@yemi33/minions 0.1.1671 → 0.1.1673

package/.claudeignore

@@ -0,0 +1,5 @@
+agents/
+engine/tmp/
+engine/contexts/
+prd/*.backup
+plans/archive/

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 0.1.1673 (2026-05-02)
+
+### Fixes
+-  cap Signed Off By column width and wrap multi-reviewer overflow
+
+## 0.1.1672 (2026-05-02)
+
+### Features
+-  fleet model validation + Copilot alias map + simplify pass
+
 ## 0.1.1671 (2026-05-02)
 
 ### Features

package/dashboard/js/render-prs.js

@@ -34,7 +34,7 @@ function prRow(pr) {
     '<td><span class="pr-agent">' + escapeHtml(pr.agent || '—') + '</span></td>' +
     '<td><span class="' + branchClass + '"' + (branchError ? ' title="' + escapeHtml(branchError) + '"' : '') + '>' + escapeHtml(branchLabel) + '</span>' + pendingReasonHtml + '</td>' +
     '<td><span class="pr-badge ' + reviewClass + '"' + (reviewTitle ? ' title="' + escapeHtml(reviewTitle) + '"' : '') + '>' + escapeHtml(reviewLabel) + '</span></td>' +
-    '<td>' + (sq.reviewer && sq.status !== 'waiting' ? '<span class="pr-agent" title="' + escapeHtml(sq.note || '') + '">' + escapeHtml(sq.reviewer) + '</span>' : sq.reviewer && sq.status === 'waiting' ? '<span class="pr-agent" style="color:var(--muted)" title="Vote pending confirmation">' + escapeHtml(sq.reviewer) + '…</span>' : pr.reviewedBy && pr.reviewedBy.length ? '<span class="pr-agent">' + escapeHtml(pr.reviewedBy.join(', ')) + '</span>' : '<span style="color:var(--muted);font-size:11px">—</span>') + '</td>' +
+    '<td class="pr-col-signoff">' + (sq.reviewer && sq.status !== 'waiting' ? '<span class="pr-agent" title="' + escapeHtml(sq.note || '') + '">' + escapeHtml(sq.reviewer) + '</span>' : sq.reviewer && sq.status === 'waiting' ? '<span class="pr-agent" style="color:var(--muted)" title="Vote pending confirmation">' + escapeHtml(sq.reviewer) + '…</span>' : pr.reviewedBy && pr.reviewedBy.length ? '<span class="pr-agent">' + escapeHtml(pr.reviewedBy.join(', ')) + '</span>' : '<span style="color:var(--muted);font-size:11px">—</span>') + '</td>' +
     '<td><span class="pr-badge ' + buildClass + '"' + (buildTitle ? ' title="' + escapeHtml(buildTitle) + '"' : '') + '>' + escapeHtml(buildLabel) + '</span></td>' +
     '<td><span class="pr-badge ' + statusClass + '">' + escapeHtml(statusLabel) + '</span></td>' +
     '<td><span class="pr-date">' + escapeHtml((pr.created || '—').slice(0, 16).replace('T', ' ')) + '</span></td>' +
@@ -44,7 +44,7 @@ function prRow(pr) {
 
 function prTableHtml(rows) {
   return '<div class="pr-table-wrap"><table class="pr-table"><thead><tr>' +
-    '<th>PR</th><th>Title</th><th>Agent</th><th>Branch</th><th>Review</th><th>Signed Off By</th><th>Build</th><th>Status</th><th>Created</th><th></th>' +
+    '<th>PR</th><th>Title</th><th>Agent</th><th>Branch</th><th>Review</th><th class="pr-col-signoff">Signed Off By</th><th>Build</th><th>Status</th><th>Created</th><th></th>' +
     '</tr></thead><tbody>' + rows + '</tbody></table></div>';
 }
 

package/dashboard/styles.css

@@ -213,7 +213,6 @@
   .prd-item-row.st-in-progress { border-left-color: var(--yellow); animation: prdWipPulse 2s infinite; }
   @keyframes prdWipPulse { 0%, 100% { box-shadow: 0 0 0 0 rgba(210,153,34,0); } 50% { box-shadow: 0 0 0 4px rgba(210,153,34,0.2); } }
   .prd-item-row.st-failed { border-left-color: var(--red); }
-  .prd-item-row.st-needs-human-review { border-left-color: var(--orange); }
   .prd-item-row.st-updated { border-left-color: var(--purple); }
   .prd-item-row.st-paused { border-left-color: var(--muted); opacity: 0.5; }
   .prd-item-id { font-family: Consolas, monospace; color: var(--muted); min-width: 36px; font-size: 0.9em; }
@@ -252,6 +251,7 @@
   .pr-table-wrap { overflow-x: auto; }
   .pr-table { width: 100%; border-collapse: collapse; font-size: var(--text-md); table-layout: auto; }
   .pr-table th:last-child, .pr-table td:last-child { width: 36px; min-width: 36px; text-align: center; }
+  .pr-table th.pr-col-signoff, .pr-table td.pr-col-signoff { width: 140px; max-width: 140px; white-space: normal; word-break: break-word; }
   .pr-table th { text-align: left; color: var(--muted); font-weight: 500; font-size: var(--text-base); text-transform: uppercase; letter-spacing: 0.5px; padding: var(--space-4) var(--space-5); border-bottom: 1px solid var(--border); }
   .pr-table td { padding: var(--space-5); border-bottom: 1px solid var(--border); vertical-align: middle; white-space: nowrap; }
   .pr-table tr:last-child td { border-bottom: none; }
@@ -268,7 +268,6 @@
   .pr-badge.active { background: rgba(88,166,255,0.15); color: var(--blue); border: 1px solid var(--blue); }
   .pr-badge.approved { background: rgba(63,185,80,0.15); color: var(--green); border: 1px solid var(--green); }
   .pr-badge.rejected { background: rgba(248,81,73,0.15); color: var(--red); border: 1px solid var(--red); }
-  .pr-badge.needs-review { background: rgba(227,179,65,0.15); color: var(--orange); border: 1px solid var(--orange); }
   .pr-badge.review-escalated { background: rgba(227,179,65,0.22); color: var(--orange); border: 1px dashed var(--orange); font-weight: 700; }
   .pr-badge.merged { background: rgba(188,140,255,0.15); color: var(--purple); border: 1px solid var(--purple); }
   .pr-badge.building { background: rgba(210,153,34,0.15); color: var(--yellow); border: 1px solid var(--yellow); animation: pulse 1.5s infinite; }

package/dashboard.js

@@ -3357,7 +3357,7 @@ const server = http.createServer(async (req, res) => {
   async function _runKbSweepBackground(body, sweepToken) {
     try {
       const { runKbSweep } = require('./engine/kb-sweep');
-      const result = await runKbSweep({ pinnedKeys: body.pinnedKeys });
+      const result = await runKbSweep({ pinnedKeys: body.pinnedKeys, engineConfig: CONFIG.engine });
       global._kbSweepLastResult = result;
       global._kbSweepLastCompletedAt = Date.now();
     } catch (e) {
@@ -5286,7 +5286,10 @@ What would you like to discuss or change? When you're happy, say "approve" and I
 
     const prompt = `Convert this schedule description to a 3-field cron expression (minute hour dayOfWeek, where dayOfWeek is 0=Sun..6=Sat or ranges like 1-5). Return JSON only: {"cron": "...", "description": "..."}. Input: ${text.trim()}`;
     try {
-      const result = await llm.callLLM(prompt, '', { model: 'haiku', maxTurns: 1, timeout: 30000, label: 'schedule-parse', direct: true });
+      const result = await llm.callLLM(prompt, '', {
+        model: 'haiku', maxTurns: 1, timeout: 30000, label: 'schedule-parse', direct: true,
+        engineConfig: CONFIG.engine,
+      });
       const parsed = JSON.parse(result.text.trim());
       if (!parsed.cron) return jsonReply(res, 422, { error: 'Could not parse schedule' });
       return jsonReply(res, 200, { cron: parsed.cron, description: parsed.description || '' });
@@ -5379,6 +5382,17 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       if (!config.agents) config.agents = {};
 
       const _clamped = [];
+      const _engineModelDiscovery = require('./engine/model-discovery');
+      const _engineRuntimes = require('./engine/runtimes');
+      function _resolveModelForRuntime(modelStr, runtimeName) {
+        try {
+          const adapter = _engineRuntimes.resolveRuntime(runtimeName);
+          if (adapter && typeof adapter.resolveModel === 'function') {
+            return adapter.resolveModel(modelStr) || modelStr;
+          }
+        } catch { /* unknown/free-text runtime */ }
+        return modelStr;
+      }
       if (body.engine) {
         const e = body.engine;
         const D = shared.ENGINE_DEFAULTS;
@@ -5440,10 +5454,9 @@ What would you like to discuss or change? When you're happy, say "approve" and I
         // (where gpt-5.5 doesn't actually exist) cascaded into every agent
         // that didn't pin its own model. Reject when the model is known to
         // belong to a different runtime than the one it'll spawn against.
-        const _engineModelDiscovery = require('./engine/model-discovery');
-        const _engineRuntimes = require('./engine/runtimes');
         async function _validateFleetModel(modelStr, resolvedRuntime) {
           if (!modelStr) return null;
+          const runtimeModelStr = _resolveModelForRuntime(modelStr, resolvedRuntime);
           let knownForResolved = null;
           try {
             const list = await _engineModelDiscovery.getRuntimeModels(resolvedRuntime, { config });
@@ -5451,11 +5464,11 @@ What would you like to discuss or change? When you're happy, say "approve" and I
               knownForResolved = new Set(list.models.map(m => m.id || m.name).filter(Boolean));
             }
           } catch { /* unknown runtime */ }
-          if (knownForResolved && !knownForResolved.has(modelStr)) {
+          if (knownForResolved && !knownForResolved.has(runtimeModelStr)) {
             return `not a valid model for runtime "${resolvedRuntime}" (known: ${[...knownForResolved].slice(0, 4).join(', ')}${knownForResolved.size > 4 ? '…' : ''})`;
           }
           if (!knownForResolved) {
-            // Free-text runtime (Claude). Reject only if model belongs to a different runtime's published list.
+            // Free-text runtime (Claude). Reject only if the raw model belongs to a different runtime's published list.
             for (const rt of _engineRuntimes.listRuntimes()) {
               if (rt === resolvedRuntime) continue;
               try {
@@ -5548,13 +5561,12 @@ What would you like to discuss or change? When you're happy, say "approve" and I
         // claude+gpt-* / copilot+claude-* combinations before they crash a
         // dispatch (see #model-validation: a stray engine.defaultModel='gpt-5.5'
         // pinned every Claude agent into a 404 spawn loop).
-        const _modelDiscovery = require('./engine/model-discovery');
         const _runtimeModelsCache = new Map(); // runtimeName → Set<modelId> (or null when unknown / Claude)
         async function _modelsFor(runtimeName) {
           if (_runtimeModelsCache.has(runtimeName)) return _runtimeModelsCache.get(runtimeName);
           let set = null;
           try {
-            const list = await _modelDiscovery.getRuntimeModels(runtimeName, { config });
+            const list = await _engineModelDiscovery.getRuntimeModels(runtimeName, { config });
             if (Array.isArray(list?.models) && list.models.length > 0) {
               set = new Set(list.models.map(m => m.id || m.name).filter(Boolean));
             }
@@ -5562,11 +5574,11 @@ What would you like to discuss or change? When you're happy, say "approve" and I
           _runtimeModelsCache.set(runtimeName, set);
           return set;
         }
-        // Returns the runtime that "owns" this model, or null if no other
+        // Returns the runtime that "owns" this raw model, or null if no other
         // runtime claims it. Catches "claude + gpt-5.5" by spotting that
         // gpt-5.5 belongs to copilot's list.
         async function _ownerOfModel(modelId) {
-          for (const rt of require('./engine/runtimes').listRuntimes()) {
+          for (const rt of _engineRuntimes.listRuntimes()) {
             const set = await _modelsFor(rt);
             if (set && set.has(modelId)) return rt;
           }
@@ -5594,6 +5606,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
             else {
               const candidate = String(updates.model);
               const resolvedCli = config.agents[id].cli || config.engine.defaultCli || 'claude';
+              const runtimeModelStr = _resolveModelForRuntime(candidate, resolvedCli);
               const knownModels = await _modelsFor(resolvedCli);
               // Two validation paths:
               //   1. If the runtime publishes a model list, enforce membership.
@@ -5601,7 +5614,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
               //      model belongs to a DIFFERENT runtime's list — that's how
               //      we catch claude+gpt-5.5 (gpt-5.5 is in Copilot's list).
               let rejection = null;
-              if (knownModels && !knownModels.has(candidate)) {
+              if (knownModels && !knownModels.has(runtimeModelStr)) {
                 rejection = `not a valid model for runtime "${resolvedCli}" (known: ${[...knownModels].slice(0, 4).join(', ')}${knownModels.size > 4 ? '…' : ''})`;
               } else if (!knownModels) {
                 const owner = await _ownerOfModel(candidate);

package/docs/copilot-cli-schema.md

@@ -20,7 +20,7 @@
 | `capabilities.systemPromptFile` | **`false`** | No `--system-prompt-file` flag exists. Inject system prompt via a `<system>` block prepended to stdin. |
 | `capabilities.effortLevels` | **`true`** | `--effort` accepts `low|medium|high|xhigh` (no `max`). Adapter must map `'max' → 'xhigh'`. |
 | `capabilities.costTracking` | **`false`** | `result.usage` contains `premiumRequests` (count, not USD), no token counts, no cost. |
-| `capabilities.modelShorthands` | **`false`** | Models are full IDs (`claude-sonnet-4.5`, `gpt-5.4`). No `sonnet`/`opus`/`haiku` aliasing on the Copilot side. |
+| `capabilities.modelShorthands` | **`false`** | The Copilot CLI requires full model IDs (`claude-sonnet-4.5`, `gpt-5.4`). Minions may accept internal aliases (`haiku`, `sonnet`, `opus`), but the adapter translates them to Copilot model IDs before invoking the CLI. |
 | `capabilities.budgetCap` | **`false`** | No `--max-budget-usd` flag. |
 | `capabilities.bareMode` | **`false`** | No `--bare`. Closest equivalent is `--no-custom-instructions` (suppresses AGENTS.md only, not all auto-discovery). |
 | `capabilities.fallbackModel` | **`false`** | No `--fallback-model` flag. |
@@ -594,9 +594,11 @@ When implementing `engine/runtimes/copilot.js`:
    **Never** emit `--verbose`.
 4. `buildPrompt()` injects `<system>...</system>\n\n` block when sysprompt is
    non-empty; passthrough otherwise (§2).
-5. `resolveModel()` is verbatim passthrough; emit a one-time `console.warn`
-   when input is `'sonnet' | 'opus' | 'haiku'` (Claude shorthand the user
-   probably meant to set on the Claude adapter).
+5. `resolveModel()` translates Minions internal aliases before the CLI boundary:
+   `'haiku'` → `'claude-haiku-4.5'`, `'sonnet'` → `'claude-sonnet-4.5'`, and
+   `'opus'` → `'claude-opus-4.5'`. All other model IDs pass through unchanged.
+   Keep `capabilities.modelShorthands` false because aliases are never passed
+   to the Copilot CLI.
 6. `_mapEffort()` private helper does `'max' → 'xhigh'`; pass through otherwise.
 7. `parseOutput(raw)` produces:
    - `text`: concatenation of all `assistant.message.data.content` (multi-turn

package/engine/cleanup.js

@@ -39,6 +39,30 @@ function worktreeDirMatchesBranch(dirLower, branch) {
   return dirLower === branchSlug || dirLower.includes(branchSlug + '-') || dirLower.endsWith('-' + branchSlug);
 }
 
+let _orphanPidProcessNamesCache = null;
+function _orphanPidProcessNames() {
+  if (_orphanPidProcessNamesCache) return _orphanPidProcessNamesCache;
+  const names = new Set(['node']);
+  try {
+    for (const name of require('./runtimes').listRuntimes()) names.add(String(name).toLowerCase());
+    // Copilot can run through the GitHub CLI fallback (`gh copilot`), so allow
+    // gh only when the copilot runtime is registered.
+    if (names.has('copilot')) names.add('gh');
+  } catch {
+    names.add('claude');
+  }
+  _orphanPidProcessNamesCache = names;
+  return names;
+}
+
+function _processNameAllowedForOrphanKill(processText) {
+  const firstLine = String(processText || '').trim().split(/\r?\n/).find(Boolean) || '';
+  const imageName = path.basename(firstLine.trim().split(/\s+/)[0] || '').toLowerCase().replace(/\.exe$/, '');
+  if (!imageName) return false;
+  return _orphanPidProcessNames().has(imageName);
+}
+
+
 /**
  * Kill orphaned processes whose dispatch ID appears in the worktree dir name.
  * Only kills processes NOT in the active dispatch queue — never kills live agents.
@@ -68,18 +92,18 @@ function _killProcessInWorktree(dir, activeProcesses, activeIds) {
       if (isActive) continue; // still active — do not kill
       const pid = parseInt(fs.readFileSync(path.join(tmpDir, f), 'utf8').trim(), 10);
       if (pid > 0) {
-        // Verify the process is actually a node/claude process before killing
+        // Verify the PID still belongs to a Minions runtime process before killing
         try {
           if (process.platform === 'win32') {
             const taskInfo = exec(`tasklist /FI "PID eq ${pid}" /NH`, { encoding: 'utf8', timeout: 3000, windowsHide: true });
             const taskLower = taskInfo.toLowerCase();
-            if (!taskLower.includes('node') && !taskLower.includes('claude')) continue;
+            if (!_processNameAllowedForOrphanKill(taskLower)) continue;
             exec(`taskkill /F /T /PID ${pid}`, { stdio: 'pipe', timeout: 5000, windowsHide: true });
           } else {
-            // Verify it's a node process before killing (prevent recycled PID kill)
+            // Verify the process name before killing (prevent recycled PID kill)
             try {
               const psOut = exec(`ps -p ${pid} -o comm=`, { encoding: 'utf8', timeout: 3000 }).trim();
-              if (!psOut.includes('node') && !psOut.includes('claude')) continue;
+              if (!_processNameAllowedForOrphanKill(psOut)) continue;
             } catch { continue; } // process dead or ps failed
             try { process.kill(-pid, 'SIGKILL'); } catch { process.kill(pid, 'SIGKILL'); }
           }
@@ -331,7 +355,7 @@ function runCleanup(config, verbose = false) {
     } // end worktreeRoots loop
   }
 
-  // 4. Kill zombie claude processes not tracked by the engine
+  // 4. Kill zombie agent processes not tracked by the engine
   // List all node processes, check if any are running spawn-agent.js for our minions
   try {
     const dispatch = getDispatch();

package/engine/cli.js

@@ -157,8 +157,8 @@ function _modelLooksIncompatible(runtime, model) {
     return true; // gpt-*, o3-*, codex, etc. — wrong CLI for these
   }
   if (runtime === 'copilot') {
-    // Copilot accepts the full catalog by ID; only Claude shorthands are wrong.
-    return m === 'sonnet' || m === 'opus' || m === 'haiku';
+    // Copilot adapter maps Minions' family aliases before spawning.
+    return false;
   }
   return false;
 }

package/engine/consolidation.js

@@ -8,11 +8,11 @@ const fs = require('fs');
 const path = require('path');
 const crypto = require('crypto');
 const shared = require('./shared');
-const { safeRead, safeWrite, safeUnlink, runFile, cleanChildEnv,
-  parseStreamJsonOutput, classifyInboxItem, KB_CATEGORIES, log, ts, dateStamp } = shared;
-const { trackEngineUsage } = require('./llm');
+const { safeRead, safeWrite,
+  classifyInboxItem, KB_CATEGORIES, log, ts, dateStamp } = shared;
+const { callLLM, trackEngineUsage } = require('./llm');
 const queries = require('./queries');
-const { getInboxFiles, getNotes, INBOX_DIR, ENGINE_DIR, MINIONS_DIR,
+const { getInboxFiles, getNotes, INBOX_DIR, ENGINE_DIR,
   NOTES_PATH, KNOWLEDGE_DIR, ARCHIVE_DIR } = queries;
 
 // Track in-flight LLM consolidation to prevent concurrent runs
@@ -150,51 +150,18 @@ function consolidateWithLLM(items, existingNotes, files, config) {
   });
 
   const prompt = buildConsolidationPrompt(items, existingNotes, kbPaths);
-
-  const tmpDir = path.join(ENGINE_DIR, 'tmp');
-  if (!fs.existsSync(tmpDir)) fs.mkdirSync(tmpDir, { recursive: true });
-  const promptPath = path.join(tmpDir, 'consolidate-prompt.md');
-  safeWrite(promptPath, prompt);
-
   const sysPrompt = 'You are a concise knowledge manager. Output only markdown. No preamble. No code fences around your output.';
-  const sysPromptPath = path.join(tmpDir, 'consolidate-sysprompt.md');
-  safeWrite(sysPromptPath, sysPrompt);
-
-  const spawnScript = path.join(ENGINE_DIR, 'spawn-agent.js');
-  const args = [
-    '--output-format', 'stream-json',
-    '--max-turns', '1',
-    '--model', 'haiku',
-    '--permission-mode', 'bypassPermissions',
-    '--verbose',
-  ];
-
-  log('info', 'Spawning Haiku for LLM consolidation...');
-
-  const proc = runFile(process.execPath, [spawnScript, promptPath, sysPromptPath, ...args], {
-    cwd: MINIONS_DIR,
-    stdio: ['pipe', 'pipe', 'pipe'],
-    env: cleanChildEnv()
-  });
 
-  let stdout = '';
-  let stderr = '';
-  proc.stdout.on('data', d => { stdout += d.toString(); if (stdout.length > 100000) stdout = stdout.slice(-50000); });
-  proc.stderr.on('data', d => { stderr += d.toString(); if (stderr.length > 50000) stderr = stderr.slice(-25000); });
+  log('info', 'Starting LLM consolidation...');
 
-  const timeout = setTimeout(() => {
-    log('warn', 'LLM consolidation timed out after 3m — killing and falling back to regex');
-    shared.killGracefully(proc, 10000);
-    _forceResetTimeout = setTimeout(() => {
-      if (!_cleared) log('warn', 'Consolidation flag force-reset after SIGKILL');
-      _clearProcessingState();
-    }, 10000);
-  }, 180000);
+  let _cleared = false; // idempotency guard — timeout and promise resolution can race
+  let timeoutHandle = null;
+  let fallbackDone = false;
 
-  let _cleared = false; // idempotency guard — both 'error' and 'close' can fire for the same process
   function _clearProcessingState() {
     if (_cleared) return;
     _cleared = true;
+    clearTimeout(timeoutHandle);
     clearTimeout(_forceResetTimeout);
     _forceResetTimeout = null;
     for (const f of files) _processingFiles.delete(f);
@@ -202,17 +169,43 @@ function consolidateWithLLM(items, existingNotes, files, config) {
     _consolidationStartedAt = 0;
   }
 
-  proc.on('close', (code) => {
-    clearTimeout(timeout);
-    try { safeUnlink(promptPath); } catch (err) { log('warn', `Temp file cleanup failed: ${promptPath} — ${err.message}`); }
-    try { safeUnlink(sysPromptPath); } catch (err) { log('warn', `Temp file cleanup failed: ${sysPromptPath} — ${err.message}`); }
+  function _fallback(message, err) {
+    if (_cleared || fallbackDone) return;
+    fallbackDone = true;
+    if (message) log('warn', message);
+    if (err?.message) log('debug', `LLM error: ${err.message}`);
+    consolidateWithRegex(items, files);
+  }
 
-    const parsed = parseStreamJsonOutput(stdout);
-    const extractedText = parsed.text;
-    trackEngineUsage('consolidation', parsed.usage);
+  const llmCall = callLLM(prompt, sysPrompt, {
+    timeout: 180000,
+    label: 'consolidation',
+    model: 'haiku',
+    maxTurns: 1,
+    direct: true,
+    engineConfig: config.engine,
+  });
 
-    if (code === 0 && (extractedText || stdout).trim().length > 50) {
-      let digest = (extractedText || stdout).trim();
+  timeoutHandle = setTimeout(() => {
+    if (_cleared) return;
+    log('warn', 'LLM consolidation timed out after 3m — aborting and falling back to regex');
+    try { if (llmCall && typeof llmCall.abort === 'function') llmCall.abort(); } catch { /* ignore */ }
+    _forceResetTimeout = setTimeout(() => {
+      if (!_cleared) log('warn', 'Consolidation flag force-reset after timeout');
+      _fallback();
+      _clearProcessingState();
+    }, 10000);
+  }, 180000);
+
+  llmCall.then((result) => {
+    if (_cleared) return;
+    clearTimeout(timeoutHandle);
+    trackEngineUsage('consolidation', result.usage);
+
+    const extractedText = result.text || '';
+    const rawText = result.raw || '';
+    if (result.code === 0 && (extractedText || rawText).trim().length > 50) {
+      let digest = (extractedText || rawText).trim();
       digest = digest.replace(/^\`\`\`\w*\n?/gm, '').replace(/\n?\`\`\`$/gm, '').trim();
 
       if (!digest.startsWith('### ')) {
@@ -220,9 +213,7 @@ function consolidateWithLLM(items, existingNotes, files, config) {
         if (sectionIdx >= 0) {
           digest = digest.slice(sectionIdx);
         } else {
-          log('warn', 'LLM consolidation output missing expected format — falling back to regex');
-          consolidateWithRegex(items, files);
-          _clearProcessingState();
+          _fallback('LLM consolidation output missing expected format — falling back to regex');
           return;
         }
       }
@@ -257,21 +248,15 @@ function consolidateWithLLM(items, existingNotes, files, config) {
       });
       classifyToKnowledgeBase(items);
       archiveInboxFiles(files);
-      log('info', `LLM consolidation complete: ${files.length} notes processed by Haiku`);
+      log('info', `LLM consolidation complete: ${files.length} notes processed`);
     } else {
-      log('warn', `LLM consolidation failed (code=${code}) — falling back to regex`);
-      if (stderr) log('debug', `LLM stderr: ${stderr.slice(0, 500)}`);
-      consolidateWithRegex(items, files);
+      _fallback(`LLM consolidation failed (code=${result.code}) — falling back to regex`, result.stderr ? { message: result.stderr.slice(0, 500) } : null);
     }
-    _clearProcessingState();
-  });
-
-  proc.on('error', (err) => {
-    clearTimeout(timeout);
-    log('warn', `LLM consolidation spawn error: ${err.message} — falling back to regex`);
-    try { safeUnlink(promptPath); } catch (unlinkErr) { log('warn', `Temp file cleanup failed: ${promptPath} — ${unlinkErr.message}`); }
-    try { safeUnlink(sysPromptPath); } catch (unlinkErr) { log('warn', `Temp file cleanup failed: ${sysPromptPath} — ${unlinkErr.message}`); }
-    consolidateWithRegex(items, files);
+  }).catch((err) => {
+    if (_cleared) return;
+    clearTimeout(timeoutHandle);
+    _fallback(`LLM consolidation error: ${err.message} — falling back to regex`, err);
+  }).finally(() => {
     _clearProcessingState();
   });
 }

package/engine/copilot-models.json

@@ -1,5 +1,5 @@
 {
   "runtime": "copilot",
   "models": null,
-  "cachedAt": "2026-05-02T02:37:22.632Z"
+  "cachedAt": "2026-05-02T04:13:14.794Z"
 }

package/engine/kb-sweep.js

@@ -104,7 +104,7 @@ function _hashDedup(manifest, opts = {}) {
 }
 
 /** Batched LLM sweep — finds within-batch dupes, reclassifies, removes stale. */
-async function _llmBatchSweep(manifest, callLLM, trackEngineUsage) {
+async function _llmBatchSweep(manifest, callLLM, trackEngineUsage, opts = {}) {
   const plan = { duplicates: [], reclassify: [], remove: [] };
   const batches = [];
   for (let i = 0; i < manifest.length; i += LLM_BATCH_SIZE) {
@@ -130,7 +130,10 @@ If nothing to do: { "duplicates": [], "reclassify": [], "remove": [] }`;
 
     let result;
     try {
-      result = await callLLM(prompt, 'Output only JSON.', { timeout: 120000, label: 'kb-sweep', model: 'haiku', maxTurns: 1, direct: true });
+      result = await callLLM(prompt, 'Output only JSON.', {
+        timeout: 120000, label: 'kb-sweep', model: 'haiku', maxTurns: 1, direct: true,
+        engineConfig: opts.engineConfig,
+      });
       trackEngineUsage('kb-sweep', result.usage);
     } catch (e) { log('warn', `[kb-sweep] batch ${b + 1} LLM error: ${e.message}`); continue; }
 
@@ -206,6 +209,7 @@ ${body}`;
       try {
         const result = await callLLM(REWRITE_PROMPT(c.entry, c.body), 'Output ONLY the template body.', {
           timeout: 120000, label: 'kb-rewrite', model: 'haiku', maxTurns: 1, direct: true,
+          engineConfig: opts.engineConfig,
         });
         trackEngineUsage('kb-sweep', result.usage);
         let newBody = (result.text || '').trim();
@@ -326,7 +330,7 @@ async function runKbSweep(opts = {}) {
   // 2. LLM batch sweep — within-batch dupes + reclassify + remove stale
   // Only runs against survivors, but we need indices that match the LIST sent to the LLM
   const llmManifest = afterHash;
-  const plan = await _llmBatchSweep(llmManifest, callLLM, trackEngineUsage);
+  const plan = await _llmBatchSweep(llmManifest, callLLM, trackEngineUsage, opts);
   const llmActions = _applyLlmPlan(plan, llmManifest, opts);
   summary.llmDuplicatesArchived = llmActions.merged;
   summary.staleRemoved = llmActions.removed;

package/engine/llm.js

@@ -403,6 +403,12 @@ function _resolveModelFor(callOpts) {
   return undefined;
 }
 
+function _resolveModelForRuntime(runtime, callOpts) {
+  const selected = _resolveModelFor(callOpts || {});
+  if (!runtime || typeof runtime.resolveModel !== 'function') return selected;
+  return runtime.resolveModel(selected);
+}
+
 function _resolveRuntimeFeatureOpts({
   stream, disableBuiltinMcps, suppressAgentsMd, reasoningSummaries, engineConfig,
 } = {}) {
@@ -431,7 +437,7 @@ function callLLM(promptText, sysPromptText, opts = {}) {
   } = opts;
 
   const runtime = _resolveRuntimeFor({ cli: cliOverride, engineConfig });
-  const model = _resolveModelFor({ model: modelOverride, engineConfig });
+  const model = _resolveModelForRuntime(runtime, { model: modelOverride, engineConfig });
   const runtimeFeatureOpts = _resolveRuntimeFeatureOpts({
     stream, disableBuiltinMcps, suppressAgentsMd, reasoningSummaries, engineConfig,
   });
@@ -528,7 +534,7 @@ function callLLMStreaming(promptText, sysPromptText, opts = {}) {
   } = opts;
 
   const runtime = _resolveRuntimeFor({ cli: cliOverride, engineConfig });
-  const model = _resolveModelFor({ model: modelOverride, engineConfig });
+  const model = _resolveModelForRuntime(runtime, { model: modelOverride, engineConfig });
   const runtimeFeatureOpts = _resolveRuntimeFeatureOpts({
     stream, disableBuiltinMcps, suppressAgentsMd, reasoningSummaries, engineConfig,
   });
@@ -619,5 +625,6 @@ module.exports = {
   _resetBinCache,
   _resolveRuntimeFor,
   _resolveModelFor,
+  _resolveModelForRuntime,
   _resolveRuntimeFeatureOpts,
 };

package/engine/pipeline.js

@@ -486,6 +486,7 @@ async function executePlanStage(stage, stageState, run, config) {
     const fullPrompt = meetingContext + '\n\n---\n\n' + planPrompt;
     const result = await llm.callLLM(fullPrompt, '', {
       timeout: 120000, label: 'pipeline-plan', model: 'sonnet', maxTurns: 1,
+      engineConfig: config.engine,
     });
     if (result.text) {
       content = result.text;

package/engine/runtimes/copilot.js

@@ -15,10 +15,10 @@
  *                                    buildPrompt() prepends a <system> block.
  *   - costTracking: false           — result.usage has premiumRequests count
  *                                    and durations only; no USD or per-token.
- *   - modelShorthands: false        — full model IDs like "claude-sonnet-4.5",
- *                                    "gpt-5.4". Bare "sonnet" / "opus" / "haiku"
- *                                    is a Claude-ism — log a one-time warning
- *                                    when seen so the user notices the mistake.
+ *   - modelShorthands: false        — Copilot itself expects full model IDs like
+ *                                    "claude-sonnet-4.5" or "gpt-5.4". The
+ *                                    adapter translates Minions' internal family
+ *                                    aliases before argv construction.
  *   - modelDiscovery: true          — GET https://api.githubcopilot.com/models
  *                                    with `gh auth token` Bearer returns the
  *                                    catalog (24 models on the test account).
@@ -152,26 +152,23 @@ function resolveBinary({ env = process.env } = {}) {
 
 // ── Model Resolution ────────────────────────────────────────────────────────
 //
-// Copilot models are full IDs (`claude-sonnet-4.5`, `gpt-5.4`, ...). The
-// adapter passes them through verbatim. When we see a Claude shorthand
-// ('sonnet', 'opus', 'haiku') we log ONCE — a stronger signal than silently
-// passing it to Copilot, which would respond with an unknown-model error.
-
-const _CLAUDE_SHORTHANDS = new Set(['sonnet', 'opus', 'haiku']);
-let _shorthandWarningLogged = false;
-
-function _resetShorthandWarning() { _shorthandWarningLogged = false; }
+// Copilot models are full IDs (`claude-sonnet-4.5`, `gpt-5.4`, ...). Minions
+// still uses the family aliases `haiku` / `sonnet` / `opus` internally, so this
+// adapter maps those aliases to Copilot model IDs while passing all other input
+// through verbatim. The capability remains false: Copilot CLI does not accept
+// these aliases directly.
+
+const _MINIONS_MODEL_ALIASES = {
+  haiku: 'claude-haiku-4.5',
+  sonnet: 'claude-sonnet-4.5',
+  opus: 'claude-opus-4.5',
+};
 
-function resolveModel(input, { logger = console } = {}) {
+function resolveModel(input) {
   if (input == null || input === '') return undefined;
   const s = String(input);
-  if (_CLAUDE_SHORTHANDS.has(s.toLowerCase()) && !_shorthandWarningLogged) {
-    _shorthandWarningLogged = true;
-    try {
-      const warn = (logger && typeof logger.warn === 'function') ? logger.warn.bind(logger) : null;
-      if (warn) warn(`[copilot] "${s}" is a Claude family shorthand; Copilot expects a full model id (e.g. claude-sonnet-4.5). Passing through verbatim — Copilot will likely reject it.`);
-    } catch { /* logger may be unwired during tests */ }
-  }
+  const mapped = _MINIONS_MODEL_ALIASES[s.toLowerCase()];
+  if (mapped) return mapped;
   return s;
 }
 
@@ -783,8 +780,7 @@ module.exports = {
   createStreamConsumer,
   // Exposed for unit tests — engine code MUST go through resolveRuntime + the
   // adapter contract; never reach into these helpers directly.
-  _CLAUDE_SHORTHANDS,
-  _resetShorthandWarning,
+  _MINIONS_MODEL_ALIASES,
   _mapEffort,
   _copilotAssistantMessageHasTools,
   KNOWN_EVENT_TYPES,

package/package.json

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