@yemi33/minions 0.1.1821 → 0.1.1823

package/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## 0.1.1823 (2026-05-09)
+
+### Other
+- docs(copilot): document test runner specifics for agents
+
 ## 0.1.1821 (2026-05-09)
 
 ### Features

package/dashboard/js/command-center.js

@@ -20,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.

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);
@@ -3361,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
@@ -3377,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;
             }
@@ -3396,12 +3463,17 @@ async function executeCCActions(actions, { source = 'command-center', inferredPr
           }
           // 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;
           }

package/engine/copilot-models.json

@@ -1,5 +1,5 @@
 {
   "runtime": "copilot",
   "models": null,
-  "cachedAt": "2026-05-09T18:10:43.397Z"
+  "cachedAt": "2026-05-09T18:16:37.444Z"
 }

package/package.json

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