prreviewbuddy 0.90.0 → 0.92.0

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

@@ -4206,6 +4206,18 @@ var REVIEW_ATTENTIONS = [
 	"medium",
 	"low"
 ];
+/** In the order the Issues pane draws them. */
+var IMPACT_GROUPS = [
+	"failure",
+	"risk",
+	"intent",
+	"maintenance"
+];
+var STEP_KINDS = [
+	"verify",
+	"decide",
+	"fix"
+];
 /**
 * The JSON in a reply, whether or not the model wrapped it in a fence.
 *
@@ -4373,10 +4385,71 @@ 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 STEP_KINDS.includes(item.kind) ? {
+		kind: item.kind,
+		text
+	} : { text };
+}
+/**
+* A question's scan-first framing, or nothing.
+*
+* Well-formedness only, the consequence's rule: a framing with no question or no reason has nothing
+* to say and is dropped, so the card shows the original question rather than an empty headline.
+*/
+function normalizeQuestionFraming(value) {
+	if (typeof value !== "object" || value === null || Array.isArray(value)) return void 0;
+	const item = value;
+	const question = asString$1(item.question).trim();
+	const why = asString$1(item.why).trim();
+	if (!question || !why) return void 0;
+	const needed = asString$1(item.needed).trim();
+	return {
+		question,
+		why,
+		...needed ? { needed } : {}
+	};
+}
+function withFraming(framing) {
+	return framing ? { framing } : {};
+}
+/** 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;
@@ -4403,7 +4476,8 @@ function normalizeQuestions(value) {
 			importance: REVIEW_ATTENTIONS.includes(item.importance) ? item.importance : void 0,
 			whyThisMatters: typeof item.whyThisMatters === "string" ? item.whyThisMatters : void 0,
 			file: typeof item.file === "string" ? item.file : void 0,
-			sectionId: asSectionId(item)
+			sectionId: asSectionId(item),
+			...withFraming(normalizeQuestionFraming(item.framing))
 		};
 	}).filter((question) => question.question.trim() !== "");
 }
@@ -5693,6 +5767,20 @@ function reviewerDispositions(session) {
 		isStale: (issue) => issue.reconciliationStatus === "affected" && (byId.get(issue.id)?.sessionId ?? session.id) !== session.id
 	};
 }
+function framableFindings(session) {
+	const dispositions = reviewerDispositions(session);
+	const answered = new Set(session.answeredQuestionIds ?? []);
+	const verdicts = verdictClosure(session);
+	return {
+		issue: (issue) => isIssueOutstanding(issue, dispositions),
+		question: (question) => isQuestionOutstanding(question, answered, verdicts.question)
+	};
+}
+/** For a caller with no session: nothing may gain framing. */
+var NOTHING_FRAMABLE = {
+	issue: () => false,
+	question: () => false
+};
 /**
 * An issue is done when the commits fixed it, or when the reviewer dismissed it and that decision
 * still stands.
@@ -12503,6 +12591,76 @@ 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\" | \"fix\", \"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 specific step this issue calls for, when there is one. "kind" is "verify" for a test or evidence-gathering step that would settle whether it happens (e.g. "Try a receipt with an unreadable amount."), "decide" for an acceptance, requirement or intent call only the reviewer can make (e.g. "Confirm this broader change is intended."), and "fix" for a concrete corrective action the finding itself supports (e.g. "Update the recovery runbook to name the new restore command."). A fix is not a verification: do not label a remedy "verify". Omit "next" rather than write a generic step or invent a remedy the evidence does not support.
+  - "group", by the consequence and the evidence behind it, never by the kind of file, never by how sure you sound and independently of "severity":
+    - "failure" when the evidence you read establishes an incorrect outcome under a stated condition (e.g. "Saving a meal with no ingredients deletes the existing ones.").
+    - "risk" when a plausible adverse consequence still needs verifying (e.g. "Two people editing the same plan at once may overwrite each other; nothing shows whether saves are serialised."), and say what is unverified.
+    - "intent" when there is a concrete behavioural consequence whose acceptability depends on requirements or intent (e.g. "Rotating the sign-in secret now signs every customer out.").
+    - "maintenance" when it constrains future development, maintainability or reliability and no concrete failure or risk fits better (e.g. "Each new payment provider has to be added in four places that must stay in step.").
+    - Documentation, runbooks, configuration and scripts are classified by what they cause like any code: recovery instructions that restore to the wrong database cause an operational failure when the text establishes it, or a risk when unverified. They are not "maintenance" because they are documentation.
+    - "confidence" is reported separately and does not decide the group. A well-evidenced failure stays a failure at medium confidence; a high severity does not make an unverified risk a failure; a low confidence does not by itself turn a failure into a risk.
+  - 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.
+A finding marked [consequence: none yet, write one] is still outstanding and has no impact summary: unless you mark it "resolved", return a "consequence" for it, written from the evidence its description already gives together with anything these changes show. Classify it into whichever of the four groups that evidence supports; do not default to "risk", and do not demand fresh proof to keep a failure the existing evidence already establishes. Keep its uncertainty as stated: rewording a finding is not re-verifying it, so never describe it as newly confirmed or tested. Never return "consequence" for a finding with neither 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 is marked
+* as needing one only when the update may add it (`framable`, an outstanding finding); otherwise it
+* shows nothing, which is what tells the model not to add one.
+*/
+function formatPriorConsequence(issue, framable = false) {
+	const consequence = issue.consequence;
+	if (!consequence) return framable ? "\n   [consequence: none yet, write one]" : "";
+	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}`;
+}
+/** A question's scan-first framing, as every prompt that asks questions asks for it. The whole shape. */
+var QUESTION_FRAMING_FIELD = "\"framing\": { \"question\": string, \"why\": string, \"needed\"?: string }";
+/** The same shape on a prior question's entry in an update, where null withdraws it and absence keeps it. */
+var QUESTION_FRAMING_REVISION_FIELD = `${QUESTION_FRAMING_FIELD.replace("\"framing\": ", "\"framing\"?: ")} | null`;
+var QUESTION_FRAMING_GUIDANCE = `"framing" on a question: the same question, asked so a reviewer who knows the product but not the code can see at a glance what is being asked and why. It is shown first; the original "question" and "whyThisMatters" stay the technical account behind it. It is a presentation of this question, never a second one.
+  - "question": short, at product or operations level (e.g. "Should rotating the sign-in secret require all customers to sign in again?").
+  - "why": one or two sentences on why the answer matters, keeping every condition and uncertainty (e.g. "The current encryption ties stored sessions to that secret."). A conditional premise stays conditional: do not turn "if" into a fact.
+  - "needed": the specific answer or confirmation required, when the question does not already make it plain (e.g. "Confirm whether this consequence is acceptable and documented."). Omit it otherwise.
+  - A question with several parts names every part, so none is lost from view. Do not answer the question in its framing.`;
+var QUESTION_FRAMING_UPDATE_GUIDANCE = `"framing" on a question entry: a question marked [framing: ...] has a short framing the reviewer reads first. Omit "framing" when it still describes the question accurately. Return a restated "framing" (same shape as on a new question) when these changes alter what is being asked, its condition or what is needed; in particular, when you mark a question "answered" or the changes answer part of it, restate it to name only what is still open, or return null if nothing in it still needs asking. Return null when it no longer holds and you cannot restate it.
+A question marked [framing: none yet, write one] is still outstanding and has no framing: unless you mark it "answered", return a "framing" for it from what the question and its reasons already say together with anything these changes show, keeping its conditions and uncertainty. Never return "framing" for a question with neither marker.`;
+/** A prior question's framing, or the marker asking for one, as the update shows it. */
+function formatPriorFraming(question, framable = false) {
+	const framing = question.framing;
+	if (!framing) return framable ? "\n   [framing: none yet, write one]" : "";
+	const needed = framing.needed ? ` Needed: ${framing.needed.trim()}` : "";
+	return `\n   [framing] ${framing.question.trim()} ${framing.why.trim()}${needed}`;
+}
+//#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 +13423,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,8 +13438,8 @@ 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" } }],
-  "questions": [{ "id": string, "question": string, "importance"?: "high" | "medium" | "low", "whyThisMatters"?: string, "file"?: string, "sectionId"?: string }],
+  "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, ${QUESTION_FRAMING_FIELD}, "file"?: string, "sectionId"?: string }],
   "recommendation": string,
   "nextSteps": string[]
 }
@@ -13299,6 +13458,9 @@ 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}
+- ${QUESTION_FRAMING_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 +13518,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),
@@ -17995,13 +18158,13 @@ function priorFindingVerdict(issue, flag) {
 * reviewer may have moved it, and a model told only the analysis's grade would re-judge from where
 * the review no longer stands. A caller with no ledger passes `ANALYSIS_ONLY`.
 */
-function formatPriorFindings(issues, assess, reassess = /* @__PURE__ */ new Map(), status = NO_STATUS) {
+function formatPriorFindings(issues, assess, reassess = /* @__PURE__ */ new Map(), status = NO_STATUS, framable = NOTHING_FRAMABLE) {
 	if (!issues.length) return "(The previous analysis raised no issues.)";
 	return issues.map((issue, index) => {
 		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, framable.issue(issue))}`;
 	}).join("\n");
 }
 /**
@@ -18010,10 +18173,10 @@ function formatPriorFindings(issues, assess, reassess = /* @__PURE__ */ new Map(
 * of the issue shape returned issues with no severity, no file and in some cases no content.
 * Keep in step with STRUCTURED_OUTPUT_INSTRUCTIONS in pipeline.ts.
 */
-var QUESTION_SHAPE_LINE = "  \"questions\": [{ \"sourceIndex\": number, \"status\": \"answered\" | \"still-open\" | \"affected\", \"answer\"?: string, \"note\"?: string, \"importance\"?: \"high\" | \"medium\" | \"low\", \"importanceReason\"?: string }],";
+var QUESTION_SHAPE_LINE = `  "questions": [{ "sourceIndex": number, "status": "answered" | "still-open" | "affected", "answer"?: string, "note"?: string, ${QUESTION_FRAMING_REVISION_FIELD}, "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 +18191,7 @@ var RECONCILIATION_SHAPE = `{
     "reviewWeight": 1 | 2 | 3 | 4 | 5
   }],
   "summary": string,
+  "orientation": string,
   "overallRisk": "low" | "medium" | "high",
   "recommendation": string,
   "nextSteps": string[]
@@ -18055,7 +18219,7 @@ function reconciliationShape(includeQuestions, withDiscussion = false) {
 * question's own id is withheld so it cannot be echoed back in place of an index, and enough
 * context travels for a judgement without inviting a rewrite of the question itself.
 */
-function formatPriorQuestions(questions, assess, status = NO_STATUS) {
+function formatPriorQuestions(questions, assess, status = NO_STATUS, framable = NOTHING_FRAMABLE) {
 	if (!questions.length) return "(The previous analysis raised no questions.)";
 	return questions.map((question, index) => {
 		const related = (question.relatedFiles ?? []).map((file) => ` [related: ${file}, not changed]`).join("");
@@ -18064,7 +18228,7 @@ function formatPriorQuestions(questions, assess, status = NO_STATUS) {
 		const verdict = alreadyAnswered(question) ? " [previously: answered]" : priorVerdict(question);
 		const assessment = assess("question", question);
 		const importance = assessment.current ? ` [importance: ${promptGrade(assessment)}]` : "";
-		return `${index}.${location}${verdict}${acceptedVerdict(status("question", question.id))}${importance} ${question.question.trim()}${why ? `\n   ${why}` : ""}`;
+		return `${index}.${location}${verdict}${acceptedVerdict(status("question", question.id))}${importance} ${question.question.trim()}${why ? `\n   ${why}` : ""}${formatPriorFraming(question, framable.question(question))}`;
 	}).join("\n");
 }
 /**
@@ -18145,7 +18309,7 @@ function buildReconciliationInstructions(input, options = {}) {
 	const discussion = options.discussion;
 	const conversationOnly = discussion !== void 0 && !discussion.codeChanged;
 	const truncationWarning = possiblyTruncated ? `\n\nIMPORTANT: the list of changes above is INCOMPLETE — more files changed than could be retrieved. Do not mark a previous finding "resolved" on the strength of absence: a file you cannot see may or may not contain the fix. Prefer "still-open" or "affected" where you cannot see the relevant code, and say so in the note.` : "";
-	const questionList = includeQuestions ? `\n\nPrevious open questions, numbered — refer to each by its number in "sourceIndex":\n${formatPriorQuestions(previous.questions, input.assess, input.status)}` : "";
+	const questionList = includeQuestions ? `\n\nPrevious open questions, numbered — refer to each by its number in "sourceIndex":\n${formatPriorQuestions(previous.questions, input.assess, input.status, input.framable)}` : "";
 	const tasks = [
 		`${discussion ? `For EVERY numbered finding above, decide what ${conversationOnly ? "the changes in the conversation" : "the new commits and the changes in the conversation"} did to it:
    - "resolved": ${conversationOnly ? "" : "the new commits fix it, and you can point at the change that does so; or "}the conversation gives evidence that invalidates its premise, under the rules below.
@@ -18168,7 +18332,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.
 
@@ -18183,7 +18347,7 @@ Overall risk: ${previous.overallRisk}
 Recommendation: ${previous.recommendation}
 
 Previous findings, numbered — refer to each by its number in "sourceIndex":
-${formatPriorFindings(previous.issues, input.assess, reassess, input.status)}${questionList}${truncationWarning}
+${formatPriorFindings(previous.issues, input.assess, reassess, input.status, input.framable)}${questionList}${truncationWarning}
 ${discussion ? `\n${DISCUSSION_RULES}\n` : ""}
 YOUR TASK
 
@@ -18194,6 +18358,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}
+${includeQuestions ? `\n${QUESTION_FRAMING_UPDATE_GUIDANCE}\n\n${QUESTION_FRAMING_GUIDANCE}\n` : ""}
 "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 +18480,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 +18505,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)
@@ -18350,15 +18523,16 @@ function parseReconciliationResult(raw, priorCount, options = {}) {
 * Every prior finding therefore survives into the new result, in its original order, carrying its
 * original id, severity and text. Only the status is added.
 */
-function reconcileIssues(prior, findings, record) {
+function reconcileIssues(prior, findings, record, framable = NOTHING_FRAMABLE) {
 	const byIndex = new Map(findings.map((f) => [f.sourceIndex, f]));
 	return prior.map((issue, index) => {
 		const verdict = byIndex.get(index);
 		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, false);
 			return {
-				...issue,
+				...reassessed,
 				reconciliationStatus: entry.status,
 				...entry.reason ? { reconciliationNote: entry.reason } : {},
 				statusHistory: appendStatusChange(issue, entry),
@@ -18368,16 +18542,56 @@ 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, status !== "resolved" && framable.issue(issue));
 		return {
-			...issue,
+			...withNote(current, verdict?.note, isStatusChange(issue.reconciliationStatus, status)),
+			...fixedInCode && verdict?.resolvedBehaviour ? { resolvedBehaviour: verdict.resolvedBehaviour } : {},
 			reconciliationStatus: status,
-			...verdict?.note ? { reconciliationNote: verdict.note } : {},
 			...recorded,
 			...record?.against ? { reconciledAgainst: verdict?.evidence ? "conversation" : record.against } : {}
 		};
 	});
 }
 /**
+* The finding with the note this round gave, or the one it had. A note argues for the status it was
+* written with, so silence keeps it only while that status holds: one this round changed leaves it
+* behind, where it would contradict the new status on the card.
+*/
+function withNote(finding, note, statusChanged) {
+	if (note) return {
+		...finding,
+		reconciliationNote: note
+	};
+	if (!statusChanged) return finding;
+	const { reconciliationNote: _superseded, ...rest } = finding;
+	return rest;
+}
+/**
+* The issue with its consequence kept, restated, withdrawn or added, as the update said.
+*
+* Silence keeps it: the model was shown it and asked to change it only where it no longer holds.
+* An issue stored without one gains the one offered only when `mayFrame`: it is still outstanding
+* after this update, so the card the reviewer is about to work from becomes scan-first. A resolved,
+* dismissed or verdict-closed issue is never given presentation it was not written with, and a
+* `null` for an issue that has nothing to withdraw changes nothing.
+*/
+function withRevisedConsequence(issue, verdict, mayFrame) {
+	if (verdict?.consequence === void 0) return issue;
+	if (!issue.consequence) return mayFrame && verdict.consequence ? {
+		...issue,
+		consequence: verdict.consequence
+	} : 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
@@ -18400,10 +18614,12 @@ function parseQuestions(value, priorCount) {
 		const note = asString$1(item.note).trim();
 		const answer = asString$1(item.answer).trim();
 		const grade = parseGrade("question", item.importance, item.importanceReason);
+		const framing = item.framing === null ? null : normalizeQuestionFraming(item.framing);
 		questions.push({
 			sourceIndex,
 			status,
 			...note ? { note } : {},
+			...framing !== void 0 ? { framing } : {},
 			...status === "answered" && answer ? { answer } : {},
 			...grade ? { grade: {
 				to: grade.to,
@@ -18438,7 +18654,7 @@ function parseQuestions(value, priorCount) {
 function alreadyAnswered(question) {
 	return question.reconciliationStatus === "answered" || question.reconciliationAnswer !== void 0;
 }
-function reconcileQuestions(prior, reconciled, against) {
+function reconcileQuestions(prior, reconciled, against, framable = NOTHING_FRAMABLE) {
 	const recorded = against ? { reconciledAgainst: against } : {};
 	const byIndex = new Map(reconciled.map((entry) => [entry.sourceIndex, entry]));
 	return prior.map((question, index) => {
@@ -18454,15 +18670,35 @@ function reconcileQuestions(prior, reconciled, against) {
 		};
 		const answered = verdict.status === "answered" && verdict.answer !== void 0;
 		const status = verdict.status !== "answered" ? verdict.status : answered ? "answered" : "affected";
+		const changed = (question.reconciliationStatus ?? "still-open") !== status;
 		return {
-			...question,
+			...withNote(withRevisedFraming(question, verdict, status !== "answered" && framable.question(question)), verdict.note, changed),
 			reconciliationStatus: status,
-			...verdict.note ? { reconciliationNote: verdict.note } : {},
 			...answered ? { reconciliationAnswer: verdict.answer } : {},
 			...recorded
 		};
 	});
 }
+/**
+* The question with its framing kept, restated, withdrawn or added: `withRevisedConsequence`'s rule.
+* A framed question is revised whatever its new status, because an answer is exactly what can make
+* its "needed" line stale; an unframed one gains a framing only while it is still outstanding.
+*/
+function withRevisedFraming(question, verdict, mayFrame) {
+	if (verdict.framing === void 0) return question;
+	if (!question.framing) return mayFrame && verdict.framing ? {
+		...question,
+		framing: verdict.framing
+	} : question;
+	if (verdict.framing === null) {
+		const { framing: _withdrawn, ...rest } = question;
+		return rest;
+	}
+	return {
+		...question,
+		framing: verdict.framing
+	};
+}
 /** Marks a reconciliation's own findings as new, so the UI can tell them from carried-forward ones. */
 function markNewIssues(issues, against) {
 	return issues.map((issue) => ({
@@ -18565,12 +18801,13 @@ function applyReconciliation(previous, parsed, context) {
 		basis: context.codeChanged ? "code" : "conversation",
 		flags,
 		against
-	}), ...markNewIssues(parsed.newIssues, against)];
-	const questions = reconcileQuestions(previous.questions, parsed.questions, against);
+	}, context.framable), ...markNewIssues(parsed.newIssues, against)];
+	const questions = reconcileQuestions(previous.questions, parsed.questions, against, context.framable);
 	const { motivation } = 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,
@@ -19005,7 +19242,8 @@ async function reconcile(workspace, runners, reviewedSha, remote, conversationOk
 		codeChanged: delta !== null,
 		conversation: prepared,
 		assess: assessmentLookup(workspace.session),
-		motivation
+		motivation,
+		framable: framableFindings(workspace.session)
 	});
 	signal.throwIfAborted();
 	cancellers.delete(jobId);
@@ -19514,6 +19752,7 @@ ${buildReconciliationInstructions({
 		previous,
 		assess: assessmentLookup(workspace.session),
 		status: statusLookup(workspace.session),
+		framable: framableFindings(workspace.session),
 		fromSha: reviewedSha,
 		toSha: newHead,
 		totalCommits: commits,
@@ -21233,4 +21472,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 };