lsm / neokai / 24960922202

82%

push

github

fix(daemon): treat tasks in review/approved as at-rest in recoverSingleRun (#1645)

A canonical task in `review` is paused awaiting human approval (the end-node
agent finished and all sibling executions correctly went `idle`); same applies
to `approved` while a post-approval executor is in flight. Neither is a
stalled run, but `recoverSingleRun` was only checking `done`/`cancelled`/
`reportedStatus !== null` and forcibly transitioning everything else to
`blocked` on daemon restart — silently flipping perfectly healthy
review-pending tasks to `blocked` with reason `execution_failed`.

Add `review` and `approved` to the at-rest set, document the rationale, and
cover with three regression tests (plain `review`, `review` + `task_completion`
checkpoint, and `approved`).

Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>

9244 of 12413 branches covered (74.47%)

Branch coverage included in aggregate %.

2 of 2 new or added lines in 1 file covered. (100.0%)

66433 of 78804 relevant lines covered (84.3%)

299.03 hits per line