Skip to content

Move sandbox policy transforms into codex-sandboxing#15599

Merged
pakrym-oai merged 12 commits intomainfrom
pakrym/sandboxing-crate-step-3-sandbox-transforms
Mar 24, 2026
Merged

Move sandbox policy transforms into codex-sandboxing#15599
pakrym-oai merged 12 commits intomainfrom
pakrym/sandboxing-crate-step-3-sandbox-transforms

Conversation

@pakrym-oai
Copy link
Copy Markdown
Collaborator

Summary

  • move the pure sandbox policy transform helpers from codex-core into codex-sandboxing
  • move the corresponding unit tests with the extracted implementation
  • update core and app-server callers to import the moved APIs directly, without re-exports or proxy methods

Testing

  • cargo test -p codex-sandboxing
  • cargo test -p codex-core sandboxing
  • cargo test -p codex-app-server --lib
  • just fix -p codex-sandboxing
  • just fix -p codex-core
  • just fix -p codex-app-server
  • just fmt
  • just argument-comment-lint
@pakrym-oai pakrym-oai marked this pull request as ready for review March 24, 2026 03:50
Base automatically changed from pakrym/sandboxing-crate-step-2-macos-permissions to main March 24, 2026 04:26
pakrym-oai added a commit that referenced this pull request Mar 24, 2026
@pakrym-oai pakrym-oai force-pushed the pakrym/sandboxing-crate-step-3-sandbox-transforms branch from 2778e5c to c6a34b2 Compare March 24, 2026 04:54
@pakrym-oai pakrym-oai merged commit 9deb8ce into main Mar 24, 2026
65 of 75 checks passed
@pakrym-oai pakrym-oai deleted the pakrym/sandboxing-crate-step-3-sandbox-transforms branch March 24, 2026 05:22
@github-actions github-actions bot locked and limited conversation to collaborators Mar 24, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

2 participants