@yemi33/minions 0.1.1054 → 0.1.1056

package/CHANGELOG.md

@@ -1,11 +1,13 @@
 # Changelog
 
-## 0.1.1054 (2026-04-17)
+## 0.1.1056 (2026-04-17)
 
 ### Features
 - seed realActivityMap at spawn time, stamp pid in live-output (#1200)
 
 ### Fixes
+-  reassign pending items from unspawned temp agents to idle named agents (#1204) (#1212)
+- 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,66 @@ 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}`); }
+    }
+    // #1204: Pre-assigned unspawned temp agents never unblock naturally.
+    // When a batch discovery saturates maxConcurrent, resolveAgent hands out temp
+    // IDs that get stamped onto pending items. Because those temp IDs are never
+    // in busyAgents (they were never spawned), the agent-busy reassignment path
+    // never fires — items sit forever even when named agents go idle. Re-route
+    // them eagerly before the busy check so an idle named agent can pick up.
+    const isUnspawnedTemp = item.agent?.startsWith('temp-') && !busyAgents.has(item.agent);
+    if (isUnspawnedTemp) {
+      const altAgent = resolveAgent(item.type, config);
+      if (altAgent && altAgent !== item.agent) {
+        const prevAgent = item.agent;
+        item.agent = altAgent;
+        item.agentName = config.agents[altAgent]?.name || tempAgents.get(altAgent)?.name || altAgent;
+        item.agentRole = config.agents[altAgent]?.role || tempAgents.get(altAgent)?.role || 'Agent';
+        delete item._agentBusySince;
+        delete item.skipReason;
+        log('info', `Reassigning ${item.id} from unspawned temp ${prevAgent} to ${altAgent} — temp agent never spawned`);
+        // Persist reassignment to dispatch.json so it survives restarts/ticks
+        mutateDispatch((dp) => {
+          const p = (dp.pending || []).find(d => d.id === item.id);
+          if (p) {
+            p.agent = altAgent;
+            p.agentName = item.agentName;
+            p.agentRole = item.agentRole;
+            delete p._agentBusySince;
+            delete p.skipReason;
+          }
+          return dp;
+        });
+      }
+    }
     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.1056",
   "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"