fix(builtin): prevent usage with InputArtifact directories These don't work with RBE, and only work accidentally with actions.
diff --git a/nodejs/directory_file_path.bzl b/nodejs/directory_file_path.bzl index 0027fb7..846a119 100644 --- a/nodejs/directory_file_path.bzl +++ b/nodejs/directory_file_path.bzl
@@ -9,8 +9,8 @@ load("//nodejs/private/providers:directory_file_path_info.bzl", "DirectoryFilePathInfo") def _directory_file_path(ctx): - if not ctx.file.directory.is_source and not ctx.file.directory.is_directory: - fail("directory attribute must be a source directory or created with Bazel declare_directory (TreeArtifact)") + if not ctx.file.directory.is_directory: + fail("directory attribute must be created with ctx.declare_directory (TreeArtifact)") return [DirectoryFilePathInfo(path = ctx.attr.path, directory = ctx.file.directory)] directory_file_path = rule(