prreviewbuddy 0.55.1 → 0.55.6

package/CHANGELOG.md

@@ -1,5 +1,79 @@
 # Changelog
 
+## 0.55.6
+
+**File paths are shown whole.** A deep directory above a file name was cut short with an ellipsis,
+and the full path appeared only in a browser tooltip on hover. The directory now wraps onto a
+second line where it needs one, so the whole path is always on screen, and the tooltips are gone.
+This covers file locations on issues and questions, related files, the Guide's file rows, the Files
+tab and KISS locations, in the workspace and the extension. The extension's file list, which had no
+tooltip, no longer hides the end of a long directory either. (Extension 0.25.4.)
+
+## 0.55.5
+
+**Why this change? says what is known, plainly.** Each state now has a label anyone can read and a
+colour that says how sure it is: **Author's reason** (green), the author's own words in the PR
+description; **Possible reason** (gold), "The PR description doesn't give a clear reason. Based on
+the change itself, this appears to…"; and **Reason unclear** (grey), "It isn't clear why this
+change is needed.", with **Ask the author** and the question to ask. Colour is only a thin rule
+beside the reason and the label's own colour; the text stays dark, and the label says it all
+without colour. Once that question is marked answered the section reads **Answer available**: "This
+question has been marked answered. Run the review again after adding the answer to the PR
+description."
+
+**A refused reason is never shown as nothing.** When the analysis says the author gave a reason but
+the words it quotes can't be confirmed in the PR description, the section used to show nothing at
+all. It now shows Reason unclear, asking the analysis's own question when it gave one, or "What
+problem does this change solve, or what goal does it serve?". That question has the same id on
+every analysis, so marking it answered survives a rerun, even when the analysis happens to ask it
+in the same words. The refused wording is still never shown, and the evidence the refused reason
+came with is not presented as a reason.
+
+**What the harness corrected moves into Analysis details.** The amber box above the review, which
+showed on every tab, is gone; its notes are now listed under Analysis details on the Overview. The
+note for a refused reason says only what the check found, since a quote also fails when it is two
+words or is the template's text. The refused quote (cut at 500 characters) and its evidence are
+kept in the review's audit record, or the extension's dev trace.
+
+The prompt asks for the author's words exactly as written, backticks and punctuation included, and
+treats a reason found only in commits, review comments or a linked issue as inferred. Under
+Possible reason, the kind of each piece of evidence ("Test", "Code") is now the size of the text
+beside it. (Extension 0.25.3.)
+
+## 0.55.4
+
+**Empty pages use the full width.** On the PR context page, the note saying no pull request
+discussion was brought in, and on Diagrams, the card offering to draw a change map or an
+implementation diagram, were held to a narrow column. They now fill the page's width like the
+rest of the workspace.
+
+## 0.55.3
+
+**Template comments stay hidden.** HTML comments in a PR description, such as a template's
+`<!-- Describe your change -->`, showed as literal text in Author's context, in both the full view
+and the preview. They are now hidden, as on GitHub. The same goes for comments in review comments
+on the PR context page. A comment inside a code block or written as inline code is still shown,
+since there it is code, and a code fence inside a comment no longer turns the rest of the
+description into code. The preview never shows part of a comment. Model output is unchanged:
+nothing it writes is hidden.
+
+## 0.55.2
+
+**A lighter Overview.** Author's context, Why this change? and What's changing no longer read as
+one block:
+
+- **Author's context** opens on a short preview of the first thing the author wrote, past any
+  template heading such as "Summary": a paragraph's first two sentences (or about 400 characters,
+  never cut mid-word), a list's first three items, or "Code block included" for a long code block.
+  **View full PR description** shows the whole description, as written. A description that fits
+  the preview is shown whole, with no button.
+- **Why this change?** shows the label and the motive. The author's quote behind a stated motive
+  is under **Author's words**, one click away, since the motive usually says the same thing.
+- **What's changing** is asked for in about 80 to 120 words, and says what the change does without
+  repeating why it exists or judging it. Analyses made before this version keep their summary.
+- More space between the three sections, and more again before the risk verdict and the
+  suggested review path.
+
 ## 0.55.1
 
 **Load more on the Notifications page.** The list shows the newest fifteen, and Load more reveals

package/dist/{delete_review-B2bqhgtx.js → delete_review-DEvRanb6.js}

@@ -9395,13 +9395,252 @@ function normalizeSections(value) {
 	return sections;
 }
 //#endregion
+//#region ../../packages/review-harness/src/domain/utils/html.ts
+function escapeHtml(value) {
+	return value.replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;").replace(/"/g, "&quot;").replace(/'/g, "&#39;");
+}
+function renderInlineText(value) {
+	return escapeHtml(value).replace(/`([^`\n]+)`/g, "<code>$1</code>");
+}
+//#endregion
+//#region ../../packages/review-harness/src/domain/utils/code_block.ts
+var CODE_BLOCK_CLASS = "pr-helper-code-block";
+var CODE_COPY_CLASS = "pr-helper-code-copy";
+//#endregion
+//#region ../../packages/review-harness/src/domain/utils/markdown.ts
+var BLOCK_MARKER = "\0";
+var INLINE_MARKER = "
";
+var BLOCK_TOKEN = /\u0000(\d+)\u0000/g;
+var BLOCK_TOKEN_LINE = /^\u0000\d+\u0000$/;
+var INLINE_TOKEN = /\u0001(\d+)\u0001/g;
+var SAFE_HREF = /^(?:https?:\/\/|#)/i;
+function renderMarkdown(value, options = {}) {
+	const { text, blocks } = extractFencedBlocks(scanMarkdownLines(value.replace(/[\u0000\u0001]/g, ""), options));
+	return restoreFencedBlocks(renderBlocks(text), blocks);
+}
+/**
+* The fence character a line opens a code block with, or null when it opens none. Shared with the
+* Author's context preview, which must find the same code blocks this renderer does.
+*/
+function fenceOpening(line) {
+	return line.match(/^\s*(`{3,}|~{3,})\s*[\w+#.-]*\s*$/)?.[1][0] ?? null;
+}
+/** Whether a line closes a code block opened with `fence`: only a fence of the same character does. */
+function closesFence(line, fence) {
+	return new RegExp(`^\\s*${fence}{3,}\\s*$`).test(line);
+}
+var COMMENT_START_OR_CODE_SPAN = /`[^`\n]+`|<!--/g;
+function commentStart(line, from) {
+	COMMENT_START_OR_CODE_SPAN.lastIndex = from;
+	for (let match; match = COMMENT_START_OR_CODE_SPAN.exec(line);) if (match[0] === "<!--") return match.index;
+	return -1;
+}
+/**
+* Each line's part in a code block and, when comments are hidden, what of it is outside one.
+*
+* Code blocks and comments are read in one pass, in the order they open, as GitHub reads them: a
+* comment inside a code block is code, and a fence inside a comment is not a fence. The renderer and
+* the Author's context preview both read Markdown through this, so they agree on what is code and
+* what is hidden. A comment never closed is not a comment: its text stays, rather than hiding the
+* rest of what was said.
+*/
+function scanMarkdownLines(value, options = {}) {
+	const result = [];
+	let fence = null;
+	let comment = false;
+	let opener = null;
+	let offset = 0;
+	for (const raw of value.split("\n")) {
+		const lineStart = offset;
+		offset += raw.length + 1;
+		if (fence) {
+			const close = closesFence(raw, fence);
+			if (close) fence = null;
+			result.push({
+				raw,
+				visible: raw,
+				code: close ? "close" : "body"
+			});
+			continue;
+		}
+		if (!comment) {
+			fence = fenceOpening(raw);
+			if (fence) {
+				result.push({
+					raw,
+					visible: raw,
+					code: "open"
+				});
+				continue;
+			}
+		}
+		if (!options.hideHtmlComments) {
+			result.push({
+				raw,
+				visible: raw
+			});
+			continue;
+		}
+		const joining = comment ? opener : null;
+		let visible = "";
+		let pos = 0;
+		while (pos < raw.length) {
+			if (comment) {
+				const end = raw.indexOf("-->", pos);
+				if (end === -1) break;
+				pos = end + 3;
+				comment = false;
+				continue;
+			}
+			const start = commentStart(raw, pos);
+			if (start === -1 || value.indexOf("-->", lineStart + start + 4) === -1) {
+				visible += raw.slice(pos);
+				break;
+			}
+			visible += raw.slice(pos, start);
+			pos = start + 4;
+			comment = true;
+		}
+		if (joining) {
+			joining.visible += visible;
+			result.push({
+				raw,
+				visible: "",
+				absent: true
+			});
+			continue;
+		}
+		const line = {
+			raw,
+			visible
+		};
+		if (comment) opener = visible.trim() === "" ? null : line;
+		result.push(line);
+	}
+	return result;
+}
+function extractFencedBlocks(lines) {
+	const blocks = [];
+	const out = [];
+	let body = null;
+	for (const line of lines) {
+		if (line.absent) continue;
+		if (line.code === "open") body = [];
+		else if (line.code === "body") body.push(line.raw);
+		else if (line.code === "close") {
+			out.push(`${BLOCK_MARKER}${blocks.length}${BLOCK_MARKER}`);
+			blocks.push(body.join("\n"));
+			body = null;
+		} else out.push(line.visible);
+	}
+	if (body) {
+		out.push(`${BLOCK_MARKER}${blocks.length}${BLOCK_MARKER}`);
+		blocks.push(body.join("\n"));
+	}
+	return {
+		text: out.join("\n"),
+		blocks
+	};
+}
+function restoreFencedBlocks(html, blocks) {
+	return html.replace(BLOCK_TOKEN, (_match, index) => `<div class="${CODE_BLOCK_CLASS}"><pre><code>${escapeHtml(blocks[Number(index)])}</code></pre><button type="button" class="${CODE_COPY_CLASS}" aria-label="Copy code">Copy</button></div>`);
+}
+function renderBlocks(text) {
+	const out = [];
+	let paragraph = [];
+	let quote = [];
+	let list = null;
+	const flush = () => {
+		if (paragraph.length) out.push(`<p>${renderInline(paragraph.join(" "))}</p>`);
+		if (quote.length) out.push(`<blockquote>${renderInline(quote.join(" "))}</blockquote>`);
+		if (list) out.push(`<${list.tag}>${list.items.map((item) => `<li>${renderInline(item)}</li>`).join("")}</${list.tag}>`);
+		paragraph = [];
+		quote = [];
+		list = null;
+	};
+	for (const raw of text.split("\n")) {
+		const line = raw.trim();
+		if (!line) {
+			flush();
+			continue;
+		}
+		if (BLOCK_TOKEN_LINE.test(line)) {
+			flush();
+			out.push(line);
+			continue;
+		}
+		if (/^(?:-{3,}|\*{3,}|_{3,})$/.test(line)) {
+			flush();
+			out.push("<hr>");
+			continue;
+		}
+		const heading = line.match(/^(#{1,6})\s+(.+)$/);
+		if (heading) {
+			flush();
+			const level = Math.min(heading[1].length, 3) + 3;
+			out.push(`<h${level}>${renderInline(heading[2])}</h${level}>`);
+			continue;
+		}
+		const bullet = line.match(/^[-*+]\s+(.+)$/);
+		if (bullet) {
+			if (list?.tag !== "ul") {
+				flush();
+				list = {
+					tag: "ul",
+					items: []
+				};
+			}
+			list.items.push(bullet[1]);
+			continue;
+		}
+		const ordered = line.match(/^\d{1,9}[.)]\s+(.+)$/);
+		if (ordered) {
+			if (list?.tag !== "ol") {
+				flush();
+				list = {
+					tag: "ol",
+					items: []
+				};
+			}
+			list.items.push(ordered[1]);
+			continue;
+		}
+		const quoted = line.match(/^>\s?(.*)$/);
+		if (quoted) {
+			if (!quote.length) flush();
+			quote.push(quoted[1]);
+			continue;
+		}
+		if (list && list.items.length) {
+			list.items[list.items.length - 1] += ` ${line}`;
+			continue;
+		}
+		if (quote.length) {
+			quote[quote.length - 1] += ` ${line}`;
+			continue;
+		}
+		paragraph.push(line);
+	}
+	flush();
+	return out.join("");
+}
+function renderInline(value) {
+	const codeSpans = [];
+	return escapeHtml(value).replace(/`([^`\n]+)`/g, (_match, code) => {
+		codeSpans.push(code);
+		return `${INLINE_MARKER}${codeSpans.length - 1}${INLINE_MARKER}`;
+	}).replace(/(?<!!)\[([^\]\n]+)\]\(([^)\s]+)\)/g, (match, label, href) => SAFE_HREF.test(href) ? `<a href="${href}" target="_blank" rel="noopener noreferrer">${label}</a>` : match).replace(/\*\*([^*\n]+)\*\*/g, "<strong>$1</strong>").replace(/__([^_\n]+)__/g, "<strong>$1</strong>").replace(/(^|[^*\w])\*([^*\n]+)\*/g, "$1<em>$2</em>").replace(/(^|[^_\w])_([^_\n]+)_(?![\w_])/g, "$1<em>$2</em>").replace(INLINE_TOKEN, (_match, index) => `<code>${codeSpans[Number(index)]}</code>`);
+}
+//#endregion
 //#region ../../packages/review-harness/src/domain/ai_review/author_description.ts
 /**
 * Reading a pull request's description for whether it says anything, without ever changing it.
 *
-* Nothing here returns a copy to store or show. The description is shown verbatim, so a cleaned
-* copy is only ever a means of asking a question of it: whether it says anything, and whether a
-* quote is something the author said.
+* Nothing here returns a copy to store, or a reworded one to show. The description is shown
+* verbatim, so a cleaned copy is only ever a means of asking a question of it: whether it says
+* anything, whether a quote is something the author said, and where its preview ends. The preview
+* itself is the author's own lines, cut short, never edited, less the HTML comments the full view
+* hides too.
 */
 /**
 * Lines that are a pull request template's scaffolding when they stand alone.
@@ -9435,7 +9674,6 @@ var TEMPLATE_LABELS = /* @__PURE__ */ new Set([
 	"related issue",
 	"links"
 ]);
-var HTML_COMMENT = /<!--[\s\S]*?-->/g;
 var HEADING_MARKER = /^#{1,6}\s*/;
 var UNCHECKED_TASK = /^[-*+]\s+\[ \]/;
 var EMPTY_LIST_ITEM = /^(?:[-*+]|\d+[.)])\s*$/;
@@ -9457,7 +9695,7 @@ function substance(line) {
 */
 function authorText(description) {
 	if (!description) return "";
-	return description.replace(HTML_COMMENT, "").split("\n").map(substance).filter((line) => line !== "").join("\n");
+	return visibleLines(description).map((line) => substance(line.visible)).filter((line) => line !== "").join("\n");
 }
 /**
 * Whether a description has anything in it beyond an unfilled template.
@@ -9468,9 +9706,138 @@ function authorText(description) {
 function hasAuthorContent(description) {
 	return authorText(description) !== "";
 }
-function isLongDescription(description) {
-	const trimmed = description.trim();
-	return trimmed.split("\n").length > 10 || trimmed.length > 700;
+var LIST_ITEM = /^\s*(?:[-*+]|\d+[.)])\s/;
+var ATX_HEADING = /^#{1,6}(?:\s|$)/;
+/**
+* The description as the full view reads it, by the renderer's own scan, so the two always agree
+* on what is code and what is a hidden comment.
+*/
+function visibleLines(description) {
+	return scanMarkdownLines(description, { hideHtmlComments: true }).filter((line) => !line.absent);
+}
+/**
+* The description's Markdown blocks, with a code block kept whole and always a block of its own.
+*
+* Split where Markdown itself would end one: a blank line, a heading (always a block of its own)
+* and a list that starts straight after a paragraph. Descriptions are often written with no blank
+* lines at all, and on blank lines alone those would be one block. A line that is only a comment
+* ends a block like a blank one.
+*/
+function blocks(description) {
+	const result = [];
+	let current = [];
+	const end = () => {
+		if (current.length) result.push(current);
+		current = [];
+	};
+	for (const line of visibleLines(description)) if (line.code === "open") {
+		end();
+		current.push(line);
+	} else if (line.code === "body") current.push(line);
+	else if (line.code === "close") {
+		current.push(line);
+		end();
+	} else if (line.visible.trim() === "") end();
+	else if (ATX_HEADING.test(line.visible.trim())) {
+		end();
+		result.push([line]);
+	} else if (LIST_ITEM.test(line.visible) && current.length && !LIST_ITEM.test(current[0].visible)) {
+		end();
+		current.push(line);
+	} else current.push(line);
+	end();
+	return result;
+}
+var ELLIPSIS = "…";
+/**
+* Whether cutting a paragraph after `text` leaves valid Markdown: no code span, link or bold left
+* open. Approximate by design, and erring towards an earlier cut or none.
+*/
+function closesCleanly(text) {
+	if ((text.match(/`/g) ?? []).length % 2) return false;
+	if ((text.match(/\*\*/g) ?? []).length % 2) return false;
+	if ((text.replace(/\*\*/g, "").match(/\*/g) ?? []).length % 2) return false;
+	if ((text.match(/__/g) ?? []).length % 2) return false;
+	if ((text.match(/\[/g) ?? []).length !== (text.match(/\]/g) ?? []).length) return false;
+	if ((text.match(/\]\(/g) ?? []).length !== (text.match(/\]\([^)]*\)/g) ?? []).length) return false;
+	return true;
+}
+/**
+* A paragraph cut to its first sentences, or to a word boundary near the character limit, at a
+* point that leaves no Markdown open. Null when it fits whole.
+*/
+function boundParagraph(paragraph) {
+	const text = paragraph.trimEnd();
+	const cuts = [];
+	for (const match of text.matchAll(/\s+/g)) {
+		const at = match.index;
+		if (!closesCleanly(text.slice(0, at))) continue;
+		cuts.push({
+			at,
+			sentence: /[.!?]["')\]]?$/.test(text.slice(0, at))
+		});
+	}
+	const bySentence = cuts.filter((cut) => cut.sentence)[1];
+	if (text.length <= 400 && !bySentence) return null;
+	if (bySentence && bySentence.at <= 400) return `${text.slice(0, bySentence.at)} ${ELLIPSIS}`;
+	if (text.length <= 400) return null;
+	const within = cuts.filter((cut) => cut.at <= 400).pop() ?? cuts.find((cut) => cut.at > 400);
+	return within ? `${text.slice(0, within.at)} ${ELLIPSIS}` : null;
+}
+/** A list cut to its first items, each whole with any lines that continue it. Null when it fits. */
+function boundList(lines) {
+	const indent = (line) => line.length - line.trimStart().length;
+	const base = indent(lines[0]);
+	const starts = lines.flatMap((line, i) => LIST_ITEM.test(line) && indent(line) <= base ? [i] : []);
+	if (starts.length <= 3) return null;
+	return `${lines.slice(0, starts[3]).join("\n")}\n\n${ELLIPSIS}`;
+}
+/**
+* What the Overview shows of a description before the reader asks for all of it.
+*
+* The first block the author wrote decides what to show; the bounds decide how much of it. A
+* template's labels, comments and unticked items ahead of it are passed over, so "## Summary" opens
+* on the summary itself. A heading of the author's own is kept, with the block it introduces.
+* A paragraph is cut after its first sentences or near a character limit, a list after its first
+* items, and a long code block is left out; nothing is cut mid-word or left as open Markdown.
+*
+* Null when the whole description fits: there is nothing more to open.
+*/
+function authorPreview(description) {
+	if (!hasAuthorContent(description)) return null;
+	const all = blocks(description);
+	const shown = [];
+	let codeOmitted = false;
+	let cut = false;
+	let i = 0;
+	for (; i < all.length; i++) {
+		const lines = all[i];
+		const first = lines.findIndex((line) => substance(line.visible) !== "");
+		if (first === -1) continue;
+		const raw = lines.slice(first).map((line) => line.visible);
+		if (raw.length === 1 && ATX_HEADING.test(lines[first].visible.trim())) {
+			shown.push(raw[0]);
+			if (shown.length > 1) break;
+			continue;
+		}
+		if (lines[first].code === "open") {
+			if (lines.filter((line) => line.code === "body").length > 6) {
+				codeOmitted = true;
+				cut = true;
+			} else shown.push(raw.join("\n"));
+		} else {
+			const bounded = LIST_ITEM.test(raw[0]) ? boundList(raw) : boundParagraph(raw.join("\n"));
+			cut = bounded !== null;
+			shown.push(bounded ?? raw.join("\n"));
+		}
+		break;
+	}
+	const rest = all.slice(i + 1).flat().map((line) => line.visible);
+	if (!cut && !rest.some((line) => substance(line) !== "")) return null;
+	return {
+		markdown: shown.join("\n\n"),
+		codeOmitted
+	};
 }
 //#endregion
 //#region ../../packages/review-harness/src/domain/ai_review/motivation.ts
@@ -9488,7 +9855,7 @@ var MOTIVATION_EVIDENCE_KINDS = [
 	"code"
 ];
 /** Ours, not the model's: what an unsupported claim is replaced with when a question remains. */
-var UNESTABLISHED_MOTIVATION = "The reason for this change isn't established by the description or the code.";
+var UNESTABLISHED_MOTIVATION = "It isn't clear why this change is needed.";
 /** Stable, so "Mark answered" on it survives a reload like any other question's. */
 var INTENT_QUESTION_ID = "intent-motivation";
 /**
@@ -9555,9 +9922,9 @@ function isAuthorQuote(quote, description) {
 * fields that belong to the other statuses are dropped rather than carried, so a `stated` motive
 * cannot also show a guess and an `unclear` one cannot also show a quote.
 *
-* A claim that fails becomes unclear only if the model also asked the question that would settle
-* it, and then in our words rather than its own. Otherwise there is nothing to show that the
-* reviewer could act on, and nothing is shown.
+* A claim that fails keeps none of its summary and becomes unclear, in our words, asking the
+* model's question or ours. Its evidence goes to the audit. An unclear motive that asked nothing
+* keeps its own summary, which claims nothing, and gets our question.
 */
 function verifyMotivation(candidate, authorDescription) {
 	if (!candidate) return {};
@@ -9584,24 +9951,30 @@ function verifyMotivation(candidate, authorDescription) {
 	const correction = {
 		status: candidate.status,
 		summary: candidate.summary,
-		reason
+		reason,
+		...rejectedQuote(candidate, reason),
+		...candidate.evidence.length > 0 ? { discardedEvidence: candidate.evidence } : {}
 	};
-	if (candidate.status !== "unclear" && candidate.authorQuestion) return {
+	return {
 		motivation: {
 			contractVersion: 1,
 			status: "unclear",
-			summary: UNESTABLISHED_MOTIVATION,
-			authorQuestion: candidate.authorQuestion
+			summary: candidate.status === "unclear" ? candidate.summary : UNESTABLISHED_MOTIVATION,
+			authorQuestion: candidate.authorQuestion ?? "What problem does this change solve, or what goal does it serve?"
 		},
 		correction: {
 			...correction,
 			outcome: "downgraded"
 		}
 	};
-	return { correction: {
-		...correction,
-		outcome: "withheld"
-	} };
+}
+function rejectedQuote(candidate, reason) {
+	if (reason !== "unverified-quote" || !candidate.sourceQuote) return {};
+	const quote = candidate.sourceQuote;
+	return quote.length > 500 ? {
+		rejectedQuote: quote.slice(0, 500),
+		rejectedQuoteTruncated: true
+	} : { rejectedQuote: quote };
 }
 /**
 * The one evidence-link policy, for both hosts.
@@ -9647,21 +10020,23 @@ function sameQuestion(a, b) {
 /**
 * File an unclear motive's question with the rest, as the one Intent question.
 *
-* A question the model already asked in the same words is tagged rather than repeated, so the
-* reader meets it once, in the Intent group. Telling two differently worded questions apart is
-* judgement and is left to the prompt.
+* A question the model already asked in the same words becomes the Intent question rather than a
+* second copy, so the reader meets it once. It takes the stable id either way: a model's ids are
+* new on every analysis, and "Mark answered" is stored by id. Telling two differently worded
+* questions apart is judgement and is left to the prompt.
 */
 function withIntentQuestion(questions, motivation) {
 	if (motivation?.status !== "unclear" || !motivation.authorQuestion) return questions;
 	const asked = motivation.authorQuestion;
 	const existing = questions.findIndex((q) => sameQuestion(q.question, asked));
+	const taken = new Set(questions.filter((_, index) => index !== existing).map((q) => q.id));
+	let id = INTENT_QUESTION_ID;
+	for (let n = 2; taken.has(id); n += 1) id = `${INTENT_QUESTION_ID}-${n}`;
 	if (existing >= 0) return questions.map((q, index) => index === existing ? {
 		...q,
+		id,
 		category: "intent"
 	} : q);
-	const taken = new Set(questions.map((q) => q.id));
-	let id = INTENT_QUESTION_ID;
-	for (let n = 2; taken.has(id); n += 1) id = `${INTENT_QUESTION_ID}-${n}`;
 	return [...questions, {
 		id,
 		question: asked,
@@ -9682,6 +10057,15 @@ function applyMotivation(result, candidate, authorDescription) {
 	};
 }
 //#endregion
+//#region ../../packages/review-harness/src/domain/ai_review/summary_guidance.ts
+/**
+* What the "summary" field is for, as both the first analysis and a reconciliation ask for it.
+*
+* One string so the two prompts cannot drift. It renders as "What's changing" on the Overview,
+* between "Why this change?" and the risk card, so it says neither why nor how good.
+*/
+var SUMMARY_GUIDANCE = `"summary": WHAT the change does, for a reviewer about to read it. Roughly 80-120 words, and fewer for a small change. Name the parts that change and what each now does. Do not explain why the change exists, and do not judge whether it is correct or safe: other fields carry both.`;
+//#endregion
 //#region ../../packages/review-harness/src/domain/ai_review/pipeline.ts
 var STRUCTURED_OUTPUT_INSTRUCTIONS = `In addition to the review principles above, first understand the implementation, then respond with a single JSON object (no markdown, no surrounding text) matching exactly this shape:
 {
@@ -9715,8 +10099,9 @@ var STRUCTURED_OUTPUT_INSTRUCTIONS = `In addition to the review principles above
 }
 
 Field guidance:
+- ${SUMMARY_GUIDANCE}
 - "motivation": WHY this change exists, which is a different question from what it does ("summary" covers that). Name the problem being solved or the goal being served, never the mechanics: "adds a retry loop" is what the code does, "the sync job was failing on transient network errors" is why. Weigh sources in this order: the PR Description and any issue it links; then tests and user-visible behaviour; then commit messages and docs; then the implementation itself. Choose exactly one "status":
-  - "stated": the author's own words in the PR Description say why. "sourceQuote" is REQUIRED: a short passage (one sentence or less, at least three words) copied character for character from the PR Description, which says why. Only the PR Description counts as the author stating it. Never use "stated" when the description only restates what the code does.
+  - "stated": the author's own words in the PR Description say why. "sourceQuote" is REQUIRED: a short passage (one sentence or less, at least three words) copied character for character from the PR Description, which says why. Copy it exactly as written, keeping backticks, punctuation, hyphenation and spelling; never paraphrase, shorten words, join two sentences or correct a typo. Only the PR Description counts as the author stating it: a reason that appears only in commit messages, review comments, or an issue the description merely links to is not "stated" (use "inferred" and cite it as evidence). Never use "stated" when the description only restates what the code does. If the change itself also shows the reason, include it as "evidence".
   - "inferred": nothing states it, but the change gives real evidence of it. "evidence" is REQUIRED and non-empty: each entry names what points to the motive, such as a test asserting the fixed behaviour, an issue reference, or a commit message, with "file" set to the changed file it is in where there is one. Never infer a motive from names alone (a branch, a function or a file being called "fix-x" is not evidence). Word the summary as a likely reading, not a fact: "This appears to..." rather than "This fixes...".
   - "unclear": neither the description nor the change establishes why, AND knowing why would change how this change should be reviewed (for example, whether a behaviour change is intended). "authorQuestion" is REQUIRED: one focused question for the author that would settle it. Do not also ask that question, in any wording, in "questions".
   If the motive is unclear and knowing it would not change the review, omit "motivation" entirely. Do not add it just to fill the field.
@@ -12107,7 +12492,7 @@ function buildReconciliationInstructions(input, options = {}) {
    The completeness rule applies here too: one entry per numbered question, and "still-open" with a note rather than an omission when you are unsure. Do not manufacture an answer out of intent you cannot see. A question about why something was done the way it was is often not answerable from code at all, and "still-open" is the correct answer to it.`] : [],
 		`Read the changed code above as a reviewer, not only as an auditor of the numbered list. These commits have not been reviewed by anyone. Report every genuinely NEW problem you find in them in "newIssues", whether or not it relates to anything on the list — code written to fix a finding is exactly where a regression gets introduced, and reassessing the list and stopping there is half the job. Do NOT repeat a previous finding here: if it still stands, that is what "still-open" is for.`,
 		`If the new changes introduce an area the earlier analysis had no section for, add it to "newSections". Otherwise return an empty array. Do not restate or restructure the earlier analysis's existing sections.`,
-		`Give a fresh "summary", "overallRisk", "recommendation" and "nextSteps" for the pull request AS IT NOW STANDS, taking both the earlier findings that survive and the new changes into account. Unlike a partial review, you have been shown the earlier analysis in full and are expected to revise its verdict where the new changes justify it.`
+		`Give a fresh "summary", "overallRisk", "recommendation" and "nextSteps" for the pull request AS IT NOW STANDS, taking both the earlier findings that survive and the new changes into account. Unlike a partial review, you have been shown the earlier analysis in full and are expected to revise its verdict where the new changes justify it. ${SUMMARY_GUIDANCE}`
 	].map((task, index) => `${index + 1}. ${task}`).join("\n");
 	return `You are RECONCILING an earlier analysis of this pull request against the changes pushed since that analysis was made.
 
@@ -13433,4 +13818,4 @@ function lineageIds(id) {
 	return [.../* @__PURE__ */ new Set([id, ...lineage.map((review) => review.id)])];
 }
 //#endregion
-export { relativeTime as $, AGENT_IDS as $n, kissDecisionRefusal as $t, createPrState as A, readQueue as An, checkCodeFreshness as At, toneOf as B, WITHHELD_REASON as Bn, PHASES as Bt, readEvents as C, offersLifecycle as Cn, statedWorkspacePort as Ct, announceJob as D, queueAddMany as Dn, forgeFor as Dt, telemetryUploadConsent as E, queueAdd as En, forgeResolver as Et, markAllRead as F, questionsOutstanding as Fn, endedAt as Ft, isLongDescription as G, git as Gn, clearClaim as Gt, withUsageRecorded as H, resolveTarget as Hn, phasesFor as Ht, markRead as I, reviewerDispositions as In, fail as It, ReviewBeingDeletedError as J, clearAgentPreference as Jn, decideKissRecommendation as Jt, asString as K, resolveAgent as Kn, driverGone as Kt, pruneNotifications as L, processDiscussion as Ln, isTerminal as Lt, eventName as M, isIssueOutstanding as Mn, followedBranch as Mt, headline as N, isQuestionOutstanding as Nn, reviewedShaOf as Nt, branchOf as O, queueRemove as On, remoteHost as Ot, listNotifications as P, issuesOutstanding as Pn, allJobs as Pt, modelHelpLines as Q, STORE_ROOT as Qn, indexReviewGroups as Qt, readNotification as R, recommendationsOf as Rn, loadJob as Rt, startJob as S, lifecyclePill as Sn, resolveWorkspacePort as St, recordTelemetryUploadConsent as T, sectionOf as Tn, claimRequestUrl as Tt, EXPLAIN_SIMPLY_PROMPT as U, displayRef as Un, progressSteps as Ut, unreadCount as V, kissOutcome as Vn, isLiveJob as Vt, hasAuthorContent as W, isCommitReview as Wn, purposeOf as Wt, readMarker as X, writeAgentPreference as Xn, groupByLineage as Xt, MANAGED_ROOT as Y, readAgentPreference as Yn, followedRefName as Yt, removeWorktree as Z, CONFIG_PATH as Zn, indexKeptIds as Zt, agentEnvOf as _, LIFECYCLES as _n, feedbackUrl as _t, wasBlocked as a, matchingWorkspaceIds as an, AgentUnavailableError as ar, bootstrapUrl as at, reanalyseReview as b, lifecycleAction as bn, MIN_WORKSPACE_PORT as bt, RefreshUnavailableError as c, presentedKissRunOf as cn, readServerRecord as ct, refreshPrContext as d, reviewedCommit as dn, spawnServer as dt, kissDecisionsOf as en, DEFAULT_AGENT_ID as er, activateQueued as et, isUnchanged as f, reviewedRepositories as fn, stopServer as ft, runningJobs as g, workspaceStamps as gn, PACKAGE_NAME as gt, liveUpdateFor as h, workspaceRevision as hn, BUILD_VERSION as ht, liveJobsFor as i, loadWorkspace as in, AgentCancelledError as ir, readIndexToken as it, createPrUpdate as j, doneVerb as jn, checkReview as jt, subjectOf as k, queueStamp as kn, fillFileUrlTemplate as kt, mergeUpdateInto as l, previousKissResult as ln, reviewsUrl as lt, UpdateAlreadyRunningError as m, touchWorkspace as mn, writeServerRecord as mt, deleteReview as n, lineageKeyFor as nn, agentFor as nr, queueForReview as nt, discardJob as o, mutateWorkspace as on, clearServerRecord as ot, askCheckout as p, summarise as pn, workspaceUrl as pt, extractJsonText as q, resolveModel as qn, isClaimed as qt, lineageIds as r, lineagePosition as rn, detectAgents as rr, settleQueue as rt, FETCH_TIMEOUT_MS as s, positionInLineage as sn, ensureServer as st, deleteLineage as t, latestKissRun as tn, agentById as tr, compactQueue as tt, updateReview as u, recentWorkspaces as un, serverEntryExists as ut, recordKissRun as v, activityLabel as vn, DEFAULT_WORKSPACE_PORT as vt, record as w, parseLifecycle as wn, writePortPreference as wt, runJob as x, lifecycleChip as xn, parseWorkspacePort as xt, startKissJob as y, isLifecycle as yn, MAX_WORKSPACE_PORT as yt, removeNotification as z, describeAuthorship as zn, saveJob as zt };
+export { extractJsonText as $, resolveModel as $n, isClaimed as $t, createPrState as A, parseLifecycle as An, writePortPreference as At, toneOf as B, issuesOutstanding as Bn, allJobs as Bt, readEvents as C, LIFECYCLES as Cn, feedbackUrl as Ct, announceJob as D, lifecycleChip as Dn, parseWorkspacePort as Dt, telemetryUploadConsent as E, lifecycleAction as En, MIN_WORKSPACE_PORT as Et, markAllRead as F, queueStamp as Fn, fillFileUrlTemplate as Ft, authorPreview as G, describeAuthorship as Gn, saveJob as Gt, withUsageRecorded as H, reviewerDispositions as Hn, fail as Ht, markRead as I, readQueue as In, checkCodeFreshness as It, CODE_BLOCK_CLASS as J, resolveTarget as Jn, phasesFor as Jt, hasAuthorContent as K, WITHHELD_REASON as Kn, PHASES as Kt, pruneNotifications as L, doneVerb as Ln, checkReview as Lt, eventName as M, queueAdd as Mn, forgeResolver as Mt, headline as N, queueAddMany as Nn, forgeFor as Nt, branchOf as O, lifecyclePill as On, resolveWorkspacePort as Ot, listNotifications as P, queueRemove as Pn, remoteHost as Pt, asString as Q, resolveAgent as Qn, driverGone as Qt, readNotification as R, isIssueOutstanding as Rn, followedBranch as Rt, startJob as S, workspaceStamps as Sn, PACKAGE_NAME as St, recordTelemetryUploadConsent as T, isLifecycle as Tn, MAX_WORKSPACE_PORT as Tt, EXPLAIN_SIMPLY_PROMPT as U, processDiscussion as Un, isTerminal as Ut, unreadCount as V, questionsOutstanding as Vn, endedAt as Vt, UNESTABLISHED_MOTIVATION as W, recommendationsOf as Wn, loadJob as Wt, escapeHtml as X, isCommitReview as Xn, purposeOf as Xt, CODE_COPY_CLASS as Y, displayRef as Yn, progressSteps as Yt, renderInlineText as Z, git as Zn, clearClaim as Zt, agentEnvOf as _, reviewedCommit as _n, spawnServer as _t, wasBlocked as a, kissDecisionRefusal as an, AGENT_IDS as ar, relativeTime as at, reanalyseReview as b, touchWorkspace as bn, writeServerRecord as bt, RefreshUnavailableError as c, lineageKeyFor as cn, agentFor as cr, queueForReview as ct, refreshPrContext as d, matchingWorkspaceIds as dn, AgentUnavailableError as dr, bootstrapUrl as dt, decideKissRecommendation as en, clearAgentPreference as er, ReviewBeingDeletedError as et, isUnchanged as f, mutateWorkspace as fn, clearServerRecord as ft, runningJobs as g, recentWorkspaces as gn, serverEntryExists as gt, liveUpdateFor as h, previousKissResult as hn, reviewsUrl as ht, liveJobsFor as i, indexReviewGroups as in, STORE_ROOT as ir, modelHelpLines as it, createPrUpdate as j, sectionOf as jn, claimRequestUrl as jt, subjectOf as k, offersLifecycle as kn, statedWorkspacePort as kt, mergeUpdateInto as l, lineagePosition as ln, detectAgents as lr, settleQueue as lt, UpdateAlreadyRunningError as m, presentedKissRunOf as mn, readServerRecord as mt, deleteReview as n, groupByLineage as nn, writeAgentPreference as nr, readMarker as nt, discardJob as o, kissDecisionsOf as on, DEFAULT_AGENT_ID as or, activateQueued as ot, askCheckout as p, positionInLineage as pn, ensureServer as pt, renderMarkdown as q, kissOutcome as qn, isLiveJob as qt, lineageIds as r, indexKeptIds as rn, CONFIG_PATH as rr, removeWorktree as rt, FETCH_TIMEOUT_MS as s, latestKissRun as sn, agentById as sr, compactQueue as st, deleteLineage as t, followedRefName as tn, readAgentPreference as tr, MANAGED_ROOT as tt, updateReview as u, loadWorkspace as un, AgentCancelledError as ur, readIndexToken as ut, recordKissRun as v, reviewedRepositories as vn, stopServer as vt, record as w, activityLabel as wn, DEFAULT_WORKSPACE_PORT as wt, runJob as x, workspaceRevision as xn, BUILD_VERSION as xt, startKissJob as y, summarise as yn, workspaceUrl as yt, removeNotification as z, isQuestionOutstanding as zn, reviewedShaOf as zt };

package/dist/main.js

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { $ as relativeTime, $n as AGENT_IDS, Bn as WITHHELD_REASON, Bt as PHASES, Ct as statedWorkspacePort, E as telemetryUploadConsent, Et as forgeResolver, Gn as git, Gt as clearClaim, Hn as resolveTarget, Kn as resolveAgent, Kt as driverGone, Lt as isTerminal, Pt as allJobs, Q as modelHelpLines, Qn as STORE_ROOT, Rt as loadJob, S as startJob, T as recordTelemetryUploadConsent, Un as displayRef, Ut as progressSteps, Vn as kissOutcome, Wt as purposeOf, X as readMarker, Xn as writeAgentPreference, Xt as groupByLineage, Y as MANAGED_ROOT, Yn as readAgentPreference, Yt as followedRefName, Z as removeWorktree, Zn as CONFIG_PATH, a as wasBlocked, an as matchingWorkspaceIds, at as bootstrapUrl, b as reanalyseReview, bt as MIN_WORKSPACE_PORT, er as DEFAULT_AGENT_ID, fn as reviewedRepositories, ft as stopServer, g as runningJobs, gt as PACKAGE_NAME, h as liveUpdateFor, ht as BUILD_VERSION, i as liveJobsFor, in as loadWorkspace, it as readIndexToken, jt as checkReview, l as mergeUpdateInto, lt as reviewsUrl, m as UpdateAlreadyRunningError, n as deleteReview$1, nn as lineageKeyFor, nt as queueForReview, on as mutateWorkspace, pn as summarise, pt as workspaceUrl, qn as resolveModel, qt as isClaimed, rr as detectAgents, sn as positionInLineage, st as ensureServer, tr as agentById, u as updateReview, un as recentWorkspaces, v as recordKissRun, vt as DEFAULT_WORKSPACE_PORT, w as record$1, wt as writePortPreference, x as runJob, xt as parseWorkspacePort, yt as MAX_WORKSPACE_PORT } from "./delete_review-B2bqhgtx.js";
+import { $n as resolveModel, $t as isClaimed, At as writePortPreference, Bt as allJobs, Dt as parseWorkspacePort, E as telemetryUploadConsent, Et as MIN_WORKSPACE_PORT, Jn as resolveTarget, Kn as WITHHELD_REASON, Kt as PHASES, Lt as checkReview, Mt as forgeResolver, Qn as resolveAgent, Qt as driverGone, S as startJob, St as PACKAGE_NAME, T as recordTelemetryUploadConsent, Tt as MAX_WORKSPACE_PORT, Ut as isTerminal, Wt as loadJob, Xt as purposeOf, Yn as displayRef, Yt as progressSteps, Zn as git, Zt as clearClaim, a as wasBlocked, ar as AGENT_IDS, at as relativeTime, b as reanalyseReview, cn as lineageKeyFor, ct as queueForReview, dn as matchingWorkspaceIds, dt as bootstrapUrl, fn as mutateWorkspace, g as runningJobs, gn as recentWorkspaces, h as liveUpdateFor, ht as reviewsUrl, i as liveJobsFor, ir as STORE_ROOT, it as modelHelpLines, kt as statedWorkspacePort, l as mergeUpdateInto, lr as detectAgents, m as UpdateAlreadyRunningError, n as deleteReview$1, nn as groupByLineage, nr as writeAgentPreference, nt as readMarker, or as DEFAULT_AGENT_ID, pn as positionInLineage, pt as ensureServer, qn as kissOutcome, rr as CONFIG_PATH, rt as removeWorktree, sr as agentById, tn as followedRefName, tr as readAgentPreference, tt as MANAGED_ROOT, u as updateReview, un as loadWorkspace, ut as readIndexToken, v as recordKissRun, vn as reviewedRepositories, vt as stopServer, w as record$1, wt as DEFAULT_WORKSPACE_PORT, x as runJob, xt as BUILD_VERSION, yn as summarise, yt as workspaceUrl } from "./delete_review-DEvRanb6.js";
 import { basename, dirname, join, resolve } from "node:path";
 import { spawn } from "node:child_process";
 import { existsSync, mkdirSync, readdirSync, realpathSync, rmSync, statSync, writeFileSync } from "node:fs";