In one unattended run, the worker on handoff #1835 broke two of the unattended-handoff skill's own worker rules in the same round.
What happened
- It waited in the background. Worker step 4, "Wait in the foreground", says each wait is one bounded command run in the worker's own turn, since a subagent receives no completion notification. The worker instead handed its local strict review pass to a background subagent and handed back before that pass reported.
- It returned
STOP for a per-issue condition. The skill reserves STOP for "a state of the repository or the session rather than of one issue". An unfinished review pass on one branch is a state of one issue, so the right outcomes were to wait for the pass and continue, or to park per "Parking". STOP ended the whole run after three rounds with backlog still pickable.
- The handoff was left neither closed nor parked. It carries no
blocked label and no decision issue, so a later picker reads it as a lane whose worker died, and the maintainer's attended session gets no parked signal for it.
The pass later returned clean, the worker woke and commented the state on #1835, then stopped again with the fix still unpushed.
Proposed direction
- Make worker step 4 name the local strict review's subagent explicitly: its reviewer is dispatched and waited on in the foreground, never backgrounded.
- State in "Return Lines" that a worker which cannot finish one issue parks it rather than returning
STOP, and list what does qualify for STOP from a worker.
Both are rule text, so the wording is the maintainer's to settle in review.
In one unattended run, the worker on handoff #1835 broke two of the
unattended-handoffskill's own worker rules in the same round.What happened
STOPfor a per-issue condition. The skill reservesSTOPfor "a state of the repository or the session rather than of one issue". An unfinished review pass on one branch is a state of one issue, so the right outcomes were to wait for the pass and continue, or to park per "Parking".STOPended the whole run after three rounds with backlog still pickable.blockedlabel and nodecisionissue, so a later picker reads it as a lane whose worker died, and the maintainer's attended session gets no parked signal for it.The pass later returned clean, the worker woke and commented the state on #1835, then stopped again with the fix still unpushed.
Proposed direction
STOP, and list what does qualify forSTOPfrom a worker.Both are rule text, so the wording is the maintainer's to settle in review.