@yemi33/minions 0.1.1820 → 0.1.1822

package/CHANGELOG.md

@@ -1,9 +1,13 @@
 # Changelog
 
-## 0.1.1820 (2026-05-09)
+## 0.1.1822 (2026-05-09)
 
 ### Features
-- add --open flag (and MINIONS_FORCE_OPEN env) to bypass the recent-browser-tab heuristic (#2274)
+- keep CC fixes on referenced PR (#2260)
+- harden CC action parsing (#2261)
+- correlate CC action failures (#2264)
+
+## 0.1.1819 (2026-05-09)
 
 ### Fixes
 - stop visibility=hidden beacons from racing pagehide and resurrecting closed-tab entries in dashboard-browser.json (#2275)

package/dashboard/js/command-center.js

@@ -10,6 +10,7 @@ var _ccTabs = [];         // [{id, title, sessionId, messages: [{role, html}]}]
 var _ccActiveTabId = null;
 var _ccOpen = false;
 var _ccRetrySeq = 0;
+var _ccMessageSeq = 0;
 // Per-tab sending state stored on tab objects: tab._sending, tab._queue, tab._abortController
 // Legacy globals for backward compat (badge, drawer close check)
 var _ccSending = false; // true if active tab is sending (UI indicator only)
@@ -19,11 +20,11 @@ try { localStorage.removeItem('cc-sending'); } catch {}
 function _ccStripActionBlockFromText(value) {
   var text = value || '';
   if (!text) return text;
-  // Tier 1 — strict: 3 leading + 0-3 trailing equals on its own line.
-  var full = /(?:^|\r?\n)===ACTIONS={0,3}[ \t]*(?=\r?\n|$)/m.exec(text);
+  // Tier 1 — strict: exact canonical delimiter on its own line.
+  var full = /(?:^|\r?\n)===ACTIONS===[ \t]*(?=\r?\n|$)/m.exec(text);
   if (full) return text.slice(0, full.index + full[0].indexOf('===ACTIONS')).trim();
-  // Tier 2 — loose: ===ACTIONS followed by punctuation/extra-equals to EOL.
-  var block = /(?:^|\r?\n)===ACTIONS\b[^\r\n]*(?=\r?\n|$)/m.exec(text);
+  // Tier 2 — loose: sentinel-looking malformed delimiters; not prose.
+  var block = /(?:^|\r?\n)===ACTIONS(?:[ \t]*(?:[-=]>?|={1,}|$)|[^A-Za-z0-9_\s\r\n][^\r\n]*)(?=\r?\n|$)/m.exec(text);
   if (block) return text.slice(0, block.index + block[0].indexOf('===ACTIONS')).trim();
   // Tier 3 — very loose: 2+ leading equals + ACTIONS keyword + 0+ trailing
   // equals, case-insensitive. Catches ====ACTIONS===, ===actions===, etc.
@@ -341,7 +342,7 @@ function ccSwitchTab(id) {
   el.innerHTML = '';
   // Re-render messages from the tab's data
   for (var i = 0; i < tab.messages.length; i++) {
-    ccAddMessage(tab.messages[i].role, tab.messages[i].html, true);
+    ccAddMessage(tab.messages[i].role, tab.messages[i].html, true, null, tab.messages[i]);
   }
   // If this tab is still processing, restore the full streaming UX (tools, partial text, thinking)
   if (tab._sending) {
@@ -454,7 +455,7 @@ function ccRestoreMessages() {
   if (!tab) return;
   if (el.children.length > 0 || tab.messages.length === 0) return; // Already rendered or nothing to restore
   for (var i = 0; i < tab.messages.length; i++) {
-    ccAddMessage(tab.messages[i].role, tab.messages[i].html, true);
+    ccAddMessage(tab.messages[i].role, tab.messages[i].html, true, null, tab.messages[i]);
   }
   // Restore "thinking" indicator if CC was mid-request when page refreshed
   try {
@@ -502,7 +503,7 @@ function ccSaveState() {
           if (!m || typeof m.html !== 'string') return m;
           var stripped = _ccStripActionBlockFromText(m.html);
           if (stripped === m.html) return m;
-          return { role: m.role, html: stripped };
+          return Object.assign({}, m, { html: stripped });
         });
         return { id: t.id, title: t.title, sessionId: t.sessionId, messages: msgs };
       });
@@ -526,11 +527,21 @@ function ccUpdateSessionIndicator() {
   }
 }
 
+function _ccNewMessageId(prefix) {
+  _ccMessageSeq++;
+  return (prefix || 'cc-msg') + '-' + Date.now().toString(36) + '-' + _ccMessageSeq;
+}
+
+function _ccMessageDomId(messageId) {
+  return 'cc-msg-' + String(messageId || '').replace(/[^a-zA-Z0-9_-]/g, '-');
+}
+
 function ccAddMessage(role, html, skipSave, targetTabId, meta) {
   var isUser = role === 'user';
   var isSystem = role === 'system';
   var isAction = role === 'action';
   var isAssistant = !isUser && !isSystem && !isAction;
+  var messageId = meta && (meta.messageId || meta._messageId) ? String(meta.messageId || meta._messageId) : '';
   var targetTab = targetTabId ? _ccTabs.find(function(t) { return t.id === targetTabId; }) : _ccActiveTab();
   // Only render to DOM if this message is for the currently visible tab
   var isVisible = !targetTabId || targetTabId === _ccActiveTabId;
@@ -538,6 +549,10 @@ function ccAddMessage(role, html, skipSave, targetTabId, meta) {
     var el = document.getElementById('cc-messages');
     var div = document.createElement('div');
     div.className = isAssistant ? 'cc-msg-assistant' : '';
+    if (messageId) {
+      div.id = _ccMessageDomId(messageId);
+      div.setAttribute('data-cc-message-id', messageId);
+    }
     if (meta && meta.retryId) div.setAttribute('data-cc-retry-id', meta.retryId);
     div.style.cssText = 'padding:8px 12px;border-radius:8px;font-size:12px;line-height:1.6;max-width:95%;' +
       (isUser ? 'background:var(--blue);color:#fff;align-self:flex-end' : isSystem ? 'align-self:center;max-width:100%' : isAction ? 'align-self:flex-start;padding:2px 0' : 'background:var(--surface2);color:var(--text);align-self:flex-start;border:1px solid var(--border);position:relative');
@@ -550,6 +565,7 @@ function ccAddMessage(role, html, skipSave, targetTabId, meta) {
     var tab = targetTab;
     if (tab) {
       var msg = { role: role, html: html };
+      if (messageId) msg._messageId = messageId;
       if (meta && meta.retryId) msg._retryId = meta.retryId;
       tab.messages.push(msg);
       // Auto-title from first user message
@@ -565,6 +581,7 @@ function ccAddMessage(role, html, skipSave, targetTabId, meta) {
     }
     ccSaveState();
   }
+  return messageId;
 }
 
 async function ccSend(options) {
@@ -809,34 +826,26 @@ async function _ccDoSend(message, skipUserMsg, forceTabId, intentMetadata) {
           addMsg('system', '<div style="text-align:center;padding:6px 12px;font-size:11px;color:var(--muted);background:var(--surface2);border-radius:6px;margin:4px 0">' + resetText + '</div>', false, activeTabId);
         }
         var finalText = _ccMergeStreamText(streamedText, evt.text || '');
+        if (evt.actions && evt.actions.length > 0) _tagServerExecuted(evt.actions, evt.actionResults);
         var rendered = renderMd(finalText || streamedText || '');
-        addMsg('assistant', rendered + _ccElapsedFooter('{seconds}s'));
+        var assistantMessageId = _ccNewMessageId('cc-turn');
+        var actionFeedbackHtml = _ccBuildActionResultFeedbackHtml(evt.actions || [], evt.actionResults || [], {
+          delayed: !!isReconnect,
+          emittedAt: evt.actionResultsAt || evt.emittedAt || ''
+        });
+        addMsg('assistant', rendered + _ccElapsedFooter('{seconds}s') + actionFeedbackHtml, false, { messageId: assistantMessageId });
         if (evt.sessionId !== undefined) {
           var originTab = _ccTabs.find(function(t) { return t.id === activeTabId; });
           if (originTab) { originTab.sessionId = evt.sessionId || null; }
           ccSaveState(); ccUpdateSessionIndicator();
         }
         if (evt.actions && evt.actions.length > 0) {
-          _tagServerExecuted(evt.actions, evt.actionResults);
-          for (var ai = 0; ai < evt.actions.length; ai++) { await ccExecuteAction(evt.actions[ai], activeTabId); }
-          // Surface per-action errors/warnings inline alongside the prose so the user can see
-          // exactly which actions failed or completed with caveats. Previously these only
-          // appeared as a small "executed" pill which gave no detail.
-          if (evt.actionResults && Array.isArray(evt.actionResults)) {
-            var failures = evt.actionResults.filter(function(r) { return r && r.error; });
-            var warnings = evt.actionResults.filter(function(r) { return r && r.warning; });
-            if (failures.length > 0) {
-              var failHtml = failures.map(function(r) {
-                return '<li>' + escHtml(r.type || (r.actionContext && r.actionContext.type) || 'action') + ': ' + escHtml(r.error) + _ccActionContextSuffix(r.actionContext) + '</li>';
-              }).join('');
-              addMsg('system', '<div style="padding:6px 12px;font-size:11px;color:var(--red);background:var(--surface2);border-radius:6px;margin:4px 0">⚠️ ' + failures.length + ' action' + (failures.length > 1 ? 's' : '') + ' failed:<ul style="margin:4px 0 0 16px;padding:0">' + failHtml + '</ul></div>', false, activeTabId);
-            }
-            if (warnings.length > 0) {
-              var warnHtml = warnings.map(function(r) {
-                return '<li>' + escHtml(r.type || (r.actionContext && r.actionContext.type) || 'action') + ': ' + escHtml(r.warning) + _ccActionContextSuffix(r.actionContext) + '</li>';
-              }).join('');
-              addMsg('system', '<div style="padding:6px 12px;font-size:11px;color:var(--orange);background:var(--surface2);border-radius:6px;margin:4px 0">ℹ️ ' + warnings.length + ' action' + (warnings.length > 1 ? '' : '') + ' completed with warnings:<ul style="margin:4px 0 0 16px;padding:0">' + warnHtml + '</ul></div>', false, activeTabId);
+          for (var ai = 0; ai < evt.actions.length; ai++) {
+            if (evt.actions[ai] && evt.actions[ai]._serverExecuted) {
+              _ccRunServerActionSideEffects(evt.actions[ai]);
+              continue;
             }
+            await ccExecuteAction(evt.actions[ai], activeTabId, { originMessageId: assistantMessageId });
           }
         } else if (evt.actionParseError) {
           // Issue #1834: server saw ===ACTIONS=== but couldn't parse the JSON.
@@ -1016,6 +1025,79 @@ async function _ccFetch(url, body, method) {
   return res;
 }
 
+function _ccActionResultSubject(action, result) {
+  var value = (result && (result.title || result.id || result.duplicateOf || result.file || result.target || result.endpoint)) ||
+    (action && (action.title || action.id || action.file || action.target || action.endpoint)) || '';
+  return String(value || '').replace(/\s+/g, ' ').trim().slice(0, 120);
+}
+
+function _ccActionResultType(action, result) {
+  return String((result && result.type) || (action && action.type) || 'action').trim() || 'action';
+}
+
+function _ccActionResultLine(action, result) {
+  var type = _ccActionResultType(action, result);
+  var subject = _ccActionResultSubject(action, result);
+  var label = escHtml(type) + (subject ? ' <strong>' + escHtml(subject) + '</strong>' : '');
+  var contextSuffix = result && result.actionContext ? _ccActionContextSuffix(result.actionContext) : '';
+  if (result && result.error) {
+    return '<li class="cc-action-feedback-row cc-action-feedback-error">&#10007; ' + label + ': ' + escHtml(result.error) + contextSuffix + '</li>';
+  }
+  if (result && result.warning) {
+    return '<li class="cc-action-feedback-row cc-action-feedback-warning">&#9888; ' + label + ': ' + escHtml(result.warning) + contextSuffix + '</li>';
+  }
+  if (result && result.ok) {
+    var duplicate = result.duplicate ? ' <span style="color:var(--orange)">already exists</span>' : '';
+    return '<li class="cc-action-feedback-row cc-action-feedback-ok">&#10003; ' + label + duplicate + contextSuffix + '</li>';
+  }
+  return '';
+}
+
+function _ccBuildActionResultFeedbackHtml(actions, actionResults, opts) {
+  if (!Array.isArray(actions) || !Array.isArray(actionResults)) return '';
+  var rows = [];
+  for (var i = 0; i < actions.length && i < actionResults.length; i++) {
+    var r = actionResults[i];
+    if (!r || (!r.ok && !r.error && !r.warning)) continue;
+    var row = _ccActionResultLine(actions[i] || {}, r);
+    if (row) rows.push(row);
+  }
+  if (rows.length === 0) return '';
+  var failures = actionResults.filter(function(r) { return r && r.error; }).length;
+  var warnings = actionResults.filter(function(r) { return r && r.warning; }).length;
+  var delayed = !!(opts && opts.delayed);
+  var emittedAt = opts && opts.emittedAt ? String(opts.emittedAt) : '';
+  var label = delayed ? 'Action results from previous turn' : 'Action results';
+  var timing = emittedAt ? ' <span style="color:var(--muted);font-weight:400">(' + escHtml(emittedAt) + ')</span>' : '';
+  var color = failures > 0 ? 'var(--red)' : warnings > 0 ? 'var(--orange)' : 'var(--green)';
+  return '<div class="cc-action-feedback" data-cc-action-feedback="true" style="margin-top:8px;padding:6px 10px;border:1px dashed var(--border);border-radius:6px;background:var(--surface);font-size:11px;color:' + color + '">' +
+    '<div style="font-weight:700">' + label + timing + '</div>' +
+    '<ul style="margin:4px 0 0 16px;padding:0">' + rows.join('') + '</ul>' +
+    '</div>';
+}
+
+function _ccAppendHtmlToMessage(tabOrId, messageId, html) {
+  if (!messageId || !html) return false;
+  var tab = typeof tabOrId === 'object' && tabOrId ? tabOrId : _ccTabs.find(function(t) { return t.id === tabOrId; });
+  if (!tab || !Array.isArray(tab.messages)) return false;
+  var msg = tab.messages.find(function(m) { return m && m._messageId === messageId; });
+  if (!msg) return false;
+  msg.html = (msg.html || '') + html;
+  if (typeof _ccActiveTabId !== 'undefined' && tab.id === _ccActiveTabId && typeof document !== 'undefined') {
+    var el = document.getElementById(_ccMessageDomId(messageId));
+    if (el && typeof el.insertAdjacentHTML === 'function') el.insertAdjacentHTML('beforeend', html);
+  }
+  return true;
+}
+
+function _ccRunServerActionSideEffects(action) {
+  var type = String(action && action.type || '');
+  if (['dispatch','fix','explore','review','test','create-meeting','plan','execute-plan','approve-plan','resume-plan','trigger-pipeline','advance-meeting','trigger-verify','regenerate-plan','continue-pipeline'].indexOf(type) >= 0) {
+    wakeEngine();
+  }
+  refresh();
+}
+
 var CC_ACTION_CONTEXT_STALE_MS = 2 * 60 * 1000;
 function _ccActionContextIsStale(ctx, nowMs) {
   if (!ctx || !ctx.requestedAt) return false;
@@ -1059,7 +1141,7 @@ function _tagServerExecuted(actions, actionResults) {
   }
 }
 
-async function ccExecuteAction(action, targetTabId) {
+async function ccExecuteAction(action, targetTabId, opts) {
   action = _ccNormalizeDispatchAction(action);
   var status = document.createElement('div');
   status.style.cssText = 'padding:4px 10px;border-radius:4px;font-size:10px;align-self:flex-start;border:1px dashed var(--border);color:var(--muted)';
@@ -1081,6 +1163,11 @@ async function ccExecuteAction(action, targetTabId) {
         _ccActionContextSuffix(action._serverContext);
       status.style.color = 'var(--green)';
     }
+    if (opts && opts.originMessageId && _ccAppendHtmlToMessage(targetTabId || _ccActiveTabId, opts.originMessageId, status.outerHTML)) {
+      ccSaveState();
+      refresh();
+      return;
+    }
     ccAddMessage('action', status.outerHTML, false, targetTabId);
     if (['dispatch','fix','explore','review','test','create-meeting'].includes(action.type)) wakeEngine();
     refresh();
@@ -1584,6 +1671,11 @@ async function ccExecuteAction(action, targetTabId) {
     status.style.color = 'var(--red)';
   }
 
+  if (opts && opts.originMessageId && _ccAppendHtmlToMessage(targetTabId || _ccActiveTabId, opts.originMessageId, status.outerHTML)) {
+    ccSaveState();
+    refresh();
+    return;
+  }
   ccAddMessage('action', status.outerHTML, false, targetTabId);
   refresh();
 }

package/dashboard.js

@@ -233,9 +233,30 @@ function normalizePrMetadata(metadata) {
   };
 }
 
-function getWorkItemPrRef(input) {
-  if (!input || typeof input !== 'object') return null;
-  return input.targetPr || input.pr || input.prId || input.prNumber || input.pullRequest || input.sourcePr || input.prUrl || null;
+function normalizePrTitleMatchText(value) {
+  return String(value || '').trim().toLowerCase().replace(/\s+/g, ' ');
+}
+
+function findPrRecordReferencedByText(prs, text, project = null) {
+  if (!Array.isArray(prs) || !String(text || '').trim()) return null;
+  const explicitRef = extractPrRefFromText(text);
+  if (explicitRef) return shared.findPrRecord(prs, explicitRef, project);
+  const normalizedText = normalizePrTitleMatchText(text);
+  if (!normalizedText) return null;
+  const matches = prs.filter(pr => {
+    const title = normalizePrTitleMatchText(pr?.title);
+    return title.length >= 8 && normalizedText.includes(title);
+  });
+  return matches.length === 1 ? matches[0] : null;
+}
+
+function inferActionPrRecord(action, prs, project = null) {
+  const text = [
+    action?.title,
+    action?.description,
+    action?.reason,
+  ].filter(value => typeof value === 'string' && value.trim()).join('\n');
+  return findPrRecordReferencedByText(prs, text, project);
 }
 
 function copyWorkItemPrFields(item, input, pr = null) {
@@ -1901,11 +1922,11 @@ function findCCActionsDelimiter(text) {
 // but are not parsed (they shouldn't reach the user as actions).
 function findCCActionsHeader(text) {
   if (!text) return null;
-  // Tier 1 — strict, parseable: 3 leading + 0-3 trailing equals, well-formed line.
-  const strict = /(?:^|\r?\n)===ACTIONS={0,3}[ \t]*(?=\r?\n|$)/m.exec(text);
+  // Tier 1 — strict, parseable: the exact canonical delimiter on its own line.
+  const strict = /(?:^|\r?\n)===ACTIONS===[ \t]*(?=\r?\n|$)/m.exec(text);
   if (strict) {
     const headerStart = strict.index + strict[0].indexOf('===ACTIONS');
-    const headerMatch = text.slice(headerStart).match(/^===ACTIONS={0,3}[ \t]*/);
+    const headerMatch = text.slice(headerStart).match(/^===ACTIONS===[ \t]*/);
     return {
       index: headerStart,
       headerLength: headerMatch ? headerMatch[0].length : '===ACTIONS==='.length,
@@ -2020,6 +2041,32 @@ function _extractActionsJson(segment) {
 const CC_DISPATCH_ACTION_ALIASES = new Set(['fix', 'explore', 'review', 'test', 'implement', 'implement:large', 'ask', 'verify']);
 const CC_ACTION_INTENT_WORK_TYPES = new Set(['fix', 'implement', 'implement:large', 'explore', 'review', 'test', 'ask', 'verify']);
 
+function getWorkItemPrRef(input) {
+  if (!input || typeof input !== 'object') return null;
+  return input.targetPr || input.pr || input.prId || input.prNumber || input.pullRequest || input.sourcePr || input.prUrl || null;
+}
+
+function isPrTargetedWorkType(workType) {
+  return ['fix', 'review', 'test'].includes(String(workType || '').toLowerCase());
+}
+
+function trimTrailingPrRefPunctuation(value) {
+  return String(value || '').replace(/[),.;:]+$/g, '');
+}
+
+function extractPrRefFromText(value) {
+  const text = String(value || '');
+  if (!text.trim()) return null;
+  const urlMatch = text.match(/https?:\/\/[^\s<>()]+(?:\/pull\/\d+|\/pullrequest\/\d+)[^\s<>()]*/i);
+  if (urlMatch) return trimTrailingPrRefPunctuation(urlMatch[0]);
+  const canonicalMatch = text.match(/\b(?:github|ado):[^\s#]+#\d+\b/i);
+  if (canonicalMatch) return canonicalMatch[0];
+  const legacyMatch = text.match(/\bPR-(\d+)\b/i);
+  if (legacyMatch) return legacyMatch[1];
+  const numberMatch = text.match(/\b(?:pr|pull\s+request|pullrequest)\s*#?\s*(\d+)\b/i);
+  return numberMatch ? numberMatch[1] : null;
+}
+
 function normalizeCCAction(action) {
   if (!action || typeof action !== 'object') return action;
   if (typeof action.type !== 'string') return action;
@@ -2051,7 +2098,7 @@ function _ccCleanIntentString(value, max = 500) {
 function _ccNormalizeIntentMetadata(meta) {
   if (!meta || typeof meta !== 'object' || Array.isArray(meta)) return {};
   const out = {};
-  for (const key of ['intent', 'type', 'title', 'description', 'priority', 'project', 'branchStrategy', 'branch_strategy']) {
+  for (const key of ['intent', 'type', 'title', 'description', 'priority', 'project', 'branchStrategy', 'branch_strategy', 'pr', 'targetPr', 'prId', 'prNumber', 'pullRequest', 'sourcePr', 'prUrl']) {
     const value = _ccCleanIntentString(meta[key], key === 'description' ? 2000 : 300);
     if (value) out[key] = value;
   }
@@ -2165,7 +2212,7 @@ function _actionsWithIntentFallback(actions, opts = {}) {
   const { message = '', intentMetadata = null, source = 'command-center', filePath = null, title: docTitle = null, answerText = '', toolUses = [] } = opts;
   const existing = (Array.isArray(actions) ? actions.map(normalizeCCAction) : [])
     .filter(action => !_ccShouldSuppressAnsweredAskDispatch(action, { message, source, answerText, toolUses }));
-  if (_messageRequestsDirectHandling(message)) return existing.filter(a => normalizeCCAction(a)?.type !== 'dispatch');
+  if (_messageRequestsDirectHandling(message, { answerText, toolUses })) return existing.filter(a => normalizeCCAction(a)?.type !== 'dispatch');
   if (existing.some(a => normalizeCCAction(a)?.type === 'dispatch')) return existing;
   if (existing.length > 0) return existing;
   const meta = _ccNormalizeIntentMetadata(intentMetadata);
@@ -2199,6 +2246,8 @@ function _actionsWithIntentFallback(actions, opts = {}) {
     workType: intent.workType,
     ...common,
   };
+  const prRef = getWorkItemPrRef(meta) || extractPrRefFromText([message, meta.title, meta.description].filter(Boolean).join('\n'));
+  if (prRef && isPrTargetedWorkType(intent.workType)) action.pr = prRef;
   if (!hasTarget) action._intentFallbackError = _ccFallbackMissingTargetError(intent);
   if (_ccShouldSuppressAnsweredAskDispatch(action, { message, source, answerText, toolUses })) return existing;
   return [action];
@@ -2246,7 +2295,11 @@ function parseCCActions(text) {
       if (jsonStr) {
         try {
           const parsed = JSON.parse(jsonStr);
-          actions = Array.isArray(parsed) ? parsed : [parsed];
+          if (Array.isArray(parsed)) {
+            actions = parsed;
+          } else {
+            parseError = 'actions JSON must be an array after ===ACTIONS=== delimiter';
+          }
         } catch (e) {
           parseError = e.message || 'invalid JSON';
         }
@@ -2261,17 +2314,6 @@ function parseCCActions(text) {
       parseError = 'Malformed ===ACTIONS=== delimiter (extra equals, lowercase, or trailing punctuation). Actions silently discarded — fix the model output.';
     }
   }
-  if (actions.length === 0) {
-    const actionRegex = /`{3,}\s*action\s*\r?\n([\s\S]*?)`{3,}/g;
-    let match;
-    while ((match = actionRegex.exec(displayText)) !== null) {
-      try { actions.push(JSON.parse(match[1].trim())); } catch {}
-    }
-    if (actions.length > 0) {
-      displayText = displayText.replace(/`{3,}\s*action\s*\r?\n[\s\S]*?`{3,}\n?/g, '').trim();
-      parseError = null; // legacy fallback recovered actions
-    }
-  }
   actions = actions.map(normalizeCCAction);
   const result = { text: displayText, actions };
   if (parseError && actions.length === 0) {
@@ -2455,15 +2497,33 @@ function _messageExplicitlyRequestsDelegation(message) {
   return false;
 }
 
-function _messageRequestsDirectHandling(message) {
+function _messageRequestsDirectHandling(message, opts = {}) {
   const normalized = _normalizeIntentText(message);
   if (!normalized.trim()) return false;
+  if (_messageRequestsInformationOnly(message) && !_unansweredInvestigationSignal(opts)) return true;
   if (_intentHasAnyPhrase(normalized, DIRECT_HANDLING_PHRASES)) return true;
   if (_intentHasAnyToken(normalized, DIRECT_QUICK_TERMS) && _intentHasAnyToken(normalized, DIRECT_QUICK_OBJECTS)) return true;
   if (_intentHasVerbObject(normalized, DIRECT_REPLY_TERMS, DIRECT_REPLY_TARGETS)) return true;
   return _intentHasVerbObject(normalized, DIRECT_SELF_TERMS, DIRECT_SELF_TARGETS);
 }
 
+function _unansweredInvestigationSignal({ answerText = '', toolUses = [] } = {}) {
+  const answerLen = String(answerText || '').trim().length;
+  const toolCount = Array.isArray(toolUses) ? toolUses.length : 0;
+  return toolCount >= 4 && answerLen < ANSWERED_ASK_MIN_CHARS;
+}
+
+function _messageRequestsInformationOnly(message) {
+  const normalized = _normalizeIntentText(message);
+  const tokens = _intentTokens(normalized);
+  if (!tokens.length) return false;
+  if (['what', 'which', 'who', 'when', 'where', 'why', 'how'].includes(tokens[0])) return true;
+  if (['show', 'list'].includes(tokens[0])) return true;
+  if (tokens[0] === 'tell' && tokens[1] === 'me') return true;
+  if (String(message || '').trim().endsWith('?') && !_intentHasAnyToken(normalized, DELEGATION_ACTION_TERMS)) return true;
+  return false;
+}
+
 function _messageIsSmallDocOnlyRequest(normalized) {
   if (_intentHasAnyToken(normalized, DOC_CHAT_DIRECT_DOC_TERMS)) return true;
   return _intentHasVerbObject(normalized, DOC_CHAT_DOC_ACTION_TERMS, DOC_CHAT_DOC_OBJECTS);
@@ -2575,7 +2635,7 @@ function _priorityFromDelegationMessage(message) {
 }
 
 function _inferDelegationActionFromMessage(message, { source = 'command-center', filePath = null, title = null, answerText = '', toolUses = [] } = {}) {
-  if (_messageRequestsDirectHandling(message)) return null;
+  if (_messageRequestsDirectHandling(message, { answerText, toolUses })) return null;
   const explicit = _messageHasDelegationIntent(message);
   const mediumLarge = _messageHasMediumLargeWorkIntent(message, { source });
   const toolCount = Array.isArray(toolUses) ? toolUses.length : 0;
@@ -2606,7 +2666,7 @@ function _inferDelegationActionFromMessage(message, { source = 'command-center',
 function _ensureDelegationForIntent(actions, opts = {}) {
   const list = (Array.isArray(actions) ? actions.map(normalizeCCAction) : [])
     .filter(action => !_ccShouldSuppressAnsweredAskDispatch(action, opts));
-  if (_messageRequestsDirectHandling(opts.message)) return list.filter(a => normalizeCCAction(a)?.type !== 'dispatch');
+  if (_messageRequestsDirectHandling(opts.message, opts)) return list.filter(a => normalizeCCAction(a)?.type !== 'dispatch');
   if (list.some(a => normalizeCCAction(a)?.type === 'dispatch')) return list;
   if (list.length > 0) return list;
   const inferred = _inferDelegationActionFromMessage(opts.message, opts);
@@ -3325,6 +3385,16 @@ async function _ccExecuteLocalApiAction(action) {
   };
 }
 
+function _ccActionFailureLabel(action) {
+  const type = String(action?.type || 'action').trim() || 'action';
+  const title = String(action?.title || action?.id || action?.file || action?.target || '').replace(/\s+/g, ' ').trim();
+  return title ? `${type} failed for '${title.slice(0, 120)}'` : `${type} failed`;
+}
+
+function _ccMultiProjectRequiredError(action) {
+  return `${_ccActionFailureLabel(action)}: project field is required when ${PROJECTS.length} projects are configured: ${PROJECTS.map(p => p.name).join(', ')}`;
+}
+
 async function executeCCActions(actions, { source = 'command-center', inferredProject = null } = {}) {
   const results = [];
   const dispatchIdsCreatedInThisCall = new Map();
@@ -3351,8 +3421,15 @@ async function executeCCActions(actions, { source = 'command-center', inferredPr
           const workType = routing.normalizeWorkType(action.workType || (action.type !== 'dispatch' ? action.type : WORK_TYPE.IMPLEMENT), WORK_TYPE.IMPLEMENT);
           const id = 'W-' + shared.uid();
           const project = action.project || '';
-          const prRef = getWorkItemPrRef(action);
+          const prTargetedWorkType = isPrTargetedWorkType(workType);
+          let prRef = getWorkItemPrRef(action);
           let linkedPr = null;
+          let allPrsForAction = null;
+          if (!prRef && prTargetedWorkType) {
+            allPrsForAction = getPullRequests().filter(p => !p._ghost);
+            linkedPr = inferActionPrRecord(action, allPrsForAction, null);
+            if (linkedPr) prRef = linkedPr.id || linkedPr.url || linkedPr.prNumber;
+          }
 
           // Strict project resolution. Silent fallback to PROJECTS[0] when the model named an unknown
           // project caused work items to land in the wrong repo. Now: unknown name → error; ambiguous
@@ -3367,8 +3444,8 @@ async function executeCCActions(actions, { source = 'command-center', inferredPr
               break;
             }
           } else if (prRef) {
-            const allPrs = getPullRequests().filter(p => !p._ghost);
-            linkedPr = shared.findPrRecord(allPrs, prRef) || null;
+            const allPrs = allPrsForAction || getPullRequests().filter(p => !p._ghost);
+            if (!linkedPr) linkedPr = shared.findPrRecord(allPrs, prRef) || null;
             if (linkedPr?._project && linkedPr._project !== 'central') {
               targetProject = resolveProjectSourceTarget(linkedPr._project, PROJECTS, { allowCentral: false }).project || null;
             }
@@ -3379,19 +3456,24 @@ async function executeCCActions(actions, { source = 'command-center', inferredPr
           }
           if (!targetProject) {
             if (PROJECTS.length > 1) {
-              results.push({ type: action.type, error: `project field is required when ${PROJECTS.length} projects are configured: ${PROJECTS.map(p => p.name).join(', ')}` });
+              results.push({ type: action.type, error: _ccMultiProjectRequiredError(action) });
               break;
             }
             if (PROJECTS.length === 1) targetProject = PROJECTS[0];
           }
           // PROJECTS.length === 0 → targetProject stays null, falls back to root work-items.json (existing behavior).
 
-          if (prRef && !linkedPr && targetProject) {
+          if (targetProject && (!linkedPr || !prRef)) {
             const projectPrs = shared.safeJson(shared.projectPrPath(targetProject)) || [];
             shared.normalizePrRecords(projectPrs, targetProject);
-            linkedPr = shared.findPrRecord(projectPrs, prRef, targetProject) || null;
+            if (!prRef && prTargetedWorkType) {
+              linkedPr = inferActionPrRecord(action, projectPrs, targetProject);
+              if (linkedPr) prRef = linkedPr.id || linkedPr.url || linkedPr.prNumber;
+            } else if (prRef && !linkedPr) {
+              linkedPr = shared.findPrRecord(projectPrs, prRef, targetProject) || null;
+            }
           }
-          if (prRef && (workType === WORK_TYPE.FIX || workType === WORK_TYPE.REVIEW || workType === WORK_TYPE.TEST) && !linkedPr) {
+          if (prRef && prTargetedWorkType && !linkedPr) {
             results.push({ type: action.type, error: `PR not found: ${prRef}` });
             break;
           }
@@ -3541,7 +3623,7 @@ async function executeCCActions(actions, { source = 'command-center', inferredPr
           }
           if (!targetProject) {
             if (PROJECTS.length > 1) {
-              results.push({ type: 'reopen-work-item', id: action.id, error: `project field is required when ${PROJECTS.length} projects are configured: ${PROJECTS.map(p => p.name).join(', ')}` });
+              results.push({ type: 'reopen-work-item', id: action.id, error: _ccMultiProjectRequiredError({ ...action, type: 'reopen-work-item' }) });
               break;
             }
             if (PROJECTS.length === 1) targetProject = PROJECTS[0];
@@ -7263,6 +7345,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
             await executeCCActions(parsed.actions),
             { message: body.message, requestedAt: actionRequestedAt }
           );
+          parsed.actionResultsAt = new Date().toISOString();
         }
         // Mirror only user-facing text to Teams; never send the internal action block.
         if (!tabId.startsWith('teams-')) {
@@ -7594,14 +7677,16 @@ What would you like to discuss or change? When you're happy, say "approve" and I
           { message: body.message, intentMetadata: body.intentMetadata, source: 'command-center', answerText: displayText, toolUses }
         );
         let actionResults;
+        let actionResultsAt;
         if (actions.length > 0) {
           actionResults = _annotateCCActionResults(
             actions,
             await executeCCActions(actions),
             { message: body.message, requestedAt: actionRequestedAt }
           );
+          actionResultsAt = new Date().toISOString();
         }
-        const donePayload = { type: 'done', text: displayText, actions, actionResults, sessionId: responseSessionId, newSession: !wasResume };
+        const donePayload = { type: 'done', text: displayText, actions, actionResults, actionResultsAt, sessionId: responseSessionId, newSession: !wasResume };
         // Issue #1834: surface action JSON parse failures so the UI can warn
         // instead of silently dropping. Client renders this as a small notice.
         if (_actionParseError) donePayload.actionParseError = _actionParseError;

package/engine/copilot-models.json

@@ -1,5 +1,5 @@
 {
   "runtime": "copilot",
   "models": null,
-  "cachedAt": "2026-05-09T17:43:03.560Z"
+  "cachedAt": "2026-05-09T18:11:55.798Z"
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.1820",
+  "version": "0.1.1822",
   "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/playbooks/fix.md

@@ -11,6 +11,8 @@ Repo: {{repo_name}} | Org: {{ado_org}} | Project: {{ado_project}}
 Fix issues found by {{reviewer}} on **{{pr_id}}**: {{pr_title}}
 Branch: `{{pr_branch}}`
 
+When `{{pr_id}}`/`{{pr_branch}}` identify an existing pull request, this is a fix-on-existing-PR task. Work in the provided PR branch, push the fix commit to that branch, and do not create a replacement branch or a new PR.
+
 {{checkpoint_context}}
 
 ## Review Findings to Address

package/prompts/cc-system.md

@@ -97,7 +97,8 @@ I'll dispatch dallas to fix that bug.
 
 Core action types:
 - **dispatch**: title (REQUIRED), workType, priority (low/medium/high), agents[] or agent (optional — both shapes accepted), project (REQUIRED when multi-project unless `pr` resolves to a tracked PR), description, pr (optional PR number/id/url for work that targets an existing PR), scope (`"fan-out"` only when the user explicitly asks to fan out to all agents). Do not emit `type:"fix"` or `type:"implement"`; emit `type:"dispatch"` with `workType:"fix"` or emit `type:"dispatch"` with `workType:"implement"`.
-  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)
+  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` whenever the user references an existing PR by number, URL, or title, including "fix/bypass this policy on this PR", review comments, or build failures), `review` (code review, NO PR), `test` (tests, PR if new), `verify` (merge/build/maintenance, NO PR)
+  For fix-style requests that name or contextually refer to an existing PR, preserving `pr` is mandatory: the agent must check out that PR's branch and push a fix commit there, not create a new branch or new PR.
   Do not dispatch an `ask` work item for a question you already answered inline in the same turn, especially after using tools. If the answer needs deeper investigation, give a brief handoff note and dispatch; do not both provide a complete answer and queue an agent to repeat it.
   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"`.