@yemi33/minions 0.1.1716 → 0.1.1718

package/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## 0.1.1718 (2026-05-04)
+
+### Features
+- prevent deferred steering crash (#2060)
+
 ## 0.1.1716 (2026-05-04)
 
 ### Features

package/bin/minions.js

@@ -25,7 +25,7 @@
 const fs = require('fs');
 const path = require('path');
 const os = require('os');
-const { spawn, execSync } = require('child_process');
+const { spawn, spawnSync, execSync } = require('child_process');
 
 const PKG_ROOT = path.resolve(__dirname, '..');
 
@@ -205,6 +205,8 @@ let force = rest.includes('--force');
 const skipScan = rest.includes('--skip-scan');
 const skipStart = rest.includes('--skip-start') || rest.includes('--no-start');
 const MINIONS_HOME = resolveMinionsHome(cmd === 'init');
+const POST_UPDATE_INIT_TIMEOUT_MS = 120000;
+const POST_UPDATE_RESTART_TIMEOUT_MS = 60000;
 
 function isSubpath(parent, child) {
   const rel = path.relative(path.resolve(parent), path.resolve(child));
@@ -432,6 +434,70 @@ function showChangelog(fromVersion) {
   console.log(`    cat ${changelogPath}\n`);
 }
 
+function writeCommandOutput(stream, output) {
+  if (!output) return;
+  stream.write(Buffer.isBuffer(output) ? output.toString('utf8') : String(output));
+}
+
+function formatPackageCliCommand(args) {
+  const initScript = path.join(PKG_ROOT, 'bin', 'minions.js');
+  return `node "${initScript}" ${args.join(' ')}`;
+}
+
+function runPackageCli(args, timeout) {
+  const initScript = path.join(PKG_ROOT, 'bin', 'minions.js');
+  const result = spawnSync(process.execPath, [initScript, ...args], {
+    cwd: process.cwd(),
+    env: { ...process.env, MINIONS_HOME },
+    encoding: 'utf8',
+    timeout,
+    windowsHide: true,
+  });
+
+  writeCommandOutput(process.stdout, result.stdout);
+  writeCommandOutput(process.stderr, result.stderr);
+  return result;
+}
+
+function runPostUpdateInit() {
+  const args = ['init', '--force', '--skip-start'];
+  const result = runPackageCli(args, POST_UPDATE_INIT_TIMEOUT_MS);
+
+  if (!result.error && result.status === 0) return;
+
+  const timedOut = result.error && result.error.code === 'ETIMEDOUT';
+  if (timedOut) {
+    console.error(`\n  ERROR: Post-update initialization timed out after ${POST_UPDATE_INIT_TIMEOUT_MS / 1000}s.`);
+  } else {
+    const detail = result.error ? result.error.message : `exit code ${result.status}`;
+    console.error(`\n  ERROR: Post-update initialization failed (${detail}).`);
+  }
+  console.error('  The npm package update completed, but runtime files were not fully synchronized.');
+  console.error('  After npm finishes settling, run:');
+  console.error(`    ${formatPackageCliCommand(args)}`);
+  console.error(`    ${formatPackageCliCommand(['restart'])}\n`);
+  process.exit(1);
+}
+
+function runPostUpdateRestart() {
+  const args = ['restart'];
+  const result = runPackageCli(args, POST_UPDATE_RESTART_TIMEOUT_MS);
+
+  if (!result.error && result.status === 0) return;
+
+  const timedOut = result.error && result.error.code === 'ETIMEDOUT';
+  if (timedOut) {
+    console.error(`\n  ERROR: Post-update restart timed out after ${POST_UPDATE_RESTART_TIMEOUT_MS / 1000}s.`);
+  } else {
+    const detail = result.error ? result.error.message : `exit code ${result.status}`;
+    console.error(`\n  ERROR: Post-update restart failed (${detail}).`);
+  }
+  console.error('  Runtime files were synchronized, but the engine/dashboard were not restarted.');
+  console.error('  Run this command to finish the update without using the Windows command shim:');
+  console.error(`    ${formatPackageCliCommand(args)}\n`);
+  process.exit(1);
+}
+
 // ─── Version command ────────────────────────────────────────────────────────
 
 function showVersion() {
@@ -544,11 +610,13 @@ if (!cmd || cmd === 'help' || cmd === '--help' || cmd === '-h') {
       console.error('  npm update failed:', e.message);
       process.exit(1);
     }
-    execSync('minions init --force --skip-start', { stdio: 'inherit', timeout: 120000 });
+    // Equivalent to `minions init --force --skip-start`, but avoids recursing through
+    // the global shim while npm is still settling the updated install.
+    runPostUpdateInit();
   }
   // Restart engine + dashboard so they pick up the new code
   console.log('\n  Restarting engine and dashboard...\n');
-  execSync('minions restart', { stdio: 'inherit', timeout: 60000 });
+  runPostUpdateRestart();
 } else if (cmd === 'version' || cmd === '--version' || cmd === '-v') {
   showVersion();
 } else if (cmd === 'add' || cmd === 'remove' || cmd === 'list' || cmd === 'scan') {

package/engine/copilot-models.json

@@ -1,5 +1,5 @@
 {
   "runtime": "copilot",
   "models": null,
-  "cachedAt": "2026-05-04T22:24:13.376Z"
+  "cachedAt": "2026-05-04T23:02:50.359Z"
 }

package/engine/runtimes/claude.js

@@ -456,7 +456,9 @@ function parseError(rawOutput) {
   if (!text) return { message: '', code: null, retriable: true };
   const lower = text.toLowerCase();
 
-  if (/invalid api key|api key.*invalid|authentication.*fail|unauthorized|401|403 forbidden|please.*log.*in|claude\.ai\/login/i.test(text)) {
+  const hasExplicitAuthFailure = /invalid api key|api key.*invalid|authentication.*fail|\bunauthorized\b|please.*log.*in|claude\.ai\/login/i.test(text);
+  const hasAuthStatusCode = /\b(?:http(?:\/\d(?:\.\d)?)?|status(?:\s+code)?|statuscode|response(?:\s+status)?|api(?:\s+(?:error|response|status))?)\s*[:=]?\s*(?:401|403)\b|\b(?:401\s+unauthorized|403\s+forbidden)\b/i.test(text);
+  if (hasExplicitAuthFailure || hasAuthStatusCode) {
     return { message: 'Claude authentication failed', code: 'auth-failure', retriable: false };
   }
   if (/prompt is too long|context window|context.*length.*exceeded|token limit|conversation.*too long/i.test(text)) {

package/engine.js

@@ -1400,9 +1400,9 @@ async function spawnAgent(dispatchItem, config) {
     const hardContractFail = completionContractFailure?.severity === 'hard'
       || completionContractFailure?.nonTerminal === true;
     const effectiveResult = hardContractFail ? DISPATCH_RESULT.ERROR : (((code === 0 && !agentReportedFailure) || autoRecovered) ? DISPATCH_RESULT.SUCCESS : DISPATCH_RESULT.ERROR);
-    const completionReportPath = structuredCompletion?._path || dispatchItem.meta?.completionReportPath || shared.dispatchCompletionReportPath(id);
+    const finalCompletionReportPath = structuredCompletion?._path || dispatchItem.meta?.completionReportPath || shared.dispatchCompletionReportPath(id);
     const completionOpts = {
-      ...(completionReportPath ? { completionReportPath } : {}),
+      ...(finalCompletionReportPath ? { completionReportPath: finalCompletionReportPath } : {}),
       ...(structuredCompletion ? { structuredCompletion } : {}),
     };
     const completeOpts = hardContractFail

package/package.json

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