Skip to content

Commit 846925e

Browse files
avrabeclaude
andcommitted
fix: remove 'tools' parameter from file_ops actions to enable runfiles
When a binary is passed as 'executable' in ctx.actions.run(), Bazel automatically provides runfiles support. However, also passing it in the 'tools' parameter prevents runfiles from being properly initialized in the sandbox. Remove the 'tools = [file_ops_tool]' parameter from both: - prepare_workspace_action() / PrepareWorkspaceHermetic action - setup_js_workspace_action() / SetupJSWorkspace action This allows the file_ops binary to properly initialize its runfiles and locate its data dependencies (wasmtime binary, WASM components, etc.) Fixes: "Failed to initialize runfiles: runfiles: no runfiles found" error 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
1 parent 527685b commit 846925e

File tree

1 file changed

+0
-2
lines changed

1 file changed

+0
-2
lines changed

tools/bazel_helpers/file_ops_actions.bzl

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -231,7 +231,6 @@ def prepare_workspace_action(ctx, config):
231231
config.get("workspace_type", "generic"),
232232
ctx.label,
233233
),
234-
tools = [file_ops_tool],
235234
)
236235

237236
return workspace_dir
@@ -458,7 +457,6 @@ def setup_js_workspace_action(ctx, sources, package_json = None, npm_deps = None
458457
outputs = [workspace_dir],
459458
mnemonic = "SetupJSWorkspace",
460459
progress_message = "Setting up JavaScript workspace for %s" % ctx.label,
461-
tools = [file_ops_tool],
462460
)
463461

464462
return workspace_dir

0 commit comments

Comments
 (0)