Skip to content

fix(agent): run sandbox commands directly instead of via node's entrypoint - #66

Merged
radim10 merged 1 commit into
masterfrom
fix/agent-sandbox-image-entrypoint
Oct 9, 2026
Merged

radim10 merged 1 commit into
masterfrom
fix/agent-sandbox-image-entrypoint

Conversation

@radim10

@radim10 radim10 commented Oct 9, 2026

Copy link
Copy Markdown
Member

Summary

  • A misspelled agent gave a misleading Node error. The default sandbox image inherited the node base image's entrypoint, which runs any command it can't find as a Node script, so agent run -- claude. failed with Error: Cannot find module '/workspace/claude.'.
  • The image now clears the entrypoint (ENTRYPOINT []). The command is exec'd directly and a typo fails with exec claude. failed: No such file or directory (exit 127). Every command the image runs is a real executable: the agent, the network holder's sleep, and agent docker shell, which already passes its own --entrypoint bash.
  • Existing installs keep the old image until it's rebuilt with stashbase agent docker build --force.

Test plan

  • Unit test: the embedded Dockerfile clears the entrypoint
  • cargo fmt --check and full cargo test pass
  • Checked by hand on a Linux VM with the rebuilt image: the typo gives the exec error and exit 127; a normal run still starts the network holder and the agent

…point

The default sandbox image inherited the node base image's entrypoint,
which runs any command it can't find as a Node script. A misspelled
agent (`agent run -- claude.`) failed with "Error: Cannot find module
'/workspace/claude.'". The image now clears the entrypoint, so the
command is exec'd directly and a typo fails with "exec claude. failed:
No such file or directory" (exit 127). Every command the image runs is a
real executable: the agent, the network holder's `sleep`, and
`agent docker shell`, which already sets its own entrypoint.

Existing installs keep the old image until it is rebuilt with
`stashbase agent docker build --force`.
@radim10 radim10 self-assigned this Oct 9, 2026
@radim10
radim10 merged commit fd20a2b into master Oct 9, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant