Skip to content

feat: Refactor bash.rs and Move sandbox runtime dirs out of workspace#3122

Open
cv70 wants to merge 1 commit into
ultraworkers:mainfrom
cv70:feat/sandbox-path-remap
Open

feat: Refactor bash.rs and Move sandbox runtime dirs out of workspace#3122
cv70 wants to merge 1 commit into
ultraworkers:mainfrom
cv70:feat/sandbox-path-remap

Conversation

@cv70
Copy link
Copy Markdown

@cv70 cv70 commented May 26, 2026

Summary

  • Move bash sandbox HOME and TMPDIR paths out of the active workspace.
  • Add shared sandbox runtime path helpers that place per-workspace sandbox state under the OS temp directory.
  • Remove the tracked rust/.sandbox-home/.rustup/settings.toml artifact from the repository tree.
  • Add regression coverage to ensure sandbox directories are not recreated under the current working directory.

Anti-slop triage

Verification

  • Targeted tests/docs checks ran, or the gap is explicitly recorded.
  • git diff --check passes.
  • No live secrets, tokens, private logs, or unrelated generated churn are included.

Resolution gate

  • If this PR resolves an issue, the issue number and fix evidence are linked.
  • If this PR should not merge, the rejection/defer rationale is evidence-backed and does not rely on vibes.
  • I did not merge/close remote PRs or issues from an automation lane without owner approval.

Issue link: #3121. Fix evidence is the new runtime path helper plus the bash regression test that asserts .sandbox-home and .sandbox-tmp are not created inside the workspace.

@cv70 cv70 changed the title feat: Move sandbox runtime dirs out of workspace feat: Refactor bash.rs and Move sandbox runtime dirs out of workspace May 26, 2026
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