fix: unblock_task returns to todo and clears stale claims (#25)

unblock_task checked started_at to decide status, always returning to
in_progress for tasks that had been worked on. It also preserved stale
claim fields, which suppressed the task.next trigger's claim guard.

Now always returns to todo and clears claim_owner/lease so the task.next
event fires and workers can pick up the task.

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

Author: danielkov

.boop/changelogs/patch-01kj3r2xz0yw7dtzzzz7q0qsne.md

@@ -0,0 +1,10 @@
+### Fix: unblock returns tasks to \`todo\` and clears stale claims
+
+Previously, \`unblock_task\` checked \`started_at\` to decide whether to return a task to \`in_progress\` or \`todo\`. Since agents set \`started_at\` when they begin work, unblocking always returned tasks to \`in_progress\` — even when the block was due to external factors (e.g., missing files from concurrent projects) and the task needed to re-enter the scheduling queue.
+
+Additionally, unblock didn't clear \`claim_owner\` or \`claim_lease_expires_at\`. This meant the \`task.next\` SQLite trigger's claim guard would suppress the event even after manually cycling the task through \`draft → todo\`. The net effect was that unblocked tasks became invisible to the worker daemon — no \`task.next\` event fired, so no worker would pick them up.
+
+**What changed:**
+- \`unblock_task\` now always transitions to \`todo\`, regardless of \`started_at\`
+- Stale claim fields (\`claim_owner\`, \`claim_claimed_at\`, \`claim_lease_expires_at\`) are cleared on unblock
+- The \`task.next\` trigger now fires correctly when tasks are unblocked

src/services/task_service.rs

@@ -271,13 +271,12 @@ pub async fn block_task(pool: &SqlitePool, id: &str, reason: &str) -> Result<Tas
 pub async fn unblock_task(pool: &SqlitePool, id: &str) -> Result<Task> {
     let mut task = get_task(pool, id).await?;
 
-    // Return to in_progress if it was started, otherwise todo
-    task.status = if task.started_at.is_some() {
-        TaskStatus::InProgress.as_str().to_string()
-    } else {
-        TaskStatus::Todo.as_str().to_string()
-    };
+    task.status = TaskStatus::Todo.as_str().to_string();
     task.blocked_reason = None;
+    // Clear stale claim so task.next trigger can fire
+    task.claim_owner = None;
+    task.claim_claimed_at = None;
+    task.claim_lease_expires_at = None;
     // Set actor for trigger-based events (system action)
     task.last_edited_by = None;