Skip to content

Improve Guardian report diagnostics - #42033

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

Improve Guardian report diagnostics#42033
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/ae7916c27a3c279e899cd5a6ead8a4b7e0cc26a1

Conversation

@copyberry

@copyberry copyberry Bot commented Sep 1, 2026

Copy link
Copy Markdown

Improve Guardian report diagnostics

Why

Reports from a large thread tree can exceed the rollout attachment limit, making
the threads involved in a failed Guardian review harder to diagnose.

What changed

  • Prioritize descendants with retained failed reviews when selecting bounded
    report rollouts, while always preserving the reported thread.
  • Include each selected thread's available Guardian trunk rollout alongside its
    conversation rollout.
  • Add a thread index attachment with selected filenames, bounded omission details,
    retained failure threads, and the process-wide discarded-record count.
  • Link failed-review records to their reviewed turn and target item when available.

Testing

  • Cover failed-review prioritization, bounded omission metadata, discarded-record
    accounting, and turn/item linkage in Guardian report records.

## Why

Reports from a large thread tree can exceed the rollout attachment limit, making
the threads involved in a failed Guardian review harder to diagnose.

## What changed

- Prioritize descendants with retained failed reviews when selecting bounded
  report rollouts, while always preserving the reported thread.
- Include each selected thread's available Guardian trunk rollout alongside its
  conversation rollout.
- Add a thread index attachment with selected filenames, bounded omission details,
  retained failure threads, and the process-wide discarded-record count.
- Link failed-review records to their reviewed turn and target item when available.

## Testing

- Cover failed-review prioritization, bounded omission metadata, discarded-record
  accounting, and turn/item linkage in Guardian report records.

GitOrigin-RevId: ae7916c27a3c279e899cd5a6ead8a4b7e0cc26a1
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/ae7916c27a3c279e899cd5a6ead8a4b7e0cc26a1 branch from 92b95c1 to 2e5ee41 Compare September 1, 2026 11:15
@copyberry
copyberry Bot merged commit 2e5ee41 into main Sep 1, 2026
26 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/ae7916c27a3c279e899cd5a6ead8a4b7e0cc26a1 branch September 1, 2026 11:16
@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.

7 participants