@dev.fast/whiteboard 0.4.0 → 0.4.1-preview.20261026.138

package/dist/{lens-selection-9byzEgvU.js → lens-selection-EuZdag4Q.js}

@@ -1,4 +1,4 @@
-import "./src-DYHVzX3b.js";
+import "./src-DEOc8bvC.js";
 import { z } from "zod";
 //#region src/viewed-coverage.ts
 const coverageSchema = z.object({

package/dist/{local-data-DHedfhxw.js → local-data-BCSeTyic.js}

@@ -1,22 +1,22 @@
-import { T as runVcsSync, _ as listTrackedFilesAtCommit, a as diff, b as resolveRepoContext, c as diffFileSummariesWorkingTree, d as git, f as gitAt, g as listCommitRange, h as jjRevisionIsConflicted, i as detectLocalVcs, l as diffTrees, m as gitCommonDirSync, n as createBlobBatchReader, p as gitCommonDir, s as diffFileSummariesTrees, t as LocalVcsToolsMissingError, u as diffWorkingTree, v as parseGitRemote, w as runVcs, x as resolveRevision, y as readFileAtCommit } from "./src-X9ZebeOD.js";
+import { T as runVcsSync, _ as listTrackedFilesAtCommit, a as diff, b as resolveRepoContext, c as diffFileSummariesWorkingTree, d as git, f as gitAt, g as listCommitRange, h as jjRevisionIsConflicted, i as detectLocalVcs, l as diffTrees, m as gitCommonDirSync, n as createBlobBatchReader, p as gitCommonDir, s as diffFileSummariesTrees, t as LocalVcsToolsMissingError, u as diffWorkingTree, v as parseGitRemote, w as runVcs, x as resolveRevision, y as readFileAtCommit } from "./src-DXnn7RrI.js";
 import { c as jsonString, m as isStringValue, n as isJsonObject, o as jsonObject, u as parseJsonText } from "./src-X9phtB2j.js";
-import { F as structuralChangeCounts, P as decodeStructuralDiffEvent, c as boardIconKind, l as boardTitle } from "./src-DYHVzX3b.js";
-import "./src-Z1x4-Qmb.js";
-import { b as updateCoverage, c as parseAnchor, f as sourceAnchors, g as intersectIntervals, h as emptyCoverage, p as coverageSchema, v as subtractIntervals, y as unionIntervals } from "./lens-selection-9byzEgvU.js";
-import { $ as lensEditSchema, A as editSchema, B as resourceReferences, C as applyEdit, D as boardRepositoryId, E as boardLinks, F as pinnedRepositoryIds, H as respellDocumentLinks, I as pinsSchema, J as checkReferences, K as sourceReferences, M as explicitPins, O as commitLinkCommits, P as pinDocumentReferences, Q as applyLensEdit, S as anchorPins, T as assignMissingIds, U as respellEditLinks, V as respellCommitLinks, W as reviewTargetSchema, Y as documentAdvice, _ as fetchPullRequest, a as SoftwareModelValidationError, at as requireVisibleSource, b as textIncludesQuote, c as traceSchema, ct as markdownText, d as setPullRequest, f as sourceBranchName, g as defaultPullRequestDeps, h as GitHubRefusedRead, i as readStoredDocument, it as fileLineRangeSchema, j as elements, k as documentSchema, l as pullRequestKey, lt as parseMarkdown, m as resolvePullRequest, n as migrateStoredDocument, nt as SourceRangeError, o as defineSoftwareMap, ot as sliceSourceRange, q as summarizeEdit, r as migrateStoredLenses, rt as checkSourcePath, st as markdownNodes, tt as lensSelections, u as pullRequestUrl, v as remoteUrls, w as assignFreshIds, x as adoptFlowUnits, y as azureRepositoryOf, z as resourceReference } from "./stored-document-migration-DEt8iwem.js";
+import { F as decodeStructuralDiffEvent, I as structuralChangeCounts, P as applyStructuralUpdate, c as boardIconKind, l as boardTitle } from "./src-DEOc8bvC.js";
+import "./src-DjRaCctk.js";
+import { b as updateCoverage, c as parseAnchor, f as sourceAnchors, g as intersectIntervals, h as emptyCoverage, p as coverageSchema, v as subtractIntervals, y as unionIntervals } from "./lens-selection-EuZdag4Q.js";
+import { A as commitLinkCommits, C as adoptFlowUnits, D as assignMissingIds, E as assignFreshIds, G as respellEditLinks, H as resourceReferences, I as pinDocumentReferences, J as sourceReferences, K as reviewTargetSchema, L as pinnedRepositoryIds, M as editSchema, N as elements, O as boardLinks, P as explicitPins, R as pinsSchema, S as parseUnifiedPatch, T as applyEdit, U as respellCommitLinks, V as resourceReference, W as respellDocumentLinks, X as checkReferences, Y as summarizeEdit, Z as documentAdvice, _ as fetchPullRequest, a as SoftwareModelValidationError, at as checkSourcePath, b as textIncludesQuote, c as traceSchema, ct as sliceSourceRange, d as setPullRequest, dt as parseMarkdown, et as applyLensEdit, f as sourceBranchName, g as defaultPullRequestDeps, h as GitHubRefusedRead, i as readStoredDocument, it as SourceRangeError, j as documentSchema, k as boardRepositoryId, l as pullRequestKey, lt as markdownNodes, m as resolvePullRequest, n as migrateStoredDocument, o as defineSoftwareMap, ot as fileLineRangeSchema, r as migrateStoredLenses, rt as lensSelections, st as requireVisibleSource, tt as lensEditSchema, u as pullRequestUrl, ut as markdownText, v as remoteUrls, w as anchorPins, x as textualRows, y as azureRepositoryOf } from "./stored-document-migration-CwmTwTb8.js";
 import { t as ReviewInputError } from "./input-error-OLgB_h31.js";
-import { A as processStartIdentity, L as writePrivateJsonAtomic, O as pidReused, j as withFileLock, k as processIsAlive } from "./s3-config-CjcLyFV5.js";
+import { A as processStartIdentity, L as writePrivateJsonAtomic, O as pidReused, j as withFileLock, k as processIsAlive } from "./s3-config-D7Yt0VhS.js";
 import { t as errorMessage } from "./error-message-OtiDonty.js";
-import { s as safeStorageSegment } from "./review-home-paths-CG-sb9Cw.js";
+import { s as safeStorageSegment } from "./review-home-paths-DcB4xLD-.js";
 import { r as isMissingFileError } from "./fs-utils-BMPLt0cr.js";
-import { a as askHistoryEntrySchema, i as askEntrySchema, o as askOfferSchema, s as askPermissionsSchema } from "./thread-state-CO97FUqK.js";
+import { a as askHistoryEntrySchema, i as askEntrySchema, o as askOfferSchema, s as askPermissionsSchema } from "./thread-state-CbLB0jUA.js";
 import { createRequire } from "node:module";
 import crypto, { createHash, randomUUID } from "node:crypto";
 import { z } from "zod";
+import path, { dirname, isAbsolute, relative, resolve, sep } from "node:path";
 import { existsSync, readFileSync, readdirSync, realpathSync, rmSync, statSync } from "node:fs";
 import * as fs$1 from "node:fs/promises";
 import { lstat, mkdir, readFile, readdir, realpath, rm, stat, writeFile } from "node:fs/promises";
-import path, { dirname, isAbsolute, relative, resolve, sep } from "node:path";
 import { homedir } from "node:os";
 import { spawn } from "node:child_process";
 import { isDeepStrictEqual } from "node:util";
@@ -819,7 +819,7 @@ async function* structuralDiff(input) {
 			break;
 		case "merge-base": args.push(`${comparison.base}...${comparison.head}`);
 	}
-	args.push("--", ...input.paths ?? []);
+	args.push("--defer", "--", ...input.paths ?? []);
 	const startupAbort = new AbortController();
 	const startup = setTimeout(() => startupAbort.abort(), 12e4);
 	const signal = AbortSignal.any([startupAbort.signal, input.signal]);
@@ -850,6 +850,7 @@ async function* structuralDiff(input) {
 	let completed = false;
 	let aborted;
 	let failed = 0;
+	const sent = /* @__PURE__ */ new Set();
 	const lines = createInterface({
 		input: child.stdout,
 		crlfDelay: Infinity
@@ -868,7 +869,8 @@ async function* structuralDiff(input) {
 				completed = true;
 				failed = event.failed;
 				aborted = event.aborted;
-			} else if (event.type !== "file") throw new Error(`Unexpected diffr event: ${event.type}`);
+			} else if (event.type === "file") sent.add((event.file.rhs ?? event.file.lhs).path);
+			else if (event.type !== "update" || !sent.has((event.file.rhs ?? event.file.lhs).path)) throw new Error(`Unexpected diffr event: ${event.type}`);
 			yield event;
 		}
 		if (!completed) {
@@ -908,11 +910,11 @@ var StructuralComparisons = class {
 			yield* entry.read(input.signal);
 		} finally {
 			entry.readers--;
-			if (!entry.done && !entry.initialReady && !entry.readers || entry.error || entry.failed) {
+			if (!entry.done && !entry.readers || entry.error || entry.failed) {
 				entry.abort.abort();
 				this.entries.delete(key);
 			}
-			const idle = [...this.entries].filter(([, value]) => (value.done || value.initialReady) && !value.readers);
+			const idle = [...this.entries].filter(([, value]) => value.done && !value.readers);
 			for (const [oldKey, old] of idle.slice(0, -10)) {
 				old.abort.abort();
 				this.entries.delete(oldKey);
@@ -959,7 +961,6 @@ var Comparison = class {
 	done = false;
 	error;
 	failed = false;
-	initialReady = false;
 	remaining;
 	listeners = /* @__PURE__ */ new Set();
 	constructor(input) {
@@ -971,13 +972,10 @@ var Comparison = class {
 				...input,
 				signal: this.abort.signal
 			})) {
-				if (event.type === "start") {
-					this.remaining = new Set(event.files.map((entry) => (entry.file.rhs ?? entry.file.lhs).path));
-					this.initialReady = this.remaining.size === 0;
-				} else if (event.type === "file") {
+				if (event.type === "start") this.remaining = new Set(event.files.map((entry) => (entry.file.rhs ?? entry.file.lhs).path));
+				else if (event.type === "file") {
 					const path = (event.file.rhs ?? event.file.lhs).path;
 					if (!this.remaining?.delete(path)) throw new Error(`Unexpected structural result: ${path}`);
-					this.initialReady = this.remaining.size === 0;
 				}
 				if (event.type === "complete") this.failed = event.failed > 0 || !!event.aborted;
 				this.events.push(event);
@@ -1015,65 +1013,6 @@ var Comparison = class {
 	}
 };
 //#endregion
-//#region src/unified-diff.ts
-const HUNK_HEADER = /^@@ -(?<oldStart>\d+)(?:,(?<oldLines>\d+))? \+(?<newStart>\d+)(?:,(?<newLines>\d+))? @@/;
-function parseUnifiedPatch(file, patch) {
-	if (!patch) return [];
-	const hunks = [];
-	let current = null;
-	let oldLine = 0;
-	let newLine = 0;
-	for (const row of patch.split(/\r?\n/)) {
-		const header = HUNK_HEADER.exec(row);
-		if (header?.groups) {
-			current = {
-				file,
-				oldStart: Number(header.groups.oldStart),
-				oldLines: Number(header.groups.oldLines ?? "1"),
-				newStart: Number(header.groups.newStart),
-				newLines: Number(header.groups.newLines ?? "1"),
-				lines: []
-			};
-			oldLine = current.oldStart;
-			newLine = current.newStart;
-			hunks.push(current);
-			continue;
-		}
-		if (!current || row.startsWith("\")) continue;
-		const marker = row[0];
-		const text = row.slice(1);
-		let line = null;
-		if (marker === " ") {
-			line = {
-				kind: "context",
-				oldLine,
-				newLine,
-				text
-			};
-			oldLine += 1;
-			newLine += 1;
-		} else if (marker === "+") {
-			line = {
-				kind: "add",
-				oldLine: null,
-				newLine,
-				text
-			};
-			newLine += 1;
-		} else if (marker === "-") {
-			line = {
-				kind: "remove",
-				oldLine,
-				newLine: null,
-				text
-			};
-			oldLine += 1;
-		}
-		if (line) current.lines.push(line);
-	}
-	return hunks;
-}
-//#endregion
 //#region src/software-map-diff-counts.ts
 async function resolveSoftwareMapDiffCounts(input) {
 	const baseRef = input.baseRef?.trim();
@@ -1347,32 +1286,6 @@ function unquoteGitPath(value) {
 */
 const checkoutFs = process.versions.electron ? createRequire(import.meta.url)("original-fs").promises : fs$1;
 //#endregion
-//#region src/review-api/lens-alignment.ts
-/** Project the already-computed patch's runs, including unchanged gaps. */
-function textualRows(path, patch, baseCount, headCount) {
-	const rows = [];
-	let base = 0, head = 0;
-	const append = (baseEnd, headEnd) => {
-		while (base < baseEnd || head < headEnd) rows.push([base < baseEnd ? base++ : null, head < headEnd ? head++ : null]);
-	};
-	for (const hunk of parseUnifiedPatch(path, patch)) {
-		append(hunk.oldLines ? hunk.oldStart - 1 : hunk.oldStart, hunk.newLines ? hunk.newStart - 1 : hunk.newStart);
-		let removed = 0, added = 0;
-		const flush = () => {
-			append(base + removed, head + added);
-			removed = added = 0;
-		};
-		for (const line of hunk.lines) if (line.kind === "context") {
-			flush();
-			append(base + 1, head + 1);
-		} else if (line.kind === "add") added++;
-		else removed++;
-		flush();
-	}
-	append(baseCount, headCount);
-	return rows;
-}
-//#endregion
 //#region src/review-api/comparison-coverage.ts
 const hash = (parts) => createHash("sha256").update(JSON.stringify(parts)).digest("hex");
 /** Immutable comparison facts shared by catalog totals and review progress. */
@@ -1436,53 +1349,73 @@ async function comparisonCoverage(data, reviewId, pins, signal, publish) {
 	publish?.(current());
 	const baseFiles = new Map(coverage.files.map((file, index) => [file.previousPath ?? file.path, index]));
 	const headFiles = new Map(coverage.files.map((file, index) => [file.path, index]));
-	const remaining = /* @__PURE__ */ new Set();
-	for await (const event of data.structuralChanges({
-		reviewId,
-		pins,
-		signal
-	})) {
-		if (event.type === "start") {
-			for (const entry of event.files) remaining.add((entry.file.rhs ?? entry.file.lhs).path);
-			if (!remaining.size) break;
-			continue;
-		}
-		if (event.type === "complete" && (event.failed || event.aborted)) throw new Error(event.aborted?.message ?? "Structural coverage is incomplete.");
-		if (event.type !== "file") continue;
-		const path = (event.file.rhs ?? event.file.lhs).path;
-		if (!remaining.delete(path)) throw new Error(`Unexpected structural result: ${path}`);
-		const baseIndex = event.file.lhs && baseFiles.get(event.file.lhs.path);
-		const headIndex = event.file.rhs && headFiles.get(event.file.rhs.path);
-		if (event.error) {
-			if (event.error.code !== "unsupported_file_type" && event.error.code !== "not_utf8") throw new Error(`Cannot count ${path}: ${event.error.message}`);
-			for (const index of [baseIndex, headIndex]) if (index !== void 0) unsupported.add(coverage.files[index].path);
-		} else if (event.diff.type === "text") {
-			const diff = event.diff;
-			const folded = foldedChanges(diff);
-			for (const side of ["base", "head"]) {
-				const source = side === "base" ? event.file.lhs : event.file.rhs;
-				const index = source && (side === "base" ? baseFiles : headFiles).get(source.path);
-				if (index === void 0) continue;
-				const file = coverage.files[index];
-				const visible = subtractIntervals(diff.structural_changes[side], folded[side]);
-				coverage.files[index] = {
-					...file,
-					folded: {
-						...file.folded ?? emptyCoverage(),
-						[side]: subtractIntervals(file.changed[side], visible)
+	return new Promise((resolve, reject) => {
+		let resolved = false;
+		const counted = () => {
+			resolved = true;
+			resolve(current());
+		};
+		(async () => {
+			const remaining = /* @__PURE__ */ new Set();
+			const diffs = /* @__PURE__ */ new Map();
+			for await (const event of data.structuralChanges({
+				reviewId,
+				pins,
+				signal
+			})) {
+				if (event.type === "start") {
+					for (const entry of event.files) remaining.add((entry.file.rhs ?? entry.file.lhs).path);
+					if (!remaining.size) break;
+					continue;
+				}
+				if (event.type === "complete" && event.aborted) throw new Error(event.aborted.message);
+				if (event.type !== "file" && event.type !== "update") continue;
+				const path = (event.file.rhs ?? event.file.lhs).path;
+				const baseIndex = event.file.lhs && baseFiles.get(event.file.lhs.path);
+				const headIndex = event.file.rhs && headFiles.get(event.file.rhs.path);
+				let diff;
+				if (event.type === "update") {
+					const previous = diffs.get(path);
+					if (!previous) throw new Error(`Unexpected structural result: ${path}`);
+					if (event.error) continue;
+					diff = applyStructuralUpdate(previous, event);
+				} else if (!remaining.delete(path)) throw new Error(`Unexpected structural result: ${path}`);
+				else if (event.error) {
+					if (event.error.code !== "unsupported_file_type" && event.error.code !== "not_utf8") throw new Error(`Cannot count ${path}: ${event.error.message}`);
+					for (const index of [baseIndex, headIndex]) if (index !== void 0) unsupported.add(coverage.files[index].path);
+				} else diff = event.diff;
+				if (diff?.type === "text") {
+					diffs.set(path, diff);
+					const folded = foldedChanges(diff);
+					for (const side of ["base", "head"]) {
+						const source = side === "base" ? event.file.lhs : event.file.rhs;
+						const index = source && (side === "base" ? baseFiles : headFiles).get(source.path);
+						if (index === void 0) continue;
+						const file = coverage.files[index];
+						const visible = subtractIntervals(diff.structural_changes[side], folded[side]);
+						coverage.files[index] = {
+							...file,
+							folded: {
+								...file.folded ?? emptyCoverage(),
+								[side]: subtractIntervals(file.changed[side], visible)
+							}
+						};
 					}
-				};
-			}
-			const index = headIndex ?? baseIndex;
-			if (index !== void 0) {
-				if ((coverage.fileSources.get(coverage.files[index].path) ?? []).every((source) => source.file === (source.side === "base" ? event.file.lhs : event.file.rhs)?.path)) coverage.alignments = new Map(coverage.alignments).set(coverage.files[index].path, structuralRows(diff));
+					const index = headIndex ?? baseIndex;
+					if (event.type === "file" && index !== void 0) {
+						if ((coverage.fileSources.get(coverage.files[index].path) ?? []).every((source) => source.file === (source.side === "base" ? event.file.lhs : event.file.rhs)?.path)) coverage.alignments = new Map(coverage.alignments).set(coverage.files[index].path, structuralRows(diff));
+					}
+				}
+				publish?.(current());
+				if (!remaining.size && !resolved) counted();
 			}
-		}
-		publish?.(current());
-		if (!remaining.size) break;
-	}
-	if (remaining.size) throw new Error("Structural coverage is incomplete.");
-	return current();
+			if (remaining.size) throw new Error("Structural coverage is incomplete.");
+			if (!resolved) counted();
+		})().catch((error) => {
+			if (resolved) console.warn("Could not refine folded coverage:", error);
+			else reject(error);
+		});
+	});
 }
 /**
 * The changed lines diffr folds by default. A file it hides (lockfiles,
@@ -6125,14 +6058,25 @@ var LocalReviewData = class {
 		const key = JSON.stringify(pins);
 		let entry = this.coverageCache.get(key);
 		if (!entry || entry.state === "error") {
-			const promise = comparisonCoverage(this, reviewId, pins, this.coverageAbort.signal, (partial) => {
+			const worktree = pins.worktreeRevision ? JSON.stringify({
+				...pins,
+				worktreeRevision: void 0
+			}) : void 0;
+			for (const [cachedKey, cached] of this.coverageCache) if (worktree && cached.worktree === worktree) {
+				cached.abort.abort();
+				this.coverageCache.delete(cachedKey);
+			}
+			const abort = new AbortController();
+			const promise = comparisonCoverage(this, reviewId, pins, AbortSignal.any([this.coverageAbort.signal, abort.signal]), (partial) => {
 				const current = this.coverageCache.get(key);
 				if (current) current.partial = partial;
 				this.notifyCoverage();
 			});
 			entry = {
 				promise,
-				state: "pending"
+				state: "pending",
+				abort,
+				worktree
 			};
 			this.coverageCache.set(key, entry);
 			const current = entry;
@@ -6146,7 +6090,7 @@ var LocalReviewData = class {
 				}
 			};
 			promise.then((value) => {
-				current.partial = value;
+				current.partial ??= value;
 				if (!this.closed) this.store.setDiffStats(pins, {
 					fileCount: value.files.length,
 					additions: value.files.reduce((sum, file) => sum + structuralChangeCounts(file.changed).added, 0),
@@ -6155,7 +6099,7 @@ var LocalReviewData = class {
 				settled("ready");
 			}, (error) => settled("error", error));
 		}
-		return entry.promise;
+		return entry.state === "ready" && entry.partial ? Promise.resolve(entry.partial) : entry.promise;
 	}
 	async changes(pins, file, previousPath) {
 		if (file === void 0) return this.summaries(pins);
@@ -6408,4 +6352,4 @@ async function readWorkspace(file) {
 	}
 }
 //#endregion
-export { projectOfCheckout as a, activityBeginSchema as c, textualRows as d, invalidateStructuralComparisons as f, CheckoutGroups as g, readGitHubPullRequests as h, inspectSnapshot as i, activityEndSchema as l, diffrMissingError as m, uploadSchema as n, groupCheckoutsAhead as o, diffrExecutable as p, commandSchema as r, initializeReviewStoreSchema as s, openLocalReviewStore as t, activityUpdateSchema as u };
+export { projectOfCheckout as a, activityBeginSchema as c, invalidateStructuralComparisons as d, diffrExecutable as f, CheckoutGroups as h, inspectSnapshot as i, activityEndSchema as l, readGitHubPullRequests as m, uploadSchema as n, groupCheckoutsAhead as o, diffrMissingError as p, commandSchema as r, initializeReviewStoreSchema as s, openLocalReviewStore as t, activityUpdateSchema as u };

package/dist/{mcp-Br-r_wwR.js → mcp-_Xg1gQuA.js}

@@ -1,7 +1,8 @@
-import { a as toolResultText, s as ReviewApiError } from "./agent-client-Dc0vHFJt.js";
+import { n as isJsonObject } from "./src-X9phtB2j.js";
+import { a as toolResultText, s as ReviewApiError, t as ToolText } from "./agent-client-DfpM8F5N.js";
 import { t as ReviewInputError } from "./input-error-OLgB_h31.js";
-import { n as REVIEW_STATUS_TOOL, t as authoringTools } from "./authoring-tools-D8EkIDjm.js";
-import { i as publicTool, n as RECOVERY, r as callPublicTool, t as toolFailure } from "./tool-failure-C1L1vJyz.js";
+import { n as REVIEW_STATUS_TOOL, t as authoringTools } from "./authoring-tools-bm39DRoV.js";
+import { i as publicTool, n as RECOVERY, r as callPublicTool, t as toolFailure } from "./tool-failure-DZaQzi4f.js";
 import { Server } from "@modelcontextprotocol/sdk/server/index.js";
 import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
 import { CallToolRequestSchema, ListToolsRequestSchema } from "@modelcontextprotocol/sdk/types.js";
@@ -32,6 +33,7 @@ function mcpAuthoringGuidance(context) {
 	return [
 		"Whiteboard explains code in documents the user reads in Whiteboard Desktop. Call session_get_instructions before creating or editing a Whiteboard and follow it. Read session_capabilities before authoring; session_create opens the new review in Desktop when it is available, so call session_open only for an existing review, and generate software maps only when softwareMapEnabled is true. When the user asks for a Whiteboard or to use Whiteboard (for example to plan work, review a branch or PR stack, or explain a system in Whiteboard), author a Whiteboard with the default topic.",
 		...context.traceEnabled ? ["For why code exists, what an agent was thinking, or whether an agent solved something before, call session_get_instructions({topic:\"trace-archaeology\"})."] : [],
+		"For programmatic MCP calls, use structuredContent when present; otherwise parse JSON from the text content when the result is JSON. An insert through session_edit uses edit.content for the actual document component, never the MCP transport wrapper containing content or structuredContent.",
 		"Never read or write Whiteboard files or SQL."
 	].join(" ");
 }
@@ -103,10 +105,15 @@ async function serveReviewMcp(connect, stdin, stdout, stderr = process.stderr, t
 					type: "text",
 					text: RECOVERY
 				}] };
-				if (request.params.name === REVIEW_STATUS_TOOL.name && offlineStatus) return { content: [{
-					type: "text",
-					text: JSON.stringify(await offlineStatus(error instanceof Error ? error.message : String(error)))
-				}] };
+				if (request.params.name === REVIEW_STATUS_TOOL.name && offlineStatus) {
+					const result = await offlineStatus(error instanceof Error ? error.message : String(error));
+					const response = { content: [{
+						type: "text",
+						text: JSON.stringify(result)
+					}] };
+					if (isJsonObject(result)) response.structuredContent = result;
+					return response;
+				}
 				throw error;
 			}
 			phase = "call";
@@ -115,10 +122,12 @@ async function serveReviewMcp(connect, stdin, stdout, stderr = process.stderr, t
 			const result = await callPublicTool(client, tool, request.params.arguments ?? {}, extra.signal);
 			const text = toolResultText(tool, result);
 			report();
-			return { content: [{
+			const response = { content: [{
 				type: "text",
 				text: listedWhileDown && tool.name === instructionsTool.name ? `${RELOAD_TOOLS}\n\n${text}` : text
 			}] };
+			if (!(result instanceof ToolText) && isJsonObject(result)) response.structuredContent = result;
+			return response;
 		} catch (error) {
 			report(toolFailure(error, phase));
 			return {

package/dist/{package-paths-BeYHYjeR.js → package-paths-Bg731FHS.js}

@@ -1,7 +1,7 @@
 import { c as jsonString, o as jsonObject, u as parseJsonText } from "./src-X9phtB2j.js";
-import "./src-Z1x4-Qmb.js";
-import { readFileSync } from "node:fs";
+import "./src-DjRaCctk.js";
 import path from "node:path";
+import { readFileSync } from "node:fs";
 import { fileURLToPath } from "node:url";
 //#region ../trace-core/src/package-root.ts
 /** Finds the package containing a source or bundled module URL. */

package/dist/profile-C8zL4u3L.js

@@ -0,0 +1,2 @@
+import { t as openReviewProfile } from "./profile-xzrUFgbT.js";
+export { openReviewProfile };

package/dist/{profile-D9wdlPgw.js → profile-xzrUFgbT.js}

@@ -1,8 +1,8 @@
-import { a as projectOfCheckout, g as CheckoutGroups, o as groupCheckoutsAhead, s as initializeReviewStoreSchema, t as openLocalReviewStore } from "./local-data-DHedfhxw.js";
-import "./src-Z1x4-Qmb.js";
-import { j as withFileLock } from "./s3-config-CjcLyFV5.js";
-import { access, mkdir } from "node:fs/promises";
+import { a as projectOfCheckout, h as CheckoutGroups, o as groupCheckoutsAhead, s as initializeReviewStoreSchema, t as openLocalReviewStore } from "./local-data-BCSeTyic.js";
+import "./src-DjRaCctk.js";
+import { j as withFileLock } from "./s3-config-D7Yt0VhS.js";
 import path from "node:path";
+import { access, mkdir } from "node:fs/promises";
 import { DatabaseSync } from "node:sqlite";
 //#region src/review-api/profile.ts
 const lockOptions = {

package/dist/{remote-attach-DSGpTOJ4.js → remote-attach-DF25Epoa.js}

@@ -1,7 +1,7 @@
-import { i as readReviewPackageVersion } from "./package-paths-BeYHYjeR.js";
-import { a as readReviewServerHealth, l as serverNotReady } from "./server-discovery-DKLIOgg3.js";
-import { c as stopBackgroundServer, o as recordedBackgroundServer, r as ensureBackgroundServer } from "./background-server-DHWHzpo5.js";
-import { t as ensureRemoteLanguageServer } from "./remote-language-server-Dx13Mg2m.js";
+import { i as readReviewPackageVersion } from "./package-paths-Bg731FHS.js";
+import { a as readReviewServerHealth, l as serverNotReady } from "./server-discovery-aHK_1r3U.js";
+import { c as stopBackgroundServer, o as recordedBackgroundServer, r as ensureBackgroundServer } from "./background-server-B--5p72o.js";
+import { t as ensureRemoteLanguageServer } from "./remote-language-server-CH8sG_CU.js";
 import path from "node:path";
 import { spawn } from "node:child_process";
 import { gt, valid } from "semver";

package/dist/{remote-extensions-WtJO7pkb.js → remote-extensions-BJamgsCe.js}

@@ -1,2 +1,2 @@
-import { t as ensureRemoteExtensions } from "./remote-extensions-CsI_BJ_f.js";
+import { t as ensureRemoteExtensions } from "./remote-extensions-CeywZL-Z.js";
 export { ensureRemoteExtensions };

package/dist/{remote-extensions-CsI_BJ_f.js → remote-extensions-CeywZL-Z.js}

@@ -1,12 +1,12 @@
 import { u as parseJsonText } from "./src-X9phtB2j.js";
-import { N as devReviewHome } from "./s3-config-CjcLyFV5.js";
-import { t as findReviewPackageRoot } from "./package-paths-BeYHYjeR.js";
-import "./review-home-paths-CG-sb9Cw.js";
+import { N as devReviewHome } from "./s3-config-D7Yt0VhS.js";
+import { t as findReviewPackageRoot } from "./package-paths-Bg731FHS.js";
+import "./review-home-paths-DcB4xLD-.js";
 import { createHash } from "node:crypto";
 import { z } from "zod";
+import path from "node:path";
 import { createReadStream, createWriteStream } from "node:fs";
 import { chmod, mkdir, readFile, readdir, rename, rm, writeFile } from "node:fs/promises";
-import path from "node:path";
 import { Readable } from "node:stream";
 import { execFile } from "node:child_process";
 import { promisify } from "node:util";

package/dist/{remote-language-server-Dx13Mg2m.js → remote-language-server-CH8sG_CU.js}

@@ -1,12 +1,12 @@
-import "./src-Z1x4-Qmb.js";
-import { A as processStartIdentity, j as withFileLock } from "./s3-config-CjcLyFV5.js";
-import { t as findReviewPackageRoot } from "./package-paths-BeYHYjeR.js";
-import { n as cliSpawn, s as spawnDetached } from "./background-server-DHWHzpo5.js";
-import { r as remoteServerPaths, t as ensureRemoteExtensions } from "./remote-extensions-CsI_BJ_f.js";
+import "./src-DjRaCctk.js";
+import { A as processStartIdentity, j as withFileLock } from "./s3-config-D7Yt0VhS.js";
+import { t as findReviewPackageRoot } from "./package-paths-Bg731FHS.js";
+import { n as cliSpawn, s as spawnDetached } from "./background-server-B--5p72o.js";
+import { r as remoteServerPaths, t as ensureRemoteExtensions } from "./remote-extensions-CeywZL-Z.js";
 import { randomBytes } from "node:crypto";
 import { z } from "zod";
-import { mkdir, readFile, rename, stat, writeFile } from "node:fs/promises";
 import path from "node:path";
+import { mkdir, readFile, rename, stat, writeFile } from "node:fs/promises";
 import { setTimeout as setTimeout$1 } from "node:timers/promises";
 //#region src/remote-language-server.ts
 const START_TIMEOUT_MS = 15e3;

package/dist/{remote-uninstall-Dp2o3qz0.js → remote-uninstall-Cq-lkBY7.js}

@@ -1,12 +1,12 @@
-import { C as REVIEW_REMOTE_INSTALL_LOCK, w as REVIEW_REMOTE_WRAPPER_MARK } from "./src-DYHVzX3b.js";
-import { M as DEV_REVIEW_HOME_ENV } from "./s3-config-CjcLyFV5.js";
-import "./review-home-paths-CG-sb9Cw.js";
-import { c as stopBackgroundServer, o as recordedBackgroundServer } from "./background-server-DHWHzpo5.js";
-import { i as whiteboardRemoteHome } from "./remote-extensions-CsI_BJ_f.js";
-import { n as remoteLanguageServerGroups, r as stopProcessGroup } from "./remote-language-server-Dx13Mg2m.js";
+import { C as REVIEW_REMOTE_INSTALL_LOCK, w as REVIEW_REMOTE_WRAPPER_MARK } from "./src-DEOc8bvC.js";
+import { M as DEV_REVIEW_HOME_ENV } from "./s3-config-D7Yt0VhS.js";
+import "./review-home-paths-DcB4xLD-.js";
+import { c as stopBackgroundServer, o as recordedBackgroundServer } from "./background-server-B--5p72o.js";
+import { i as whiteboardRemoteHome } from "./remote-extensions-CeywZL-Z.js";
+import { n as remoteLanguageServerGroups, r as stopProcessGroup } from "./remote-language-server-CH8sG_CU.js";
+import path from "node:path";
 import { existsSync } from "node:fs";
 import { lstat, mkdir, readFile, readdir, rm, stat } from "node:fs/promises";
-import path from "node:path";
 import { execFile } from "node:child_process";
 import { promisify } from "node:util";
 //#region src/remote-uninstall.ts

package/dist/{request-origin-CNSLVn7x.js → request-origin-CargJFvH.js}

@@ -1,7 +1,7 @@
 import { a as jsonNumber, c as jsonString, i as jsonBoolean, n as isJsonObject } from "./src-X9phtB2j.js";
-import { _ as REVIEW_DIFF_LAYOUTS, g as REVIEW_COLOR_THEME_KINDS, v as REVIEW_DIFF_LAYOUT_SCOPES } from "./src-DYHVzX3b.js";
-import { t as SELECTION_UNAVAILABLE_REASONS } from "./lens-selection-9byzEgvU.js";
-import { n as askAgentIds } from "./thread-state-CO97FUqK.js";
+import { _ as REVIEW_DIFF_LAYOUTS, g as REVIEW_COLOR_THEME_KINDS, v as REVIEW_DIFF_LAYOUT_SCOPES } from "./src-DEOc8bvC.js";
+import { t as SELECTION_UNAVAILABLE_REASONS } from "./lens-selection-EuZdag4Q.js";
+import { n as askAgentIds } from "./thread-state-CbLB0jUA.js";
 import { z } from "zod";
 //#region src/telemetry-clean-text.ts
 const USER_DATA_REGEXES = [
@@ -234,6 +234,7 @@ const SETTING_NAME = [
 	"sharing_enabled",
 	"diffr_config",
 	"theme",
+	"diff_theme",
 	"ctrl_tab",
 	"open_files_in",
 	"ready_notification",
@@ -241,6 +242,7 @@ const SETTING_NAME = [
 	"code_font_size",
 	"document_font_size",
 	"reading_font",
+	"reading_font_family",
 	"code_font_family",
 	"font_ligatures",
 	"diff_font_size",
@@ -712,6 +714,15 @@ const UI_TELEMETRY_EVENTS = {
 			files: "number"
 		}
 	},
+	diff_loaded: {
+		event: "review_diff_loaded",
+		properties: {
+			first_file_ms: "number",
+			updates_ms: "number",
+			files: "number",
+			update_errors: "number"
+		}
+	},
 	discord_clicked: {
 		event: "review_discord_clicked",
 		properties: { via: DISCORD_VIA }

package/dist/{review-app-launcher-BZAnR8Vp.js → review-app-launcher-D8ApNDUV.js}

@@ -1,7 +1,7 @@
-import { o as parseReviewVerbResponse } from "./src-DYHVzX3b.js";
-import { b as selectReviewInstance, m as healthyReviewInstance, v as reviewInstanceStartHint, y as reviewInstanceUnavailable } from "./server-discovery-DKLIOgg3.js";
-import { closeSync, existsSync, fstatSync, mkdtempSync, openSync, readSync } from "node:fs";
+import { o as parseReviewVerbResponse } from "./src-DEOc8bvC.js";
+import { b as selectReviewInstance, m as healthyReviewInstance, v as reviewInstanceStartHint, y as reviewInstanceUnavailable } from "./server-discovery-aHK_1r3U.js";
 import path from "node:path";
+import { closeSync, existsSync, fstatSync, mkdtempSync, openSync, readSync } from "node:fs";
 import { homedir, tmpdir } from "node:os";
 import { execFileSync, spawn } from "node:child_process";
 //#region src/review-app-launcher.ts

package/dist/{review-home-paths-CG-sb9Cw.js → review-home-paths-DcB4xLD-.js}

@@ -1,5 +1,5 @@
-import "./src-Z1x4-Qmb.js";
-import { N as devReviewHome } from "./s3-config-CjcLyFV5.js";
+import "./src-DjRaCctk.js";
+import { N as devReviewHome } from "./s3-config-D7Yt0VhS.js";
 import crypto from "node:crypto";
 import path from "node:path";
 //#region src/review-home-paths.ts

package/dist/{review-telemetry-BM-_KdLi.js → review-telemetry-BW7gJrSo.js}

@@ -1,15 +1,15 @@
 import { c as jsonString, l as jsonValueSchema, o as jsonObject, u as parseJsonText } from "./src-X9phtB2j.js";
-import "./src-DYHVzX3b.js";
-import "./src-Z1x4-Qmb.js";
-import { F as writeFileAtomic, N as devReviewHome, j as withFileLock, k as processIsAlive } from "./s3-config-CjcLyFV5.js";
-import { i as readReviewPackageVersion, t as findReviewPackageRoot } from "./package-paths-BeYHYjeR.js";
-import "./review-home-paths-CG-sb9Cw.js";
-import "./request-origin-CNSLVn7x.js";
+import "./src-DEOc8bvC.js";
+import "./src-DjRaCctk.js";
+import { F as writeFileAtomic, N as devReviewHome, j as withFileLock, k as processIsAlive } from "./s3-config-D7Yt0VhS.js";
+import { i as readReviewPackageVersion, t as findReviewPackageRoot } from "./package-paths-Bg731FHS.js";
+import "./review-home-paths-DcB4xLD-.js";
+import "./request-origin-CargJFvH.js";
 import { createHmac, randomUUID } from "node:crypto";
 import { z } from "zod";
+import path from "node:path";
 import { existsSync, readFileSync, rmSync } from "node:fs";
 import { readFile, readdir, rm } from "node:fs/promises";
-import path from "node:path";
 import os from "node:os";
 import { valid } from "semver";
 //#region src/agent-session-ref.ts

package/dist/{runtime-C6_1K-3Z.js → runtime-DY6Ctyog.js}

@@ -1,4 +1,4 @@
-import { w as runVcs } from "./src-X9ZebeOD.js";
+import { w as runVcs } from "./src-DXnn7RrI.js";
 import path from "node:path";
 //#region src/runtime.ts
 async function resolveReviewRoot(cwd, execFile = execFilePromise) {