prreviewbuddy 0.90.0 → 0.91.1

package/CHANGELOG.md

@@ -1,5 +1,31 @@
 # Changelog
 
+## 0.91.1
+
+**Issues lead with what they mean.** Each issue from a new analysis now opens with its consequence:
+a short title saying what can happen, a sentence or two on when and how, and, when there is a
+specific one, the step that would settle it, marked **To verify** or **To decide**. The technical
+title, the reasoning, the code location and the history are under **Technical details** on the
+card, one click (or Enter) away, with nothing to wait for.
+
+Outstanding issues are grouped by the decision they ask of you: **Failures**, **Possible risks**,
+**Intent to confirm** and **Maintenance**, each ordered by severity. The group comes from the
+evidence the analysis read, not from its severity or how confident it is; both are still shown on
+every card. The pane opens with one sentence on what the change touches, a count of what is
+outstanding, and buttons to jump to each group. Folding a group is remembered for the next review.
+
+Resolved and dismissed issues stay in the closed list at the bottom. An issue the new commits fixed
+leads with what the code now does, when the update could see it in the code; one settled by a
+comment never claims a fix. Updates keep each issue's consequence true: they are shown it, and
+restate or withdraw it when the new commits change what it describes.
+
+Reviews made before this are shown as they were, and are not rewritten. Beside the new groups their
+issues sit under **Other issues**; on their own they need no heading.
+
+For coding agents, `list_findings` and `get_finding` (and `prreviewbuddy findings --json` and
+`finding --json`) carry each issue's `consequence` and, on an issue the code resolved,
+`resolved_behaviour`, so a list of findings can be triaged without fetching each one.
+
 ## 0.90.0
 
 **Connect your coding agent from Settings.** A new **AI integrations** section in Settings shows,

package/dist/main.js

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { $ as readAskRequest, $i as explanationOf, Ar as attemptIdOf, At as UpdateAlreadyRunningError, Bt as append, Dt as parkedUpdateRefusal, E as relativeTime, Ea as resolveTarget, Er as forgeResolver, F as spawnRunner, Fa as git, Ft as prepareReview, G as reviewTag, Ga as AGENT_IDS, Gr as isClaimed, Gt as recordTelemetryUploadConsent, H as findingRef, Ha as writeAgentPreference, Hr as purposeOf, I as applyUpdate, It as settleReview, J as forgetAskRequest, Ja as agentIdentityOf, Ji as reviewerDispositions, Jr as followedRefName, Jt as BUILD_VERSION, Ka as agentById, Kt as telemetryUploadConsent, L as modelProvenanceOf, La as resolveAgent, Lr as PHASES, Lt as reanalyseReview, Ma as AmbiguousRefError, Mt as runningJobs, Nr as isTerminal$1, Nt as failureOf, P as runDetached, Pr as loadJob, Qa as runnableAgents, Ra as resolveModel, T as runBatch, Ta as kissOutcome, Ua as CONFIG_PATH, Ui as isIssueOutstanding, Ur as clearClaim, Ut as stamp, Va as readAgentPreference, Vr as progressSteps, Wa as STORE_ROOT, Wi as isQuestionOutstanding, Wr as driverGone, Wt as deliver, Xa as isAgentId, Y as interruption, Ya as detectAgents, Yi as verdictClosure, Yt as PACKAGE_NAME, Z as isTerminalAsk, Za as parkedReason, _t as parseWorkspacePort, at as healthyPort, br as findExistingReview, bt as writePortPreference, ca as findingAssessment, di as reviewedRepositories, dr as MANAGED_ROOT, dt as stopServer, et as recordAskAccepted, fi as summarise, ft as workspaceUrl, gr as removeWorktree, gt as MIN_WORKSPACE_PORT, hr as readMarker, ht as MAX_WORKSPACE_PORT, i as deleteReview$1, ii as matchingWorkspaceIds, it as ensureServer, j as RUNNER_KINDS, jt as liveUpdateFor, k as queueForReview, ka as TargetError, kr as allJobs, li as recentWorkspaces, mi as workspaceRevision, mt as DEFAULT_WORKSPACE_PORT, ni as lineagePosition, nt as bootstrapUrl, o as liveJobsFor, oi as positionInLineage, ot as readServerRecord, q as endAskRequest, qt as telemetryUploadEnabled, ri as loadWorkspace, s as wasBlocked, sr as modelHelpLines, st as reviewsUrl, t as MCP_SERVER_INFO_NAME, tt as readIndexToken, u as recordKissRun, ua as findingStatus, ui as reviewedCommit, vr as describeMovement, wa as WITHHELD_REASON, x as batchIdentity, xt as COMMIT_REVIEW_CANNOT_UPDATE, yr as describeSubject, yt as statedWorkspacePort, za as agentIdOf, zt as startJob } from "./mcp_tools-CSj5vcSE.js";
+import { $ as readAskRequest, $a as runnableAgents, $i as explanationOf, Aa as TargetError, Ar as attemptIdOf, At as UpdateAlreadyRunningError, Ba as agentIdOf, Bt as append, Da as resolveTarget, Dt as parkedUpdateRefusal, E as relativeTime, Ea as kissOutcome, Er as forgeResolver, F as spawnRunner, Ft as prepareReview, G as reviewTag, Ga as STORE_ROOT, Gr as isClaimed, Gt as recordTelemetryUploadConsent, H as findingRef, Ha as readAgentPreference, Hr as purposeOf, I as applyUpdate, Ia as git, It as settleReview, J as forgetAskRequest, Ji as reviewerDispositions, Jr as followedRefName, Jt as BUILD_VERSION, Ka as AGENT_IDS, Kt as telemetryUploadConsent, L as modelProvenanceOf, Lr as PHASES, Lt as reanalyseReview, Mt as runningJobs, Na as AmbiguousRefError, Nr as isTerminal$1, Nt as failureOf, P as runDetached, Pr as loadJob, Qa as parkedReason, Ra as resolveAgent, T as runBatch, Ta as WITHHELD_REASON, Ua as writeAgentPreference, Ui as isIssueOutstanding, Ur as clearClaim, Ut as stamp, Vr as progressSteps, Wa as CONFIG_PATH, Wi as isQuestionOutstanding, Wr as driverGone, Wt as deliver, Xa as detectAgents, Y as interruption, Ya as agentIdentityOf, Yi as verdictClosure, Yt as PACKAGE_NAME, Z as isTerminalAsk, Za as isAgentId, _t as parseWorkspacePort, at as healthyPort, br as findExistingReview, bt as writePortPreference, ca as findingAssessment, di as reviewedRepositories, dr as MANAGED_ROOT, dt as stopServer, et as recordAskAccepted, fi as summarise, ft as workspaceUrl, gr as removeWorktree, gt as MIN_WORKSPACE_PORT, hr as readMarker, ht as MAX_WORKSPACE_PORT, i as deleteReview$1, ii as matchingWorkspaceIds, it as ensureServer, j as RUNNER_KINDS, jt as liveUpdateFor, k as queueForReview, kr as allJobs, li as recentWorkspaces, mi as workspaceRevision, mt as DEFAULT_WORKSPACE_PORT, ni as lineagePosition, nt as bootstrapUrl, o as liveJobsFor, oi as positionInLineage, ot as readServerRecord, q as endAskRequest, qa as agentById, qt as telemetryUploadEnabled, ri as loadWorkspace, s as wasBlocked, sr as modelHelpLines, st as reviewsUrl, t as MCP_SERVER_INFO_NAME, tt as readIndexToken, u as recordKissRun, ua as findingStatus, ui as reviewedCommit, vr as describeMovement, x as batchIdentity, xt as COMMIT_REVIEW_CANNOT_UPDATE, yr as describeSubject, yt as statedWorkspacePort, za as resolveModel, zt as startJob } from "./mcp_tools-BR7_XprF.js";
 import { basename, dirname, isAbsolute, join, resolve } from "node:path";
 import { spawn } from "node:child_process";
 import { existsSync, mkdirSync, readdirSync, realpathSync, rmSync, statSync, writeFileSync } from "node:fs";
@@ -1044,7 +1044,14 @@ function summaryOf(tag, entry, judge) {
 		...finding.file ? { location: {
 			path: finding.file,
 			...issue?.line ? { line: issue.line } : {}
-		} } : {}
+		} } : {},
+		...issue?.consequence ? { consequence: {
+			title: issue.consequence.title,
+			scenario: issue.consequence.scenario,
+			...issue.consequence.group ? { group: issue.consequence.group } : {},
+			...issue.consequence.next ? { next: { ...issue.consequence.next } } : {}
+		} } : {},
+		...issue?.resolvedBehaviour && status === "resolved" ? { resolved_behaviour: issue.resolvedBehaviour } : {}
 	};
 }
 /** A question can run to a paragraph. The list carries its first line; `get_finding` has the rest. */

package/dist/{mcp_tools-CSj5vcSE.js → mcp_tools-BR7_XprF.js}

@@ -4206,6 +4206,13 @@ var REVIEW_ATTENTIONS = [
 	"medium",
 	"low"
 ];
+/** In the order the Issues pane draws them. */
+var IMPACT_GROUPS = [
+	"failure",
+	"risk",
+	"intent",
+	"maintenance"
+];
 /**
 * The JSON in a reply, whether or not the model wrapped it in a fence.
 *
@@ -4373,10 +4380,49 @@ function normalizeIssues(value) {
 			line: typeof item.line === "number" ? item.line : void 0,
 			sectionId: asSectionId(item),
 			confidence: REVIEW_ATTENTIONS.includes(item.confidence) ? item.confidence : void 0,
-			existingDiscussion: asClaimedThreadRef(item.existingDiscussion)
+			existingDiscussion: asClaimedThreadRef(item.existingDiscussion),
+			...withConsequence(normalizeConsequence(item.consequence))
 		};
 	}).filter((issue) => issue.title.trim() !== "" || issue.description.trim() !== "");
 }
+/**
+* An issue's consequence, or nothing.
+*
+* Well-formedness only. A consequence with no title or no scenario has nothing to say and is
+* dropped, so the card falls back to its technical account rather than showing an empty headline.
+* An unknown group is left out rather than defaulted, because every default would be a claim: the
+* card then sits under "Other issues". A group is never derived from severity or confidence; the
+* two stay separate attributes, as the model stated them.
+*/
+function normalizeConsequence(value) {
+	if (typeof value !== "object" || value === null || Array.isArray(value)) return void 0;
+	const item = value;
+	const title = asString$1(item.title).trim();
+	const scenario = asString$1(item.scenario).trim();
+	if (!title || !scenario) return void 0;
+	const group = IMPACT_GROUPS.includes(item.group) ? item.group : void 0;
+	const next = normalizeStep(item.next);
+	return {
+		title,
+		scenario,
+		...next ? { next } : {},
+		...group ? { group } : {}
+	};
+}
+function normalizeStep(value) {
+	if (typeof value !== "object" || value === null) return void 0;
+	const item = value;
+	const text = asString$1(item.text).trim();
+	if (!text) return void 0;
+	return item.kind === "verify" || item.kind === "decide" ? {
+		kind: item.kind,
+		text
+	} : { text };
+}
+/** Spread form, so an issue without one carries no `consequence` key at all. */
+function withConsequence(consequence) {
+	return consequence ? { consequence } : {};
+}
 function asClaimedThreadRef(value) {
 	if (typeof value !== "object" || value === null) return void 0;
 	const claim = value;
@@ -12503,6 +12549,50 @@ function applyMotivation(result, candidate, authorDescription) {
 */
 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/consequence_guidance.ts
+/**
+* What an issue's "consequence" is, as every prompt that raises issues asks for it.
+*
+* One fragment and one guidance string, so the first analysis, follow-ups, updates and the
+* extension's investigation cannot drift apart. The fragment is the whole shape, never an
+* abbreviation: a model cannot follow a schema it was not shown.
+*/
+var CONSEQUENCE_FIELD = "\"consequence\": { \"title\": string, \"scenario\": string, \"group\": \"failure\" | \"risk\" | \"intent\" | \"maintenance\", \"next\"?: { \"kind\": \"verify\" | \"decide\", \"text\": string } }";
+/** The same shape on a prior finding's entry in an update, where null withdraws it and absence keeps it. */
+var CONSEQUENCE_REVISION_FIELD = `${CONSEQUENCE_FIELD.replace("\"consequence\": ", "\"consequence\"?: ")} | null`;
+var CONSEQUENCE_GUIDANCE = `"consequence" on an issue: the same issue told as what it means, for someone who knows the product but may not know its implementation. It is shown first; "title" and "description" stay the technical account behind it. Reframe around the consequence, do not just simplify the technical wording.
+  - "title": the observable consequence or the meaningful development constraint, as a short sentence (e.g. "Creating a meal can fail halfway, leaving it outside the planner").
+  - "scenario": one or two sentences: the triggering condition and what happens (e.g. "If adding it to the day fails after saving, the user sees an error but the meal still exists. Retrying may then be blocked by its name.").
+  - "next": the one step that would settle it, when there is a specific one. "kind" is "verify" for a check or test that would establish whether it happens (e.g. "Try a receipt with an unreadable amount.") and "decide" for an acceptance or intent call only the reviewer can make (e.g. "Confirm this broader change is intended."). Omit "next" rather than writing a generic step.
+  - "group", by the evidence behind the issue, never by how sure you sound and independently of "severity": "failure" when the code you read establishes an incorrect outcome under a stated condition; "risk" when a plausible undesirable consequence still needs verifying; "intent" when the change has a concrete behavioural consequence whose correctness depends on what was intended; "maintenance" when it constrains future development, maintainability or reliability rather than current behaviour. "confidence" is reported separately and does not decide the group: if the evidence does not establish the failure, it is a "risk", and say what is unverified.
+  - Keep every condition and qualification that affects the decision. Avoid code identifiers unless the consequence cannot be understood without one. Do not state severity.
+  - Never claim user harm, a measured performance effect, a reproduction or a certainty the evidence does not give. A performance concern nobody measured says so (e.g. "Opening a meal may do unnecessary work for nested meals... The effect has not been measured."). A maintenance or intent issue is described as one, not as a user-facing failure.
+  - An issue names a concrete consequence. A request for missing requirements with no concrete consequence is a question for the author, not an issue, and is never raised as both.`;
+/** What "orientation" is for, as the first analysis and an update ask for it. */
+var ORIENTATION_GUIDANCE = `"orientation": one sentence, at most about 25 words, naming the behaviour or product area this change touches, for a reviewer about to decide on its issues (e.g. "This change affects receipt extraction and how duplicate receipt details are combined."). Say what it touches, not whether it is safe, and do not summarise the issues.`;
+/**
+* How an update keeps a prior finding's consequence and resolution true, beside its status.
+*
+* The technical text of a prior finding never changes in an update, but what it means can: the
+* condition it describes may have narrowed, the evidence moved, or the step become moot. A
+* consequence that no longer describes the finding is worse than none, so the model is shown each
+* one and must keep, restate or withdraw it.
+*/
+var CONSEQUENCE_UPDATE_GUIDANCE = `"consequence" on a finding entry: a finding marked [consequence: ...] has an impact summary the reviewer reads first. Omit "consequence" when it still describes the finding accurately, which is the usual case. Return a restated "consequence" (same shape as on a new issue) when what has changed since alters the condition, the evidence, the next step or the group it describes, most often for an "affected" finding. Return null when it no longer holds and you cannot restate it accurately. Never return "consequence" for a finding without a [consequence: ...] marker.
+"resolvedBehaviour" on a finding entry: only with "resolved", and only when the changed code shows what now happens under the finding's condition, in one sentence for someone who knows the product (e.g. "If adding the meal to the planner fails, the meal is no longer saved."). Omit it when the code does not show that, and whenever the finding is settled by a comment rather than by code.`;
+/**
+* The impact summary the reviewer reads first, shown under a prior finding so a later pass can keep,
+* restate or withdraw it (`CONSEQUENCE_UPDATE_GUIDANCE`, and the investigation's own rule). Group
+* and step are included: either can go stale on its own. A finding stored without one shows
+* nothing, which is also what tells the model not to add one.
+*/
+function formatPriorConsequence(issue) {
+	const consequence = issue.consequence;
+	if (!consequence) return "";
+	const next = consequence.next ? ` Next (${consequence.next.kind ?? "step"}): ${consequence.next.text}` : "";
+	return `\n   [consequence: group ${consequence.group ?? "none"}] ${consequence.title.trim().replace(/[.!?]$/, "")}. ${consequence.scenario.trim()}${next}`;
+}
+//#endregion
 //#region ../../packages/review-harness/src/domain/discussion/discussion_delta.ts
 /**
 * Which collections of a snapshot were capped. A snapshot from before coverage was recorded can
@@ -13265,6 +13355,7 @@ var STRUCTURED_OUTPUT_INSTRUCTIONS = `In addition to the review principles above
     "authorQuestion"?: string
   },
   "overallRisk": "low" | "medium" | "high",
+  "orientation": string,
   "sections": [{
     "id": string,
     "order": number,
@@ -13279,7 +13370,7 @@ var STRUCTURED_OUTPUT_INSTRUCTIONS = `In addition to the review principles above
     "reviewWeight": 1 | 2 | 3 | 4 | 5,
     "recommendedStartingPoint": boolean
   }],
-  "issues": [{ "id": string, "severity": "critical" | "major" | "minor" | "nit", "title": string, "description": string, "file"?: string, "line"?: number, "sectionId"?: string, "confidence"?: "high" | "medium" | "low", "existingDiscussion"?: { "threadId": string, "relation": "same" | "related" } }],
+  "issues": [{ "id": string, "severity": "critical" | "major" | "minor" | "nit", "title": string, "description": string, "file"?: string, "line"?: number, "sectionId"?: string, "confidence"?: "high" | "medium" | "low", ${CONSEQUENCE_FIELD}, "existingDiscussion"?: { "threadId": string, "relation": "same" | "related" } }],
   "questions": [{ "id": string, "question": string, "importance"?: "high" | "medium" | "low", "whyThisMatters"?: string, "file"?: string, "sectionId"?: string }],
   "recommendation": string,
   "nextSteps": string[]
@@ -13299,6 +13390,8 @@ Field guidance:
 - "recommendation": a 1-2 sentence merge verdict aimed at the developer (e.g. "Approve" or "Request changes — fix the unhandled rejection before merging"). Do not just restate the risk level, and do not copy Guide section content verbatim into this field.
 - "nextSteps": concrete actions to take before merging (specific tests to run, files to double-check), not a restatement of the Guide.
 - "issues": report only issues worth the reviewer's attention — do not invent issues to fill out the array, and do not omit significant issues. Let the count reflect the actual PR (a trivial PR may have zero or one issue; a large or risky PR may have several).
+- ${CONSEQUENCE_GUIDANCE}
+- ${ORIENTATION_GUIDANCE}
 - "existingDiscussion" on an issue: set this only when a thread in the PR DISCUSSION CONTEXT block already raises this same concern, and you have independently confirmed from the diff that it still applies. "threadId" must be copied verbatim from that thread's [T_...] marker — never invent an id, never guess, and omit the field entirely if no thread matches. Set "relation" to "same" when the thread already makes this exact point, or "related" when it discusses this area without covering the finding — a reviewer acts differently on "someone said this already" than on "someone was looking near here". Still write the issue normally; this field only records that the point was raised before, so the reviewer is not shown it as a fresh discovery. Do not use it to pad the review with issues you would otherwise have left out.
 - "confidence" on an issue: how sure you are the issue is real, on the same high/medium/low scale as a question's "importance". This is a separate axis from "severity", which is how much it would matter if real — a critical issue you are unsure about and a nit you are certain of are different things to act on, so do not let one field stand in for the other. Use LOW when you are reasoning from a diff excerpt without the surrounding code, and say what would confirm it in the description. Omit the field rather than guessing a middle value.
 - "questions": each one is a genuine open item the reviewer needs answered before they can sign off, not a rhetorical prompt. "importance" is how much attention it deserves, on the same high/medium/low scale as a section's "reviewAttention" (e.g. a question about a possibly-exposed secret is "high"; a naming preference is "low"). "whyThisMatters" is 1-2 sentences explaining the concrete consequence of the question going unanswered — not a restatement of the question itself.
@@ -13356,6 +13449,7 @@ function parseUnchecked(raw) {
 			schemaVersion: 1,
 			summary,
 			overallRisk: RISKS.includes(obj.overallRisk) ? obj.overallRisk : "medium",
+			...asString$1(obj.orientation).trim() ? { orientation: asString$1(obj.orientation).trim() } : {},
 			sections: normalizeSections(obj.sections),
 			issues: normalizeIssues(obj.issues),
 			questions: normalizeQuestions(obj.questions),
@@ -18001,7 +18095,7 @@ function formatPriorFindings(issues, assess, reassess = /* @__PURE__ */ new Map(
 		const related = (issue.relatedFiles ?? []).map((file) => ` [related: ${file}, not changed]`).join("");
 		const location = (issue.file ? ` [${issue.file}${issue.line ? `:${issue.line}` : ""}]` : "") + related;
 		const description = issue.description.trim();
-		return `${index}. (${promptGrade(assess("issue", issue))})${location}${priorFindingVerdict(issue, reassess.get(index))}${acceptedVerdict(status("issue", issue.id))} ${issue.title.trim()}${description ? `\n   ${description}` : ""}`;
+		return `${index}. (${promptGrade(assess("issue", issue))})${location}${priorFindingVerdict(issue, reassess.get(index))}${acceptedVerdict(status("issue", issue.id))} ${issue.title.trim()}${description ? `\n   ${description}` : ""}${formatPriorConsequence(issue)}`;
 	}).join("\n");
 }
 /**
@@ -18012,8 +18106,8 @@ function formatPriorFindings(issues, assess, reassess = /* @__PURE__ */ new Map(
 */
 var QUESTION_SHAPE_LINE = "  \"questions\": [{ \"sourceIndex\": number, \"status\": \"answered\" | \"still-open\" | \"affected\", \"answer\"?: string, \"note\"?: string, \"importance\"?: \"high\" | \"medium\" | \"low\", \"importanceReason\"?: string }],";
 var RECONCILIATION_SHAPE = `{
-  "findings": [{ "sourceIndex": number, "status": "resolved" | "still-open" | "affected", "note"?: string, "severity"?: "critical" | "major" | "minor" | "nit", "severityReason"?: string }],
-  "newIssues": [{ "id": string, "severity": "critical" | "major" | "minor" | "nit", "title": string, "description": string, "file"?: string, "line"?: number, "confidence"?: "high" | "medium" | "low" }],
+  "findings": [{ "sourceIndex": number, "status": "resolved" | "still-open" | "affected", "note"?: string, ${CONSEQUENCE_REVISION_FIELD}, "resolvedBehaviour"?: string, "severity"?: "critical" | "major" | "minor" | "nit", "severityReason"?: string }],
+  "newIssues": [{ "id": string, "severity": "critical" | "major" | "minor" | "nit", "title": string, "description": string, "file"?: string, "line"?: number, ${CONSEQUENCE_FIELD}, "confidence"?: "high" | "medium" | "low" }],
   "newSections": [{
     "id": string,
     "order": number,
@@ -18028,6 +18122,7 @@ var RECONCILIATION_SHAPE = `{
     "reviewWeight": 1 | 2 | 3 | 4 | 5
   }],
   "summary": string,
+  "orientation": string,
   "overallRisk": "low" | "medium" | "high",
   "recommendation": string,
   "nextSteps": string[]
@@ -18168,7 +18263,7 @@ ${REASSESS_TASK}` : ""}
    ${GRADE_TASK.question}`] : [],
 		conversationOnly ? `No commits were pushed, so there is no new code to review. Report a problem in "newIssues" only where the conversation points at something in the code the earlier analysis missed and you have confirmed it by reading the code. Do NOT repeat a previous finding here: if it still stands, that is what "still-open" is for.` : `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. ${SUMMARY_GUIDANCE}`
+		`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} ${ORIENTATION_GUIDANCE}`
 	].map((task, index) => `${index + 1}. ${task}`).join("\n");
 	return `${conversationOnly ? `You are RECONCILING an earlier analysis of this pull request against what has changed in its conversation since that analysis was made.
 
@@ -18194,6 +18289,10 @@ ${reconciliationShape(includeQuestions, discussion !== void 0)}
 
 "note" is one short sentence on *why* the status is what it is — name the change that resolved it, or what leaves it standing.${discussion ? " Where a comment is the reason, name its author." : ""} Omit it rather than restating the finding.
 
+${CONSEQUENCE_UPDATE_GUIDANCE}
+
+${CONSEQUENCE_GUIDANCE}
+
 "severity" on a new issue is how much it would matter if real; "confidence" is how sure you are that it is real. Separate axes — do not let one stand in for the other, and omit "confidence" rather than guessing a middle. Report only what is worth the reviewer's attention: empty arrays are valid answers and are far better than padded ones.`;
 }
 /**
@@ -18312,10 +18411,14 @@ function parseReconciliationResult(raw, priorCount, options = {}) {
 		const note = asString$1(item.note).trim();
 		const grade = parseGrade("issue", item.severity, item.severityReason);
 		const cites = parseCitation(item.evidence);
+		const resolvedBehaviour = asString$1(item.resolvedBehaviour).trim();
+		const consequence = item.consequence === null ? null : normalizeConsequence(item.consequence);
 		findings.push({
 			sourceIndex,
 			status,
 			...note ? { note } : {},
+			...consequence !== void 0 ? { consequence } : {},
+			...resolvedBehaviour && status === "resolved" ? { resolvedBehaviour } : {},
 			...grade ? { grade: {
 				to: grade.to,
 				reason: grade.reason
@@ -18333,6 +18436,7 @@ function parseReconciliationResult(raw, priorCount, options = {}) {
 			recommendedStartingPoint: false
 		})),
 		summary: asString$1(obj.summary).trim(),
+		orientation: asString$1(obj.orientation).trim(),
 		overallRisk: RISKS.includes(obj.overallRisk) ? obj.overallRisk : null,
 		recommendation: asString$1(obj.recommendation).trim(),
 		nextSteps: asStringArray(obj.nextSteps)
@@ -18357,8 +18461,9 @@ function reconcileIssues(prior, findings, record) {
 		const flag = issue.reconciliationStatus === "resolved" ? record?.flags?.get(index) : void 0;
 		if (flag && record) {
 			const entry = reassessedEntry(verdict, flag, record);
+			const { resolvedBehaviour: _reopened, ...reassessed } = withRevisedConsequence(issue, verdict);
 			return {
-				...issue,
+				...reassessed,
 				reconciliationStatus: entry.status,
 				...entry.reason ? { reconciliationNote: entry.reason } : {},
 				statusHistory: appendStatusChange(issue, entry),
@@ -18368,8 +18473,11 @@ function reconcileIssues(prior, findings, record) {
 		if (issue.reconciliationStatus === "resolved") return issue;
 		const status = verdict?.status ?? "still-open";
 		const recorded = record && isStatusChange(issue.reconciliationStatus, status) ? { statusHistory: appendStatusChange(issue, changeEntry(status, verdict, record)) } : {};
+		const fixedInCode = status === "resolved" && !verdict?.evidence && record?.basis !== "conversation";
+		const { resolvedBehaviour: _previous, ...current } = withRevisedConsequence(issue, verdict);
 		return {
-			...issue,
+			...current,
+			...fixedInCode && verdict?.resolvedBehaviour ? { resolvedBehaviour: verdict.resolvedBehaviour } : {},
 			reconciliationStatus: status,
 			...verdict?.note ? { reconciliationNote: verdict.note } : {},
 			...recorded,
@@ -18378,6 +18486,24 @@ function reconcileIssues(prior, findings, record) {
 	});
 }
 /**
+* The issue with its consequence kept, restated or withdrawn, as the update said.
+*
+* Only for an issue that already had one. An update is never the place a historical issue gains
+* an impact summary it was not written with, so an answer for one without is ignored. Silence
+* keeps it: the model was shown it and asked to change it only where it no longer holds.
+*/
+function withRevisedConsequence(issue, verdict) {
+	if (!issue.consequence || verdict?.consequence === void 0) return issue;
+	if (verdict.consequence === null) {
+		const { consequence: _withdrawn, ...rest } = issue;
+		return rest;
+	}
+	return {
+		...issue,
+		consequence: verdict.consequence
+	};
+}
+/**
 * The same defences the findings get, applied to questions.
 *
 * Out-of-range and duplicate indices are dropped rather than allowed to throw, and the first answer
@@ -18571,6 +18697,7 @@ function applyReconciliation(previous, parsed, context) {
 	const composed = {
 		schemaVersion: 1,
 		summary: parsed.summary || previous.summary,
+		...parsed.orientation || previous.orientation ? { orientation: parsed.orientation || previous.orientation } : {},
 		...motivation ? { motivation } : {},
 		overallRisk: parsed.overallRisk ?? previous.overallRisk,
 		recommendation: parsed.recommendation || previous.recommendation,
@@ -21233,4 +21360,4 @@ var MCP_TOOL_NAMES = [
 	"ask_buddy"
 ];
 //#endregion
-export { readAskRequest as $, versionFrom as $a, explanationOf as $i, authorPreview as $n, kissDecisionsOf as $r, recordRuntimeBestEffort as $t, settleQueue as A, isCommitReview as Aa, createWithMembers as Ai, moveIds as An, attemptIdOf as Ar, UpdateAlreadyRunningError as At, humanizeModelId as B, clearAgentPreference as Ba, queueStamp as Bi, existingPaths as Bn, phasesFor as Br, append as Bt, parsePullRequestLink as C, describeAuthorship as Ca, lifecyclePillAction as Ci, createPrState as Cn, followedBranch as Cr, RefreshUnavailableError as Ct, activateQueued as D, originRemote as Da, sectionOf as Di, markAllRead as Dn, fillFileUrlTemplate as Dr, parkedUpdateRefusal as Dt, relativeTime as E, resolveTarget as Ea, parseLifecycle as Ei, listNotifications as En, forgeResolver as Er, mergeUpdateInto as Et, spawnRunner as F, git as Fa, setMembership as Fi, retireNotification as Fn, pruneJobResidue as Fr, prepareReview as Ft, reviewTag as G, AGENT_IDS as Ga, issuesOutstanding as Gi, storedConversation as Gn, isClaimed as Gr, recordTelemetryUploadConsent as Gt, findingRef as H, writeAgentPreference as Ha, doneVerb as Hi, adoptLegacyBaseline as Hn, purposeOf as Hr, record as Ht, applyUpdate as I, refExists as Ia, updateCollections as Ii, toneOf as In, saveJob as Ir, settleReview as It, forgetAskRequest as J, agentIdentityOf as Ja, reviewerDispositions as Ji, isUnchanged as Jn, followedRefName as Jr, BUILD_VERSION as Jt, reviewTagsIn as K, agentById as Ka, questionsOutstanding as Ki, unforeseenFailure as Kn, CLAIM_LEASE_MS as Kr, telemetryUploadConsent as Kt, modelProvenanceOf as L, resolveAgent as La, queueAdd as Li, unreadAround as Ln, PHASES as Lr, reanalyseReview as Lt, RUNNER_MARKER as M, AmbiguousRefError as Ma, nameProblemText as Mi, readNotification as Mn, fail as Mr, runningJobs as Mt, cliEntry as N, shortNameOf as Na, readCollections as Ni, readStateOf as Nn, isTerminal as Nr, failureOf as Nt, compactQueue as O, DetachedHeadError as Oa, collectionsStamp as Oi, markRead as On, RUN_RECORD_RETENTION_MS as Or, updateReview as Ot, runDetached as P, shorthandOf as Pa, renameCollection as Pi, removeNotification as Pn, loadJob as Pr, observeReview as Pt, markAskRunning as Q, runnableAgents as Qa, explanationActionOf as Qi, UNESTABLISHED_MOTIVATION as Qn, kissDecisionRefusal as Qr, mapRuntimes as Qt, provenanceDetailLines as R, resolveModel as Ra, queueAddMany as Ri, unreadCount as Rn, isLiveJob as Rr, runJob as Rt, extractPullRequestLinks as S, objectWithAnyOf as Sa, lifecycleName as Si, createArchitectureNotification as Sn, checkReview as Sr, FETCH_TIMEOUT_MS as St, runBatch as T, kissOutcome as Ta, offersLifecycle as Ti, eventName as Tn, fetchHead as Tr, cancellableHere as Tt, findingRefs as U, CONFIG_PATH as Ua, isIssueOutstanding as Ui, prepareConversationWithin as Un, clearClaim as Ur, stamp as Ut, findingAtRef as V, readAgentPreference as Va, readQueue as Vi, ASK_REREAD_CEILING_MS as Vn, progressSteps as Vr, readEvents as Vt, refOf as W, STORE_ROOT as Wa, isQuestionOutstanding as Wi, saveRefresh as Wn, driverGone as Wr, deliver as Wt, isRequestId as X, isAgentId as Xa, processDiscussion as Xi, EXPLAIN_SIMPLY_PROMPT as Xn, indexKeptIds as Xr, feedbackUrl as Xt, interruption as Y, detectAgents as Ya, verdictClosure as Yi, agentEnvOf as Yn, groupByLineage as Yr, PACKAGE_NAME as Yt, isTerminalAsk as Z, parkedReason as Za, recommendationsOf as Zi, chatConversationSection as Zn, indexReviewGroups as Zr, waitingHistory as Zt, endRun as _, statusLookup as _a, activityAttributes as _i, writeJsonAtomically as _n, requestJobStop as _r, parseWorkspacePort as _t, lineageIds as a, assessmentLookup as aa, mutateWorkspace as ai, loadArchitecture as an, escapeHtml as ar, healthyPort as at, stopRun as b, asString$1 as ba, lifecycleChip as bi, subjectOf as bn, findExistingReview as br, writePortPreference as bt, discardJob as c, findingAssessment as ca, previousKissResult as ci, repositoryKey as cn, ReviewBeingDeletedError as cr, serverEntryExists as ct, startKissJob as d, gradeScale as da, reviewedRepositories as di, trackedFiles as dn, MANAGED_ROOT as dr, stopServer as dt, isExplanationAction as ea, latestKissRun as ei, RUNTIME_WORDS as en, AGENT_RUN_MARKER as eo, hasAuthorContent as er, recordAskAccepted as et, isDriving as f, isGradeFor as fa, summarise as fi, ownersOf as fn, createWorktree as fr, workspaceUrl as ft, completeRun as g, sourcePhrase as ga, LIFECYCLES as gi, isArchitectureId as gn, removeWorktree as gr, MIN_WORKSPACE_PORT as gt, beginRun as h, reassessIn as ha, workspaceStamps as hi, ancestorsOf as hn, readMarker as hr, MAX_WORKSPACE_PORT as ht, deleteReview as i, analysisRefOf as ia, matchingWorkspaceIds as ii, checkArchitecture as in, CODE_COPY_CLASS as ir, ensureServer as it, RUNNER_KINDS as j, resolveBase as ja, deleteCollection as ji, pruneNotifications as jn, endedAt as jr, liveUpdateFor as jt, queueForReview as k, TargetError as ka, createCollection as ki, markReadForReview as kn, allJobs as kr, askCheckout as kt, kissAgentId as l, findingIn as la, recentWorkspaces as li, storeArchitecture as ln, isBusy as lr, spawnServer as lt, RUN_KINDS as m, promptGrade as ma, workspaceRevision as mi, REPOSITORY_SCOPE as mn, isWorktreeHeld as mr, DEFAULT_WORKSPACE_PORT as mt, MCP_TOOL_NAMES as n, ANALYSIS_ONLY as na, lineagePosition as ni, formatDuration as nn, AgentUnavailableError as no, renderMarkdown as nr, bootstrapUrl as nt, liveJobsFor as o, closesFinding as oa, positionInLineage as oi, partFiles as on, renderInlineText as or, readServerRecord as ot, track as p, isOffered as pa, touchWorkspace as pi, QUOTE_LIMITS as pn, holdWorktree as pr, writeServerRecord as pt, endAskRequest as q, agentFor as qa, resolvedByConversation as qi, refreshPrContext as qn, decideKissRecommendation as qr, telemetryUploadEnabled as qt, deleteLineage as r, NO_STATUS as ra, loadWorkspace as ri, ARCHITECTURE_DIR as rn, CODE_BLOCK_CLASS as rr, clearServerRecordOwnedBy as rt, wasBlocked as s, currentGradeIn as sa, presentedKissRunOf as si, readArchitecture as sn, modelHelpLines as sr, reviewsUrl as st, MCP_SERVER_INFO_NAME as t, withExplanation as ta, lineageKey as ti, describeRuntimes as tn, AgentCancelledError as to, GITHUB_MARKDOWN as tr, readIndexToken as tt, recordKissRun as u, findingStatus as ua, reviewedCommit as ui, storedScopes as un, withReviewLock as ur, stopChild as ut, heldRuns as v, wasReassessed as va, activityLabel as vi, announceJob as vn, describeMovement as vr, resolveWorkspacePort as vt, unsupportedReason as w, WITHHELD_REASON as wa, lifecyclePillLabel as wi, createPrUpdate as wn, reviewedShaOf as wr, cancelUpdate as wt, batchIdentity as x, extractJsonText as xa, lifecycleItemLabel as xi, categoryOf as xn, checkCodeFreshness as xr, COMMIT_REVIEW_CANNOT_UPDATE as xt, reportRun as y, followUpFindingId as ya, isLifecycle as yi, branchOf as yn, describeSubject as yr, statedWorkspacePort as yt, agentDisplayName as z, agentIdOf as za, queueRemove as zi, withUsageRecorded as zn, isTerminalJobPhase as zr, startJob as zt };
+export { readAskRequest as $, runnableAgents as $a, explanationOf as $i, authorPreview as $n, kissDecisionsOf as $r, recordRuntimeBestEffort as $t, settleQueue as A, TargetError as Aa, createWithMembers as Ai, moveIds as An, attemptIdOf as Ar, UpdateAlreadyRunningError as At, humanizeModelId as B, agentIdOf as Ba, queueStamp as Bi, existingPaths as Bn, phasesFor as Br, append as Bt, parsePullRequestLink as C, objectWithAnyOf as Ca, lifecyclePillAction as Ci, createPrState as Cn, followedBranch as Cr, RefreshUnavailableError as Ct, activateQueued as D, resolveTarget as Da, sectionOf as Di, markAllRead as Dn, fillFileUrlTemplate as Dr, parkedUpdateRefusal as Dt, relativeTime as E, kissOutcome as Ea, parseLifecycle as Ei, listNotifications as En, forgeResolver as Er, mergeUpdateInto as Et, spawnRunner as F, shorthandOf as Fa, setMembership as Fi, retireNotification as Fn, pruneJobResidue as Fr, prepareReview as Ft, reviewTag as G, STORE_ROOT as Ga, issuesOutstanding as Gi, storedConversation as Gn, isClaimed as Gr, recordTelemetryUploadConsent as Gt, findingRef as H, readAgentPreference as Ha, doneVerb as Hi, adoptLegacyBaseline as Hn, purposeOf as Hr, record as Ht, applyUpdate as I, git as Ia, updateCollections as Ii, toneOf as In, saveJob as Ir, settleReview as It, forgetAskRequest as J, agentFor as Ja, reviewerDispositions as Ji, isUnchanged as Jn, followedRefName as Jr, BUILD_VERSION as Jt, reviewTagsIn as K, AGENT_IDS as Ka, questionsOutstanding as Ki, unforeseenFailure as Kn, CLAIM_LEASE_MS as Kr, telemetryUploadConsent as Kt, modelProvenanceOf as L, refExists as La, queueAdd as Li, unreadAround as Ln, PHASES as Lr, reanalyseReview as Lt, RUNNER_MARKER as M, resolveBase as Ma, nameProblemText as Mi, readNotification as Mn, fail as Mr, runningJobs as Mt, cliEntry as N, AmbiguousRefError as Na, readCollections as Ni, readStateOf as Nn, isTerminal as Nr, failureOf as Nt, compactQueue as O, originRemote as Oa, collectionsStamp as Oi, markRead as On, RUN_RECORD_RETENTION_MS as Or, updateReview as Ot, runDetached as P, shortNameOf as Pa, renameCollection as Pi, removeNotification as Pn, loadJob as Pr, observeReview as Pt, markAskRunning as Q, parkedReason as Qa, explanationActionOf as Qi, UNESTABLISHED_MOTIVATION as Qn, kissDecisionRefusal as Qr, mapRuntimes as Qt, provenanceDetailLines as R, resolveAgent as Ra, queueAddMany as Ri, unreadCount as Rn, isLiveJob as Rr, runJob as Rt, extractPullRequestLinks as S, extractJsonText as Sa, lifecycleName as Si, createArchitectureNotification as Sn, checkReview as Sr, FETCH_TIMEOUT_MS as St, runBatch as T, WITHHELD_REASON as Ta, offersLifecycle as Ti, eventName as Tn, fetchHead as Tr, cancellableHere as Tt, findingRefs as U, writeAgentPreference as Ua, isIssueOutstanding as Ui, prepareConversationWithin as Un, clearClaim as Ur, stamp as Ut, findingAtRef as V, clearAgentPreference as Va, readQueue as Vi, ASK_REREAD_CEILING_MS as Vn, progressSteps as Vr, readEvents as Vt, refOf as W, CONFIG_PATH as Wa, isQuestionOutstanding as Wi, saveRefresh as Wn, driverGone as Wr, deliver as Wt, isRequestId as X, detectAgents as Xa, processDiscussion as Xi, EXPLAIN_SIMPLY_PROMPT as Xn, indexKeptIds as Xr, feedbackUrl as Xt, interruption as Y, agentIdentityOf as Ya, verdictClosure as Yi, agentEnvOf as Yn, groupByLineage as Yr, PACKAGE_NAME as Yt, isTerminalAsk as Z, isAgentId as Za, recommendationsOf as Zi, chatConversationSection as Zn, indexReviewGroups as Zr, waitingHistory as Zt, endRun as _, statusLookup as _a, activityAttributes as _i, writeJsonAtomically as _n, requestJobStop as _r, parseWorkspacePort as _t, lineageIds as a, assessmentLookup as aa, mutateWorkspace as ai, loadArchitecture as an, escapeHtml as ar, healthyPort as at, stopRun as b, SEVERITIES as ba, lifecycleChip as bi, subjectOf as bn, findExistingReview as br, writePortPreference as bt, discardJob as c, findingAssessment as ca, previousKissResult as ci, repositoryKey as cn, ReviewBeingDeletedError as cr, serverEntryExists as ct, startKissJob as d, gradeScale as da, reviewedRepositories as di, trackedFiles as dn, MANAGED_ROOT as dr, stopServer as dt, isExplanationAction as ea, latestKissRun as ei, RUNTIME_WORDS as en, versionFrom as eo, hasAuthorContent as er, recordAskAccepted as et, isDriving as f, isGradeFor as fa, summarise as fi, ownersOf as fn, createWorktree as fr, workspaceUrl as ft, completeRun as g, sourcePhrase as ga, LIFECYCLES as gi, isArchitectureId as gn, removeWorktree as gr, MIN_WORKSPACE_PORT as gt, beginRun as h, reassessIn as ha, workspaceStamps as hi, ancestorsOf as hn, readMarker as hr, MAX_WORKSPACE_PORT as ht, deleteReview as i, analysisRefOf as ia, matchingWorkspaceIds as ii, checkArchitecture as in, CODE_COPY_CLASS as ir, ensureServer as it, RUNNER_KINDS as j, isCommitReview as ja, deleteCollection as ji, pruneNotifications as jn, endedAt as jr, liveUpdateFor as jt, queueForReview as k, DetachedHeadError as ka, createCollection as ki, markReadForReview as kn, allJobs as kr, askCheckout as kt, kissAgentId as l, findingIn as la, recentWorkspaces as li, storeArchitecture as ln, isBusy as lr, spawnServer as lt, RUN_KINDS as m, promptGrade as ma, workspaceRevision as mi, REPOSITORY_SCOPE as mn, isWorktreeHeld as mr, DEFAULT_WORKSPACE_PORT as mt, MCP_TOOL_NAMES as n, ANALYSIS_ONLY as na, lineagePosition as ni, formatDuration as nn, AgentCancelledError as no, renderMarkdown as nr, bootstrapUrl as nt, liveJobsFor as o, closesFinding as oa, positionInLineage as oi, partFiles as on, renderInlineText as or, readServerRecord as ot, track as p, isOffered as pa, touchWorkspace as pi, QUOTE_LIMITS as pn, holdWorktree as pr, writeServerRecord as pt, endAskRequest as q, agentById as qa, resolvedByConversation as qi, refreshPrContext as qn, decideKissRecommendation as qr, telemetryUploadEnabled as qt, deleteLineage as r, NO_STATUS as ra, loadWorkspace as ri, ARCHITECTURE_DIR as rn, AgentUnavailableError as ro, CODE_BLOCK_CLASS as rr, clearServerRecordOwnedBy as rt, wasBlocked as s, currentGradeIn as sa, presentedKissRunOf as si, readArchitecture as sn, modelHelpLines as sr, reviewsUrl as st, MCP_SERVER_INFO_NAME as t, withExplanation as ta, lineageKey as ti, describeRuntimes as tn, AGENT_RUN_MARKER as to, GITHUB_MARKDOWN as tr, readIndexToken as tt, recordKissRun as u, findingStatus as ua, reviewedCommit as ui, storedScopes as un, withReviewLock as ur, stopChild as ut, heldRuns as v, wasReassessed as va, activityLabel as vi, announceJob as vn, describeMovement as vr, resolveWorkspacePort as vt, unsupportedReason as w, describeAuthorship as wa, lifecyclePillLabel as wi, createPrUpdate as wn, reviewedShaOf as wr, cancelUpdate as wt, batchIdentity as x, asString$1 as xa, lifecycleItemLabel as xi, categoryOf as xn, checkCodeFreshness as xr, COMMIT_REVIEW_CANNOT_UPDATE as xt, reportRun as y, followUpFindingId as ya, isLifecycle as yi, branchOf as yn, describeSubject as yr, statedWorkspacePort as yt, agentDisplayName as z, resolveModel as za, queueRemove as zi, withUsageRecorded as zn, isTerminalJobPhase as zr, startJob as zt };