Skip to content

Port compact auto-review store#137

Merged
shiny-code-bot merged 1 commit into
mainfrom
auto-review-compact-store
Jun 18, 2026
Merged

Port compact auto-review store#137
shiny-code-bot merged 1 commit into
mainfrom
auto-review-compact-store

Conversation

@shiny-code-bot

Copy link
Copy Markdown
Collaborator

Summary

  • convert auto-review persistence to a canonical compact runs index plus ReviewOutputEvent sidecars
  • remove legacy/unscoped/sidecar-recovery compatibility paths and dead compact-store schema fields
  • preserve sidecar-backed detail reads, including omitted f21+ findings, and scope orphan recovery to background auto-review runs
  • move background snapshot epoch stamping into the successful start path and keep spawned review persistence aligned

Validation

  • cargo fmt --all -- --check
  • git diff --check
  • just test -p codex-auto-review
  • just test -p codex-app-server review
  • just test -p codex-core auto_review_awareness
  • final read-only review agent: no issues found

Part of #126 and corrects the PR #63 audit slice toward the no-legacy/no-dead-code canonical cutover.

@shiny-code-bot shiny-code-bot merged commit a3421b7 into main Jun 18, 2026
6 checks passed
@shiny-code-bot shiny-code-bot deleted the auto-review-compact-store branch June 18, 2026 23:46
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