Skip to content

Read turn permissions from the current configuration - #36930

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/d67c5d2bcbe6dc76f15b56defb485b155fd1f138
Aug 4, 2026
Merged

Read turn permissions from the current configuration#36930
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/d67c5d2bcbe6dc76f15b56defb485b155fd1f138

Conversation

@copyberry

@copyberry copyberry Bot commented Aug 4, 2026

Copy link
Copy Markdown

Read turn permissions from the current configuration

What changed

  • Remove the cached permission profile from TurnContext and derive the
    effective profile, filesystem policy, network policy, and legacy sandbox
    policy from config.permissions.
  • Update sandbox consumers and tests to use the current turn configuration.
  • Verify that role-based agent spawning reapplies runtime permissions and that
    cold-resumed agents retain their disabled permission profile.

## What changed

- Remove the cached permission profile from `TurnContext` and derive the
  effective profile, filesystem policy, network policy, and legacy sandbox
  policy from `config.permissions`.
- Update sandbox consumers and tests to use the current turn configuration.
- Verify that role-based agent spawning reapplies runtime permissions and that
  cold-resumed agents retain their disabled permission profile.

GitOrigin-RevId: d67c5d2bcbe6dc76f15b56defb485b155fd1f138
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/d67c5d2bcbe6dc76f15b56defb485b155fd1f138 branch from 98a5ef4 to 90314a9 Compare August 4, 2026 17:14
@copyberry
copyberry Bot merged commit 90314a9 into main Aug 4, 2026
22 of 26 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/d67c5d2bcbe6dc76f15b56defb485b155fd1f138 branch August 4, 2026 17:15
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 4, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants