@yemi33/minions 0.1.1772 → 0.1.1774

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## 0.1.1774 (2026-05-07)
+
+### Features
+- suppress stale doc-chat model errors
+
+### Fixes
+- yemi33/minions#2168
+
 ## 0.1.1772 (2026-05-07)
 
 ### Features

package/README.md

@@ -127,21 +127,31 @@ minions work "Explore the codebase and document the architecture"
 | `minions init` | Bootstrap `~/.minions/` with default agents and config |
 | `minions update` | Update to latest version (npm update + apply) |
 | `minions version` | Show installed vs package version |
+| `minions doctor` | Check prerequisites and runtime health |
 | `minions scan [dir] [depth]` | Scan for git repos and multi-select to add (default: ~, depth 3) |
 | `minions add <dir>` | Link a single project (auto-detects settings from git, prompts to confirm) |
 | `minions remove <dir-or-name> [--keep-data \| --purge --force]` | Unlink a project: cancels pending work items, drains dispatch + kills active agents, cleans worktrees, disables linked schedules, archives `projects/<name>/` to `projects/.archived/<name>-YYYYMMDD/`. Use `--keep-data` to leave the data dir in place, or `--purge --force` to delete it. |
 | `minions list` | List all linked projects with descriptions |
+| `minions restart` | Start engine and dashboard together (recommended after reboot) |
 | `minions start` | Start engine daemon (ticks every 60s, auto-syncs MCP servers) |
 | `minions stop` | Stop the engine |
 | `minions status` | Show agents, projects, dispatch queue, quality metrics |
 | `minions pause` / `resume` | Pause/resume dispatching |
 | `minions dispatch` | Force a dispatch cycle |
 | `minions discover` | Dry-run work discovery |
+| `minions queue` | Show dispatch queue (pending, active, completed) |
+| `minions sources` | Show work source status per project |
 | `minions work <title> [opts]` | Add to central work queue |
 | `minions spawn <agent> <prompt>` | Manually spawn an agent |
 | `minions plan <file\|text> [proj]` | Run a plan |
+| `minions kill` | Kill all active agents and reset their dispatches to pending |
+| `minions complete <dispatch-id>` | Manually mark a dispatch as completed |
+| `minions config set-cli <R> [--model M]` | Persist the default runtime/model without starting the engine |
+| `minions mcp-sync` | Sync MCP servers from `~/.claude.json` |
 | `minions cleanup` | Run cleanup manually (temp files, worktrees, zombies) |
 | `minions dash` | Open dashboard (starts if not already running, port 7331) |
+| `minions nuke --confirm` | Factory reset runtime state and reset config to defaults |
+| `minions uninstall --confirm` | Remove Minions state and uninstall the npm package |
 
 You can also run scripts directly: `node ~/.minions/engine.js start`, `node ~/.minions/dashboard.js`, etc.
 

package/bin/minions.js

@@ -8,18 +8,29 @@
  *   minions add <project-dir>        Link a project (interactive)
  *   minions remove <project-dir>     Unlink a project
  *   minions list                     List linked projects
+ *   minions update                   Update to latest version
+ *   minions version                  Show installed and package versions
+ *   minions doctor                   Check prerequisites and runtime health
+ *   minions restart                  Start engine + dashboard
  *   minions start                    Start the engine
  *   minions stop                     Stop the engine
  *   minions status                   Show engine status
  *   minions pause / resume           Pause/resume dispatching
+ *   minions queue                    Show dispatch queue
+ *   minions sources                  Show work source status
  *   minions dash                     Start the dashboard
  *   minions work <title> [opts-json] Add a work item
  *   minions spawn <agent> <prompt>   Manually spawn an agent
  *   minions dispatch                 Force a dispatch cycle
  *   minions discover                 Dry-run work discovery
  *   minions cleanup                  Run cleanup manually
+ *   minions kill                     Kill active agents and reset to pending
+ *   minions complete <dispatch-id>   Mark a dispatch completed
+ *   minions config set-cli <R> [--model M]  Persist default runtime/model
  *   minions plan <file|text> [proj]  Run a plan
- *   minions version                  Show installed and package versions
+ *   minions mcp-sync                 Sync MCP servers from ~/.claude.json
+ *   minions nuke --confirm           Factory reset runtime state/config
+ *   minions uninstall --confirm      Remove Minions and uninstall package
  */
 
 const fs = require('fs');
@@ -642,6 +653,9 @@ if (!cmd || cmd === 'help' || cmd === '--help' || cmd === '-h') {
     minions plan <file|text> [proj]  Run a plan
     minions kill                     Kill all active agents and reset to pending
     minions complete <dispatch-id>   Manually mark a dispatch as completed
+    minions config set-cli <R> [--model M]
+                                      Persist default runtime/model without starting
+    minions mcp-sync                 Sync MCP servers from ~/.claude.json
     minions cleanup                  Clean temp files, worktrees, zombies
     minions nuke --confirm           Factory reset (delete state, reset config to defaults)
     minions uninstall --confirm      Remove everything + uninstall npm package

package/dashboard/js/modal-qa.js

@@ -30,6 +30,32 @@ const QA_QUEUE_CAP = 10; // max queued messages
 const QA_STREAM_STALL_MS = 6 * 60 * 1000; // allow the full doc-chat timeout before treating heartbeat-only streams as stalled
 let _qaSessionKey = ''; // key for current conversation (title or filePath)
 
+const QA_STICKY_BOTTOM_PX = 80;
+let _qaThreadShouldFollow = true;
+
+function _qaIsNearThreadBottom(thread) {
+  if (!thread) return true;
+  return thread.scrollHeight - thread.scrollTop - thread.clientHeight <= QA_STICKY_BOTTOM_PX;
+}
+
+function _qaShouldFollowThread(thread) {
+  return _qaThreadShouldFollow && _qaIsNearThreadBottom(thread);
+}
+
+function _qaSetThreadFollowFromScroll(thread) {
+  _qaThreadShouldFollow = _qaIsNearThreadBottom(thread);
+}
+
+function _qaScrollThreadToBottom(thread) {
+  if (!thread) return;
+  thread.scrollTop = thread.scrollHeight;
+  _qaThreadShouldFollow = true;
+}
+
+function _qaMaybeScrollThreadToBottom(thread, shouldFollow) {
+  if (shouldFollow) _qaScrollThreadToBottom(thread);
+}
+
 // Insert html at the bottom of the thread but above any pending "Queued: ..."
 // strips, so queued messages always remain visually below the active progress
 // UX / answer / errors.
@@ -47,7 +73,7 @@ function _renderQaUserMessage(thread, message, selection) {
   }
   qHtml += '</div>';
   _qaInsertBeforeQueued(thread, qHtml);
-  thread.scrollTop = thread.scrollHeight;
+  _qaScrollThreadToBottom(thread);
   _showThreadWrap();
 }
 const _qaSessions = new Map(); // persist conversations across modal open/close {key → {history, threadHtml}}
@@ -91,6 +117,10 @@ function _qaThreadEl() {
   return document.getElementById('modal-qa-thread');
 }
 
+document.addEventListener('scroll', function(e) {
+  if (e.target && e.target.id === 'modal-qa-thread') _qaSetThreadFollowFromScroll(e.target);
+}, true);
+
 function _qaThreadHtml() {
   return (_qaThreadEl() || {}).innerHTML || '';
 }
@@ -308,8 +338,9 @@ function _qaMutateThreadHtml(key, mutate) {
     const wasCollapsed = _qaIsThreadCollapsed();
     const thread = _qaThreadEl();
     if (thread) {
+      const shouldFollow = _qaShouldFollowThread(thread);
       thread.innerHTML = html;
-      thread.scrollTop = thread.scrollHeight;
+      _qaMaybeScrollThreadToBottom(thread, shouldFollow);
     }
     if (wasCollapsed) _setQaThreadCollapsed(true);
     else _showThreadWrap();
@@ -386,7 +417,7 @@ function _initQaSession() {
     _showThreadWrap();
     requestAnimationFrame(function() {
       var thread = document.getElementById('modal-qa-thread');
-      if (thread) thread.scrollTop = thread.scrollHeight;
+      _qaScrollThreadToBottom(thread);
     });
     if (_qaQueue.length > 0 && !_qaProcessing) {
       setTimeout(_qaResumeQueuedMessages, 0);
@@ -394,6 +425,7 @@ function _initQaSession() {
   } else {
     _qaHistory = [];
     document.getElementById('modal-qa-thread').innerHTML = '';
+    _qaThreadShouldFollow = true;
     var wrap = document.getElementById('modal-qa-thread-wrap');
     var expandBar = document.getElementById('qa-expand-bar');
     if (wrap) wrap.style.display = 'none';
@@ -410,6 +442,7 @@ function clearQaConversation() {
   _qaProcessing = false;
   _qaAbortController = null;
   document.getElementById('modal-qa-thread').innerHTML = '';
+  _qaThreadShouldFollow = true;
   var wrap = document.getElementById('modal-qa-thread-wrap');
   var expandBar = document.getElementById('qa-expand-bar');
   if (wrap) wrap.style.display = 'none';
@@ -455,7 +488,7 @@ function modalSend() {
     }
     _qaQueue.push({ message, selection });
     thread.insertAdjacentHTML('beforeend', _qaBuildQueuedHtml(message));
-    thread.scrollTop = thread.scrollHeight;
+    _qaScrollThreadToBottom(thread);
     _showThreadWrap();
     _qaSaveActiveSessionState();
     return;

package/dashboard.js

@@ -281,12 +281,23 @@ function createWorkItemWithDedup(wiPath, item, options = {}) {
   return result || { created: false, item: null };
 }
 
+function formatUnknownProjectError(projectName, projects = []) {
+  const known = projects.map(p => p.name).filter(Boolean).join(', ') || '(none configured)';
+  return `Project "${projectName}" not found. Known projects: ${known}`;
+}
+
+function findProjectByName(projects, projectName) {
+  const name = String(projectName || '').trim().toLowerCase();
+  if (!name) return null;
+  return projects.find(p => p.name?.toLowerCase() === name) || null;
+}
+
 function resolveWorkItemsCreateTarget(projectName, projects = PROJECTS) {
   const project = String(projectName || '').trim();
   let targetProject = null;
   if (project) {
-    targetProject = projects.find(p => p.name === project) || (projects.length > 0 ? projects[0] : null);
-    if (!targetProject) return { error: 'No projects configured' };
+    targetProject = findProjectByName(projects, project);
+    if (!targetProject) return { error: formatUnknownProjectError(project, projects) };
   } else if (projects.length === 1) {
     targetProject = projects[0];
   }
@@ -326,7 +337,13 @@ function linkPullRequestForTracking({ url, title, project: projectName, autoObse
     throw err;
   }
   const projects = shared.getProjects(config);
-  const targetProject = projectName ? projects.find(p => p.name?.toLowerCase() === projectName.toLowerCase()) : (projects[0] || null);
+  const explicitProjectName = String(projectName || '').trim();
+  const targetProject = explicitProjectName ? findProjectByName(projects, explicitProjectName) : (projects[0] || null);
+  if (explicitProjectName && !targetProject) {
+    const err = new Error(formatUnknownProjectError(explicitProjectName, projects));
+    err.statusCode = 400;
+    throw err;
+  }
   const prPath = targetProject ? shared.projectPrPath(targetProject) : path.join(MINIONS_DIR, 'pull-requests.json');
 
   const prNumMatch = url.match(/\/pull\/(\d+)|pullrequest\/(\d+)/);
@@ -2704,23 +2721,19 @@ function _formatDocChatContext({ document, title, filePath, selection, canEdit,
   return context;
 }
 
-// Map errorClass codes from the runtime adapter to actionable user-facing messages.
+// Map runtime failures to user-facing messages.
 // sessionPreserved=true means ccCall preserved the session — user can retry immediately.
 // toolUses=[] from result.toolUses lets the message warn that tools may have already
 // modified files/state before the failure — the user shouldn't assume nothing happened.
-// errorMessage is the runtime adapter's own remediation string (from parseError) —
-// authoritative because it knows whether the runtime is Claude, Copilot, or another;
-// dashboard falls back to generic copy only when the adapter didn't supply one.
+// errorMessage is the runtime adapter's own text from parseError; when present it
+// is authoritative and is shown directly rather than replaced with dashboard copy.
 function _docChatErrorMessage(errorClass, sessionPreserved = false, toolUses = [], errorMessage = null) {
   const tools = Array.isArray(toolUses) ? toolUses : [];
   const toolHint = tools.length > 0
     ? ` (${tools.length} tool${tools.length === 1 ? '' : 's'} ran before the failure: ${tools.slice(0, 5).map(t => t.name).join(', ')}${tools.length > 5 ? '…' : ''} — files or state may have been modified.)`
     : '';
-  if (errorClass === 'auth-failure') return (errorMessage || 'Runtime authentication failed — check your CLI auth or API key, then try again.') + toolHint;
-  if (errorClass === 'context-limit') return 'Session context is too long. Click "Clear" to start a fresh conversation.' + toolHint;
-  if (errorClass === 'budget-exceeded') return (errorMessage || 'Runtime budget exceeded — check your account or quota.') + toolHint;
-  if (errorClass === 'crash') return (errorMessage || 'Runtime crashed unexpectedly. Try again.') + toolHint;
-  if (errorClass === 'unknown-model') return (errorMessage || 'Configured model is not valid for the active runtime. Update engine.ccModel or engine.defaultModel.') + toolHint;
+  const directError = typeof errorMessage === 'string' ? errorMessage.trim() : '';
+  if (directError) return directError + toolHint;
   if (sessionPreserved) return 'Temporary connection issue — your conversation is intact, send your message again.' + toolHint;
   if (tools.length > 0) return 'The agent stopped responding before producing a final answer.' + toolHint;
   return 'Failed to process request. Try again.';
@@ -2729,14 +2742,26 @@ function _docChatErrorMessage(errorClass, sessionPreserved = false, toolUses = [
 // Secondary note rendered alongside a recovered partial answer — distinct from the
 // hard-failure message because the answer/actions/document-edit DID land. The user
 // just needs to know the run wasn't clean.
-function _docChatPartialWarning(errorClass) {
-  if (errorClass === 'auth-failure') return 'Note: auth failed — answer recovered from partial output, but follow-up turns may not work.';
-  if (errorClass === 'context-limit') return 'Note: session context was too long — answer recovered. Click "Clear" before continuing.';
-  if (errorClass === 'budget-exceeded') return 'Note: runtime budget exceeded — answer recovered, but further calls may fail.';
-  if (errorClass === 'crash') return 'Note: runtime crashed before clean exit, but a complete response was recovered.';
+function _docChatPartialWarning(errorClass, errorMessage = null) {
+  const directError = typeof errorMessage === 'string' ? errorMessage.trim() : '';
+  if (directError) return `Note: ${directError}`;
   return 'Note: the agent exited unexpectedly. A complete response was recovered — verify any saved files or dispatched actions.';
 }
 
+function _docChatResultHasVisibleError(result) {
+  if (!result) return false;
+  if (result.errorClass) return true;
+  if (typeof result.errorMessage === 'string' && result.errorMessage.trim()) return true;
+  // stderr without a classified/runtime error is often CLI diagnostics; hard
+  // failures with no answer still surface stderr through _docChatFailureResponse.
+  return false;
+}
+
+function _docChatResultLooksSuccessful(result) {
+  if (!result || !result.text) return false;
+  return result.code === 0 || !_docChatResultHasVisibleError(result);
+}
+
 // Build the doc-chat extraContext for a single ccCall pass — refreshed on retry
 // so a fresh-session retry includes the full document instead of relying on the
 // dead session's prior turn for context.
@@ -2813,7 +2838,7 @@ function _recoverPartialDocChatResponse(result, sessionKey) {
   return {
     ...parsed,
     partial: true,
-    warning: _docChatPartialWarning(result.errorClass),
+    warning: _docChatPartialWarning(result.errorClass, result.errorMessage || result.stderr || null),
     toolUses: Array.isArray(result.toolUses) ? result.toolUses : [],
     // Recovery path still attaches the raw runtime failure — the answer landed
     // despite a non-zero exit; users still benefit from seeing why.
@@ -2821,6 +2846,41 @@ function _recoverPartialDocChatResponse(result, sessionKey) {
   };
 }
 
+function _shouldSuppressDocChatPostPatchError(ccError, finalize) {
+  if (!finalize || finalize.edited !== true) return false;
+  if (!ccError || ccError.errorClass !== 'unknown-model') return false;
+  return String(ccError.runtime || '').toLowerCase() === 'copilot';
+}
+
+function _buildDocChatResponsePayload({
+  answer,
+  actions,
+  actionResults,
+  actionParseError,
+  ccError,
+  partial,
+  warning,
+  toolUses,
+  finalize,
+} = {}) {
+  const final = finalize || { edited: false, content: null, answerSuffix: '' };
+  const suppressPostPatchError = _shouldSuppressDocChatPostPatchError(ccError, final);
+  const visibleAnswer = suppressPostPatchError && !partial ? 'Updated the document.' : answer;
+  const finalAnswer = final.answerSuffix ? visibleAnswer + final.answerSuffix : visibleAnswer;
+  return {
+    ok: !(ccError && !suppressPostPatchError),
+    answer: finalAnswer,
+    actions,
+    ...(actionResults ? { actionResults } : {}),
+    ...(actionParseError ? { actionParseError } : {}),
+    ...(ccError && !suppressPostPatchError ? { error: ccError } : {}),
+    ...(partial && !suppressPostPatchError ? { partial: true, warning } : {}),
+    ...(Array.isArray(toolUses) && toolUses.length ? { toolUses } : {}),
+    edited: final.edited,
+    ...(final.edited && final.content !== null ? { content: final.content } : {}),
+  };
+}
+
 
 // True when the file is a meeting JSON whose status forbids edits. Loaded
 // fresh on each call because meeting status can change while a doc-chat is
@@ -2973,7 +3033,7 @@ async function ccDocCall({ message, document, title, filePath, selection, canEdi
     // bleed into future interactions under the same key.
     docSessions.delete(sessionKey);
     schedulePersistDocSessions();
-  } else if (result.code === 0 && result.sessionId) {
+  } else if (_docChatResultLooksSuccessful(result) && result.sessionId) {
     // Store doc hash for next call's unchanged check
     const session = resolveSession('doc', sessionKey);
     if (session) session._docHash = initialPass.docHash;
@@ -2983,7 +3043,7 @@ async function ccDocCall({ message, document, title, filePath, selection, canEdi
     return { answer: result.text || result.stderr || 'Minions runtime is not installed or configured.', content: null, actions: [] };
   }
 
-  if (result.code !== 0 || !result.text) {
+  if (!_docChatResultLooksSuccessful(result)) {
     // Try to salvage a parseable answer / action / document edit before failing.
     const recovered = _recoverPartialDocChatResponse(result, sessionKey);
     if (recovered) return recovered;
@@ -3034,7 +3094,7 @@ async function ccDocCallStreaming({ message, document, title, filePath, selectio
   if (freshSession && sessionKey) {
     docSessions.delete(sessionKey);
     schedulePersistDocSessions();
-  } else if (result.code === 0 && result.sessionId) {
+  } else if (_docChatResultLooksSuccessful(result) && result.sessionId) {
     const session = resolveSession('doc', sessionKey);
     if (session) session._docHash = initialPass.docHash;
   }
@@ -3043,7 +3103,7 @@ async function ccDocCallStreaming({ message, document, title, filePath, selectio
     return { answer: result.text || result.stderr || 'Minions runtime is not installed or configured.', content: null, actions: [] };
   }
 
-  if (result.code !== 0 || !result.text) {
+  if (!_docChatResultLooksSuccessful(result)) {
     const recovered = _recoverPartialDocChatResponse(result, sessionKey);
     if (recovered) return recovered;
     const sessionPreserved = !!(resolveSession('doc', sessionKey)?.sessionId);
@@ -5184,20 +5244,11 @@ What would you like to discuss or change? When you're happy, say "approve" and I
         filePath: body.filePath, fullPath, isJson, canEdit,
         originalContent: currentContent, delimiterContent: content,
       });
-      const finalAnswer = finalize.answerSuffix ? answer + finalize.answerSuffix : answer;
-      _docDone = true;
-      return jsonReply(res, 200, {
-        ok: !ccError,
-        answer: finalAnswer,
-        actions,
-        ...(actionResults ? { actionResults } : {}),
-        ...(actionParseError ? { actionParseError } : {}),
-        ...(ccError ? { error: ccError } : {}),
-        ...(partial ? { partial: true, warning } : {}),
-        ...(Array.isArray(toolUses) && toolUses.length ? { toolUses } : {}),
-        edited: finalize.edited,
-        ...(finalize.edited && finalize.content !== null ? { content: finalize.content } : {}),
+      const payload = _buildDocChatResponsePayload({
+        answer, actions, actionResults, actionParseError, ccError, partial, warning, toolUses, finalize,
       });
+      _docDone = true;
+      return jsonReply(res, 200, payload);
       } finally { _docAbort = null; _docDone = true; docChatInFlight.delete(docKey); }
     } catch (e) { return jsonReply(res, e.statusCode || 500, { error: e.message }); }
   }
@@ -5286,18 +5337,14 @@ What would you like to discuss or change? When you're happy, say "approve" and I
           filePath: body.filePath, fullPath, isJson, canEdit,
           originalContent: currentContent, delimiterContent: content,
         });
-        const finalAnswer = finalize.answerSuffix ? answer + finalize.answerSuffix : answer;
+        const payload = _buildDocChatResponsePayload({
+          answer, actions, actionResults, actionParseError, ccError, partial, warning, toolUses, finalize,
+        });
+        const { answer: finalAnswer, ...donePayload } = payload;
         writeDocEvent({
           type: 'done',
           text: finalAnswer,
-          actions,
-          ...(actionResults ? { actionResults } : {}),
-          ...(actionParseError ? { actionParseError } : {}),
-          ...(ccError ? { error: ccError } : {}),
-          ...(partial ? { partial: true, warning } : {}),
-          ...(Array.isArray(toolUses) && toolUses.length ? { toolUses } : {}),
-          edited: finalize.edited,
-          ...(finalize.edited && finalize.content !== null ? { content: finalize.content } : {}),
+          ...donePayload,
         });
         _docStreamEnded = true;
         res.end();
@@ -7121,6 +7168,13 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       if (!url) return jsonReply(res, 400, { error: 'url required' });
 
       reloadConfig();
+      const explicitProjectName = String(body.project || '').trim();
+      if (explicitProjectName) {
+        const projects = shared.getProjects(CONFIG);
+        if (!findProjectByName(projects, explicitProjectName)) {
+          return jsonReply(res, 400, { error: formatUnknownProjectError(explicitProjectName, projects) }, req);
+        }
+      }
       const adoTarget = parseAdoPrMetadataTarget(url);
       let initialPrData = null;
       if (adoTarget) {
@@ -7130,7 +7184,13 @@ What would you like to discuss or change? When you're happy, say "approve" and I
           shared.log('warn', `ADO PR link metadata fetch failed for ${url}: ${e.message}`);
         }
       }
-      const { id: prId, prPath, prNum, created, linked } = linkPullRequestForTracking(body, CONFIG, { metadata: initialPrData });
+      let linkResult;
+      try {
+        linkResult = linkPullRequestForTracking(body, CONFIG, { metadata: initialPrData });
+      } catch (e) {
+        return jsonReply(res, e.statusCode || 400, { error: e.message }, req);
+      }
+      const { id: prId, prPath, prNum, created, linked } = linkResult;
       invalidateStatusCache();
       jsonReply(res, 200, { ok: true, id: prId, created, linked });
 
@@ -7268,6 +7328,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
     { method: 'POST', path: /^\/api\/agent\/([\w-]+)\/kill$/, desc: 'Kill a running agent: stop process, clear dispatch, reset work items to pending', handler: handleAgentKill },
     { method: 'GET', path: /^\/api\/agent\/([\w-]+)\/live-stream(?:\?.*)?$/, desc: 'SSE real-time live output streaming', handler: handleAgentLiveStream },
     { method: 'GET', path: /^\/api\/agent\/([\w-]+)\/live(?:\?.*)?$/, desc: 'Tail live output for a working agent', params: 'tail? (bytes, default 8192)', handler: handleAgentLive },
+    { method: 'GET', path: /^\/api\/agent\/([\w-]+)\/live-output(?:\?.*)?$/, desc: 'Tail live output for a working agent (alias for /live)', params: 'tail? (bytes, default 8192)', handler: handleAgentLive },
     { method: 'GET', path: /^\/api\/agent\/([\w-]+)\/output(?:\?.*)?$/, desc: 'Fetch final output.log for an agent', handler: handleAgentOutput },
     { method: 'GET', path: /^\/api\/agent\/([\w-]+)$/, desc: 'Get detailed agent info', handler: handleAgentDetail },
     { method: 'GET', path: /^\/api\/dispatch\/([\w.-]+)\/completion-report$/, desc: 'Read structured completion report for a dispatch', handler: (req, res, match) => {
@@ -7722,6 +7783,10 @@ module.exports = {
   _docChatPartialWarning,
   _docChatFailureResponse,
   _recoverPartialDocChatResponse,
+  _docChatResultHasVisibleError,
+  _docChatResultLooksSuccessful,
+  _shouldSuppressDocChatPostPatchError,
+  _buildDocChatResponsePayload,
   _linkPullRequestForTracking: linkPullRequestForTracking,
   _resolveSkillReadPath,
   DOC_CHAT_DOCUMENT_DELIMITER,

package/engine/copilot-models.json

@@ -1,5 +1,5 @@
 {
   "runtime": "copilot",
   "models": null,
-  "cachedAt": "2026-05-07T17:21:57.007Z"
+  "cachedAt": "2026-05-07T21:34:32.453Z"
 }

package/engine/runtimes/copilot.js

@@ -634,25 +634,77 @@ function parseStreamChunk(line) {
 
 // ── Error Normalization ─────────────────────────────────────────────────────
 
-function parseError(rawOutput) {
+function _collectErrorSignal(rawOutput) {
   const text = rawOutput == null ? '' : String(rawOutput);
+  if (!text) return '';
+
+  const signals = [];
+  let sawJsonLine = false;
+  for (const rawLine of text.split('\n')) {
+    const line = rawLine.trim();
+    if (!line) continue;
+    if (!line.startsWith('{')) {
+      signals.push(line);
+      continue;
+    }
+
+    let obj;
+    try { obj = JSON.parse(line); } catch { continue; }
+    if (!obj || typeof obj !== 'object') continue;
+    sawJsonLine = true;
+
+    const type = String(obj.type || '');
+    const subtype = String(obj.subtype || '');
+    const eventType = String(obj.event?.type || '');
+    const isErrorEvent = type === 'error'
+      || eventType === 'error'
+      || subtype.startsWith('error')
+      || obj.is_error === true
+      || obj.error != null
+      || obj.data?.error != null;
+    if (!isErrorEvent) continue;
+
+    for (const value of [
+      obj.error,
+      obj.message,
+      obj.stderr,
+      obj.result,
+      obj.data?.error,
+      obj.data?.message,
+      obj.data?.stderr,
+      obj.event?.error,
+      obj.event?.message,
+      subtype,
+    ]) {
+      if (typeof value === 'string' && value.trim()) signals.push(value.trim());
+    }
+  }
+
+  if (signals.length > 0) return signals.join('\n');
+  return sawJsonLine ? '' : text;
+}
+
+function parseError(rawOutput) {
+  const text = _collectErrorSignal(rawOutput);
   if (!text) return { message: '', code: null, retriable: true };
   const lower = text.toLowerCase();
 
-  if (/not authenticated|copilot login|please.*log.*in|401|403 forbidden|unauthorized/i.test(text)) {
-    return { message: 'Copilot/GitHub authentication failed. Run `gh auth login` or provide GH_TOKEN/COPILOT_GITHUB_TOKEN with Copilot access.', code: 'auth-failure', retriable: false };
+  const hasExplicitAuthFailure = /not authenticated|copilot login|please.*log.*in|\bunauthorized\b/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: text, code: 'auth-failure', retriable: false };
   }
   if (/rate limit|too many requests|\b429\b/i.test(text)) {
-    return { message: 'Copilot rate limit hit', code: 'rate-limit', retriable: true };
+    return { message: text, code: 'rate-limit', retriable: true };
   }
   if (/unknown model|model not found|model.*invalid|invalid model/i.test(text)) {
-    return { message: 'Copilot rejected the requested model', code: 'unknown-model', retriable: false };
+    return { message: text, code: 'unknown-model', retriable: false };
   }
   if (/budget.*exceed|premium.*limit.*reach|quota.*exceed/i.test(lower)) {
-    return { message: 'Copilot premium-request budget exceeded — check your GitHub Copilot quota.', code: 'budget-exceeded', retriable: false };
+    return { message: text, code: 'budget-exceeded', retriable: false };
   }
   if (/internal error|panic|uncaught|copilot.*crashed|fatal: copilot/i.test(lower)) {
-    return { message: 'Copilot CLI crashed unexpectedly. Try again.', code: 'crash', retriable: true };
+    return { message: text, code: 'crash', retriable: true };
   }
   return { message: '', code: null, retriable: true };
 }

package/package.json

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