@yemi33/minions 0.1.1815 → 0.1.1817

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 0.1.1817 (2026-05-09)
+
+### Features
+- suppress duplicate inline ask dispatch (#2256)
+- correlate CC action failures (#2254)
+
+### Other
+- test(shared): add unit tests for project resolution and path comparison helpers (#2268)
+- test(cli): add unit tests for pid/dispatch helpers and help formatter (#2267)
+
 ## 0.1.1815 (2026-05-09)
 
 ### Features

package/dashboard/js/command-center.js

@@ -826,11 +826,15 @@ async function _ccDoSend(message, skipUserMsg, forceTabId, intentMetadata) {
             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 || 'action') + ': ' + escHtml(r.error) + '</li>'; }).join('');
+              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 || 'action') + ': ' + escHtml(r.warning) + '</li>'; }).join('');
+              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);
             }
           }
@@ -1012,6 +1016,28 @@ async function _ccFetch(url, body, method) {
   return res;
 }
 
+var CC_ACTION_CONTEXT_STALE_MS = 2 * 60 * 1000;
+function _ccActionContextIsStale(ctx, nowMs) {
+  if (!ctx || !ctx.requestedAt) return false;
+  var ts = Date.parse(ctx.requestedAt);
+  if (!Number.isFinite(ts)) return false;
+  var now = Number.isFinite(nowMs) ? nowMs : Date.now();
+  return now - ts > CC_ACTION_CONTEXT_STALE_MS;
+}
+
+function _ccActionContextSuffix(ctx, nowMs) {
+  if (!ctx || typeof ctx !== 'object') return '';
+  var parts = [];
+  if (ctx.title) parts.push('action "' + escHtml(ctx.title) + '"');
+  if (ctx.request) parts.push('request "' + escHtml(ctx.request) + '"');
+  if (ctx.requestedAt) parts.push('started ' + escHtml(ctx.requestedAt));
+  if (parts.length === 0) return '';
+  var stale = _ccActionContextIsStale(ctx, nowMs)
+    ? ' <strong style="color:var(--orange)">possibly stale</strong>'
+    : '';
+  return '<div style="font-size:10px;color:var(--muted);margin-top:2px">Context: ' + parts.join(' | ') + stale + '</div>';
+}
+
 // Tag actions that the server already executed so ccExecuteAction skips the API call
 function _tagServerExecuted(actions, actionResults) {
   if (!actionResults || !Array.isArray(actionResults)) return;
@@ -1022,10 +1048,12 @@ function _tagServerExecuted(actions, actionResults) {
       if (r.id) actions[i]._serverId = r.id;
       if (r.warning) actions[i]._serverWarning = r.warning;
       if (r.duplicate) actions[i]._serverDuplicate = true;
+      if (r.actionContext) actions[i]._serverContext = r.actionContext;
       if (r.reusedFromAction !== undefined) actions[i]._serverHidden = true;
     } else if (r && r.error) {
       actions[i]._serverExecuted = true;
       actions[i]._serverError = r.error;
+      if (r.actionContext) actions[i]._serverContext = r.actionContext;
     }
     // clientExecuted: false means server didn't handle it — frontend must execute
   }
@@ -1040,7 +1068,8 @@ async function ccExecuteAction(action, targetTabId) {
   if (action._serverExecuted) {
     if (action._serverHidden) return;
     if (action._serverError) {
-      status.innerHTML = '&#10007; ' + escHtml(action.type) + ' failed: ' + escHtml(action._serverError);
+      status.innerHTML = '&#10007; ' + escHtml(action.type) + ' failed: ' + escHtml(action._serverError) +
+        _ccActionContextSuffix(action._serverContext);
       status.style.color = 'var(--red)';
     } else {
       var label = action._serverId ? escHtml(action._serverId) : escHtml(action.title || action.type);
@@ -1048,7 +1077,8 @@ async function ccExecuteAction(action, targetTabId) {
       var successLabel = serverActionType === 'dispatch' ? 'Dispatched' : serverActionType;
       status.innerHTML = '&#10003; ' + escHtml(successLabel) + ': <strong>' + label + '</strong>' +
         (action._serverDuplicate ? '<div style="font-size:10px;color:var(--orange);margin-top:2px">Already existed from a previous request; no duplicate work item was created.</div>' : '') +
-        (action._serverWarning ? '<div style="font-size:10px;color:var(--muted);margin-top:2px">' + escHtml(action._serverWarning) + '</div>' : '');
+        (action._serverWarning ? '<div style="font-size:10px;color:var(--muted);margin-top:2px">' + escHtml(action._serverWarning) + '</div>' : '') +
+        _ccActionContextSuffix(action._serverContext);
       status.style.color = 'var(--green)';
     }
     ccAddMessage('action', status.outerHTML, false, targetTabId);

package/dashboard.js

@@ -2163,7 +2163,8 @@ function _ccFallbackMissingTargetError(intent) {
 
 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) : [];
+  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 (existing.some(a => normalizeCCAction(a)?.type === 'dispatch')) return existing;
   if (existing.length > 0) return existing;
@@ -2199,9 +2200,38 @@ function _actionsWithIntentFallback(actions, opts = {}) {
     ...common,
   };
   if (!hasTarget) action._intentFallbackError = _ccFallbackMissingTargetError(intent);
+  if (_ccShouldSuppressAnsweredAskDispatch(action, { message, source, answerText, toolUses })) return existing;
   return [action];
 }
 
+function _ccActionContextValue(action) {
+  if (!action || typeof action !== 'object') return '';
+  const candidates = [action.title, action.id, action.file, action.target, action.pr, action.endpoint];
+  for (const value of candidates) {
+    const cleaned = _ccCleanIntentString(value, 160);
+    if (cleaned) return cleaned;
+  }
+  return '';
+}
+
+function _annotateCCActionResults(actions, actionResults, opts = {}) {
+  if (!Array.isArray(actionResults)) return actionResults;
+  const requestedAt = _ccCleanIntentString(opts.requestedAt || new Date().toISOString(), 80);
+  const request = _ccCleanIntentString(opts.message, 180);
+  return actionResults.map((result, idx) => {
+    if (!result || typeof result !== 'object') return result;
+    const action = Array.isArray(actions) ? normalizeCCAction(actions[idx]) : null;
+    const context = {
+      type: _ccCleanIntentString(result.type || action?.type || 'action', 80) || 'action',
+    };
+    const title = _ccActionContextValue(action);
+    if (title) context.title = title;
+    if (request) context.request = request;
+    if (requestedAt) context.requestedAt = requestedAt;
+    return { ...result, actionContext: context };
+  });
+}
+
 function parseCCActions(text) {
   let actions = [];
   let displayText = stripCCActionsForDisplay(text);
@@ -2258,6 +2288,18 @@ function parseCCActions(text) {
 }
 
 const DELEGATION_ACTION_TERMS = ['dispatch', 'delegate', 'assign', 'queue', 'enqueue'];
+const DELEGATION_REQUEST_PREFIX_TERMS = ['please', 'can', 'could', 'would', 'will', 'should', 'you', 'cc', 'minions', 'minion'];
+const DELEGATION_PERSON_ACTION_TERMS = ['ask', 'tell', 'have'];
+// Subset of delegation action terms that double as common nouns in status
+// questions (e.g. "queue status", "dispatch contents"). When followed by a
+// status-noun follow-on these are noun usage, not imperative delegation.
+const DELEGATION_AMBIGUOUS_NOUN_TERMS = new Set(['dispatch', 'queue']);
+const DELEGATION_NON_VERB_FOLLOW_ONS = new Set([
+  'status', 'contents', 'content', 'state', 'count', 'list', 'items', 'item',
+  'size', 'length', 'depth', 'health', 'info', 'order', 'queue', 'history',
+  'log', 'logs', 'summary', 'overview', 'report', 'is', 'was', 'are', 'were',
+  'has', 'have', 'looks', 'shows',
+]);
 const DELEGATION_MINIONS_PHRASES = ['have minions', 'ask minions', 'tell minions', 'hand off', 'hand it off', 'hand this off', 'send to agent', 'send it to agent', 'send this to agent'];
 const MEDIUM_INVESTIGATION_TERMS = ['audit', 'investigate', 'research', 'explore', 'analyze', 'analyse'];
 const MEDIUM_INVESTIGATION_PHRASES = ['deep dive', 'root cause'];
@@ -2288,6 +2330,15 @@ const DIRECT_HANDLING_PHRASES = [
   'no dispatch', 'no delegate', 'no delegation', 'no work item',
   'without dispatching', 'without delegating', 'without creating work item',
 ];
+const ANSWERED_ASK_MIN_CHARS = 80;
+const ANSWERED_ASK_INCOMPLETE_PHRASES = [
+  'i will dispatch', 'ill dispatch', 'i will delegate', 'ill delegate',
+  'i will ask', 'ill ask', 'need to dispatch', 'needs to dispatch',
+  'need to delegate', 'needs to delegate', 'need an agent', 'needs an agent',
+  'hand off', 'hand this off', 'deeper investigation', 'further investigation',
+  'cannot answer', 'cant answer', 'could not answer', 'couldnt answer',
+  'not enough information',
+];
 
 function _isIntentWordChar(ch) {
   const code = ch.charCodeAt(0);
@@ -2373,6 +2424,37 @@ function _messageHasDelegationIntent(message) {
   return _intentHasAnyPhrase(normalized, DELEGATION_MINIONS_PHRASES);
 }
 
+function _delegationTokenIsImperativeVerb(tokens, idx) {
+  const term = tokens[idx];
+  if (!term) return false;
+  if (!DELEGATION_AMBIGUOUS_NOUN_TERMS.has(term)) return true;
+  const next = tokens[idx + 1];
+  if (!next) return true;
+  return !DELEGATION_NON_VERB_FOLLOW_ONS.has(next);
+}
+
+function _messageExplicitlyRequestsDelegation(message) {
+  const normalized = _normalizeIntentText(message);
+  const tokens = _intentTokens(normalized);
+  if (!tokens.length) return false;
+  if (_intentHasAnyPhrase(normalized, DELEGATION_MINIONS_PHRASES)) return true;
+  if (_intentHasPhrase(normalized, 'work item') && _intentHasAnyToken(normalized, ['create', 'open', 'add'])) return true;
+
+  let i = 0;
+  while (DELEGATION_REQUEST_PREFIX_TERMS.includes(tokens[i])) i++;
+  if (DELEGATION_ACTION_TERMS.includes(tokens[i]) && _delegationTokenIsImperativeVerb(tokens, i)) return true;
+  if (DELEGATION_PERSON_ACTION_TERMS.includes(tokens[i])) {
+    return tokens.slice(i + 1, i + 7).includes('to');
+  }
+  for (let j = 0; j < tokens.length && j <= 6; j++) {
+    if (!DELEGATION_ACTION_TERMS.includes(tokens[j])) continue;
+    if (!_delegationTokenIsImperativeVerb(tokens, j)) continue;
+    const prefix = tokens.slice(0, j);
+    if (prefix.includes('you') || prefix.includes('minions') || prefix.includes('minion')) return true;
+  }
+  return false;
+}
+
 function _messageRequestsDirectHandling(message) {
   const normalized = _normalizeIntentText(message);
   if (!normalized.trim()) return false;
@@ -2412,6 +2494,24 @@ function _inferDelegatedWorkType(message) {
   return 'ask';
 }
 
+function _ccAnswerLooksCompleteForAsk(answerText, toolUses = []) {
+  if (!Array.isArray(toolUses) || toolUses.length === 0) return false;
+  const answer = _collapseWhitespace(stripCCActionSyntax(answerText || ''));
+  if (answer.length < ANSWERED_ASK_MIN_CHARS) return false;
+  const normalized = _normalizeIntentText(answer);
+  if (_intentHasAnyPhrase(normalized, ANSWERED_ASK_INCOMPLETE_PHRASES)) return false;
+  if (_intentHasVerbObject(normalized, ['need', 'needs', 'require', 'requires'], ['agent', 'delegation', 'investigation', 'research'])) return false;
+  return true;
+}
+
+function _ccShouldSuppressAnsweredAskDispatch(action, opts = {}) {
+  const normalized = normalizeCCAction(action);
+  if (normalized?.type !== 'dispatch') return false;
+  if (String(normalized.workType || '').trim().toLowerCase() !== 'ask') return false;
+  if (_messageExplicitlyRequestsDelegation(opts.message)) return false;
+  return _ccAnswerLooksCompleteForAsk(opts.answerText, opts.toolUses);
+}
+
 function _collapseWhitespace(text) {
   let out = '';
   let lastWasSpace = true;
@@ -2504,11 +2604,13 @@ function _inferDelegationActionFromMessage(message, { source = 'command-center',
 }
 
 function _ensureDelegationForIntent(actions, opts = {}) {
-  const list = Array.isArray(actions) ? actions.map(normalizeCCAction) : [];
+  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 (list.some(a => normalizeCCAction(a)?.type === 'dispatch')) return list;
   if (list.length > 0) return list;
   const inferred = _inferDelegationActionFromMessage(opts.message, opts);
+  if (_ccShouldSuppressAnsweredAskDispatch(inferred, opts)) return list;
   return inferred ? [...list, inferred] : list;
 }
 
@@ -7100,6 +7202,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
     try {
       const body = await readBody(req);
       if (!body.message) return jsonReply(res, 400, { error: 'message required' });
+      const actionRequestedAt = new Date().toISOString();
 
       // Per-tab concurrency guard
       tabId = body.tabId || 'default';
@@ -7155,7 +7258,11 @@ 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: parsed.text, toolUses }
         );
         if (parsed.actions.length > 0) {
-          parsed.actionResults = await executeCCActions(parsed.actions);
+          parsed.actionResults = _annotateCCActionResults(
+            parsed.actions,
+            await executeCCActions(parsed.actions),
+            { message: body.message, requestedAt: actionRequestedAt }
+          );
         }
         // Mirror only user-facing text to Teams; never send the internal action block.
         if (!tabId.startsWith('teams-')) {
@@ -7259,6 +7366,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       const body = await readBody(req);
       if (!body.message && !body.reconnect) { res.statusCode = 400; res.end('message required'); return; }
       tabId = body.tabId || 'default';
+      const actionRequestedAt = new Date().toISOString();
       if (body.reconnect) {
         const live = _getCcLiveStream(tabId);
         if (!live) { res.statusCode = 409; res.end('No live command-center response to reconnect'); return; }
@@ -7487,7 +7595,11 @@ What would you like to discuss or change? When you're happy, say "approve" and I
         );
         let actionResults;
         if (actions.length > 0) {
-          actionResults = await executeCCActions(actions);
+          actionResults = _annotateCCActionResults(
+            actions,
+            await executeCCActions(actions),
+            { message: body.message, requestedAt: actionRequestedAt }
+          );
         }
         const donePayload = { type: 'done', text: displayText, actions, actionResults, sessionId: responseSessionId, newSession: !wasResume };
         // Issue #1834: surface action JSON parse failures so the UI can warn

package/engine/cli.js

@@ -1531,4 +1531,10 @@ module.exports = {
   _controlBelongsToOwner: controlBelongsToOwner,
   _markControlStoppingForOwner: markControlStoppingForOwner,
   _markControlStoppedForOwner: markControlStoppedForOwner,
+  _isPidAlive: isPidAlive,
+  _summarizeActiveDispatchPids: summarizeActiveDispatchPids,
+  _dispatchSafeId: dispatchSafeId,
+  _readDispatchPid: readDispatchPid,
+  _normalizeSessionBranch: normalizeSessionBranch,
+  _dispatchSessionBranch: dispatchSessionBranch,
 };

package/engine/copilot-models.json

@@ -1,5 +1,5 @@
 {
   "runtime": "copilot",
   "models": null,
-  "cachedAt": "2026-05-09T15:24:57.566Z"
+  "cachedAt": "2026-05-09T15:29:07.860Z"
 }

package/engine/shared.js

@@ -3451,6 +3451,10 @@ module.exports = {
   legacyProjectStatePath,
   ensureProjectStateFiles,
   sameResolvedPath,
+  projectStateRecordKey, // exported for testing
+  mergeProjectStateArrays, // exported for testing
+  realPathForComparison, // exported for testing
+  prPathComparisonCandidates, // exported for testing
   resolveProjectForPrPath, // exported for testing
   getPrLinks,
   addPrLink,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.1815",
+  "version": "0.1.1817",
   "description": "Multi-agent AI dev team that runs from ~/.minions/ — five autonomous agents share a single engine, dashboard, and knowledge base",
   "bin": {
     "minions": "bin/minions.js"

package/prompts/cc-system.md

@@ -98,6 +98,7 @@ 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)
+  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"`.
   After emitting a dispatch-like action, return immediately; do not poll, monitor, watch, wait, or check until completion, and do not add follow-up status actions. Only create a watch, monitor, poll, or periodically check when the human explicitly asks for monitoring, watching, periodic checks, or notification on completion.