Skip to content

Consolidate Guardian tests at shared policy and context boundaries - #46064

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

copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/b39ef21475e2f5d99d758f2d27615449b252dc65

Conversation

@copyberry

@copyberry copyberry Bot commented Sep 17, 2026 •

Copy link
Copy Markdown

Consolidate Guardian tests at shared policy and context boundaries

What changed

  • Move approval routing tests into guardian-reviewer, covering both reviewer choices across UnlessTrusted, OnRequest, Granular, and Never without host session setup.
  • Combine reviewer configuration checks into one test for isolation from parent customizations.
  • Cover filtering of ordinary developer messages while preserving manual approvals in guardian-context, removing the duplicate async scorer test.

…46064)

## What changed

- Move approval routing tests into `guardian-reviewer`, covering both reviewer choices across `UnlessTrusted`, `OnRequest`, `Granular`, and `Never` without host session setup.
- Combine reviewer configuration checks into one test for isolation from parent customizations.
- Cover filtering of ordinary developer messages while preserving manual approvals in `guardian-context`, removing the duplicate async scorer test.

GitOrigin-RevId: b39ef21475e2f5d99d758f2d27615449b252dc65
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/b39ef21475e2f5d99d758f2d27615449b252dc65 branch from 09e0991 to 08663cc Compare September 17, 2026 00:39
@copyberry
copyberry Bot merged commit 08663cc into main Sep 17, 2026
1 check passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/b39ef21475e2f5d99d758f2d27615449b252dc65 branch September 17, 2026 00:39
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 17, 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.

1 participant