Skip to content

test(tui): cover auto review lifecycle metadata gap#307

Merged
cbusillo merged 1 commit into
mainfrom
feat/auto-review-lifecycle-harness
Jun 1, 2026
Merged

test(tui): cover auto review lifecycle metadata gap#307
cbusillo merged 1 commit into
mainfrom
feat/auto-review-lifecycle-harness

Conversation

@cbusillo
Copy link
Copy Markdown
Owner

@cbusillo cbusillo commented Jun 1, 2026

Summary

  • extract Auto Review active-run matching into an AutoReviewRunIdentity helper
  • treat missing session/snapshot metadata as non-authoritative without permanently marking the agent processed
  • add a lifecycle regression that ignores a pre-start metadata-free terminal status, then accepts the later matching status after start metadata arrives

Validation

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

Closes part of #76 by adding deterministic lifecycle coverage for stale/slow Auto Review event ordering.

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