Skip to content

Deduplicate retained instructions in async Guardian context - #48110

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

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

Conversation

@copyberry

@copyberry copyberry Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

Deduplicate retained instructions in async Guardian context

Why

Async Guardian requests include retained user instructions even when complete copies are already present in the same request's transcript.

What changed

  • Include retained source metadata and order labels in async user transcript entries.
  • Omit retained instructions when the current transcript contains the same complete source revision. Keep ordering guidance in every async sample and never rely on previous samples for deduplication.
  • Preserve retained originals when transcript copies are incomplete or lack source metadata.

Testing

Extend retained-instruction tests to cover async deduplication, preservation of originals without complete source proof, and budget protection. Update async scorer expectations for source order labels and deduplicated instructions.

## Why

Async Guardian requests include retained user instructions even when complete copies are already present in the same request's transcript.

## What changed

- Include retained source metadata and order labels in async user transcript entries.
- Omit retained instructions when the current transcript contains the same complete source revision. Keep ordering guidance in every async sample and never rely on previous samples for deduplication.
- Preserve retained originals when transcript copies are incomplete or lack source metadata.

## Testing

Extend retained-instruction tests to cover async deduplication, preservation of originals without complete source proof, and budget protection. Update async scorer expectations for source order labels and deduplicated instructions.

GitOrigin-RevId: 7029731b296d2342eea4ad74ffa9d3447578d83e
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/7029731b296d2342eea4ad74ffa9d3447578d83e branch from 3379cc7 to 8592869 Compare September 25, 2026 14:55
@copyberry
copyberry Bot merged commit 8592869 into main Sep 25, 2026
1 check passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/7029731b296d2342eea4ad74ffa9d3447578d83e branch September 25, 2026 14:55
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 25, 2026
@wenzo
wenzo deployed to issue-triage September 25, 2026 15:02 — with GitHub Actions Active
@wenzo
wenzo deployed to issue-triage September 25, 2026 15:02 — with GitHub Actions Active
@wenzo
wenzo deployed to issue-triage September 25, 2026 15:02 — with GitHub Actions Active

This branch was successfully deployed

1 active deployment
issue-triage — 85928697 Deployed Sep 25, 2026 by wenzo via Translate non-English issue #15105
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