Skip to content

Wire Auto Review lifecycle state into durable store#333

Merged
shiny-code-bot merged 1 commit into
mainfrom
feat/auto-review-lifecycle-store-wiring
Jun 2, 2026
Merged

Wire Auto Review lifecycle state into durable store#333
shiny-code-bot merged 1 commit into
mainfrom
feat/auto-review-lifecycle-store-wiring

Conversation

@shiny-code-bot
Copy link
Copy Markdown
Collaborator

Summary

  • Persist TUI Auto Review launch, start, active/inactive status, skip, stale-loss, and terminal completion updates into the durable AutoReviewRun store.
  • Persist exec-observed Auto Review agent lifecycle updates into the same store, reusing existing rows by agent id after restart and enriching rows with heuristic summaries plus structured output sidecars when available.
  • Bound stored finding digests for future low-token ledger use while keeping full structured review output in sidecars.
  • Reconcile orphaned in-flight runs on TUI/exec startup so restarts do not leave stale rows looking active.

Validation

  • cargo test -p code-core review_store
  • ./build-fast.sh
  • git diff --check

Agent Review Notes

  • Agents reviewed the slice for lifecycle correctness and long-term Auto Review goals.
  • Addressed review feedback by adding startup reconciliation, stale-run loss marking, exec row reuse by agent id, heuristic exec finding summaries, and bounded finding digests.

Closes #326

@shiny-code-bot shiny-code-bot merged commit 960608a into main Jun 2, 2026
1 check passed
@shiny-code-bot shiny-code-bot deleted the feat/auto-review-lifecycle-store-wiring branch June 2, 2026 13:29
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.

Wire TUI and exec Auto Review lifecycle events into durable state

2 participants