Skip to content

Wire durable auto-review runtime dedupe#132

Merged
shiny-code-bot merged 1 commit into
mainfrom
auto-review-runtime-dedupe
Jun 18, 2026
Merged

Wire durable auto-review runtime dedupe#132
shiny-code-bot merged 1 commit into
mainfrom
auto-review-runtime-dedupe

Conversation

@shiny-code-bot

Copy link
Copy Markdown
Collaborator

Summary

  • Wire the durable auto-review store into background runtime scheduling so equivalent completed background reviews are reused instead of spending another review request.
  • Add target-filtered duplicate lookup/supersession helpers so unowned in-flight records do not shadow reusable completed runs, and clean terminal supersession does not cross branch/head boundaries.
  • Persist duplicate candidates as skipped records with duplicate metadata while preserving the existing app-server/TUI background status contract.

Refs #130

Validation

  • cargo test -p codex-auto-review
  • cargo test -p codex-core save_skipped_duplicate_records_duplicate_metadata
  • cargo test -p codex-core automatic_background_review_reuses_completed_durable_duplicate --test all -- --nocapture
  • cargo test -p codex-core automatic_background_review_suppresses_unchanged_dirty_diff --test all -- --nocapture
  • cargo fmt --manifest-path codex-rs/Cargo.toml --all --check (passes with existing stable-rustfmt warnings)
  • git diff --check
  • Read-only GPT-5.5 patch review after repairs: no issues found.
  • Built-in mini auto-review failed before findings on this patch (fb268d17-bafa-415c-b232-208b85991b2d), so it produced no actionable findings.

@shiny-code-bot shiny-code-bot merged commit dc66809 into main Jun 18, 2026
6 checks passed
@shiny-code-bot shiny-code-bot deleted the auto-review-runtime-dedupe branch June 18, 2026 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant