prreviewbuddy 0.95.2 → 0.96.2

package/dist/{mcp_tools-DDbcXys2.js → mcp_tools-DmQKsAkC.js}

@@ -851,6 +851,12 @@ var CLAUDE_AGENT_ID = "claude";
 * about sign-in to errors that have nothing to do with it, which is worse than staying quiet.
 */
 var AUTHENTICATION$2 = /\b(?:auth(?:enticat\w*)?|oauth|credential|session expired|not logged in|log ?in)\b/i;
+/**
+* What Claude Code prints, and exits 1 on, for an argument it does not know: `error: unknown option
+* '--setting-sources'` (seen on 2.1.318 with a made-up flag). Every argument comes from
+* `claudeArgs`, so this means the installed CLI is older than one of them.
+*/
+var UNKNOWN_OPTION = /unknown option '--[\w-]+'/i;
 /** Said to the caller that asked for the stop, and never shown to the reviewer who pressed it. */
 var STOPPED$2 = "The agent was stopped.";
 /** How long a stopped agent gets to end on its own terms before it is ended for it. */
@@ -1060,6 +1066,17 @@ var VERSION_TIMEOUT_MS$2 = 1e4;
 * `--strict-mcp-config`, with no `--mcp-config` beside it, is the containment clause of the agent
 * contract as Claude satisfies it. It tells the CLI to load no MCP servers at all, so `prb_review`
 * is not there to be reached no matter which `CLAUDE_CONFIG_DIR` the spawn inherits.
+*
+* `--setting-sources user` is the other half. The run stands in a worktree of the change under
+* review, and by default the CLI also loads that worktree's `.claude/settings.json` and
+* `settings.local.json`: settings written by the change's author. Their hooks run as the reviewer,
+* before the model sees anything and outside `--allowedTools`, and their permissions widen the tool
+* policy. Observed on Claude Code 2.1.318 with a canary `SessionStart` hook: it ran under the
+* arguments above without this flag and not with it, while the reviewer's own hooks ran both times.
+* The same flag keeps out the worktree's `CLAUDE.md`, `.claude/agents` and `.claude/skills`, which
+* reach the model as instructions: a canary of each was absent from the `init` event and unknown to
+* the model under these arguments (2026-11-08), while the reviewer's own agents and skills remained.
+* `--bare` would also skip them, but it skips the reviewer's sign-in too.
 */
 function claudeArgs(capability, requestedModel) {
 	return [
@@ -1070,6 +1087,8 @@ function claudeArgs(capability, requestedModel) {
 		"--allowedTools",
 		toolsFor(capability),
 		"--strict-mcp-config",
+		"--setting-sources",
+		"user",
 		...requestedModel ? ["--model", requestedModel] : []
 	];
 }
@@ -1109,6 +1128,7 @@ var claudeAgent = {
 	* situation nobody is in until they have tried the obvious thing first.
 	*/
 	explainFailure(detail, env = {}) {
+		if (UNKNOWN_OPTION.test(detail)) return "This Claude Code is too old for PR Review Buddy. Update it with `claude update`, then try again.";
 		if (!AUTHENTICATION$2.test(detail)) return null;
 		return `Claude Code could not sign in. This review was started using ${env.CLAUDE_CONFIG_DIR ? `the Claude profile at \`${env.CLAUDE_CONFIG_DIR}\`` : "your default Claude profile"}, so make sure that one is signed in.`;
 	},
@@ -1653,14 +1673,25 @@ var codexMcpRegistration = {
 * `--no-mcp`. Two things were established by running read-only commands rather than by reading
 * documentation, and they are the reason this adapter exists at all:
 *
-*   - MCP servers resolve only from `$CODEX_HOME/config.toml`, which `--ignore-user-config` drops
-*     while leaving the reviewer's login working. So containment costs no credential.
+*   - Configured MCP servers resolve from `$CODEX_HOME/config.toml`, which `--ignore-user-config`
+*     drops while leaving the reviewer's login working. So containment costs no credential. Plugins
+*     are the exception: a remote plugin's server (Notion, from the plugin cache) still started under
+*     it on codex-cli 0.157.0, and only `--disable plugins` stopped it. Connected apps are the other:
+*     the reviewer's ChatGPT apps arrive as a built-in `codex_apps` server (GitHub with
+*     `merge_pull_request`, Google Drive with `share_file`, a web search) whose calls run on OpenAI's
+*     side, where the read-only sandbox does not reach. `--disable apps` removes all of them.
 *   - A repository's own `.codex/config.toml` contributes nothing. This was the open question, and
 *     it mattered because a review runs in a worktree cut from the repository under review: a
 *     project-level server would have been an untrusted repository injecting a tool into a run that
 *     is supposed to have none. A canary server declared in project config does not appear in
 *     `codex mcp list` run from that repository.
 *
+* The reviewed repository's `AGENTS.md` is kept out with `-c project_doc_max_bytes=0`, for the same
+* reason the Claude adapter keeps out `CLAUDE.md`: the author of the change can edit it in the same
+* change. `-c web_search="disabled"` removes the web tool, which a read-only review has no use for.
+* The repository's `.codex/` config and hooks need no flag: with user config ignored the repository
+* is untrusted, so its project layer is never loaded (verified 2026-11-08 with a canary repository).
+*
 * `-c mcp_servers={}` is deliberately not used: the override layer is deep-merged, so an empty
 * table iterates zero keys and leaves the configured servers exactly as they were. It looks like
 * containment and is not.
@@ -1682,6 +1713,39 @@ var STOPPED = "The run was stopped.";
 */
 var CARRIED = ["CODEX_HOME", "PATH"];
 var VERSION_TIMEOUT_MS = 1e4;
+/**
+* The oldest Codex a review will start on.
+*
+* `-c web_search="disabled"` is what keeps a review off the web, and Codex ignores a `-c` key it
+* does not know (exit 0 on 0.157.0), so on a Codex older than the key the switch would do nothing
+* and say nothing. This is the oldest release where the switch is known to work: 0.122.0 is also
+* the first with `--ignore-user-config` and `--ignore-rules` (#18646), so nothing older could run
+* a review anyway. Established 2026-11-08 from the openai/codex source at each tag:
+*
+*   - the hosted web tool is built only for `Cached` or `Live` (`create_web_search_tool` returns
+*     `None` for `Disabled` at 0.122.0; `hosted_spec.rs` at 0.134.0), and
+*   - the standalone web search extension, from 0.135.0, is enabled only when the mode is not
+*     `Disabled` (checked at 0.135, 0.140, 0.145, 0.150 and 0.157).
+*
+* And on 0.157.0 by the tools the Responses API echoed back: `web.run` and `web__run` without the
+* switch, neither with it. Older macOS builds could not be run to repeat that: every release before
+* 0.132.0 is signed with a revoked certificate (0.131.0 is not notarized) and macOS refuses it.
+*/
+var MINIMUM_CODEX_VERSION = "0.122.0";
+/**
+* Why a review will not start on this Codex, or null when it will.
+*
+* An unreadable version is refused rather than assumed, and a pre-release of the minimum counts as
+* older than it: neither is known to have what the release has.
+*/
+function codexVersionRefusal(version) {
+	const update = "Update it with `codex update`, or with the package manager you installed it from if that command is missing, then try again.";
+	const parsed = version ? /^(\d+)\.(\d+)\.(\d+)(.*)$/.exec(version) : null;
+	if (!parsed) return `PR Review Buddy could not tell which Codex is installed, and needs Codex ${MINIMUM_CODEX_VERSION} or newer. ${update}`;
+	const have = parsed.slice(1, 4).map(Number);
+	const need = MINIMUM_CODEX_VERSION.split(".").map(Number);
+	return (have.map((part, i) => part - need[i]).find((diff) => diff !== 0) ?? (parsed[4].startsWith("-") ? -1 : 0)) < 0 ? `Codex ${version} is too old for PR Review Buddy, which needs Codex ${MINIMUM_CODEX_VERSION} or newer. ${update}` : null;
+}
 var DEFAULT_TIMEOUT_MS = { "read-code": 12e5 };
 function codexArgs(_capability, requestedModel) {
 	return [
@@ -1692,6 +1756,14 @@ function codexArgs(_capability, requestedModel) {
 		"read-only",
 		"--ignore-user-config",
 		"--ignore-rules",
+		"--disable",
+		"plugins",
+		"--disable",
+		"apps",
+		"-c",
+		"project_doc_max_bytes=0",
+		"-c",
+		"web_search=\"disabled\"",
 		"--ephemeral",
 		...requestedModel ? ["-m", requestedModel] : []
 	];
@@ -1822,6 +1894,14 @@ var AUTHENTICATION = /not (?:logged|signed) in|unauthori[sz]ed|401|authenticat|c
 * authentication failure, so no Retry is offered.
 */
 var REJECTED_KEY = /incorrect api key|invalid[_ ]api[_ ]key/i;
+/**
+* A Codex older than a switch a review passes: an unknown feature name (`Unknown feature flag:
+* <name>` on 0.157.0) or, older still, an option it has never heard of. Fixed by updating, so it is
+* said as that. `codex update` exists on 0.157.0, but the Codex reading this is older by definition
+* and may predate it, so the package manager is named as the fallback. Neither refusal reads as a sign-in failure, so it stays an execution failure and
+* Retry is there after the update.
+*/
+var TOO_OLD = /unknown feature flag|unexpected argument '--[\w-]+'/i;
 function isAuthentication(detail) {
 	return AUTHENTICATION.test(detail) || REJECTED_KEY.test(detail);
 }
@@ -1845,6 +1925,7 @@ var codexAgent = {
 	mcpRegistration: codexMcpRegistration,
 	/** Which tool, which profile, and the one command that changes it. See `agent.ts` for the budget. */
 	explainFailure(detail, env = {}) {
+		if (TOO_OLD.test(detail)) return "This Codex is too old for PR Review Buddy. Update it with `codex update`, or with the package manager you installed it from if that command is missing, then try again.";
 		if (!isAuthentication(detail)) return null;
 		const profile = env.CODEX_HOME ? `the Codex profile at \`${env.CODEX_HOME}\`` : "your default Codex profile";
 		if (REJECTED_KEY.test(detail)) return `OpenAI refused a request from ${profile} even though Codex reports being signed in, which usually points to Codex's own setup rather than your sign-in. Run \`codex doctor\` to see what it is using, and check \`codex features list\` for an experimental feature to turn off with \`codex features disable <name>\`.`;
@@ -1864,11 +1945,20 @@ var codexAgent = {
 				encoding: "utf8",
 				env: spawnEnv(env)
 			}, (error, stdout) => {
-				if (!error) return resolvePromise({
-					ok: true,
-					version: versionFrom(stdout),
-					path: whichOn("codex", spawnEnv(env).PATH)
-				});
+				if (!error) {
+					const version = versionFrom(stdout);
+					const refusal = codexVersionRefusal(version);
+					if (refusal) return resolvePromise({
+						ok: false,
+						found: true,
+						reason: refusal
+					});
+					return resolvePromise({
+						ok: true,
+						version,
+						path: whichOn("codex", spawnEnv(env).PATH)
+					});
+				}
 				const found = error.code !== "ENOENT";
 				resolvePromise({
 					ok: false,
@@ -1881,6 +1971,9 @@ var codexAgent = {
 	async run(input) {
 		guardPrompt(input);
 		if (input.signal?.aborted) throw new AgentCancelledError(STOPPED);
+		const installed = await codexAgent.available(input.env);
+		if (!installed.ok) throw new CodexUnavailableError(installed.reason);
+		if (input.signal?.aborted) throw new AgentCancelledError(STOPPED);
 		const reader = assemble$2(input.onProgress ?? (() => {}));
 		return new Promise((resolvePromise, reject) => {
 			const child = spawn("codex", codexArgs(input.capability, input.requestedModel), {
@@ -6556,6 +6649,159 @@ function lifecycleItemLabel(target, lifecycle, activity) {
 	return lifecycleAction(target);
 }
 //#endregion
+//#region ../../packages/review-harness/src/workspace/reviewer_outcome.ts
+var REVIEWER_OUTCOMES = [
+	{
+		id: "no-concerns",
+		name: "Reviewed: no concerns",
+		baseline: true
+	},
+	{
+		id: "needs-changes",
+		name: "Needs changes",
+		baseline: true
+	},
+	{
+		id: "needs-decision",
+		name: "Needs a decision / answer",
+		baseline: true
+	},
+	{
+		id: "deeper-review",
+		name: "Needs deeper review",
+		baseline: false
+	},
+	{
+		id: "later",
+		name: "Later",
+		baseline: false
+	}
+];
+function parseOutcomeSnapshot(value) {
+	if (typeof value !== "object" || value === null) return null;
+	const raw = value;
+	if (typeof raw.sessionId !== "string" || typeof raw.commitSha !== "string" || typeof raw.analysisRevision !== "string") return null;
+	return {
+		sessionId: raw.sessionId,
+		commitSha: raw.commitSha,
+		analysisRevision: raw.analysisRevision
+	};
+}
+function outcomeDefinition(kind) {
+	return REVIEWER_OUTCOMES.find((outcome) => outcome.id === kind);
+}
+function isBaselineKind(kind) {
+	return outcomeDefinition(kind)?.baseline === true;
+}
+function parseStoredOutcome(value) {
+	if (typeof value !== "object" || value === null) return void 0;
+	const raw = value;
+	if (typeof raw.kind !== "string" || !outcomeDefinition(raw.kind)) return void 0;
+	if (typeof raw.at !== "number" || !Number.isFinite(raw.at)) return void 0;
+	if (typeof raw.commitSha !== "string" || typeof raw.sessionId !== "string") return void 0;
+	return {
+		kind: raw.kind,
+		at: raw.at,
+		commitSha: raw.commitSha,
+		sessionId: raw.sessionId,
+		analysisRevision: typeof raw.analysisRevision === "string" ? raw.analysisRevision : "",
+		...typeof raw.recordId === "string" ? { recordId: raw.recordId } : {},
+		...typeof raw.analysisJobId === "string" ? { analysisJobId: raw.analysisJobId } : {},
+		...Array.isArray(raw.answeredQuestions) ? { answeredQuestions: raw.answeredQuestions.filter((q) => typeof q === "string") } : {}
+	};
+}
+function parseOutcomes(value) {
+	if (!Array.isArray(value)) return [];
+	return value.map(parseStoredOutcome).filter((outcome) => outcome !== void 0);
+}
+/** Newest last, capped. The baseline lives in its own field, so nothing here has to survive pruning. */
+function appendOutcome(history, outcome) {
+	return [...history, outcome].slice(-20);
+}
+/** The newer of two baselines; a non-baseline outcome never replaces one. */
+function nextBaseline(current, outcome) {
+	if (!isBaselineKind(outcome.kind)) return current;
+	return !current || outcome.at >= current.at ? outcome : current;
+}
+/** The newest baseline across a lineage's records, so a re-analysis that made a new record keeps it. */
+function lastBaseline(baselines) {
+	let newest;
+	for (const baseline of baselines) if (baseline && (!newest || baseline.at > newest.at)) newest = baseline;
+	return newest;
+}
+function sameSha(a, b) {
+	if (!a || !b) return false;
+	const length = Math.min(a.length, b.length);
+	return length >= 7 && a.slice(0, length) === b.slice(0, length);
+}
+/** How an outcome relates to the analysis on screen. */
+function outcomeStanding(outcome, snapshot) {
+	if (!sameSha(outcome.commitSha, snapshot.commitSha)) return "earlier-snapshot";
+	if (outcome.sessionId !== snapshot.sessionId || outcome.analysisRevision !== snapshot.analysisRevision) return "earlier-analysis";
+	return "current";
+}
+/**
+* Verdicts come from the history and from the stored baseline, which is never pruned: twenty
+* deferrals after a verdict push it out of the history, but not out of the reading.
+*/
+function readOutcomes(history, snapshot, baseline) {
+	const verdicts = history.filter((outcome) => isBaselineKind(outcome.kind));
+	if (baseline && isBaselineKind(baseline.kind) && !verdicts.some((outcome) => outcome.at === baseline.at && outcome.kind === baseline.kind)) verdicts.push(baseline);
+	verdicts.sort((a, b) => a.at - b.at);
+	const newestVerdict = verdicts.at(-1);
+	if (!snapshot) return newestVerdict ? { previous: {
+		outcome: newestVerdict,
+		standing: "earlier-snapshot"
+	} } : {};
+	const verdict = verdicts.filter((outcome) => outcomeStanding(outcome, snapshot) === "current").at(-1);
+	const deferral = history.filter((outcome) => !isBaselineKind(outcome.kind) && outcomeStanding(outcome, snapshot) === "current" && (!verdict || outcome.at > verdict.at)).at(-1);
+	const reading = {};
+	if (verdict) reading.verdict = verdict;
+	else if (newestVerdict) reading.previous = {
+		outcome: newestVerdict,
+		standing: outcomeStanding(newestVerdict, snapshot)
+	};
+	if (deferral) reading.deferral = deferral;
+	return reading;
+}
+/** "Needs changes", or how an older one reads beside a newer analysis. */
+function outcomeLabel(outcome, standing) {
+	const name = outcomeDefinition(outcome.kind)?.name ?? outcome.kind;
+	if (standing === "current") return name;
+	if (standing === "earlier-analysis") return `Outcome on an earlier analysis: ${name}`;
+	return `${name} on ${outcome.commitSha.slice(0, 7)} (earlier snapshot)`;
+}
+//#endregion
+//#region ../../packages/review-harness/src/workspace/analysis_revision.ts
+/**
+* A short fingerprint of the analysis a reviewer is reading: its findings, statuses, notes and text.
+*
+* The session id and the commit do not move when a Re-check reconciles findings at the same commit,
+* so a verdict checked against those two alone would be accepted for findings the reviewer never saw.
+* The reviewer's own ticks are not part of it: those are the reviewer's acts, not the analysis.
+*/
+function analysisRevision(workspace) {
+	const result = workspace.session.result;
+	if (!result) return "";
+	return createHash("sha256").update(JSON.stringify(result)).digest("hex").slice(0, 16);
+}
+/** The analysed commit, falling back the way `reviewedCommit` does. */
+function analysedCommit(workspace) {
+	return workspace.session.metadata?.commitSha || workspace.changeSet.data.pullRequest.head.sha || "";
+}
+function analysisSnapshot(workspace) {
+	if (!workspace.session.result) return null;
+	return {
+		sessionId: workspace.session.id,
+		commitSha: analysedCommit(workspace),
+		analysisRevision: analysisRevision(workspace)
+	};
+}
+/** When the analysis on screen was last produced or reconciled: the upper time bound for its history. */
+function analysedAt(workspace) {
+	return Math.max(workspace.session.metadata?.generatedAt ?? 0, workspace.lastUpdate?.at ?? 0, workspace.lastRecheck?.at ?? 0, workspace.createdAt);
+}
+//#endregion
 //#region ../../packages/review-harness/src/workspace/store.ts
 var WORKSPACES_DIR = join(STORE_ROOT, "workspaces");
 /**
@@ -6750,6 +6996,18 @@ function workspaceRevision(workspace) {
 	}
 }
 /**
+* Moves whenever any review is written or deleted, by this process or another: every write stages a
+* file in the directory and renames it into place, and a delete unlinks one. One stat, where
+* `workspaceStamps` is one per review, for a cache to ask whether anything changed at all.
+*/
+function workspacesRevision() {
+	try {
+		return statSync(WORKSPACES_DIR).mtimeMs;
+	} catch {
+		return 0;
+	}
+}
+/**
 * Every stored review as an id and a revision token, without reading one of them.
 *
 * Here rather than in `reviews_revision.ts` because this module owns the layout of `workspaces/`,
@@ -7031,6 +7289,14 @@ function groupByLineage(reviews) {
 	}
 	return [...groups].map(([key, members]) => {
 		const history = [...members].sort((a, b) => b.createdAt - a.createdAt);
+		const [newest] = history;
+		if (newest.outcomeInputs && history.length > 1) {
+			const { outcome: _outcome, deferral: _deferral, ...rest } = newest;
+			history[0] = {
+				...rest,
+				...lineageOutcome(newest.outcomeInputs, history.map((review) => review.outcomeInputs))
+			};
+		}
 		return {
 			key,
 			latest: history[0],
@@ -7161,6 +7427,8 @@ function summarise(workspace) {
 		})(),
 		lastViewedAt: workspace.lastViewedAt,
 		completedAt: workspace.completedAt,
+		outcomeInputs: outcomeInputsOf(workspace),
+		...outcomeSummary(outcomeInputsOf(workspace)),
 		pinned: workspace.pinned === true,
 		lifecycle: parseLifecycle(workspace.lifecycle),
 		paths: files.slice(0, MAX_INDEXED_PATHS).map((file) => file.name),
@@ -7171,6 +7439,42 @@ function summarise(workspace) {
 		authorName: workspace.prContext?.pullRequest.author?.name || void 0
 	};
 }
+function outcomeInputsOf(workspace) {
+	const baseline = parseStoredOutcome(workspace.reviewedBaseline);
+	return {
+		history: parseOutcomes(workspace.reviewerOutcomes),
+		snapshot: analysisSnapshot(workspace),
+		...baseline ? { baseline } : {}
+	};
+}
+/**
+* The newest record's outcomes read with the newest baseline of its whole lineage, as Quick Review
+* reads them. A fresh analysis makes a record with no baseline of its own, and the verdict on the
+* one before must still show on the card, as an earlier one, instead of vanishing from it.
+*/
+function lineageOutcome(newest, lineage) {
+	return outcomeSummary(newest, lastBaseline([newest.baseline, ...lineage.map((inputs) => inputs?.baseline)]));
+}
+function outcomeSummary(inputs, baseline = inputs.baseline) {
+	const reading = readOutcomes(inputs.history, inputs.snapshot, baseline);
+	const verdict = reading.verdict ? {
+		outcome: reading.verdict,
+		standing: "current"
+	} : reading.previous;
+	return {
+		...verdict ? { outcome: {
+			kind: verdict.outcome.kind,
+			standing: verdict.standing,
+			label: outcomeLabel(verdict.outcome, verdict.standing),
+			at: verdict.outcome.at
+		} } : {},
+		...reading.deferral ? { deferral: {
+			kind: reading.deferral.kind,
+			label: outcomeLabel(reading.deferral, "current"),
+			at: reading.deferral.at
+		} } : {}
+	};
+}
 /**
 * The commit a review was made from, however the record spells it. Null when it recorded none.
 *
@@ -8386,6 +8690,14 @@ function sanitisePullRequest(value) {
 	if (template) pr.fileUrlTemplate = template;
 	const author = pullRequestAuthor(value.author);
 	if (author) pr.author = author;
+	if (isRecord$1(value.headRepository)) {
+		const owner = label(value.headRepository.owner);
+		const repo = label(value.headRepository.repo);
+		if (owner && repo) pr.headRepository = {
+			owner,
+			repo
+		};
+	}
 	return pr;
 }
 /**
@@ -9093,7 +9405,7 @@ function slug(repo) {
 * `baseRefOid` is a diff taken against the wrong *commit*, which is worse because the branch name
 * is right and everything reads as normal. A list nobody asserts is a list somebody tidies.
 */
-var PR_FIELDS = "number,url,title,body,headRefName,headRefOid,baseRefName,baseRefOid,changedFiles,author";
+var PR_FIELDS = "number,url,title,body,headRefName,headRefOid,baseRefName,baseRefOid,changedFiles,author,headRepository,headRepositoryOwner";
 async function viewRequest(repo, reference, cwd, env) {
 	return await ghJson([
 		"pr",
@@ -9142,7 +9454,11 @@ function identity(raw) {
 		author: author ? {
 			name: text$2(author.name),
 			handle: text$2(author.login)
-		} : null
+		} : null,
+		...text$2(raw.headRepositoryOwner?.login) && text$2(raw.headRepository?.name) ? { headRepository: {
+			owner: text$2(raw.headRepositoryOwner?.login),
+			repo: text$2(raw.headRepository?.name)
+		} } : {}
 	};
 }
 /**
@@ -11337,6 +11653,32 @@ ${diff}`;
 	};
 }
 //#endregion
+//#region ../../packages/review-harness/src/domain/ai_review/behaviour_change.ts
+/** Both sides present and non-empty, or nothing: half an explanation is not one. */
+function parseBehaviourChange(value) {
+	if (typeof value !== "object" || value === null || Array.isArray(value)) return void 0;
+	const raw = value;
+	const before = typeof raw.before === "string" ? raw.before.trim() : "";
+	const after = typeof raw.after === "string" ? raw.after.trim() : "";
+	return before && after ? {
+		before,
+		after
+	} : void 0;
+}
+function withBehaviourChange(change) {
+	return change ? { behaviourChange: change } : {};
+}
+/**
+* The behaviourChange an update keeps.
+*
+* Restated: the new one. New code and none restated: dropped, because the old one describes a head
+* that no longer exists. A conversation-only pass read no new code, so the previous one still holds.
+*/
+function mergeBehaviourChange(previous, restated, codeChanged) {
+	if (restated) return restated;
+	return codeChanged ? void 0 : previous;
+}
+//#endregion
 //#region ../../packages/review-harness/src/domain/utils/html.ts
 function escapeHtml(value) {
 	return value.replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;").replace(/"/g, "&quot;").replace(/'/g, "&#39;");
@@ -12652,6 +12994,10 @@ var CONSEQUENCE_GUIDANCE = `"consequence" on an issue: the same issue told as wh
   - 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.`;
+/** The behaviourChange shape, written out whole for every prompt that asks for it. */
+var BEHAVIOUR_CHANGE_FIELD = "\"behaviourChange\"?: { \"before\": string, \"after\": string }";
+/** What "behaviourChange" is for, as the first analysis and an update ask for it. */
+var BEHAVIOUR_CHANGE_GUIDANCE = `"behaviourChange": how the behaviour this pull request touches works today and how it works with the whole pull request applied, for a reviewer who knows the product. "before" is the behaviour at the base the pull request targets; "after" is the behaviour at the head you analysed. One to three plain sentences each, leading with what a user or the system experiences, not with the implementation (e.g. before: "A failed payment is shown as an error and the order is cancelled.", after: "A failed payment is retried twice before the order is cancelled, and the customer is emailed after the last attempt."). Describe the whole pull request, not one commit. Omit the field entirely when the code you read does not establish how it works today: never guess "before".`;
 /**
 * How an update keeps a prior finding's consequence and resolution true, beside its status.
 *
@@ -13458,6 +13804,7 @@ var STRUCTURED_OUTPUT_INSTRUCTIONS = `In addition to the review principles above
   },
   "overallRisk": "low" | "medium" | "high",
   "orientation": string,
+  ${BEHAVIOUR_CHANGE_FIELD},
   "sections": [{
     "id": string,
     "order": number,
@@ -13495,6 +13842,7 @@ Field guidance:
 - ${CONSEQUENCE_GUIDANCE}
 - ${QUESTION_FRAMING_GUIDANCE}
 - ${ORIENTATION_GUIDANCE}
+- ${BEHAVIOUR_CHANGE_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.
@@ -13553,6 +13901,7 @@ function parseUnchecked(raw) {
 			summary,
 			overallRisk: RISKS.includes(obj.overallRisk) ? obj.overallRisk : "medium",
 			...asString$1(obj.orientation).trim() ? { orientation: asString$1(obj.orientation).trim() } : {},
+			...withBehaviourChange(parseBehaviourChange(obj.behaviourChange)),
 			sections: normalizeSections(obj.sections),
 			issues: normalizeIssues(obj.issues),
 			questions: normalizeQuestions(obj.questions),
@@ -18226,6 +18575,7 @@ var RECONCILIATION_SHAPE = `{
   }],
   "summary": string,
   "orientation": string,
+  ${BEHAVIOUR_CHANGE_FIELD},
   "overallRisk": "low" | "medium" | "high",
   "recommendation": string,
   "nextSteps": string[]
@@ -18333,6 +18683,12 @@ var GRADE_TASK = {
 * written to fix a finding is precisely where a regression gets introduced. A run that only
 * reassessed the list has done half the work, and it is the half that looks complete.
 */
+/** The earlier before-and-after, quoted so the model can restate it rather than start over. */
+function previousBehaviourChange(previous) {
+	const change = previous.behaviourChange;
+	if (!change) return "";
+	return ` (the earlier analysis said: before: ${JSON.stringify(change.before)}; after: ${JSON.stringify(change.after)}. Keep what still holds, correct what the new changes alter)`;
+}
 function buildReconciliationInstructions(input, options = {}) {
 	const { previous, totalCommits, changedFiles, possiblyTruncated } = input;
 	const reassess = input.reassess ?? /* @__PURE__ */ new Map();
@@ -18366,7 +18722,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} ${ORIENTATION_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} ${BEHAVIOUR_CHANGE_GUIDANCE} Restate "behaviourChange" for the whole pull request as it now stands${previousBehaviourChange(previous)}, or omit it when the evidence does not establish it.`
 	].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.
 
@@ -18540,6 +18896,7 @@ function parseReconciliationResult(raw, priorCount, options = {}) {
 		})),
 		summary: asString$1(obj.summary).trim(),
 		orientation: asString$1(obj.orientation).trim(),
+		...withBehaviourChange(parseBehaviourChange(obj.behaviourChange)),
 		overallRisk: RISKS.includes(obj.overallRisk) ? obj.overallRisk : null,
 		recommendation: asString$1(obj.recommendation).trim(),
 		nextSteps: asStringArray(obj.nextSteps)
@@ -18817,6 +19174,15 @@ function findingsChanged(counts) {
 }
 //#endregion
 //#region ../../packages/review-harness/src/domain/ai_review/update_reconciliation.ts
+/**
+* Folding an update's answer onto the analysis it brings up to date.
+*
+* One function for every client that updates a review, so the workspace and the extension cannot
+* come to read the same evidence differently: what a cited comment may settle, when a resolution is
+* marked for reassessment instead, which findings a changed comment reopens, what each status is
+* recorded against, and which threads a new finding may claim. A client differs only in how it
+* fetched the changes and ran the model.
+*/
 function reconciledAgainstOf(basis) {
 	return basis.code ? basis.discussion ? "commits-and-conversation" : "commits" : "conversation";
 }
@@ -18849,6 +19215,7 @@ function applyReconciliation(previous, parsed, context) {
 		schemaVersion: 1,
 		summary: parsed.summary || previous.summary,
 		...parsed.orientation || previous.orientation ? { orientation: parsed.orientation || previous.orientation } : {},
+		...withBehaviourChange(mergeBehaviourChange(previous.behaviourChange, parsed.behaviourChange, context.codeChanged)),
 		...motivation ? { motivation } : {},
 		overallRisk: parsed.overallRisk ?? previous.overallRisk,
 		recommendation: parsed.recommendation || previous.recommendation,
@@ -22175,4 +22542,4 @@ var MCP_TOOL_NAMES = [
 	"ask_buddy"
 ];
 //#endregion
-export { statusPillLabel as $, TargetError as $a, lifecycleItemLabel as $i, writeJsonAtomically as $n, findExistingReview as $r, statedWorkspacePort as $t, anythingRunning as A, currentGradeIn as Aa, indexReviewGroups as Ai, BUILD_VERSION as An, deleteCollection as Ao, chatConversationSection as Ar, isRequestId as At, completedLabel as B, statusLookup as Ba, previousKissResult as Bi, checkArchitecture as Bn, modelHelpLines as Br, readServerRecord as Bt, parsePullRequestLink as C, isExplanationAction as Ca, driverGone as Ci, readEvents as Cn, AGENT_RUN_MARKER as Co, ASK_REREAD_CEILING_MS as Cr, findingRefs as Ct, INDEX_SECTIONS as D, analysisRefOf as Da, followedRefName as Di, recordTelemetryUploadConsent as Dn, collectionsStamp as Do, isUnchanged as Dr, endAskRequest as Dt, relativeTime as E, NO_STATUS as Ea, decideKissRecommendation as Ei, deliver$1 as En, cleanName as Eo, storedConversation as Er, reviewTagsIn as Et, collectionFold as F, isGradeFor as Fa, loadWorkspace as Fi, recordRuntimeBestEffort as Fn, STORE_ROOT as Fo, renderMarkdown as Fr, readIndexToken as Ft, markItemLabel as G, extractJsonText as Ga, touchWorkspace as Gi, storeArchitecture as Gn, createWorktree as Gr, stopServer as Gt, homeSection as H, followUpFindingId as Ha, reviewedCommit as Hi, partFiles as Hn, isBusy as Hr, serverEntryExists as Ht, collectionHash as I, isOffered as Ia, matchingWorkspaceIds as Ii, RUNTIME_WORDS as In, CODE_BLOCK_CLASS as Ir, bootstrapUrl as It, queueItemLabel as J, WITHHELD_REASON as Ja, LIFECYCLES as Ji, ownersOf as Jn, readMarker as Jr, DEFAULT_WORKSPACE_PORT as Jt, omittedText as K, objectWithAnyOf as Ka, workspaceRevision as Ki, storedScopes as Kn, holdWorktree as Kr, workspaceUrl as Kt, collectionPlacement as L, promptGrade as La, mutateWorkspace as Li, describeRuntimes as Ln, CODE_COPY_CLASS as Lr, clearServerRecordOwnedBy as Lt, authorOptions as M, findingIn as Ma, kissDecisionsOf as Mi, feedbackUrl as Mn, readCollections as Mo, authorPreview as Mr, markAskRunning as Mt, collectionCountLabel as N, findingStatus as Na, latestKissRun as Ni, waitingHistory as Nn, renameCollection as No, hasAuthorContent as Nr, readAskRequest as Nt, INDEX_SECTION_NAMES as O, assessmentLookup as Oa, groupByLineage as Oi, telemetryUploadConsent as On, createCollection as Oo, agentEnvOf as Or, forgetAskRequest as Ot, collectionEmptyText as P, gradeScale as Pa, lineagePosition as Pi, mapRuntimes as Pn, setMembership as Po, GITHUB_MARKDOWN as Pr, recordAskAccepted as Pt, statusLabel as Q, DetachedHeadError as Qa, lifecycleChip as Qi, isArchitectureId as Qn, describeSubject as Qr, resolveWorkspacePort as Qt, collectionsLine as R, reassessIn as Ra, positionInLineage as Ri, formatDuration as Rn, escapeHtml as Rr, ensureServer as Rt, extractPullRequestLinks as S, explanationOf as Sa, clearClaim as Si, append as Sn, versionFrom as So, existingPaths as Sr, findingRef as St, runBatch as T, ANALYSIS_ONLY as Ta, CLAIM_LEASE_MS as Ti, stamp as Tn, AgentUnavailableError as To, saveRefresh as Tr, reviewTag as Tt, isCollectionId as U, SEVERITIES as Ua, reviewedRepositories as Ui, readArchitecture as Un, withReviewLock as Ur, spawnServer as Ut, countLabel as V, wasReassessed as Va, recentWorkspaces as Vi, loadArchitecture as Vn, ReviewBeingDeletedError as Vr, reviewsUrl as Vt, isFoldKey as W, asString$1 as Wa, summarise as Wi, repositoryKey as Wn, MANAGED_ROOT as Wr, stopChild as Wt, sectionOpen as X, resolveTarget as Xa, activityLabel as Xi, REPOSITORY_SCOPE as Xn, requestJobStop as Xr, MIN_WORKSPACE_PORT as Xt, queueOptionLabel as Y, kissOutcome as Ya, activityAttributes as Yi, QUOTE_LIMITS as Yn, removeWorktree as Yr, MAX_WORKSPACE_PORT as Yt, selectAllLabel as Z, originRemote as Za, isLifecycle as Zi, ancestorsOf as Zn, describeMovement as Zr, parseWorkspacePort as Zt, endRun as _, reviewerDispositions as _a, isLiveJob as _i, prepareReview as _n, agentIdentityOf as _o, retireNotification as _r, modelProvenanceOf as _t, lineageIds as a, sectionOf as aa, forgeResolver as ai, cancellableHere as an, git as ao, createPrState as ar, lineageKeysFor as at, stopRun as b, recommendationsOf as ba, progressSteps as bi, runJob as bn, parkedReason as bo, unreadCount as br, humanizeModelId as bt, discardJob as c, queueRemove as ca, allJobs as ci, recheckConversation as cn, resolveModel as co, listNotifications as cr, compactQueue as ct, startKissJob as d, doneVerb as da, fail as di, findingsChanged as dn, readAgentPreference as do, markReadForReview as dr, RUNNER_KINDS as dt, lifecycleName$1 as ea, checkCodeFreshness as ei, writePortPreference as en, isCommitReview as eo, announceJob as er, deliver as et, isDriving as f, isIssueOutstanding as fa, isTerminal as fi, UpdateAlreadyRunningError as fn, writeAgentPreference as fo, moveIds as fr, RUNNER_MARKER as ft, completeRun as g, resolvedByConversation as ga, PHASES as gi, observeReview as gn, agentFor as go, removeNotification as gr, applyUpdate as gt, beginRun as h, questionsOutstanding as ha, saveJob as hi, failureOf as hn, agentById as ho, readStateOf as hr, spawnRunner as ht, deleteReview as i, parseLifecycle as ia, fetchHead as ii, cancelUpdate as in, shorthandOf as io, createArchitectureNotification as ir, retryableMiss as it, authorOptionLabel as j, findingAssessment as ja, kissDecisionRefusal as ji, PACKAGE_NAME as jn, nameProblemText as jo, UNESTABLISHED_MOTIVATION as jr, isTerminalAsk as jt, PR_FILTER_OPTIONS as k, closesFinding as ka, indexKeptIds as ki, telemetryUploadEnabled as kn, createWithMembers as ko, EXPLAIN_SIMPLY_PROMPT as kr, interruption as kt, kissAgentId as l, queueStamp as la, attemptIdOf as li, updateReview as ln, agentIdOf as lo, markAllRead as lr, queueForReview as lt, RUN_KINDS as m, issuesOutstanding as ma, pruneJobResidue as mi, runningJobs as mn, AGENT_IDS as mo, readNotification as mr, runDetached as mt, MCP_TOOL_NAMES as n, lifecyclePillLabel as na, followedBranch as ni, FETCH_TIMEOUT_MS as nn, AmbiguousRefError as no, subjectOf as nr, missedDelivery as nt, liveJobsFor as o, queueAdd as oa, fillFileUrlTemplate as oi, mergeUpdateInto as on, refExists as oo, createPrUpdate as or, settleCollections as ot, track as p, isQuestionOutstanding as pa, loadJob as pi, liveUpdateFor as pn, CONFIG_PATH as po, pruneNotifications as pr, cliEntry as pt, pinItemLabel as q, describeAuthorship as qa, workspaceStamps as qi, trackedFiles as qn, isWorktreeHeld as qr, writeServerRecord as qt, deleteLineage as r, offersLifecycle as ra, reviewedShaOf as ri, RefreshUnavailableError as rn, shortNameOf as ro, categoryOf as rr, openDestination as rt, wasBlocked as s, queueAddMany as sa, RUN_RECORD_RETENTION_MS as si, parkedUpdateRefusal as sn, resolveAgent as so, eventName as sr, activateQueued as st, MCP_SERVER_INFO_NAME as t, lifecyclePillAction as ta, checkReview as ti, COMMIT_REVIEW_CANNOT_UPDATE as tn, resolveBase as to, branchOf as tr, isDelivered as tt, recordKissRun as u, readQueue as ua, endedAt as ui, askCheckout as un, clearAgentPreference as uo, markRead as ur, settleQueue as ut, heldRuns as v, verdictClosure as va, isTerminalJobPhase as vi, settleReview as vn, detectAgents as vo, toneOf as vr, provenanceDetailLines as vt, unsupportedReason as w, withExplanation as wa, isClaimed as wi, record as wn, AgentCancelledError as wo, prepareConversationWithin as wr, refOf as wt, batchIdentity as x, explanationActionOf as xa, purposeOf as xi, startJob as xn, runnableAgents as xo, withUsageRecorded as xr, findingAtRef as xt, reportRun as y, processDiscussion as ya, phasesFor as yi, reanalyseReview as yn, isAgentId as yo, unreadAround as yr, agentDisplayName as yt, completedFold as z, sourcePhrase as za, presentedKissRunOf as zi, ARCHITECTURE_DIR as zn, renderInlineText as zr, healthyPort as zt };
+export { statusLabel as $, gradeScale as $a, workspacesRevision as $i, isArchitectureId as $n, setMembership as $o, describeSubject as $r, resolveWorkspacePort as $t, PR_FILTER_OPTIONS as A, doneVerb as Aa, indexKeptIds as Ai, telemetryUploadEnabled as An, readAgentPreference as Ao, EXPLAIN_SIMPLY_PROMPT as Ar, interruption as At, completedFold as B, explanationActionOf as Ba, matchingWorkspaceIds as Bi, ARCHITECTURE_DIR as Bn, runnableAgents as Bo, renderInlineText as Br, healthyPort as Bt, parsePullRequestLink as C, parseLifecycle as Ca, clearClaim as Ci, append as Cn, shorthandOf as Co, existingPaths as Cr, findingRef as Ct, INDEX_SECTIONS as D, queueRemove as Da, decideKissRecommendation as Di, deliver$1 as Dn, resolveModel as Do, storedConversation as Dr, reviewTagsIn as Dt, relativeTime as E, queueAddMany as Ea, CLAIM_LEASE_MS as Ei, stamp as En, resolveAgent as Eo, saveRefresh as Er, reviewTag as Et, collectionEmptyText as F, resolvedByConversation as Fa, lineageKey as Fi, mapRuntimes as Fn, agentFor as Fo, GITHUB_MARKDOWN as Fr, recordAskAccepted as Ft, isFoldKey as G, NO_STATUS as Ga, previousKissResult as Gi, repositoryKey as Gn, cleanName as Go, MANAGED_ROOT as Gr, stopChild as Gt, countLabel as H, isExplanationAction as Ha, outcomeInputsOf as Hi, loadArchitecture as Hn, AGENT_RUN_MARKER as Ho, ReviewBeingDeletedError as Hr, reviewsUrl as Ht, collectionFold as I, reviewerDispositions as Ia, lineageKeyFor as Ii, recordRuntimeBestEffort as In, agentIdentityOf as Io, renderMarkdown as Ir, readIndexToken as It, pinItemLabel as J, closesFinding as Ja, reviewedRepositories as Ji, trackedFiles as Jn, createWithMembers as Jo, isWorktreeHeld as Jr, writeServerRecord as Jt, markItemLabel as K, analysisRefOf as Ka, recentWorkspaces as Ki, storeArchitecture as Kn, collectionsStamp as Ko, createWorktree as Kr, stopServer as Kt, collectionHash as L, verdictClosure as La, lineageOutcome as Li, RUNTIME_WORDS as Ln, detectAgents as Lo, CODE_BLOCK_CLASS as Lr, bootstrapUrl as Lt, authorOptionLabel as M, isQuestionOutstanding as Ma, kissDecisionRefusal as Mi, PACKAGE_NAME as Mn, CONFIG_PATH as Mo, UNESTABLISHED_MOTIVATION as Mr, isTerminalAsk as Mt, authorOptions as N, issuesOutstanding as Na, kissDecisionsOf as Ni, feedbackUrl as Nn, AGENT_IDS as No, authorPreview as Nr, markAskRunning as Nt, INDEX_SECTION_NAMES as O, queueStamp as Oa, followedRefName as Oi, recordTelemetryUploadConsent as On, agentIdOf as Oo, isUnchanged as Or, endAskRequest as Ot, collectionCountLabel as P, questionsOutstanding as Pa, latestKissRun as Pi, waitingHistory as Pn, agentById as Po, hasAuthorContent as Pr, readAskRequest as Pt, selectAllLabel as Q, findingStatus as Qa, workspaceStamps as Qi, ancestorsOf as Qn, renameCollection as Qo, describeMovement as Qr, parseWorkspacePort as Qt, collectionPlacement as R, processDiscussion as Ra, lineagePosition as Ri, describeRuntimes as Rn, isAgentId as Ro, CODE_COPY_CLASS as Rr, clearServerRecordOwnedBy as Rt, extractPullRequestLinks as S, offersLifecycle as Sa, purposeOf as Si, startJob as Sn, shortNameOf as So, withUsageRecorded as Sr, findingAtRef as St, runBatch as T, queueAdd as Ta, isClaimed as Ti, record as Tn, refExists as To, prepareConversationWithin as Tr, refOf as Tt, homeSection as U, withExplanation as Ua, positionInLineage as Ui, partFiles as Un, AgentCancelledError as Uo, isBusy as Ur, serverEntryExists as Ut, completedLabel as V, explanationOf as Va, mutateWorkspace as Vi, checkArchitecture as Vn, versionFrom as Vo, modelHelpLines as Vr, readServerRecord as Vt, isCollectionId as W, ANALYSIS_ONLY as Wa, presentedKissRunOf as Wi, readArchitecture as Wn, AgentUnavailableError as Wo, withReviewLock as Wr, spawnServer as Wt, queueOptionLabel as X, findingAssessment as Xa, touchWorkspace as Xi, QUOTE_LIMITS as Xn, nameProblemText as Xo, removeWorktree as Xr, MAX_WORKSPACE_PORT as Xt, queueItemLabel as Y, currentGradeIn as Ya, summarise as Yi, ownersOf as Yn, deleteCollection as Yo, readMarker as Yr, DEFAULT_WORKSPACE_PORT as Yt, sectionOpen as Z, findingIn as Za, workspaceRevision as Zi, REPOSITORY_SCOPE as Zn, readCollections as Zo, requestJobStop as Zr, MIN_WORKSPACE_PORT as Zt, endRun as _, lifecycleChip as _a, PHASES as _i, observeReview as _n, DetachedHeadError as _o, removeNotification as _r, applyUpdate as _t, lineageIds as a, nextBaseline as aa, fetchHead as ai, cancelUpdate as an, statusLookup as ao, createArchitectureNotification as ar, retryableMiss as at, stopRun as b, lifecyclePillAction as ba, phasesFor as bi, reanalyseReview as bn, resolveBase as bo, unreadAround as br, agentDisplayName as bt, discardJob as c, outcomeStanding as ca, RUN_RECORD_RETENTION_MS as ci, parkedUpdateRefusal as cn, SEVERITIES as co, eventName as cr, activateQueued as ct, startKissJob as d, readOutcomes as da, endedAt as di, askCheckout as dn, objectWithAnyOf as do, markRead as dr, settleQueue as dt, analysedAt as ea, findExistingReview as ei, statedWorkspacePort as en, isGradeFor as eo, writeJsonAtomically as er, withWriteLock as es, statusPillLabel as et, isDriving as f, sameSha as fa, fail as fi, findingsChanged as fn, describeAuthorship as fo, markReadForReview as fr, RUNNER_KINDS as ft, completeRun as g, isLifecycle as ga, saveJob as gi, failureOf as gn, originRemote as go, readStateOf as gr, spawnRunner as gt, beginRun as h, activityLabel as ha, pruneJobResidue as hi, runningJobs as hn, resolveTarget as ho, readNotification as hr, runDetached as ht, deleteReview as i, lastBaseline as ia, reviewedShaOf as ii, RefreshUnavailableError as in, sourcePhrase as io, categoryOf as ir, openDestination as it, anythingRunning as j, isIssueOutstanding as ja, indexReviewGroups as ji, BUILD_VERSION as jn, writeAgentPreference as jo, chatConversationSection as jr, isRequestId as jt, LIST_SECTIONS as k, readQueue as ka, groupByLineage as ki, telemetryUploadConsent as kn, clearAgentPreference as ko, agentEnvOf as kr, forgetAskRequest as kt, kissAgentId as l, parseOutcomeSnapshot as la, allJobs as li, recheckConversation as ln, asString$1 as lo, listNotifications as lr, compactQueue as lt, RUN_KINDS as m, activityAttributes as ma, loadJob as mi, liveUpdateFor as mn, kissOutcome as mo, pruneNotifications as mr, cliEntry as mt, MCP_TOOL_NAMES as n, REVIEWER_OUTCOMES as na, checkReview as ni, COMMIT_REVIEW_CANNOT_UPDATE as nn, promptGrade as no, branchOf as nr, isDelivered as nt, liveJobsFor as o, outcomeDefinition as oa, forgeResolver as oi, cancellableHere as on, wasReassessed as oo, createPrState as or, lineageKeysFor as ot, track as p, LIFECYCLES as pa, isTerminal as pi, UpdateAlreadyRunningError as pn, WITHHELD_REASON as po, moveIds as pr, RUNNER_MARKER as pt, omittedText as q, assessmentLookup as qa, reviewedCommit as qi, storedScopes as qn, createCollection as qo, holdWorktree as qr, workspaceUrl as qt, deleteLineage as r, appendOutcome as ra, followedBranch as ri, FETCH_TIMEOUT_MS as rn, reassessIn as ro, subjectOf as rr, missedDelivery as rt, wasBlocked as s, outcomeLabel as sa, fillFileUrlTemplate as si, mergeUpdateInto as sn, followUpFindingId as so, createPrUpdate as sr, settleCollections as st, MCP_SERVER_INFO_NAME as t, analysisSnapshot as ta, checkCodeFreshness as ti, writePortPreference as tn, isOffered as to, announceJob as tr, STORE_ROOT as ts, deliver as tt, recordKissRun as u, parseOutcomes as ua, attemptIdOf as ui, updateReview as un, extractJsonText as uo, markAllRead as ur, queueForReview as ut, heldRuns as v, lifecycleItemLabel as va, isLiveJob as vi, prepareReview as vn, TargetError as vo, retireNotification as vr, modelProvenanceOf as vt, unsupportedReason as w, sectionOf as wa, driverGone as wi, readEvents as wn, git as wo, ASK_REREAD_CEILING_MS as wr, findingRefs as wt, batchIdentity as x, lifecyclePillLabel as xa, progressSteps as xi, runJob as xn, AmbiguousRefError as xo, unreadCount as xr, humanizeModelId as xt, reportRun as y, lifecycleName$1 as ya, isTerminalJobPhase as yi, settleReview as yn, isCommitReview as yo, toneOf as yr, provenanceDetailLines as yt, collectionsLine as z, recommendationsOf as za, loadWorkspace as zi, formatDuration as zn, parkedReason as zo, escapeHtml as zr, ensureServer as zt };