@zhixuan92/multi-model-agent-core 4.0.2 → 4.0.4

package/dist/reporting/headline-templates/debug.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"debug.d.ts","sourceRoot":"","sources":["../../../src/reporting/headline-templates/debug.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,yBAAyB,CAAC;AAEhE;;;;GAIG;AACH,eAAO,MAAM,qBAAqB,EAAE,gBAQnC,CAAC"}
+{"version":3,"file":"debug.d.ts","sourceRoot":"","sources":["../../../src/reporting/headline-templates/debug.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,yBAAyB,CAAC;AAGhE;;;;;;;;;;;;;;;;;;;;;;;GAuBG;AACH,eAAO,MAAM,qBAAqB,EAAE,gBAsBnC,CAAC"}

package/dist/reporting/headline-templates/debug.js

@@ -1,15 +1,45 @@
+import { countHighOrCritical, parseNarrativeFindings } from '../severity.js';
 /**
- * Compose a terminal headline for debug. Matches the composeTerminalHeadline
- * format that the legacy executor produced: "debug: 1/1 tasks complete".
- * Debug always dispatches exactly 1 task.
+ * Compose a terminal headline for debug.
+ *
+ * Tool sweep #4 rewrite: bring debug into the same shape as audit and
+ * review/verify so operator-facing logs stay consistent across tools:
+ *
+ *     [ok]    debug <path>: 3 findings (0 high)
+ *     [error] debug: 1 findings (1 high)
+ *     [ok]    debug completed
+ *
+ * Previously emitted "debug: 1/1 tasks complete" with no status prefix
+ * and no findings count — operator could not tell ok from error and
+ * had no signal about how many real findings landed.
+ *
+ * Source priority for findings (parallel to audit):
+ *   1. runResult.annotatedFindings (canonical — annotator's structured
+ *      output when verdict='annotated').
+ *   2. parseNarrativeFindings(runResult.output) — recovers `## Finding N:`
+ *      blocks from the implementer's narrative when the annotator errored.
+ *
+ * Note: debug's reportSchema.parse is intentionally a thrower (the tool
+ * doesn't emit a structured report), so `report` is always notApplicable
+ * here — there's no `report.findings` source to read.
  */
 export const debugHeadlineTemplate = {
-    compose({ report }) {
-        const r = report;
-        if (r && typeof r.rootCause === 'string' && r.rootCause.length > 0) {
-            return `debug: root cause — ${r.rootCause}`;
+    compose({ status, runResult, task }) {
+        const annotated = runResult?.annotatedFindings ?? [];
+        let findings = annotated;
+        if (findings.length === 0 && typeof runResult?.output === 'string') {
+            const narrative = parseNarrativeFindings(runResult.output);
+            if (narrative.length > 0)
+                findings = narrative;
         }
-        return 'debug: 1/1 tasks complete';
+        const path = task?.filePaths?.[0] || '';
+        if (findings.length === 0 && !path) {
+            return `[${status}] debug completed`;
+        }
+        const high = countHighOrCritical(findings);
+        return path
+            ? `[${status}] debug ${path}: ${findings.length} findings (${high} high)`
+            : `[${status}] debug: ${findings.length} findings (${high} high)`;
     },
 };
 //# sourceMappingURL=debug.js.map

package/dist/reporting/headline-templates/debug.js.map

@@ -1 +1 @@
-{"version":3,"file":"debug.js","sourceRoot":"","sources":["../../../src/reporting/headline-templates/debug.ts"],"names":[],"mappings":"AAEA;;;;GAIG;AACH,MAAM,CAAC,MAAM,qBAAqB,GAAqB;IACrD,OAAO,CAAC,EAAE,MAAM,EAAE;QAChB,MAAM,CAAC,GAAG,MAA6C,CAAC;QACxD,IAAI,CAAC,IAAI,OAAO,CAAC,CAAC,SAAS,KAAK,QAAQ,IAAI,CAAC,CAAC,SAAS,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACnE,OAAO,uBAAuB,CAAC,CAAC,SAAS,EAAE,CAAC;QAC9C,CAAC;QACD,OAAO,2BAA2B,CAAC;IACrC,CAAC;CACF,CAAC"}
+{"version":3,"file":"debug.js","sourceRoot":"","sources":["../../../src/reporting/headline-templates/debug.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,mBAAmB,EAAE,sBAAsB,EAAE,MAAM,gBAAgB,CAAC;AAE7E;;;;;;;;;;;;;;;;;;;;;;;GAuBG;AACH,MAAM,CAAC,MAAM,qBAAqB,GAAqB;IACrD,OAAO,CAAC,EAAE,MAAM,EAAE,SAAS,EAAE,IAAI,EAAE;QACjC,MAAM,SAAS,GAAG,SAAS,EAAE,iBAAiB,IAAI,EAAE,CAAC;QACrD,IAAI,QAAQ,GACV,SAA0C,CAAC;QAC7C,IAAI,QAAQ,CAAC,MAAM,KAAK,CAAC,IAAI,OAAO,SAAS,EAAE,MAAM,KAAK,QAAQ,EAAE,CAAC;YACnE,MAAM,SAAS,GAAG,sBAAsB,CAAC,SAAS,CAAC,MAAM,CAAC,CAAC;YAC3D,IAAI,SAAS,CAAC,MAAM,GAAG,CAAC;gBAAE,QAAQ,GAAG,SAAS,CAAC;QACjD,CAAC;QAED,MAAM,IAAI,GACP,IAA6C,EAAE,SAAS,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;QAEvE,IAAI,QAAQ,CAAC,MAAM,KAAK,CAAC,IAAI,CAAC,IAAI,EAAE,CAAC;YACnC,OAAO,IAAI,MAAM,mBAAmB,CAAC;QACvC,CAAC;QAED,MAAM,IAAI,GAAG,mBAAmB,CAAC,QAAQ,CAAC,CAAC;QAC3C,OAAO,IAAI;YACT,CAAC,CAAC,IAAI,MAAM,WAAW,IAAI,KAAK,QAAQ,CAAC,MAAM,cAAc,IAAI,QAAQ;YACzE,CAAC,CAAC,IAAI,MAAM,YAAY,QAAQ,CAAC,MAAM,cAAc,IAAI,QAAQ,CAAC;IACtE,CAAC;CACF,CAAC"}

package/dist/reporting/headline-templates/delegate.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"delegate.d.ts","sourceRoot":"","sources":["../../../src/reporting/headline-templates/delegate.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,yBAAyB,CAAC;AAIhE,eAAO,MAAM,wBAAwB,EAAE,gBAQtC,CAAC"}
+{"version":3,"file":"delegate.d.ts","sourceRoot":"","sources":["../../../src/reporting/headline-templates/delegate.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,yBAAyB,CAAC;AAKhE,eAAO,MAAM,wBAAwB,EAAE,gBA0CtC,CAAC"}

package/dist/reporting/headline-templates/delegate.js

@@ -1,12 +1,42 @@
 import { isNotApplicable } from '../not-applicable.js';
+import { firstSentenceOrTruncate } from '../headline-text.js';
 export const delegateHeadlineTemplate = {
-    compose({ report, status }) {
-        if (!report || isNotApplicable(report))
-            return `[${status}] no structured report available`;
+    compose({ report, status, runResult }) {
         const r = report;
-        const files = Array.isArray(r.filesChanged) ? r.filesChanged : [];
-        const summary = typeof r.summary === 'string' ? r.summary : '';
-        return `[${status}] ${summary} (${files.length} file${files.length === 1 ? '' : 's'})`;
+        const reportInapplicable = !r || isNotApplicable(r);
+        // Gap 13 fix (4.0.3+): file count source priority is
+        // `report.filesChanged` (rare — only when the worker emits proper
+        // structured output) → `runResult.filesWritten` (canonical
+        // runner-shell signal that always reflects reality, including
+        // synthetic `shell:<cmd>` entries from Gap 11). Pre-fix, the
+        // composer only read report.filesChanged and reported "(0 files)"
+        // even when the worker had successfully edited files via
+        // edit_file or run_shell.
+        const reportFiles = !reportInapplicable && Array.isArray(r?.filesChanged) ? r.filesChanged : [];
+        const runFiles = Array.isArray(runResult?.filesWritten) ? runResult.filesWritten : [];
+        const fileCount = reportFiles.length > 0 ? reportFiles.length : runFiles.length;
+        if (reportInapplicable && fileCount === 0) {
+            return `[${status}] delegate: no structured report available`;
+        }
+        // Gap 12 fix (4.0.3+): trim worker's narrative `summary` to first
+        // sentence (or 80-char truncate). Pre-fix, the entire summary —
+        // sometimes multi-sentence prose ending mid-thought — was inlined
+        // into the headline.
+        //
+        // Tool sweep #6 follow-up: when the structured report carries no
+        // summary, fall back to the worker's `output` text (also trimmed to
+        // first sentence) so the headline is informative even on no-op /
+        // 0-file outcomes. Pre-fix, those collapsed to `[incomplete] (0
+        // files)` with NO clue why.
+        const rawSummary = (!reportInapplicable && typeof r?.summary === 'string' && r.summary.length > 0)
+            ? r.summary
+            : (typeof runResult?.output === 'string' ? runResult.output : '');
+        const summary = firstSentenceOrTruncate(rawSummary);
+        const summaryClause = summary.length > 0 ? ` ${summary}` : '';
+        // Tool sweep #6 follow-up: prefix with `delegate:` for parity with
+        // audit / review / verify / debug / investigate headlines so the
+        // operator can tell the route at a glance.
+        return `[${status}] delegate:${summaryClause} (${fileCount} file${fileCount === 1 ? '' : 's'})`;
     },
 };
 //# sourceMappingURL=delegate.js.map

package/dist/reporting/headline-templates/delegate.js.map

@@ -1 +1 @@
-{"version":3,"file":"delegate.js","sourceRoot":"","sources":["../../../src/reporting/headline-templates/delegate.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,eAAe,EAAE,MAAM,sBAAsB,CAAC;AAEvD,MAAM,CAAC,MAAM,wBAAwB,GAAqB;IACxD,OAAO,CAAC,EAAE,MAAM,EAAE,MAAM,EAAE;QACxB,IAAI,CAAC,MAAM,IAAI,eAAe,CAAC,MAAM,CAAC;YAAE,OAAO,IAAI,MAAM,kCAAkC,CAAC;QAC5F,MAAM,CAAC,GAAG,MAAkC,CAAC;QAC7C,MAAM,KAAK,GAAG,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,EAAE,CAAC;QAClE,MAAM,OAAO,GAAG,OAAO,CAAC,CAAC,OAAO,KAAK,QAAQ,CAAC,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,CAAC;QAC/D,OAAO,IAAI,MAAM,KAAK,OAAO,KAAK,KAAK,CAAC,MAAM,QAAQ,KAAK,CAAC,MAAM,KAAK,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,GAAG,GAAG,CAAC;IACzF,CAAC;CACF,CAAC"}
+{"version":3,"file":"delegate.js","sourceRoot":"","sources":["../../../src/reporting/headline-templates/delegate.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,eAAe,EAAE,MAAM,sBAAsB,CAAC;AACvD,OAAO,EAAE,uBAAuB,EAAE,MAAM,qBAAqB,CAAC;AAE9D,MAAM,CAAC,MAAM,wBAAwB,GAAqB;IACxD,OAAO,CAAC,EAAE,MAAM,EAAE,MAAM,EAAE,SAAS,EAAE;QACnC,MAAM,CAAC,GAAG,MAA8D,CAAC;QACzE,MAAM,kBAAkB,GAAG,CAAC,CAAC,IAAI,eAAe,CAAC,CAAC,CAAC,CAAC;QAEpD,qDAAqD;QACrD,kEAAkE;QAClE,2DAA2D;QAC3D,8DAA8D;QAC9D,6DAA6D;QAC7D,kEAAkE;QAClE,yDAAyD;QACzD,0BAA0B;QAC1B,MAAM,WAAW,GAAG,CAAC,kBAAkB,IAAI,KAAK,CAAC,OAAO,CAAC,CAAC,EAAE,YAAY,CAAC,CAAC,CAAC,CAAC,CAAE,CAAC,YAAY,CAAC,CAAC,CAAC,EAAE,CAAC;QACjG,MAAM,QAAQ,GAAG,KAAK,CAAC,OAAO,CAAC,SAAS,EAAE,YAAY,CAAC,CAAC,CAAC,CAAC,SAAU,CAAC,YAAY,CAAC,CAAC,CAAC,EAAE,CAAC;QACvF,MAAM,SAAS,GAAG,WAAW,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,WAAW,CAAC,MAAM,CAAC,CAAC,CAAC,QAAQ,CAAC,MAAM,CAAC;QAEhF,IAAI,kBAAkB,IAAI,SAAS,KAAK,CAAC,EAAE,CAAC;YAC1C,OAAO,IAAI,MAAM,4CAA4C,CAAC;QAChE,CAAC;QAED,kEAAkE;QAClE,gEAAgE;QAChE,kEAAkE;QAClE,qBAAqB;QACrB,EAAE;QACF,iEAAiE;QACjE,oEAAoE;QACpE,iEAAiE;QACjE,gEAAgE;QAChE,4BAA4B;QAC5B,MAAM,UAAU,GAAG,CAAC,CAAC,kBAAkB,IAAI,OAAO,CAAC,EAAE,OAAO,KAAK,QAAQ,IAAI,CAAC,CAAC,OAAO,CAAC,MAAM,GAAG,CAAC,CAAC;YAChG,CAAC,CAAC,CAAC,CAAC,OAAO;YACX,CAAC,CAAC,CAAC,OAAO,SAAS,EAAE,MAAM,KAAK,QAAQ,CAAC,CAAC,CAAC,SAAS,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC;QACpE,MAAM,OAAO,GAAG,uBAAuB,CAAC,UAAU,CAAC,CAAC;QAEpD,MAAM,aAAa,GAAG,OAAO,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,IAAI,OAAO,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QAC9D,mEAAmE;QACnE,iEAAiE;QACjE,2CAA2C;QAC3C,OAAO,IAAI,MAAM,cAAc,aAAa,KAAK,SAAS,QAAQ,SAAS,KAAK,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,GAAG,GAAG,CAAC;IAClG,CAAC;CACF,CAAC"}

package/dist/reporting/headline-templates/execute-plan.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"execute-plan.d.ts","sourceRoot":"","sources":["../../../src/reporting/headline-templates/execute-plan.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,yBAAyB,CAAC;AAYhE,eAAO,MAAM,2BAA2B,EAAE,gBAczC,CAAC"}
+{"version":3,"file":"execute-plan.d.ts","sourceRoot":"","sources":["../../../src/reporting/headline-templates/execute-plan.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,yBAAyB,CAAC;AAahE,eAAO,MAAM,2BAA2B,EAAE,gBAyBzC,CAAC"}

package/dist/reporting/headline-templates/execute-plan.js

@@ -1,14 +1,26 @@
+import { firstSentenceOrTruncate } from '../headline-text.js';
 export const executePlanHeadlineTemplate = {
-    compose({ report, status }) {
+    compose({ report, status, runResult }) {
         const r = report;
+        // Tool sweep #7 (execute-plan): use the dash-cased route name
+        // `execute-plan` to match the HTTP path + envelope shapes (the
+        // pre-fix `execute_plan` underscore was the only tool diverging
+        // from kebab-case in operator-facing output) and ALWAYS include
+        // the [<status>] prefix for parity with audit/review/verify/debug/
+        // delegate.
         if (r?.taskOutcomes && Array.isArray(r.taskOutcomes) && r.taskOutcomes.length > 0) {
             const completed = r.taskOutcomes.filter((t) => t.status === 'completed' || t.status === 'success').length;
-            return `execute_plan: ${completed}/${r.taskOutcomes.length} tasks complete`;
+            return `[${status}] execute-plan: ${completed}/${r.taskOutcomes.length} tasks complete`;
         }
-        if (r?.summary && typeof r.summary === 'string') {
-            return `[${status}] execute_plan: ${r.summary}`;
-        }
-        return `[${status}] execute_plan`;
+        // Tool sweep #7: fall back to runResult.output when report has no
+        // summary. Mirrors the delegate fix — operator gets a meaningful
+        // reason on no-op outcomes instead of a bare `[<status>] execute-plan`.
+        const fallbackSrc = (r?.summary && typeof r.summary === 'string' && r.summary.length > 0)
+            ? r.summary
+            : (typeof runResult?.output === 'string' ? runResult.output : '');
+        const summary = firstSentenceOrTruncate(fallbackSrc);
+        const summaryClause = summary.length > 0 ? ` ${summary}` : '';
+        return `[${status}] execute-plan:${summaryClause}`;
     },
 };
 //# sourceMappingURL=execute-plan.js.map

package/dist/reporting/headline-templates/execute-plan.js.map

@@ -1 +1 @@
-{"version":3,"file":"execute-plan.js","sourceRoot":"","sources":["../../../src/reporting/headline-templates/execute-plan.ts"],"names":[],"mappings":"AAYA,MAAM,CAAC,MAAM,2BAA2B,GAAqB;IAC3D,OAAO,CAAC,EAAE,MAAM,EAAE,MAAM,EAAE;QACxB,MAAM,CAAC,GAAG,MAA2C,CAAC;QACtD,IAAI,CAAC,EAAE,YAAY,IAAI,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,YAAY,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAClF,MAAM,SAAS,GAAG,CAAC,CAAC,YAAY,CAAC,MAAM,CACrC,CAAC,CAAc,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,KAAK,WAAW,IAAI,CAAC,CAAC,MAAM,KAAK,SAAS,CACvE,CAAC,MAAM,CAAC;YACT,OAAO,iBAAiB,SAAS,IAAI,CAAC,CAAC,YAAY,CAAC,MAAM,iBAAiB,CAAC;QAC9E,CAAC;QACD,IAAI,CAAC,EAAE,OAAO,IAAI,OAAO,CAAC,CAAC,OAAO,KAAK,QAAQ,EAAE,CAAC;YAChD,OAAO,IAAI,MAAM,mBAAmB,CAAC,CAAC,OAAO,EAAE,CAAC;QAClD,CAAC;QACD,OAAO,IAAI,MAAM,gBAAgB,CAAC;IACpC,CAAC;CACF,CAAC"}
+{"version":3,"file":"execute-plan.js","sourceRoot":"","sources":["../../../src/reporting/headline-templates/execute-plan.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,uBAAuB,EAAE,MAAM,qBAAqB,CAAC;AAY9D,MAAM,CAAC,MAAM,2BAA2B,GAAqB;IAC3D,OAAO,CAAC,EAAE,MAAM,EAAE,MAAM,EAAE,SAAS,EAAE;QACnC,MAAM,CAAC,GAAG,MAA2C,CAAC;QACtD,8DAA8D;QAC9D,+DAA+D;QAC/D,gEAAgE;QAChE,gEAAgE;QAChE,mEAAmE;QACnE,YAAY;QACZ,IAAI,CAAC,EAAE,YAAY,IAAI,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,YAAY,CAAC,IAAI,CAAC,CAAC,YAAY,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAClF,MAAM,SAAS,GAAG,CAAC,CAAC,YAAY,CAAC,MAAM,CACrC,CAAC,CAAc,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,KAAK,WAAW,IAAI,CAAC,CAAC,MAAM,KAAK,SAAS,CACvE,CAAC,MAAM,CAAC;YACT,OAAO,IAAI,MAAM,mBAAmB,SAAS,IAAI,CAAC,CAAC,YAAY,CAAC,MAAM,iBAAiB,CAAC;QAC1F,CAAC;QACD,kEAAkE;QAClE,iEAAiE;QACjE,wEAAwE;QACxE,MAAM,WAAW,GAAG,CAAC,CAAC,EAAE,OAAO,IAAI,OAAO,CAAC,CAAC,OAAO,KAAK,QAAQ,IAAI,CAAC,CAAC,OAAO,CAAC,MAAM,GAAG,CAAC,CAAC;YACvF,CAAC,CAAC,CAAC,CAAC,OAAO;YACX,CAAC,CAAC,CAAC,OAAO,SAAS,EAAE,MAAM,KAAK,QAAQ,CAAC,CAAC,CAAC,SAAS,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC;QACpE,MAAM,OAAO,GAAG,uBAAuB,CAAC,WAAW,CAAC,CAAC;QACrD,MAAM,aAAa,GAAG,OAAO,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,IAAI,OAAO,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QAC9D,OAAO,IAAI,MAAM,kBAAkB,aAAa,EAAE,CAAC;IACrD,CAAC;CACF,CAAC"}

package/dist/reporting/headline-templates/review.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"review.d.ts","sourceRoot":"","sources":["../../../src/reporting/headline-templates/review.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,yBAAyB,CAAC;AAIhE,eAAO,MAAM,sBAAsB,EAAE,gBAYpC,CAAC"}
+{"version":3,"file":"review.d.ts","sourceRoot":"","sources":["../../../src/reporting/headline-templates/review.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,yBAAyB,CAAC;AAKhE,eAAO,MAAM,sBAAsB,EAAE,gBAwCpC,CAAC"}

package/dist/reporting/headline-templates/review.js

@@ -1,15 +1,39 @@
 import { isNotApplicable } from '../not-applicable.js';
+import { countHighOrCritical, parseNarrativeFindings } from '../severity.js';
 export const reviewHeadlineTemplate = {
-    compose({ report, status, taskBrief }) {
-        if (isNotApplicable(report)) {
-            return `[${status}] review: ${taskBrief}`;
-        }
+    compose({ report, status, taskBrief, runResult, task }) {
         const r = report;
-        if (!r?.findings) {
-            return `[${status}] review: ${taskBrief}`;
+        const reportInapplicable = !r || isNotApplicable(r);
+        // Source priority (4.0.3+, parallel to audit):
+        //   1. report.findings (structured)
+        //   2. runResult.annotatedFindings (annotator success path)
+        //   3. parseNarrativeFindings(runResult.output) (annotator error
+        //      fallback — recover from implementer's `## Finding N:` blocks)
+        const reportFindings = !reportInapplicable && Array.isArray(r?.findings) ? r.findings : [];
+        const annotated = runResult?.annotatedFindings ?? [];
+        let findings = reportFindings.length > 0
+            ? reportFindings
+            : annotated;
+        if (findings.length === 0 && typeof runResult?.output === 'string') {
+            const narrative = parseNarrativeFindings(runResult.output);
+            if (narrative.length > 0)
+                findings = narrative;
+        }
+        const blocking = countHighOrCritical(findings);
+        const path = (!reportInapplicable && typeof r?.filePath === 'string' ? r.filePath : '')
+            || task?.filePaths?.[0]
+            || '';
+        // Mirror audit's gap-A fix: only collapse to "review completed"
+        // when we have no path AND no findings. With a path, always emit
+        // the structured form so a clean review still names the file:
+        //   [ok] review packages/foo.ts: 0 findings (0 blocking)
+        // Previously this branch fell back to `${taskBrief}` which the
+        // brief compiler reduces to the route name, producing the
+        // operator-useless headline "[ok] review: review".
+        if (findings.length === 0 && !path) {
+            return `[${status}] review completed`;
         }
-        const blocking = r.findings.filter(f => f.severity === 'high').length;
-        return `[${status}] review ${r.filePath}: ${r.findings.length} findings (${blocking} blocking)`;
+        return `[${status}] review ${path}: ${findings.length} findings (${blocking} blocking)`;
     },
 };
 //# sourceMappingURL=review.js.map

package/dist/reporting/headline-templates/review.js.map

@@ -1 +1 @@
-{"version":3,"file":"review.js","sourceRoot":"","sources":["../../../src/reporting/headline-templates/review.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,eAAe,EAAE,MAAM,sBAAsB,CAAC;AAEvD,MAAM,CAAC,MAAM,sBAAsB,GAAqB;IACtD,OAAO,CAAC,EAAE,MAAM,EAAE,MAAM,EAAE,SAAS,EAAE;QACnC,IAAI,eAAe,CAAC,MAAM,CAAC,EAAE,CAAC;YAC5B,OAAO,IAAI,MAAM,aAAa,SAAS,EAAE,CAAC;QAC5C,CAAC;QACD,MAAM,CAAC,GAAG,MAAsB,CAAC;QACjC,IAAI,CAAC,CAAC,EAAE,QAAQ,EAAE,CAAC;YACjB,OAAO,IAAI,MAAM,aAAa,SAAS,EAAE,CAAC;QAC5C,CAAC;QACD,MAAM,QAAQ,GAAG,CAAC,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,MAAM,CAAC,CAAC,MAAM,CAAC;QACtE,OAAO,IAAI,MAAM,YAAY,CAAC,CAAC,QAAQ,KAAK,CAAC,CAAC,QAAQ,CAAC,MAAM,cAAc,QAAQ,YAAY,CAAC;IAClG,CAAC;CACF,CAAC"}
+{"version":3,"file":"review.js","sourceRoot":"","sources":["../../../src/reporting/headline-templates/review.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,eAAe,EAAE,MAAM,sBAAsB,CAAC;AACvD,OAAO,EAAE,mBAAmB,EAAE,sBAAsB,EAAE,MAAM,gBAAgB,CAAC;AAE7E,MAAM,CAAC,MAAM,sBAAsB,GAAqB;IACtD,OAAO,CAAC,EAAE,MAAM,EAAE,MAAM,EAAE,SAAS,EAAE,SAAS,EAAE,IAAI,EAAE;QACpD,MAAM,CAAC,GAAG,MAAkD,CAAC;QAC7D,MAAM,kBAAkB,GAAG,CAAC,CAAC,IAAI,eAAe,CAAC,CAAC,CAAC,CAAC;QAEpD,+CAA+C;QAC/C,oCAAoC;QACpC,4DAA4D;QAC5D,iEAAiE;QACjE,qEAAqE;QACrE,MAAM,cAAc,GAAG,CAAC,kBAAkB,IAAI,KAAK,CAAC,OAAO,CAAC,CAAC,EAAE,QAAQ,CAAC,CAAC,CAAC,CAAC,CAAE,CAAC,QAAQ,CAAC,CAAC,CAAC,EAAE,CAAC;QAC5F,MAAM,SAAS,GAAG,SAAS,EAAE,iBAAiB,IAAI,EAAE,CAAC;QACrD,IAAI,QAAQ,GACV,cAAc,CAAC,MAAM,GAAG,CAAC;YACvB,CAAC,CAAE,cAAgD;YACnD,CAAC,CAAE,SAA2C,CAAC;QACnD,IAAI,QAAQ,CAAC,MAAM,KAAK,CAAC,IAAI,OAAO,SAAS,EAAE,MAAM,KAAK,QAAQ,EAAE,CAAC;YACnE,MAAM,SAAS,GAAG,sBAAsB,CAAC,SAAS,CAAC,MAAM,CAAC,CAAC;YAC3D,IAAI,SAAS,CAAC,MAAM,GAAG,CAAC;gBAAE,QAAQ,GAAG,SAAS,CAAC;QACjD,CAAC;QAED,MAAM,QAAQ,GAAG,mBAAmB,CAAC,QAAQ,CAAC,CAAC;QAC/C,MAAM,IAAI,GACR,CAAC,CAAC,kBAAkB,IAAI,OAAO,CAAC,EAAE,QAAQ,KAAK,QAAQ,CAAC,CAAC,CAAC,CAAE,CAAC,QAAQ,CAAC,CAAC,CAAC,EAAE,CAAC;eACvE,IAA6C,EAAE,SAAS,EAAE,CAAC,CAAC,CAAC;eAC9D,EAAE,CAAC;QAER,gEAAgE;QAChE,iEAAiE;QACjE,8DAA8D;QAC9D,yDAAyD;QACzD,+DAA+D;QAC/D,0DAA0D;QAC1D,mDAAmD;QACnD,IAAI,QAAQ,CAAC,MAAM,KAAK,CAAC,IAAI,CAAC,IAAI,EAAE,CAAC;YACnC,OAAO,IAAI,MAAM,oBAAoB,CAAC;QACxC,CAAC;QAED,OAAO,IAAI,MAAM,YAAY,IAAI,KAAK,QAAQ,CAAC,MAAM,cAAc,QAAQ,YAAY,CAAC;IAC1F,CAAC;CACF,CAAC"}

package/dist/reporting/headline-templates/verify.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"verify.d.ts","sourceRoot":"","sources":["../../../src/reporting/headline-templates/verify.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,yBAAyB,CAAC;AAIhE,eAAO,MAAM,sBAAsB,EAAE,gBAQpC,CAAC"}
+{"version":3,"file":"verify.d.ts","sourceRoot":"","sources":["../../../src/reporting/headline-templates/verify.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,yBAAyB,CAAC;AAIhE,eAAO,MAAM,sBAAsB,EAAE,gBA8BpC,CAAC"}

package/dist/reporting/headline-templates/verify.js

@@ -1,12 +1,29 @@
+import { parseVerifyResults } from '../report-parser-slots/verify-report.js';
 import { isNotApplicable } from '../not-applicable.js';
 export const verifyHeadlineTemplate = {
-    compose({ report, status }) {
-        if (!report || isNotApplicable(report))
-            return `[${status}] verify: no structured report available`;
+    compose({ report, status, runResult, task }) {
         const r = report;
-        const items = Array.isArray(r.results) ? r.results : [];
-        const passed = items.filter((x) => x?.pass).length;
-        return `[${status}] verify: ${passed}/${items.length} pass`;
+        const reportInapplicable = !r || isNotApplicable(r);
+        // Source priority (parallel to audit + review):
+        //   1. report.results (structured path; the primary parser now handles
+        //      both JSON and narrative).
+        //   2. parseVerifyResults(runResult.output) (defensive fallback when the
+        //      structured parser ran a non-verify schema and stripped the
+        //      narrative — same Gap-A pattern surfaced on review #2).
+        let results = !reportInapplicable && Array.isArray(r?.results) ? r.results : [];
+        if (results.length === 0 && typeof runResult?.output === 'string') {
+            const narrative = parseVerifyResults(runResult.output);
+            if (narrative.length > 0)
+                results = narrative;
+        }
+        const path = task?.filePaths?.[0] || '';
+        if (results.length === 0 && !path) {
+            return `[${status}] verify completed`;
+        }
+        const passed = results.filter((x) => x?.pass).length;
+        return path
+            ? `[${status}] verify ${path}: ${passed}/${results.length} pass`
+            : `[${status}] verify: ${passed}/${results.length} pass`;
     },
 };
 //# sourceMappingURL=verify.js.map

package/dist/reporting/headline-templates/verify.js.map

@@ -1 +1 @@
-{"version":3,"file":"verify.js","sourceRoot":"","sources":["../../../src/reporting/headline-templates/verify.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,eAAe,EAAE,MAAM,sBAAsB,CAAC;AAEvD,MAAM,CAAC,MAAM,sBAAsB,GAAqB;IACtD,OAAO,CAAC,EAAE,MAAM,EAAE,MAAM,EAAE;QACxB,IAAI,CAAC,MAAM,IAAI,eAAe,CAAC,MAAM,CAAC;YAAE,OAAO,IAAI,MAAM,0CAA0C,CAAC;QACpG,MAAM,CAAC,GAAG,MAAsB,CAAC;QACjC,MAAM,KAAK,GAAG,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,CAAC;QACxD,MAAM,MAAM,GAAG,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,EAAE,IAAI,CAAC,CAAC,MAAM,CAAC;QACnD,OAAO,IAAI,MAAM,aAAa,MAAM,IAAI,KAAK,CAAC,MAAM,OAAO,CAAC;IAC9D,CAAC;CACF,CAAC"}
+{"version":3,"file":"verify.js","sourceRoot":"","sources":["../../../src/reporting/headline-templates/verify.ts"],"names":[],"mappings":"AACA,OAAO,EAAqB,kBAAkB,EAAE,MAAM,yCAAyC,CAAC;AAChG,OAAO,EAAE,eAAe,EAAE,MAAM,sBAAsB,CAAC;AAEvD,MAAM,CAAC,MAAM,sBAAsB,GAAqB;IACtD,OAAO,CAAC,EAAE,MAAM,EAAE,MAAM,EAAE,SAAS,EAAE,IAAI,EAAE;QACzC,MAAM,CAAC,GAAG,MAAkD,CAAC;QAC7D,MAAM,kBAAkB,GAAG,CAAC,CAAC,IAAI,eAAe,CAAC,CAAC,CAAC,CAAC;QAEpD,gDAAgD;QAChD,uEAAuE;QACvE,iCAAiC;QACjC,yEAAyE;QACzE,kEAAkE;QAClE,8DAA8D;QAC9D,IAAI,OAAO,GACT,CAAC,kBAAkB,IAAI,KAAK,CAAC,OAAO,CAAC,CAAC,EAAE,OAAO,CAAC,CAAC,CAAC,CAAC,CAAE,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,CAAC;QACrE,IAAI,OAAO,CAAC,MAAM,KAAK,CAAC,IAAI,OAAO,SAAS,EAAE,MAAM,KAAK,QAAQ,EAAE,CAAC;YAClE,MAAM,SAAS,GAAG,kBAAkB,CAAC,SAAS,CAAC,MAAM,CAAC,CAAC;YACvD,IAAI,SAAS,CAAC,MAAM,GAAG,CAAC;gBAAE,OAAO,GAAG,SAAS,CAAC;QAChD,CAAC;QAED,MAAM,IAAI,GACP,IAA6C,EAAE,SAAS,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;QAEvE,IAAI,OAAO,CAAC,MAAM,KAAK,CAAC,IAAI,CAAC,IAAI,EAAE,CAAC;YAClC,OAAO,IAAI,MAAM,oBAAoB,CAAC;QACxC,CAAC;QAED,MAAM,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,EAAE,IAAI,CAAC,CAAC,MAAM,CAAC;QACrD,OAAO,IAAI;YACT,CAAC,CAAC,IAAI,MAAM,YAAY,IAAI,KAAK,MAAM,IAAI,OAAO,CAAC,MAAM,OAAO;YAChE,CAAC,CAAC,IAAI,MAAM,aAAa,MAAM,IAAI,OAAO,CAAC,MAAM,OAAO,CAAC;IAC7D,CAAC;CACF,CAAC"}

package/dist/reporting/headline-text.d.ts

@@ -0,0 +1,36 @@
+/**
+ * Helpers for headline text shaping.
+ *
+ * Headlines are operator-facing single-line strings that appear in
+ * polling output and the terminal envelope. They MUST stay short and
+ * structured — workers' free-form `summary` fields can be paragraphs
+ * long and end mid-sentence (the v4.0.3 Gap 12 case: "...now ends
+ * with:" cut off because the worker meant to put a code excerpt
+ * after the colon).
+ *
+ * `firstSentenceOrTruncate` is the single helper composers use to
+ * sanitize narrative text for headline display.
+ */
+/**
+ * Return the first sentence of `s`, or a hard-truncated form with
+ * trailing ellipsis when no sentence boundary exists in the first
+ * `max` characters.
+ *
+ * Why first-sentence? Worker summaries follow a "headline sentence,
+ * then details" pattern; the first sentence carries the operator-
+ * relevant signal. Truncating mid-sentence (the prior bug) loses
+ * that signal AND looks broken.
+ *
+ * Sentence-end heuristic: a `.!?` followed by whitespace OR end-of-
+ * string. The match is non-greedy and ranges over `max` characters,
+ * so internal punctuation (version numbers like `v4.0.3`, decimals
+ * like `1.5`, filenames like `auth.ts`) does NOT block the scan.
+ *
+ * @param s     raw text (may be empty / whitespace-only)
+ * @param max   hard truncation cap when no sentence break is found.
+ *              80 keeps headlines well under the typical 120-char
+ *              terminal-line target after the bracket prefix +
+ *              file-count suffix are added.
+ */
+export declare function firstSentenceOrTruncate(s: string, max?: number): string;
+//# sourceMappingURL=headline-text.d.ts.map

package/dist/reporting/headline-text.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"headline-text.d.ts","sourceRoot":"","sources":["../../src/reporting/headline-text.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAEH;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,wBAAgB,uBAAuB,CAAC,CAAC,EAAE,MAAM,EAAE,GAAG,SAAK,GAAG,MAAM,CAkCnE"}

package/dist/reporting/headline-text.js

@@ -0,0 +1,73 @@
+/**
+ * Helpers for headline text shaping.
+ *
+ * Headlines are operator-facing single-line strings that appear in
+ * polling output and the terminal envelope. They MUST stay short and
+ * structured — workers' free-form `summary` fields can be paragraphs
+ * long and end mid-sentence (the v4.0.3 Gap 12 case: "...now ends
+ * with:" cut off because the worker meant to put a code excerpt
+ * after the colon).
+ *
+ * `firstSentenceOrTruncate` is the single helper composers use to
+ * sanitize narrative text for headline display.
+ */
+/**
+ * Return the first sentence of `s`, or a hard-truncated form with
+ * trailing ellipsis when no sentence boundary exists in the first
+ * `max` characters.
+ *
+ * Why first-sentence? Worker summaries follow a "headline sentence,
+ * then details" pattern; the first sentence carries the operator-
+ * relevant signal. Truncating mid-sentence (the prior bug) loses
+ * that signal AND looks broken.
+ *
+ * Sentence-end heuristic: a `.!?` followed by whitespace OR end-of-
+ * string. The match is non-greedy and ranges over `max` characters,
+ * so internal punctuation (version numbers like `v4.0.3`, decimals
+ * like `1.5`, filenames like `auth.ts`) does NOT block the scan.
+ *
+ * @param s     raw text (may be empty / whitespace-only)
+ * @param max   hard truncation cap when no sentence break is found.
+ *              80 keeps headlines well under the typical 120-char
+ *              terminal-line target after the bracket prefix +
+ *              file-count suffix are added.
+ */
+export function firstSentenceOrTruncate(s, max = 80) {
+    if (!s || typeof s !== 'string')
+        return '';
+    const trimmed = s.trim();
+    if (trimmed.length === 0)
+        return '';
+    // F2 fix (audit, low): defend against invalid `max` values. Headlines
+    // are single-line operator-facing strings; a buggy caller passing
+    // `max=0`, negative, `NaN`, or `Infinity` shouldn't make the regex
+    // throw or overrun. Coerce to a sane bound: integer in [1, 2000].
+    const safeMax = Number.isFinite(max) && max >= 1
+        ? Math.min(Math.floor(max), 2000)
+        : 80;
+    // N2 fix (audit-2, low): collapse whitespace BEFORE sentence detection.
+    // Doing it after meant a sentence wrapping across a newline before
+    // its terminator (e.g., "Fixed auth\nissue. More") never matched the
+    // boundary regex and fell through to a generic truncate.
+    const oneLine = collapseNewlines(trimmed);
+    // N1 fix (audit-2, low): cap the captured sentence at exactly `safeMax`
+    // chars including the terminating punctuation. The leading run is
+    // therefore at most `safeMax - 1` chars. Skip sentence detection
+    // entirely when there's no room for even "X." (safeMax < 2).
+    // Lazy quantifier `{1,N}?` lets internal `.!?` characters pass
+    // through (version numbers, decimals, filenames) until we hit one
+    // followed by whitespace or end-of-string — the real sentence boundary.
+    if (safeMax >= 2) {
+        const sentenceEnd = new RegExp(`^(.{1,${safeMax - 1}}?[.!?])(\\s|$)`);
+        const m = oneLine.match(sentenceEnd);
+        if (m)
+            return m[1];
+    }
+    return oneLine.length > safeMax ? oneLine.slice(0, safeMax - 1) + '…' : oneLine;
+}
+/** Replace any whitespace run (including newlines, tabs, and CRs) with
+ *  a single space so the returned headline stays on one line. */
+function collapseNewlines(s) {
+    return s.replace(/\s+/g, ' ').trim();
+}
+//# sourceMappingURL=headline-text.js.map

package/dist/reporting/headline-text.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"headline-text.js","sourceRoot":"","sources":["../../src/reporting/headline-text.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAEH;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,MAAM,UAAU,uBAAuB,CAAC,CAAS,EAAE,GAAG,GAAG,EAAE;IACzD,IAAI,CAAC,CAAC,IAAI,OAAO,CAAC,KAAK,QAAQ;QAAE,OAAO,EAAE,CAAC;IAC3C,MAAM,OAAO,GAAG,CAAC,CAAC,IAAI,EAAE,CAAC;IACzB,IAAI,OAAO,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,EAAE,CAAC;IAEpC,sEAAsE;IACtE,kEAAkE;IAClE,mEAAmE;IACnE,kEAAkE;IAClE,MAAM,OAAO,GACX,MAAM,CAAC,QAAQ,CAAC,GAAG,CAAC,IAAI,GAAG,IAAI,CAAC;QAC9B,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,EAAE,IAAI,CAAC;QACjC,CAAC,CAAC,EAAE,CAAC;IAET,wEAAwE;IACxE,mEAAmE;IACnE,qEAAqE;IACrE,yDAAyD;IACzD,MAAM,OAAO,GAAG,gBAAgB,CAAC,OAAO,CAAC,CAAC;IAE1C,wEAAwE;IACxE,kEAAkE;IAClE,iEAAiE;IACjE,6DAA6D;IAC7D,+DAA+D;IAC/D,kEAAkE;IAClE,wEAAwE;IACxE,IAAI,OAAO,IAAI,CAAC,EAAE,CAAC;QACjB,MAAM,WAAW,GAAG,IAAI,MAAM,CAAC,SAAS,OAAO,GAAG,CAAC,iBAAiB,CAAC,CAAC;QACtE,MAAM,CAAC,GAAG,OAAO,CAAC,KAAK,CAAC,WAAW,CAAC,CAAC;QACrC,IAAI,CAAC;YAAE,OAAO,CAAC,CAAC,CAAC,CAAC,CAAC;IACrB,CAAC;IAED,OAAO,OAAO,CAAC,MAAM,GAAG,OAAO,CAAC,CAAC,CAAC,OAAO,CAAC,KAAK,CAAC,CAAC,EAAE,OAAO,GAAG,CAAC,CAAC,GAAG,GAAG,CAAC,CAAC,CAAC,OAAO,CAAC;AAClF,CAAC;AAED;iEACiE;AACjE,SAAS,gBAAgB,CAAC,CAAS;IACjC,OAAO,CAAC,CAAC,OAAO,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC,IAAI,EAAE,CAAC;AACvC,CAAC"}

package/dist/reporting/report-parser-slots/verify-report.d.ts

@@ -6,5 +6,22 @@ export interface VerifyReport {
         evidence: string;
     }>;
 }
+/**
+ * Extract verify checklist results from `## Finding N:` narrative blocks.
+ *
+ * The verify tool prompt instructs workers to emit results in this exact
+ * format (and explicitly NOT JSON):
+ *
+ *     ## Finding 1: <title>
+ *     - Severity: low|high|...
+ *     - Item: <criterion text>
+ *     - Result: PASS | FAIL
+ *     - Evidence: <evidence>
+ *
+ * Returns one entry per finding block. `pass` is true iff the Result line
+ * normalizes to "pass" (case-insensitive). Missing labels become empty
+ * strings / false (matches the schema's required-field shape).
+ */
+export declare function parseVerifyResults(output: string): VerifyReport['results'];
 export declare const verifyReportSchema: ReportSchema<VerifyReport>;
 //# sourceMappingURL=verify-report.d.ts.map

package/dist/reporting/report-parser-slots/verify-report.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"verify-report.d.ts","sourceRoot":"","sources":["../../../src/reporting/report-parser-slots/verify-report.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,gCAAgC,CAAC;AAEnE,MAAM,WAAW,YAAY;IAC3B,OAAO,EAAE,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,OAAO,CAAC;QAAC,QAAQ,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;CACnE;AAED,eAAO,MAAM,kBAAkB,EAAE,YAAY,CAAC,YAAY,CAMzD,CAAC"}
+{"version":3,"file":"verify-report.d.ts","sourceRoot":"","sources":["../../../src/reporting/report-parser-slots/verify-report.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,gCAAgC,CAAC;AAEnE,MAAM,WAAW,YAAY;IAC3B,OAAO,EAAE,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,OAAO,CAAC;QAAC,QAAQ,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;CACnE;AAED;;;;;;;;;;;;;;;GAeG;AACH,wBAAgB,kBAAkB,CAChC,MAAM,EAAE,MAAM,GACb,YAAY,CAAC,SAAS,CAAC,CAezB;AAED,eAAO,MAAM,kBAAkB,EAAE,YAAY,CAAC,YAAY,CAezD,CAAC"}

package/dist/reporting/report-parser-slots/verify-report.js

@@ -1,9 +1,51 @@
+/**
+ * Extract verify checklist results from `## Finding N:` narrative blocks.
+ *
+ * The verify tool prompt instructs workers to emit results in this exact
+ * format (and explicitly NOT JSON):
+ *
+ *     ## Finding 1: <title>
+ *     - Severity: low|high|...
+ *     - Item: <criterion text>
+ *     - Result: PASS | FAIL
+ *     - Evidence: <evidence>
+ *
+ * Returns one entry per finding block. `pass` is true iff the Result line
+ * normalizes to "pass" (case-insensitive). Missing labels become empty
+ * strings / false (matches the schema's required-field shape).
+ */
+export function parseVerifyResults(output) {
+    if (!output || typeof output !== 'string')
+        return [];
+    const blocks = output.split(/^##\s+Finding\s+\d+\s*:?/im);
+    const out = [];
+    for (let i = 1; i < blocks.length; i++) {
+        const block = blocks[i];
+        const itemMatch = block.match(/^\s*[-*]?\s*Item\s*:\s*(.+?)\s*$/im);
+        const resultMatch = block.match(/^\s*[-*]?\s*Result\s*:\s*(\w+)/im);
+        const evidenceMatch = block.match(/^\s*[-*]?\s*Evidence\s*:\s*([\s\S]+?)(?=\n\s*[-*]\s|\n##|$)/im);
+        const item = itemMatch ? itemMatch[1].trim() : '';
+        const pass = resultMatch ? /^pass$/i.test(resultMatch[1].trim()) : false;
+        const evidence = evidenceMatch ? evidenceMatch[1].trim() : '';
+        out.push({ item, pass, evidence });
+    }
+    return out;
+}
 export const verifyReportSchema = {
     parse(text) {
+        // Primary path: JSON block (legacy; some workers may still emit it).
         const m = text.match(/```json\n([\s\S]+?)\n```/);
-        if (!m)
-            throw new Error('verify report missing JSON block');
-        return JSON.parse(m[1]);
+        if (m)
+            return JSON.parse(m[1]);
+        // Narrative path (the actual prompt format — the verify prompt
+        // explicitly says "Do NOT emit JSON"). Recover one result per
+        // `## Finding N:` block. If neither path produced anything,
+        // throw so the parent falls back to notApplicable.
+        const narrative = parseVerifyResults(text);
+        if (narrative.length === 0) {
+            throw new Error('verify report missing JSON block and no `## Finding N:` narrative results');
+        }
+        return { results: narrative };
     },
 };
 //# sourceMappingURL=verify-report.js.map

package/dist/reporting/report-parser-slots/verify-report.js.map

@@ -1 +1 @@
-{"version":3,"file":"verify-report.js","sourceRoot":"","sources":["../../../src/reporting/report-parser-slots/verify-report.ts"],"names":[],"mappings":"AAMA,MAAM,CAAC,MAAM,kBAAkB,GAA+B;IAC5D,KAAK,CAAC,IAAY;QAChB,MAAM,CAAC,GAAG,IAAI,CAAC,KAAK,CAAC,0BAA0B,CAAC,CAAC;QACjD,IAAI,CAAC,CAAC;YAAE,MAAM,IAAI,KAAK,CAAC,kCAAkC,CAAC,CAAC;QAC5D,OAAO,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;IAC1B,CAAC;CACF,CAAC"}
+{"version":3,"file":"verify-report.js","sourceRoot":"","sources":["../../../src/reporting/report-parser-slots/verify-report.ts"],"names":[],"mappings":"AAMA;;;;;;;;;;;;;;;GAeG;AACH,MAAM,UAAU,kBAAkB,CAChC,MAAc;IAEd,IAAI,CAAC,MAAM,IAAI,OAAO,MAAM,KAAK,QAAQ;QAAE,OAAO,EAAE,CAAC;IACrD,MAAM,MAAM,GAAG,MAAM,CAAC,KAAK,CAAC,4BAA4B,CAAC,CAAC;IAC1D,MAAM,GAAG,GAA4B,EAAE,CAAC;IACxC,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,MAAM,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACvC,MAAM,KAAK,GAAG,MAAM,CAAC,CAAC,CAAC,CAAC;QACxB,MAAM,SAAS,GAAG,KAAK,CAAC,KAAK,CAAC,oCAAoC,CAAC,CAAC;QACpE,MAAM,WAAW,GAAG,KAAK,CAAC,KAAK,CAAC,kCAAkC,CAAC,CAAC;QACpE,MAAM,aAAa,GAAG,KAAK,CAAC,KAAK,CAAC,+DAA+D,CAAC,CAAC;QACnG,MAAM,IAAI,GAAG,SAAS,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QAClD,MAAM,IAAI,GAAG,WAAW,CAAC,CAAC,CAAC,SAAS,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC;QACzE,MAAM,QAAQ,GAAG,aAAa,CAAC,CAAC,CAAC,aAAa,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QAC9D,GAAG,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,QAAQ,EAAE,CAAC,CAAC;IACrC,CAAC;IACD,OAAO,GAAG,CAAC;AACb,CAAC;AAED,MAAM,CAAC,MAAM,kBAAkB,GAA+B;IAC5D,KAAK,CAAC,IAAY;QAChB,qEAAqE;QACrE,MAAM,CAAC,GAAG,IAAI,CAAC,KAAK,CAAC,0BAA0B,CAAC,CAAC;QACjD,IAAI,CAAC;YAAE,OAAO,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;QAC/B,+DAA+D;QAC/D,8DAA8D;QAC9D,4DAA4D;QAC5D,mDAAmD;QACnD,MAAM,SAAS,GAAG,kBAAkB,CAAC,IAAI,CAAC,CAAC;QAC3C,IAAI,SAAS,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YAC3B,MAAM,IAAI,KAAK,CAAC,2EAA2E,CAAC,CAAC;QAC/F,CAAC;QACD,OAAO,EAAE,OAAO,EAAE,SAAS,EAAE,CAAC;IAChC,CAAC;CACF,CAAC"}

package/dist/reporting/severity.d.ts

@@ -0,0 +1,62 @@
+/**
+ * Severity helpers shared across headline composers and telemetry
+ * event-builder. Per the wire-telemetry-gaps plan (Gap 2 + round-2 F1):
+ * THREE distinct helpers, NOT one — conflating headline counting with
+ * telemetry bucketing would corrupt the wire's findings_critical /
+ * findings_high columns.
+ */
+export type FindingSeverity = 'critical' | 'high' | 'medium' | 'low';
+/**
+ * Normalize a raw severity string to the canonical lowercase enum.
+ * Returns null for unknown / unparseable values.
+ *
+ * Used by both headline counting and telemetry bucketing — single
+ * source of truth for "what's the severity of this finding".
+ */
+export declare function normalizeSeverity(raw: unknown): FindingSeverity | null;
+/**
+ * Headline-only helper. Counts findings whose normalized severity is
+ * 'high' OR 'critical'. Used by audit + review headline composers to
+ * surface the "(N high)" annotation.
+ *
+ * MUST NOT be used for telemetry bucketing — see
+ * `bucketFindingsBySeverity` for that.
+ */
+export declare function countHighOrCritical(findings: ReadonlyArray<{
+    severity?: unknown;
+}>): number;
+/**
+ * Telemetry-only helper. Returns separate per-severity buckets:
+ * `{ critical, high, medium, low }`. Used by event-builder so the
+ * wire's findings_critical / findings_high / findings_medium /
+ * findings_low DB columns each carry exact counts.
+ *
+ * MUST NOT be replaced by countHighOrCritical — that conflates
+ * critical and high into one count.
+ */
+export declare function bucketFindingsBySeverity(findings: ReadonlyArray<{
+    severity?: unknown;
+}>): Record<FindingSeverity, number>;
+/**
+ * Extract findings from an implementer's narrative `## Finding N:` output
+ * block-by-block. The audit / review / verify / debug / investigate
+ * tool prompts instruct workers to emit findings in this exact format:
+ *
+ *     ## Finding 1: <title>
+ *     - Severity: critical | high | medium | low
+ *     - Location: file:line
+ *     - Issue: ...
+ *     - Suggestion: ...
+ *
+ * The annotator normally consumes that narrative and emits a structured
+ * JSON array. When the annotator errors (parse failure, exhaustion
+ * mid-block, etc.), this helper recovers the findings from the
+ * implementer's output so the headline + envelope still report the
+ * correct count. Returns one entry per finding with the parsed severity
+ * (lowercase canonical, or null if no severity line was found).
+ */
+export declare function parseNarrativeFindings(output: string): Array<{
+    severity: FindingSeverity | null;
+    claim: string;
+}>;
+//# sourceMappingURL=severity.d.ts.map

package/dist/reporting/severity.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"severity.d.ts","sourceRoot":"","sources":["../../src/reporting/severity.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,MAAM,MAAM,eAAe,GAAG,UAAU,GAAG,MAAM,GAAG,QAAQ,GAAG,KAAK,CAAC;AAIrE;;;;;;GAMG;AACH,wBAAgB,iBAAiB,CAAC,GAAG,EAAE,OAAO,GAAG,eAAe,GAAG,IAAI,CAItE;AAED;;;;;;;GAOG;AACH,wBAAgB,mBAAmB,CAAC,QAAQ,EAAE,aAAa,CAAC;IAAE,QAAQ,CAAC,EAAE,OAAO,CAAA;CAAE,CAAC,GAAG,MAAM,CAO3F;AAED;;;;;;;;GAQG;AACH,wBAAgB,wBAAwB,CACtC,QAAQ,EAAE,aAAa,CAAC;IAAE,QAAQ,CAAC,EAAE,OAAO,CAAA;CAAE,CAAC,GAC9C,MAAM,CAAC,eAAe,EAAE,MAAM,CAAC,CAOjC;AAED;;;;;;;;;;;;;;;;;GAiBG;AACH,wBAAgB,sBAAsB,CACpC,MAAM,EAAE,MAAM,GACb,KAAK,CAAC;IAAE,QAAQ,EAAE,eAAe,GAAG,IAAI,CAAC;IAAC,KAAK,EAAE,MAAM,CAAA;CAAE,CAAC,CAgB5D"}