Skip to content

Enforce auto review dedupe freshness policy#380

Merged
cbusillo merged 1 commit into
mainfrom
fix/auto-review-dedupe-policy
Jun 5, 2026
Merged

Enforce auto review dedupe freshness policy#380
cbusillo merged 1 commit into
mainfrom
fix/auto-review-dedupe-policy

Conversation

@cbusillo
Copy link
Copy Markdown
Owner

@cbusillo cbusillo commented Jun 5, 2026

Summary

  • make Auto Review duplicate lookup target-aware so stale detached or older snapshot runs are not reused for the active checkout
  • keep diff fingerprints based on effective diff content, while preserving snapshot/head/epoch evidence separately for applicability checks
  • keep adopted in-flight reviews visible by retargeting the TUI to the adopted run and replaying terminal statuses safely
  • promote healthy long-running reviews to LongRunning freshness in TUI and exec status writers

Validation

  • cargo test --manifest-path code-rs/Cargo.toml -p code-core duplicate_lookup
  • cargo test --manifest-path code-rs/Cargo.toml -p code-core auto_review_freshness_classifier
  • cargo test --manifest-path code-rs/Cargo.toml -p code-tui auto_review_diff_fingerprint
  • cargo test --manifest-path code-rs/Cargo.toml -p code-tui adopted_background_review
  • ./build-fast.sh

Fixes #329

@cbusillo cbusillo merged commit c848e1c into main Jun 5, 2026
1 check passed
@cbusillo cbusillo deleted the fix/auto-review-dedupe-policy branch June 5, 2026 13:45
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.

Enforce Auto Review dedupe freshness supersede and cancellation policy

1 participant