rlang::env_binding_are_lazy() fails when a names() method is defined

[moodymudskipper]

env <- rlang::env(x = 1)
class(env) <- "foo"
names.foo <- function(x) "y"
names(env)
#> [1] "y"
rlang::env_binding_are_lazy(env)
#> Error in env_binding_types(env, nms): no binding for "y"

^{Created on 2025-03-14 with reprex v2.1.0}

Real life example with reticulate objects :

library(reticulate)
sentence_transformers <- import("sentence_transformers")
model <- sentence_transformers$SentenceTransformer("all-MiniLM-L6-v2")
sentences <- c("This is an example sentence.", "Another sentence.")
embeddings <- model$encode(sentences)
rlang::env_binding_are_lazy(embeddings)
#> Error in env_binding_types(env, nms): no binding for "all"

^{Created on 2025-03-14 with reprex v2.1.0}

I think ls(, all.names = TRUE, sorted = FALSE) should be used instead of env_names()

Edit: same for env_binding_are_locked()

[moodymudskipper]

Luckily we have a nms arg so if we provide the correct names there it works

env <- rlang::env(x = 1)
class(env) <- "foo"
names.foo <- function(x) "y"
rlang::env_binding_are_lazy(env, ls(env, all.names = TRUE, sorted = FALSE))
#>     x 
#> FALSE

^{Created on 2025-03-14 with reprex v2.1.0}