@yemi33/minions 0.1.1819 → 0.1.1821

package/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## 0.1.1821 (2026-05-09)
+
+### Features
+- correlate CC action failures (#2264)
+
 ## 0.1.1819 (2026-05-09)
 
 ### Fixes

package/bin/minions.js

@@ -11,7 +11,7 @@
  *   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 restart [--open]         Start engine + dashboard (--open forces a new browser tab)
  *   minions start                    Start the engine
  *   minions stop                     Stop the engine
  *   minions status                   Show engine status
@@ -256,6 +256,11 @@ const [cmd, ...rest] = process.argv.slice(2);
 let force = rest.includes('--force');
 const skipScan = rest.includes('--skip-scan');
 const skipStart = rest.includes('--skip-start') || rest.includes('--no-start');
+// `--open` (or env MINIONS_FORCE_OPEN=1) bypasses the recent-browser-tab heuristic
+// in restart/init and forces the new dashboard to auto-open a browser tab. Useful
+// when the heuristic mis-fires (e.g. you closed your dashboard tab seconds before
+// running restart and its final heartbeat was still inside the 45s window).
+const forceOpen = rest.includes('--open') || process.env.MINIONS_FORCE_OPEN === '1';
 const MINIONS_HOME = resolveMinionsHome(cmd === 'init');
 process.env.MINIONS_HOME = MINIONS_HOME;
 const POST_UPDATE_INIT_TIMEOUT_MS = 120000;
@@ -410,7 +415,7 @@ function init() {
   // was recently polling the dashboard. A bare/orphan dashboard process on the
   // port is not enough; cold starts should still open the UI.
   const dashWasUp = isPortListening(DASH_PORT);
-  const suppressDashboardOpen = dashWasUp && hasRecentDashboardBrowserTab(MINIONS_HOME);
+  const suppressDashboardOpen = !forceOpen && dashWasUp && hasRecentDashboardBrowserTab(MINIONS_HOME);
   if (isUpgrade) {
     try { execSync(`node "${path.join(MINIONS_HOME, 'engine.js')}" stop`, { stdio: 'ignore', cwd: MINIONS_HOME, timeout: 10000, windowsHide: true }); } catch {}
     // Free the dashboard port too — without this the new dashboard EADDRINUSE-dies
@@ -624,7 +629,7 @@ if (!cmd || cmd === 'help' || cmd === '--help' || cmd === '-h') {
     minions list                     List linked projects
 
   Engine:
-    minions restart                   Start engine + dashboard (use after reboot)
+    minions restart [--open]         Start engine + dashboard (use after reboot; --open forces a new browser tab)
     minions start                    Start engine daemon only
     minions stop                     Stop the engine
     minions status                   Show agents, projects, queue
@@ -688,7 +693,7 @@ if (!cmd || cmd === 'help' || cmd === '--help' || cmd === '-h') {
   // was recently polling the dashboard. A bare/orphan dashboard process on the
   // port is not enough; cold starts should still open the UI.
   const dashWasUp = isPortListening(DASH_PORT);
-  const suppressDashboardOpen = dashWasUp && hasRecentDashboardBrowserTab(MINIONS_HOME);
+  const suppressDashboardOpen = !forceOpen && dashWasUp && hasRecentDashboardBrowserTab(MINIONS_HOME);
   // Layered kill — each step is best-effort, layered so the next still runs if
   // one fails. Goal: the old engine is gone before we spawn a new one, even if
   // PowerShell is unavailable, the engine is hung, or its cmdline doesn't match.

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)
@@ -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

@@ -3325,6 +3325,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();
@@ -3379,7 +3389,7 @@ 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];
@@ -3541,7 +3551,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 +7273,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 +7605,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:41:50.994Z"
+  "cachedAt": "2026-05-09T18:10:43.397Z"
 }

package/package.json

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