Skip to content

feat(sandbox): support launch-configured permission profiles - #3

Merged
zfy0701 merged 1 commit into
mainfrom
codex/protected-credential-roots
Aug 2, 2026
Merged

feat(sandbox): support launch-configured permission profiles#3
zfy0701 merged 1 commit into
mainfrom
codex/protected-credential-roots

Conversation

@zfy0701

@zfy0701 zfy0701 commented Aug 2, 2026

Copy link
Copy Markdown

Summary

  • allow a trusted launcher to pass Codex app-server configuration overrides and map ACP modes to named permission profiles
  • select the mapped profile for new, resumed, and loaded threads, and update it when the ACP mode changes
  • preserve additional workspace roots and omit the legacy per-turn sandbox policy while a named profile is active
  • keep profile definitions opaque to the adapter and keep approval-reviewer selection independent

Motivation

Named Codex permission profiles can express runtime policy that the legacy ACP sandbox policy cannot represent. This adds a generic integration point for ACP hosts without embedding any operator-specific policy in codex-acp.

Validation

  • npm run typecheck
  • npm test (345 passed, 28 skipped)
  • npm run build
  • Linux app-server smoke test covering session creation and all three ACP mode transitions; the selected named profile remained active, additional workspace roots were forwarded, and Auto reviewer remained independent
  • Linux sandbox smoke test confirming that a profile-level file deny remained effective without a legacy per-turn sandbox override

Created by Codex . GPT-5.6

@zfy0701
zfy0701 force-pushed the codex/protected-credential-roots branch 2 times, most recently from e056dfa to da5af3f Compare August 2, 2026 09:35
@zfy0701
zfy0701 force-pushed the codex/protected-credential-roots branch from da5af3f to c84e801 Compare August 2, 2026 10:06
@zfy0701
zfy0701 force-pushed the codex/protected-credential-roots branch from c84e801 to 9714397 Compare August 2, 2026 10:20
@zfy0701 zfy0701 changed the title fix(sandbox): protect credentials from model tools feat(sandbox): support launch-configured permission profiles Aug 2, 2026
@zfy0701
zfy0701 merged commit 2e60833 into main Aug 2, 2026
2 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