@gethmy/agent 1.17.0 → 1.18.0

package/dist/cli.js

@@ -105,6 +105,7 @@ var init_log = __esm(() => {
 var exports_board_helpers = {};
 __export(exports_board_helpers, {
   resolveCardLabels: () => resolveCardLabels,
+  releaseAssignedAgent: () => releaseAssignedAgent,
   moveCardToColumn: () => moveCardToColumn,
   moveCardAndAddLabel: () => moveCardAndAddLabel,
   hasLabel: () => hasLabel,
@@ -126,6 +127,9 @@ function resolveCardLabels(card, labelMap) {
 function hasLabel(cardLabels, labelName) {
   return cardLabels.some((l) => l.name.toLowerCase() === labelName.toLowerCase());
 }
+async function releaseAssignedAgent(client, cardId) {
+  await client.updateCard(cardId, { assignedAgentId: null });
+}
 async function moveCardToColumn(client, card, targetColumnName) {
   try {
     const board = await client.getBoard(card.project_id);
@@ -3862,6 +3866,11 @@ async function runCompletion(client, card, branchName, worktreePath, config, wor
   }
   if (config.completion.moveToColumn) {
     await moveCardToColumn(client, card, config.completion.moveToColumn);
+    try {
+      await releaseAssignedAgent(client, card.id);
+    } catch (err) {
+      log.warn(TAG15, `assignment release failed for #${card.short_id}: ${err instanceof Error ? err.message : err}`);
+    }
     if (onMovedToCompletion) {
       try {
         await onMovedToCompletion(card);
@@ -6190,6 +6199,7 @@ async function runTransition(client, card, plan, opts = {}) {
   const board = await withRetry("move", shortId, () => client.getBoard(card.project_id), attempts, backoffMs);
   const columns = board.columns;
   const labels = board.labels ?? [];
+  let moveLanded = false;
   if (plan.move) {
     const target = columns.find((c) => c.name.toLowerCase() === plan.move.columnName.toLowerCase());
     if (!target) {
@@ -6202,8 +6212,14 @@ async function runTransition(client, card, plan, opts = {}) {
       await withRetry("move", shortId, () => client.moveCard(card.id, target.id), attempts, backoffMs);
       log.info(TAG23, `#${shortId} → "${target.name}"`);
       card.column_id = target.id;
+      moveLanded = true;
+    } else {
+      moveLanded = true;
     }
   }
+  if (moveLanded && plan.onMoved) {
+    await plan.onMoved();
+  }
   if (plan.addLabels?.length) {
     const existing = new Set(card.labelIds ?? []);
     for (const { name, color } of plan.addLabels) {
@@ -6439,7 +6455,15 @@ class ReviewWorker {
       const cwd = this.worktreePath;
       if (!localMode) {
         log.info(this.tag, `Starting dev server on port ${port}...`);
-        this.devServerProcess = spawnInGroup("bun", ["run", "dev", "--", "--port", String(port)], { cwd, stdio: ["ignore", "pipe", "pipe"] });
+        const [devCmd, devArgs] = spawnRunArgs("dev", "--port", String(port));
+        this.devServerProcess = spawnInGroup(devCmd, devArgs, {
+          cwd,
+          stdio: ["ignore", "pipe", "pipe"]
+        });
+        let devServerSpawnError = null;
+        this.devServerProcess.once("error", (err) => {
+          devServerSpawnError = err;
+        });
         await this.client.updateAgentProgress(card.id, {
           agentIdentifier: agentIdentifier(this.id),
           agentName: `${AGENT_NAME} (Review)`,
@@ -6447,6 +6471,9 @@ class ReviewWorker {
           currentTask: `Starting dev server on port ${port}…`,
           progressPercent: 10
         });
+        if (devServerSpawnError) {
+          throw new DevServerReadinessError(`dev server failed to start (${devCmd}): ${devServerSpawnError.message}`);
+        }
         await waitForDevServer(this.devServerProcess, 30000);
         await probeDevServer(port);
         log.info(this.tag, `Dev server ready on port ${port}`);
@@ -6701,7 +6728,7 @@ class ReviewWorker {
 [parse_error] ${msg}
 `);
       });
-      if (this.process.stdout) {
+      if (this.process?.stdout) {
         parser.attach(this.process.stdout);
         if (runLog) {
           this.process.stdout.on("data", (chunk) => {
@@ -6710,14 +6737,14 @@ class ReviewWorker {
         }
       }
       let stderr = "";
-      this.process.stderr?.on("data", (data) => {
+      this.process?.stderr?.on("data", (data) => {
         stderr += data.toString();
         runLog?.stream.write(`[stderr] ${data.toString()}`);
       });
-      this.process.on("error", (err) => {
+      this.process?.on("error", (err) => {
         reject(new Error(`Failed to spawn claude: ${err.message}`));
       });
-      this.process.on("close", (code) => {
+      this.process?.on("close", (code) => {
         this.process = null;
         const stdout = textChunks.join("");
         const stats = tracker.stats;
@@ -6847,6 +6874,7 @@ var init_review_worker = __esm(() => {
   init_gate_collectors();
   init_git_diff_stat();
   init_log();
+  init_pm();
   init_process_group();
   init_progress_tracker();
   init_review_completion();
@@ -7502,16 +7530,16 @@ async function advanceStageOnGate(card, stage, stageIndex, def, evaluation, deps
     currentStage: next.stage.id,
     done: false
   });
-  deps.sink?.recordPlaybookAdvanced({
-    fromStageId: stage.id,
-    fromStageName: stage.name,
-    toStageId: next.stage.id,
-    toStageName: next.stage.name,
-    advancedBy: "system",
-    reason: summary
-  });
   await runTransition(deps.client, card, {
     move: { columnName: toColumn },
+    onMoved: () => deps.sink?.recordPlaybookAdvanced({
+      fromStageId: stage.id,
+      fromStageName: stage.name,
+      toStageId: next.stage.id,
+      toStageName: next.stage.name,
+      advancedBy: "system",
+      reason: summary
+    }),
     addLabels: [{ name: AGENT_LABEL }],
     ...isAgentRunnableOwner(next.stage.owner) ? { assignAgent: deps.agentId } : {}
   }, { store: deps.stateStore, runId: deps.runId });
@@ -9220,6 +9248,22 @@ var init_recovery = __esm(() => {
   init_worktree();
 });
 
+// src/claim.ts
+async function claimReviewCard(client, cardId, agentId) {
+  try {
+    const { claimed } = await client.claimCard(cardId, agentId);
+    log.debug(TAG34, `claim ${cardId} → ${claimed ? "won" : "lost"}`);
+    return claimed;
+  } catch (err) {
+    log.error(TAG34, `claim ${cardId} failed: ${err instanceof Error ? err.message : err}`);
+    return false;
+  }
+}
+var TAG34 = "claim";
+var init_claim = __esm(() => {
+  init_log();
+});
+
 // src/strand-recovery.ts
 var exports_strand_recovery = {};
 __export(exports_strand_recovery, {
@@ -9266,19 +9310,27 @@ async function reclaimPreReviewStrands(opts) {
     const prUrl = resolvePrUrl(card.description ?? null, branch, cwd, provider);
     if (prUrl)
       continue;
-    log.warn(TAG34, `#${card.short_id} stranded in review (branch pushed, no PR, unowned) — re-asserting daemon assignment`);
-    try {
-      await client.updateCard(card.id, { assignedAgentId: agentId });
-      reclaimed.push(card.id);
-    } catch (err) {
-      log.error(TAG34, `review re-claim failed for #${card.short_id}: ${err instanceof Error ? err.message : err}`);
+    const won = await claimReviewCard(client, card.id, agentId);
+    if (!won) {
+      log.debug(TAG35, `#${card.short_id} — lost the review claim race, skipping`);
+      continue;
+    }
+    log.warn(TAG35, `#${card.short_id} claimed for review (branch pushed, no PR, unowned)`);
+    reclaimed.push(card.id);
+    if (opts.onClaimed) {
+      try {
+        await opts.onClaimed(card);
+      } catch (err) {
+        log.error(TAG35, `onClaimed for #${card.short_id} failed: ${err instanceof Error ? err.message : err}`);
+      }
     }
   }
   return reclaimed;
 }
-var TAG34 = "strand-recovery";
+var TAG35 = "strand-recovery";
 var init_strand_recovery = __esm(() => {
   init_board_helpers();
+  init_claim();
   init_git_pr();
   init_log();
   init_review_worktree();
@@ -9327,7 +9379,7 @@ class Reconciler {
       clearInterval(this.timer);
       this.timer = null;
     }
-    log.info(TAG35, "Heartbeat stopped");
+    log.info(TAG36, "Heartbeat stopped");
   }
   async recoverStaleRuns() {
     if (!this.stateStore || !this.agentConfig)
@@ -9344,7 +9396,7 @@ class Reconciler {
       if (!daemonDead && !(heartbeatStale && ourZombie))
         continue;
       const reason = daemonDead ? `foreign daemon ${run.daemonPid} is dead` : `our worker lost card ${run.cardId} with ${Math.round((now - run.lastHeartbeatAt) / 1000)}s stale heartbeat`;
-      log.warn(TAG35, `zombie run ${run.runId} (#${run.cardShortId}): ${reason} — recovering`);
+      log.warn(TAG36, `zombie run ${run.runId} (#${run.cardShortId}): ${reason} — recovering`);
       await recoverRun(run, this.stateStore, this.client, this.agentConfig, {
         runId: run.runId,
         cardId: run.cardId,
@@ -9371,11 +9423,11 @@ class Reconciler {
       const stalledAt = Date.parse(card.updated_at ?? "");
       if (!Number.isFinite(stalledAt) || now - stalledAt < graceMs)
         continue;
-      log.warn(TAG35, `#${card.short_id} stranded in "${inProgressCol.name}" (no live run) — requeueing to "${pickupCol.name}"`);
+      log.warn(TAG36, `#${card.short_id} stranded in "${inProgressCol.name}" (no live run) — requeueing to "${pickupCol.name}"`);
       try {
         await this.client.moveCard(card.id, pickupCol.id);
       } catch (err) {
-        log.error(TAG35, `stranded requeue failed for #${card.short_id}: ${err instanceof Error ? err.message : err}`);
+        log.error(TAG36, `stranded requeue failed for #${card.short_id}: ${err instanceof Error ? err.message : err}`);
       }
     }
   }
@@ -9397,10 +9449,19 @@ class Reconciler {
       labelMap,
       reviewColumns: this.reviewColumns,
       approvedLabel: this.approvedLabel,
-      graceMs: this.agentConfig?.timing.staleHeartbeatMs ?? 120000,
+      graceMs: 0,
       knownCardIds,
       cwd: process.cwd(),
-      provider: this.gitProvider
+      provider: this.gitProvider,
+      onClaimed: async (card) => {
+        const column = columns.find((c) => c.id === card.column_id);
+        if (!column)
+          return;
+        const cardLabels = resolveCardLabels(card, labelMap);
+        const subtasks = card.subtasks ?? [];
+        log.info(TAG36, `Enqueuing claimed review card #${card.short_id} (agent-agnostic pickup)`);
+        await this.pool.enqueue(card, column, cardLabels, subtasks, "review");
+      }
     });
   }
   async releaseStalledApprovals(cards, columns, knownCardIds) {
@@ -9422,11 +9483,11 @@ class Reconciler {
       const parkedAt = Date.parse(card.updated_at ?? "");
       if (!Number.isFinite(parkedAt) || now - parkedAt < ttlMs)
         continue;
-      log.warn(TAG35, `#${card.short_id} parked for approval > ${planning.approvalTtlHours}h — auto-releasing to "${pickupCol.name}"`);
+      log.warn(TAG36, `#${card.short_id} parked for approval > ${planning.approvalTtlHours}h — auto-releasing to "${pickupCol.name}"`);
       try {
         await this.client.moveCard(card.id, pickupCol.id);
       } catch (err) {
-        log.error(TAG35, `auto-release failed for #${card.short_id}: ${err instanceof Error ? err.message : err}`);
+        log.error(TAG36, `auto-release failed for #${card.short_id}: ${err instanceof Error ? err.message : err}`);
       }
     }
   }
@@ -9469,21 +9530,21 @@ class Reconciler {
           const subtasks = card.subtasks ?? [];
           const mode = route.mode;
           if (route.stage) {
-            log.info(TAG35, `Stage card #${card.short_id} (stage "${card.current_stage}") in "${column.name}" — routing to the stage executor (implement) regardless of column`);
+            log.info(TAG36, `Stage card #${card.short_id} (stage "${card.current_stage}") in "${column.name}" — routing to the stage executor (implement) regardless of column`);
           }
           if (mode === "review" && this.approvedLabel && hasLabel(cardLabels, this.approvedLabel)) {
-            log.debug(TAG35, `Skipping #${card.short_id} — already has "${this.approvedLabel}" label`);
+            log.debug(TAG36, `Skipping #${card.short_id} — already has "${this.approvedLabel}" label`);
             continue;
           }
           if (mode === "review" && hasLabel(cardLabels, NEED_REVIEW_LABEL)) {
-            log.debug(TAG35, `Skipping #${card.short_id} — has "${NEED_REVIEW_LABEL}" label (needs human)`);
+            log.debug(TAG36, `Skipping #${card.short_id} — has "${NEED_REVIEW_LABEL}" label (needs human)`);
             continue;
           }
           if (mode === "review" && !extractBranchFromDescription(card.description)) {
-            log.debug(TAG35, `Skipping #${card.short_id} — no branch reference (not qualified for auto-review)`);
+            log.debug(TAG36, `Skipping #${card.short_id} — no branch reference (not qualified for auto-review)`);
             continue;
           }
-          log.info(TAG35, `Missed assignment: #${card.short_id} "${card.title}" (${mode}) — enqueueing`);
+          log.info(TAG36, `Missed assignment: #${card.short_id} "${card.title}" (${mode}) — enqueueing`);
           await this.pool.enqueue(card, column, cardLabels, subtasks, mode);
         }
       }
@@ -9494,18 +9555,18 @@ class Reconciler {
       await this.recoverStrandedReview(cards, columns, labelMap, knownCardIds);
       for (const knownId of knownCardIds) {
         if (!allAgentCardIds.has(knownId)) {
-          log.info(TAG35, `Missed unassign: ${knownId} — removing`);
+          log.info(TAG36, `Missed unassign: ${knownId} — removing`);
           await this.pool.removeCard(knownId);
         }
       }
       await this.releaseStalledApprovals(cards, columns, knownCardIds);
-      log.debug(TAG35, `Reconciled: ${assignedCards.length} assigned, ${knownCardIds.size} known`);
+      log.debug(TAG36, `Reconciled: ${assignedCards.length} assigned, ${knownCardIds.size} known`);
     } catch (err) {
-      log.error(TAG35, `Heartbeat failed: ${err instanceof Error ? err.message : err}`);
+      log.error(TAG36, `Heartbeat failed: ${err instanceof Error ? err.message : err}`);
     }
   }
 }
-var TAG35 = "reconcile";
+var TAG36 = "reconcile";
 var init_reconcile = __esm(() => {
   init_board_helpers();
   init_git_pr();
@@ -9545,7 +9606,7 @@ function prettyBanner(config, version) {
       checks.push({ kind: "ok", message });
     },
     warn(message) {
-      log.warn(TAG36, message);
+      log.warn(TAG37, message);
       checks.push({ kind: "warn", message: message.split(`
 `, 1)[0] });
     },
@@ -9570,25 +9631,25 @@ function prettyBanner(config, version) {
   };
 }
 function jsonBanner(config, version) {
-  log.info(TAG36, `Harmony Agent Daemon v${version} starting...`);
-  log.info(TAG36, `Project: ${config.projectId} | Pool: ${config.agent.poolSize} | Model: ${config.agent.claude.model} | Runner: ${config.agent.runner} | Pickup: ${config.agent.pickupColumns.join(", ")}`);
+  log.info(TAG37, `Harmony Agent Daemon v${version} starting...`);
+  log.info(TAG37, `Project: ${config.projectId} | Pool: ${config.agent.poolSize} | Model: ${config.agent.claude.model} | Runner: ${config.agent.runner} | Pickup: ${config.agent.pickupColumns.join(", ")}`);
   if (config.agent.review.enabled) {
-    log.info(TAG36, `Review: enabled | Columns: ${config.agent.review.pickupColumns.join(", ")} | → ${config.agent.review.moveToColumn} / ${config.agent.review.failColumn}`);
+    log.info(TAG37, `Review: enabled | Columns: ${config.agent.review.pickupColumns.join(", ")} | → ${config.agent.review.moveToColumn} / ${config.agent.review.failColumn}`);
   }
   let failed = false;
   return {
     setProjectName(_name) {},
     setGitProvider(provider) {
-      log.info(TAG36, `Git provider: ${provider}`);
+      log.info(TAG37, `Git provider: ${provider}`);
     },
     setHttpPort(port) {
-      log.info(TAG36, `HTTP server on port ${port}`);
+      log.info(TAG37, `HTTP server on port ${port}`);
     },
     check(message) {
-      log.info(TAG36, message);
+      log.info(TAG37, message);
     },
     warn(message) {
-      log.warn(TAG36, message);
+      log.warn(TAG37, message);
     },
     fail() {
       failed = true;
@@ -9596,7 +9657,7 @@ function jsonBanner(config, version) {
     async ready(message) {
       if (failed)
         return;
-      log.info(TAG36, message);
+      log.info(TAG37, message);
     }
   };
 }
@@ -9677,7 +9738,7 @@ function cyan(s) {
 function yellow(s) {
   return `${ANSI.yellow}${s}${ANSI.reset}`;
 }
-var TAG36 = "daemon", RULE_WIDTH = 70, ANSI;
+var TAG37 = "daemon", RULE_WIDTH = 70, ANSI;
 var init_startup_banner = __esm(() => {
   init_log();
   ANSI = {
@@ -9828,13 +9889,13 @@ class Watcher {
   }
   async start() {
     if (!isPretty()) {
-      log.info(TAG37, "Connecting to Supabase realtime (broadcast)...");
+      log.info(TAG38, "Connecting to Supabase realtime (broadcast)...");
     }
     this.supabase = createClient(this.credentials.supabaseUrl, this.credentials.supabaseAnonKey);
     const presenceChannel = this.supabase.channel(`board-presence-${this.projectId}`);
     this.subscribeBroadcast();
     presenceChannel.on("presence", { event: "sync" }, () => {
-      log.debug(TAG37, "Presence sync");
+      log.debug(TAG38, "Presence sync");
     }).subscribe(async (status) => {
       if (status === "SUBSCRIBED") {
         await presenceChannel.track({
@@ -9847,7 +9908,7 @@ class Watcher {
           agentName: this.identity.agentName
         });
         if (!isPretty() || !this.suppressStartupLogs) {
-          log.info(TAG37, "Presence tracked on board-presence channel");
+          log.info(TAG38, "Presence tracked on board-presence channel");
         }
         this.presenceTracked = true;
         this.maybeResolveReady();
@@ -9860,13 +9921,13 @@ class Watcher {
       return;
     const gen = ++this.broadcastGen;
     this.channel = this.supabase.channel(`board-${this.projectId}`).on("broadcast", { event: "card_update" }, (msg) => {
-      log.debug(TAG37, `Broadcast: card_update ${JSON.stringify(msg.payload)}`);
+      log.debug(TAG38, `Broadcast: card_update ${JSON.stringify(msg.payload)}`);
       this.onCardBroadcast({
         event: "card_update",
         payload: msg.payload ?? {}
       });
     }).on("broadcast", { event: "card_created" }, (msg) => {
-      log.debug(TAG37, `Broadcast: card_created ${JSON.stringify(msg.payload)}`);
+      log.debug(TAG38, `Broadcast: card_created ${JSON.stringify(msg.payload)}`);
       this.onCardBroadcast({
         event: "card_created",
         payload: msg.payload ?? {}
@@ -9876,7 +9937,7 @@ class Watcher {
       const cardId = payload.card_id;
       const command = payload.command;
       if (cardId && command) {
-        log.info(TAG37, `Broadcast: agent_command ${command} for ${cardId}`);
+        log.info(TAG38, `Broadcast: agent_command ${command} for ${cardId}`);
         this.onAgentCommand?.({ cardId, command });
       }
     }).subscribe((status) => {
@@ -9886,13 +9947,13 @@ class Watcher {
         this.connected = true;
         this.reconnectAttempts = 0;
         if (!isPretty() || !this.suppressStartupLogs) {
-          log.info(TAG37, "Broadcast subscription active");
+          log.info(TAG38, "Broadcast subscription active");
         }
         this.maybeResolveReady();
       } else if (status === "CHANNEL_ERROR" || status === "TIMED_OUT" || status === "CLOSED") {
         this.connected = false;
         if (!this.stopping) {
-          log.warn(TAG37, `Broadcast subscription ${status} — scheduling reconnect`);
+          log.warn(TAG38, `Broadcast subscription ${status} — scheduling reconnect`);
           this.scheduleReconnect();
         }
       }
@@ -9911,7 +9972,7 @@ class Watcher {
   async reconnectBroadcast() {
     if (this.stopping || !this.supabase)
       return;
-    log.warn(TAG37, `Reconnecting broadcast subscription (attempt ${this.reconnectAttempts})`);
+    log.warn(TAG38, `Reconnecting broadcast subscription (attempt ${this.reconnectAttempts})`);
     if (this.channel) {
       const old = this.channel;
       this.channel = null;
@@ -9941,10 +10002,10 @@ class Watcher {
       this.supabase = null;
     }
     this.connected = false;
-    log.info(TAG37, "Broadcast subscription stopped");
+    log.info(TAG38, "Broadcast subscription stopped");
   }
 }
-var TAG37 = "watcher";
+var TAG38 = "watcher";
 var init_watcher = __esm(() => {
   init_log();
 });
@@ -10031,10 +10092,10 @@ function runWorktreeGc(basePath, store, opts = {}) {
     });
   } catch {}
   if (result.removed.length > 0) {
-    log.info(TAG38, `GC removed ${result.removed.length} orphan worktree(s): ${result.removed.map((p) => p.split("/").pop()).join(", ")}`);
+    log.info(TAG39, `GC removed ${result.removed.length} orphan worktree(s): ${result.removed.map((p) => p.split("/").pop()).join(", ")}`);
   }
   if (result.errors.length > 0) {
-    log.warn(TAG38, `GC had ${result.errors.length} error(s): ${result.errors.map((e) => `${e.path}: ${e.error}`).join("; ")}`);
+    log.warn(TAG39, `GC had ${result.errors.length} error(s): ${result.errors.map((e) => `${e.path}: ${e.error}`).join("; ")}`);
   }
   return result;
 }
@@ -10064,7 +10125,7 @@ function pruneFailedRemoteBranches(opts) {
   } catch (err) {
     const detail = gitErrorDetail2(err);
     if (isTransientGitNetworkError(detail)) {
-      log.debug(TAG38, `Remote branch GC skipped — remote unreachable: ${detail}`);
+      log.debug(TAG39, `Remote branch GC skipped — remote unreachable: ${detail}`);
       return result;
     }
     result.errors.push({ ref: "fetch", error: detail });
@@ -10103,7 +10164,7 @@ function pruneFailedRemoteBranches(opts) {
       continue;
     }
     if (clock() > sweepDeadline) {
-      log.debug(TAG38, `Remote branch GC budget spent — removed ${result.removed.length}, remaining deferred to next tick`);
+      log.debug(TAG39, `Remote branch GC budget spent — removed ${result.removed.length}, remaining deferred to next tick`);
       break;
     }
     try {
@@ -10116,17 +10177,17 @@ function pruneFailedRemoteBranches(opts) {
     } catch (err) {
       const detail = gitErrorDetail2(err);
       if (isTransientGitNetworkError(detail)) {
-        log.debug(TAG38, `Remote branch GC interrupted — remote unreachable: ${detail}`);
+        log.debug(TAG39, `Remote branch GC interrupted — remote unreachable: ${detail}`);
         break;
       }
       result.errors.push({ ref, error: detail });
     }
   }
   if (result.removed.length > 0) {
-    log.info(TAG38, `Pruned ${result.removed.length} stale remote branch(es) under ${opts.prefix}: ${result.removed.join(", ")}`);
+    log.info(TAG39, `Pruned ${result.removed.length} stale remote branch(es) under ${opts.prefix}: ${result.removed.join(", ")}`);
   }
   if (result.errors.length > 0) {
-    log.warn(TAG38, `Remote branch GC had ${result.errors.length} error(s): ${result.errors.map((e) => `${e.ref}: ${e.error}`).join("; ")}`);
+    log.warn(TAG39, `Remote branch GC had ${result.errors.length} error(s): ${result.errors.map((e) => `${e.ref}: ${e.error}`).join("; ")}`);
   }
   return result;
 }
@@ -10157,13 +10218,13 @@ class WorktreeGc {
     try {
       runWorktreeGc(this.basePath, this.store);
     } catch (err) {
-      log.warn(TAG38, `GC tick failed: ${err instanceof Error ? err.message : err}`);
+      log.warn(TAG39, `GC tick failed: ${err instanceof Error ? err.message : err}`);
     }
     if (this.remoteOpts) {
       try {
         pruneFailedRemoteBranches(this.remoteOpts);
       } catch (err) {
-        log.warn(TAG38, `Remote GC tick failed: ${err instanceof Error ? err.message : err}`);
+        log.warn(TAG39, `Remote GC tick failed: ${err instanceof Error ? err.message : err}`);
       }
     }
   }
@@ -10177,7 +10238,7 @@ function getRepoRoot2() {
     return null;
   }
 }
-var TAG38 = "worktree-gc", GIT_NETWORK_TIMEOUT_MS = 30000, GIT_SSH_CONNECT_TIMEOUT_SECS = 10, GIT_PRUNE_SWEEP_BUDGET_MS = 60000, GIT_NETWORK_EXEC, TRANSIENT_GIT_NETWORK_ERROR;
+var TAG39 = "worktree-gc", GIT_NETWORK_TIMEOUT_MS = 30000, GIT_SSH_CONNECT_TIMEOUT_SECS = 10, GIT_PRUNE_SWEEP_BUDGET_MS = 60000, GIT_NETWORK_EXEC, TRANSIENT_GIT_NETWORK_ERROR;
 var init_worktree_gc = __esm(() => {
   init_log();
   init_worktree();
@@ -10281,7 +10342,7 @@ async function main() {
   } catch (err) {
     if (err instanceof ConfigValidationError) {
       banner.fail();
-      log.error(TAG39, err.message);
+      log.error(TAG40, err.message);
       process.exit(1);
     }
     throw err;
@@ -10291,7 +10352,7 @@ async function main() {
   } catch (err) {
     if (err instanceof ConfigValidationError) {
       banner.fail();
-      log.error(TAG39, err.message);
+      log.error(TAG40, err.message);
       process.exit(1);
     }
     throw err;
@@ -10401,7 +10462,7 @@ async function main() {
     if (shuttingDown)
       return;
     shuttingDown = true;
-    log.info(TAG39, `Received ${signal}, shutting down gracefully...`);
+    log.info(TAG40, `Received ${signal}, shutting down gracefully...`);
     reconciler.stop();
     mergeMonitor?.stop();
     worktreeGc.stop();
@@ -10411,18 +10472,18 @@ async function main() {
     }
     await watcher.stop();
     await pool.shutdown();
-    log.info(TAG39, "Daemon stopped.");
+    log.info(TAG40, "Daemon stopped.");
     process.exit(exitCode);
   };
   process.on("SIGINT", () => shutdown("SIGINT"));
   process.on("SIGTERM", () => shutdown("SIGTERM"));
   process.on("uncaughtException", (err) => {
-    log.error(TAG39, `Uncaught exception: ${err.message}`);
+    log.error(TAG40, `Uncaught exception: ${err.message}`);
     exitCode = 1;
     shutdown("uncaughtException");
   });
   process.on("unhandledRejection", (reason) => {
-    log.error(TAG39, `Unhandled rejection: ${reason instanceof Error ? reason.message : String(reason)}`);
+    log.error(TAG40, `Unhandled rejection: ${reason instanceof Error ? reason.message : String(reason)}`);
     exitCode = 1;
     shutdown("unhandledRejection");
   });
@@ -10475,29 +10536,29 @@ async function handleBroadcast(event, client, pool, config, agentId) {
   if (assignedAgentId === undefined)
     return;
   if (assignedAgentId === agentId) {
-    log.info(TAG39, `Broadcast: card ${cardId} assigned to agent`);
+    log.info(TAG40, `Broadcast: card ${cardId} assigned to agent`);
     try {
       await pool.resetAttemptsForReassign(cardId);
       await tryEnqueueCard(cardId, client, pool, config, agentId);
     } catch (err) {
-      log.error(TAG39, `Failed to process assignment: ${err instanceof Error ? err.message : err}`);
+      log.error(TAG40, `Failed to process assignment: ${err instanceof Error ? err.message : err}`);
     }
   } else if (pool.isCardKnown(cardId)) {
-    log.info(TAG39, `Broadcast: card ${cardId} unassigned from agent`);
+    log.info(TAG40, `Broadcast: card ${cardId} unassigned from agent`);
     await pool.removeCard(cardId);
   }
 }
 async function tryEnqueueCard(cardId, client, pool, config, agentId) {
   const { card } = await client.getCard(cardId);
   if (card.assigned_agent_id !== agentId) {
-    log.debug(TAG39, `Card ${cardId} no longer assigned to agent — skipping`);
+    log.debug(TAG40, `Card ${cardId} no longer assigned to agent — skipping`);
     return;
   }
   const board = await client.getBoard(config.projectId, { summary: true });
   const columns = board.columns;
   const column = columns.find((c) => c.id === card.column_id);
   if (!column) {
-    log.warn(TAG39, `Column not found for card ${cardId}`);
+    log.warn(TAG40, `Column not found for card ${cardId}`);
     return;
   }
   const route = classifyPickup(card, column.name, {
@@ -10506,27 +10567,27 @@ async function tryEnqueueCard(cardId, client, pool, config, agentId) {
     playbooks: config.agent.playbooks
   });
   if (!route) {
-    log.info(TAG39, `Card #${card.short_id} is in "${column.name}", not a pickup/review/stage column — skipping`);
+    log.info(TAG40, `Card #${card.short_id} is in "${column.name}", not a pickup/review/stage column — skipping`);
     return;
   }
   if (route.stage) {
-    log.info(TAG39, `Card #${card.short_id} is a playbook stage card (stage "${card.current_stage}") in "${column.name}" — routing to the stage executor (implement pool) regardless of column`);
+    log.info(TAG40, `Card #${card.short_id} is a playbook stage card (stage "${card.current_stage}") in "${column.name}" — routing to the stage executor (implement pool) regardless of column`);
   }
   const mode = route.mode;
   const labelMap = buildLabelMap(board.labels ?? []);
   const cardLabels = resolveCardLabels(card, labelMap);
   const subtasks = card.subtasks ?? [];
   if (mode === "review" && config.agent.review.approvedLabel && hasLabel(cardLabels, config.agent.review.approvedLabel)) {
-    log.debug(TAG39, `Card #${card.short_id} already has "${config.agent.review.approvedLabel}" — skipping review`);
+    log.debug(TAG40, `Card #${card.short_id} already has "${config.agent.review.approvedLabel}" — skipping review`);
     return;
   }
   if (mode === "review" && !extractBranchFromDescription(card.description)) {
-    log.info(TAG39, `Card #${card.short_id} has no branch reference — skipping auto-review`);
+    log.info(TAG40, `Card #${card.short_id} has no branch reference — skipping auto-review`);
     return;
   }
   await pool.enqueue(card, column, cardLabels, subtasks, mode);
 }
-var TAG39 = "daemon", PKG_VERSION;
+var TAG40 = "daemon", PKG_VERSION;
 var init_src = __esm(() => {
   init_board_helpers();
   init_config();

package/dist/index.js

@@ -104,6 +104,7 @@ var init_log = __esm(() => {
 var exports_board_helpers = {};
 __export(exports_board_helpers, {
   resolveCardLabels: () => resolveCardLabels,
+  releaseAssignedAgent: () => releaseAssignedAgent,
   moveCardToColumn: () => moveCardToColumn,
   moveCardAndAddLabel: () => moveCardAndAddLabel,
   hasLabel: () => hasLabel,
@@ -125,6 +126,9 @@ function resolveCardLabels(card, labelMap) {
 function hasLabel(cardLabels, labelName) {
   return cardLabels.some((l) => l.name.toLowerCase() === labelName.toLowerCase());
 }
+async function releaseAssignedAgent(client, cardId) {
+  await client.updateCard(cardId, { assignedAgentId: null });
+}
 async function moveCardToColumn(client, card, targetColumnName) {
   try {
     const board = await client.getBoard(card.project_id);
@@ -3861,6 +3865,11 @@ async function runCompletion(client, card, branchName, worktreePath, config, wor
   }
   if (config.completion.moveToColumn) {
     await moveCardToColumn(client, card, config.completion.moveToColumn);
+    try {
+      await releaseAssignedAgent(client, card.id);
+    } catch (err) {
+      log.warn(TAG15, `assignment release failed for #${card.short_id}: ${err instanceof Error ? err.message : err}`);
+    }
     if (onMovedToCompletion) {
       try {
         await onMovedToCompletion(card);
@@ -6189,6 +6198,7 @@ async function runTransition(client, card, plan, opts = {}) {
   const board = await withRetry("move", shortId, () => client.getBoard(card.project_id), attempts, backoffMs);
   const columns = board.columns;
   const labels = board.labels ?? [];
+  let moveLanded = false;
   if (plan.move) {
     const target = columns.find((c) => c.name.toLowerCase() === plan.move.columnName.toLowerCase());
     if (!target) {
@@ -6201,8 +6211,14 @@ async function runTransition(client, card, plan, opts = {}) {
       await withRetry("move", shortId, () => client.moveCard(card.id, target.id), attempts, backoffMs);
       log.info(TAG23, `#${shortId} → "${target.name}"`);
       card.column_id = target.id;
+      moveLanded = true;
+    } else {
+      moveLanded = true;
     }
   }
+  if (moveLanded && plan.onMoved) {
+    await plan.onMoved();
+  }
   if (plan.addLabels?.length) {
     const existing = new Set(card.labelIds ?? []);
     for (const { name, color } of plan.addLabels) {
@@ -6438,7 +6454,15 @@ class ReviewWorker {
       const cwd = this.worktreePath;
       if (!localMode) {
         log.info(this.tag, `Starting dev server on port ${port}...`);
-        this.devServerProcess = spawnInGroup("bun", ["run", "dev", "--", "--port", String(port)], { cwd, stdio: ["ignore", "pipe", "pipe"] });
+        const [devCmd, devArgs] = spawnRunArgs("dev", "--port", String(port));
+        this.devServerProcess = spawnInGroup(devCmd, devArgs, {
+          cwd,
+          stdio: ["ignore", "pipe", "pipe"]
+        });
+        let devServerSpawnError = null;
+        this.devServerProcess.once("error", (err) => {
+          devServerSpawnError = err;
+        });
         await this.client.updateAgentProgress(card.id, {
           agentIdentifier: agentIdentifier(this.id),
           agentName: `${AGENT_NAME} (Review)`,
@@ -6446,6 +6470,9 @@ class ReviewWorker {
           currentTask: `Starting dev server on port ${port}…`,
           progressPercent: 10
         });
+        if (devServerSpawnError) {
+          throw new DevServerReadinessError(`dev server failed to start (${devCmd}): ${devServerSpawnError.message}`);
+        }
         await waitForDevServer(this.devServerProcess, 30000);
         await probeDevServer(port);
         log.info(this.tag, `Dev server ready on port ${port}`);
@@ -6700,7 +6727,7 @@ class ReviewWorker {
 [parse_error] ${msg}
 `);
       });
-      if (this.process.stdout) {
+      if (this.process?.stdout) {
         parser.attach(this.process.stdout);
         if (runLog) {
           this.process.stdout.on("data", (chunk) => {
@@ -6709,14 +6736,14 @@ class ReviewWorker {
         }
       }
       let stderr = "";
-      this.process.stderr?.on("data", (data) => {
+      this.process?.stderr?.on("data", (data) => {
         stderr += data.toString();
         runLog?.stream.write(`[stderr] ${data.toString()}`);
       });
-      this.process.on("error", (err) => {
+      this.process?.on("error", (err) => {
         reject(new Error(`Failed to spawn claude: ${err.message}`));
       });
-      this.process.on("close", (code) => {
+      this.process?.on("close", (code) => {
         this.process = null;
         const stdout = textChunks.join("");
         const stats = tracker.stats;
@@ -6846,6 +6873,7 @@ var init_review_worker = __esm(() => {
   init_gate_collectors();
   init_git_diff_stat();
   init_log();
+  init_pm();
   init_process_group();
   init_progress_tracker();
   init_review_completion();
@@ -7501,16 +7529,16 @@ async function advanceStageOnGate(card, stage, stageIndex, def, evaluation, deps
     currentStage: next.stage.id,
     done: false
   });
-  deps.sink?.recordPlaybookAdvanced({
-    fromStageId: stage.id,
-    fromStageName: stage.name,
-    toStageId: next.stage.id,
-    toStageName: next.stage.name,
-    advancedBy: "system",
-    reason: summary
-  });
   await runTransition(deps.client, card, {
     move: { columnName: toColumn },
+    onMoved: () => deps.sink?.recordPlaybookAdvanced({
+      fromStageId: stage.id,
+      fromStageName: stage.name,
+      toStageId: next.stage.id,
+      toStageName: next.stage.name,
+      advancedBy: "system",
+      reason: summary
+    }),
     addLabels: [{ name: AGENT_LABEL }],
     ...isAgentRunnableOwner(next.stage.owner) ? { assignAgent: deps.agentId } : {}
   }, { store: deps.stateStore, runId: deps.runId });
@@ -9219,6 +9247,22 @@ var init_recovery = __esm(() => {
   init_worktree();
 });
 
+// src/claim.ts
+async function claimReviewCard(client, cardId, agentId) {
+  try {
+    const { claimed } = await client.claimCard(cardId, agentId);
+    log.debug(TAG34, `claim ${cardId} → ${claimed ? "won" : "lost"}`);
+    return claimed;
+  } catch (err) {
+    log.error(TAG34, `claim ${cardId} failed: ${err instanceof Error ? err.message : err}`);
+    return false;
+  }
+}
+var TAG34 = "claim";
+var init_claim = __esm(() => {
+  init_log();
+});
+
 // src/strand-recovery.ts
 var exports_strand_recovery = {};
 __export(exports_strand_recovery, {
@@ -9265,19 +9309,27 @@ async function reclaimPreReviewStrands(opts) {
     const prUrl = resolvePrUrl(card.description ?? null, branch, cwd, provider);
     if (prUrl)
       continue;
-    log.warn(TAG34, `#${card.short_id} stranded in review (branch pushed, no PR, unowned) — re-asserting daemon assignment`);
-    try {
-      await client.updateCard(card.id, { assignedAgentId: agentId });
-      reclaimed.push(card.id);
-    } catch (err) {
-      log.error(TAG34, `review re-claim failed for #${card.short_id}: ${err instanceof Error ? err.message : err}`);
+    const won = await claimReviewCard(client, card.id, agentId);
+    if (!won) {
+      log.debug(TAG35, `#${card.short_id} — lost the review claim race, skipping`);
+      continue;
+    }
+    log.warn(TAG35, `#${card.short_id} claimed for review (branch pushed, no PR, unowned)`);
+    reclaimed.push(card.id);
+    if (opts.onClaimed) {
+      try {
+        await opts.onClaimed(card);
+      } catch (err) {
+        log.error(TAG35, `onClaimed for #${card.short_id} failed: ${err instanceof Error ? err.message : err}`);
+      }
     }
   }
   return reclaimed;
 }
-var TAG34 = "strand-recovery";
+var TAG35 = "strand-recovery";
 var init_strand_recovery = __esm(() => {
   init_board_helpers();
+  init_claim();
   init_git_pr();
   init_log();
   init_review_worktree();
@@ -9326,7 +9378,7 @@ class Reconciler {
       clearInterval(this.timer);
       this.timer = null;
     }
-    log.info(TAG35, "Heartbeat stopped");
+    log.info(TAG36, "Heartbeat stopped");
   }
   async recoverStaleRuns() {
     if (!this.stateStore || !this.agentConfig)
@@ -9343,7 +9395,7 @@ class Reconciler {
       if (!daemonDead && !(heartbeatStale && ourZombie))
         continue;
       const reason = daemonDead ? `foreign daemon ${run.daemonPid} is dead` : `our worker lost card ${run.cardId} with ${Math.round((now - run.lastHeartbeatAt) / 1000)}s stale heartbeat`;
-      log.warn(TAG35, `zombie run ${run.runId} (#${run.cardShortId}): ${reason} — recovering`);
+      log.warn(TAG36, `zombie run ${run.runId} (#${run.cardShortId}): ${reason} — recovering`);
       await recoverRun(run, this.stateStore, this.client, this.agentConfig, {
         runId: run.runId,
         cardId: run.cardId,
@@ -9370,11 +9422,11 @@ class Reconciler {
       const stalledAt = Date.parse(card.updated_at ?? "");
       if (!Number.isFinite(stalledAt) || now - stalledAt < graceMs)
         continue;
-      log.warn(TAG35, `#${card.short_id} stranded in "${inProgressCol.name}" (no live run) — requeueing to "${pickupCol.name}"`);
+      log.warn(TAG36, `#${card.short_id} stranded in "${inProgressCol.name}" (no live run) — requeueing to "${pickupCol.name}"`);
       try {
         await this.client.moveCard(card.id, pickupCol.id);
       } catch (err) {
-        log.error(TAG35, `stranded requeue failed for #${card.short_id}: ${err instanceof Error ? err.message : err}`);
+        log.error(TAG36, `stranded requeue failed for #${card.short_id}: ${err instanceof Error ? err.message : err}`);
       }
     }
   }
@@ -9396,10 +9448,19 @@ class Reconciler {
       labelMap,
       reviewColumns: this.reviewColumns,
       approvedLabel: this.approvedLabel,
-      graceMs: this.agentConfig?.timing.staleHeartbeatMs ?? 120000,
+      graceMs: 0,
       knownCardIds,
       cwd: process.cwd(),
-      provider: this.gitProvider
+      provider: this.gitProvider,
+      onClaimed: async (card) => {
+        const column = columns.find((c) => c.id === card.column_id);
+        if (!column)
+          return;
+        const cardLabels = resolveCardLabels(card, labelMap);
+        const subtasks = card.subtasks ?? [];
+        log.info(TAG36, `Enqueuing claimed review card #${card.short_id} (agent-agnostic pickup)`);
+        await this.pool.enqueue(card, column, cardLabels, subtasks, "review");
+      }
     });
   }
   async releaseStalledApprovals(cards, columns, knownCardIds) {
@@ -9421,11 +9482,11 @@ class Reconciler {
       const parkedAt = Date.parse(card.updated_at ?? "");
       if (!Number.isFinite(parkedAt) || now - parkedAt < ttlMs)
         continue;
-      log.warn(TAG35, `#${card.short_id} parked for approval > ${planning.approvalTtlHours}h — auto-releasing to "${pickupCol.name}"`);
+      log.warn(TAG36, `#${card.short_id} parked for approval > ${planning.approvalTtlHours}h — auto-releasing to "${pickupCol.name}"`);
       try {
         await this.client.moveCard(card.id, pickupCol.id);
       } catch (err) {
-        log.error(TAG35, `auto-release failed for #${card.short_id}: ${err instanceof Error ? err.message : err}`);
+        log.error(TAG36, `auto-release failed for #${card.short_id}: ${err instanceof Error ? err.message : err}`);
       }
     }
   }
@@ -9468,21 +9529,21 @@ class Reconciler {
           const subtasks = card.subtasks ?? [];
           const mode = route.mode;
           if (route.stage) {
-            log.info(TAG35, `Stage card #${card.short_id} (stage "${card.current_stage}") in "${column.name}" — routing to the stage executor (implement) regardless of column`);
+            log.info(TAG36, `Stage card #${card.short_id} (stage "${card.current_stage}") in "${column.name}" — routing to the stage executor (implement) regardless of column`);
           }
           if (mode === "review" && this.approvedLabel && hasLabel(cardLabels, this.approvedLabel)) {
-            log.debug(TAG35, `Skipping #${card.short_id} — already has "${this.approvedLabel}" label`);
+            log.debug(TAG36, `Skipping #${card.short_id} — already has "${this.approvedLabel}" label`);
             continue;
           }
           if (mode === "review" && hasLabel(cardLabels, NEED_REVIEW_LABEL)) {
-            log.debug(TAG35, `Skipping #${card.short_id} — has "${NEED_REVIEW_LABEL}" label (needs human)`);
+            log.debug(TAG36, `Skipping #${card.short_id} — has "${NEED_REVIEW_LABEL}" label (needs human)`);
             continue;
           }
           if (mode === "review" && !extractBranchFromDescription(card.description)) {
-            log.debug(TAG35, `Skipping #${card.short_id} — no branch reference (not qualified for auto-review)`);
+            log.debug(TAG36, `Skipping #${card.short_id} — no branch reference (not qualified for auto-review)`);
             continue;
           }
-          log.info(TAG35, `Missed assignment: #${card.short_id} "${card.title}" (${mode}) — enqueueing`);
+          log.info(TAG36, `Missed assignment: #${card.short_id} "${card.title}" (${mode}) — enqueueing`);
           await this.pool.enqueue(card, column, cardLabels, subtasks, mode);
         }
       }
@@ -9493,18 +9554,18 @@ class Reconciler {
       await this.recoverStrandedReview(cards, columns, labelMap, knownCardIds);
       for (const knownId of knownCardIds) {
         if (!allAgentCardIds.has(knownId)) {
-          log.info(TAG35, `Missed unassign: ${knownId} — removing`);
+          log.info(TAG36, `Missed unassign: ${knownId} — removing`);
           await this.pool.removeCard(knownId);
         }
       }
       await this.releaseStalledApprovals(cards, columns, knownCardIds);
-      log.debug(TAG35, `Reconciled: ${assignedCards.length} assigned, ${knownCardIds.size} known`);
+      log.debug(TAG36, `Reconciled: ${assignedCards.length} assigned, ${knownCardIds.size} known`);
     } catch (err) {
-      log.error(TAG35, `Heartbeat failed: ${err instanceof Error ? err.message : err}`);
+      log.error(TAG36, `Heartbeat failed: ${err instanceof Error ? err.message : err}`);
     }
   }
 }
-var TAG35 = "reconcile";
+var TAG36 = "reconcile";
 var init_reconcile = __esm(() => {
   init_board_helpers();
   init_git_pr();
@@ -9544,7 +9605,7 @@ function prettyBanner(config, version) {
       checks.push({ kind: "ok", message });
     },
     warn(message) {
-      log.warn(TAG36, message);
+      log.warn(TAG37, message);
       checks.push({ kind: "warn", message: message.split(`
 `, 1)[0] });
     },
@@ -9569,25 +9630,25 @@ function prettyBanner(config, version) {
   };
 }
 function jsonBanner(config, version) {
-  log.info(TAG36, `Harmony Agent Daemon v${version} starting...`);
-  log.info(TAG36, `Project: ${config.projectId} | Pool: ${config.agent.poolSize} | Model: ${config.agent.claude.model} | Runner: ${config.agent.runner} | Pickup: ${config.agent.pickupColumns.join(", ")}`);
+  log.info(TAG37, `Harmony Agent Daemon v${version} starting...`);
+  log.info(TAG37, `Project: ${config.projectId} | Pool: ${config.agent.poolSize} | Model: ${config.agent.claude.model} | Runner: ${config.agent.runner} | Pickup: ${config.agent.pickupColumns.join(", ")}`);
   if (config.agent.review.enabled) {
-    log.info(TAG36, `Review: enabled | Columns: ${config.agent.review.pickupColumns.join(", ")} | → ${config.agent.review.moveToColumn} / ${config.agent.review.failColumn}`);
+    log.info(TAG37, `Review: enabled | Columns: ${config.agent.review.pickupColumns.join(", ")} | → ${config.agent.review.moveToColumn} / ${config.agent.review.failColumn}`);
   }
   let failed = false;
   return {
     setProjectName(_name) {},
     setGitProvider(provider) {
-      log.info(TAG36, `Git provider: ${provider}`);
+      log.info(TAG37, `Git provider: ${provider}`);
     },
     setHttpPort(port) {
-      log.info(TAG36, `HTTP server on port ${port}`);
+      log.info(TAG37, `HTTP server on port ${port}`);
     },
     check(message) {
-      log.info(TAG36, message);
+      log.info(TAG37, message);
     },
     warn(message) {
-      log.warn(TAG36, message);
+      log.warn(TAG37, message);
     },
     fail() {
       failed = true;
@@ -9595,7 +9656,7 @@ function jsonBanner(config, version) {
     async ready(message) {
       if (failed)
         return;
-      log.info(TAG36, message);
+      log.info(TAG37, message);
     }
   };
 }
@@ -9676,7 +9737,7 @@ function cyan(s) {
 function yellow(s) {
   return `${ANSI.yellow}${s}${ANSI.reset}`;
 }
-var TAG36 = "daemon", RULE_WIDTH = 70, ANSI;
+var TAG37 = "daemon", RULE_WIDTH = 70, ANSI;
 var init_startup_banner = __esm(() => {
   init_log();
   ANSI = {
@@ -9827,13 +9888,13 @@ class Watcher {
   }
   async start() {
     if (!isPretty()) {
-      log.info(TAG37, "Connecting to Supabase realtime (broadcast)...");
+      log.info(TAG38, "Connecting to Supabase realtime (broadcast)...");
     }
     this.supabase = createClient(this.credentials.supabaseUrl, this.credentials.supabaseAnonKey);
     const presenceChannel = this.supabase.channel(`board-presence-${this.projectId}`);
     this.subscribeBroadcast();
     presenceChannel.on("presence", { event: "sync" }, () => {
-      log.debug(TAG37, "Presence sync");
+      log.debug(TAG38, "Presence sync");
     }).subscribe(async (status) => {
       if (status === "SUBSCRIBED") {
         await presenceChannel.track({
@@ -9846,7 +9907,7 @@ class Watcher {
           agentName: this.identity.agentName
         });
         if (!isPretty() || !this.suppressStartupLogs) {
-          log.info(TAG37, "Presence tracked on board-presence channel");
+          log.info(TAG38, "Presence tracked on board-presence channel");
         }
         this.presenceTracked = true;
         this.maybeResolveReady();
@@ -9859,13 +9920,13 @@ class Watcher {
       return;
     const gen = ++this.broadcastGen;
     this.channel = this.supabase.channel(`board-${this.projectId}`).on("broadcast", { event: "card_update" }, (msg) => {
-      log.debug(TAG37, `Broadcast: card_update ${JSON.stringify(msg.payload)}`);
+      log.debug(TAG38, `Broadcast: card_update ${JSON.stringify(msg.payload)}`);
       this.onCardBroadcast({
         event: "card_update",
         payload: msg.payload ?? {}
       });
     }).on("broadcast", { event: "card_created" }, (msg) => {
-      log.debug(TAG37, `Broadcast: card_created ${JSON.stringify(msg.payload)}`);
+      log.debug(TAG38, `Broadcast: card_created ${JSON.stringify(msg.payload)}`);
       this.onCardBroadcast({
         event: "card_created",
         payload: msg.payload ?? {}
@@ -9875,7 +9936,7 @@ class Watcher {
       const cardId = payload.card_id;
       const command = payload.command;
       if (cardId && command) {
-        log.info(TAG37, `Broadcast: agent_command ${command} for ${cardId}`);
+        log.info(TAG38, `Broadcast: agent_command ${command} for ${cardId}`);
         this.onAgentCommand?.({ cardId, command });
       }
     }).subscribe((status) => {
@@ -9885,13 +9946,13 @@ class Watcher {
         this.connected = true;
         this.reconnectAttempts = 0;
         if (!isPretty() || !this.suppressStartupLogs) {
-          log.info(TAG37, "Broadcast subscription active");
+          log.info(TAG38, "Broadcast subscription active");
         }
         this.maybeResolveReady();
       } else if (status === "CHANNEL_ERROR" || status === "TIMED_OUT" || status === "CLOSED") {
         this.connected = false;
         if (!this.stopping) {
-          log.warn(TAG37, `Broadcast subscription ${status} — scheduling reconnect`);
+          log.warn(TAG38, `Broadcast subscription ${status} — scheduling reconnect`);
           this.scheduleReconnect();
         }
       }
@@ -9910,7 +9971,7 @@ class Watcher {
   async reconnectBroadcast() {
     if (this.stopping || !this.supabase)
       return;
-    log.warn(TAG37, `Reconnecting broadcast subscription (attempt ${this.reconnectAttempts})`);
+    log.warn(TAG38, `Reconnecting broadcast subscription (attempt ${this.reconnectAttempts})`);
     if (this.channel) {
       const old = this.channel;
       this.channel = null;
@@ -9940,10 +10001,10 @@ class Watcher {
       this.supabase = null;
     }
     this.connected = false;
-    log.info(TAG37, "Broadcast subscription stopped");
+    log.info(TAG38, "Broadcast subscription stopped");
   }
 }
-var TAG37 = "watcher";
+var TAG38 = "watcher";
 var init_watcher = __esm(() => {
   init_log();
 });
@@ -10030,10 +10091,10 @@ function runWorktreeGc(basePath, store, opts = {}) {
     });
   } catch {}
   if (result.removed.length > 0) {
-    log.info(TAG38, `GC removed ${result.removed.length} orphan worktree(s): ${result.removed.map((p) => p.split("/").pop()).join(", ")}`);
+    log.info(TAG39, `GC removed ${result.removed.length} orphan worktree(s): ${result.removed.map((p) => p.split("/").pop()).join(", ")}`);
   }
   if (result.errors.length > 0) {
-    log.warn(TAG38, `GC had ${result.errors.length} error(s): ${result.errors.map((e) => `${e.path}: ${e.error}`).join("; ")}`);
+    log.warn(TAG39, `GC had ${result.errors.length} error(s): ${result.errors.map((e) => `${e.path}: ${e.error}`).join("; ")}`);
   }
   return result;
 }
@@ -10063,7 +10124,7 @@ function pruneFailedRemoteBranches(opts) {
   } catch (err) {
     const detail = gitErrorDetail2(err);
     if (isTransientGitNetworkError(detail)) {
-      log.debug(TAG38, `Remote branch GC skipped — remote unreachable: ${detail}`);
+      log.debug(TAG39, `Remote branch GC skipped — remote unreachable: ${detail}`);
       return result;
     }
     result.errors.push({ ref: "fetch", error: detail });
@@ -10102,7 +10163,7 @@ function pruneFailedRemoteBranches(opts) {
       continue;
     }
     if (clock() > sweepDeadline) {
-      log.debug(TAG38, `Remote branch GC budget spent — removed ${result.removed.length}, remaining deferred to next tick`);
+      log.debug(TAG39, `Remote branch GC budget spent — removed ${result.removed.length}, remaining deferred to next tick`);
       break;
     }
     try {
@@ -10115,17 +10176,17 @@ function pruneFailedRemoteBranches(opts) {
     } catch (err) {
       const detail = gitErrorDetail2(err);
       if (isTransientGitNetworkError(detail)) {
-        log.debug(TAG38, `Remote branch GC interrupted — remote unreachable: ${detail}`);
+        log.debug(TAG39, `Remote branch GC interrupted — remote unreachable: ${detail}`);
         break;
       }
       result.errors.push({ ref, error: detail });
     }
   }
   if (result.removed.length > 0) {
-    log.info(TAG38, `Pruned ${result.removed.length} stale remote branch(es) under ${opts.prefix}: ${result.removed.join(", ")}`);
+    log.info(TAG39, `Pruned ${result.removed.length} stale remote branch(es) under ${opts.prefix}: ${result.removed.join(", ")}`);
   }
   if (result.errors.length > 0) {
-    log.warn(TAG38, `Remote branch GC had ${result.errors.length} error(s): ${result.errors.map((e) => `${e.ref}: ${e.error}`).join("; ")}`);
+    log.warn(TAG39, `Remote branch GC had ${result.errors.length} error(s): ${result.errors.map((e) => `${e.ref}: ${e.error}`).join("; ")}`);
   }
   return result;
 }
@@ -10156,13 +10217,13 @@ class WorktreeGc {
     try {
       runWorktreeGc(this.basePath, this.store);
     } catch (err) {
-      log.warn(TAG38, `GC tick failed: ${err instanceof Error ? err.message : err}`);
+      log.warn(TAG39, `GC tick failed: ${err instanceof Error ? err.message : err}`);
     }
     if (this.remoteOpts) {
       try {
         pruneFailedRemoteBranches(this.remoteOpts);
       } catch (err) {
-        log.warn(TAG38, `Remote GC tick failed: ${err instanceof Error ? err.message : err}`);
+        log.warn(TAG39, `Remote GC tick failed: ${err instanceof Error ? err.message : err}`);
       }
     }
   }
@@ -10176,7 +10237,7 @@ function getRepoRoot2() {
     return null;
   }
 }
-var TAG38 = "worktree-gc", GIT_NETWORK_TIMEOUT_MS = 30000, GIT_SSH_CONNECT_TIMEOUT_SECS = 10, GIT_PRUNE_SWEEP_BUDGET_MS = 60000, GIT_NETWORK_EXEC, TRANSIENT_GIT_NETWORK_ERROR;
+var TAG39 = "worktree-gc", GIT_NETWORK_TIMEOUT_MS = 30000, GIT_SSH_CONNECT_TIMEOUT_SECS = 10, GIT_PRUNE_SWEEP_BUDGET_MS = 60000, GIT_NETWORK_EXEC, TRANSIENT_GIT_NETWORK_ERROR;
 var init_worktree_gc = __esm(() => {
   init_log();
   init_worktree();
@@ -10280,7 +10341,7 @@ async function main() {
   } catch (err) {
     if (err instanceof ConfigValidationError) {
       banner.fail();
-      log.error(TAG39, err.message);
+      log.error(TAG40, err.message);
       process.exit(1);
     }
     throw err;
@@ -10290,7 +10351,7 @@ async function main() {
   } catch (err) {
     if (err instanceof ConfigValidationError) {
       banner.fail();
-      log.error(TAG39, err.message);
+      log.error(TAG40, err.message);
       process.exit(1);
     }
     throw err;
@@ -10400,7 +10461,7 @@ async function main() {
     if (shuttingDown)
       return;
     shuttingDown = true;
-    log.info(TAG39, `Received ${signal}, shutting down gracefully...`);
+    log.info(TAG40, `Received ${signal}, shutting down gracefully...`);
     reconciler.stop();
     mergeMonitor?.stop();
     worktreeGc.stop();
@@ -10410,18 +10471,18 @@ async function main() {
     }
     await watcher.stop();
     await pool.shutdown();
-    log.info(TAG39, "Daemon stopped.");
+    log.info(TAG40, "Daemon stopped.");
     process.exit(exitCode);
   };
   process.on("SIGINT", () => shutdown("SIGINT"));
   process.on("SIGTERM", () => shutdown("SIGTERM"));
   process.on("uncaughtException", (err) => {
-    log.error(TAG39, `Uncaught exception: ${err.message}`);
+    log.error(TAG40, `Uncaught exception: ${err.message}`);
     exitCode = 1;
     shutdown("uncaughtException");
   });
   process.on("unhandledRejection", (reason) => {
-    log.error(TAG39, `Unhandled rejection: ${reason instanceof Error ? reason.message : String(reason)}`);
+    log.error(TAG40, `Unhandled rejection: ${reason instanceof Error ? reason.message : String(reason)}`);
     exitCode = 1;
     shutdown("unhandledRejection");
   });
@@ -10474,29 +10535,29 @@ async function handleBroadcast(event, client, pool, config, agentId) {
   if (assignedAgentId === undefined)
     return;
   if (assignedAgentId === agentId) {
-    log.info(TAG39, `Broadcast: card ${cardId} assigned to agent`);
+    log.info(TAG40, `Broadcast: card ${cardId} assigned to agent`);
     try {
       await pool.resetAttemptsForReassign(cardId);
       await tryEnqueueCard(cardId, client, pool, config, agentId);
     } catch (err) {
-      log.error(TAG39, `Failed to process assignment: ${err instanceof Error ? err.message : err}`);
+      log.error(TAG40, `Failed to process assignment: ${err instanceof Error ? err.message : err}`);
     }
   } else if (pool.isCardKnown(cardId)) {
-    log.info(TAG39, `Broadcast: card ${cardId} unassigned from agent`);
+    log.info(TAG40, `Broadcast: card ${cardId} unassigned from agent`);
     await pool.removeCard(cardId);
   }
 }
 async function tryEnqueueCard(cardId, client, pool, config, agentId) {
   const { card } = await client.getCard(cardId);
   if (card.assigned_agent_id !== agentId) {
-    log.debug(TAG39, `Card ${cardId} no longer assigned to agent — skipping`);
+    log.debug(TAG40, `Card ${cardId} no longer assigned to agent — skipping`);
     return;
   }
   const board = await client.getBoard(config.projectId, { summary: true });
   const columns = board.columns;
   const column = columns.find((c) => c.id === card.column_id);
   if (!column) {
-    log.warn(TAG39, `Column not found for card ${cardId}`);
+    log.warn(TAG40, `Column not found for card ${cardId}`);
     return;
   }
   const route = classifyPickup(card, column.name, {
@@ -10505,27 +10566,27 @@ async function tryEnqueueCard(cardId, client, pool, config, agentId) {
     playbooks: config.agent.playbooks
   });
   if (!route) {
-    log.info(TAG39, `Card #${card.short_id} is in "${column.name}", not a pickup/review/stage column — skipping`);
+    log.info(TAG40, `Card #${card.short_id} is in "${column.name}", not a pickup/review/stage column — skipping`);
     return;
   }
   if (route.stage) {
-    log.info(TAG39, `Card #${card.short_id} is a playbook stage card (stage "${card.current_stage}") in "${column.name}" — routing to the stage executor (implement pool) regardless of column`);
+    log.info(TAG40, `Card #${card.short_id} is a playbook stage card (stage "${card.current_stage}") in "${column.name}" — routing to the stage executor (implement pool) regardless of column`);
   }
   const mode = route.mode;
   const labelMap = buildLabelMap(board.labels ?? []);
   const cardLabels = resolveCardLabels(card, labelMap);
   const subtasks = card.subtasks ?? [];
   if (mode === "review" && config.agent.review.approvedLabel && hasLabel(cardLabels, config.agent.review.approvedLabel)) {
-    log.debug(TAG39, `Card #${card.short_id} already has "${config.agent.review.approvedLabel}" — skipping review`);
+    log.debug(TAG40, `Card #${card.short_id} already has "${config.agent.review.approvedLabel}" — skipping review`);
     return;
   }
   if (mode === "review" && !extractBranchFromDescription(card.description)) {
-    log.info(TAG39, `Card #${card.short_id} has no branch reference — skipping auto-review`);
+    log.info(TAG40, `Card #${card.short_id} has no branch reference — skipping auto-review`);
     return;
   }
   await pool.enqueue(card, column, cardLabels, subtasks, mode);
 }
-var TAG39 = "daemon", PKG_VERSION;
+var TAG40 = "daemon", PKG_VERSION;
 var init_src = __esm(() => {
   init_board_helpers();
   init_config();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gethmy/agent",
-  "version": "1.17.0",
+  "version": "1.18.0",
   "description": "Push-based agent daemon for Harmony — watches board assignments and spawns Claude CLI workers",
   "type": "module",
   "main": "dist/index.js",