zyndo 0.3.3 → 0.3.4

package/dist/sellerDaemon.js

@@ -23,7 +23,12 @@ const HEARTBEAT_INTERVAL_MS = 45_000;
 // broker's authoritative task list. This catches missed `task.assigned`
 // events caused by broker restarts (in-memory event queue reset), network
 // partitions, or dropped SSE frames. 8 cycles at 25s = ~200s safety net.
-const RECONCILE_EVERY_N_POLLS = 8;
+// Periodic reconcile safety net. Every N poll cycles, check /agent/tasks for
+// anything the daemon doesn't know about. Was 8 (~200s at 25s polls). Too
+// slow — users saw multi-minute hire pickup delays after broker restarts
+// broke the event cursor. 2 (~50s) is fast enough to be invisible while
+// still being cheap. Incident 2026-04-14.
+const RECONCILE_EVERY_N_POLLS = 2;
 // ---------------------------------------------------------------------------
 // Reconnect error classification (incident 2026-04-09 follow-up)
 //
@@ -178,7 +183,21 @@ export async function startSellerDaemon(config, opts) {
             logger.error(`Identity registration failed: ${err instanceof Error ? err.message : String(err)}. Deliveries will be unsigned.`);
         }
     }
-    let lastEventId = loadLastEventId();
+    // Reset the event cursor on every daemon startup. The persisted value from
+    // a previous session can be higher than the broker's in-memory event
+    // counter after a broker restart (Railway auto-deploys reset the counter),
+    // causing pollEvents to return empty indefinitely — tasks could only be
+    // recovered via the periodic reconcile loop, producing multi-minute hire
+    // pickup delays. Starting from 0 makes pollEvents instantly see any current
+    // event stream. The startup reconcile already catches any in-flight task
+    // the broker persisted, and the activeTasks Set dedupes any replayed
+    // events so no work is done twice. Incident 2026-04-14.
+    const persistedCursor = loadLastEventId();
+    if (persistedCursor > 0) {
+        logger.info(`Event cursor reset from ${persistedCursor} → 0 on startup (broker may have restarted; reconcile will recover in-flight tasks).`);
+    }
+    let lastEventId = 0;
+    saveLastEventId(0);
     let lastHeartbeat = Date.now();
     const activeTasks = new Set();
     let pollsSinceReconcile = 0;
@@ -330,12 +349,23 @@ export async function startSellerDaemon(config, opts) {
                             logger.info(`Reconnected successfully (attempt ${attempt + 1}).`);
                             lastHeartbeat = Date.now();
                             reconnected = true;
-                            // Force a reconcile on the next tick: the broker may have
-                            // restarted and wiped its in-memory event queue, so our
-                            // lastEventId cursor is now stale relative to the fresh
-                            // nextEventId counter. Without this the seller polls forever
-                            // and never sees task.assigned events that happened while we
-                            // were disconnected or that were issued after the reset.
+                            // The broker almost certainly restarted (that's why the
+                            // heartbeat failed in the first place). Its in-memory event
+                            // counter has reset to 0 while our in-memory lastEventId is
+                            // still at the old high value — every subsequent pollEvents
+                            // call would return empty because we'd be asking for events
+                            // "since <high>" from a stream that now starts at 1. Reset
+                            // the cursor in memory AND on disk so the next poll sees the
+                            // current event stream. Incident 2026-04-14.
+                            if (lastEventId > 0) {
+                                logger.info(`Event cursor reset ${lastEventId} → 0 after reconnect (broker restart suspected).`);
+                            }
+                            lastEventId = 0;
+                            saveLastEventId(0);
+                            // Also run an immediate reconcile so any task that was
+                            // already assigned to us before the broker restarted (and
+                            // is now past the event horizon) gets picked up without
+                            // waiting for the periodic cycle.
                             pollsSinceReconcile = RECONCILE_EVERY_N_POLLS;
                             await reconcileTasks('reconnect');
                             break;
@@ -374,6 +404,14 @@ export async function startSellerDaemon(config, opts) {
                             logger.info(`Re-registered: agentId=${holder.current.agentId}`);
                             lastHeartbeat = Date.now();
                             reconnected = true;
+                            // Same cursor reset as the reconnect branch — a fresh connect
+                            // against a post-restart broker means the event stream is
+                            // brand new, so ask for "since=0".
+                            if (lastEventId > 0) {
+                                logger.info(`Event cursor reset ${lastEventId} → 0 after re-register.`);
+                            }
+                            lastEventId = 0;
+                            saveLastEventId(0);
                             pollsSinceReconcile = RECONCILE_EVERY_N_POLLS;
                             await reconcileTasks('re-register');
                         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "zyndo",
-  "version": "0.3.3",
+  "version": "0.3.4",
   "description": "The agent-to-agent CLI tool for sellers in the Zyndo Marketplace",
   "type": "module",
   "license": "MIT",