Skip to content

Centralize Guardian context composition - #42085

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

Centralize Guardian context composition#42085
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/9268343631fd6417bb05a3c0abce883559a37685

Conversation

@copyberry

@copyberry copyberry Bot commented Sep 1, 2026

Copy link
Copy Markdown

Centralize Guardian context composition

What changed

  • Add SectionRegistry::compose to group collected authorization fragments and transcript entries while preserving fragment boundaries and registration order.
  • Use the shared composed context in the core Guardian prompt and Guardian V2 synchronous and asynchronous reviewers.
  • Keep transcript rendering, retention, and delta handling with each caller while removing duplicated section grouping and authorization rendering.

Testing

  • Cover composition for synchronous and asynchronous targets, including root conversation role handling, trusted user answers, transcript entries, empty context, and contributor failures.
  • Verify the synchronous reviewer preserves authorization fragment boundaries in prompt items.

## What changed

- Add `SectionRegistry::compose` to group collected authorization fragments and transcript entries while preserving fragment boundaries and registration order.
- Use the shared composed context in the core Guardian prompt and Guardian V2 synchronous and asynchronous reviewers.
- Keep transcript rendering, retention, and delta handling with each caller while removing duplicated section grouping and authorization rendering.

## Testing

- Cover composition for synchronous and asynchronous targets, including root conversation role handling, trusted user answers, transcript entries, empty context, and contributor failures.
- Verify the synchronous reviewer preserves authorization fragment boundaries in prompt items.

GitOrigin-RevId: 9268343631fd6417bb05a3c0abce883559a37685
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/9268343631fd6417bb05a3c0abce883559a37685 branch from cfd953b to 6127478 Compare September 1, 2026 15:55
@copyberry
copyberry Bot merged commit 6127478 into main Sep 1, 2026
26 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/9268343631fd6417bb05a3c0abce883559a37685 branch September 1, 2026 15:56
@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.

1 participant