Skip to content

Preserve encrypted agent messages in Guardian reviews - #49038

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

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

Conversation

@copyberry

@copyberry copyberry Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

Preserve encrypted agent messages in Guardian reviews

Why

Guardian transcripts omitted encrypted agent messages, leaving reviews without evidence such as encrypted parent replies.

What changed

  • Carry encrypted agent_message items through synchronous reviews and asynchronous scoring, preserving their order, author, recipient, and ciphertext.
  • Account for native messages in context budgets and omit them whole with an explicit notice when limits require it.
  • Support mixed text and native messages during review admission while retaining user-content annotations.

Testing

Add coverage for transcript ordering and budget eviction in both reviewer profiles, plus a mocked scenario verifying that an encrypted parent reply appears exactly once in each of two incremental Guardian reviews.

## Why

Guardian transcripts omitted encrypted agent messages, leaving reviews without evidence such as encrypted parent replies.

## What changed

- Carry encrypted `agent_message` items through synchronous reviews and asynchronous scoring, preserving their order, author, recipient, and ciphertext.
- Account for native messages in context budgets and omit them whole with an explicit notice when limits require it.
- Support mixed text and native messages during review admission while retaining user-content annotations.

## Testing

Add coverage for transcript ordering and budget eviction in both reviewer profiles, plus a mocked scenario verifying that an encrypted parent reply appears exactly once in each of two incremental Guardian reviews.

GitOrigin-RevId: abc2f1ce5accd83beb06f799ef9ed9d7fa2cd90a
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/abc2f1ce5accd83beb06f799ef9ed9d7fa2cd90a branch from 11aa1d8 to 368e5ea Compare September 28, 2026 17:40
@copyberry
copyberry Bot merged commit 368e5ea into main Sep 28, 2026
1 check passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/abc2f1ce5accd83beb06f799ef9ed9d7fa2cd90a branch September 28, 2026 17:40
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 28, 2026

This branch was successfully deployed

1 active deployment
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.

3 participants