Skip to content

feat(sandbox): add configurable Unix-local environment isolation - #4640

Merged
seratch merged 1 commit into
mainfrom
fix/unix-local-environment-isolation
Aug 25, 2026
Merged

feat(sandbox): add configurable Unix-local environment isolation#4640
seratch merged 1 commit into
mainfrom
fix/unix-local-environment-isolation

Conversation

@seratch

@seratch seratch commented Aug 24, 2026

Copy link
Copy Markdown
Member

This pull request supersedes #4617 and adds opt-in, configurable environment isolation for Unix-local sandboxes without changing existing default behavior.

UnixLocalSandboxClient continues to inherit the complete host environment by default. Applications can set inherit_host_environment=False to use the SDK's safe built-in allowlist, or additionally provide host_environment_allowlist to replace it with an exact application-controlled set of variable names.

Manifest.environment continues to override the selected host baseline, and HOME remains pinned to the workspace. The inheritance policy is trusted runtime-only configuration applied to both created and resumed sessions; it is never serialized into session state.

Co-authored-by: simpleqt <89645338+simpleqt@users.noreply.github.com>
@seratch seratch added this to the 0.22.x milestone Aug 24, 2026
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Aug 24, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-08-24T23:56:04.904363Z e552eee PR opened
🔒 Security Review Completed 2026-08-24T23:56:45.446293Z e552eee PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@seratch seratch changed the title feat: add configurable Unix-local environment isolation feat(sandbox): add configurable Unix-local environment isolation Aug 25, 2026
@seratch
seratch enabled auto-merge (squash) August 25, 2026 00:05
@seratch
seratch merged commit 91f8c49 into main Aug 25, 2026
33 of 34 checks passed
@seratch
seratch deleted the fix/unix-local-environment-isolation branch August 25, 2026 00:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant