@zhixuan92/multi-model-agent-core 4.0.6 → 4.2.0

package/dist/tools/investigate/implementer-criteria.js

@@ -1,33 +1,129 @@
 /**
  * Investigate-specific implementer criteria.
  *
+ * INVESTIGATE'S PURPOSE — read this before adding categories.
+ * mma-investigate answers a question about the codebase. The caller is
+ * about to ACT on your answer — write code, edit a file, choose between
+ * approaches. The success criterion is:
+ *
+ *   "If the caller acts on this answer literally — opens the cited
+ *    files, follows the cited chain, takes the synthesis at face value
+ *    — will they end up with correct code?"
+ *
+ * That criterion is what makes a finding load-bearing. A wrong file
+ * path, a stale quote, a hand-waved synthesis, an overstated confidence
+ * — all become bugs the caller writes. The investigate-equivalent of
+ * "fix is unimplementable" is "the answer points at a file that does
+ * not contain what you said it contained."
+ *
  * Investigate answers a question about the codebase. Findings can be
  * code-level citations, project-level synthesis, or NEGATIVE results
- * ("searched X, not found"). The shared evidence rule that demands a
- * code quote for every finding wrongly suppresses negative results,
- * which are often the most useful answer to "is X still used?" or
- * "where does Y live?".
+ * ("searched X, not found"). Negative findings are legitimate answers
+ * to "is X still used?" or "where does Y live?" and must not be
+ * suppressed.
  *
  * Note: investigate does NOT use SEVERITY_LADDER — its findings are
- * citations and synthesis, not severity-rated issues.
+ * citations and synthesis, not severity-rated issues. Confidence is the
+ * calibration dial, not severity.
+ */
+/**
+ * The orientation block. Goes at the TOP of every investigate prompt.
+ *
+ * Without an explicit purpose statement, workers default to "give a
+ * plausible-sounding answer" — which produces hallucinated citations
+ * and overstated confidence. With this orientation, every claim is
+ * ground-truthed against the file system.
  */
+export const INVESTIGATE_PURPOSE_ORIENTATION = [
+    'Why this investigation exists:',
+    'mma-investigate is the answer-and-act loop. The caller will use your answer to make code edits — open the cited files, take the synthesis at face value, choose an approach based on your confidence rating. A wrong file path becomes a bug; a stale quote becomes a wrong edit; an overstated confidence becomes a misallocated effort.',
+    '',
+    'For your output to clear that bar, every load-bearing claim must answer:',
+    '- Where exactly is this — file:line for present things, or "searched <pattern> in <path>, not found" for absent things?',
+    '- Did I read the file just now, or am I reasoning from training data? (only the former counts as evidence)',
+    '- For synthesis claims (e.g. "X is used by Y via Z"), is each link in the chain backed by a file:line?',
+    '- Is my confidence calibrated to evidence strength, or to how certain I sound?',
+    '',
+    'A claim without a citation is a guess. A citation that does not match the file currently on disk is a hallucination. A "high confidence" verdict on a synthesis with one weak link is overstatement.',
+    '',
+    'The completion test: would a caller who reads only your investigation report and the named files end up with the same answer if they re-investigated themselves — or would they find the cited file does not say what you said it said?',
+].join('\n');
 export const EVIDENCE_RULE_INVESTIGATE = [
     'Evidence grounding (REQUIRED for every citation):',
-    '- For present things: `file:line` (or `file:line-line` for spans) plus a quote or summary of what you found.',
+    '- For present things: `file:line` (or `file:line-line` for spans) plus a quote or summary of what you found. The cited line MUST contain the cited content as of your read — do NOT cite from training-data memory.',
     '- For absent things: explicit `searched <pattern> in <path>, no matches` — negative findings are legitimate answers and should be emitted, not suppressed.',
-    '- For synthesis findings (e.g. "X uses Y indirectly via Z"): cite each link in the chain by `file:line`.',
+    '- For synthesis findings (e.g. "X uses Y indirectly via Z"): cite each link in the chain by `file:line`. A synthesis claim with even one un-cited link is a hand-wave.',
+    '- For project-level claims that no single file demonstrates (e.g. "the codebase has no shared error type"): write the negative ("searched the repo for `class.*Error` declarations: only X, Y, Z found, none shared") rather than asserting the absence without evidence.',
+    '- If you have not read a file, do NOT cite from it. Reasoning-from-training-data is the most common hallucination source — refuse it explicitly.',
 ].join('\n');
 export const SCOPE_RULE_INVESTIGATE = [
     'Scope:',
     '- Wherever the question leads. The question may not name files; you choose where to look.',
-    '- Out of scope: drift into issues unrelated to the question; opportunistic code review of the code you are investigating (raise that separately, not as an investigation finding).',
+    '- If the question is broad (e.g. "how does X work overall?"), break it into sub-questions and answer each with citations rather than producing one un-grounded narrative.',
+    '- Out of scope: drift into issues unrelated to the question; opportunistic code review of the code you are investigating (raise that separately, not as an investigation finding); fixes / suggestions / improvements (this is a read-only Q&A — propose nothing).',
+].join('\n');
+/**
+ * The failure-mode taxonomy for investigations.
+ *
+ * Without this block, workers tend to give plausible-sounding answers
+ * with shaky citations. The 8 categories below are the specific ways
+ * an investigation answer becomes a bug when the caller acts on it.
+ */
+export const INVESTIGATE_FAILURE_MODES = [
+    'Five parallel perspectives for ANSWERING the user\'s question. From your assigned perspective, propose one or more candidate answers (or sub-answers) to the question. Each finding is a candidate answer; severity = your confidence in this answer. Pay extra care to AVOID common citation-quality failures (no hallucinated file:line; no quoting from memory — always re-read).',
+    '',
+    '1. DIRECT-SYMBOL-TRACE PERSPECTIVE — start from the symbols/files named in the question (or directly implied by it). Read the named file(s) top-to-bottom, follow imports/calls/types step-by-step. Your candidate answer is the chain of file:line references that, when followed in order, mechanically resolves the question.',
+    '2. CALLER-ANALYSIS PERSPECTIVE — grep for callers/consumers of the symbols in the question. Who depends on this code? What do they pass / expect / assert? Your candidate answer comes from the contract the callers assume — the question often resolves to "this code does X because callers depend on X".',
+    '3. TEST-DRIVEN PERSPECTIVE — find sibling tests for the symbols/files in question (test files often co-located or under tests/). Read what the tests assert about the behavior. Your candidate answer is "the tests show the intended behavior is X" — backed by test name + assertion citation.',
+    '4. CROSS-FILE DEPENDENCY-MAP PERSPECTIVE — what other modules participate in the data path / orchestration around the question? Map the boundary: which files import the named symbols, which configure them, which receive their output. Your candidate answer comes from the system-level picture — "the answer involves X handing off to Y, configured by Z".',
+    '5. DOCUMENTATION/COMMENT-LENS PERSPECTIVE — read docstrings, README, design docs, in-code comments adjacent to the symbols. Sometimes the answer is stated in prose by the original author. Your candidate answer is the prose explanation, cited; cross-check it against current code (docs may be stale).',
+    '',
+    'Confidence calibration for each candidate answer:',
+    '- critical: THE answer — multiple grounded file:line citations from THIS perspective, no inferred steps in the chain. The user can act on this without re-verification.',
+    '- high: strong answer — fully grounded with file:line, evidence chain has at most one inferred step. The user should sanity-check the inferred step.',
+    '- medium: likely / partial answer — cited inference from this perspective; some gaps in the chain. Mark "verify by reading <file>" so the user knows where to confirm.',
+    '- low: candidate / weak answer — minimal evidence from this perspective, presented as an alternative for the user to weigh against other perspectives\' answers. Better than silence — silence loses information the merge annotator could rank.',
+    '',
+    'Quality discipline (apply WITHIN your perspective, do not let it stop you from emitting):',
+    '- Always re-read files this session before quoting — never quote from memory.',
+    '- Verify every file:line citation actually exists on disk at that range.',
+    '- If your perspective produces multiple distinct candidate answers (e.g. caller-analysis surfaces two callers with conflicting expectations), emit BOTH as separate findings.',
+    '- Do not pad — but absent total impossibility, every perspective should produce at least one candidate (even low confidence). The merge annotator will dedup and rank.',
+].join('\n');
+/**
+ * Confidence-discipline reminder.
+ *
+ * The shared SEVERITY_LADDER does not apply to investigate (findings
+ * are citations, not severity-rated). Instead, confidence is the
+ * calibration dial. The common failure mode is over-confidence —
+ * stating "high confidence" because the worker sounds certain,
+ * not because the evidence is strong. This block tells the worker
+ * confidence reflects evidence strength only.
+ */
+export const CONFIDENCE_REMINDER_INVESTIGATE = [
+    'Confidence-discipline reminder:',
+    '- Confidence reflects EVIDENCE STRENGTH (how completely the citation chain supports the answer), not ASSERTION STRENGTH (how certain you sound).',
+    '- For each load-bearing claim, ask: "if the caller followed this citation and re-read the file themselves, would they reach the same conclusion?" If yes for every claim → `high`. If yes for most but inference fills the gaps → `medium`. If significant gaps remain → `low`.',
+    '- Do NOT use confidence to communicate certainty about the question being answered. Use it to communicate certainty that your answer is CORRECT given your evidence.',
+    '- A short investigation that found a clean answer can legitimately be `high`. A long investigation that found a partial answer is `medium` or `low`, no matter how thorough it felt.',
+    '',
+    'Citation-chain walk (REQUIRED on every load-bearing claim):',
+    '- Before writing the Summary, list every claim that drives the answer. For each, ask: "do I have a file:line for this, and did I read the file in this session?"',
+    '- If the answer to either is no, the claim is inference. Either downgrade Confidence, or add the citation by reading the file now.',
+    '- Worked example. Question: "how does the audit prompt assemble the failure-mode taxonomy?" Naive answer: "The audit tool config imports DOC_AUDIT_FAILURE_MODES from implementer-criteria.ts and joins it into the prompt — confidence: high." Better answer: cite the import line (e.g. `tool-config.ts:14 — import { DOC_AUDIT_FAILURE_MODES, ... }`) AND the consumer line where it is joined into the prompt (e.g. `tool-config.ts:152 — DOC_AUDIT_FAILURE_MODES,` inside the FINDING_FORMAT_INSTRUCTIONS array). Two citations, both verified by reading the file → high confidence is now backed. The naive version asserts the same conclusion but with no actual file:line; if the file has been refactored, the answer is silently wrong.',
+    '- Most workers miss findings of this shape on first pass because the answer "feels right". The citation-chain walk forces the file-system check.',
 ].join('\n');
 export const ANNOTATOR_AWARENESS_INVESTIGATE = [
     'After your output, an annotator validates each finding against this investigate rubric:',
-    '- Does each finding answer some part of the question?',
-    '- Are present-thing citations to real `file:line` from files you actually read?',
-    '- Are negative findings explicit ("searched X, not found") rather than silent omissions?',
-    '- Does the confidence reflect the strength of evidence?',
-    'Self-check before emitting. Findings that fail any check are downgraded or dropped.',
+    '- Does each citation answer some part of the question (not an adjacent question)?',
+    '- Are present-thing citations to real `file:line` from files actually read this session?',
+    '- Are negative findings explicit ("searched X in Y, not found") rather than silent omissions?',
+    '- For synthesis claims, is each link in the chain cited?',
+    '- Does the confidence reflect evidence strength (not assertion strength)?',
+    '- Is the answer to the asked question, not a shifted version of it?',
+    'Self-check before emitting. Findings that fail any check are downgraded or dropped — but negative findings ("searched, not found") and inference-with-citations ("I infer X from Y:42, Z:18") are FULLY VALID. Do NOT downgrade negative findings for lacking a code quote, and do NOT downgrade inference-with-citations as "speculation" if the cited links are real.',
 ].join('\n');
+import { parseCriteria } from '../criteria-types.js';
+/** Structured per-criterion array for parallel-criteria fan-out. */
+export const INVESTIGATE_CRITERIA = parseCriteria(INVESTIGATE_FAILURE_MODES);
 //# sourceMappingURL=implementer-criteria.js.map

package/dist/tools/investigate/implementer-criteria.js.map

@@ -1 +1 @@
-{"version":3,"file":"implementer-criteria.js","sourceRoot":"","sources":["../../../src/tools/investigate/implementer-criteria.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAEH,MAAM,CAAC,MAAM,yBAAyB,GAAG;IACvC,mDAAmD;IACnD,8GAA8G;IAC9G,4JAA4J;IAC5J,0GAA0G;CAC3G,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAEb,MAAM,CAAC,MAAM,sBAAsB,GAAG;IACpC,QAAQ;IACR,2FAA2F;IAC3F,oLAAoL;CACrL,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAEb,MAAM,CAAC,MAAM,+BAA+B,GAAG;IAC7C,yFAAyF;IACzF,uDAAuD;IACvD,iFAAiF;IACjF,0FAA0F;IAC1F,yDAAyD;IACzD,qFAAqF;CACtF,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC"}
+{"version":3,"file":"implementer-criteria.js","sourceRoot":"","sources":["../../../src/tools/investigate/implementer-criteria.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;GA2BG;AAEH;;;;;;;GAOG;AACH,MAAM,CAAC,MAAM,+BAA+B,GAAG;IAC7C,gCAAgC;IAChC,2UAA2U;IAC3U,EAAE;IACF,0EAA0E;IAC1E,yHAAyH;IACzH,4GAA4G;IAC5G,wGAAwG;IACxG,gFAAgF;IAChF,EAAE;IACF,sMAAsM;IACtM,EAAE;IACF,yOAAyO;CAC1O,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAEb,MAAM,CAAC,MAAM,yBAAyB,GAAG;IACvC,mDAAmD;IACnD,qNAAqN;IACrN,4JAA4J;IAC5J,wKAAwK;IACxK,2QAA2Q;IAC3Q,kJAAkJ;CACnJ,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAEb,MAAM,CAAC,MAAM,sBAAsB,GAAG;IACpC,QAAQ;IACR,2FAA2F;IAC3F,2KAA2K;IAC3K,oQAAoQ;CACrQ,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAEb;;;;;;GAMG;AACH,MAAM,CAAC,MAAM,yBAAyB,GAAG;IACvC,sXAAsX;IACtX,EAAE;IACF,kUAAkU;IAClU,8SAA8S;IAC9S,kSAAkS;IAClS,kWAAkW;IAClW,6SAA6S;IAC7S,EAAE;IACF,mDAAmD;IACnD,yKAAyK;IACzK,sJAAsJ;IACtJ,wKAAwK;IACxK,kPAAkP;IAClP,EAAE;IACF,2FAA2F;IAC3F,+EAA+E;IAC/E,0EAA0E;IAC1E,+KAA+K;IAC/K,wKAAwK;CACzK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAEb;;;;;;;;;GASG;AACH,MAAM,CAAC,MAAM,+BAA+B,GAAG;IAC7C,iCAAiC;IACjC,kJAAkJ;IAClJ,iRAAiR;IACjR,sKAAsK;IACtK,sLAAsL;IACtL,EAAE;IACF,6DAA6D;IAC7D,kKAAkK;IAClK,oIAAoI;IACpI,qtBAAqtB;IACrtB,kJAAkJ;CACnJ,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAEb,MAAM,CAAC,MAAM,+BAA+B,GAAG;IAC7C,yFAAyF;IACzF,mFAAmF;IACnF,0FAA0F;IAC1F,+FAA+F;IAC/F,0DAA0D;IAC1D,2EAA2E;IAC3E,qEAAqE;IACrE,yWAAyW;CAC1W,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAEb,OAAO,EAAE,aAAa,EAAuB,MAAM,sBAAsB,CAAC;AAE1E,oEAAoE;AACpE,MAAM,CAAC,MAAM,oBAAoB,GAA8B,aAAa,CAAC,yBAAyB,CAAC,CAAC"}

package/dist/tools/investigate/tool-config.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"tool-config.d.ts","sourceRoot":"","sources":["../../../src/tools/investigate/tool-config.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,mBAAmB,EAAE,MAAM,6CAA6C,CAAC;AAElF,OAAO,KAAK,EAAE,KAAK,EAAE,MAAM,aAAa,CAAC;AAEzC,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,sCAAsC,CAAC;AAGvE,OAAO,KAAK,EAAE,uBAAuB,EAAE,MAAM,2DAA2D,CAAC;AAUzG,wBAAgB,mBAAmB,CAAC,QAAQ,EAAE,mBAAmB,GAAG,IAAI,CAYvE;AAKD,MAAM,WAAW,oBAAoB;IACnC,EAAE,EAAE,MAAM,CAAC;IACX,OAAO,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,wBAAyB,SAAQ,KAAK;IACrD,qBAAqB,EAAE,oBAAoB,EAAE,CAAC;IAC9C,sBAAsB,EAAE,MAAM,EAAE,CAAC;IACjC,0BAA0B,EAAE,MAAM,EAAE,CAAC;CACtC;AAED,MAAM,WAAW,gBAAgB;IAC/B,+DAA+D;IAC/D,QAAQ,EAAE,MAAM,CAAC;IACjB;;;;;;;;;;OAUG;IACH,cAAc,EAAE,MAAM,CAAC;IACvB,SAAS,EAAE,MAAM,EAAE,CAAC;IACpB,eAAe,EAAE,MAAM,EAAE,CAAC;IAC1B,KAAK,CAAC,EAAE,MAAM,GAAG,UAAU,CAAC;CAC7B;AAqDD,eAAO,MAAM,UAAU,EAAE,UAAU,CAAC,wBAAwB,EAAE,gBAAgB,EAAE,uBAAuB,CA2DtG,CAAC"}
+{"version":3,"file":"tool-config.d.ts","sourceRoot":"","sources":["../../../src/tools/investigate/tool-config.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,mBAAmB,EAAE,MAAM,6CAA6C,CAAC;AAElF,OAAO,KAAK,EAAE,KAAK,EAAE,MAAM,aAAa,CAAC;AAEzC,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,sCAAsC,CAAC;AAGvE,OAAO,KAAK,EAAE,uBAAuB,EAAE,MAAM,2DAA2D,CAAC;AAazG,wBAAgB,mBAAmB,CAAC,QAAQ,EAAE,mBAAmB,GAAG,IAAI,CAYvE;AAKD,MAAM,WAAW,oBAAoB;IACnC,EAAE,EAAE,MAAM,CAAC;IACX,OAAO,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,wBAAyB,SAAQ,KAAK;IACrD,qBAAqB,EAAE,oBAAoB,EAAE,CAAC;IAC9C,sBAAsB,EAAE,MAAM,EAAE,CAAC;IACjC,0BAA0B,EAAE,MAAM,EAAE,CAAC;CACtC;AAED,MAAM,WAAW,gBAAgB;IAC/B,+DAA+D;IAC/D,QAAQ,EAAE,MAAM,CAAC;IACjB;;;;;;;;;;OAUG;IACH,cAAc,EAAE,MAAM,CAAC;IACvB,SAAS,EAAE,MAAM,EAAE,CAAC;IACpB,eAAe,EAAE,MAAM,EAAE,CAAC;IAC1B,KAAK,CAAC,EAAE,MAAM,GAAG,UAAU,CAAC;CAC7B;AAkED,eAAO,MAAM,UAAU,EAAE,UAAU,CAAC,wBAAwB,EAAE,gBAAgB,EAAE,uBAAuB,CAgEtG,CAAC"}

package/dist/tools/investigate/tool-config.js

@@ -4,7 +4,7 @@ import { investigateReportSchema } from '../../reporting/report-parser-slots/inv
 import { investigateHeadlineTemplate } from '../../reporting/headline-templates/investigate.js';
 import { deriveInvestigateWorkerStatus } from '../../reporting/derive-investigate-status.js';
 import { DEFAULT_TASK_TIMEOUT_MS } from '../../config/schema.js';
-import { EVIDENCE_RULE_INVESTIGATE, SCOPE_RULE_INVESTIGATE, ANNOTATOR_AWARENESS_INVESTIGATE, } from './implementer-criteria.js';
+import { INVESTIGATE_PURPOSE_ORIENTATION, EVIDENCE_RULE_INVESTIGATE, SCOPE_RULE_INVESTIGATE, ANNOTATOR_AWARENESS_INVESTIGATE, INVESTIGATE_FAILURE_MODES, CONFIDENCE_REMINDER_INVESTIGATE, } from './implementer-criteria.js';
 export function registerInvestigate(registry) {
     registry.register({
         routeName: 'investigate',
@@ -20,6 +20,12 @@ export function registerInvestigate(registry) {
 }
 function compilePrompt(input) {
     const promptParts = [];
+    // Orientation goes FIRST — the worker needs to know why this
+    // investigation exists (caller will act on this answer; wrong file
+    // path becomes a bug) before reading the format spec / taxonomy.
+    // Without it, workers default to plausible-sounding answers with
+    // shaky citations.
+    promptParts.push(INVESTIGATE_PURPOSE_ORIENTATION);
     promptParts.push([
         'Produce an investigation report in this EXACT structured format. The deterministic',
         'parser extracts citations, confidence, and unresolved items by section — do NOT emit',
@@ -29,14 +35,15 @@ function compilePrompt(input) {
         'One paragraph stating the answer to the question, in plain prose.',
         '',
         '## Citations',
-        'One bullet per evidence item, in this exact format:',
-        '`- file/path.ts:LINE — claim` (em-dash, OR `--` is also accepted)',
-        'Use a `LINE-LINE` range when an evidence span covers multiple lines.',
-        'If the question is fully project-level (no code evidence applies), write `(none)`',
-        'on its own line — but only when Confidence is `low`.',
+        'One bullet per evidence item. Each bullet must start with `-` and contain `<file>:<LINE> — <claim>` (em-dash, OR `--` is also accepted). The parser tolerates an optional pair of backticks wrapping the path:line portion (e.g. `` `src/foo.ts:42` — claim ``) but the canonical form is without backticks.',
+        'Examples (use either form):',
+        '  - src/foo.ts:42 — claim about line 42',
+        '  - src/foo.ts:42-58 — claim about a span',
+        'Use a LINE-LINE range when an evidence span covers multiple lines.',
+        'If the question is fully project-level (no code evidence applies), write `(none)` on its own line — but only when Confidence is `low`.',
         '',
         '## Confidence',
-        'One of `high`, `medium`, or `low`, optionally followed by ` — <one-line rationale>`.',
+        'One of high, medium, or low, optionally followed by ` — <one-line rationale>`. Do NOT wrap the level in backticks; emit it as plain text.',
         '',
         '## Unresolved',
         'Optional bullets describing follow-up questions; write `(none)` if there are none.',
@@ -59,7 +66,7 @@ function compilePrompt(input) {
     // but evidence-grounding + scope-discipline + annotator-awareness
     // apply just as much. Workers that cite hallucinated lines or
     // speculate about unread files now have the rubric inline.
-    promptParts.push(EVIDENCE_RULE_INVESTIGATE, SCOPE_RULE_INVESTIGATE, ANNOTATOR_AWARENESS_INVESTIGATE);
+    promptParts.push(INVESTIGATE_FAILURE_MODES, CONFIDENCE_REMINDER_INVESTIGATE, EVIDENCE_RULE_INVESTIGATE, SCOPE_RULE_INVESTIGATE, ANNOTATOR_AWARENESS_INVESTIGATE);
     return promptParts.join('\n\n');
 }
 export const toolConfig = {
@@ -78,6 +85,11 @@ export const toolConfig = {
     },
     buildTaskSpec: (brief, ctx) => ({
         prompt: brief.compiledPrompt,
+        // Pure user question for the parallel-criteria dispatcher's cached
+        // prefix — avoids embedding the legacy ## Summary/## Citations format
+        // spec from compiledPrompt, which would compete with the per-sub-worker
+        // ## Finding N: format the dispatcher uses.
+        parallelTarget: `Question: ${brief.question}`,
         agentType: 'complex',
         reviewPolicy: 'quality_only',
         cwd: ctx.projectContext?.cwd ?? ctx.cwd,

package/dist/tools/investigate/tool-config.js.map

@@ -1 +1 @@
-{"version":3,"file":"tool-config.js","sourceRoot":"","sources":["../../../src/tools/investigate/tool-config.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,WAAW,EAAE,MAAM,aAAa,CAAC;AAE1C,OAAO,EAAE,0BAA0B,EAAE,MAAM,iCAAiC,CAAC;AAG7E,OAAO,EAAE,uBAAuB,EAAE,MAAM,2DAA2D,CAAC;AAEpG,OAAO,EAAE,2BAA2B,EAAE,MAAM,mDAAmD,CAAC;AAChG,OAAO,EAAE,6BAA6B,EAAE,MAAM,8CAA8C,CAAC;AAC7F,OAAO,EAAE,uBAAuB,EAAE,MAAM,wBAAwB,CAAC;AACjE,OAAO,EACL,yBAAyB,EACzB,sBAAsB,EACtB,+BAA+B,GAChC,MAAM,2BAA2B,CAAC;AAEnC,MAAM,UAAU,mBAAmB,CAAC,QAA6B;IAC/D,QAAQ,CAAC,QAAQ,CAAC;QAChB,SAAS,EAAE,aAAa;QACxB,UAAU,EAAE,MAAM;QAClB,QAAQ,EAAE,cAAc;QACxB,OAAO,EAAE,MAAM;QACf,MAAM,EAAE,WAAW;QACnB,YAAY,EAAE,WAAW;QACzB,gBAAgB,EAAE,SAAS;QAC3B,oBAAoB,EAAE,KAAK;QAC3B,iBAAiB,EAAE,eAAe;KACnC,CAAC,CAAC;AACL,CAAC;AAoCD,SAAS,aAAa,CAAC,KAA+B;IACpD,MAAM,WAAW,GAAa,EAAE,CAAC;IACjC,WAAW,CAAC,IAAI,CACd;QACE,oFAAoF;QACpF,sFAAsF;QACtF,sFAAsF;QACtF,EAAE;QACF,YAAY;QACZ,mEAAmE;QACnE,EAAE;QACF,cAAc;QACd,qDAAqD;QACrD,mEAAmE;QACnE,sEAAsE;QACtE,mFAAmF;QACnF,sDAAsD;QACtD,EAAE;QACF,eAAe;QACf,sFAAsF;QACtF,EAAE;QACF,eAAe;QACf,oFAAoF;QACpF,iFAAiF;QACjF,kDAAkD;KACnD,CAAC,IAAI,CAAC,IAAI,CAAC,CACb,CAAC;IACF,KAAK,MAAM,KAAK,IAAI,KAAK,CAAC,qBAAqB,EAAE,CAAC;QAChD,WAAW,CAAC,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;IAClC,CAAC;IACD,IAAI,KAAK,CAAC,0BAA0B,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAChD,WAAW,CAAC,IAAI,CACd,gEAAgE;YAChE,KAAK,CAAC,0BAA0B,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAC/D,CAAC;IACJ,CAAC;IACD,WAAW,CAAC,IAAI,CAAC,aAAa,KAAK,CAAC,QAAQ,EAAE,CAAC,CAAC;IAChD,IAAI,KAAK,CAAC,qBAAqB,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC3C,WAAW,CAAC,IAAI,CACd,sMAAsM,CACvM,CAAC;IACJ,CAAC;IACD,6DAA6D;IAC7D,mEAAmE;IACnE,kEAAkE;IAClE,8DAA8D;IAC9D,2DAA2D;IAC3D,WAAW,CAAC,IAAI,CAAC,yBAAyB,EAAE,sBAAsB,EAAE,+BAA+B,CAAC,CAAC;IACrG,OAAO,WAAW,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;AAClC,CAAC;AAED,MAAM,CAAC,MAAM,UAAU,GAAoF;IACzG,IAAI,EAAE,aAAa;IACnB,QAAQ,EAAE,WAAW;IACrB,SAAS,EAAE,SAAS;IACpB,SAAS,EAAE,CAAC,KAA+B,EAAsB,EAAE;QACjE,MAAM,cAAc,GAAG,aAAa,CAAC,KAAK,CAAC,CAAC;QAC5C,OAAO,CAAC;gBACN,QAAQ,EAAE,KAAK,CAAC,QAAQ;gBACxB,cAAc;gBACd,SAAS,EAAE,KAAK,CAAC,sBAAsB;gBACvC,eAAe,EAAE,KAAK,CAAC,eAAe,IAAI,EAAE;gBAC5C,KAAK,EAAE,KAAK,CAAC,KAAK;aACnB,CAAC,CAAC;IACL,CAAC;IACD,aAAa,EAAE,CAAC,KAAuB,EAAE,GAAqB,EAAE,EAAE,CAAC,CAAC;QAClE,MAAM,EAAE,KAAK,CAAC,cAAc;QAC5B,SAAS,EAAE,SAAkB;QAC7B,YAAY,EAAE,cAAuB;QACrC,GAAG,EAAE,GAAG,CAAC,cAAc,EAAE,GAAG,IAAI,GAAG,CAAC,GAAG;QACvC,eAAe,EAAE,KAAK,CAAC,eAAe;QACtC,SAAS,EAAE,KAAK,CAAC,SAAS;QAC1B,KAAK,EAAE,KAAK,CAAC,KAAK,IAAI,GAAG,CAAC,MAAM,CAAC,QAAQ,EAAE,KAAK,IAAI,MAAM;QAC1D,SAAS,EAAE,GAAG,CAAC,MAAM,CAAC,QAAQ,EAAE,SAAS,IAAI,uBAAuB;QACpE,UAAU,EAAE,GAAG,CAAC,MAAM,CAAC,QAAQ,EAAE,UAAU,IAAI,EAAE;QACjD,aAAa,EAAE,GAAG,CAAC,MAAM,CAAC,QAAQ,EAAE,aAAa,IAAI,UAAU;QAC/D,SAAS,EAAE,GAAG,CAAC,SAAS,IAAI,SAAS;KACtC,CAAC;IACF,YAAY,EAAE,uBAAuB;IACrC,gBAAgB,EAAE,2BAA2B;IAC7C,eAAe,EAAE;QACf,SAAS,EAAE,0BAA0B;KACtC;IACD,mBAAmB,EAAE,CAAC,QAAQ,EAAE,IAAI,EAAE,EAAE;QACtC,MAAM,MAAM,GAAG,QAAQ,CAAC,gBAAuD,CAAC;QAChF,MAAM,aAAa,GAAG,MAAM,EAAE,IAAI,KAAK,mBAAmB,CAAC,CAAC,CAAC,MAAM,CAAC,aAAa,CAAC,CAAC,CAAC,IAAI,CAAC;QACzF,MAAM,YAAY,GAAG,aAAa,EAAE,wBAAwB,IAAI,KAAK,CAAC;QAEtE,MAAM,OAAO,GAAG,6BAA6B,CAAC;YAC5C,YAAY;YACZ,WAAW,EAAE,MAAM,EAAE,IAAI,KAAK,mBAAmB;gBAC/C,CAAC,CAAC,EAAE,IAAI,EAAE,mBAAmB,EAAE,aAAa,EAAE,MAAM,CAAC,aAAa,EAAE,eAAe,EAAE,MAAM,CAAC,eAAe,EAAE;gBAC7G,CAAC,CAAC,EAAE,IAAI,EAAE,sBAAsB,EAAE;SACrC,CAAC,CAAC;QAEH,+DAA+D;QAC/D,IAAI,aAAa,IAAI,QAAQ,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,CAAC;YACzC,IAAI,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,gBAAgB,EAAE,CAAC;gBACzC,QAAQ,CAAC,OAAO,CAAC,CAAC,CAAS,CAAC,gBAAgB,GAAG,EAAE,aAAa,EAAE,CAAC;YACpE,CAAC;iBAAM,CAAC;gBACL,QAAQ,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,gBAAwB,CAAC,aAAa,GAAG,aAAa,CAAC;YAC9E,CAAC;YACA,QAAQ,CAAC,OAAO,CAAC,CAAC,CAAS,CAAC,YAAY,GAAG,OAAO,CAAC,YAAY,CAAC;YACjE,IAAI,OAAO,CAAC,gBAAgB,KAAK,SAAS,EAAE,CAAC;gBAC1C,QAAQ,CAAC,OAAO,CAAC,CAAC,CAAS,CAAC,gBAAgB,GAAG,OAAO,CAAC,gBAAgB,CAAC;YAC3E,CAAC;QACH,CAAC;QAED,OAAO,QAAQ,CAAC;IAClB,CAAC;CACF,CAAC"}
+{"version":3,"file":"tool-config.js","sourceRoot":"","sources":["../../../src/tools/investigate/tool-config.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,WAAW,EAAE,MAAM,aAAa,CAAC;AAE1C,OAAO,EAAE,0BAA0B,EAAE,MAAM,iCAAiC,CAAC;AAG7E,OAAO,EAAE,uBAAuB,EAAE,MAAM,2DAA2D,CAAC;AAEpG,OAAO,EAAE,2BAA2B,EAAE,MAAM,mDAAmD,CAAC;AAChG,OAAO,EAAE,6BAA6B,EAAE,MAAM,8CAA8C,CAAC;AAC7F,OAAO,EAAE,uBAAuB,EAAE,MAAM,wBAAwB,CAAC;AACjE,OAAO,EACL,+BAA+B,EAC/B,yBAAyB,EACzB,sBAAsB,EACtB,+BAA+B,EAC/B,yBAAyB,EACzB,+BAA+B,GAChC,MAAM,2BAA2B,CAAC;AAEnC,MAAM,UAAU,mBAAmB,CAAC,QAA6B;IAC/D,QAAQ,CAAC,QAAQ,CAAC;QAChB,SAAS,EAAE,aAAa;QACxB,UAAU,EAAE,MAAM;QAClB,QAAQ,EAAE,cAAc;QACxB,OAAO,EAAE,MAAM;QACf,MAAM,EAAE,WAAW;QACnB,YAAY,EAAE,WAAW;QACzB,gBAAgB,EAAE,SAAS;QAC3B,oBAAoB,EAAE,KAAK;QAC3B,iBAAiB,EAAE,eAAe;KACnC,CAAC,CAAC;AACL,CAAC;AAoCD,SAAS,aAAa,CAAC,KAA+B;IACpD,MAAM,WAAW,GAAa,EAAE,CAAC;IACjC,6DAA6D;IAC7D,mEAAmE;IACnE,iEAAiE;IACjE,iEAAiE;IACjE,mBAAmB;IACnB,WAAW,CAAC,IAAI,CAAC,+BAA+B,CAAC,CAAC;IAClD,WAAW,CAAC,IAAI,CACd;QACE,oFAAoF;QACpF,sFAAsF;QACtF,sFAAsF;QACtF,EAAE;QACF,YAAY;QACZ,mEAAmE;QACnE,EAAE;QACF,cAAc;QACd,8SAA8S;QAC9S,6BAA6B;QAC7B,yCAAyC;QACzC,2CAA2C;QAC3C,oEAAoE;QACpE,wIAAwI;QACxI,EAAE;QACF,eAAe;QACf,2IAA2I;QAC3I,EAAE;QACF,eAAe;QACf,oFAAoF;QACpF,iFAAiF;QACjF,kDAAkD;KACnD,CAAC,IAAI,CAAC,IAAI,CAAC,CACb,CAAC;IACF,KAAK,MAAM,KAAK,IAAI,KAAK,CAAC,qBAAqB,EAAE,CAAC;QAChD,WAAW,CAAC,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;IAClC,CAAC;IACD,IAAI,KAAK,CAAC,0BAA0B,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAChD,WAAW,CAAC,IAAI,CACd,gEAAgE;YAChE,KAAK,CAAC,0BAA0B,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAC/D,CAAC;IACJ,CAAC;IACD,WAAW,CAAC,IAAI,CAAC,aAAa,KAAK,CAAC,QAAQ,EAAE,CAAC,CAAC;IAChD,IAAI,KAAK,CAAC,qBAAqB,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC3C,WAAW,CAAC,IAAI,CACd,sMAAsM,CACvM,CAAC;IACJ,CAAC;IACD,6DAA6D;IAC7D,mEAAmE;IACnE,kEAAkE;IAClE,8DAA8D;IAC9D,2DAA2D;IAC3D,WAAW,CAAC,IAAI,CACd,yBAAyB,EACzB,+BAA+B,EAC/B,yBAAyB,EACzB,sBAAsB,EACtB,+BAA+B,CAChC,CAAC;IACF,OAAO,WAAW,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;AAClC,CAAC;AAED,MAAM,CAAC,MAAM,UAAU,GAAoF;IACzG,IAAI,EAAE,aAAa;IACnB,QAAQ,EAAE,WAAW;IACrB,SAAS,EAAE,SAAS;IACpB,SAAS,EAAE,CAAC,KAA+B,EAAsB,EAAE;QACjE,MAAM,cAAc,GAAG,aAAa,CAAC,KAAK,CAAC,CAAC;QAC5C,OAAO,CAAC;gBACN,QAAQ,EAAE,KAAK,CAAC,QAAQ;gBACxB,cAAc;gBACd,SAAS,EAAE,KAAK,CAAC,sBAAsB;gBACvC,eAAe,EAAE,KAAK,CAAC,eAAe,IAAI,EAAE;gBAC5C,KAAK,EAAE,KAAK,CAAC,KAAK;aACnB,CAAC,CAAC;IACL,CAAC;IACD,aAAa,EAAE,CAAC,KAAuB,EAAE,GAAqB,EAAE,EAAE,CAAC,CAAC;QAClE,MAAM,EAAE,KAAK,CAAC,cAAc;QAC5B,mEAAmE;QACnE,sEAAsE;QACtE,wEAAwE;QACxE,4CAA4C;QAC5C,cAAc,EAAE,aAAa,KAAK,CAAC,QAAQ,EAAE;QAC7C,SAAS,EAAE,SAAkB;QAC7B,YAAY,EAAE,cAAuB;QACrC,GAAG,EAAE,GAAG,CAAC,cAAc,EAAE,GAAG,IAAI,GAAG,CAAC,GAAG;QACvC,eAAe,EAAE,KAAK,CAAC,eAAe;QACtC,SAAS,EAAE,KAAK,CAAC,SAAS;QAC1B,KAAK,EAAE,KAAK,CAAC,KAAK,IAAI,GAAG,CAAC,MAAM,CAAC,QAAQ,EAAE,KAAK,IAAI,MAAM;QAC1D,SAAS,EAAE,GAAG,CAAC,MAAM,CAAC,QAAQ,EAAE,SAAS,IAAI,uBAAuB;QACpE,UAAU,EAAE,GAAG,CAAC,MAAM,CAAC,QAAQ,EAAE,UAAU,IAAI,EAAE;QACjD,aAAa,EAAE,GAAG,CAAC,MAAM,CAAC,QAAQ,EAAE,aAAa,IAAI,UAAU;QAC/D,SAAS,EAAE,GAAG,CAAC,SAAS,IAAI,SAAS;KACtC,CAAC;IACF,YAAY,EAAE,uBAAuB;IACrC,gBAAgB,EAAE,2BAA2B;IAC7C,eAAe,EAAE;QACf,SAAS,EAAE,0BAA0B;KACtC;IACD,mBAAmB,EAAE,CAAC,QAAQ,EAAE,IAAI,EAAE,EAAE;QACtC,MAAM,MAAM,GAAG,QAAQ,CAAC,gBAAuD,CAAC;QAChF,MAAM,aAAa,GAAG,MAAM,EAAE,IAAI,KAAK,mBAAmB,CAAC,CAAC,CAAC,MAAM,CAAC,aAAa,CAAC,CAAC,CAAC,IAAI,CAAC;QACzF,MAAM,YAAY,GAAG,aAAa,EAAE,wBAAwB,IAAI,KAAK,CAAC;QAEtE,MAAM,OAAO,GAAG,6BAA6B,CAAC;YAC5C,YAAY;YACZ,WAAW,EAAE,MAAM,EAAE,IAAI,KAAK,mBAAmB;gBAC/C,CAAC,CAAC,EAAE,IAAI,EAAE,mBAAmB,EAAE,aAAa,EAAE,MAAM,CAAC,aAAa,EAAE,eAAe,EAAE,MAAM,CAAC,eAAe,EAAE;gBAC7G,CAAC,CAAC,EAAE,IAAI,EAAE,sBAAsB,EAAE;SACrC,CAAC,CAAC;QAEH,+DAA+D;QAC/D,IAAI,aAAa,IAAI,QAAQ,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,CAAC;YACzC,IAAI,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,gBAAgB,EAAE,CAAC;gBACzC,QAAQ,CAAC,OAAO,CAAC,CAAC,CAAS,CAAC,gBAAgB,GAAG,EAAE,aAAa,EAAE,CAAC;YACpE,CAAC;iBAAM,CAAC;gBACL,QAAQ,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,gBAAwB,CAAC,aAAa,GAAG,aAAa,CAAC;YAC9E,CAAC;YACA,QAAQ,CAAC,OAAO,CAAC,CAAC,CAAS,CAAC,YAAY,GAAG,OAAO,CAAC,YAAY,CAAC;YACjE,IAAI,OAAO,CAAC,gBAAgB,KAAK,SAAS,EAAE,CAAC;gBAC1C,QAAQ,CAAC,OAAO,CAAC,CAAC,CAAS,CAAC,gBAAgB,GAAG,OAAO,CAAC,gBAAgB,CAAC;YAC3E,CAAC;QACH,CAAC;QAED,OAAO,QAAQ,CAAC;IAClB,CAAC;CACF,CAAC"}

package/dist/tools/parallel-criteria-prompt.d.ts

@@ -0,0 +1,106 @@
+import type { CriterionEntry } from './criteria-types.js';
+/**
+ * Per-route semantics for findings. Same wire shape (`## Finding N:`) and
+ * same severity tiers (critical / high / medium / low) across all five
+ * read-only routes — only the *meaning* of each tier and the per-sub-worker
+ * goal differ.
+ *
+ * Audit / review / debug: each finding = an issue. severity = impact.
+ * Verify: each finding = a verification verdict. severity = how decisive
+ *   the FAIL signal is (or how strong the PASS evidence is).
+ * Investigate: each finding = a candidate answer (or sub-answer) to the
+ *   user's question. severity = confidence in the answer.
+ */
+export interface RouteSemantics {
+    /** One-line goal sentence injected at the top of the per-sub-worker
+     *  assignment block. e.g. "find issues of THIS specific kind" (audit). */
+    goalLine: string;
+    /** What "no findings" means for this route, when the sub-worker has
+     *  nothing to emit. Investigate's empty case is rare (we can almost
+     *  always produce SOME candidate answer, even low-confidence). */
+    emptyOutcomeLine: string;
+    /** Per-tier severity meaning for THIS route. Replaces the generic
+     *  SEVERITY_LADDER inside the cached prefix so the worker calibrates
+     *  to the right semantic. Order: critical, high, medium, low. */
+    severityMeanings: {
+        critical: string;
+        high: string;
+        medium: string;
+        low: string;
+    };
+    /** One-paragraph clarifier of what a "finding" represents on this
+     *  route. Written into the cached prefix above the format spec so
+     *  the worker doesn't default to "find a problem" when the route
+     *  semantic is "propose an answer" (investigate) or "report a
+     *  verification verdict" (verify). */
+    findingMeaningParagraph: string;
+    /** Whether each sub-worker MUST emit at least one finding.
+     *
+     *  - `false` (problem-finding routes — audit/review): "no findings
+     *    for this criterion" is a valid honest result; the artifact may
+     *    have no problems in the category.
+     *  - `true` (answer-finding routes — debug/verify/investigate): the
+     *    user asked a question; every parallel angle owes at least one
+     *    contribution (even low-severity / low-confidence). The merge
+     *    annotator dedups + ranks; soft signals are valuable, not noise.
+     */
+    mustEmitAtLeastOne: boolean;
+}
+/**
+ * Shared cached-prefix builder for parallel-criteria fan-out across the
+ * five read-only routes. The prefix carries everything that's identical
+ * across all N sub-workers + the cache-warmer call:
+ *   - route-specific orientation, evidence rule, scope rule, annotator awareness
+ *   - SEVERITY_LADDER (shared across routes)
+ *   - finding-format spec (shared shape: `## Finding N:` blocks)
+ *   - the full criterion taxonomy listed by id+title+description so each
+ *     sub-worker knows what's NOT theirs
+ *   - target content (inlined document and/or file contents pre-read by the
+ *     dispatcher)
+ *
+ * The cache_control marker is attached at the adapter layer; this helper
+ * only assembles the text. The per-sub-worker user message (the variable
+ * suffix) is built by `buildReadOnlyCriterionSuffix` below.
+ *
+ * THOROUGHNESS_REMINDER_* / CONFIDENCE_REMINDER_* are intentionally OMITTED.
+ * They were calibrated for a single worker covering all categories; with
+ * one criterion per worker, that pressure backfires (workers invent weak
+ * findings to hit the implied quota when their criterion legitimately has
+ * zero matches in the artifact).
+ */
+export interface CachedPrefixBlocks {
+    /** Route-specific orientation, e.g. AUDIT_PURPOSE_ORIENTATION. */
+    orientation: string;
+    /** Route-specific EVIDENCE_RULE_*. */
+    evidenceRule: string;
+    /** Route-specific SCOPE_RULE_*. */
+    scopeRule: string;
+    /** Route-specific ANNOTATOR_AWARENESS_*. */
+    annotatorAwareness: string;
+    /** Per-route finding format text. */
+    findingFormat: string;
+    /** The full criterion taxonomy in structured form. */
+    criteria: readonly CriterionEntry[];
+    /** Route-specific semantics for what a finding represents and what
+     *  each severity tier means. */
+    semantics: RouteSemantics;
+}
+export interface CachedPrefixTarget {
+    /** Inlined document content, when present (audit). */
+    document?: string;
+    /** Map of file path → file contents pre-read by the dispatcher. */
+    preReadFiles?: Record<string, string>;
+    /** File paths declared by the request, even when their contents weren't
+     *  pre-read (sub-worker may grep on demand). */
+    filePaths?: readonly string[];
+}
+export declare function buildReadOnlyCachedPrefix(blocks: CachedPrefixBlocks, target: CachedPrefixTarget): string;
+/** Per-sub-worker user message: assigns ONE criterion. The semantics
+ *  arg controls how the assignment is framed (find issues / answer
+ *  question / verify criterion / find root cause / find quality issues
+ *  in the prose).
+ *
+ *  Same shape across all routes; only `goalLine` and `emptyOutcomeLine`
+ *  differ per route. */
+export declare function buildReadOnlyCriterionSuffix(semantics: RouteSemantics, criterion: CriterionEntry): string;
+//# sourceMappingURL=parallel-criteria-prompt.d.ts.map

package/dist/tools/parallel-criteria-prompt.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"parallel-criteria-prompt.d.ts","sourceRoot":"","sources":["../../src/tools/parallel-criteria-prompt.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,qBAAqB,CAAC;AAE1D;;;;;;;;;;;GAWG;AACH,MAAM,WAAW,cAAc;IAC7B;8EAC0E;IAC1E,QAAQ,EAAE,MAAM,CAAC;IACjB;;sEAEkE;IAClE,gBAAgB,EAAE,MAAM,CAAC;IACzB;;qEAEiE;IACjE,gBAAgB,EAAE;QAAE,QAAQ,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAC;QAAC,MAAM,EAAE,MAAM,CAAC;QAAC,GAAG,EAAE,MAAM,CAAA;KAAE,CAAC;IAClF;;;;0CAIsC;IACtC,uBAAuB,EAAE,MAAM,CAAC;IAChC;;;;;;;;;OASG;IACH,kBAAkB,EAAE,OAAO,CAAC;CAC7B;AAED;;;;;;;;;;;;;;;;;;;;;GAqBG;AACH,MAAM,WAAW,kBAAkB;IACjC,kEAAkE;IAClE,WAAW,EAAE,MAAM,CAAC;IACpB,sCAAsC;IACtC,YAAY,EAAE,MAAM,CAAC;IACrB,mCAAmC;IACnC,SAAS,EAAE,MAAM,CAAC;IAClB,4CAA4C;IAC5C,kBAAkB,EAAE,MAAM,CAAC;IAC3B,qCAAqC;IACrC,aAAa,EAAE,MAAM,CAAC;IACtB,sDAAsD;IACtD,QAAQ,EAAE,SAAS,cAAc,EAAE,CAAC;IACpC;oCACgC;IAChC,SAAS,EAAE,cAAc,CAAC;CAC3B;AAaD,MAAM,WAAW,kBAAkB;IACjC,sDAAsD;IACtD,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,mEAAmE;IACnE,YAAY,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IACtC;oDACgD;IAChD,SAAS,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;CAC/B;AAED,wBAAgB,yBAAyB,CACvC,MAAM,EAAE,kBAAkB,EAC1B,MAAM,EAAE,kBAAkB,GACzB,MAAM,CAwCR;AAED;;;;;;wBAMwB;AACxB,wBAAgB,4BAA4B,CAC1C,SAAS,EAAE,cAAc,EACzB,SAAS,EAAE,cAAc,GACxB,MAAM,CA4BR"}

package/dist/tools/parallel-criteria-prompt.js

@@ -0,0 +1,86 @@
+function renderSeverityLadder(meanings) {
+    return [
+        'Severity (your judgment, calibrated to the meanings below):',
+        `- critical: ${meanings.critical}`,
+        `- high:     ${meanings.high}`,
+        `- medium:   ${meanings.medium}`,
+        `- low:      ${meanings.low}`,
+        'Use the FULL ladder. Calibrate to actual signal strength, not how alarming or assertive the wording sounds.',
+    ].join('\n');
+}
+export function buildReadOnlyCachedPrefix(blocks, target) {
+    const parts = [
+        blocks.orientation,
+        '',
+        'What a "finding" means on this route:',
+        blocks.semantics.findingMeaningParagraph,
+        '',
+        blocks.findingFormat,
+        '',
+        renderSeverityLadder(blocks.semantics.severityMeanings),
+        '',
+        blocks.evidenceRule,
+        '',
+        blocks.scopeRule,
+        '',
+        blocks.annotatorAwareness,
+        '',
+        'Reference taxonomy (other parallel sub-workers cover the categories not assigned to you):',
+        ...blocks.criteria.map(c => `- Criterion ${c.id} — ${c.title}: ${c.description}`),
+    ];
+    if (target.document && target.document.trim().length > 0) {
+        parts.push('', 'Target document (inlined):', '', target.document);
+    }
+    const filePaths = target.filePaths ?? [];
+    const preRead = target.preReadFiles ?? {};
+    if (filePaths.length > 0) {
+        parts.push('', 'Target files (pre-read where contents are below; otherwise the sub-worker may read on demand):');
+        for (const fp of filePaths) {
+            const content = preRead[fp];
+            if (content !== undefined) {
+                parts.push('', `--- ${fp} ---`, content);
+            }
+            else {
+                parts.push('', `--- ${fp} ---`, '(not pre-read; read with read_file if you need its contents)');
+            }
+        }
+    }
+    return parts.join('\n');
+}
+/** Per-sub-worker user message: assigns ONE criterion. The semantics
+ *  arg controls how the assignment is framed (find issues / answer
+ *  question / verify criterion / find root cause / find quality issues
+ *  in the prose).
+ *
+ *  Same shape across all routes; only `goalLine` and `emptyOutcomeLine`
+ *  differ per route. */
+export function buildReadOnlyCriterionSuffix(semantics, criterion) {
+    const minimumBlock = semantics.mustEmitAtLeastOne
+        ? [
+            'HARD REQUIREMENT (this is non-negotiable):',
+            'Your response MUST contain at least one `## Finding N:` block. Empty / narrative-only / commentary-only responses are WORKFLOW ERRORS and will be discarded.',
+            '',
+            'Three valid finding shapes — use whichever fits, in this order of preference:',
+            '',
+            '1. SUBSTANTIVE FINDING — your angle produced a real on-question signal. Emit at any severity (critical → low) per the calibration above. Multiple substantive findings from one angle are fine.',
+            '',
+            '2. PARTIAL FINDING — your angle produced a weak / inferred signal. Emit at severity=low with the partial information, marked "verify by reading <file>" or "cross-check against <another angle>".',
+            '',
+            '3. NOT-APPLICABLE FINDING — your angle is genuinely orthogonal to the question (e.g. CONCURRENCY angle on a pure-sync function; TEST-DRIVEN angle when no tests exist for the area). Emit ONE finding with severity=low, title prefixed with "[N/A]", and Issue="This perspective does not apply because <concrete reason>." The merge annotator drops these from the final report; they exist so the user can see ALL angles were attempted.',
+            '',
+            'DO NOT pad with speculation to satisfy the count. If you don\'t have a substantive or partial signal, the [N/A] finding is the right choice — the merge annotator treats N/A as completeness coverage, not as a real finding.',
+        ].join('\n')
+        : semantics.emptyOutcomeLine;
+    return [
+        `Your assignment: criterion ${criterion.id} — "${criterion.title}".`,
+        '',
+        criterion.description,
+        '',
+        semantics.goalLine,
+        '',
+        minimumBlock,
+        '',
+        'Do NOT drift outside this angle; other parallel sub-workers cover the other angles.',
+    ].join('\n');
+}
+//# sourceMappingURL=parallel-criteria-prompt.js.map

package/dist/tools/parallel-criteria-prompt.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"parallel-criteria-prompt.js","sourceRoot":"","sources":["../../src/tools/parallel-criteria-prompt.ts"],"names":[],"mappings":"AAqFA,SAAS,oBAAoB,CAAC,QAA4C;IACxE,OAAO;QACL,6DAA6D;QAC7D,eAAe,QAAQ,CAAC,QAAQ,EAAE;QAClC,eAAe,QAAQ,CAAC,IAAI,EAAE;QAC9B,eAAe,QAAQ,CAAC,MAAM,EAAE;QAChC,eAAe,QAAQ,CAAC,GAAG,EAAE;QAC7B,6GAA6G;KAC9G,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AACf,CAAC;AAYD,MAAM,UAAU,yBAAyB,CACvC,MAA0B,EAC1B,MAA0B;IAE1B,MAAM,KAAK,GAAa;QACtB,MAAM,CAAC,WAAW;QAClB,EAAE;QACF,uCAAuC;QACvC,MAAM,CAAC,SAAS,CAAC,uBAAuB;QACxC,EAAE;QACF,MAAM,CAAC,aAAa;QACpB,EAAE;QACF,oBAAoB,CAAC,MAAM,CAAC,SAAS,CAAC,gBAAgB,CAAC;QACvD,EAAE;QACF,MAAM,CAAC,YAAY;QACnB,EAAE;QACF,MAAM,CAAC,SAAS;QAChB,EAAE;QACF,MAAM,CAAC,kBAAkB;QACzB,EAAE;QACF,2FAA2F;QAC3F,GAAG,MAAM,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,eAAe,CAAC,CAAC,EAAE,MAAM,CAAC,CAAC,KAAK,KAAK,CAAC,CAAC,WAAW,EAAE,CAAC;KAClF,CAAC;IAEF,IAAI,MAAM,CAAC,QAAQ,IAAI,MAAM,CAAC,QAAQ,CAAC,IAAI,EAAE,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACzD,KAAK,CAAC,IAAI,CAAC,EAAE,EAAE,4BAA4B,EAAE,EAAE,EAAE,MAAM,CAAC,QAAQ,CAAC,CAAC;IACpE,CAAC;IAED,MAAM,SAAS,GAAG,MAAM,CAAC,SAAS,IAAI,EAAE,CAAC;IACzC,MAAM,OAAO,GAAG,MAAM,CAAC,YAAY,IAAI,EAAE,CAAC;IAC1C,IAAI,SAAS,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACzB,KAAK,CAAC,IAAI,CAAC,EAAE,EAAE,gGAAgG,CAAC,CAAC;QACjH,KAAK,MAAM,EAAE,IAAI,SAAS,EAAE,CAAC;YAC3B,MAAM,OAAO,GAAG,OAAO,CAAC,EAAE,CAAC,CAAC;YAC5B,IAAI,OAAO,KAAK,SAAS,EAAE,CAAC;gBAC1B,KAAK,CAAC,IAAI,CAAC,EAAE,EAAE,OAAO,EAAE,MAAM,EAAE,OAAO,CAAC,CAAC;YAC3C,CAAC;iBAAM,CAAC;gBACN,KAAK,CAAC,IAAI,CAAC,EAAE,EAAE,OAAO,EAAE,MAAM,EAAE,8DAA8D,CAAC,CAAC;YAClG,CAAC;QACH,CAAC;IACH,CAAC;IAED,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC1B,CAAC;AAED;;;;;;wBAMwB;AACxB,MAAM,UAAU,4BAA4B,CAC1C,SAAyB,EACzB,SAAyB;IAEzB,MAAM,YAAY,GAAG,SAAS,CAAC,kBAAkB;QAC/C,CAAC,CAAC;YACE,4CAA4C;YAC5C,8JAA8J;YAC9J,EAAE;YACF,+EAA+E;YAC/E,EAAE;YACF,iMAAiM;YACjM,EAAE;YACF,mMAAmM;YACnM,EAAE;YACF,+aAA+a;YAC/a,EAAE;YACF,+NAA+N;SAChO,CAAC,IAAI,CAAC,IAAI,CAAC;QACd,CAAC,CAAC,SAAS,CAAC,gBAAgB,CAAC;IAC/B,OAAO;QACL,8BAA8B,SAAS,CAAC,EAAE,OAAO,SAAS,CAAC,KAAK,IAAI;QACpE,EAAE;QACF,SAAS,CAAC,WAAW;QACrB,EAAE;QACF,SAAS,CAAC,QAAQ;QAClB,EAAE;QACF,YAAY;QACZ,EAAE;QACF,qFAAqF;KACtF,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AACf,CAAC"}

package/dist/tools/review/implementer-criteria.d.ts

@@ -1,11 +1,60 @@
 /**
  * Review-specific implementer criteria.
  *
+ * REVIEW'S PURPOSE — read this before adding categories.
+ * mma-review is the pre-merge gate. The maintainer accepting your verdict
+ * will NOT re-investigate before pressing merge — your output is treated
+ * as authoritative. The success criterion is:
+ *
+ *   "After fixes, will the merge be safe, correct, and maintainable —
+ *    such that a regression is unlikely to ship?"
+ *
+ * That criterion makes a finding load-bearing. A nit that doesn't change
+ * whether the merge is safe is low-priority no matter how clean the
+ * suggested rewrite reads. A cross-file ripple that breaks a caller
+ * not in the diff is the audit-equivalent of an unimplementable fix —
+ * load-bearing even though the named file looks fine in isolation.
+ *
  * Review examines source code in named files against a focus area
  * (security/correctness/performance/style). Findings should be
- * line-quotable — that's the natural shape of code defects.
+ * line-quotable — that's the natural shape of code defects — but
+ * cross-file findings backed by call-site references are also valid.
+ */
+/**
+ * The orientation block. Goes at the TOP of every review prompt.
+ *
+ * This is the load-bearing addition. Without an explicit purpose
+ * statement, workers default to "find issues in this file" — which
+ * produces line-by-line proofreading and misses the cross-file rippe,
+ * test-gap, and implicit-contract findings that actually block merges.
  */
+export declare const REVIEW_PURPOSE_ORIENTATION: string;
 export declare const EVIDENCE_RULE_REVIEW: string;
 export declare const SCOPE_RULE_REVIEW: string;
+/**
+ * The failure-mode taxonomy for code reviews.
+ *
+ * Without this block, workers default to line-by-line proofreading of
+ * the named file and miss cross-file ripples, test gaps, and
+ * implicit-contract regressions — the findings that actually block
+ * merges. The 10 categories below are what a careful maintainer would
+ * scan for before pressing merge.
+ */
+export declare const CODE_REVIEW_FAILURE_MODES: string;
+/**
+ * Counter-balance to the SEVERITY_LADDER's anti-inflation hint.
+ *
+ * The shared severity ladder ends with "Workers commonly inflate —
+ * resist the urge." That bias is correct in the limit (no, the missing
+ * comma is not critical) but produces UNDER-finding when combined with
+ * a thin per-tool rubric. For code review specifically, the typical
+ * failure is missing the cross-file ripple or test gap because the
+ * worker only looked at the diff in the named file. This block tells
+ * the worker that under-finding is the more common review failure.
+ */
+export declare const THOROUGHNESS_REMINDER_REVIEW: string;
 export declare const ANNOTATOR_AWARENESS_REVIEW: string;
+import { type CriterionEntry } from '../criteria-types.js';
+/** Structured per-criterion array for parallel-criteria fan-out. */
+export declare const REVIEW_CRITERIA: readonly CriterionEntry[];
 //# sourceMappingURL=implementer-criteria.d.ts.map

package/dist/tools/review/implementer-criteria.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"implementer-criteria.d.ts","sourceRoot":"","sources":["../../../src/tools/review/implementer-criteria.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,eAAO,MAAM,oBAAoB,QAKrB,CAAC;AAEb,eAAO,MAAM,iBAAiB,QAIlB,CAAC;AAEb,eAAO,MAAM,0BAA0B,QAO3B,CAAC"}
+{"version":3,"file":"implementer-criteria.d.ts","sourceRoot":"","sources":["../../../src/tools/review/implementer-criteria.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;GAqBG;AAEH;;;;;;;GAOG;AACH,eAAO,MAAM,0BAA0B,QAmB3B,CAAC;AAEb,eAAO,MAAM,oBAAoB,QAQrB,CAAC;AAEb,eAAO,MAAM,iBAAiB,QAOlB,CAAC;AAEb;;;;;;;;GAQG;AACH,eAAO,MAAM,yBAAyB,QAmB1B,CAAC;AAEb;;;;;;;;;;GAUG;AACH,eAAO,MAAM,4BAA4B,QAW7B,CAAC;AAEb,eAAO,MAAM,0BAA0B,QAO3B,CAAC;AAEb,OAAO,EAAiB,KAAK,cAAc,EAAE,MAAM,sBAAsB,CAAC;AAE1E,oEAAoE;AACpE,eAAO,MAAM,eAAe,EAAE,SAAS,cAAc,EAA6C,CAAC"}