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

package/dist/index.js

@@ -419,10 +419,10 @@ function readInjected(value) {
 }
 function getDaemonBuildInfo() {
   if (cached) return cached;
-  const commit = readInjected(true ? "4bcba3bd84cc9f4cd08a3d93965f10f43fcf5dad" : void 0) ?? "unknown";
-  const commitShort = readInjected(true ? "4bcba3bd" : void 0) ?? (commit !== "unknown" ? commit.slice(0, 7) : "unknown");
-  const version = readInjected(true ? "0.9.82-rc.533" : void 0) ?? readInjected(typeof process !== "undefined" ? process.env?.ADHDEV_PKG_VERSION : void 0) ?? "unknown";
-  const builtAt = readInjected(true ? "2026-07-15T07:45:39.050Z" : 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;
 }
@@ -5632,12 +5632,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 {
   }
 }
@@ -7685,6 +7689,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) {
@@ -7693,13 +7716,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();
@@ -20627,17 +20712,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);
@@ -22077,6 +22152,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;
@@ -22088,13 +22166,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,
@@ -22649,7 +22747,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";
@@ -22683,6 +22781,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();