@yemi33/minions 0.1.1715 → 0.1.1717

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## 0.1.1717 (2026-05-04)
+
+### Features
+- Harden update init timeout (#2059)
+- harden claude auth classifier (#2058)
+- add CC post-dispatch stop rule (#2057)
+
 ## 0.1.1715 (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-04T18:47:12.870Z"
+  "cachedAt": "2026-05-04T22:25:10.942Z"
 }

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/package.json

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

package/prompts/cc-system.md

@@ -90,6 +90,7 @@ Core action types:
   workTypes: `explore` (research/report only, NO PR), `ask` (answer/report, NO PR), `implement` (new code, PR REQUIRED), `fix` (standalone bug fix creates a PR; include `pr` when fixing review comments/build failures on an existing PR), `review` (code review, NO PR), `test` (tests, PR if new), `verify` (merge/build/maintenance, NO PR)
   If the user wants a design/architecture artifact committed through a PR, dispatch `implement` or `docs` rather than `explore`.
   When the user names a specific agent ("assign this to lambert"), put exactly that one name in `agents` (e.g. `"agents": ["lambert"]`). A single-agent assignment is hard-pinned by the server — it will queue for that agent only and skip the routing table. If the user explicitly asks for fan-out/all agents, set `scope: "fan-out"`.
+  After emitting a dispatch, fix, or implement action, return immediately; do not poll, monitor, watch, wait, or check until completion, and do not add follow-up status actions. Only create a watch, monitor, poll, or periodically check when the human explicitly asks for monitoring, watching, periodic checks, or notification on completion.
 - **build-and-test**: pr, project (optional), agent (optional) — Run the build-and-test playbook against a PR. The agent will checkout the PR branch, run the project's build/test commands, and report results. Use when the user asks to "run tests on PR X" or "build PR X" or after a fix to verify nothing regressed.
   Example: user says "run build and test on PR 1834" → `{"type":"build-and-test","pr":"1834"}`
 - **note**: title, content — save to inbox