@zhixuan92/multi-model-agent-core 4.0.3 → 4.0.5

package/dist/review/reviewer-output-parser.js

@@ -2,20 +2,164 @@ function extractSummarySection(text) {
     const match = text.match(/##\s*Summary\s*\n([\s\S]*?)(?=\n##\s|$)/i);
     return match ? match[1].trim() : null;
 }
-function extractDeviationsAndUnresolved(text) {
-    const concerns = [];
-    const jsonMatch = text.match(/```json\s*\n([\s\S]*?)\n```/i);
-    if (jsonMatch) {
+/**
+ * Extract the verdict from a JSON block in the reviewer's output.
+ * Two acceptable shapes:
+ *   1. Fenced: ```json {"verdict":"...","concerns":[...]} ```
+ *   2. Bare:   {"verdict":"...","concerns":[...]}    (no fence)
+ *
+ * Some models freelance the format and skip the ```json``` wrapper —
+ * the fenced-only parser would then drop a perfectly valid verdict
+ * and fall through to the markdown-section path, which also fails
+ * (since the JSON template doesn't emit `## Summary`), causing the
+ * spurious "missing structured verdict" meta-concern that triggered
+ * spec_rework spirals.
+ *
+ * Recognized verdict values (case-insensitive):
+ *   - "approved" / "approve" → ReviewerVerdict.approved
+ *   - "changes_required" / "changes-required" → changes_required
+ *   - "concerns" → mapped to approved (parser convention: concerns
+ *     ≠ blocking; use changes_required for blocking).
+ */
+function extractJsonVerdict(text) {
+    // Pass 1: fenced ```json ... ``` block.
+    const fenced = text.match(/```json\s*\n([\s\S]*?)\n```/i);
+    const fencedVerdict = fenced ? tryReadVerdict(fenced[1]) : null;
+    if (fencedVerdict)
+        return fencedVerdict;
+    // Pass 2: scan for the first balanced `{...}` that parses as a JSON
+    // object containing a "verdict" key. Pre-fix the parser only tried
+    // FIRST `{` to LAST `}` — that fails when the text has prose with
+    // its own `{...}` (e.g., "the diff matches {file} criteria...
+    // {"verdict":"approved",...}"). We now walk each `{` left-to-right
+    // and find its matching `}` via balanced-brace counting (string-
+    // literal aware) so prose braces and JSON braces don't collide.
+    return findFirstParseableJsonVerdict(text);
+}
+/**
+ * Walk every `{` in `text` from left to right, find its balanced `}`
+ * (respecting strings + escapes), and try to parse the slice as JSON
+ * with a "verdict" field. Returns the first successful match, or null.
+ */
+function findFirstParseableJsonVerdict(text) {
+    const len = text.length;
+    for (let start = text.indexOf('{'); start !== -1; start = text.indexOf('{', start + 1)) {
+        const end = matchingBrace(text, start);
+        if (end === -1)
+            continue;
+        const candidate = text.slice(start, end + 1);
+        if (!/"verdict"\s*:/i.test(candidate))
+            continue;
+        const v = tryReadVerdict(candidate);
+        if (v)
+            return v;
+        // Bound the scan: don't try crazy-large prefixes from the same
+        // position. matchingBrace already returns at most `len`.
+        if (end >= len - 1)
+            break;
+    }
+    return null;
+}
+/**
+ * Return the index of the `}` that balances the `{` at `openPos`,
+ * accounting for nested objects and string literals. Returns -1 when
+ * unbalanced (don't try to parse).
+ */
+function matchingBrace(text, openPos) {
+    let depth = 0;
+    let inString = false;
+    let escape = false;
+    for (let i = openPos; i < text.length; i++) {
+        const ch = text[i];
+        if (inString) {
+            if (escape) {
+                escape = false;
+                continue;
+            }
+            if (ch === '\\') {
+                escape = true;
+                continue;
+            }
+            if (ch === '"') {
+                inString = false;
+                continue;
+            }
+            continue;
+        }
+        if (ch === '"') {
+            inString = true;
+            continue;
+        }
+        if (ch === '{')
+            depth++;
+        else if (ch === '}') {
+            depth--;
+            if (depth === 0)
+                return i;
+        }
+    }
+    return -1;
+}
+function readConcernsFromJsonText(text) {
+    const tryParse = (s) => {
         try {
-            const parsed = JSON.parse(jsonMatch[1]);
-            if (Array.isArray(parsed.concerns)) {
-                concerns.push(...parsed.concerns);
+            const obj = JSON.parse(s);
+            if (Array.isArray(obj.concerns)) {
+                return obj.concerns.filter((x) => typeof x === 'string');
             }
         }
         catch {
-            // fall through to markdown sections
+            /* not parseable */
         }
+        return [];
+    };
+    const fenced = text.match(/```json\s*\n([\s\S]*?)\n```/i);
+    if (fenced) {
+        const out = tryParse(fenced[1]);
+        if (out.length > 0)
+            return out;
+    }
+    // Bare JSON fallback — same balanced-brace walk as extractJsonVerdict
+    // so the two helpers stay in sync. Pre-fix used a fragile first-`{`-
+    // to-last-`}` slice that broke when prose contained `{}`.
+    for (let start = text.indexOf('{'); start !== -1; start = text.indexOf('{', start + 1)) {
+        const end = matchingBrace(text, start);
+        if (end === -1)
+            continue;
+        const candidate = text.slice(start, end + 1);
+        if (!/"concerns"\s*:/i.test(candidate))
+            continue;
+        const out = tryParse(candidate);
+        if (out.length > 0)
+            return out;
+    }
+    return [];
+}
+function tryReadVerdict(jsonText) {
+    let parsed;
+    try {
+        parsed = JSON.parse(jsonText);
+    }
+    catch {
+        return null;
     }
+    const obj = parsed;
+    const v = typeof obj.verdict === 'string' ? obj.verdict.toLowerCase().trim() : '';
+    if (v === 'approved' || v === 'approve')
+        return 'approved';
+    if (v === 'changes_required' || v === 'changes-required')
+        return 'changes_required';
+    if (v === 'concerns')
+        return 'approved';
+    return null;
+}
+function extractDeviationsAndUnresolved(text) {
+    const concerns = [];
+    // Try fenced ```json``` first, then bare JSON (same lenient pattern
+    // as extractJsonVerdict — keeps both helpers in sync so a parser
+    // that recognizes the verdict also recognizes the concerns).
+    const fromJson = readConcernsFromJsonText(text);
+    concerns.push(...fromJson);
     const devMatch = text.match(/##\s*Deviations from brief\s*\n([\s\S]*?)(?=\n##\s|$)/i);
     if (devMatch) {
         const lines = devMatch[1].trim().split('\n').filter(l => l.trim().startsWith('-'));
@@ -40,19 +184,30 @@ function extractDiffVerdict(text) {
 }
 export class ReviewerOutputParser {
     parse(text) {
-        // Lenient parse: a missing `## Summary` section used to throw and crash
-        // the entire run with `runner_crash`. Reviewer models that don't follow
-        // the format strictly (some reasoning models freelance the response
-        // shape) would take the whole task down. Treat malformed output as
-        // `changes_required` with a meta-concern surfacing the format failure
-        // so the rework loop can re-prompt instead of crashing.
-        const summary = extractSummarySection(text);
+        // Source priority:
+        //   1. JSON block ```json ... ``` containing {"verdict":"...","concerns":[...]}
+        //      — what spec/quality templates actually instruct the LLM to emit
+        //      (tool sweep #6 rewrite). The pre-fix parser only looked at
+        //      `## Summary` markdown sections, defaulted every JSON-only
+        //      response to `changes_required` with a meta-concern, and
+        //      triggered exactly the spec_rework spirals we're trying to
+        //      eliminate.
+        //   2. `## Summary` markdown section (back-compat for any reviewer
+        //      that still emits the older format).
+        //   3. Fall through to changes_required with a meta-concern when
+        //      neither path produced a verdict — keeps malformed output
+        //      from crashing the run.
+        const jsonVerdict = extractJsonVerdict(text);
         const concerns = extractDeviationsAndUnresolved(text);
+        if (jsonVerdict) {
+            return { verdict: jsonVerdict, concerns };
+        }
+        const summary = extractSummarySection(text);
         if (!summary) {
             return {
                 verdict: 'changes_required',
                 concerns: [
-                    'reviewer output missing `## Summary` section — defaulting verdict to changes_required',
+                    'reviewer output missing structured verdict (no JSON block, no `## Summary` section) — defaulting to changes_required',
                     ...concerns,
                 ],
             };

package/dist/review/reviewer-output-parser.js.map

@@ -1 +1 @@
-{"version":3,"file":"reviewer-output-parser.js","sourceRoot":"","sources":["../../src/review/reviewer-output-parser.ts"],"names":[],"mappings":"AAYA,SAAS,qBAAqB,CAAC,IAAY;IACzC,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,0CAA0C,CAAC,CAAC;IACrE,OAAO,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC;AACxC,CAAC;AAED,SAAS,8BAA8B,CAAC,IAAY;IAClD,MAAM,QAAQ,GAAa,EAAE,CAAC;IAE9B,MAAM,SAAS,GAAG,IAAI,CAAC,KAAK,CAAC,8BAA8B,CAAC,CAAC;IAC7D,IAAI,SAAS,EAAE,CAAC;QACd,IAAI,CAAC;YACH,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,CAAC;YACxC,IAAI,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,EAAE,CAAC;gBACnC,QAAQ,CAAC,IAAI,CAAC,GAAG,MAAM,CAAC,QAAQ,CAAC,CAAC;YACpC,CAAC;QACH,CAAC;QAAC,MAAM,CAAC;YACP,oCAAoC;QACtC,CAAC;IACH,CAAC;IAED,MAAM,QAAQ,GAAG,IAAI,CAAC,KAAK,CAAC,wDAAwD,CAAC,CAAC;IACtF,IAAI,QAAQ,EAAE,CAAC;QACb,MAAM,KAAK,GAAG,QAAQ,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,UAAU,CAAC,GAAG,CAAC,CAAC,CAAC;QACnF,QAAQ,CAAC,IAAI,CAAC,GAAG,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,EAAE,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC;IAClE,CAAC;IAED,MAAM,UAAU,GAAG,IAAI,CAAC,KAAK,CAAC,6CAA6C,CAAC,CAAC;IAC7E,IAAI,UAAU,EAAE,CAAC;QACf,MAAM,KAAK,GAAG,UAAU,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,UAAU,CAAC,GAAG,CAAC,CAAC,CAAC;QACrF,QAAQ,CAAC,IAAI,CAAC,GAAG,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,EAAE,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC;IAClE,CAAC;IAED,OAAO,QAAQ,CAAC;AAClB,CAAC;AAED,SAAS,kBAAkB,CAAC,IAAY;IACtC,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,EAAE,CAAC;IAC5B,IAAI,aAAa,CAAC,IAAI,CAAC,OAAO,CAAC;QAAE,OAAO,SAAS,CAAC;IAClD,IAAI,aAAa,CAAC,IAAI,CAAC,OAAO,CAAC;QAAE,OAAO,UAAU,CAAC;IACnD,IAAI,WAAW,CAAC,IAAI,CAAC,OAAO,CAAC;QAAE,OAAO,QAAQ,CAAC;IAC/C,OAAO,IAAI,CAAC;AACd,CAAC;AAED,MAAM,OAAO,oBAAoB;IAC/B,KAAK,CAAC,IAAY;QAChB,wEAAwE;QACxE,wEAAwE;QACxE,oEAAoE;QACpE,mEAAmE;QACnE,sEAAsE;QACtE,wDAAwD;QACxD,MAAM,OAAO,GAAG,qBAAqB,CAAC,IAAI,CAAC,CAAC;QAC5C,MAAM,QAAQ,GAAG,8BAA8B,CAAC,IAAI,CAAC,CAAC;QACtD,IAAI,CAAC,OAAO,EAAE,CAAC;YACb,OAAO;gBACL,OAAO,EAAE,kBAAkB;gBAC3B,QAAQ,EAAE;oBACR,uFAAuF;oBACvF,GAAG,QAAQ;iBACZ;aACF,CAAC;QACJ,CAAC;QACD,MAAM,KAAK,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC;QACpC,MAAM,OAAO,GAAoB,KAAK,CAAC,QAAQ,CAAC,kBAAkB,CAAC,CAAC,CAAC,CAAC,kBAAkB,CAAC,CAAC,CAAC,UAAU,CAAC;QACtG,OAAO,EAAE,OAAO,EAAE,QAAQ,EAAE,CAAC;IAC/B,CAAC;IAED,SAAS,CAAC,IAAY;QACpB,mEAAmE;QACnE,mEAAmE;QACnE,MAAM,OAAO,GAAG,kBAAkB,CAAC,IAAI,CAAC,CAAC;QACzC,MAAM,QAAQ,GAAG,8BAA8B,CAAC,IAAI,CAAC,CAAC;QACtD,IAAI,CAAC,OAAO,EAAE,CAAC;YACb,OAAO;gBACL,OAAO,EAAE,UAAU;gBACnB,QAAQ,EAAE;oBACR,oGAAoG;oBACpG,GAAG,QAAQ;iBACZ;aACF,CAAC;QACJ,CAAC;QACD,OAAO,EAAE,OAAO,EAAE,QAAQ,EAAE,CAAC;IAC/B,CAAC;CACF;AAED,MAAM,OAAO,kBAAmB,SAAQ,KAAK;IAC3C,YAAY,OAAe,IAAI,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,GAAG,oBAAoB,CAAC,CAAC,CAAC;CACnF"}
+{"version":3,"file":"reviewer-output-parser.js","sourceRoot":"","sources":["../../src/review/reviewer-output-parser.ts"],"names":[],"mappings":"AAYA,SAAS,qBAAqB,CAAC,IAAY;IACzC,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,0CAA0C,CAAC,CAAC;IACrE,OAAO,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC;AACxC,CAAC;AAED;;;;;;;;;;;;;;;;;;GAkBG;AACH,SAAS,kBAAkB,CAAC,IAAY;IACtC,wCAAwC;IACxC,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,8BAA8B,CAAC,CAAC;IAC1D,MAAM,aAAa,GAAG,MAAM,CAAC,CAAC,CAAC,cAAc,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;IAChE,IAAI,aAAa;QAAE,OAAO,aAAa,CAAC;IAExC,oEAAoE;IACpE,mEAAmE;IACnE,kEAAkE;IAClE,8DAA8D;IAC9D,mEAAmE;IACnE,iEAAiE;IACjE,gEAAgE;IAChE,OAAO,6BAA6B,CAAC,IAAI,CAAC,CAAC;AAC7C,CAAC;AAED;;;;GAIG;AACH,SAAS,6BAA6B,CAAC,IAAY;IACjD,MAAM,GAAG,GAAG,IAAI,CAAC,MAAM,CAAC;IACxB,KAAK,IAAI,KAAK,GAAG,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,EAAE,KAAK,KAAK,CAAC,CAAC,EAAE,KAAK,GAAG,IAAI,CAAC,OAAO,CAAC,GAAG,EAAE,KAAK,GAAG,CAAC,CAAC,EAAE,CAAC;QACvF,MAAM,GAAG,GAAG,aAAa,CAAC,IAAI,EAAE,KAAK,CAAC,CAAC;QACvC,IAAI,GAAG,KAAK,CAAC,CAAC;YAAE,SAAS;QACzB,MAAM,SAAS,GAAG,IAAI,CAAC,KAAK,CAAC,KAAK,EAAE,GAAG,GAAG,CAAC,CAAC,CAAC;QAC7C,IAAI,CAAC,gBAAgB,CAAC,IAAI,CAAC,SAAS,CAAC;YAAE,SAAS;QAChD,MAAM,CAAC,GAAG,cAAc,CAAC,SAAS,CAAC,CAAC;QACpC,IAAI,CAAC;YAAE,OAAO,CAAC,CAAC;QAChB,+DAA+D;QAC/D,yDAAyD;QACzD,IAAI,GAAG,IAAI,GAAG,GAAG,CAAC;YAAE,MAAM;IAC5B,CAAC;IACD,OAAO,IAAI,CAAC;AACd,CAAC;AAED;;;;GAIG;AACH,SAAS,aAAa,CAAC,IAAY,EAAE,OAAe;IAClD,IAAI,KAAK,GAAG,CAAC,CAAC;IACd,IAAI,QAAQ,GAAG,KAAK,CAAC;IACrB,IAAI,MAAM,GAAG,KAAK,CAAC;IACnB,KAAK,IAAI,CAAC,GAAG,OAAO,EAAE,CAAC,GAAG,IAAI,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QAC3C,MAAM,EAAE,GAAG,IAAI,CAAC,CAAC,CAAC,CAAC;QACnB,IAAI,QAAQ,EAAE,CAAC;YACb,IAAI,MAAM,EAAE,CAAC;gBAAC,MAAM,GAAG,KAAK,CAAC;gBAAC,SAAS;YAAC,CAAC;YACzC,IAAI,EAAE,KAAK,IAAI,EAAE,CAAC;gBAAC,MAAM,GAAG,IAAI,CAAC;gBAAC,SAAS;YAAC,CAAC;YAC7C,IAAI,EAAE,KAAK,GAAG,EAAE,CAAC;gBAAC,QAAQ,GAAG,KAAK,CAAC;gBAAC,SAAS;YAAC,CAAC;YAC/C,SAAS;QACX,CAAC;QACD,IAAI,EAAE,KAAK,GAAG,EAAE,CAAC;YAAC,QAAQ,GAAG,IAAI,CAAC;YAAC,SAAS;QAAC,CAAC;QAC9C,IAAI,EAAE,KAAK,GAAG;YAAE,KAAK,EAAE,CAAC;aACnB,IAAI,EAAE,KAAK,GAAG,EAAE,CAAC;YACpB,KAAK,EAAE,CAAC;YACR,IAAI,KAAK,KAAK,CAAC;gBAAE,OAAO,CAAC,CAAC;QAC5B,CAAC;IACH,CAAC;IACD,OAAO,CAAC,CAAC,CAAC;AACZ,CAAC;AAED,SAAS,wBAAwB,CAAC,IAAY;IAC5C,MAAM,QAAQ,GAAG,CAAC,CAAS,EAAY,EAAE;QACvC,IAAI,CAAC;YACH,MAAM,GAAG,GAAG,IAAI,CAAC,KAAK,CAAC,CAAC,CAA2B,CAAC;YACpD,IAAI,KAAK,CAAC,OAAO,CAAC,GAAG,CAAC,QAAQ,CAAC,EAAE,CAAC;gBAChC,OAAO,GAAG,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAe,EAAE,CAAC,OAAO,CAAC,KAAK,QAAQ,CAAC,CAAC;YACxE,CAAC;QACH,CAAC;QAAC,MAAM,CAAC;YACP,mBAAmB;QACrB,CAAC;QACD,OAAO,EAAE,CAAC;IACZ,CAAC,CAAC;IACF,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,8BAA8B,CAAC,CAAC;IAC1D,IAAI,MAAM,EAAE,CAAC;QACX,MAAM,GAAG,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC;QAChC,IAAI,GAAG,CAAC,MAAM,GAAG,CAAC;YAAE,OAAO,GAAG,CAAC;IACjC,CAAC;IACD,sEAAsE;IACtE,qEAAqE;IACrE,0DAA0D;IAC1D,KAAK,IAAI,KAAK,GAAG,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,EAAE,KAAK,KAAK,CAAC,CAAC,EAAE,KAAK,GAAG,IAAI,CAAC,OAAO,CAAC,GAAG,EAAE,KAAK,GAAG,CAAC,CAAC,EAAE,CAAC;QACvF,MAAM,GAAG,GAAG,aAAa,CAAC,IAAI,EAAE,KAAK,CAAC,CAAC;QACvC,IAAI,GAAG,KAAK,CAAC,CAAC;YAAE,SAAS;QACzB,MAAM,SAAS,GAAG,IAAI,CAAC,KAAK,CAAC,KAAK,EAAE,GAAG,GAAG,CAAC,CAAC,CAAC;QAC7C,IAAI,CAAC,iBAAiB,CAAC,IAAI,CAAC,SAAS,CAAC;YAAE,SAAS;QACjD,MAAM,GAAG,GAAG,QAAQ,CAAC,SAAS,CAAC,CAAC;QAChC,IAAI,GAAG,CAAC,MAAM,GAAG,CAAC;YAAE,OAAO,GAAG,CAAC;IACjC,CAAC;IACD,OAAO,EAAE,CAAC;AACZ,CAAC;AAED,SAAS,cAAc,CAAC,QAAgB;IACtC,IAAI,MAAe,CAAC;IACpB,IAAI,CAAC;QACH,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC;IAChC,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,IAAI,CAAC;IACd,CAAC;IACD,MAAM,GAAG,GAAG,MAA+B,CAAC;IAC5C,MAAM,CAAC,GAAG,OAAO,GAAG,CAAC,OAAO,KAAK,QAAQ,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,WAAW,EAAE,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;IAClF,IAAI,CAAC,KAAK,UAAU,IAAI,CAAC,KAAK,SAAS;QAAE,OAAO,UAAU,CAAC;IAC3D,IAAI,CAAC,KAAK,kBAAkB,IAAI,CAAC,KAAK,kBAAkB;QAAE,OAAO,kBAAkB,CAAC;IACpF,IAAI,CAAC,KAAK,UAAU;QAAE,OAAO,UAAU,CAAC;IACxC,OAAO,IAAI,CAAC;AACd,CAAC;AAED,SAAS,8BAA8B,CAAC,IAAY;IAClD,MAAM,QAAQ,GAAa,EAAE,CAAC;IAE9B,oEAAoE;IACpE,iEAAiE;IACjE,6DAA6D;IAC7D,MAAM,QAAQ,GAAG,wBAAwB,CAAC,IAAI,CAAC,CAAC;IAChD,QAAQ,CAAC,IAAI,CAAC,GAAG,QAAQ,CAAC,CAAC;IAE3B,MAAM,QAAQ,GAAG,IAAI,CAAC,KAAK,CAAC,wDAAwD,CAAC,CAAC;IACtF,IAAI,QAAQ,EAAE,CAAC;QACb,MAAM,KAAK,GAAG,QAAQ,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,UAAU,CAAC,GAAG,CAAC,CAAC,CAAC;QACnF,QAAQ,CAAC,IAAI,CAAC,GAAG,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,EAAE,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC;IAClE,CAAC;IAED,MAAM,UAAU,GAAG,IAAI,CAAC,KAAK,CAAC,6CAA6C,CAAC,CAAC;IAC7E,IAAI,UAAU,EAAE,CAAC;QACf,MAAM,KAAK,GAAG,UAAU,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,UAAU,CAAC,GAAG,CAAC,CAAC,CAAC;QACrF,QAAQ,CAAC,IAAI,CAAC,GAAG,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,EAAE,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC;IAClE,CAAC;IAED,OAAO,QAAQ,CAAC;AAClB,CAAC;AAED,SAAS,kBAAkB,CAAC,IAAY;IACtC,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,EAAE,CAAC;IAC5B,IAAI,aAAa,CAAC,IAAI,CAAC,OAAO,CAAC;QAAE,OAAO,SAAS,CAAC;IAClD,IAAI,aAAa,CAAC,IAAI,CAAC,OAAO,CAAC;QAAE,OAAO,UAAU,CAAC;IACnD,IAAI,WAAW,CAAC,IAAI,CAAC,OAAO,CAAC;QAAE,OAAO,QAAQ,CAAC;IAC/C,OAAO,IAAI,CAAC;AACd,CAAC;AAED,MAAM,OAAO,oBAAoB;IAC/B,KAAK,CAAC,IAAY;QAChB,mBAAmB;QACnB,gFAAgF;QAChF,uEAAuE;QACvE,kEAAkE;QAClE,iEAAiE;QACjE,+DAA+D;QAC/D,iEAAiE;QACjE,kBAAkB;QAClB,mEAAmE;QACnE,2CAA2C;QAC3C,iEAAiE;QACjE,gEAAgE;QAChE,8BAA8B;QAC9B,MAAM,WAAW,GAAG,kBAAkB,CAAC,IAAI,CAAC,CAAC;QAC7C,MAAM,QAAQ,GAAG,8BAA8B,CAAC,IAAI,CAAC,CAAC;QACtD,IAAI,WAAW,EAAE,CAAC;YAChB,OAAO,EAAE,OAAO,EAAE,WAAW,EAAE,QAAQ,EAAE,CAAC;QAC5C,CAAC;QACD,MAAM,OAAO,GAAG,qBAAqB,CAAC,IAAI,CAAC,CAAC;QAC5C,IAAI,CAAC,OAAO,EAAE,CAAC;YACb,OAAO;gBACL,OAAO,EAAE,kBAAkB;gBAC3B,QAAQ,EAAE;oBACR,sHAAsH;oBACtH,GAAG,QAAQ;iBACZ;aACF,CAAC;QACJ,CAAC;QACD,MAAM,KAAK,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC;QACpC,MAAM,OAAO,GAAoB,KAAK,CAAC,QAAQ,CAAC,kBAAkB,CAAC,CAAC,CAAC,CAAC,kBAAkB,CAAC,CAAC,CAAC,UAAU,CAAC;QACtG,OAAO,EAAE,OAAO,EAAE,QAAQ,EAAE,CAAC;IAC/B,CAAC;IAED,SAAS,CAAC,IAAY;QACpB,mEAAmE;QACnE,mEAAmE;QACnE,MAAM,OAAO,GAAG,kBAAkB,CAAC,IAAI,CAAC,CAAC;QACzC,MAAM,QAAQ,GAAG,8BAA8B,CAAC,IAAI,CAAC,CAAC;QACtD,IAAI,CAAC,OAAO,EAAE,CAAC;YACb,OAAO;gBACL,OAAO,EAAE,UAAU;gBACnB,QAAQ,EAAE;oBACR,oGAAoG;oBACpG,GAAG,QAAQ;iBACZ;aACF,CAAC;QACJ,CAAC;QACD,OAAO,EAAE,OAAO,EAAE,QAAQ,EAAE,CAAC;IAC/B,CAAC;CACF;AAED,MAAM,OAAO,kBAAmB,SAAQ,KAAK;IAC3C,YAAY,OAAe,IAAI,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,GAAG,oBAAoB,CAAC,CAAC,CAAC;CACnF"}

package/dist/review/reviewer-prompt-builder.d.ts

@@ -1,5 +1,5 @@
-import type { ReviewTemplate } from './templates/shared.js';
-export type QualityReviewRoute = 'delegate' | 'execute-plan' | 'audit' | 'review' | 'verify' | 'investigate' | 'debug' | 'explore';
+import type { ReviewTemplate, ReviewTemplateContext } from './templates/shared.js';
+export type QualityReviewRoute = 'delegate' | 'execute-plan' | 'audit' | 'review' | 'verify' | 'investigate' | 'debug' | 'research';
 export declare class ReviewerPromptBuilder {
     private templates;
     private qualityTemplates;
@@ -8,25 +8,17 @@ export declare class ReviewerPromptBuilder {
         qualityForAP: ReviewTemplate;
         diff: ReviewTemplate;
     }, qualityTemplates?: Partial<Record<QualityReviewRoute, ReviewTemplate>>);
-    buildSpec(ctx: {
-        workerOutput: string;
-        brief: string;
-    }): {
+    buildSpec(ctx: ReviewTemplateContext): {
         systemPrompt: string;
         userPrompt: string;
     };
-    buildQualityAP(ctx: {
-        workerOutput: string;
-        brief: string;
+    buildQualityAP(ctx: ReviewTemplateContext & {
         route?: QualityReviewRoute;
     }): {
         systemPrompt: string;
         userPrompt: string;
     };
-    buildDiff(ctx: {
-        workerOutput: string;
-        brief: string;
-    }): {
+    buildDiff(ctx: ReviewTemplateContext): {
         systemPrompt: string;
         userPrompt: string;
     };

package/dist/review/reviewer-prompt-builder.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"reviewer-prompt-builder.d.ts","sourceRoot":"","sources":["../../src/review/reviewer-prompt-builder.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,uBAAuB,CAAC;AAE5D,MAAM,MAAM,kBAAkB,GAAG,UAAU,GAAG,cAAc,GAAG,OAAO,GAAG,QAAQ,GAAG,QAAQ,GAAG,aAAa,GAAG,OAAO,GAAG,SAAS,CAAC;AAEnI,qBAAa,qBAAqB;IAE9B,OAAO,CAAC,SAAS;IAKjB,OAAO,CAAC,gBAAgB;gBALhB,SAAS,EAAE;QACjB,IAAI,EAAE,cAAc,CAAC;QACrB,YAAY,EAAE,cAAc,CAAC;QAC7B,IAAI,EAAE,cAAc,CAAC;KACtB,EACO,gBAAgB,GAAE,OAAO,CAAC,MAAM,CAAC,kBAAkB,EAAE,cAAc,CAAC,CAAM;IAGpF,SAAS,CAAC,GAAG,EAAE;QAAE,YAAY,EAAE,MAAM,CAAC;QAAC,KAAK,EAAE,MAAM,CAAA;KAAE,GAAG;QAAE,YAAY,EAAE,MAAM,CAAC;QAAC,UAAU,EAAE,MAAM,CAAA;KAAE;IAOrG,cAAc,CAAC,GAAG,EAAE;QAAE,YAAY,EAAE,MAAM,CAAC;QAAC,KAAK,EAAE,MAAM,CAAC;QAAC,KAAK,CAAC,EAAE,kBAAkB,CAAA;KAAE,GAAG;QAAE,YAAY,EAAE,MAAM,CAAC;QAAC,UAAU,EAAE,MAAM,CAAA;KAAE;IAQtI,SAAS,CAAC,GAAG,EAAE;QAAE,YAAY,EAAE,MAAM,CAAC;QAAC,KAAK,EAAE,MAAM,CAAA;KAAE,GAAG;QAAE,YAAY,EAAE,MAAM,CAAC;QAAC,UAAU,EAAE,MAAM,CAAA;KAAE;CAMtG"}
+{"version":3,"file":"reviewer-prompt-builder.d.ts","sourceRoot":"","sources":["../../src/review/reviewer-prompt-builder.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,qBAAqB,EAAE,MAAM,uBAAuB,CAAC;AAEnF,MAAM,MAAM,kBAAkB,GAAG,UAAU,GAAG,cAAc,GAAG,OAAO,GAAG,QAAQ,GAAG,QAAQ,GAAG,aAAa,GAAG,OAAO,GAAG,UAAU,CAAC;AAEpI,qBAAa,qBAAqB;IAE9B,OAAO,CAAC,SAAS;IAKjB,OAAO,CAAC,gBAAgB;gBALhB,SAAS,EAAE;QACjB,IAAI,EAAE,cAAc,CAAC;QACrB,YAAY,EAAE,cAAc,CAAC;QAC7B,IAAI,EAAE,cAAc,CAAC;KACtB,EACO,gBAAgB,GAAE,OAAO,CAAC,MAAM,CAAC,kBAAkB,EAAE,cAAc,CAAC,CAAM;IAGpF,SAAS,CAAC,GAAG,EAAE,qBAAqB,GAAG;QAAE,YAAY,EAAE,MAAM,CAAC;QAAC,UAAU,EAAE,MAAM,CAAA;KAAE;IAOnF,cAAc,CAAC,GAAG,EAAE,qBAAqB,GAAG;QAAE,KAAK,CAAC,EAAE,kBAAkB,CAAA;KAAE,GAAG;QAAE,YAAY,EAAE,MAAM,CAAC;QAAC,UAAU,EAAE,MAAM,CAAA;KAAE;IAQzH,SAAS,CAAC,GAAG,EAAE,qBAAqB,GAAG;QAAE,YAAY,EAAE,MAAM,CAAC;QAAC,UAAU,EAAE,MAAM,CAAA;KAAE;CAMpF"}

package/dist/review/reviewer-prompt-builder.js.map

@@ -1 +1 @@
-{"version":3,"file":"reviewer-prompt-builder.js","sourceRoot":"","sources":["../../src/review/reviewer-prompt-builder.ts"],"names":[],"mappings":"AAIA,MAAM,OAAO,qBAAqB;IAEtB;IAKA;IANV,YACU,SAIP,EACO,mBAAwE,EAAE;QAL1E,cAAS,GAAT,SAAS,CAIhB;QACO,qBAAgB,GAAhB,gBAAgB,CAA0D;IACjF,CAAC;IAEJ,SAAS,CAAC,GAA4C;QACpD,OAAO;YACL,YAAY,EAAE,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,YAAY;YAC9C,UAAU,EAAE,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,eAAe,CAAC,GAAG,CAAC;SACrD,CAAC;IACJ,CAAC;IAED,cAAc,CAAC,GAAwE;QACrF,MAAM,QAAQ,GAAG,CAAC,GAAG,CAAC,KAAK,KAAK,SAAS,IAAI,IAAI,CAAC,gBAAgB,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC,IAAI,IAAI,CAAC,SAAS,CAAC,YAAY,CAAC;QAC9G,OAAO;YACL,YAAY,EAAE,QAAQ,CAAC,YAAY;YACnC,UAAU,EAAE,QAAQ,CAAC,eAAe,CAAC,GAAG,CAAC;SAC1C,CAAC;IACJ,CAAC;IAED,SAAS,CAAC,GAA4C;QACpD,OAAO;YACL,YAAY,EAAE,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,YAAY;YAC9C,UAAU,EAAE,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,eAAe,CAAC,GAAG,CAAC;SACrD,CAAC;IACJ,CAAC;CACF"}
+{"version":3,"file":"reviewer-prompt-builder.js","sourceRoot":"","sources":["../../src/review/reviewer-prompt-builder.ts"],"names":[],"mappings":"AAIA,MAAM,OAAO,qBAAqB;IAEtB;IAKA;IANV,YACU,SAIP,EACO,mBAAwE,EAAE;QAL1E,cAAS,GAAT,SAAS,CAIhB;QACO,qBAAgB,GAAhB,gBAAgB,CAA0D;IACjF,CAAC;IAEJ,SAAS,CAAC,GAA0B;QAClC,OAAO;YACL,YAAY,EAAE,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,YAAY;YAC9C,UAAU,EAAE,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,eAAe,CAAC,GAAG,CAAC;SACrD,CAAC;IACJ,CAAC;IAED,cAAc,CAAC,GAA2D;QACxE,MAAM,QAAQ,GAAG,CAAC,GAAG,CAAC,KAAK,KAAK,SAAS,IAAI,IAAI,CAAC,gBAAgB,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC,IAAI,IAAI,CAAC,SAAS,CAAC,YAAY,CAAC;QAC9G,OAAO;YACL,YAAY,EAAE,QAAQ,CAAC,YAAY;YACnC,UAAU,EAAE,QAAQ,CAAC,eAAe,CAAC,GAAG,CAAC;SAC1C,CAAC;IACJ,CAAC;IAED,SAAS,CAAC,GAA0B;QAClC,OAAO;YACL,YAAY,EAAE,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,YAAY;YAC9C,UAAU,EAAE,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,eAAe,CAAC,GAAG,CAAC;SACrD,CAAC;IACJ,CAAC;CACF"}

package/dist/review/templates/annotator-audit.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"annotator-audit.d.ts","sourceRoot":"","sources":["../../../src/review/templates/annotator-audit.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,uBAAuB,CAAC;AAE/D,eAAO,MAAM,sBAAsB,EAAE,iBAGpC,CAAC"}
+{"version":3,"file":"annotator-audit.d.ts","sourceRoot":"","sources":["../../../src/review/templates/annotator-audit.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,uBAAuB,CAAC;AAE/D,eAAO,MAAM,sBAAsB,EAAE,iBAgBpC,CAAC"}

package/dist/review/templates/annotator-audit.js

@@ -1,5 +1,18 @@
 export const annotatorAuditTemplate = {
     role: 'audit',
     onBriefCheck: 'For each finding, ask: is this the kind of issue the audit asked for? A security audit should produce security findings, not style nits.',
+    evidenceRule: [
+        '- Audit findings come in three valid shapes:',
+        '  1. Doc quote: a verbatim passage from the document showing the issue.',
+        '  2. Absence-reference: a precise pointer to where the doc *should* address something but doesn\'t (e.g. "Section 3.2 lists failure modes but is silent on queue overflow").',
+        '  3. Claim + contradiction: the doc\'s claim plus a quote from the source it contradicts.',
+        '- A finding without one of these three forms is speculation; downgrade to low or drop.',
+    ].join('\n'),
+    scopeRule: [
+        '- The document and what it directly references are in scope.',
+        '- Cross-section / cross-doc reasoning IS the value of an audit — do not penalize as speculation.',
+        '- Repository-wide enumeration / globbing is out of scope; flag findings whose evidence depends on broad enumeration.',
+        '- Coding-style nits on inline examples belong in a code review, not an audit; flag as off-brief.',
+    ].join('\n'),
 };
 //# sourceMappingURL=annotator-audit.js.map

package/dist/review/templates/annotator-audit.js.map

@@ -1 +1 @@
-{"version":3,"file":"annotator-audit.js","sourceRoot":"","sources":["../../../src/review/templates/annotator-audit.ts"],"names":[],"mappings":"AAEA,MAAM,CAAC,MAAM,sBAAsB,GAAsB;IACvD,IAAI,EAAE,OAAO;IACb,YAAY,EAAE,0IAA0I;CACzJ,CAAC"}
+{"version":3,"file":"annotator-audit.js","sourceRoot":"","sources":["../../../src/review/templates/annotator-audit.ts"],"names":[],"mappings":"AAEA,MAAM,CAAC,MAAM,sBAAsB,GAAsB;IACvD,IAAI,EAAE,OAAO;IACb,YAAY,EAAE,0IAA0I;IACxJ,YAAY,EAAE;QACZ,8CAA8C;QAC9C,yEAAyE;QACzE,8KAA8K;QAC9K,2FAA2F;QAC3F,wFAAwF;KACzF,CAAC,IAAI,CAAC,IAAI,CAAC;IACZ,SAAS,EAAE;QACT,8DAA8D;QAC9D,kGAAkG;QAClG,sHAAsH;QACtH,kGAAkG;KACnG,CAAC,IAAI,CAAC,IAAI,CAAC;CACb,CAAC"}

package/dist/review/templates/annotator-debug.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"annotator-debug.d.ts","sourceRoot":"","sources":["../../../src/review/templates/annotator-debug.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,uBAAuB,CAAC;AAE/D,eAAO,MAAM,sBAAsB,EAAE,iBAGpC,CAAC"}
+{"version":3,"file":"annotator-debug.d.ts","sourceRoot":"","sources":["../../../src/review/templates/annotator-debug.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,uBAAuB,CAAC;AAE/D,eAAO,MAAM,sBAAsB,EAAE,iBAapC,CAAC"}

package/dist/review/templates/annotator-debug.js

@@ -1,5 +1,15 @@
 export const annotatorDebugTemplate = {
     role: 'debugging hypothesis',
-    onBriefCheck: 'Each finding should be a hypothesis, root-cause claim, or evidence (reproducer, error pattern, code path). Flag findings that do not logically follow from cited evidence or that exceed what the trace actually shows.',
+    onBriefCheck: 'Each finding should be a hypothesis, root-cause claim, or evidence (reproducer, error pattern, code path).',
+    evidenceRule: [
+        '- Debug findings are hypotheses with reasoning chains.',
+        '- Evidence: reproducer + traced code path (file:line) + observed output.',
+        '- Hypothesis-level findings with PARTIAL evidence are valid — that is debugging.',
+        '- Severity reflects evidence strength: confirmed root cause = high; plausible = medium; ruled out = low or drop.',
+    ].join('\n'),
+    scopeRule: [
+        '- Cross-file tracing is in scope and required to follow the failure path.',
+        '- Out of scope: applied fixes (the worker should propose, not apply); unrelated code-review remarks.',
+    ].join('\n'),
 };
 //# sourceMappingURL=annotator-debug.js.map

package/dist/review/templates/annotator-debug.js.map

@@ -1 +1 @@
-{"version":3,"file":"annotator-debug.js","sourceRoot":"","sources":["../../../src/review/templates/annotator-debug.ts"],"names":[],"mappings":"AAEA,MAAM,CAAC,MAAM,sBAAsB,GAAsB;IACvD,IAAI,EAAE,sBAAsB;IAC5B,YAAY,EAAE,yNAAyN;CACxO,CAAC"}
+{"version":3,"file":"annotator-debug.js","sourceRoot":"","sources":["../../../src/review/templates/annotator-debug.ts"],"names":[],"mappings":"AAEA,MAAM,CAAC,MAAM,sBAAsB,GAAsB;IACvD,IAAI,EAAE,sBAAsB;IAC5B,YAAY,EAAE,4GAA4G;IAC1H,YAAY,EAAE;QACZ,wDAAwD;QACxD,0EAA0E;QAC1E,kFAAkF;QAClF,kHAAkH;KACnH,CAAC,IAAI,CAAC,IAAI,CAAC;IACZ,SAAS,EAAE;QACT,2EAA2E;QAC3E,sGAAsG;KACvG,CAAC,IAAI,CAAC,IAAI,CAAC;CACb,CAAC"}

package/dist/review/templates/annotator-investigate.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"annotator-investigate.d.ts","sourceRoot":"","sources":["../../../src/review/templates/annotator-investigate.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,uBAAuB,CAAC;AAE/D,eAAO,MAAM,4BAA4B,EAAE,iBAG1C,CAAC"}
+{"version":3,"file":"annotator-investigate.d.ts","sourceRoot":"","sources":["../../../src/review/templates/annotator-investigate.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,uBAAuB,CAAC;AAE/D,eAAO,MAAM,4BAA4B,EAAE,iBAY1C,CAAC"}

package/dist/review/templates/annotator-investigate.js

@@ -1,5 +1,14 @@
 export const annotatorInvestigateTemplate = {
     role: 'codebase investigation',
-    onBriefCheck: 'Each finding should be relevant to the question. Findings may be code-level (file:line cited in evidence) or project-level synthesis (what was searched, what was not found). Flag findings whose evidence does not support the claim or whose claim drifts from the question.',
+    onBriefCheck: 'Each finding should be relevant to the question.',
+    evidenceRule: [
+        '- Present-thing citations: real `file:line` from files actually read, with a quote or summary.',
+        '- Absent-thing citations: explicit "searched <pattern> in <path>, no matches" — negative findings are legitimate answers and must NOT be downgraded for lacking a code quote.',
+        '- Synthesis findings: cite each link in the reasoning chain by file:line.',
+    ].join('\n'),
+    scopeRule: [
+        '- Wherever the question leads is in scope; the question may not name files.',
+        '- Drift into unrelated code-review remarks is out of scope.',
+    ].join('\n'),
 };
 //# sourceMappingURL=annotator-investigate.js.map

package/dist/review/templates/annotator-investigate.js.map

@@ -1 +1 @@
-{"version":3,"file":"annotator-investigate.js","sourceRoot":"","sources":["../../../src/review/templates/annotator-investigate.ts"],"names":[],"mappings":"AAEA,MAAM,CAAC,MAAM,4BAA4B,GAAsB;IAC7D,IAAI,EAAE,wBAAwB;IAC9B,YAAY,EAAE,gRAAgR;CAC/R,CAAC"}
+{"version":3,"file":"annotator-investigate.js","sourceRoot":"","sources":["../../../src/review/templates/annotator-investigate.ts"],"names":[],"mappings":"AAEA,MAAM,CAAC,MAAM,4BAA4B,GAAsB;IAC7D,IAAI,EAAE,wBAAwB;IAC9B,YAAY,EAAE,kDAAkD;IAChE,YAAY,EAAE;QACZ,gGAAgG;QAChG,+KAA+K;QAC/K,2EAA2E;KAC5E,CAAC,IAAI,CAAC,IAAI,CAAC;IACZ,SAAS,EAAE;QACT,6EAA6E;QAC7E,6DAA6D;KAC9D,CAAC,IAAI,CAAC,IAAI,CAAC;CACb,CAAC"}

package/dist/review/templates/annotator-review.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"annotator-review.d.ts","sourceRoot":"","sources":["../../../src/review/templates/annotator-review.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,uBAAuB,CAAC;AAE/D,eAAO,MAAM,uBAAuB,EAAE,iBAGrC,CAAC"}
+{"version":3,"file":"annotator-review.d.ts","sourceRoot":"","sources":["../../../src/review/templates/annotator-review.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,uBAAuB,CAAC;AAE/D,eAAO,MAAM,uBAAuB,EAAE,iBAarC,CAAC"}

package/dist/review/templates/annotator-review.js

@@ -1,5 +1,15 @@
 export const annotatorReviewTemplate = {
     role: 'code review',
     onBriefCheck: 'For each finding, ask: is this within the requested focus area? A security review should produce security findings, not formatting nits.',
+    evidenceRule: [
+        '- Code-review findings must cite `file:line` from the named files.',
+        '- Evidence must include a verbatim code quote, not paraphrase.',
+        '- Findings without a quotable code excerpt are speculation; downgrade or drop.',
+    ].join('\n'),
+    scopeRule: [
+        '- Only the named files are in scope. Behavior of direct callers/callees may be referenced when visible in the named files.',
+        '- Speculation about untouched files is out of scope.',
+        '- Doc/spec issues belong in an audit, not a review — flag as off-brief.',
+    ].join('\n'),
 };
 //# sourceMappingURL=annotator-review.js.map

package/dist/review/templates/annotator-review.js.map

@@ -1 +1 @@
-{"version":3,"file":"annotator-review.js","sourceRoot":"","sources":["../../../src/review/templates/annotator-review.ts"],"names":[],"mappings":"AAEA,MAAM,CAAC,MAAM,uBAAuB,GAAsB;IACxD,IAAI,EAAE,aAAa;IACnB,YAAY,EAAE,0IAA0I;CACzJ,CAAC"}
+{"version":3,"file":"annotator-review.js","sourceRoot":"","sources":["../../../src/review/templates/annotator-review.ts"],"names":[],"mappings":"AAEA,MAAM,CAAC,MAAM,uBAAuB,GAAsB;IACxD,IAAI,EAAE,aAAa;IACnB,YAAY,EAAE,0IAA0I;IACxJ,YAAY,EAAE;QACZ,oEAAoE;QACpE,gEAAgE;QAChE,gFAAgF;KACjF,CAAC,IAAI,CAAC,IAAI,CAAC;IACZ,SAAS,EAAE;QACT,4HAA4H;QAC5H,sDAAsD;QACtD,yEAAyE;KAC1E,CAAC,IAAI,CAAC,IAAI,CAAC;CACb,CAAC"}

package/dist/review/templates/annotator-shared.d.ts

@@ -5,6 +5,12 @@ export interface AnnotatorPromptContext {
 export interface AnnotatorTemplate {
     role: string;
     onBriefCheck: string;
+    /** Per-tool evidence rule. Tells the annotator what counts as
+     *  grounded evidence for findings from this tool. */
+    evidenceRule: string;
+    /** Per-tool scope rule. Tells the annotator what is in/out of scope
+     *  for findings from this tool. */
+    scopeRule: string;
 }
-export declare const ANNOTATOR_RUBRIC: string;
+export declare function buildAnnotatorRubric(template: AnnotatorTemplate): string;
 //# sourceMappingURL=annotator-shared.d.ts.map

package/dist/review/templates/annotator-shared.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"annotator-shared.d.ts","sourceRoot":"","sources":["../../../src/review/templates/annotator-shared.ts"],"names":[],"mappings":"AAGA,MAAM,WAAW,sBAAsB;IACrC,YAAY,EAAE,MAAM,CAAC;IACrB,KAAK,EAAE,MAAM,CAAC;CACf;AAED,MAAM,WAAW,iBAAiB;IAChC,IAAI,EAAE,MAAM,CAAC;IACb,YAAY,EAAE,MAAM,CAAC;CACtB;AAED,eAAO,MAAM,gBAAgB,QA0DrB,CAAC"}
+{"version":3,"file":"annotator-shared.d.ts","sourceRoot":"","sources":["../../../src/review/templates/annotator-shared.ts"],"names":[],"mappings":"AAGA,MAAM,WAAW,sBAAsB;IACrC,YAAY,EAAE,MAAM,CAAC;IACrB,KAAK,EAAE,MAAM,CAAC;CACf;AAED,MAAM,WAAW,iBAAiB;IAChC,IAAI,EAAE,MAAM,CAAC;IACb,YAAY,EAAE,MAAM,CAAC;IACrB;yDACqD;IACrD,YAAY,EAAE,MAAM,CAAC;IACrB;uCACmC;IACnC,SAAS,EAAE,MAAM,CAAC;CACnB;AAED,wBAAgB,oBAAoB,CAAC,QAAQ,EAAE,iBAAiB,GAAG,MAAM,CA+CxE"}

package/dist/review/templates/annotator-shared.js

@@ -1,6 +1,7 @@
 // Data shared by all 5 annotator templates. The actual prompt assembly
 // happens in review/annotator-prompt-builder.ts.
-export const ANNOTATOR_RUBRIC = String.raw `
+export function buildAnnotatorRubric(template) {
+    return String.raw `
 ## Output format (REQUIRED)
 
 Respond with exactly one fenced JSON code block AS THE LAST BLOCK in your
@@ -16,47 +17,35 @@ the worker presented them. Example:
     "suggestion": "Use a parameterized API or escape input",
     "annotatorConfidence": 90,
     "category": "security"
-  },
-  {
-    "id": "F2",
-    "severity": "medium",
-    "claim": "Auth check missing on /admin endpoint",
-    "evidence": "router.get('/admin', adminHandler) — no auth middleware applied",
-    "annotatorConfidence": 60,
-    "category": "security"
   }
 ]
 ` + '```' + `
 
 Field rules:
-- ` + '`id`' + `: assign sequentially F1, F2, F3, ... (your choice; must be unique).
+- ` + '`id`' + `: assign sequentially F1, F2, F3, ... (must be unique).
 - ` + '`severity`' + `: one of "critical" | "high" | "medium" | "low" — YOUR
-   final judgment, not the worker's. The worker's value is a hint; you may
-   dial it up or down based on actual impact (workers tend to inflate).
-   - critical: must fix before any other work (RCE, auth bypass, data loss)
-   - high:     serious bug / security issue, blocks release
-   - medium:   real issue, should fix soon
-   - low:      minor issue, nice to fix
-   Map worker-said "mid" -> "medium". When the worker omitted severity, judge.
+   final judgment. The worker's value is a hint; calibrate to actual impact.
 - ` + '`claim`' + `: one-sentence summary.
 - ` + '`evidence`' + `: REQUIRED, ≥20 chars, MUST be a verbatim quote from the
-   worker's output. The parser flags non-substring quotes — quote precisely.
+   worker's output.
 - ` + '`suggestion`' + `: optional; quote or paraphrase the worker's recommended fix.
-- ` + '`annotatorConfidence`' + `: integer 0-100. How confident YOU (reviewer) are
-   that the finding is correct, on-brief, and well-grounded:
-     80-100: defend without hesitation
-     60-79:  plausible, minor gaps
-     40-59:  thin evidence
-     20-39:  weak / off-brief
-      0-19:  unsupported / fabricated
+- ` + '`annotatorConfidence`' + `: integer 0-100. How confident YOU are
+   that the finding is correct, on-brief, and well-grounded.
 - ` + '`category`' + `: optional, one of: "missing_test" | "scope_creep" |
    "incomplete_impl" | "style_lint" | "security" | "performance" |
    "maintainability" | "doc_gap" | "doc_drift" | "contract_violation" |
    "coverage_gap" | "dead_code" | "queue_hygiene" | "other".
-   When omitted, the parser will infer from the claim text. Prefer to emit
-   it when the category is unambiguous from your reading.
 
-If the worker raised NO issues, return ` + '`[]`' + `. Surrounding prose is allowed
-but ignored by the parser — only the LAST ` + '```json' + ` block is read.
+## Tool-specific evidence rule (apply when judging "well-grounded")
+
+` + template.evidenceRule + `
+
+## Tool-specific scope rule (apply when judging "on-brief")
+
+` + template.scopeRule + `
+
+If the worker raised NO issues, return ` + '`[]`' + `. Surrounding prose is
+allowed but ignored by the parser — only the LAST ` + '```json' + ` block is read.
 `.trim();
+}
 //# sourceMappingURL=annotator-shared.js.map

package/dist/review/templates/annotator-shared.js.map

@@ -1 +1 @@
-{"version":3,"file":"annotator-shared.js","sourceRoot":"","sources":["../../../src/review/templates/annotator-shared.ts"],"names":[],"mappings":"AAAA,uEAAuE;AACvE,iDAAiD;AAYjD,MAAM,CAAC,MAAM,gBAAgB,GAAG,MAAM,CAAC,GAAG,CAAA;;;;;;;CAOzC,GAAG,WAAW,GAAG;;;;;;;;;;;;;;;;;;;CAmBjB,GAAG,KAAK,GAAG;;;GAGT,GAAG,MAAM,GAAG;GACZ,GAAG,YAAY,GAAG;;;;;;;;GAQlB,GAAG,SAAS,GAAG;GACf,GAAG,YAAY,GAAG;;GAElB,GAAG,cAAc,GAAG;GACpB,GAAG,uBAAuB,GAAG;;;;;;;GAO7B,GAAG,YAAY,GAAG;;;;;;;wCAOmB,GAAG,MAAM,GAAG;2CACT,GAAG,SAAS,GAAG;CACzD,CAAC,IAAI,EAAE,CAAC"}
+{"version":3,"file":"annotator-shared.js","sourceRoot":"","sources":["../../../src/review/templates/annotator-shared.ts"],"names":[],"mappings":"AAAA,uEAAuE;AACvE,iDAAiD;AAkBjD,MAAM,UAAU,oBAAoB,CAAC,QAA2B;IAC9D,OAAO,MAAM,CAAC,GAAG,CAAA;;;;;;;CAOlB,GAAG,WAAW,GAAG;;;;;;;;;;;CAWjB,GAAG,KAAK,GAAG;;;GAGT,GAAG,MAAM,GAAG;GACZ,GAAG,YAAY,GAAG;;GAElB,GAAG,SAAS,GAAG;GACf,GAAG,YAAY,GAAG;;GAElB,GAAG,cAAc,GAAG;GACpB,GAAG,uBAAuB,GAAG;;GAE7B,GAAG,YAAY,GAAG;;;;;;;CAOpB,GAAG,QAAQ,CAAC,YAAY,GAAG;;;;CAI3B,GAAG,QAAQ,CAAC,SAAS,GAAG;;wCAEe,GAAG,MAAM,GAAG;mDACD,GAAG,SAAS,GAAG;CACjE,CAAC,IAAI,EAAE,CAAC;AACT,CAAC"}

package/dist/review/templates/annotator-verify.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"annotator-verify.d.ts","sourceRoot":"","sources":["../../../src/review/templates/annotator-verify.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,uBAAuB,CAAC;AAE/D,eAAO,MAAM,uBAAuB,EAAE,iBAGrC,CAAC"}
+{"version":3,"file":"annotator-verify.d.ts","sourceRoot":"","sources":["../../../src/review/templates/annotator-verify.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,uBAAuB,CAAC;AAE/D,eAAO,MAAM,uBAAuB,EAAE,iBAarC,CAAC"}

package/dist/review/templates/annotator-verify.js

@@ -1,5 +1,15 @@
 export const annotatorVerifyTemplate = {
     role: 'verification report',
-    onBriefCheck: 'Each finding should map to one checklist item with evidence the criterion was met or unmet. Flag findings that do not correspond to any checklist item, or whose evidence does not actually demonstrate the claimed pass/fail status.',
+    onBriefCheck: 'Each finding should map to one checklist item with evidence the criterion was met or unmet.',
+    evidenceRule: [
+        '- Each Finding must map 1:1 to a checklist item.',
+        '- Evidence is execution output (test/build/command output) OR a code reference (`file:line`).',
+        '- A claimed PASS without evidence is speculation; downgrade or drop.',
+        '- Severity binding: PASS = low; FAIL = medium or high based on impact.',
+    ].join('\n'),
+    scopeRule: [
+        '- Only checklist items are in scope. Findings not tied to a checklist item are off-brief.',
+        '- All checklist items should be covered (one Finding per item, in order).',
+    ].join('\n'),
 };
 //# sourceMappingURL=annotator-verify.js.map

package/dist/review/templates/annotator-verify.js.map

@@ -1 +1 @@
-{"version":3,"file":"annotator-verify.js","sourceRoot":"","sources":["../../../src/review/templates/annotator-verify.ts"],"names":[],"mappings":"AAEA,MAAM,CAAC,MAAM,uBAAuB,GAAsB;IACxD,IAAI,EAAE,qBAAqB;IAC3B,YAAY,EAAE,uOAAuO;CACtP,CAAC"}
+{"version":3,"file":"annotator-verify.js","sourceRoot":"","sources":["../../../src/review/templates/annotator-verify.ts"],"names":[],"mappings":"AAEA,MAAM,CAAC,MAAM,uBAAuB,GAAsB;IACxD,IAAI,EAAE,qBAAqB;IAC3B,YAAY,EAAE,6FAA6F;IAC3G,YAAY,EAAE;QACZ,kDAAkD;QAClD,+FAA+F;QAC/F,sEAAsE;QACtE,wEAAwE;KACzE,CAAC,IAAI,CAAC,IAAI,CAAC;IACZ,SAAS,EAAE;QACT,2FAA2F;QAC3F,2EAA2E;KAC5E,CAAC,IAAI,CAAC,IAAI,CAAC;CACb,CAAC"}

package/dist/review/templates/diff-review.d.ts

@@ -1,3 +1,11 @@
 import type { ReviewTemplate } from './shared.js';
+/**
+ * Diff reviewer — the final cross-check that looks at the cumulative
+ * diff in isolation and decides if it's acceptable as a whole.
+ *
+ * Tool sweep #6 fix: the template name was a misnomer pre-fix —
+ * the reviewer received only the worker's text summary, never the
+ * actual diff. Now it sees the diff (matches the name).
+ */
 export declare const diffTemplate: ReviewTemplate;
 //# sourceMappingURL=diff-review.d.ts.map

package/dist/review/templates/diff-review.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"diff-review.d.ts","sourceRoot":"","sources":["../../../src/review/templates/diff-review.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,aAAa,CAAC;AAElD,eAAO,MAAM,YAAY,EAAE,cAK1B,CAAC"}
+{"version":3,"file":"diff-review.d.ts","sourceRoot":"","sources":["../../../src/review/templates/diff-review.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,aAAa,CAAC;AAElD;;;;;;;GAOG;AACH,eAAO,MAAM,YAAY,EAAE,cAgC1B,CAAC"}

package/dist/review/templates/diff-review.js

@@ -1,7 +1,39 @@
+/**
+ * Diff reviewer — the final cross-check that looks at the cumulative
+ * diff in isolation and decides if it's acceptable as a whole.
+ *
+ * Tool sweep #6 fix: the template name was a misnomer pre-fix —
+ * the reviewer received only the worker's text summary, never the
+ * actual diff. Now it sees the diff (matches the name).
+ */
 export const diffTemplate = {
-    systemPrompt: `You are reviewing a diff. Reply with EXACTLY one of: APPROVE, CONCERNS: <reasons>, or REJECT: <reason>`,
+    systemPrompt: [
+        'You are reviewing the cumulative diff produced by all rework rounds. Decide whether the overall change is acceptable.',
+        '',
+        'Reply with EXACTLY one of these single-line verdicts:',
+        '- `APPROVE` — the diff is acceptable as-is.',
+        '- `CONCERNS: <one-line summary of concerns>` — the diff has flaws but is on the right track.',
+        '- `REJECT: <one-line reason>` — the diff is wrong enough that it should not be applied.',
+        '',
+        'Decision criteria:',
+        '- Is the diff scoped to what the brief asked for? Out-of-scope edits are a CONCERN at minimum.',
+        '- Are the changes internally consistent (e.g., a renamed symbol updated everywhere it appears in the diff)?',
+        '- Does the diff introduce obvious correctness or security issues that the spec/quality reviews missed?',
+        '',
+        'Do NOT re-litigate the brief itself — that is the spec reviewer\'s job. Focus on the diff as a whole.',
+    ].join('\n'),
     buildUserPrompt(ctx) {
-        return `Task: ${ctx.brief}\n\nWorker output:\n${ctx.workerOutput}`;
+        const parts = [];
+        parts.push(`# Task brief\n${ctx.brief}`);
+        parts.push(`# Worker's most recent summary\n${ctx.workerOutput || '(no summary)'}`);
+        if (ctx.diff && ctx.diff.length > 0) {
+            parts.push(`# Cumulative diff (what you are reviewing)\n\n\`\`\`diff\n${ctx.diff}\n\`\`\``);
+        }
+        else {
+            parts.push(`# Cumulative diff\n(no file changes detected — APPROVE only if the brief requested a no-op)`);
+        }
+        parts.push(`# Decide\nReply with one of APPROVE, CONCERNS: <text>, or REJECT: <text>.`);
+        return parts.join('\n\n');
     },
 };
 //# sourceMappingURL=diff-review.js.map

package/dist/review/templates/diff-review.js.map

@@ -1 +1 @@
-{"version":3,"file":"diff-review.js","sourceRoot":"","sources":["../../../src/review/templates/diff-review.ts"],"names":[],"mappings":"AAEA,MAAM,CAAC,MAAM,YAAY,GAAmB;IAC1C,YAAY,EAAE,wGAAwG;IACtH,eAAe,CAAC,GAAG;QACjB,OAAO,SAAS,GAAG,CAAC,KAAK,uBAAuB,GAAG,CAAC,YAAY,EAAE,CAAC;IACrE,CAAC;CACF,CAAC"}
+{"version":3,"file":"diff-review.js","sourceRoot":"","sources":["../../../src/review/templates/diff-review.ts"],"names":[],"mappings":"AAEA;;;;;;;GAOG;AACH,MAAM,CAAC,MAAM,YAAY,GAAmB;IAC1C,YAAY,EAAE;QACZ,uHAAuH;QACvH,EAAE;QACF,uDAAuD;QACvD,6CAA6C;QAC7C,8FAA8F;QAC9F,yFAAyF;QACzF,EAAE;QACF,oBAAoB;QACpB,gGAAgG;QAChG,6GAA6G;QAC7G,wGAAwG;QACxG,EAAE;QACF,uGAAuG;KACxG,CAAC,IAAI,CAAC,IAAI,CAAC;IAEZ,eAAe,CAAC,GAAG;QACjB,MAAM,KAAK,GAAa,EAAE,CAAC;QAC3B,KAAK,CAAC,IAAI,CAAC,iBAAiB,GAAG,CAAC,KAAK,EAAE,CAAC,CAAC;QACzC,KAAK,CAAC,IAAI,CAAC,mCAAmC,GAAG,CAAC,YAAY,IAAI,cAAc,EAAE,CAAC,CAAC;QAEpF,IAAI,GAAG,CAAC,IAAI,IAAI,GAAG,CAAC,IAAI,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACpC,KAAK,CAAC,IAAI,CAAC,6DAA6D,GAAG,CAAC,IAAI,UAAU,CAAC,CAAC;QAC9F,CAAC;aAAM,CAAC;YACN,KAAK,CAAC,IAAI,CAAC,6FAA6F,CAAC,CAAC;QAC5G,CAAC;QAED,KAAK,CAAC,IAAI,CAAC,2EAA2E,CAAC,CAAC;QAExF,OAAO,KAAK,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;IAC5B,CAAC;CACF,CAAC"}

package/dist/review/templates/finding-criteria.d.ts

@@ -0,0 +1,15 @@
+/**
+ * Truly-shared finding criteria — the constants that apply *identically*
+ * across every consumer. Per-tool blocks live next to each tool, not here.
+ *
+ *  - SEVERITY_LADDER: same severity ladder, same anti-inflation hint, same
+ *    impact-anchored definitions for all 5 read-only tools.
+ *  - REVIEWER_AWARENESS_AP: same spec+quality reviewer expectation across
+ *    delegate / execute-plan / retry implementer prompts.
+ *
+ * Evidence + scope + annotator-awareness are calibrated per-tool; see
+ * tools/<tool>/implementer-criteria.ts for each tool's blocks.
+ */
+export declare const SEVERITY_LADDER: string;
+export declare const REVIEWER_AWARENESS_AP: string;
+//# sourceMappingURL=finding-criteria.d.ts.map