@yemi33/minions 0.1.409 → 0.1.410

package/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## 0.1.410 (2026-04-06)
+
+### Fixes
+-  steering messages retry instead of being silently dropped
+
 ## 0.1.409 (2026-04-06)
 
 ### Fixes

package/engine/cli.js

@@ -149,8 +149,8 @@ const commands = {
         if (agentPid && agentPid > 0) {
           try {
             if (process.platform === 'win32') {
-              const out = exec(`tasklist /FI "PID eq ${agentPid}" /NH`, { encoding: 'utf8', timeout: 3000 });
-              if (!out.includes(String(agentPid))) agentPid = null;
+              const out = exec(`tasklist /FI "PID eq ${agentPid}" /NH`, { encoding: 'utf8', timeout: 3000 }).trim();
+              if (!new RegExp(`\\b${agentPid}\\b`).test(out)) agentPid = null;
             } else {
               process.kill(agentPid, 0);
             }
@@ -158,7 +158,13 @@ const commands = {
         }
 
         if (agentPid) {
-          e.activeProcesses.set(item.id, { proc: { pid: agentPid > 0 ? agentPid : null }, agentId, startedAt: item.created_at, reattached: true });
+          // Load sessionId from session.json for steering support
+          let sessionId = null;
+          try {
+            const sj = safeJson(path.join(AGENTS_DIR, agentId, 'session.json'));
+            if (sj?.sessionId) sessionId = sj.sessionId;
+          } catch {}
+          e.activeProcesses.set(item.id, { proc: { pid: agentPid > 0 ? agentPid : null }, agentId, startedAt: item.created_at, reattached: true, sessionId });
           // Sync work item status to dispatched — direct file write to avoid lifecycle lazy init issues
           if (item.meta?.item?.id && item.meta?.project?.localPath) {
             try {

package/engine/timeout.js

@@ -60,16 +60,24 @@ function checkSteering(config) {
     const steerPath = path.join(AGENTS_DIR, info.agentId, 'steer.md');
     if (!fs.existsSync(steerPath)) continue;
 
-    const message = safeRead(steerPath);
-    try { fs.unlinkSync(steerPath); } catch { /* cleanup */ }
-    if (!message) continue;
-
     const sessionId = info.sessionId;
     if (!sessionId) {
-      log('warn', `Steering: no sessionId for ${info.agentId} — cannot resume. Message dropped.`);
+      // No sessionId yet — check stale (>5 min means it'll never arrive)
+      try {
+        const age = Date.now() - fs.statSync(steerPath).mtimeMs;
+        if (age > 300000) {
+          log('warn', `Steering: no sessionId for ${info.agentId} after 5m — deleting stale message`);
+          try { fs.unlinkSync(steerPath); } catch {}
+        }
+      } catch {}
+      // Leave steer.md in place — retry next tick when sessionId may be available
       continue;
     }
 
+    const message = safeRead(steerPath);
+    try { fs.unlinkSync(steerPath); } catch { /* cleanup */ }
+    if (!message) continue;
+
     log('info', `Steering: killing ${info.agentId} (${id}) for session resume with human message`);
 
     // Kill current process

package/engine.js

@@ -696,7 +696,13 @@ function spawnAgent(dispatchItem, config) {
   }, 3000);
 
   // Track process — even if PID isn't available yet (async on Windows)
-  activeProcesses.set(id, { proc, agentId, startedAt });
+  // Pre-load sessionId from session.json so steering works immediately on resume
+  let initialSessionId = null;
+  try {
+    const sj = safeJson(path.join(AGENTS_DIR, agentId, 'session.json'));
+    if (sj?.sessionId && sj.dispatchId === id) initialSessionId = sj.sessionId;
+  } catch {}
+  activeProcesses.set(id, { proc, agentId, startedAt, sessionId: initialSessionId });
 
   // Log PID and persist to registry
   if (proc.pid) {

package/package.json

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