feat: Crash recovery salvage: persist verified candidate + resume (not rebuild) + orphan sweep .gN fix (#91)#96
Conversation
…t rebuild) + orphan sweep .gN fix (#91)
There was a problem hiding this comment.
QA panel review — PASS
code-review-structural · head 3e09c8af99e9 · formal
[review-synthesizer completed: workflow code-review-structural:report]
Prose Brief
Overall risk: Low — all four LLM finders returned zero findings, and the verifier confirmed the empty panel. The diff is a focused feature addition (sweep-based iteration in loop.py with a new parent_feature_id on Worktree) touching few files.
Fix-first item: None — no defects surfaced. However, the verifier flagged a coverage gap worth a human sanity check: worktree.parent_feature_id is referenced in loop.py's _sweep method but was not found in worktree.py on the default branch at verification time. This may live in a stacked/companion PR, but if it doesn't exist anywhere, the sweep feature silently breaks at runtime.
Panel disagreement: None — all four finders independently returned empty arrays.
Verification changes: The verifier confirmed the empty array; no refuted or uncertain items to process.
Gaps: (1) The structural pass (protopatch) failed with a context-length error — we have no tool-sourced findings for this round. (2) The parent_feature_id existence question noted above.
[]
Summary
Crash recovery salvage: persist verified candidate + resume (not rebuild) + orphan sweep .gN fix (#91) (
bd-13w)See the diff for details.
Fixes #91