prreviewbuddy 0.75.0 → 0.76.9

package/dist/main.js

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { $ as MAX_WORKSPACE_PORT, A as relativeTime, An as describeMovement, B as readIndexToken, Bn as allJobs, Ci as writeAgentPreference, Cn as MANAGED_ROOT, Di as agentById, Dn as readMarker, Ei as AGENT_IDS, F as runBatch, G as reviewsUrl, Kn as PHASES, Mn as findExistingReview, On as removeWorktree, Q as DEFAULT_WORKSPACE_PORT, R as queueForReview, Rn as forgeResolver, Si as readAgentPreference, Ti as STORE_ROOT, U as ensureServer, V as bootstrapUrl, Wn as loadJob, X as workspaceUrl, Xn as purposeOf, Y as stopServer, Yn as progressSteps, Zn as clearClaim, _ as liveUpdateFor, _r as summarise, _t as telemetryUploadConsent, a as wasBlocked, b as recordKissRun, bi as resolveModel, bt as PACKAGE_NAME, cr as loadWorkspace, ct as reanalyseReview, d as mergeUpdateInto, di as WITHHELD_REASON, dr as positionInLineage, dt as append, er as followedRefName, et as MIN_WORKSPACE_PORT, f as updateReview, fi as kissOutcome, g as UpdateAlreadyRunningError, gr as reviewedRepositories, gt as recordTelemetryUploadConsent, ht as deliver, i as liveJobsFor, it as writePortPreference, j as batchIdentity, jn as describeSubject, ki as detectAgents, lr as matchingWorkspaceIds, mr as recentWorkspaces, mt as stamp, n as deleteReview$1, ot as prepareReview, pi as resolveTarget, pt as record$1, rt as statedWorkspacePort, st as settleReview, tt as parseWorkspacePort, ur as mutateWorkspace, ut as startJob, v as runningJobs, vi as git, vt as telemetryUploadEnabled, wi as CONFIG_PATH, xn as modelHelpLines, yi as resolveAgent, yt as BUILD_VERSION } from "./delete_review-BWKGCqwI.js";
+import { $n as PHASES, Ai as readAgentPreference, An as MANAGED_ROOT, B as ensureServer, Cr as reviewedRepositories, D as relativeTime, Di as resolveAgent, Ei as git, F as queueForReview, Fi as agentById, Fn as removeWorktree, Gn as forgeResolver, H as reviewsUrl, K as stopServer, L as readIndexToken, Li as detectAgents, Ln as describeMovement, M as runBatch, Mi as CONFIG_PATH, Ni as STORE_ROOT, O as batchIdentity, Oi as resolveModel, On as modelHelpLines, Pi as AGENT_IDS, Pn as readMarker, Q as parseWorkspacePort, R as bootstrapUrl, Rn as describeSubject, X as MAX_WORKSPACE_PORT, Y as DEFAULT_WORKSPACE_PORT, Z as MIN_WORKSPACE_PORT, Zn as loadJob, _ as recordKissRun, _r as mutateWorkspace, _t as PACKAGE_NAME, a as wasBlocked, at as reanalyseReview, bi as kissOutcome, ct as append, d as mergeUpdateInto, dt as stamp, et as statedWorkspacePort, f as updateReview, ft as deliver, g as runningJobs, gr as matchingWorkspaceIds, gt as BUILD_VERSION, h as liveUpdateFor, hr as loadWorkspace, ht as telemetryUploadEnabled, i as liveJobsFor, ir as clearClaim, it as settleReview, ji as writeAgentPreference, m as UpdateAlreadyRunningError, mt as telemetryUploadConsent, n as deleteReview$1, nr as progressSteps, pt as recordTelemetryUploadConsent, q as workspaceUrl, qn as allJobs, rr as purposeOf, rt as prepareReview, sr as followedRefName, st as startJob, tt as writePortPreference, ut as record$1, vr as positionInLineage, wr as summarise, xi as resolveTarget, xr as recentWorkspaces, yi as WITHHELD_REASON, zn as findExistingReview } from "./delete_review-BX3lu5Q7.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";

package/dist/server.js

@@ -1,4 +1,4 @@
-import { $ as MAX_WORKSPACE_PORT, $n as decideKissRecommendation, $r as findingIn, $t as markReadForReview, A as relativeTime$1, Ai as AgentCancelledError, Ar as sectionOf, At as partFiles, B as readIndexToken, Bn as allJobs, Br as questionsOutstanding, Bt as isArchitectureId, C as track, Ci as writeAgentPreference, Cn as MANAGED_ROOT, Cr as activityLabel, Ct as mapRuntimes, D as endRun, Di as agentById, Dn as readMarker, Dr as lifecyclePill, Dt as ARCHITECTURE_DIR, E as completeRun, Ei as AGENT_IDS, En as isWorktreeHeld, Er as lifecycleItemLabel, Et as describeRuntimes, F as runBatch, Fn as followedBranch, Fr as readQueue, Ft as trackedFiles, Gn as saveJob, Gr as explanationOf, Gt as categoryOf, H as clearServerRecordOwnedBy, Hn as fail, Hr as processDiscussion, Ht as announceJob, I as activateQueued, In as reviewedShaOf, Ir as doneVerb, It as ownersOf, J as stopChild, Jn as phasesFor, Jr as ANALYSIS_ONLY, Jt as createPrUpdate, K as serverEntryExists, Kr as isExplanationAction, Kt as createArchitectureNotification, L as compactQueue, Ln as fetchHead, Lr as isIssueOutstanding, Lt as QUOTE_LIMITS, M as extractPullRequestLinks, Mn as findExistingReview, Mr as queueAddMany, Mt as repositoryKey, N as parsePullRequestLink, Nn as checkCodeFreshness, Nr as queueRemove, Nt as storeArchitecture, O as reportRun, Oi as agentFor$1, On as removeWorktree, Or as offersLifecycle, Ot as checkArchitecture, P as unsupportedReason, Pn as checkReview, Pr as queueStamp, Pt as storedScopes, Q as DEFAULT_WORKSPACE_PORT, Qn as isClaimed, Qr as findingAssessment, Qt as markRead, Rn as forgeResolver, Rr as isQuestionOutstanding, Rt as REPOSITORY_SCOPE, S as isDriving, Si as readAgentPreference, Sn as ReviewBeingDeletedError, Sr as activityAttributes, St as waitingHistory, T as beginRun, Ti as STORE_ROOT, Tn as holdWorktree, Tr as lifecycleChip, Tt as RUNTIME_WORDS, Un as isTerminal, Ur as recommendationsOf, Ut as branchOf, Vn as endedAt, Vr as reviewerDispositions, Vt as writeJsonAtomically, W as readServerRecord, Wn as loadJob, Wr as explanationActionOf, Wt as subjectOf, Xn as purposeOf, Xr as assessmentLookup, Xt as listNotifications, Yn as progressSteps, Yr as analysisRefOf, Yt as eventName, Z as writeServerRecord, Zr as currentGradeIn, Zt as markAllRead, _ as liveUpdateFor, _i as resolveBase, _n as CODE_BLOCK_CLASS, _t as telemetryUploadConsent, a as wasBlocked, ai as wasReassessed, an as retireNotification, ar as kissDecisionsOf, at as failureOf$1, bi as resolveModel, bn as renderInlineText, br as workspaceStamps, c as RefreshUnavailableError, ci as extractJsonText, cn as unreadCount$1, cr as loadWorkspace, ct as reanalyseReview, d as mergeUpdateInto, di as WITHHELD_REASON, dn as EXPLAIN_SIMPLY_PROMPT, ei as gradeScale, en as moveIds, er as followedRefName, et as MIN_WORKSPACE_PORT, f as updateReview, fi as kissOutcome, fn as UNESTABLISHED_MOTIVATION, fr as presentedKissRunOf, ft as readEvents, g as UpdateAlreadyRunningError, gi as isCommitReview, gn as renderMarkdown, gt as recordTelemetryUploadConsent, h as askCheckout, hi as DetachedHeadError, hn as GITHUB_MARKDOWN, hr as reviewedCommit, i as liveJobsFor, ii as sourcePhrase, in as removeNotification, ir as kissDecisionRefusal, it as writePortPreference, j as batchIdentity, ji as AgentUnavailableError, jr as queueAdd, jt as readArchitecture, k as stopRun, ki as detectAgents, kn as requestJobStop, kr as parseLifecycle, kt as loadArchitecture, l as cancelUpdate, li as objectWithAnyOf, ln as withUsageRecorded, lt as runJob, m as isUnchanged, mi as originRemote, mn as hasAuthorContent, mr as recentWorkspaces, n as deleteReview, ni as promptGrade, nn as readNotification, nr as indexKeptIds, nt as resolveWorkspacePort, o as discardJob, oi as followUpFindingId, on as toneOf, or as latestKissRun, ot as prepareReview, p as refreshPrContext, pi as resolveTarget, pn as authorPreview, pr as previousKissResult, pt as record, q as spawnServer, qn as isLiveJob, qr as withExplanation, qt as createPrState, r as lineageIds, ri as reassessIn, rn as readStateOf, rr as indexReviewGroups, rt as statedWorkspacePort, s as FETCH_TIMEOUT_MS, si as asString, sn as unreadAround, sr as lineagePosition, st as settleReview, t as deleteLineage, ti as isGradeFor, tn as pruneNotifications, tr as groupByLineage, tt as parseWorkspacePort, u as cancellableHere, ui as describeAuthorship, un as existingPaths, ur as mutateWorkspace, vi as git, vn as CODE_COPY_CLASS, vr as touchWorkspace, w as RUN_KINDS, wn as createWorktree, wr as isLifecycle, wt as recordRuntimeBestEffort, x as startKissJob, xi as clearAgentPreference, xr as LIFECYCLES, xt as feedbackUrl, y as agentEnvOf, yi as resolveAgent, yn as escapeHtml$1, yr as workspaceRevision, yt as BUILD_VERSION, z as settleQueue, zn as fillFileUrlTemplate, zr as issuesOutstanding, zt as ancestorsOf } from "./delete_review-BWKGCqwI.js";
+import { $ as resolveWorkspacePort, $r as explanationOf, $t as readNotification, A as parsePullRequestLink, Ai as readAgentPreference, An as MANAGED_ROOT, Ar as activityLabel, At as storeArchitecture, Bn as checkCodeFreshness, Br as queueRemove, Bt as branchOf, C as completeRun, Ci as DetachedHeadError, Cn as renderMarkdown, Ct as describeRuntimes, D as relativeTime$1, Di as resolveAgent, Dn as renderInlineText, Dr as workspaceStamps, Dt as partFiles, E as stopRun, Ei as git, En as escapeHtml$1, Er as workspaceRevision, Et as loadArchitecture, Fi as agentById, Fn as removeWorktree, Fr as offersLifecycle, Ft as REPOSITORY_SCOPE, G as stopChild, Gn as forgeResolver, Gr as isQuestionOutstanding, Gt as createPrUpdate, Hn as followedBranch, Hr as readQueue, Ht as categoryOf, I as settleQueue, Ii as agentFor$1, In as requestJobStop, Ir as parseLifecycle, It as ancestorsOf, J as writeServerRecord, Jn as endedAt, Jr as resolvedByConversation, Jt as markAllRead, Kn as fillFileUrlTemplate, Kr as issuesOutstanding, Kt as eventName, L as readIndexToken, Li as detectAgents, Lr as sectionOf, Lt as isArchitectureId, M as runBatch, Mn as holdWorktree, Mr as lifecycleChip, Mt as trackedFiles, N as activateQueued, Ni as STORE_ROOT, Nn as isWorktreeHeld, Nr as lifecycleItemLabel, Nt as ownersOf, O as batchIdentity, Oi as resolveModel, Or as LIFECYCLES, Ot as readArchitecture, P as compactQueue, Pi as AGENT_IDS, Pn as readMarker, Pr as lifecyclePill, Pt as QUOTE_LIMITS, Q as parseWorkspacePort, Qn as saveJob, Qr as explanationActionOf, Qt as pruneNotifications, Ri as AgentCancelledError, Rr as queueAdd, Rt as writeJsonAtomically, S as beginRun, Si as originRemote, Sn as GITHUB_MARKDOWN, Sr as reviewedCommit, St as RUNTIME_WORDS, T as reportRun, Ti as resolveBase, Tn as CODE_COPY_CLASS, Tr as touchWorkspace, Tt as checkArchitecture, U as serverEntryExists, Un as reviewedShaOf, Ur as doneVerb, Ut as createArchitectureNotification, V as readServerRecord, Vn as checkReview, Vr as queueStamp, Vt as subjectOf, W as spawnServer, Wn as fetchHead, Wr as isIssueOutstanding, Wt as createPrState, X as MAX_WORKSPACE_PORT, Xn as isTerminal, Xr as processDiscussion, Xt as markReadForReview, Y as DEFAULT_WORKSPACE_PORT, Yn as fail, Yr as reviewerDispositions, Yt as markRead, Z as MIN_WORKSPACE_PORT, Zn as loadJob, Zr as recommendationsOf, Zt as moveIds, _i as objectWithAnyOf, _n as EXPLAIN_SIMPLY_PROMPT, _r as mutateWorkspace, a as wasBlocked, ai as currentGradeIn, an as unreadCount$1, ar as isClaimed, at as reanalyseReview, b as track, bi as kissOutcome, bn as authorPreview, br as previousKissResult, bt as mapRuntimes, c as RefreshUnavailableError, ci as gradeScale, cn as ASK_REREAD_CEILING_MS, cr as groupByLineage, d as mergeUpdateInto, di as reassessIn, dn as saveRefresh, dr as kissDecisionRefusal, ei as isExplanationAction, en as readStateOf, er as isLiveJob, et as statedWorkspacePort, f as updateReview, fi as sourcePhrase, fn as storedConversation, fr as kissDecisionsOf, gi as extractJsonText, gn as agentEnvOf, gt as BUILD_VERSION, h as liveUpdateFor, hi as asString, hn as isUnchanged, hr as loadWorkspace, i as liveJobsFor, ii as assessmentLookup, in as unreadAround, it as settleReview, j as unsupportedReason, ji as writeAgentPreference, jn as createWorktree, jr as isLifecycle, jt as storedScopes, k as extractPullRequestLinks, ki as clearAgentPreference, kn as ReviewBeingDeletedError, kr as activityAttributes, kt as repositoryKey, l as cancelUpdate, li as isGradeFor, ln as adoptLegacyBaseline, lr as indexKeptIds, lt as readEvents, m as UpdateAlreadyRunningError, mi as followUpFindingId, mn as refreshPrContext, mr as lineagePosition, mt as telemetryUploadConsent, n as deleteReview, ni as ANALYSIS_ONLY, nn as retireNotification, nr as progressSteps, nt as failureOf$1, o as discardJob, oi as findingAssessment, on as withUsageRecorded, or as decideKissRecommendation, ot as runJob, p as askCheckout, pi as wasReassessed, pn as unforeseenFailure, pr as latestKissRun, pt as recordTelemetryUploadConsent, qn as allJobs, qr as questionsOutstanding, qt as listNotifications, r as lineageIds, ri as analysisRefOf, rn as toneOf, rr as purposeOf, rt as prepareReview, s as FETCH_TIMEOUT_MS, si as findingIn, sn as existingPaths, sr as followedRefName, t as deleteLineage, ti as withExplanation, tn as removeNotification, tr as phasesFor, tt as writePortPreference, u as cancellableHere, ui as promptGrade, un as prepareConversationWithin, ur as indexReviewGroups, ut as record, v as startKissJob, vi as describeAuthorship, vn as chatConversationSection, vt as feedbackUrl, w as endRun, wi as isCommitReview, wn as CODE_BLOCK_CLASS, wt as ARCHITECTURE_DIR, x as RUN_KINDS, xi as resolveTarget, xn as hasAuthorContent, xr as recentWorkspaces, xt as recordRuntimeBestEffort, y as isDriving, yi as WITHHELD_REASON, yn as UNESTABLISHED_MOTIVATION, yr as presentedKissRunOf, yt as waitingHistory, z as clearServerRecordOwnedBy, zi as AgentUnavailableError, zn as findExistingReview, zr as queueAddMany, zt as announceJob } from "./delete_review-BX3lu5Q7.js";
 import { basename, isAbsolute, join, relative, resolve } from "node:path";
 import { execFile } from "node:child_process";
 import { existsSync, mkdirSync, readFileSync, readdirSync, realpathSync, rmSync, statSync, writeFileSync } from "node:fs";
@@ -1210,13 +1210,43 @@ function importanceTip(importance) {
 * tell those apart from the badge — so the copy commits to the weaker of the two readings rather
 * than claiming a judgement that may not have been made.
 */
-function findingStatusTip(status) {
+function findingStatusTip(status, against = "commits") {
 	return {
-		resolved: "Resolved — the new commits appear to address this finding. Worth a glance to confirm before you close it out.",
-		"still-open": "Still open — the new commits do not address this finding. Findings the analysis did not comment on are shown this way too, so this never disappears on its own.",
-		affected: "Affected — the code this finding concerns has changed, but not in a way that settles it. Re-read this one.",
-		new: "New — introduced by the changes pushed since your previous analysis."
-	}[status];
+		commits: {
+			resolved: "Resolved — the new commits appear to address this finding. Worth a glance to confirm before you close it out.",
+			"still-open": "Still open — the new commits do not address this finding. Findings the analysis did not comment on are shown this way too, so this never disappears on its own.",
+			affected: "Affected — the code this finding concerns has changed, but not in a way that settles it. Re-read this one.",
+			new: "New — introduced by the changes pushed since your previous analysis."
+		},
+		conversation: {
+			resolved: "Resolved — a comment on the pull request shows this finding no longer applies. Worth a glance to confirm before you close it out.",
+			"still-open": "Still open — nothing said on the pull request settles this finding. Findings the analysis did not comment on are shown this way too, so this never disappears on its own.",
+			affected: "Affected — something said on the pull request bears on this finding without settling it. Re-read this one.",
+			new: "New — raised in the pull request conversation since your previous analysis, and confirmed in the code."
+		},
+		"commits-and-conversation": {
+			resolved: "Resolved — the new commits or the pull request conversation appear to settle this finding. Worth a glance to confirm before you close it out.",
+			"still-open": "Still open — neither the new commits nor the pull request conversation settle this finding. Findings the analysis did not comment on are shown this way too, so this never disappears on its own.",
+			affected: "Affected — the new commits or the pull request conversation bear on this finding without settling it. Re-read this one.",
+			new: "New — found in the commits or the conversation since your previous analysis."
+		}
+	}[against][status];
+}
+/** What an update's note on a finding came after, as Copy heads it. */
+function reconciledAfter(against = "commits") {
+	return {
+		commits: "After the new commits",
+		conversation: "After the pull request conversation",
+		"commits-and-conversation": "After the new commits and comments"
+	}[against];
+}
+/** Where an update's answer to a question came from, as its heading. */
+function reconciledAnswerHeading(against = "commits") {
+	return {
+		commits: "Answered by the new changes",
+		conversation: "Answered by the pull request conversation",
+		"commits-and-conversation": "Answered by the new changes or comments"
+	}[against];
 }
 /** The reviewer-facing label for a status. Kept beside the tooltips so the two cannot drift. */
 function findingStatusLabel(status) {
@@ -1234,12 +1264,24 @@ function findingStatusLabel(status) {
 * for a question the reassessment never mentioned. The copy commits to the weaker reading, because
 * the reviewer must not read silence as an answer.
 */
-function questionStatusTip(status) {
+function questionStatusTip(status, against = "commits") {
 	return {
-		answered: "Answered — the new commits settle this, and the answer is below. Worth a glance to confirm before you drop it.",
-		"still-open": "Still open — the new commits do not settle this. Questions the reassessment did not comment on are shown this way too, so this never disappears on its own.",
-		affected: "Affected — the code this question is about has changed without answering it. Read the question again before you ask it."
-	}[status];
+		commits: {
+			answered: "Answered — the new commits settle this, and the answer is below. Worth a glance to confirm before you drop it.",
+			"still-open": "Still open — the new commits do not settle this. Questions the reassessment did not comment on are shown this way too, so this never disappears on its own.",
+			affected: "Affected — the code this question is about has changed without answering it. Read the question again before you ask it."
+		},
+		conversation: {
+			answered: "Answered — the pull request conversation settles this, and the answer is below. Worth a glance to confirm before you drop it.",
+			"still-open": "Still open — nothing said on the pull request settles this. Questions the reassessment did not comment on are shown this way too, so this never disappears on its own.",
+			affected: "Affected — something said on the pull request bears on this question without answering it. Read the question again before you ask it."
+		},
+		"commits-and-conversation": {
+			answered: "Answered — the new commits or the pull request conversation settle this, and the answer is below. Worth a glance to confirm before you drop it.",
+			"still-open": "Still open — neither the new commits nor the pull request conversation settle this. Questions the reassessment did not comment on are shown this way too, so this never disappears on its own.",
+			affected: "Affected — the new commits or the pull request conversation bear on this question without answering it. Read the question again before you ask it."
+		}
+	}[against][status];
 }
 /** The reviewer-facing label for a question status. Kept beside the tooltip so the two cannot drift. */
 function questionStatusLabel(status) {
@@ -1301,6 +1343,102 @@ function pillAttrs(classNames, tip) {
 	return ` class="${escapeHtml$1(classNames)} pr-helper-tip" data-tip="${escapeHtml$1(tip)}"`;
 }
 //#endregion
+//#region ../../packages/review-harness/src/domain/utils/relative_time.ts
+function formatRelativeTime(ts, now = Date.now()) {
+	const diffMs = Math.max(0, now - ts);
+	const minute = 6e4;
+	const hour = 60 * minute;
+	const day = 24 * hour;
+	if (diffMs < minute) return "Just now";
+	if (diffMs < hour) {
+		const mins = Math.floor(diffMs / minute);
+		return `${mins} min${mins === 1 ? "" : "s"} ago`;
+	}
+	if (diffMs < day) {
+		const hours = Math.floor(diffMs / hour);
+		return `${hours} hour${hours === 1 ? "" : "s"} ago`;
+	}
+	const days = Math.floor(diffMs / day);
+	return `${days} day${days === 1 ? "" : "s"} ago`;
+}
+//#endregion
+//#region ../../packages/review-harness/src/domain/ui/components/status_history.ts
+/**
+* What a step did, by where it came from. A settled finding's three outcomes are kept apart
+* because they are different statements: reopened says the finding is valid again, marked for
+* reassessment says nobody could tell, and still resolved says the resolution survived.
+*/
+function statusStepHeadline(from, step) {
+	if (step.carried) return `${findingStatusLabel(step.status)}, before history was kept`;
+	if (from === "resolved") {
+		if (step.status === "still-open") return "Reopened";
+		if (step.status === "affected") return "Marked for reassessment";
+		if (step.status === "resolved") return "Still resolved";
+	}
+	return findingStatusLabel(step.status);
+}
+/** Who or what decided a step. */
+function statusBasisPhrase(step) {
+	switch (step.basis) {
+		case "code": return step.commitSha ? `by the new commits, at ${step.commitSha.slice(0, 7)}` : "by the new commits";
+		case "conversation": return "by the pull request conversation";
+		case "manual": return "by you";
+		default: return "";
+	}
+}
+/** Why a settled finding was looked at again, when that is why. */
+function statusTriggerSentence(step) {
+	if (step.trigger === "evidence-edited") return "The comment it was resolved by was edited.";
+	if (step.trigger === "evidence-deleted") return "The comment it was resolved by was deleted.";
+	return "";
+}
+/** Whose words these are, and what they were. */
+function evidenceAttribution(evidence) {
+	const what = evidence.kind === "review" ? "review" : "comment";
+	if (evidence.deleted) return `@${evidence.author}'s ${what}, since deleted`;
+	return `@${evidence.author}'s ${what}, as it read then`;
+}
+/** The steps paired with the status each moved from: the previous step's, or none for the first. */
+function statusSteps(history) {
+	return history.map((step, index) => ({
+		from: index > 0 ? history[index - 1].status : void 0,
+		step
+	}));
+}
+/** Whether there is a history worth showing: at least one change recorded, not only a carried start. */
+function hasStatusHistory(history) {
+	return !!history?.some((step) => !step.carried);
+}
+function renderEvidence$2(evidence, now) {
+	if (!evidence) return "";
+	const edited = evidence.editedAt && Number.isFinite(Date.parse(evidence.editedAt)) ? ` · edited ${escapeHtml$1(formatRelativeTime(Date.parse(evidence.editedAt), now))}` : "";
+	const body = evidence.body ? `<p class="pr-helper-status-evidence-body">${renderInlineText(evidence.body)}</p>` : "";
+	const link = evidence.url && /^https?:\/\//i.test(evidence.url) ? `<a class="pr-helper-status-evidence-link" href="${escapeHtml$1(evidence.url)}" target="_blank" rel="noopener noreferrer">Open the ${evidence.kind === "review" ? "review" : "comment"}</a>` : "";
+	return `<blockquote class="pr-helper-status-evidence" data-status-evidence>
+          <p class="pr-helper-status-evidence-who">${escapeHtml$1(evidenceAttribution(evidence))}${edited}</p>
+          ${body}${link}
+        </blockquote>`;
+}
+function renderStep$1(from, step, now) {
+	const source = [statusBasisPhrase(step), step.at !== void 0 ? formatRelativeTime(step.at, now) : ""].filter(Boolean).join(" · ");
+	const trigger = statusTriggerSentence(step);
+	return `<li class="pr-helper-reassessment-step" data-status-step="${escapeHtml$1(step.status)}">
+        <span class="pr-helper-reassessment-step-move pr-helper-status-step-headline">${escapeHtml$1(statusStepHeadline(from, step))}</span>
+        ${source ? `<span class="pr-helper-reassessment-step-source">${escapeHtml$1(source)}</span>` : ""}
+        ${trigger ? `<p class="pr-helper-reassessment-reason">${escapeHtml$1(trigger)}</p>` : ""}
+        ${step.reason ? `<p class="pr-helper-reassessment-reason">${renderInlineText(step.reason)}</p>` : ""}
+        ${renderEvidence$2(step.evidence, now)}
+      </li>`;
+}
+/** The disclosure, closed. Nothing when the status has never changed. */
+function renderStatusHistory(history, now = Date.now()) {
+	if (!hasStatusHistory(history)) return "";
+	return `<details class="pr-helper-reassessment-history pr-helper-status-history" data-status-history>
+      <summary class="pr-helper-reassessment-history-summary">Status history</summary>
+      <ol class="pr-helper-reassessment-steps">${statusSteps(history).map(({ from, step }) => renderStep$1(from, step, now)).join("")}</ol>
+    </details>`;
+}
+//#endregion
 //#region ../../packages/review-harness/src/domain/ui/components/finding_copy.ts
 /**
 * An issue's Copy: its title in bold, the description, the update, the investigation, how its grade
@@ -1325,8 +1463,12 @@ function issueText(issue, investigation, assessment) {
 	return join$1([
 		bold(issue.title),
 		issue.description,
-		update(issue.reconciliationStatus && findingStatusLabel(issue.reconciliationStatus), issue.reconciliationNote),
-		...investigation ? [investigated(issue.investigationImpact, issue.investigationNote), reassessed(assessment)] : [],
+		update(issue.reconciliationStatus && findingStatusLabel(issue.reconciliationStatus), issue.reconciliationNote, issue.reconciledAgainst),
+		...investigation ? [
+			investigated(issue.investigationImpact, issue.investigationNote),
+			reassessed(assessment),
+			statusHistory(issue.statusHistory)
+		] : [],
 		where(issue.file, issue.line, issue.relatedFiles)
 	]);
 }
@@ -1334,8 +1476,8 @@ function questionText(question, investigation, assessment) {
 	return join$1([
 		bold(question.question),
 		question.whyThisMatters,
-		labelled("Answered by the new changes", question.reconciliationAnswer),
-		update(question.reconciliationStatus && questionStatusLabel(question.reconciliationStatus), question.reconciliationNote),
+		labelled(reconciledAnswerHeading(question.reconciledAgainst), question.reconciliationAnswer),
+		update(question.reconciliationStatus && questionStatusLabel(question.reconciliationStatus), question.reconciliationNote, question.reconciledAgainst),
 		...investigation ? [
 			labelled("Answer from investigation", question.investigationAnswer),
 			investigated(question.investigationImpact, question.investigationNote),
@@ -1345,9 +1487,9 @@ function questionText(question, investigation, assessment) {
 	]);
 }
 /** What the last update said about the finding, under the status the card shows beside it. */
-function update(status, note) {
+function update(status, note, against) {
 	if (!note?.trim()) return void 0;
-	return `**After the new commits${status ? ` (${status.toLowerCase()})` : ""}:** ${note.trim()}`;
+	return `**${reconciledAfter(against)}${status ? ` (${status.toLowerCase()})` : ""}:** ${note.trim()}`;
 }
 /**
 * What an investigation concluded, under the verdict the card's badge shows. "Investigated" is the
@@ -1367,6 +1509,24 @@ function reassessed(assessment) {
 	const move = `${assessment.original ?? "ungraded"} → ${assessment.current}, ${sourcePhrase(last.source)}`;
 	return last.reason?.trim() ? `**Reassessed (${move}):** ${last.reason.trim()}` : `**Reassessed:** ${move}.`;
 }
+/**
+* Every change of status, each with what it rested on and the evidence as it read then: the record
+* of why the review reached its conclusion, which a link to a since-edited comment could not keep.
+* Copy only, like the reassessed grade; a feedback draft is not the place for the review's working.
+*/
+function statusHistory(history) {
+	if (!hasStatusHistory(history)) return void 0;
+	return `**Status history:**\n${statusSteps(history).map(({ from, step }) => {
+		const basis = statusBasisPhrase(step);
+		const said = [statusTriggerSentence(step), step.reason?.trim()].filter(Boolean).join(" ");
+		const head = `- ${statusStepHeadline(from, step)}${basis ? ` ${basis}` : ""}${said ? `: ${said}` : ""}`;
+		const evidence = step.evidence;
+		if (!evidence) return head;
+		const quoted = evidence.body ? `: ${evidence.body.replace(/\s+/g, " ")}` : "";
+		const link = evidence.url ? ` (${evidence.url})` : "";
+		return `${head}\n  > ${evidenceAttribution(evidence)}${quoted}${link}`;
+	}).join("\n")}`;
+}
 function labelled(label, text) {
 	return text?.trim() ? `**${label}:** ${text.trim()}` : void 0;
 }
@@ -1409,25 +1569,6 @@ function shortSha$1(sha) {
 	return sha.slice(0, MIN_ABBREVIATED_LENGTH);
 }
 //#endregion
-//#region ../../packages/review-harness/src/domain/utils/relative_time.ts
-function formatRelativeTime(ts, now = Date.now()) {
-	const diffMs = Math.max(0, now - ts);
-	const minute = 6e4;
-	const hour = 60 * minute;
-	const day = 24 * hour;
-	if (diffMs < minute) return "Just now";
-	if (diffMs < hour) {
-		const mins = Math.floor(diffMs / minute);
-		return `${mins} min${mins === 1 ? "" : "s"} ago`;
-	}
-	if (diffMs < day) {
-		const hours = Math.floor(diffMs / hour);
-		return `${hours} hour${hours === 1 ? "" : "s"} ago`;
-	}
-	const days = Math.floor(diffMs / day);
-	return `${days} day${days === 1 ? "" : "s"} ago`;
-}
-//#endregion
 //#region ../../packages/review-harness/src/domain/ui/components/intent_note.ts
 /**
 * The Overview's unclear motive, kept in step with its Intent question.
@@ -2613,9 +2754,9 @@ function renderIssueConfidence(confidence) {
 * is the axis the reviewer is scanning for. They already know what the findings were; what they
 * came back to learn is which ones the author dealt with.
 */
-function renderFindingStatus(status) {
+function renderFindingStatus(status, against) {
 	if (!status) return "";
-	return `<span${pillAttrs(`pr-helper-finding-status pr-helper-finding-status-${status}`, findingStatusTip(status))}>${escapeHtml$1(findingStatusLabel(status))}</span>`;
+	return `<span${pillAttrs(`pr-helper-finding-status pr-helper-finding-status-${status}`, findingStatusTip(status, against))}>${escapeHtml$1(findingStatusLabel(status))}</span>`;
 }
 /**
 * The same pill, for a question.
@@ -2624,9 +2765,9 @@ function renderFindingStatus(status) {
 * a finding is `resolved` and a question is `answered`, which are different events. Nothing fixed
 * the question; somebody now knows the answer.
 */
-function renderQuestionStatus(status) {
+function renderQuestionStatus(status, against) {
 	if (!status) return "";
-	return `<span${pillAttrs(`pr-helper-finding-status pr-helper-finding-status-${status === "answered" ? "resolved" : status}`, questionStatusTip(status))}>${escapeHtml$1(questionStatusLabel(status))}</span>`;
+	return `<span${pillAttrs(`pr-helper-finding-status pr-helper-finding-status-${status === "answered" ? "resolved" : status}`, questionStatusTip(status, against))}>${escapeHtml$1(questionStatusLabel(status))}</span>`;
 }
 /** The model's one-line reason for a status, when it gave one. */
 function renderReconciliationNote(note) {
@@ -2771,7 +2912,7 @@ function renderIssuesTab(session, investigatingId, options) {
           ${options?.askBuddy ? renderAskBuddyMenu("issue") : ""}
         </div>
         <div class="pr-helper-issue-badges">
-          ${renderFindingStatus(issue.reconciliationStatus)}
+          ${renderFindingStatus(issue.reconciliationStatus, issue.reconciledAgainst)}
           ${renderDismissedBadge(isDismissed)}
           ${renderInvestigationImpact(issue.investigationImpact)}
           ${renderSectionTag(issue.sectionId, sections)}
@@ -2783,6 +2924,7 @@ function renderIssuesTab(session, investigatingId, options) {
       ${renderAssessmentRegion("issue", assessment, grading)}
       <p class="pr-helper-issue-desc">${renderInlineText(issue.description)}</p>${renderCardExplanation(explanationOf(session, "issue", issue.id), options)}
       ${renderReconciliationNote(issue.reconciliationNote)}
+      ${renderStatusHistory(issue.statusHistory, now)}
       ${renderInvestigationNote(issue.investigationNote)}
       ${renderExistingDiscussion(issue.existingDiscussion)}
       ${renderFileLocation(issue.file, "pr-helper-issue-loc", issue.line)}
@@ -2811,7 +2953,12 @@ function renderIssuesTab(session, investigatingId, options) {
 */
 function closedFindingsSummary(counts) {
 	const byCommits = counts.done - counts.reviewerClosed;
-	if (counts.reviewerClosed === 0) return `Resolved by the new commits (${counts.done})`;
+	if (counts.reviewerClosed === 0) {
+		const byConversation = counts.byConversation ?? 0;
+		if (byConversation === 0) return `Resolved by the new commits (${counts.done})`;
+		if (byConversation === counts.done) return `Resolved by the conversation (${counts.done})`;
+		return `Resolved (${counts.done})`;
+	}
 	if (byCommits === 0) return `Dismissed by you (${counts.done})`;
 	return `Dealt with (${counts.done})`;
 }
@@ -2854,7 +3001,8 @@ function renderStaleDismissal(stale) {
 function renderDismissToggle(issue, isDismissed, stale, dismissal) {
 	const label = stale ? "Dismiss again" : isDismissed ? "Reopen" : "Dismiss";
 	const settled = issue.reconciliationStatus === "resolved";
-	return `<button type="button" class="pr-helper-issue-dismiss-toggle" data-issue-dismissed="${escapeHtml$1(issue.id)}" data-dismissed="${isDismissed}"${stale ? " data-stale=\"true\"" : ""}${dismissal ? ` data-dismissed-in="${escapeHtml$1(dismissal.sessionId)}"` : ""}${settled ? " data-settled=\"true\"" : ""}>${label}</button>`;
+	const settledBy = settled ? ` data-settled-by="${resolvedByConversation(issue) ? "conversation" : "code"}"` : "";
+	return `<button type="button" class="pr-helper-issue-dismiss-toggle" data-issue-dismissed="${escapeHtml$1(issue.id)}" data-dismissed="${isDismissed}"${stale ? " data-stale=\"true\"" : ""}${dismissal ? ` data-dismissed-in="${escapeHtml$1(dismissal.sessionId)}"` : ""}${settled ? ` data-settled="true"${settledBy}` : ""}>${label}</button>`;
 }
 /**
 * The group an Intent question lives in, keyed like a theme's so the workspace client finds its
@@ -2873,7 +3021,7 @@ function renderQuestionCard(q, sections, answered, records, now = Date.now(), in
 	const settled = q.reconciliationStatus === "answered";
 	const answerToggle = `<button type="button" class="pr-helper-question-answer-toggle" data-question-answered="${escapeHtml$1(q.id)}" data-answered="${answered}"${settled ? " data-settled=\"true\"" : ""}>${answered ? "Mark as unanswered" : "Mark answered"}</button>`;
 	const investigationAnswer = q.investigationAnswer?.trim() ? `<h5>Answer from investigation</h5><p class="pr-helper-issue-desc pr-helper-investigation-answer">${renderInlineText(q.investigationAnswer)}</p>` : "";
-	const reconciliationAnswer = q.reconciliationAnswer?.trim() ? `<h5>Answered by the new changes</h5><p class="pr-helper-issue-desc pr-helper-investigation-answer">${renderInlineText(q.reconciliationAnswer)}</p>` : "";
+	const reconciliationAnswer = q.reconciliationAnswer?.trim() ? `<h5>${escapeHtml$1(reconciledAnswerHeading(q.reconciledAgainst))}</h5><p class="pr-helper-issue-desc pr-helper-investigation-answer">${renderInlineText(q.reconciliationAnswer)}</p>` : "";
 	return `
     <div class="pr-helper-question-card${answered ? " pr-helper-question-card-answered" : ""}${q.investigationImpact ? ` pr-helper-issue-investigation-${q.investigationImpact}` : ""}" data-finding-card="${escapeHtml$1(q.id)}"${questionHome(q) ? ` data-section-id="${escapeHtml$1(questionHome(q))}"` : ""}>
       <div class="pr-helper-issue-header">
@@ -2884,7 +3032,7 @@ function renderQuestionCard(q, sections, answered, records, now = Date.now(), in
           ${options?.askBuddy ? renderAskBuddyMenu("question") : ""}
         </div>
         <div class="pr-helper-issue-badges">
-          ${renderQuestionStatus(q.reconciliationStatus)}
+          ${renderQuestionStatus(q.reconciliationStatus, q.reconciledAgainst)}
           ${renderInvestigationImpact(q.investigationImpact)}
           <!-- Rendered on every card, hidden by CSS inside a theme group, because the heading
                above it already says the same word. The client moves cards between the group and
@@ -5217,14 +5365,19 @@ function updateFacts(outcome) {
 	const counts = outcome.counts;
 	if (!counts || !outcome.toSha) return null;
 	const plural = (n, one, many) => `${n} ${n === 1 ? one : many}`;
+	const byConversation = counts.resolvedByConversation ?? 0;
+	const fixed = counts.resolved - byConversation;
 	const changes = [
-		counts.resolved ? `${plural(counts.resolved, "issue", "issues")} fixed` : "",
+		fixed ? `${plural(fixed, "issue", "issues")} fixed` : "",
+		byConversation ? `${plural(byConversation, "issue", "issues")} resolved by the conversation` : "",
+		counts.reopened ? `${plural(counts.reopened, "issue", "issues")} reopened` : "",
 		counts.newIssues ? plural(counts.newIssues, "new issue", "new issues") : "",
 		counts.affected ? `${counts.affected} to look at again` : "",
 		counts.questionsAnswered ? `${plural(counts.questionsAnswered, "question", "questions")} answered` : ""
 	].filter(Boolean);
 	const grades = counts.gradeChanges ? [`${plural(counts.gradeChanges, "grade change", "grade changes")} suggested`] : [];
-	return [`<code>${escapeHtml$1(outcome.toSha.slice(0, 7))}</code>`, ...[...changes.length ? changes : ["No change to the findings"], ...grades].map(escapeHtml$1)];
+	const basis = outcome.reassessed;
+	return [...basis && !basis.code ? [escapeHtml$1("Reassessed against the conversation")] : [`<code>${escapeHtml$1(outcome.toSha.slice(0, 7))}</code>`, ...basis?.discussion ? [escapeHtml$1("with the conversation")] : []], ...[...changes.length ? changes : ["No change to the findings"], ...grades].map(escapeHtml$1)];
 }
 /**
 * The two things that can go quietly half-wrong, said out loud.
@@ -11951,7 +12104,7 @@ A developer is reviewing a specific code change and is looking at a review guide
 
 Answer only in relation to this change and the repository it lives in. You have the working tree, so open the files, trace callers and read history wherever that settles the question. Do not speculate about code you could have read.
 
-If a question cannot be answered from this change and this repository, say so plainly and say what would answer it. Do not pad the answer out with general advice about the topic; a developer mid-review wants the specific answer or an honest "that isn't determinable from here".
+If a question cannot be answered from this change, this repository and the pull request context below, say so plainly and say what information is missing. Do not suggest shell commands, API calls or other ways to retrieve forge data yourself. Do not pad the answer out with general advice about the topic; a developer mid-review wants the specific answer or an honest "that isn't determinable from here".
 
 Write in short paragraphs and keep it to what was asked. Markdown renders, so use it where it carries meaning: backticks for identifiers and paths, a fenced block for a snippet worth reading as code rather than described in a sentence, a bullet list where the answer genuinely is a list. Prose is still the default. No headings, and do not break a paragraph into bullets to look organised.`;
 var AskUnavailableError = class extends Error {};
@@ -12058,6 +12211,8 @@ function buildPrompt$2(workspace, question, sectionId, cwd, context = {}) {
 		"---",
 		"DIFF:",
 		workspace.diff,
+		"---",
+		chatConversationSection(context.conversation ?? storedConversation(workspace)),
 		...reading ? ["---", reading] : [],
 		...recent.length > 0 ? [
 			"---",
@@ -14320,25 +14475,15 @@ async function kissFreshnessOrNull(workspace) {
 */
 async function handleRefreshPr(res, workspace) {
 	const reattributing = JSON.stringify(workspace.session.result);
+	adoptLegacyBaseline(workspace);
+	const startedAt = Date.now();
 	let outcome;
 	try {
 		outcome = await refreshPrContext(workspace);
 	} catch {
-		outcome = {
-			at: Date.now(),
-			ok: false,
-			message: "The pull request could not be re-read. Nothing here has changed."
-		};
+		outcome = unforeseenFailure(startedAt);
 	}
-	workspace.lastPrRefresh = outcome;
-	mutateWorkspace(workspace.id, (current) => {
-		current.prContext = workspace.prContext;
-		current.lastPrRefresh = workspace.lastPrRefresh;
-		if (JSON.stringify(current.session.result) === reattributing) current.session = {
-			...current.session,
-			result: workspace.session.result
-		};
-	});
+	saveRefresh(workspace, outcome, reattributing);
 	record({
 		event: "pr_conversation_rechecked",
 		reviewId: workspace.id,
@@ -14557,10 +14702,16 @@ async function handleAsk(req, res, workspace) {
 	try {
 		let answer;
 		try {
+			const reattributing = JSON.stringify(workspace.session.result);
+			const { prepared, refresh } = await prepareConversationWithin(workspace, ASK_REREAD_CEILING_MS);
+			if (refresh) try {
+				saveRefresh(workspace, refresh, reattributing);
+			} catch {}
 			answer = await askAboutChange(workspace, question, sectionId, run.signal, {
 				...explains ? { explaining: explains } : {},
 				...continuing ? { continuing } : {},
-				...reassessed ? { reassessing: reassessed } : {}
+				...reassessed ? { reassessing: reassessed } : {},
+				conversation: prepared
 			});
 		} catch (error) {
 			failed();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "prreviewbuddy",
-  "version": "0.75.0",
+  "version": "0.76.9",
   "description": "Understand code changes before you approve them. Local branch and pull request reviews with Claude Code, Codex or Gemini CLI.",
   "license": "UNLICENSED",
   "homepage": "https://prreviewbuddy.com",

package/static/sidebar.css

@@ -5468,6 +5468,19 @@
 .pr-helper-reassessment-evidence .pr-helper-issue-loc { margin-top: 2px; }
 /* Muted, not alarming: an unverified suggestion is still a reasoned one, and can still be accepted. */
 .pr-helper-reassessment-unverified { margin: 2px 0 0; font-size: 11px; font-style: italic; color: #9a6700; }
+/* A finding's status history reuses the grade history's disclosure and steps. The quote is the
+   evidence as it read when the decision was made, which may no longer be what the thread says. */
+.pr-helper-status-step-headline { font-weight: 600; }
+.pr-helper-status-evidence {
+  margin: 4px 0 0;
+  padding: 2px 0 2px 8px;
+  border-left: 2px solid rgba(15, 23, 42, 0.18);
+  font-size: 11px;
+  color: var(--color-fg-muted, #57606a);
+}
+.pr-helper-status-evidence-who { margin: 0; font-style: italic; }
+.pr-helper-status-evidence-body { margin: 2px 0 0; line-height: 1.5; white-space: pre-wrap; overflow-wrap: anywhere; }
+.pr-helper-status-evidence-link { display: inline-block; margin-top: 2px; color: #0969da; }
 /* A proposal is a question put to the reviewer, so it gets the purple rule an investigation note
    has, and its three answers in a row beneath it. */
 .pr-helper-reassessment-proposal {