Skip to content

Fix auto-review store migration awareness#129

Merged
shiny-code-bot merged 1 commit into
mainfrom
fix-auto-review-migration-awareness
Jun 18, 2026
Merged

Fix auto-review store migration awareness#129
shiny-code-bot merged 1 commit into
mainfrom
fix-auto-review-migration-awareness

Conversation

@shiny-code-bot

Copy link
Copy Markdown
Collaborator

Summary

  • preserve visibility of pre-scoped auto-review runs from CODEX_LAB_HOME/auto-review/runs/*.json after the scoped store migration
  • keep new scoped runs.json / outputs/*.json data authoritative over duplicate legacy run ids
  • avoid auto-review awareness git/diff probing unless there is loadable legacy data, scoped store data, or a live pending/running review
  • make legacy store reads best-effort so corrupt retired artifacts cannot hide valid scoped store data

Refs #126.

Validation

  • cargo fmt --manifest-path codex-rs/Cargo.toml --all --check
  • git diff --check
  • cargo test -p codex-auto-review
  • cargo test -p codex-core auto_review_awareness
  • read-only agent review passes on migration compatibility, hot-path awareness detection, and legacy/new precedence

@shiny-code-bot shiny-code-bot merged commit b7bc5c4 into main Jun 18, 2026
6 checks passed
@shiny-code-bot shiny-code-bot deleted the fix-auto-review-migration-awareness branch June 18, 2026 03:51
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