Skip to content

fix: reindex recovered PR sessions - #561

Open
Guflly wants to merge 2 commits into
agavra:mainfrom
Guflly:fix/reindex-recovered-pr-session
Open

fix: reindex recovered PR sessions#561
Guflly wants to merge 2 commits into
agavra:mainfrom
Guflly:fix/reindex-recovered-pr-session

Conversation

@Guflly

@Guflly Guflly commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Re-index PR sessions recovered from their deterministic path so the review list and review add see them immediately. The repair only updates the manifest and leaves the session JSON unchanged.

Adds regression coverage for missing and stale manifest entries.

Tests:

  • cargo test should_reindex_recovered_pr_session
  • cargo fmt --all -- --check

Closes #449

Comment thread src/app/tests/target_selector_tests.rs Outdated
Comment thread src/app/tests/target_selector_tests.rs Outdated
Comment thread src/persistence/storage.rs
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.

Re-index PR sessions recovered from deterministic fallback path

2 participants