pi-pr-review 1.14.1 → 1.14.2

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [1.14.2](https://github.com/10ego/pi-pr-review/compare/v1.14.1...v1.14.2) (2026-08-23)
+
+
+### Bug Fixes
+
+* **extract:** scan lane evidence past the synthesis findings framing ([#77](https://github.com/10ego/pi-pr-review/issues/77)) ([dfebc36](https://github.com/10ego/pi-pr-review/commit/dfebc3632910fdde46581d4a48d236d72c7046df))
+
 ## [1.14.1](https://github.com/10ego/pi-pr-review/compare/v1.14.0...v1.14.1) (2026-08-22)
 
 

package/extensions/pr-review-subagent.ts

@@ -56,7 +56,7 @@ import {
 } from "../lib/pr-review-artifacts.ts";
 import { runWithConcurrency } from "../lib/pr-review-concurrency.ts";
 import { attemptDeadline, fallbackBudget, type ReviewBudget } from "../lib/pr-review-deadlines.ts";
-import { buildExtractionSystemPrompt, MAX_EXTRACTION_OUTPUT_BYTES } from "../lib/pr-review-extract.ts";
+import { buildExtractionSystemPrompt, buildExtractionTask, MAX_EXTRACTION_OUTPUT_BYTES } from "../lib/pr-review-extract.ts";
 import { loadReviewContext, shardUnifiedDiff } from "../lib/pr-review-context.ts";
 import {
 	combineAbortSignals,
@@ -2727,7 +2727,7 @@ export async function runFindingExtraction(
 			invocation.command,
 			invocation.args,
 			ctx.cwd,
-			input,
+			buildExtractionTask(input),
 			lease.signal,
 			undefined,
 			undefined,

package/lib/pr-review-extract.ts

@@ -89,6 +89,8 @@ export function buildExtractionSystemPrompt(): string {
 		"When the document names a file for the finding, also include: path (repo-relative file path exactly as written), start_line and end_line (integers from the document when stated), side (\"RIGHT\" for added lines, \"LEFT\" for removed lines), and location_quote (a verbatim substring of the document containing that path).",
 		"Optionally include source (the lane or section the finding came from).",
 		"Rules: report only findings actually stated in the document; never invent, merge, or upgrade severity; keep the reviewer's own wording in title and body; omit path fields when no location is stated; do not report strengths, summaries, or coverage notes as findings.",
+		"The synthesis's own Findings section is NOT authoritative about what the document states: a review summary saying \"No findings\" while a retained lane section states a concrete defect means that defect IS stated in the document and must be extracted.",
+		"Findings appear in any format: ### severity blocks, bulleted field lists (title/severity/why/location), or prose. Scan every section, including retained lane output, before deciding the document states no findings.",
 		"If the document states no findings, return {\"findings\":[]}.",
 		"Do not use tools, read files, or modify anything.",
 	].join("\n");
@@ -112,6 +114,23 @@ function truncateWithMarker(text: string, maxBytes: number, omittedBytes?: numbe
 }
 
 /** Assemble the bounded extraction payload: synthesis first, then lanes in order, each truncated with its own byte-count marker. */
+/**
+ * Host-authored framing wrapped around the data document. Without it the
+ * child treats the synthesis's own "Findings: No findings." framing as
+ * authoritative and misses defects stated in retained lane output (observed
+ * in the first production extraction run).
+ */
+export function buildExtractionTask(input: string): string {
+	return [
+		"Objective: Extract every concrete defect finding stated anywhere in this review document, including inside the retained lane output sections.",
+		"A \"No findings\" line written by the summary does not mean the document states no findings: check the lane sections too.",
+		"",
+		"--- Review document ---",
+		input,
+		"--- End of review document ---",
+	].join("\n");
+}
+
 export function buildExtractionInput(
 	rawText: string,
 	lanes: readonly ReviewLaneArtifact[],

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-pr-review",
-  "version": "1.14.1",
+  "version": "1.14.2",
   "description": "Parallel AI code review for GitHub pull requests in the Pi coding agent, with model-agnostic tiered subagents, structured findings, optional verification, and host-gated COMMENT or qualified APPROVE publishing.",
   "keywords": [
     "pi-package",