@yemi33/minions 0.1.927 → 0.1.929

package/CHANGELOG.md

@@ -1,6 +1,6 @@
 # Changelog
 
-## 0.1.927 (2026-04-13)
+## 0.1.929 (2026-04-14)
 
 ### Features
 -  doc-chat abort kills LLM process + queued messages auto-process
@@ -20,6 +20,8 @@
 - add red dot notification on CC tab when response completes (#934) (#946)
 
 ### Fixes
+- steering kill on no-session re-queues dispatch instead of erroring (#1015)
+- inject cached ADO token into spawned agents (#998) (#1012)
 -  qaAbort race + skip kill on completed doc-chat
 -  doc-chat Stop button actually works — release server guard on disconnect
 -  CC streaming auto-retries fresh session when resume fails
@@ -38,8 +40,6 @@
 -  CC queued messages sent to wrong tab after tab switch
 - pass sysPromptPath instead of steerPromptPath as system prompt in steering resume (#962)
 -  PRD item display dispatched/in-progress status conflation (closes #950) (#955)
-- remove KB watchdog — checkpoint never written, restore never worked
-- make KB sweep endpoint async with status polling (#933)
 
 ### Other
 - test(preflight): add unit tests for findClaudeBinary, runPreflight, printPreflight, checkOrExit (#953)

package/engine.js

@@ -855,6 +855,13 @@ async function spawnAgent(dispatchItem, config) {
   // Spawn the claude process
   const childEnv = shared.cleanChildEnv();
 
+  // Inject cached ADO token so agents skip re-authentication (#998)
+  // getAdoToken() returns cached token (30-min TTL) or null — never blocks on browser auth
+  try {
+    const adoToken = await getAdoToken();
+    if (adoToken) childEnv.MINIONS_ADO_TOKEN = adoToken;
+  } catch { /* non-fatal — agent can still authenticate on its own */ }
+
   // Spawn via wrapper script — node directly (no bash intermediary)
   // spawn-agent.js handles CLAUDECODE env cleanup and claude binary resolution
   const spawnScript = path.join(ENGINE_DIR, 'spawn-agent.js');
@@ -1023,6 +1030,11 @@ async function spawnAgent(dispatchItem, config) {
 
       const spawnScript = path.join(ENGINE_DIR, 'spawn-agent.js');
       const childEnv = shared.cleanChildEnv();
+      // Inject cached ADO token for steering session too (#998)
+      try {
+        const adoToken = await getAdoToken();
+        if (adoToken) childEnv.MINIONS_ADO_TOKEN = adoToken;
+      } catch { /* non-fatal */ }
       let resumeProc;
       try {
         resumeProc = runFile(process.execPath, [spawnScript, steerPromptPath, sysPromptPath, ...resumeArgs], {
@@ -1098,6 +1110,34 @@ async function spawnAgent(dispatchItem, config) {
       return;
     }
 
+    // Check if this was a no-session steering kill (#1014) — re-queue for retry instead of erroring.
+    // timeout.js sets _steeringNoSession when it kills an agent that hasn't established a sessionId.
+    // The steering message is already saved to inbox, so re-queuing lets the engine retry and the
+    // agent picks up the message on the next dispatch.
+    if (procInfo?._steeringNoSession) {
+      log('info', `Steering no-session: re-queue ${agentId} (${id}) — dispatch moved back to pending`);
+      activeProcesses.delete(id);
+      realActivityMap.delete(id);
+      // Move dispatch item from active back to pending so engine retries
+      mutateDispatch((dp) => {
+        const idx = dp.active.findIndex(d => d.id === id);
+        if (idx >= 0) {
+          const item = dp.active.splice(idx, 1)[0];
+          delete item.started_at;
+          delete item.workerState;
+          delete item.workerStateAt;
+          delete item.workerStateDetail;
+          dp.pending.push(item);
+        }
+        return dp;
+      });
+      // Cleanup temp files
+      try { fs.unlinkSync(sysPromptPath); } catch { /* cleanup */ }
+      try { fs.unlinkSync(promptPath); } catch { /* cleanup */ }
+      try { fs.unlinkSync(promptPath.replace(/prompt-/, 'pid-').replace(/\.md$/, '.pid')); } catch { /* cleanup */ }
+      return;
+    }
+
     activeProcesses.delete(id);
     realActivityMap.delete(id); // Clean up real activity tracking
 
@@ -1405,7 +1445,7 @@ function reconcileItemsWithPrs(items, allPrs, { onlyIds } = {}) {
 // ─── Inbox Consolidation (extracted to engine/consolidation.js) ──────────────
 
 const { consolidateInbox } = require('./engine/consolidation');
-const { pollPrStatus, pollPrHumanComments, reconcilePrs, checkLiveReviewStatus: adoCheckLiveReview, needsAdoPollRetry } = require('./engine/ado');
+const { pollPrStatus, pollPrHumanComments, reconcilePrs, checkLiveReviewStatus: adoCheckLiveReview, needsAdoPollRetry, getAdoToken } = require('./engine/ado');
 const { pollPrStatus: ghPollPrStatus, pollPrHumanComments: ghPollPrHumanComments, reconcilePrs: ghReconcilePrs, checkLiveReviewStatus: ghCheckLiveReview } = require('./engine/github');
 
 // ─── State Snapshot ─────────────────────────────────────────────────────────

package/package.json

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