Skip to content

Archive superseded auto-review results#310

Merged
cbusillo merged 1 commit into
mainfrom
feat/auto-review-currentness-gate
Jun 1, 2026
Merged

Archive superseded auto-review results#310
cbusillo merged 1 commit into
mainfrom
feat/auto-review-currentness-gate

Conversation

@cbusillo
Copy link
Copy Markdown
Owner

@cbusillo cbusillo commented Jun 1, 2026

Summary

  • classify Auto Review completions as current, superseded, or ignored
  • archive same-session superseded findings as historical Auto Review notices without post-turn developer injection
  • keep cross-session and missing-metadata completions fail-closed, and preserve active review state/locks

Validation

  • cargo test -p code-tui --features test-helpers auto_review
  • ./build-fast.sh

Review

  • read-only agent review found two race/lock concerns; both were fixed and covered by regression tests before this PR

Refs #76

@cbusillo cbusillo merged commit 579ef5c into main Jun 1, 2026
1 check passed
@cbusillo cbusillo deleted the feat/auto-review-currentness-gate branch June 1, 2026 21:36
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