Skip to content

Build auto-review durable lifecycle foundation#131

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

Build auto-review durable lifecycle foundation#131
shiny-code-bot merged 1 commit into
mainfrom
auto-review-durable-lifecycle

Conversation

@shiny-code-bot

Copy link
Copy Markdown
Collaborator

Refs #130

Summary

  • expand the durable auto-review store lifecycle vocabulary toward the authoritative Every Code model while preserving Codex Lab run ids, ReviewTarget, scoped store layout, and app-server/TUI contracts
  • make runs.json the canonical store record and treat output sidecars as recovery/detail cache so corrupt or premature sidecars cannot override the index
  • add tested helpers for duplicate lookup, stale-target filtering, fingerprint-scoped supersession, and lost-run reconciliation with live-run preservation

Validation

  • cargo test -p codex-auto-review
  • cargo test -p codex-core auto_review_awareness
  • cargo test -p codex-core review_persistence
  • cargo test -p codex-app-server review
  • cargo fmt --manifest-path codex-rs/Cargo.toml --all --check
  • git diff --check
  • read-only diff review agent; repaired all reported lifecycle/scope issues before opening

Notes

This is the first #130 implementation slice. It deliberately does not wire the new store helpers into runtime scheduling yet, nor does it port review.lock / snapshot.epoch. Those remain the next slices after this foundation is reviewed.

@shiny-code-bot shiny-code-bot merged commit d06c491 into main Jun 18, 2026
6 checks passed
@shiny-code-bot shiny-code-bot deleted the auto-review-durable-lifecycle branch June 18, 2026 13:37
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