@yemi33/minions 0.1.1054 → 0.1.1055

package/CHANGELOG.md

@@ -1,11 +1,12 @@
 # Changelog
 
-## 0.1.1054 (2026-04-17)
+## 0.1.1055 (2026-04-17)
 
 ### Features
 - seed realActivityMap at spawn time, stamp pid in live-output (#1200)
 
 ### Fixes
+- guard undefined agent in pending dispatch loop (closes #1206) (#1210)
 -  improve fallback meeting conclusion
 -  remove command center chevron
 - stamp live-output.log stub before spawn (#1198)

package/engine.js

@@ -3483,6 +3483,35 @@ async function tickInner() {
       log('warn', `Duplicate dispatch ID ${item.id} in pending queue — skipping`);
       continue;
     }
+    // #1206: Guard against undefined/non-string item.agent. A corrupted dispatch
+    // entry (manual edit, serialization round-trip, cleared field) would otherwise
+    // be handed to spawnAgent, which crashes with `TypeError: "path" argument must
+    // be of type string. Received undefined` and re-queues — every tick. Try to
+    // resolve a fallback via routing; if none is available, skip this tick.
+    if (!item.agent || typeof item.agent !== 'string') {
+      const fallback = resolveAgent(item.type || WORK_TYPE.FIX, config);
+      if (!fallback) {
+        log('warn', `Pending dispatch ${item.id} has no agent and routing returned no fallback — skipping`);
+        continue;
+      }
+      log('info', `Pending dispatch ${item.id} missing agent; routed → ${fallback} (#1206 guard)`);
+      item.agent = fallback;
+      item.agentName = config.agents[fallback]?.name || tempAgents.get(fallback)?.name || fallback;
+      item.agentRole = config.agents[fallback]?.role || tempAgents.get(fallback)?.role || 'Agent';
+      // Persist so the fix survives across ticks even if this dispatch is skipped
+      // later in the loop (branch lock, concurrency cap, agent busy, etc.).
+      try {
+        mutateDispatch((dp) => {
+          const p = (dp.pending || []).find(d => d.id === item.id);
+          if (p) {
+            p.agent = item.agent;
+            p.agentName = item.agentName;
+            p.agentRole = item.agentRole;
+          }
+          return dp;
+        });
+      } catch (e) { log('warn', `Persist agent resolution for ${item.id} failed: ${e.message}`); }
+    }
     if (busyAgents.has(item.agent)) {
       // Agent busy reassignment: if item has been waiting on a busy agent past the threshold,
       // try to find an alternative agent via routing. Skip explicitly assigned items.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.1054",
+  "version": "0.1.1055",
   "description": "Multi-agent AI dev team that runs from ~/.minions/ — five autonomous agents share a single engine, dashboard, and knowledge base",
   "bin": {
     "minions": "bin/minions.js"