Skip to content

Limit sandbox tool permission changes - #233

Open
wsr3005 wants to merge 1 commit into
yc-software:mainfrom
wsr3005:codex/sandbox-tool-chmod-fix
Open

Limit sandbox tool permission changes#233
wsr3005 wants to merge 1 commit into
yc-software:mainfrom
wsr3005:codex/sandbox-tool-chmod-fix

Conversation

@wsr3005

@wsr3005 wsr3005 commented Aug 5, 2026

Copy link
Copy Markdown

Summary

  • set executable mode on each copied sandbox tool with COPY --chmod=0755
  • stop recursively changing permissions on the base image's /usr/local/bin
  • cover the generated Dockerfile contract in unit and CLI end-to-end tests

Verification

  • Node 24 sandbox build unit and CLI end-to-end tests: 15 passed
  • CLI typecheck, ESLint, Prettier, and diff check
  • real Buildx build of a synthetic sandbox layer
  • local sandbox image build and live cold-start, warm-restart, persistence, and scratch smoke test

No rendered UI changes.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

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