Skip to content

Allow updating the approval reviewer for active turns - #42121

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/e1216b09c6a7972c2c9eaf4955d8d57d60604ca1
Sep 1, 2026
Merged

Allow updating the approval reviewer for active turns#42121
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/e1216b09c6a7972c2c9eaf4955d8d57d60604ca1

Conversation

@copyberry

@copyberry copyberry Bot commented Sep 1, 2026

Copy link
Copy Markdown

Allow updating the approval reviewer for active turns

What changed

  • Add approvalsReviewer to the experimental turn/settings/update API.
  • Apply reviewer changes to subsequently captured steps and new background approval requests while preserving pending approvals and future-thread defaults.
  • Allow reviewer-only updates without step_model_switching, while continuing to enforce managed reviewer restrictions and model-required auto review.
  • Use an explicit live reviewer update for MCP approvals while retaining refreshed thread defaults for clients that have not set one.

Testing

  • Cover switching between user and automatic review during a live turn, future-turn isolation, and managed-authority rejection.

## What changed

- Add `approvalsReviewer` to the experimental `turn/settings/update` API.
- Apply reviewer changes to subsequently captured steps and new background approval requests while preserving pending approvals and future-thread defaults.
- Allow reviewer-only updates without `step_model_switching`, while continuing to enforce managed reviewer restrictions and model-required auto review.
- Use an explicit live reviewer update for MCP approvals while retaining refreshed thread defaults for clients that have not set one.

## Testing

- Cover switching between user and automatic review during a live turn, future-turn isolation, and managed-authority rejection.

GitOrigin-RevId: e1216b09c6a7972c2c9eaf4955d8d57d60604ca1
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/e1216b09c6a7972c2c9eaf4955d8d57d60604ca1 branch from 260befa to 9112564 Compare September 1, 2026 19:21
@copyberry
copyberry Bot merged commit 9112564 into main Sep 1, 2026
23 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/e1216b09c6a7972c2c9eaf4955d8d57d60604ca1 branch September 1, 2026 19:22
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 1, 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.

2 participants