@yemi33/minions 0.1.930 → 0.1.932

package/CHANGELOG.md

@@ -1,6 +1,6 @@
 # Changelog
 
-## 0.1.930 (2026-04-14)
+## 0.1.932 (2026-04-14)
 
 ### Features
 -  add adoPollEnabled/ghPollEnabled engine settings
@@ -21,6 +21,10 @@
 - add red dot notification on CC tab when response completes (#934) (#946)
 
 ### Fixes
+- prevent _consolidationInFlight race from stale force-reset timeout (#1023)
+- stop perpetual ADO poll retry when token unavailable (#1021)
+- await discoverPipelineWork instead of fire-and-forget .catch (#1020)
+- add unhandledRejection/uncaughtException handlers to engine process (#1019)
 - show doc-chat badges on Notes and KB items (#1016)
 - steering kill on no-session re-queues dispatch instead of erroring (#1015)
 - inject cached ADO token into spawned agents (#998) (#1012)
@@ -37,10 +41,6 @@
 - PRD item status stuck at dispatched when fix completes (#989)
 - dep merge ancestor pruning + pre-flight simulation (#958) (#979)
 - add MAX_TURNS failure class and fix enum count test (#983)
-- prevent Create Plan from meeting saving doc-chat context bleed (#980)
-- plan completion incorrectly overrides awaiting-approval status (#970) (#978)
--  CC queued messages sent to wrong tab after tab switch
-- pass sysPromptPath instead of steerPromptPath as system prompt in steering resume (#962)
 
 ### Other
 - test(preflight): add unit tests for findClaudeBinary, runPreflight, printPreflight, checkOrExit (#953)

package/engine/ado.js

@@ -252,7 +252,8 @@ async function pollPrStatus(config) {
   const token = await getAdoToken();
   if (!token) {
     log('warn', 'Skipping PR status poll — no ADO token available');
-    _adoPollHadAuthFailure = true; // trigger retry on next tick
+    // Don't set _adoPollHadAuthFailure — getAdoToken() has its own 10-min backoff,
+    // and setting the flag would hammer pollPrStatus() every tick with no useful work.
     return;
   }
 

package/engine/cli.js

@@ -493,6 +493,23 @@ const commands = {
 
     process.on('SIGTERM', () => gracefulShutdown('SIGTERM'));
     process.on('SIGINT', () => gracefulShutdown('SIGINT'));
+
+    // Crash handlers — log the error and exit cleanly so the process is detectable as crashed
+    process.on('unhandledRejection', (reason) => {
+      const msg = reason instanceof Error ? reason.stack || reason.message : String(reason);
+      console.error(`[FATAL] Unhandled promise rejection: ${msg}`);
+      try { shared.log('fatal', `Unhandled promise rejection: ${msg}`); } catch { /* best effort */ }
+      try { shared.flushLogs(); } catch { /* best effort */ }
+      process.exit(1);
+    });
+
+    process.on('uncaughtException', (err) => {
+      const msg = err instanceof Error ? err.stack || err.message : String(err);
+      console.error(`[FATAL] Uncaught exception: ${msg}`);
+      try { shared.log('fatal', `Uncaught exception: ${msg}`); } catch { /* best effort */ }
+      try { shared.flushLogs(); } catch { /* best effort */ }
+      process.exit(1);
+    });
   },
 
   stop() {

package/engine/consolidation.js

@@ -18,6 +18,7 @@ const { getInboxFiles, getNotes, INBOX_DIR, ENGINE_DIR, MINIONS_DIR,
 // Track in-flight LLM consolidation to prevent concurrent runs
 let _consolidationInFlight = false;
 let _consolidationStartedAt = 0;
+let _forceResetTimeout = null; // force-reset handle; cancelled by _clearProcessingState
 const _processingFiles = new Set(); // files currently being consolidated (race guard)
 
 function consolidateInbox(config) {
@@ -113,6 +114,9 @@ Use today's date: ${dateStamp()}`;
 
 function consolidateWithLLM(items, existingNotes, files, config) {
 
+  // Cancel any stale force-reset from a prior run before starting fresh
+  clearTimeout(_forceResetTimeout);
+  _forceResetTimeout = null;
   _consolidationInFlight = true;
   _consolidationStartedAt = Date.now();
   for (const f of files) _processingFiles.add(f);
@@ -180,18 +184,21 @@ function consolidateWithLLM(items, existingNotes, files, config) {
   const timeout = setTimeout(() => {
     log('warn', 'LLM consolidation timed out after 3m — killing and falling back to regex');
     shared.killGracefully(proc, 10000);
-    setTimeout(() => {
-      if (_consolidationInFlight) {
-        _consolidationInFlight = false;
-        _processingFiles.clear();
-        log('warn', 'Consolidation flag force-reset after SIGKILL');
-      }
+    _forceResetTimeout = setTimeout(() => {
+      if (!_cleared) log('warn', 'Consolidation flag force-reset after SIGKILL');
+      _clearProcessingState();
     }, 10000);
   }, 180000);
 
+  let _cleared = false; // idempotency guard — both 'error' and 'close' can fire for the same process
   function _clearProcessingState() {
+    if (_cleared) return;
+    _cleared = true;
+    clearTimeout(_forceResetTimeout);
+    _forceResetTimeout = null;
     for (const f of files) _processingFiles.delete(f);
     _consolidationInFlight = false;
+    _consolidationStartedAt = 0;
   }
 
   proc.on('close', (code) => {

package/engine.js

@@ -2990,7 +2990,7 @@ async function discoverWork(config) {
   // Pipeline orchestration — check stage completions and start ready stages
   try {
     const { discoverPipelineWork } = require('./engine/pipeline');
-    discoverPipelineWork(config).catch(e => log('warn', 'discover pipeline work: ' + e.message));
+    await discoverPipelineWork(config);
   } catch (e) { log('warn', 'discover pipeline work: ' + e.message); }
 
   // Periodic plan completion sweep — catch PRDs that completed while engine was down

package/package.json

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