prreviewbuddy 0.55.1 → 0.55.6

package/dist/server.js

@@ -1,4 +1,4 @@
-import { $ as relativeTime$1, $n as AGENT_IDS, $t as kissDecisionRefusal, A as createPrState, An as readQueue, At as checkCodeFreshness, B as toneOf, Bn as WITHHELD_REASON, C as readEvents, Cn as offersLifecycle, Ct as statedWorkspacePort, D as announceJob, Dn as queueAddMany, Dt as forgeFor, E as telemetryUploadConsent, En as queueAdd, F as markAllRead, Fn as questionsOutstanding, Ft as endedAt, G as isLongDescription, Gn as git, H as withUsageRecorded, Ht as phasesFor, I as markRead, In as reviewerDispositions, It as fail, J as ReviewBeingDeletedError, Jn as clearAgentPreference, Jt as decideKissRecommendation, K as asString, L as pruneNotifications, Ln as processDiscussion, Lt as isTerminal, M as eventName, Mn as isIssueOutstanding, Mt as followedBranch, N as headline, Nn as isQuestionOutstanding, Nt as reviewedShaOf, O as branchOf, On as queueRemove, Ot as remoteHost, P as listNotifications, Pn as issuesOutstanding, Pt as allJobs, Qn as STORE_ROOT, Qt as indexReviewGroups, R as readNotification, Rn as recommendationsOf, Rt as loadJob, Sn as lifecyclePill, St as resolveWorkspacePort, T as recordTelemetryUploadConsent, Tn as sectionOf, Tt as claimRequestUrl, U as EXPLAIN_SIMPLY_PROMPT, Ut as progressSteps, V as unreadCount$1, Vn as kissOutcome, Vt as isLiveJob, W as hasAuthorContent, Wn as isCommitReview, 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, Zt as indexKeptIds, _ as agentEnvOf, _n as LIFECYCLES, _t as feedbackUrl, a as wasBlocked, ar as AgentUnavailableError, b as reanalyseReview, bn as lifecycleAction, bt as MIN_WORKSPACE_PORT, c as RefreshUnavailableError, cn as presentedKissRunOf, ct as readServerRecord, d as refreshPrContext, dn as reviewedCommit, dt as spawnServer, en as kissDecisionsOf, et as activateQueued, f as isUnchanged, gn as workspaceStamps, h as liveUpdateFor, hn as workspaceRevision, ht as BUILD_VERSION, i as liveJobsFor, in as loadWorkspace, ir as AgentCancelledError, it as readIndexToken, j as createPrUpdate, jn as doneVerb, jt as checkReview, k as subjectOf, kn as queueStamp, kt as fillFileUrlTemplate, l as mergeUpdateInto, ln as previousKissResult, m as UpdateAlreadyRunningError, mn as touchWorkspace, mt as writeServerRecord, n as deleteReview, nr as agentFor, o as discardJob, on as mutateWorkspace, ot as clearServerRecord, p as askCheckout, q as extractJsonText, qn as resolveModel, qt as isClaimed, r as lineageIds, rn as lineagePosition, rt as settleQueue, s as FETCH_TIMEOUT_MS, t as deleteLineage, tn as latestKissRun, tt as compactQueue, u as updateReview, un as recentWorkspaces, ut as serverEntryExists, vn as activityLabel, vt as DEFAULT_WORKSPACE_PORT, w as record, wn as parseLifecycle, wt as writePortPreference, x as runJob, xn as lifecycleChip, xt as parseWorkspacePort, y as startKissJob, yn as isLifecycle, yt as MAX_WORKSPACE_PORT, z as removeNotification, zn as describeAuthorship, zt as saveJob } from "./delete_review-B2bqhgtx.js";
+import { $ as extractJsonText, $n as resolveModel, $t as isClaimed, A as createPrState, An as parseLifecycle, At as writePortPreference, B as toneOf, Bn as issuesOutstanding, Bt as allJobs, C as readEvents, Cn as LIFECYCLES, Ct as feedbackUrl, D as announceJob, Dn as lifecycleChip, Dt as parseWorkspacePort, E as telemetryUploadConsent, En as lifecycleAction, Et as MIN_WORKSPACE_PORT, F as markAllRead, Fn as queueStamp, Ft as fillFileUrlTemplate, G as authorPreview, Gn as describeAuthorship, Gt as saveJob, H as withUsageRecorded, Hn as reviewerDispositions, Ht as fail, I as markRead, In as readQueue, It as checkCodeFreshness, J as CODE_BLOCK_CLASS, Jt as phasesFor, K as hasAuthorContent, Kn as WITHHELD_REASON, L as pruneNotifications, Ln as doneVerb, Lt as checkReview, M as eventName, Mn as queueAdd, N as headline, Nn as queueAddMany, Nt as forgeFor, O as branchOf, On as lifecyclePill, Ot as resolveWorkspacePort, P as listNotifications, Pn as queueRemove, Pt as remoteHost, Q as asString, R as readNotification, Rn as isIssueOutstanding, Rt as followedBranch, Sn as workspaceStamps, T as recordTelemetryUploadConsent, Tn as isLifecycle, Tt as MAX_WORKSPACE_PORT, U as EXPLAIN_SIMPLY_PROMPT, Un as processDiscussion, Ut as isTerminal, V as unreadCount$1, Vn as questionsOutstanding, Vt as endedAt, W as UNESTABLISHED_MOTIVATION, Wn as recommendationsOf, Wt as loadJob, X as escapeHtml$1, Xn as isCommitReview, Xt as purposeOf, Y as CODE_COPY_CLASS, Yt as progressSteps, Z as renderInlineText, Zn as git, _ as agentEnvOf, _n as reviewedCommit, _t as spawnServer, a as wasBlocked, an as kissDecisionRefusal, ar as AGENT_IDS, at as relativeTime$1, b as reanalyseReview, bn as touchWorkspace, bt as writeServerRecord, c as RefreshUnavailableError, cr as agentFor, d as refreshPrContext, dr as AgentUnavailableError, en as decideKissRecommendation, er as clearAgentPreference, et as ReviewBeingDeletedError, f as isUnchanged, fn as mutateWorkspace, ft as clearServerRecord, gn as recentWorkspaces, gt as serverEntryExists, h as liveUpdateFor, hn as previousKissResult, i as liveJobsFor, in as indexReviewGroups, ir as STORE_ROOT, j as createPrUpdate, jn as sectionOf, jt as claimRequestUrl, k as subjectOf, kn as offersLifecycle, kt as statedWorkspacePort, l as mergeUpdateInto, ln as lineagePosition, lt as settleQueue, m as UpdateAlreadyRunningError, mn as presentedKissRunOf, mt as readServerRecord, n as deleteReview, nn as groupByLineage, nr as writeAgentPreference, nt as readMarker, o as discardJob, on as kissDecisionsOf, ot as activateQueued, p as askCheckout, q as renderMarkdown, qn as kissOutcome, qt as isLiveJob, r as lineageIds, rn as indexKeptIds, rt as removeWorktree, s as FETCH_TIMEOUT_MS, sn as latestKissRun, st as compactQueue, t as deleteLineage, tn as followedRefName, tr as readAgentPreference, tt as MANAGED_ROOT, u as updateReview, un as loadWorkspace, ur as AgentCancelledError, ut as readIndexToken, w as record, wn as activityLabel, wt as DEFAULT_WORKSPACE_PORT, x as runJob, xn as workspaceRevision, xt as BUILD_VERSION, y as startKissJob, z as removeNotification, zn as isQuestionOutstanding, zt as reviewedShaOf } from "./delete_review-DEvRanb6.js";
 import { basename, dirname, isAbsolute, join, relative, resolve } from "node:path";
 import { existsSync, mkdirSync, readFileSync, readdirSync, renameSync, statSync, unlinkSync, writeFileSync } from "node:fs";
 import { createHash, timingSafeEqual } from "node:crypto";
@@ -233,14 +233,6 @@ function readingThemeLine(sections, sectionId) {
 	return section ? `The reviewer is currently reading the "${section.title}" theme. Answer with that in view, but do not ignore the rest of the change if it bears on the question.` : null;
 }
 //#endregion
-//#region ../../packages/review-harness/src/domain/utils/html.ts
-function escapeHtml$1(value) {
-	return value.replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;").replace(/"/g, "&quot;").replace(/'/g, "&#39;");
-}
-function renderInlineText(value) {
-	return escapeHtml$1(value).replace(/`([^`\n]+)`/g, "<code>$1</code>");
-}
-//#endregion
 //#region ../../packages/review-harness/src/workspace/reviews_index.ts
 /**
 * The sections of the list, top to bottom.
@@ -980,139 +972,6 @@ function formatRelativeTime(ts, now = Date.now()) {
 	return `${days} day${days === 1 ? "" : "s"} ago`;
 }
 //#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) {
-	const { text, blocks } = extractFencedBlocks(value.replace(/[\u0000\u0001]/g, ""));
-	return restoreFencedBlocks(renderBlocks(text), blocks);
-}
-function extractFencedBlocks(value) {
-	const blocks = [];
-	const lines = value.split("\n");
-	const out = [];
-	let i = 0;
-	while (i < lines.length) {
-		const opening = lines[i].match(/^\s*(`{3,}|~{3,})\s*[\w+#.-]*\s*$/);
-		if (!opening) {
-			out.push(lines[i]);
-			i += 1;
-			continue;
-		}
-		const closing = new RegExp(`^\\s*${opening[1][0]}{3,}\\s*$`);
-		const body = [];
-		i += 1;
-		while (i < lines.length && !closing.test(lines[i])) {
-			body.push(lines[i]);
-			i += 1;
-		}
-		i += 1;
-		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$1(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$1(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/ui/components/intent_note.ts
 /**
 * The Overview's unclear motive, kept in step with its Intent question.
@@ -1124,7 +983,7 @@ function renderInline(value) {
 * Marking a question answered records that it was answered, never what the answer was, so nothing
 * here touches the stored motivation. It only says the analysis is older than the answer.
 */
-var INTENT_NOTE_COPY = "This question is marked answered. If the answer was added to the PR description or conversation, run the review again to incorporate it.";
+var INTENT_NOTE_COPY = "This question has been marked answered. Run the review again after adding the answer to the PR description.";
 var RUN_REVIEW_AGAIN = "Run review again";
 //#endregion
 //#region ../../packages/review-harness/src/domain/ui/components/author_context_toggle.ts
@@ -1134,9 +993,13 @@ var RUN_REVIEW_AGAIN = "Run review again";
 * Its own module because both hosts call it from their own click handling, and the workspace
 * client should not have to bundle the whole tab renderer to do it. The markup it acts on is
 * written by `renderAuthorContext` in review_tabs.ts, from the same constants.
+*
+* The preview and the full description are two elements, and one of them is always `hidden`. A
+* closed description is not in the tab order or the accessibility tree, so nothing focusable can
+* sit out of sight inside it.
 */
-var AUTHOR_CONTEXT_COLLAPSED_CLASS = "pr-helper-author-context-collapsed";
-var SHOW_FULL_DESCRIPTION = "Show full description";
+var AUTHOR_CONTEXT_PREVIEW_ID = "pr-helper-author-context-preview";
+var SHOW_FULL_DESCRIPTION = "View full PR description";
 var AUTHOR_CONTEXT_SECTION_ID = "pr-helper-author-context";
 //#endregion
 //#region ../../packages/review-harness/src/domain/utils/model_label.ts
@@ -1785,20 +1648,24 @@ function renderOverviewActions(session, options) {
 * template: an empty "Author's context" would read as the author having said nothing, when on a
 * local branch there was never anyone to ask.
 *
-* Long descriptions are clamped rather than cut. The whole text is in the page, checklists and
-* testing notes included, and the button only changes how much of it shows.
+* Opens on a bounded preview of the first thing the author wrote (`authorPreview`), so it sits
+* beside the other two narrative sections rather than outweighing them. The whole description is in the page, checklists and testing notes
+* included, and the button swaps the preview for it. Nothing is cut from either.
 */
+var GITHUB_MARKDOWN = { hideHtmlComments: true };
 function renderAuthorContext(session) {
 	const description = session.metadata.authorDescription;
 	if (!hasAuthorContent(description)) return "";
 	const id = "pr-helper-author-context-body";
-	const long = isLongDescription(description);
-	const toggle = long ? `<button type="button" class="pr-helper-author-context-toggle" aria-expanded="false" aria-controls="${id}" data-author-context-toggle>${SHOW_FULL_DESCRIPTION}</button>` : "";
+	const preview = authorPreview(description);
+	const omitted = preview?.codeOmitted ? "<p class=\"pr-helper-author-context-omitted\">Code block included</p>" : "";
+	const opening = preview === null ? "" : `<div class="pr-helper-author-context-body pr-helper-markdown" id="${AUTHOR_CONTEXT_PREVIEW_ID}">${preview.markdown ? renderMarkdown(preview.markdown, GITHUB_MARKDOWN) : ""}${omitted}</div>
+      <button type="button" class="pr-helper-author-context-toggle" aria-expanded="false" aria-controls="${id}" data-author-context-toggle>${SHOW_FULL_DESCRIPTION}</button>`;
 	return `
     <section class="pr-helper-author-context" id="${AUTHOR_CONTEXT_SECTION_ID}" aria-labelledby="pr-helper-author-context-heading">
       <h4 class="pr-helper-overview-heading" id="pr-helper-author-context-heading" tabindex="-1">Author's context</h4>
-      <div class="pr-helper-author-context-body pr-helper-markdown${long ? ` ${AUTHOR_CONTEXT_COLLAPSED_CLASS}` : ""}" id="${id}">${renderMarkdown(description)}</div>
-      ${toggle}
+      <div class="pr-helper-author-context-body pr-helper-markdown" id="${id}"${preview === null ? "" : " hidden"}>${renderMarkdown(description, GITHUB_MARKDOWN)}</div>
+      ${opening}
     </section>
   `;
 }
@@ -1814,9 +1681,9 @@ function renderEvidenceFile(item) {
 	return `<span class="pr-helper-why-evidence-unverified"><code>${escapeHtml$1(item.file)}</code> · unverified path</span>`;
 }
 var MOTIVATION_LABELS = {
-	stated: "Stated by the author",
-	inferred: "Likely motivation",
-	unclear: "Motivation unclear"
+	stated: "Author's reason",
+	inferred: "Possible reason",
+	unclear: "Reason unclear"
 };
 var EVIDENCE_LABELS = {
 	test: "Test",
@@ -1829,8 +1696,8 @@ var EVIDENCE_LABELS = {
 /**
 * Why the change exists, and how the review knows.
 *
-* Every status shows what supports it: a stated motive its quote, an inferred one its evidence, an
-* unclear one the question that would settle it. `verifyMotivation` guarantees each is there, so a
+* Every status carries what supports it: a stated motive its quote (one click away), an inferred
+* one its evidence, an unclear one the question that would settle it. `verifyMotivation` guarantees each is there, so a
 * status rendered without its support would be a stored result this version did not write.
 *
 * Nothing for a result with no motivation, which covers every review made before the field and
@@ -1839,40 +1706,57 @@ var EVIDENCE_LABELS = {
 function renderWhyThisChange(motivation, session, questions) {
 	if (!motivation) return "";
 	if (motivation.status === "unclear") return renderUnclearMotive(motivation, session, questions);
+	let summary = renderInlineText(motivation.summary);
 	let support = "";
 	if (motivation.status === "stated" && motivation.sourceQuote) {
 		const source = hasAuthorContent(session.metadata.authorDescription) ? `<button type="button" class="pr-helper-why-link" data-author-context-reveal>From Author's context</button>` : "";
-		support = `<blockquote class="pr-helper-why-quote">${renderInlineText(motivation.sourceQuote)}</blockquote>${source}`;
-	} else if (motivation.status === "inferred" && motivation.evidence?.length) support = `<ul class="pr-helper-why-evidence">${motivation.evidence.map((item) => `<li><span class="pr-helper-why-evidence-kind">${EVIDENCE_LABELS[item.kind]}</span> ${renderInlineText(item.detail)}${renderEvidenceFile(item)}</li>`).join("")}</ul>`;
+		support = `<details class="pr-helper-why-source"><summary>Author's words</summary><blockquote class="pr-helper-why-quote">${renderInlineText(motivation.sourceQuote)}</blockquote>${source}</details>`;
+	} else if (motivation.status === "inferred") {
+		summary = renderInlineText(possibleReason(motivation.summary, hasAuthorContent(session.metadata.authorDescription)));
+		if (motivation.evidence?.length) support = `<ul class="pr-helper-why-evidence">${motivation.evidence.map((item) => `<li><span class="pr-helper-why-evidence-kind">${EVIDENCE_LABELS[item.kind]}</span> ${renderInlineText(item.detail)}${renderEvidenceFile(item)}</li>`).join("")}</ul>`;
+	}
 	return `
-    <section class="pr-helper-why" aria-labelledby="pr-helper-why-heading">
+    <section class="pr-helper-why pr-helper-why-${motivation.status}" aria-labelledby="pr-helper-why-heading">
       <h4 class="pr-helper-overview-heading" id="pr-helper-why-heading">Why this change?</h4>
       <p class="pr-helper-why-status pr-helper-why-status-${motivation.status}">${MOTIVATION_LABELS[motivation.status]}</p>
-      <p class="pr-helper-why-summary">${renderInlineText(motivation.summary)}</p>
+      <p class="pr-helper-why-summary">${summary}</p>
       ${support}
     </section>
   `;
 }
 /**
+* An inferred reason, said as one: where it came from, then the model's own hedged sentence.
+*
+* The prompt asks for "This appears to...", which reads on after a comma once its first letter is
+* lowered. Anything else keeps its wording and capital after a colon, so a sentence that opens on
+* a name (`GitHub`, `ErrorSeverity`) is never rewritten. With no description to speak of, nothing
+* is said about one.
+*/
+function possibleReason(summary, hasDescription) {
+	const lead = hasDescription ? "The PR description doesn't give a clear reason. " : "";
+	return /^This\s/.test(summary) ? `${lead}Based on the change itself, t${summary.slice(1)}` : `${lead}Based on the change itself: ${summary}`;
+}
+/**
 * An unclear motive, and what became of its question.
 *
-* Once the question is answered the stored motivation is still the analysis's, so it is shown as
-* that: unclear when analysed. Only a new analysis that reads the answer may replace it, and the
-* button runs one. Both halves are rendered and one is hidden, so the workspace can switch them
-* when a question is marked answered in place (`syncIntentNote`).
+* Always our own sentence, never the model's: an unclear motive is the absence of a reason, and
+* the reader needs only that and what to ask. Once the question is answered the stored motivation
+* is still the analysis's, so the button runs a new one that can read the answer. Both halves are
+* rendered and one is hidden, so the workspace can switch them when a question is marked answered
+* in place (`syncIntentNote`).
 */
 function renderUnclearMotive(motivation, session, questions) {
 	const intent = questions.find((q) => q.category === "intent");
 	const answered = intent ? !isQuestionOutstanding(intent, new Set(session.answeredQuestionIds ?? [])) : false;
 	const hide = (yes) => yes ? " hidden" : "";
 	return `
-    <section class="pr-helper-why" aria-labelledby="pr-helper-why-heading"${intent ? ` data-intent-question="${escapeHtml$1(intent.id)}"` : ""}>
+    <section class="pr-helper-why pr-helper-why-unclear" aria-labelledby="pr-helper-why-heading"${intent ? ` data-intent-question="${escapeHtml$1(intent.id)}"` : ""}>
       <h4 class="pr-helper-overview-heading" id="pr-helper-why-heading">Why this change?</h4>
       <p class="pr-helper-why-status pr-helper-why-status-unclear" data-intent-open${hide(answered)}>${MOTIVATION_LABELS.unclear}</p>
-      <p class="pr-helper-why-status pr-helper-why-status-unclear" data-intent-answered${hide(!answered)}>Motivation was unclear when analysed</p>
-      <p class="pr-helper-why-summary">${renderInlineText(motivation.summary)}</p>
-      ${motivation.authorQuestion ? `<p class="pr-helper-why-question">${renderInlineText(motivation.authorQuestion)}</p>` : ""}
-      <button type="button" class="pr-helper-why-link" data-overview-nav="questions" data-intent-open${hide(answered)}>Asked in Questions, under Intent</button>
+      <p class="pr-helper-why-status pr-helper-why-status-answered" data-intent-answered${hide(!answered)}>Answer available</p>
+      <p class="pr-helper-why-summary" data-intent-open${hide(answered)}>${UNESTABLISHED_MOTIVATION}</p>
+      ${motivation.authorQuestion ? `<button type="button" class="pr-helper-why-ask" data-overview-nav="questions" data-intent-open${hide(answered)}>Ask the author</button>
+      <p class="pr-helper-why-question">\u201c${renderInlineText(motivation.authorQuestion)}\u201d</p>` : ""}
       <div class="pr-helper-why-answered" data-intent-answered${hide(!answered)}>
         <p class="pr-helper-why-answered-note">${INTENT_NOTE_COPY}</p>
         <button type="button" class="pr-helper-why-rerun" data-run-review-again>${RUN_REVIEW_AGAIN}</button>
@@ -1923,7 +1807,7 @@ function renderFileLocation(file, className, line) {
 	const suffix = line ? `:${line}` : "";
 	const dirLine = dir ? `<span class="pr-helper-path-dir">${escapeHtml$1(dir)}</span>` : "";
 	const lineAttr = line ? ` data-line="${line}"` : "";
-	return `<span class="${className}" data-file="${escapeHtml$1(file)}"${lineAttr} title="${escapeHtml$1(file + suffix)}">${dirLine}<span class="pr-helper-path-name">${escapeHtml$1(name + suffix)}</span></span>`;
+	return `<span class="${className}" data-file="${escapeHtml$1(file)}"${lineAttr}>${dirLine}<span class="pr-helper-path-name">${escapeHtml$1(name + suffix)}</span></span>`;
 }
 /**
 * The files a finding rests on that this change does not touch.
@@ -1945,7 +1829,7 @@ function renderRelatedFiles(files) {
 	return files.map((file) => {
 		const { dir, name } = splitFilePath(file);
 		const dirLine = dir ? `<span class="pr-helper-path-dir">${escapeHtml$1(dir)}</span>` : "";
-		return `<span class="prb-file-related" data-file="${escapeHtml$1(file)}" title="${escapeHtml$1(file)}">Related file: ${dirLine}<span class="pr-helper-path-name">${escapeHtml$1(name)}</span> · not changed</span>`;
+		return `<span class="prb-file-related" data-file="${escapeHtml$1(file)}">Related file: ${dirLine}<span class="pr-helper-path-name">${escapeHtml$1(name)}</span> · not changed</span>`;
 	}).join("");
 }
 function discussionDetails(session) {
@@ -1970,7 +1854,8 @@ function plural$1(count, word) {
 function renderExistingDiscussion(ref) {
 	if (!ref) return "";
 	const state = ref.status === "resolved" ? "thread since marked resolved" : "thread still unresolved";
-	const label = `<strong class="pr-helper-issue-discussion-lead">${escapeHtml$1(ref.relation === "related" ? "Related discussion" : "Already raised")}</strong>${escapeHtml$1(` by @${ref.raisedBy} · ${state}`)}`;
+	const lead = ref.relation === "related" ? "Related discussion" : "Already raised";
+	const label = `<strong class="pr-helper-issue-discussion-lead">${escapeHtml$1(lead)}</strong>${escapeHtml$1(` by @${ref.raisedBy} · ${state}`)}`;
 	const body = ref.url ? `<a class="pr-helper-issue-discussion-link" href="${escapeHtml$1(ref.url)}" data-discussion-url="${escapeHtml$1(ref.url)}" rel="noopener noreferrer">${label}</a>` : label;
 	return `<p class="pr-helper-issue-discussion pr-helper-issue-discussion-${escapeHtml$1(ref.status)}">${body}</p>`;
 }
@@ -2098,7 +1983,7 @@ function renderGuideSectionCard(section, counts, activeGuidedSectionId, status,
 		const focus = f.focus ? `<div class="pr-helper-guide-file-focus">${renderInlineText(f.focus)}</div>` : "";
 		const { dir, name } = splitFilePath(f.path);
 		const dirLine = dir ? `<span class="pr-helper-path-dir">${escapeHtml$1(dir)}</span>` : "";
-		return `<li class="pr-helper-guide-file" data-file="${escapeHtml$1(f.path)}" data-guide-section="${escapeHtml$1(section.id)}" title="${escapeHtml$1(f.path)}">${dirLine}<span class="pr-helper-path-name">${escapeHtml$1(name)}</span>${focus}</li>`;
+		return `<li class="pr-helper-guide-file" data-file="${escapeHtml$1(f.path)}" data-guide-section="${escapeHtml$1(section.id)}">${dirLine}<span class="pr-helper-path-name">${escapeHtml$1(name)}</span>${focus}</li>`;
 	}).join("") || "<li class=\"pr-helper-review-empty\">No files listed.</li>";
 	const checklist = section.checklist.map((item) => `<li>${renderInlineText(item)}</li>`).join("") || "<li>None</li>";
 	const [minMinutes, maxMinutes] = estimateMinutesForWeight(section.reviewWeight);
@@ -2647,13 +2532,17 @@ function renderQuestionsTab(session, investigatingId, options) {
 /**
 * The panel's contents are plain sentences, escaped here, because every caller is passing facts
 * rather than markup and one of them is a model's own words about a file it cited.
+*
+* `corrections` is the exception: what the harness changed in the model's output, as markup the
+* caller has already escaped (a removed path is set as code). It lives here, behind the control,
+* because it describes how the result was made rather than the result.
 */
-function renderAnalysisDetails(items) {
-	if (items.length === 0) return "";
+function renderAnalysisDetails(items, corrections = []) {
+	if (items.length === 0 && corrections.length === 0) return "";
 	return `<details class="prb-doc-provenance" data-analysis-details>
   <summary class="prb-doc-provenance-summary">Analysis details</summary>
   <div class="prb-doc-provenance-panel">
-    <ul>${items.map((item) => `<li>${escapeHtml$1(item)}</li>`).join("")}</ul>
+    ${items.length > 0 ? `<ul>${items.map((item) => `<li>${escapeHtml$1(item)}</li>`).join("")}</ul>` : ""}${corrections.length > 0 ? `<p class="prb-doc-provenance-heading">What the harness corrected</p><ul data-corrections>${corrections.map((note) => `<li>${note}</li>`).join("")}</ul>` : ""}
   </div>
 </details>`;
 }
@@ -3603,7 +3492,8 @@ function renderAttribution(attribution, processed) {
 	if (!attribution || processed.all.length === 0) return "";
 	const { findings, matched } = attribution;
 	if (findings === 0) return "";
-	return `<p class="prb-pr-attribution" data-matched="${matched}">${escapeHtml$1(matched === 0 ? `None of the ${findings} finding${findings === 1 ? "" : "s"} in this review matched a thread here. The comparison was made; nothing lined up.` : `${matched} of ${findings} finding${findings === 1 ? "" : "s"} in this review ${matched === 1 ? "is" : "are"} matched to a thread here. Each one says so on the Issues view.`)}</p>`;
+	const text = matched === 0 ? `None of the ${findings} finding${findings === 1 ? "" : "s"} in this review matched a thread here. The comparison was made; nothing lined up.` : `${matched} of ${findings} finding${findings === 1 ? "" : "s"} in this review ${matched === 1 ? "is" : "are"} matched to a thread here. Each one says so on the Issues view.`;
+	return `<p class="prb-pr-attribution" data-matched="${matched}">${escapeHtml$1(text)}</p>`;
 }
 /**
 * Said plainly, because the alternative is a reviewer assuming their findings were checked
@@ -3688,10 +3578,11 @@ function renderDecisions(processed) {
 *
 * `renderMarkdown` escapes before it parses, so nothing in a comment can become markup this page
 * did not emit, and its href allow-list means a link is only ever a link. It was written for model
-* output injected into github.com, which is a harsher setting than this one.
+* output injected into github.com, which is a harsher setting than this one. HTML comments are
+* hidden, as GitHub hides them.
 */
 function renderBody(body) {
-	return `<div class="prb-pr-comment-body prb-pr-md">${renderMarkdown(body)}</div>`;
+	return `<div class="prb-pr-comment-body prb-pr-md">${renderMarkdown(body, { hideHtmlComments: true })}</div>`;
 }
 function renderBucket(title, note, threads) {
 	if (threads.length === 0) return "";
@@ -4435,8 +4326,9 @@ function renderEvidence(recommendation, rest) {
   </div>` : ""
 	].filter(Boolean);
 	if (body.length === 0) return "";
+	const files = rest.length > 0 ? ` · ${rest.length} more ${rest.length === 1 ? "file" : "files"}` : "";
 	return `<details class="prb-kiss-evidence" data-kiss-evidence>
-    <summary class="prb-kiss-evidence-title">Evidence &amp; trade-offs${escapeHtml$1(rest.length > 0 ? ` · ${rest.length} more ${rest.length === 1 ? "file" : "files"}` : "")}</summary>
+    <summary class="prb-kiss-evidence-title">Evidence &amp; trade-offs${escapeHtml$1(files)}</summary>
     ${body.join("\n    ")}
   </details>`;
 }
@@ -4502,7 +4394,7 @@ function renderLocationList(locations) {
 	return `<ul class="prb-kiss-locations-list">${locations.map((location) => {
 		const { dir, name } = splitFilePath(location.path);
 		const suffix = location.line ? `:${location.line}` : "";
-		return `<li><button type="button" class="prb-kiss-location" data-file="${escapeHtml$1(location.path)}"${location.line ? ` data-line="${location.line}"` : ""}${location.primary ? " data-primary" : ""} aria-expanded="false" title="${escapeHtml$1(location.path + suffix)}">
+		return `<li><button type="button" class="prb-kiss-location" data-file="${escapeHtml$1(location.path)}"${location.line ? ` data-line="${location.line}"` : ""}${location.primary ? " data-primary" : ""} aria-expanded="false">
       ${dir ? `<span class="pr-helper-path-dir">${escapeHtml$1(dir)}</span>` : ""}<span class="prb-kiss-location-row"><span class="pr-helper-path-name">${escapeHtml$1(name + suffix)}</span>${location.unchanged ? "<span class=\"prb-kiss-location-unchanged\">not changed</span>" : ""}</span>
     </button></li>`;
 	}).join("")}</ul>`;
@@ -5296,7 +5188,6 @@ function renderReviewDoc(view, sections) {
 	const result = session.result;
 	const progress = showsProgress(view);
 	return `<div class="pr-helper-sidebar prb-pane">
-      ${result ? renderDisclosure(view) : ""}
       ${result ? renderStandardViews(view, sections) : ""}
       ${progress ? `<section class="prb-view" data-view="progress" data-progress-pane data-job-phase="${escapeHtml$1(view.job.phase)}" role="status" aria-live="polite">${renderProgressView(view.job, view.changeSet.data.files.length)}</section>` : ""}
       ${renderFilesView(view, sections)}
@@ -5379,7 +5270,7 @@ function renderImplementationEmpty() {
 function renderStandardViews(view, sections) {
 	const { session } = view;
 	const result = session.result;
-	return `<section class="prb-view" data-view="overview">${renderDocHead("Review overview", `${renderAnalysisDetails(analysisDetailItems(session))}${renderFocusToggle("overview", "Overview")}`)}${renderOverviewTab(session, "", {
+	return `<section class="prb-view" data-view="overview">${renderDocHead("Review overview", `${renderAnalysisDetails(analysisDetailItems(session), correctionNotes(view))}${renderFocusToggle("overview", "Overview")}`)}${renderOverviewTab(session, "", {
 		diagrams: true,
 		detailsInHeader: true
 	})}</section>
@@ -5438,7 +5329,8 @@ function renderStandardViews(view, sections) {
 */
 function renderAttributionAge(view) {
 	if (!view.prContext) return "";
-	return `<p class="prb-doc-note">Any &ldquo;already raised&rdquo; note below describes the pull request as it stood ${escapeHtml$1(relativeTime$1(view.prContext.prContext.discussion.fetchedAt))}, when its discussion was fetched. That includes whether each thread was still open. If someone has replied or resolved a thread since then, this page has no way of knowing. Use <button type="button" class="prb-doc-note-link" data-view-nav="prcontext">Re-check the conversation</button> on the PR context view to bring it up to date.</p>`;
+	const imported = relativeTime$1(view.prContext.prContext.discussion.fetchedAt);
+	return `<p class="prb-doc-note">Any &ldquo;already raised&rdquo; note below describes the pull request as it stood ${escapeHtml$1(imported)}, when its discussion was fetched. That includes whether each thread was still open. If someone has replied or resolved a thread since then, this page has no way of knowing. Use <button type="button" class="prb-doc-note-link" data-view-nav="prcontext">Re-check the conversation</button> on the PR context view to bring it up to date.</p>`;
 }
 /**
 * The link out to this file on the pull request.
@@ -5993,14 +5885,17 @@ function railLink(view, icon, label, badge, note, alsoCurrentOn) {
   </button>`;
 }
 /**
-* What the harness corrected, stated above the review rather than buried in a details element.
-* A guide quietly repaired is a guide trusted more than it deserves.
+* What the harness corrected, as notes for Analysis details on the Overview.
+*
+* Behind the control rather than above the review: the Overview explains the result, and these
+* explain how it was made. Each correction's effect on the result is already in the result itself
+* (a removed reference is gone, a refused motive shows as unclear), so the reader acting on the
+* review loses nothing by not meeting the mechanism first.
 *
 * Corrections only. There is deliberately nothing here about how much of the change travelled
-* inline: that is provenance, it belongs in Analysis details on the Overview, and putting it in a
-* box under this heading would restate a limit that no longer exists as though it did.
+* inline: that is provenance, which Analysis details already states as a fact of its own.
 */
-function renderDisclosure(view) {
+function correctionNotes(view) {
 	const { audit } = view;
 	const notes = [];
 	if (audit.unknownPaths.length > 0) {
@@ -6012,26 +5907,28 @@ function renderDisclosure(view) {
 	} else if (audit.relocatedFindings > 0) notes.push(`${audit.relocatedFindings} finding${audit.relocatedFindings === 1 ? "" : "s"} pointed at a file that is not in this change. The finding is kept; its location is not.`);
 	const motivation = motivationNote(audit.motivationCorrection);
 	if (motivation) notes.push(motivation);
-	if (notes.length === 0) return "";
-	return `<aside class="prb-disclosure">
-  <strong>What the harness corrected</strong>
-  <ul>${notes.map((note) => `<li>${note}</li>`).join("")}</ul>
-</aside>`;
+	return notes;
 }
 /**
 * A refused motivation, described by what kind of claim it was and never by its wording: that
-* wording is exactly what the check decided the reader should not be shown.
+* wording, and any rejected quote, is exactly what the check decided the reader should not see.
 *
-* An unclear motive with no question is not mentioned. Withholding it costs the reader nothing,
-* since it said nothing they could act on.
+* An unclear motive that asked nothing is not mentioned: it claimed nothing, and the question it
+* now carries is in plain view under Intent.
 */
 function motivationNote(correction) {
 	if (!correction || correction.reason === "no-question") return null;
-	return `The review gave ${{
-		"missing-quote": "that the author had stated why this change was made, without quoting where",
-		"unverified-quote": "that the author had stated why this change was made, quoting words that are not in the description",
-		"no-evidence": "a likely reason for this change, without evidence for it"
-	}[correction.reason]}, ${correction.outcome === "downgraded" ? "so the reason is shown as unclear, with its question to the author under Intent" : "so no reason is shown"}.`;
+	const claim = {
+		"missing-quote": "The analysis attributed a reason for this change to the author, but did not quote where the PR description gives it.",
+		"unverified-quote": "The analysis attributed a reason for this change to the author, but the words it quoted could not be confirmed as the author's own in the PR description.",
+		"no-evidence": "The analysis suggested a likely reason for this change, but gave no evidence for it."
+	}[correction.reason];
+	const outcome = {
+		downgraded: "The reason is shown as unclear, with a question for the author.",
+		withheld: "No reason is shown."
+	}[correction.outcome];
+	const evidence = correction.discardedEvidence?.length ? ` Evidence it gave, not used: ${correction.discardedEvidence.map((item) => `${escapeHtml$1(item.kind)}: ${escapeHtml$1(item.detail)}`).join("; ")}.` : "";
+	return [claim, outcome].filter(Boolean).join(" ") + evidence;
 }
 /**
 * Every changed file, split by whether the guide sends the reviewer there.
@@ -6115,7 +6012,7 @@ function referencedRow(view, file, titles, refs) {
 	const counts = fileCounts(refs);
 	const named = file.sectionIds.map((id) => titles.get(id)).filter((title) => Boolean(title));
 	const by = named.length > 0 ? `<span class="prb-file-referenced-by">Referenced by: ${escapeHtml$1(named.join(", "))}</span>` : "";
-	return `<div class="prb-file prb-file-expandable prb-file-referenced" data-file="${escapeHtml$1(file.path)}" role="button" tabindex="0" aria-expanded="false" data-search="${escapeHtml$1(file.path.toLowerCase())}" title="${escapeHtml$1(file.path)}">
+	return `<div class="prb-file prb-file-expandable prb-file-referenced" data-file="${escapeHtml$1(file.path)}" role="button" tabindex="0" aria-expanded="false" data-search="${escapeHtml$1(file.path.toLowerCase())}">
     <span class="prb-file-path">
       ${dir ? `<span class="pr-helper-path-dir">${escapeHtml$1(dir)}</span>` : ""}
       <span class="pr-helper-path-name">${escapeHtml$1(name)}</span>
@@ -6181,7 +6078,7 @@ function fileRow(view, file, refs) {
 	const link = readable ? null : prFileLink(view, file.name);
 	const openPr = link ? `<a class="prb-file-open prb-file-pr" href="${escapeHtml$1(link.href)}" target="_blank" rel="noopener noreferrer" title="${escapeHtml$1(link.label)}">Pull request</a>` : "";
 	const attributes = readable ? ` data-file="${escapeHtml$1(file.name)}" role="button" tabindex="0" aria-expanded="false"` : "";
-	return `<div class="prb-file${readable ? " prb-file-expandable" : ""}"${attributes} data-search="${escapeHtml$1(file.name.toLowerCase())}" title="${escapeHtml$1(file.name)}">
+	return `<div class="prb-file${readable ? " prb-file-expandable" : ""}"${attributes} data-search="${escapeHtml$1(file.name.toLowerCase())}">
     <span class="prb-file-status" data-status="${escapeHtml$1(file.status)}">${escapeHtml$1(file.status)}</span>
     <span class="prb-file-path">
       ${dir ? `<span class="pr-helper-path-dir">${escapeHtml$1(dir)}</span>` : ""}
@@ -7778,7 +7675,8 @@ function renderHistoryRow(review, number, latest, now) {
 function renderAnalysisState(analysis) {
 	if (analysis === void 0) return "";
 	if (analysis.phase === "failed") return analysis.stopped ? "<span class=\"prb-card-analysis\" data-analysis=\"stopped\">Analysis stopped</span>" : "<span class=\"prb-card-analysis\" data-analysis=\"failed\">Analysis failed</span>";
-	return `<span class="prb-card-analysis" data-analysis="running"><i></i>${escapeHtml$1(phasesFor({ purpose: analysis.purpose })[analysis.phase].doing)}</span>`;
+	const names = phasesFor({ purpose: analysis.purpose });
+	return `<span class="prb-card-analysis" data-analysis="running"><i></i>${escapeHtml$1(names[analysis.phase].doing)}</span>`;
 }
 /**
 * What this review still has open.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "prreviewbuddy",
-  "version": "0.55.1",
+  "version": "0.55.6",
   "description": "Review a branch or pull request in an isolated checkout, from your terminal, with the coding agent you already have installed.",
   "license": "UNLICENSED",
   "homepage": "https://prreviewbuddy.com",

package/static/shell.css

@@ -728,7 +728,6 @@ a.prb-author:hover .prb-author-name { color: var(--text-primary); text-decoratio
   color: var(--pane-ink-muted);
   line-height: var(--lh-body);
 }
-.prb-pr-empty { max-width: 62ch; }
 .prb-pr-provenance {
   display: flex;
   flex-wrap: wrap;
@@ -1273,26 +1272,16 @@ a.prb-rail-item { text-decoration: none; }
 .prb-doc-provenance-panel ul { margin: 0; padding-left: 16px; }
 .prb-doc-provenance-panel li { margin-bottom: 5px; }
 .prb-doc-provenance-panel li:last-child { margin-bottom: 0; }
+.prb-doc-provenance-heading {
+  margin: 12px 0 6px;
+  font-weight: 600;
+  color: var(--pane-ink);
+}
+.prb-doc-provenance-heading:first-child { margin-top: 0; }
 
 /* A pane is present in the DOM at all times so switching costs no rebuild; only one is shown. */
 .prb-view[hidden] { display: none; }
 
-/* ── Disclosure strip ──
-   Anything the harness corrected or could not confirm. Amber, above the review, never buried:
-   a guide quietly repaired is a guide trusted more than it deserves. */
-.prb-disclosure {
-  margin: 0 0 20px;
-  padding: 12px 14px;
-  border-radius: 8px;
-  border: 1px solid rgba(154, 103, 0, 0.25);
-  background: rgba(240, 178, 50, 0.12);
-  color: var(--pane-warning);
-  font-size: var(--t-meta);
-  line-height: var(--lh-body);
-}
-.prb-disclosure strong { font-weight: 600; }
-.prb-disclosure ul { margin: 6px 0 0; padding-left: 18px; }
-
 /* ── File list ── */
 .prb-file-group { margin-bottom: 26px; }
 .prb-file-group h4 {
@@ -3890,7 +3879,6 @@ a.prb-rail-item { text-decoration: none; }
    Says what the view is for and offers the one action that spends a run. Not a dead end, and not a
    button that pretends to be free. */
 .prb-map-empty {
-  max-width: 560px;
   margin: 8px 0 20px;
   padding: 18px 20px;
   border: 1px dashed rgba(15, 23, 42, 0.2);