@adhdev/daemon-core 0.9.82-rc.532 → 0.9.82-rc.534

package/dist/index.mjs

@@ -414,10 +414,10 @@ function readInjected(value) {
 }
 function getDaemonBuildInfo() {
   if (cached) return cached;
-  const commit = readInjected(true ? "2dcb1909a41f2358986e408601c8a1284c6f3685" : void 0) ?? "unknown";
-  const commitShort = readInjected(true ? "2dcb1909" : void 0) ?? (commit !== "unknown" ? commit.slice(0, 7) : "unknown");
-  const version = readInjected(true ? "0.9.82-rc.532" : void 0) ?? readInjected(typeof process !== "undefined" ? process.env?.ADHDEV_PKG_VERSION : void 0) ?? "unknown";
-  const builtAt = readInjected(true ? "2026-07-15T07:14:52.163Z" : void 0);
+  const commit = readInjected(true ? "7b6411ebc267c71484a3f7d16c392b5bd3e71bd8" : void 0) ?? "unknown";
+  const commitShort = readInjected(true ? "7b6411eb" : void 0) ?? (commit !== "unknown" ? commit.slice(0, 7) : "unknown");
+  const version = readInjected(true ? "0.9.82-rc.534" : void 0) ?? readInjected(typeof process !== "undefined" ? process.env?.ADHDEV_PKG_VERSION : void 0) ?? "unknown";
+  const builtAt = readInjected(true ? "2026-07-15T08:30:18.262Z" : void 0);
   cached = builtAt ? { commit, commitShort, version, builtAt } : { commit, commitShort, version };
   return cached;
 }
@@ -5626,12 +5626,16 @@ function getActiveSessionDeliveries(meshId, sessionId) {
     return [];
   }
 }
+function consumeSessionDelivery(meshId, sessionId, status, taskId) {
+  try {
+    return MeshRuntimeStore.getInstance().consumeSessionDelivery(meshId, sessionId, status, taskId);
+  } catch {
+    return 0;
+  }
+}
 function markSessionDeliveriesTerminal(meshId, sessionId, terminalStatus) {
   try {
-    const active = MeshRuntimeStore.getInstance().getActiveSessionDeliveries(meshId, sessionId);
-    for (const delivery of active) {
-      MeshRuntimeStore.getInstance().updateSessionDeliveryStatus(delivery.id, terminalStatus);
-    }
+    MeshRuntimeStore.getInstance().markOpenSessionDeliveriesTerminal(meshId, sessionId, terminalStatus);
   } catch {
   }
 }
@@ -7678,6 +7682,25 @@ var init_mesh_runtime_store = __esm({
         });
         this.maybeCheckpointWal();
       }
+      // DELIVERED-NOT-CONSUMED-REDRIVE monotonic FSM: the forward-progress lifecycle of a
+      // delivery is a strictly increasing rank — a status may only advance, never regress.
+      // The redrive bug was a NON-monotonic FSM: the transport-confirm callback
+      // (mesh-queue-assignment :384) writes 'delivered' unconditionally by PK, so when the
+      // worker's agent:generating_started raced AHEAD of the confirm and already flipped the
+      // row 'delivering'→'acked', the late confirm CLOBBERED 'acked' back to 'delivered'.
+      // taskDeliveryConsumed() (which keys on 'acked'/'completed') then read false forever,
+      // and the short-grace re-drive re-opened an already-consumed task. Enforcing the rank
+      // ordering here makes the two event orders converge on the same monotone terminal state
+      // regardless of arrival order, so a late confirm can never demote a consumed delivery.
+      // 'failed'/'expired'/'cancelled' are absorbing OUTCOMES, not progress ranks — they are
+      // always allowed (a genuine dispatch failure must be recordable even from 'acked').
+      static DELIVERY_PROGRESS_RANK = {
+        queued: 0,
+        delivering: 1,
+        delivered: 2,
+        acked: 3,
+        completed: 4
+      };
       updateSessionDeliveryStatus(id, status, opts) {
         const now = (/* @__PURE__ */ new Date()).toISOString();
         if (opts?.incrementAttempt) {
@@ -7686,13 +7709,75 @@ var init_mesh_runtime_store = __esm({
                 SET status = @status, last_error = @lastError, attempt_count = attempt_count + 1, updated_at = @updatedAt
                 WHERE id = @id
             `).run({ id, status, lastError: opts?.lastError ?? null, updatedAt: now });
-        } else {
+          return;
+        }
+        const targetRank = _MeshRuntimeStore.DELIVERY_PROGRESS_RANK[status];
+        if (targetRank === void 0) {
           this.db.prepare(`
                 UPDATE mesh_session_delivery
                 SET status = @status, last_error = @lastError, updated_at = @updatedAt
                 WHERE id = @id
             `).run({ id, status, lastError: opts?.lastError ?? null, updatedAt: now });
+          return;
         }
+        this.db.prepare(`
+            UPDATE mesh_session_delivery
+            SET status = @status, last_error = @lastError, updated_at = @updatedAt
+            WHERE id = @id AND (@targetRank >= CASE status
+                WHEN 'queued' THEN 0 WHEN 'delivering' THEN 1 WHEN 'delivered' THEN 2
+                WHEN 'acked' THEN 3 WHEN 'completed' THEN 4 ELSE 99 END)
+        `).run({ id, status, lastError: opts?.lastError ?? null, updatedAt: now, targetRank });
+      }
+      /**
+       * DELIVERED-NOT-CONSUMED-REDRIVE consume path. Advance a task's delivery record(s) to a
+       * CONSUMED status ('acked' or 'completed'), matching on mesh + session (+ taskId when the
+       * event names one) and INCLUDING rows already in 'delivered'/'acked'/'delivering'.
+       *
+       * The ack/terminal callers previously routed through getActiveSessionDeliveries(), whose SQL
+       * EXCLUDES 'delivered' — so in the normal event order (transport confirm flips 'delivered'
+       * BEFORE the worker's generating_started fires) the ack matched zero rows and the delivery
+       * was stranded 'delivered', never 'acked'. This finds the row by (mesh, session[, task])
+       * directly and relies on updateSessionDeliveryStatus's monotonic guard to only advance it.
+       * Returns the number of rows advanced.
+       */
+      consumeSessionDelivery(meshId, sessionId, status, taskId) {
+        const rows = this.db.prepare(
+          taskId ? `SELECT id, session_id FROM mesh_session_delivery
+                     WHERE mesh_id = ? AND task_id = ?
+                       AND status IN ('queued','delivering','delivered','acked')` : `SELECT id, session_id FROM mesh_session_delivery
+                     WHERE mesh_id = ? AND session_id = ?
+                       AND status IN ('queued','delivering','delivered','acked')`
+        ).all(meshId, taskId ?? sessionId);
+        let advanced = 0;
+        for (const r of rows) {
+          if (!sessionIdsEquivalent(r.session_id ?? void 0, sessionId)) continue;
+          this.updateSessionDeliveryStatus(r.id, status);
+          advanced++;
+        }
+        return advanced;
+      }
+      /**
+       * DELIVERED-NOT-CONSUMED-REDRIVE terminal path. Mark every OPEN delivery for a session
+       * (queued/delivering/delivered/acked) terminal on task completion/failure. The prior
+       * markSessionDeliveriesTerminal() routed through getActiveSessionDeliveries(), whose SQL
+       * EXCLUDES 'delivered'/'completed' — so a 'delivered' row (the common case, since the
+       * transport confirm flips it before the completion event) was never marked terminal and
+       * stayed 'delivered', keeping taskDeliveryConsumed() false and feeding the false re-drive.
+       * We match rows in OPEN states directly here. 'completed' advances monotonically (it is the
+       * top progress rank); 'failed' is an absorbing outcome written unconditionally.
+       */
+      markOpenSessionDeliveriesTerminal(meshId, sessionId, terminalStatus) {
+        const rows = this.db.prepare(
+          `SELECT id, session_id FROM mesh_session_delivery
+               WHERE mesh_id = ? AND status IN ('queued','delivering','delivered','acked')`
+        ).all(meshId);
+        let marked = 0;
+        for (const r of rows) {
+          if (!sessionIdsEquivalent(r.session_id ?? void 0, sessionId)) continue;
+          this.updateSessionDeliveryStatus(r.id, terminalStatus);
+          marked++;
+        }
+        return marked;
       }
       getActiveSessionDeliveries(meshId, sessionId) {
         const now = (/* @__PURE__ */ new Date()).toISOString();
@@ -20629,17 +20714,7 @@ function injectMeshSystemMessage(components, args) {
           updateDirectDispatchStatus(args.meshId, sessionId, "acked", soleTaskId);
         }
       }
-      const activeDeliveries = (() => {
-        try {
-          return MeshRuntimeStore.getInstance().getActiveSessionDeliveries(args.meshId, sessionId);
-        } catch {
-          return [];
-        }
-      })();
-      const deliveriesToAck = startedTaskId ? activeDeliveries.filter((d) => d.taskId === startedTaskId) : activeDeliveries;
-      for (const d of deliveriesToAck) {
-        updateSessionDeliveryStatus(d.id, "acked");
-      }
+      consumeSessionDelivery(args.meshId, sessionId, "acked", startedTaskId);
     }
   } else if (args.event === "agent:stopped") {
     const sessionId = resolveEventSessionId(args.metadataEvent, args.sourceInstanceId);
@@ -22079,6 +22154,9 @@ function recoverStrandedAssignedDispatches(components, meshId, store) {
   for (const key2 of [...deliveredNoTurnUnknownStreak.keys()]) {
     if (key2.startsWith(meshKeyPrefix) && !assignedKeys.has(key2)) deliveredNoTurnUnknownStreak.delete(key2);
   }
+  for (const key2 of [...deliveredUnconsumedUnknownStreak.keys()]) {
+    if (key2.startsWith(meshKeyPrefix) && !assignedKeys.has(key2)) deliveredUnconsumedUnknownStreak.delete(key2);
+  }
   for (const row of assigned) {
     const dispatchedAtMs = Date.parse(row.dispatchTimestamp ?? "");
     if (!Number.isFinite(dispatchedAtMs)) continue;
@@ -22090,13 +22168,33 @@ function recoverStrandedAssignedDispatches(components, meshId, store) {
         updateTaskStatus(meshId, row.id, status);
         continue;
       }
+      const shortStreakKey = `${meshId}::${row.id}`;
       const verdict = row.assignedSessionId ? resolveSessionBusyVerdict(components, row.assignedSessionId) : "IDLE_CONFIRMED";
-      if (verdict !== "GENERATING") {
+      if (verdict === "GENERATING") {
+        deliveredUnconsumedUnknownStreak.delete(shortStreakKey);
+      } else {
+        if (verdict === "IDLE_CONFIRMED") {
+          deliveredUnconsumedUnknownStreak.delete(shortStreakKey);
+        } else {
+          const streak = (deliveredUnconsumedUnknownStreak.get(shortStreakKey) ?? 0) + 1;
+          deliveredUnconsumedUnknownStreak.set(shortStreakKey, streak);
+          if (streak < RECLAIM_UNKNOWN_GRACE_TICKS) {
+            traceMeshEventDrop("short_redrive_deferred_unknown_verdict", {
+              taskId: row.id,
+              sessionId: row.assignedSessionId,
+              nodeId: row.assignedNodeId,
+              meshId,
+              event: "agent:generating_started"
+            }, `unknown ${streak}/${RECLAIM_UNKNOWN_GRACE_TICKS}`);
+            continue;
+          }
+        }
         const redriven = reclaimStrandedAssignedTask(meshId, row.id, {
           reason: "delivered_not_consumed_redrive",
           ageMs
         });
         if (redriven) {
+          deliveredUnconsumedUnknownStreak.delete(shortStreakKey);
           LOG.warn("MeshReconcile", `Re-drove delivered-but-unconsumed task ${row.id} on mesh ${meshId} (node=${row.assignedNodeId ?? "?"} session=${row.assignedSessionId ?? "?"}, delivered but no generating_started in ${Math.round(ageMs / 1e3)}s, verdict ${verdict} \u2192 ${redriven.status})`);
           traceMeshEventDrop("assigned_delivered_not_consumed_redrive", {
             taskId: row.id,
@@ -22651,7 +22749,7 @@ function setupMeshReconcileLoop(components) {
     }
   };
 }
-var coordinatorModalParkState, heldEventLedgerRecorded, ASSIGNED_STRANDED_DEADLINE_MS, DELIVERED_NO_TURN_DEADLINE_MS, ASSIGNED_DELIVERED_UNCONSUMED_REDRIVE_MS, RECLAIM_UNKNOWN_GRACE_TICKS, deliveredNoTurnUnknownStreak, ZOMBIE_ASSIGNED_MIN_AGE_MS, STRICT_SESSION_MATCH_TTL_MS, unresolvedForwardRejectionCounts, MAX_FORWARD_REJECTIONS, UNRESOLVED_FORWARD_NUDGE_DELAY_MS, unresolvedForwardNudgeTimer, unresolvedForwardNudgeRunning;
+var coordinatorModalParkState, heldEventLedgerRecorded, ASSIGNED_STRANDED_DEADLINE_MS, DELIVERED_NO_TURN_DEADLINE_MS, ASSIGNED_DELIVERED_UNCONSUMED_REDRIVE_MS, RECLAIM_UNKNOWN_GRACE_TICKS, deliveredNoTurnUnknownStreak, deliveredUnconsumedUnknownStreak, ZOMBIE_ASSIGNED_MIN_AGE_MS, STRICT_SESSION_MATCH_TTL_MS, unresolvedForwardRejectionCounts, MAX_FORWARD_REJECTIONS, UNRESOLVED_FORWARD_NUDGE_DELAY_MS, unresolvedForwardNudgeTimer, unresolvedForwardNudgeRunning;
 var init_mesh_reconcile_loop = __esm({
   "src/mesh/mesh-reconcile-loop.ts"() {
     "use strict";
@@ -22685,6 +22783,7 @@ var init_mesh_reconcile_loop = __esm({
     ASSIGNED_DELIVERED_UNCONSUMED_REDRIVE_MS = 25e3;
     RECLAIM_UNKNOWN_GRACE_TICKS = 3;
     deliveredNoTurnUnknownStreak = /* @__PURE__ */ new Map();
+    deliveredUnconsumedUnknownStreak = /* @__PURE__ */ new Map();
     ZOMBIE_ASSIGNED_MIN_AGE_MS = 30 * 60 * 1e3;
     STRICT_SESSION_MATCH_TTL_MS = 6e4;
     unresolvedForwardRejectionCounts = /* @__PURE__ */ new Map();
@@ -26901,8 +27000,8 @@ ${lastSnapshot}`;
         const stableMs = this.lastScreenChangeAt ? now - this.lastScreenChangeAt : 0;
         if (stableMs < 2e3) return;
         const startupModal = this.runParseApproval(this.recentOutputBuffer);
-        const startupStatus = this.runDetectStatus(screenText || this.recentOutputBuffer);
-        if (!startupModal && startupStatus !== "idle") {
+        const startupIdle = this.detectIdleHonoringOnNoMatch(screenText || this.recentOutputBuffer);
+        if (!startupModal && !startupIdle) {
           this.scheduleStartupSettleCheck();
           return;
         }
@@ -27038,6 +27137,38 @@ ${lastSnapshot}`;
           tailScreen: buildCliScreenSnapshot(tail)
         });
       }
+      /**
+       * WRITE-READINESS ONNOMATCH (opencode): resolve whether the session is
+       * genuinely idle *for the purpose of opening the PTY write gate*, honoring
+       * the manifest's `dispatchOrder.onNoMatch` policy.
+       *
+       * The split-brain this closes: the engine's settled evaluation runs
+       * detectStatus through parseSession, whose builder collapses a null verdict
+       * to `'idle'` (buildParseSessionFromTui: `detectStatus(input) ?? 'idle'`),
+       * so the dashboard status flips to idle via `script_detect`. But the
+       * write-readiness gates (resolveStartupState / sendMessage recovery) call
+       * `runDetectStatus` *directly* and require the literal `=== 'idle'` return —
+       * they never apply the `onNoMatch` policy. opencode's only idle cue is the
+       * `Ask anything` composer placeholder in the last-8-lines scope with
+       * `onNoMatch: preserve-last`; when that placeholder is momentarily out of
+       * frame the raw detector returns null, the gate stays shut, `this.ready`
+       * never flips, and the first queued prompt sits in `not_ready_pending_prompt`
+       * forever (no turn ever starts, so no turn-completion drain fires).
+       *
+       * The fix keeps the raw detector as the primary signal (unchanged behavior
+       * for providers whose detector returns a literal idle) and only falls back
+       * when BOTH (a) the manifest policy is idle-preserving (`preserve-last` or
+       * `idle`) AND (b) the engine has *durably* settled to idle (no in-flight
+       * turn, no modal, no parse error). That guard makes the fallback safe: it
+       * cannot open the gate mid-turn or while a modal is up.
+       */
+      detectIdleHonoringOnNoMatch(text) {
+        if (this.runDetectStatus(text) === "idle") return true;
+        const dispatchOrder = this.provider.tui?.dispatchOrder;
+        const onNoMatch = dispatchOrder?.onNoMatch;
+        if (onNoMatch !== "preserve-last" && onNoMatch !== "idle") return false;
+        return this.engine.currentStatus === "idle" && !this.engine.isWaitingForResponse && !this.engine.currentTurnScope && !this.engine.activeModal && !this.parseErrorMessage;
+      }
       runParseApproval(tail) {
         const screenText = this.terminalScreen.getText();
         const buffer = screenText || this.accumulatedBuffer;
@@ -27570,7 +27701,7 @@ ${lastSnapshot}`;
         }
         if (!this.ready) {
           this.resolveStartupState("send_precheck");
-          if (this.runDetectStatus(this.recentOutputBuffer) === "idle") {
+          if (!this.ready && this.detectIdleHonoringOnNoMatch(this.recentOutputBuffer)) {
             this.ready = true;
             this.startupParseGate = false;
             this.engine.setStatus("idle", "send_message_idle_prompt_recovery");