@yemi33/minions 0.1.457 → 0.1.459

package/CHANGELOG.md

@@ -1,8 +1,14 @@
 # Changelog
 
-## 0.1.457 (2026-04-07)
+## 0.1.459 (2026-04-07)
 
 ### Fixes
+-  steering no longer causes pending/active flip
+
+## 0.1.458 (2026-04-07)
+
+### Fixes
+-  steering resume failure no longer burns retry slots or flips status
 -  shared-branch worktree creates branch from main if not found
 -  pre-dispatch live vote check prevents reviewing approved PRs
 

package/engine/timeout.js

@@ -84,6 +84,7 @@ function checkSteering(config) {
     // Store steering context for re-spawn on close
     info._steeringMessage = message;
     info._steeringSessionId = sessionId;
+    info._steeringAt = Date.now(); // prevent timeout checker from treating this as orphaned
   }
 }
 
@@ -203,6 +204,10 @@ function checkTimeouts(config) {
 
     const effectiveTimeout = isBlocking ? blockingTimeout : heartbeatTimeout;
 
+    // Skip recently-steered agents — they're being killed and re-spawned
+    const procInfo = activeProcesses.get(item.id);
+    if (procInfo?._steeringAt && Date.now() - procInfo._steeringAt < 60000) continue;
+
     if (!hasProcess && silentMs > effectiveTimeout && Date.now() > engineRestartGraceUntil) {
       // No tracked process AND no recent output past effective timeout AND grace period expired → orphaned
       log('warn', `Orphan detected: ${item.agent} (${item.id}) — no process tracked, silent for ${silentSec}s${isBlocking ? ' (blocking timeout exceeded)' : ''}`);
@@ -242,6 +247,9 @@ function checkTimeouts(config) {
         if (item.status !== WI_STATUS.DISPATCHED) continue;
         // Never revert completed items
         if (item.completedAt || item.status === WI_STATUS.DONE) continue;
+        // Skip recently-steered agents (being killed and re-spawned)
+        const steerInfo = activeProcesses.get(item.id) || [...activeProcesses.values()].find(p => p.agentId && item.dispatched_to === p.agentId);
+        if (steerInfo?._steeringAt && Date.now() - steerInfo._steeringAt < 60000) continue;
         // Check if any active dispatch references this item
         const projectNames = getProjects(config).map(p => p.name);
         const possibleKeys = [

package/engine.js

@@ -618,11 +618,23 @@ function spawnAgent(dispatchItem, config) {
       });
 
       // Re-wire close handler for the resumed process
-      resumeProc.on('close', onAgentClose);
+      resumeProc.on('close', (resumeCode) => {
+        if (resumeCode !== 0) {
+          // Resume failed — don't burn a retry slot. Complete the dispatch as success
+          // (the original work was already done up to the kill point) and let the
+          // work item be re-discovered on the next tick if still pending.
+          log('warn', `Steering resume for ${agentId} exited with code ${resumeCode} — completing dispatch without error`);
+          activeProcesses.delete(id);
+          completeDispatch(id, DISPATCH_RESULT.SUCCESS, 'Steering resume failed but original work completed', '', { processWorkItemFailure: false });
+          return;
+        }
+        // Successful resume — run normal close handler
+        onAgentClose(resumeCode);
+      });
       resumeProc.on('error', (err) => {
-        log('error', `Steering re-spawn failed for ${agentId}: ${err.message}`);
+        log('warn', `Steering re-spawn error for ${agentId}: ${err.message}`);
         activeProcesses.delete(id);
-        completeDispatch(id, DISPATCH_RESULT.ERROR, `Steering re-spawn error: ${err.message}`);
+        completeDispatch(id, DISPATCH_RESULT.SUCCESS, 'Steering re-spawn error but original work completed', '', { processWorkItemFailure: false });
       });
 
       // Don't run completion hooks — agent is still working

package/package.json

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