Skip to content

Preserve Guardian review evidence across compaction - #41879

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/a18b1176f14a4d006c263aa82d4e593afee66bc3
Aug 31, 2026
Merged

Preserve Guardian review evidence across compaction#41879
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/a18b1176f14a4d006c263aa82d4e593afee66bc3

Conversation

@copyberry

@copyberry copyberry Bot commented Aug 31, 2026

Copy link
Copy Markdown

Preserve Guardian review evidence across compaction

Why

Guardian reviews need the original conversation evidence even after the parent context has been compacted, while rollback must still remove evidence from reverted turns.

What changed

  • Build synchronous and asynchronous Guardian transcripts from the retained review-history snapshot.
  • Use the shared Guardian transcript collector for consistent entry formatting, tool attribution, and per-entry truncation while preserving the existing transcript retention budgets.

Testing

  • Add app-server coverage for retaining user and tool evidence after compaction and discarding it after rollback.
  • Extend transcript tests for standalone tool outputs, shell calls, attribution, truncation, and retention limits.

## Why

Guardian reviews need the original conversation evidence even after the parent context has been compacted, while rollback must still remove evidence from reverted turns.

## What changed

- Build synchronous and asynchronous Guardian transcripts from the retained review-history snapshot.
- Use the shared Guardian transcript collector for consistent entry formatting, tool attribution, and per-entry truncation while preserving the existing transcript retention budgets.

## Testing

- Add app-server coverage for retaining user and tool evidence after compaction and discarding it after rollback.
- Extend transcript tests for standalone tool outputs, shell calls, attribution, truncation, and retention limits.

GitOrigin-RevId: a18b1176f14a4d006c263aa82d4e593afee66bc3
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/a18b1176f14a4d006c263aa82d4e593afee66bc3 branch from dd3f3a8 to 9f97cb7 Compare August 31, 2026 17:11
@copyberry
copyberry Bot merged commit 9f97cb7 into main Aug 31, 2026
27 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/a18b1176f14a4d006c263aa82d4e593afee66bc3 branch August 31, 2026 17:12
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 31, 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.

5 participants