fix: base the human auto-request on this handoff, not review history #38

Merged
dan-claude-bot merged 1 commit from fix/human-handoff-guard into main 2026-07-18 20:31:51 +00:00

1 commit

Author SHA1 Message Date
dan-claude-bot
ec7096c08e fix: base the human auto-request on THIS handoff, not review history
codex's late #85/#98 round-3 finding, valid post-merge: the needs-human
auto-request fired only when the human had NEVER reviewed, so any earlier
human comment or stale approval left a fully-approved PR labeled
needs-human with nobody actually requested — a wedged handoff.
human_request_needed() now asks whether a fresh head-current human review
is missing (live request or head-current approval → nothing to ask;
anything else → request). Five new fixtures cover the wedge, the stale
approval, the satisfied handoff, and request suppression (19 total).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
2026-07-18 20:16:57 +00:00