Skip to content

fix(tui): identify auto review lifecycle runs#308

Merged
cbusillo merged 1 commit into
mainfrom
feat/auto-review-run-id
Jun 1, 2026
Merged

fix(tui): identify auto review lifecycle runs#308
cbusillo merged 1 commit into
mainfrom
feat/auto-review-run-id

Conversation

@cbusillo
Copy link
Copy Markdown
Owner

@cbusillo cbusillo commented Jun 1, 2026

Summary

  • add a generated Auto Review lifecycle run id to background start/finish events
  • track the active background review run id in the TUI and ignore start/finish events from older runs
  • cover wrong-run start and finish events so stale lifecycle callbacks cannot overwrite or clear the current review

Validation

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

Continues #76 by making Auto Review lifecycle callbacks identify the active launched run, not just the reviewed snapshot/session metadata.

@cbusillo cbusillo merged commit faedc4a into main Jun 1, 2026
1 check passed
@cbusillo cbusillo deleted the feat/auto-review-run-id branch June 1, 2026 20: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