@yemi33/minions 0.1.2134 → 0.1.2136

package/engine.js

@@ -7700,33 +7700,11 @@ async function tickInner() {
     } catch (err) { log('warn', `Stall detection error: ${err?.message || err}`); }
   }
 
-  // 3. Discover new work from sources
-  // Cap temp-agent creation at (maxConcurrent - activeCount) for this tick so
-  // temps count against maxConcurrent like named agents. Without this, a mass
-  // discovery pass (e.g. 20 PR build failures) would stamp a fresh temp agent
-  // onto every pending item regardless of concurrency cap, and those temps
-  // would spawn over subsequent ticks — overwhelming the OS and mass-orphaning
-  // the dispatches (#1209).
-  {
-    const dispatchPre = getDispatch();
-    const activeCountPre = (dispatchPre.active || []).length;
-    const maxC = resolveMaxConcurrent(config);
-    setTempBudget(Math.max(0, maxC - activeCountPre));
-  }
   try { pruneStalePrDispatches(config); } catch (e) { log('warn', 'prune stale PR dispatches: ' + e.message); }
-  let discoveryOk = true;
-  try { await discoverWork(config); } catch (e) { log('warn', 'discoverWork: ' + e.message); discoveryOk = false; }
-  if (_isTickStale(myGeneration)) return;
-
-  // 4. Update snapshot
-  safe('updateSnapshot', () => updateSnapshot(config));
-
-  if (!discoveryOk) {
-    log('warn', 'Skipping dispatch — discovery failed, stale data risk');
-    return;
-  }
 
-  // 5. Process pending dispatches — auto-spawn agents
+  // 3. Process pending dispatches before discovery. discoverWork() runs
+  // pre-dispatch LLM validation for newly found work; a slow validator must not
+  // starve runnable entries that are already durable in dispatch.pending.
   const dispatch = getDispatch();
   const activeCount = (dispatch.active || []).length;
   const maxConcurrent = resolveMaxConcurrent(config);
@@ -8017,6 +7995,31 @@ async function tickInner() {
     if (_isTickStale(myGeneration)) return;
     mutateDispatch((dp) => { dp.pending = postDispatch.pending; return dp; });
   }
+
+  // 4. Discover new work from sources. Newly discovered dispatches are eligible
+  // for the next tick; existing pending work has already had its spawn chance.
+  // Cap temp-agent creation at (maxConcurrent - activeCount) for this tick so
+  // temps count against maxConcurrent like named agents. Without this, a mass
+  // discovery pass (e.g. 20 PR build failures) would stamp a fresh temp agent
+  // onto every pending item regardless of concurrency cap, and those temps
+  // would spawn over subsequent ticks — overwhelming the OS and mass-orphaning
+  // the dispatches (#1209).
+  {
+    const dispatchPre = getDispatch();
+    const activeCountPre = (dispatchPre.active || []).length;
+    const maxC = resolveMaxConcurrent(config);
+    setTempBudget(Math.max(0, maxC - activeCountPre));
+  }
+  let discoveryOk = true;
+  try { await discoverWork(config); } catch (e) { log('warn', 'discoverWork: ' + e.message); discoveryOk = false; }
+  if (_isTickStale(myGeneration)) return;
+
+  // 5. Update snapshot
+  safe('updateSnapshot', () => updateSnapshot(config));
+
+  if (!discoveryOk) {
+    log('warn', 'Discovery failed after pending dispatch pass — new work may be stale until next tick');
+  }
 }
 
 // ─── Exports (for engine/cli.js and other modules) ──────────────────────────

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.2134",
+  "version": "0.1.2136",
   "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"