@yemi33/minions 0.1.408 → 0.1.410

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 0.1.410 (2026-04-06)
+
+### Fixes
+-  steering messages retry instead of being silently dropped
+
+## 0.1.409 (2026-04-06)
+
+### Fixes
+-  npm version check — use npm.cmd on Windows, log errors, expose in API
+
 ## 0.1.408 (2026-04-06)
 
 ### Fixes

package/dashboard.js

@@ -181,15 +181,18 @@ async function checkNpmVersion() {
   try {
     // Use npm view — respects user's .npmrc proxy/registry config (unlike raw https.get)
     const { execFile } = require('child_process');
+    // On Windows, detached processes may not have npm on PATH — use npm.cmd explicitly
+    const npmCmd = process.platform === 'win32' ? 'npm.cmd' : 'npm';
     const version = await new Promise((resolve, reject) => {
-      execFile('npm', ['view', PKG_NAME, 'version'], { timeout: 15000, windowsHide: true }, (err, stdout) => {
+      execFile(npmCmd, ['view', PKG_NAME, 'version'], { timeout: 15000, windowsHide: true, env: process.env }, (err, stdout) => {
         if (err) reject(err); else resolve((stdout || '').trim());
       });
     });
     _npmVersionCache = { latest: version || null, checkedAt: new Date().toISOString() };
     _npmVersionCacheTs = now;
-  } catch {
-    _npmVersionCache = _npmVersionCache || { latest: null, checkedAt: null, error: 'check failed' };
+  } catch (e) {
+    console.error('[version-check] npm view failed:', e.message?.split('\n')[0]);
+    _npmVersionCache = _npmVersionCache || { latest: null, checkedAt: null, error: e.message?.split('\n')[0] || 'check failed' };
   }
   return _npmVersionCache;
 }
@@ -338,6 +341,7 @@ function getStatus() {
         stale: engineStale || dashboardStale,
         latest: _npmVersionCache?.latest || null,
         updateAvailable: !!(diskVersion && _npmVersionCache?.latest && _npmVersionCache.latest !== diskVersion && _compareVersions(_npmVersionCache.latest, diskVersion) > 0),
+        _npmCheckError: _npmVersionCache?.error || null,
       };
     })(),
     timestamp: new Date().toISOString(),

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.408",
+  "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"