fix: routing bug — getNextTask ignores board_truth, parked_pass, blocked_external

[itskai-dev]

Summary

Two-part routing fix:

Part 1 — sweeper bounce path (PR #2769, commit 4d5a603)

was requeuing PR-merged tasks to todo when missing artifacts. Two pr_merged guards added:

• Artifact-rejection path: PR merged + no qa_bundle → close to done
• Drift-repair dupeErr path: PR merged + approved → close to done

Part 2 — getNextTask selector (this PR, commit 8fbcd95)

getNextTask() was surfacing closed/stale tasks because it had no awareness of:

• review_state=parked_pass/fail — QA-complete tasks
• board_truth metadata — explicit per-task lane routing
• blocked_external flags
• reopen=true — tasks being actively reworked

Fix

Added four filters to getNextTask():

// Skip parked tasks — review_state=parked_pass/fail means QA is done
if (reviewState === 'parked_pass' || reviewState === 'parked_fail') return false

// Skip explicitly reopened tasks
if (meta?.reopen === true) return false

// Skip tasks with explicit board truth routing
if (meta?.board_truth && typeof meta.board_truth === 'string') return false

// Skip blocked external tasks
if (meta?.blocked_external === true) return false

Why Two Fixes

PR #2769 closed the sweeper bounce path. This PR closes the selector path. Together they stop:

1. Tasks incorrectly requeued to todo by sweeper (artifact rejection bounce)
2. Closed/stale tasks being surfaced by /tasks/next despite board truth

Reported by link: M cut bounced 13+ times. Root cause was both paths combined.

Testing

• TypeScript clean (npx tsc --noEmit)
• task-1777172195778-9zofmfuya (3B post-merge, parked_pass) — correctly filtered
• task-1777356670701-lo4e5fiim (board_truth → link, blocked_external) — correctly filtered
• Need fresh /tasks/next pull proof on rn-b1964d01-fhdmps