@dev.fast/whiteboard 0.2.4-preview.20261009.108 → 0.2.4

package/dist/agent-cli-BR2j-2n5.js

@@ -0,0 +1,2 @@
+import { n as runReviewAgentCli } from "./agent-cli-BrFqhnzM.js";
+export { runReviewAgentCli };

package/dist/{agent-cli-BYMMEOlO.js → agent-cli-BrFqhnzM.js}

@@ -23,7 +23,7 @@ async function runReviewAgentCli(input) {
 			[REVIEW_AGENT_HEADER]: reviewSessionAgent(env)
 		};
 		if (mode === "mcp") {
-			const { serveReviewMcp } = await import("./mcp-BWp_kC0V.js");
+			const { serveReviewMcp } = await import("./mcp-BDy9Aybd.js");
 			await serveReviewMcp(async (key, agentKind) => {
 				const connected = await connectReviewInstance(key ? {
 					...env,

package/dist/{authoring-tools-CUJ_56bd.js → authoring-tools-DvIO906l.js}

@@ -1,4 +1,4 @@
-import { g as activityUpdateSchema, h as activityEndSchema, i as commandSchema, m as activityBeginSchema, n as uploadSchema } from "./local-data-BfAaQmqC.js";
+import { g as activityUpdateSchema, h as activityEndSchema, i as commandSchema, m as activityBeginSchema, n as uploadSchema } from "./local-data-Bm84Xdck.js";
 import { x as publishedEditSchema, y as pathTargetSchema } from "./stored-document-migration-BxZWQQjv.js";
 import { t as ReviewInputError } from "./input-error-OLgB_h31.js";
 import { t as findReviewPackageRoot } from "./package-paths-B6-zxvIO.js";

package/dist/build-info.json

@@ -1 +1 @@
-{"version":"0.2.4-preview.20261009.108","commit":"449d5ddd42bb5bd075544be06a5dd3ff5826ab52","dirty":true,"builtAt":"2026-10-09T02:49:52.008Z"}
+{"version":"0.2.4","commit":"5b00f0e66d034e187a29d270b3690a37b57ccf7b","dirty":true,"builtAt":"2026-10-09T06:23:07.418Z"}

package/dist/{cli-runner-CyBLX8Oj.js → cli-runner-Cjl8h376.js}

@@ -13,7 +13,7 @@ import { t as ReviewTelemetry } from "./review-telemetry-DBy3eVCo.js";
 import { a as isOwnedShim, f as windowsInstallerCommand, m as connectPrompts, o as pathShimPath, p as scanLegacySkills, t as aliasInstallationToAccount } from "./account-alias-_1_7vROx.js";
 import { t as fuzzyRank } from "./fuzzy-match-BoAmcyak.js";
 import { n as readReviewServerDiscovery, o as reviewServerStateDir, r as readReviewServerHealth, s as serverNotReady } from "./server-discovery-Dbzn3w6Z.js";
-import { t as reviewAgentCliHelp } from "./agent-cli-BYMMEOlO.js";
+import { t as reviewAgentCliHelp } from "./agent-cli-BrFqhnzM.js";
 import { closeSync, existsSync, fstatSync, mkdirSync, mkdtempSync, openSync, readSync, renameSync, writeFileSync } from "node:fs";
 import { realpath } from "node:fs/promises";
 import path from "node:path";
@@ -4495,7 +4495,7 @@ async function runReviewCli(input) {
 		process.once("SIGINT", stop);
 		process.once("SIGTERM", stop);
 		try {
-			const { runHeadlessServer } = await import("./headless-host-wMPUx0Aj.js");
+			const { runHeadlessServer } = await import("./headless-host-DcWaxx9P.js");
 			await runHeadlessServer({
 				stateDir,
 				port,
@@ -4546,7 +4546,7 @@ async function runReviewCli(input) {
 		if (problem) throw new Error(`Cannot tell whether a Whiteboard Desktop is using ${stateDir}: ${problem.message}`);
 		const inUse = () => /* @__PURE__ */ new Error(`A Whiteboard server is using ${stateDir}. Stop it first, then run whiteboard server reset-id again.`);
 		if (desktops.instances.some(({ discovery }) => [discovery.appPid, discovery.serverPid].some((pid) => processIsAlive(pid)))) throw inUse();
-		const [{ openReviewProfile }, { withHeadlessServerLock }] = await Promise.all([import("./profile-YLwWPRQp.js"), import("./headless-host-wMPUx0Aj.js")]);
+		const [{ openReviewProfile }, { withHeadlessServerLock }] = await Promise.all([import("./profile-BQGaX8aX.js"), import("./headless-host-DcWaxx9P.js")]);
 		const reset = await withHeadlessServerLock(await realpath(stateDir), async () => {
 			const local = await openReviewProfile(stateDir, { manageWorkspaces: false });
 			try {
@@ -4774,7 +4774,7 @@ async function runReviewCli(input) {
 		return commands;
 	} });
 	for (const [name, description] of [["api", "Call a JSON review authoring tool on the running server"], ["mcp", "Serve the JSON review authoring tools over stdio MCP"]]) configureOutput(program.command(name).description(description).argument("[args...]", "tool name and JSON input").allowUnknownOption().allowExcessArguments().helpOption(false).passThroughOptions().addHelpText("after", `\n${reviewAgentCliHelp}`), "plain").action(async (args) => {
-		const { runReviewAgentCli } = await import("./agent-cli-BJ_SxT1P.js");
+		const { runReviewAgentCli } = await import("./agent-cli-BR2j-2n5.js");
 		state.exitCode = await runReviewAgentCli({
 			...input,
 			env: authoringEnv(),

package/dist/cli.js

@@ -17,7 +17,7 @@ async function runCli(effectivePath) {
 `);
 		return 1;
 	}
-	const { runReviewCli } = await import("./cli-runner-CyBLX8Oj.js");
+	const { runReviewCli } = await import("./cli-runner-Cjl8h376.js");
 	return runReviewCli({
 		argv,
 		cliPaths: {

package/dist/{headless-host-wMPUx0Aj.js → headless-host-DcWaxx9P.js}

@@ -1,8 +1,8 @@
 import { p as isObjectValue } from "./src-X9phtB2j.js";
 import { O as withFileLock, P as writePrivateJsonAtomic } from "./s3-config-CAxhO9u_.js";
 import "./review-home-paths-6zZH-1c9.js";
-import { t as openReviewProfile } from "./profile-CviK33_F.js";
-import { i as createWhiteboardCore, l as createNodeRequestListener, m as migrateDiffrConfig, n as installProcessErrorTelemetry, p as GlobalReviewDesktopVerbRelay, t as drainServerCrashReport } from "./process-error-telemetry-WjZ2FTnX.js";
+import { t as openReviewProfile } from "./profile-RDEioF8b.js";
+import { i as createWhiteboardCore, l as createNodeRequestListener, m as migrateDiffrConfig, n as installProcessErrorTelemetry, p as GlobalReviewDesktopVerbRelay, t as drainServerCrashReport } from "./process-error-telemetry-Dzh5a3s_.js";
 import { i as reviewServerDiscoveryPath, t as headlessServerLockPath } from "./server-discovery-Dbzn3w6Z.js";
 import { randomBytes, randomUUID } from "node:crypto";
 import { mkdir, realpath, rm } from "node:fs/promises";

package/dist/{local-data-BfAaQmqC.js → local-data-Bm84Xdck.js}

@@ -425,9 +425,9 @@ async function* structuralDiff(input) {
 		case "merge-base": args.push(`${comparison.base}...${comparison.head}`);
 	}
 	args.push("--", ...input.paths ?? []);
-	const idleAbort = new AbortController();
-	const idle = setTimeout(() => idleAbort.abort(), 12e4);
-	const signal = AbortSignal.any([idleAbort.signal, input.signal]);
+	const startupAbort = new AbortController();
+	const startup = setTimeout(() => startupAbort.abort(), 12e4);
+	const signal = AbortSignal.any([startupAbort.signal, input.signal]);
 	console.info(`[Review] structural diff: ${executable} ${args.join(" ")}`);
 	const child = spawn(executable, args, {
 		cwd: input.repositoryPath,
@@ -461,7 +461,6 @@ async function* structuralDiff(input) {
 	});
 	try {
 		for await (const line of lines) {
-			idle.refresh();
 			if (Buffer.byteLength(line) > 100663296) throw new Error("Structural diff record exceeded 96 MiB.");
 			if (!line.trim()) continue;
 			const event = decodeStructuralDiffEvent(line);
@@ -469,6 +468,7 @@ async function* structuralDiff(input) {
 			if (!started) {
 				if (event.type !== "start" || event.version !== 3) throw new Error("Unsupported diffr stream protocol.");
 				started = true;
+				clearTimeout(startup);
 			} else if (event.type === "complete") {
 				completed = true;
 				failed = event.failed;
@@ -484,7 +484,7 @@ async function* structuralDiff(input) {
 		const code = await exited;
 		if (code !== 0 && !(code === 2 && (failed > 0 || aborted !== void 0))) throw exitError(code);
 	} finally {
-		clearTimeout(idle);
+		clearTimeout(startup);
 		lines.close();
 		if (child.exitCode === null) child.kill();
 		await exited.catch(() => {});
@@ -960,9 +960,9 @@ async function comparisonCoverage(data, reviewId, pins, mode, signal, publish) {
 	const fileSources = /* @__PURE__ */ new Map();
 	const alignments = /* @__PURE__ */ new Map();
 	const files = [];
-	if (mode === "textual") for (const file of await data.changes(pins)) {
+	for (const file of await data.changes(pins)) {
 		signal.throwIfAborted();
-		const patch = await data.changes(pins, file.path);
+		const patch = await data.changes(pins, file.path, file.previousPath);
 		const changed = emptyCoverage();
 		for (const hunk of parseUnifiedPatch(file.path, patch)) for (const line of hunk.lines) {
 			if (line.kind === "add") changed.head.push([line.newLine - 1, line.newLine]);
@@ -1001,80 +1001,72 @@ async function comparisonCoverage(data, reviewId, pins, mode, signal, publish) {
 			changed,
 			viewed: emptyCoverage()
 		});
-		publish?.({
+		if (mode === "textual") publish?.({
+			counted: false,
 			files: [...files],
 			fileSources: new Map(fileSources),
 			alignments: new Map(alignments)
 		});
 	}
-	else {
-		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;
+	const coverage = {
+		files,
+		fileSources,
+		alignments,
+		counted: true
+	};
+	if (mode === "textual") return coverage;
+	const publishCurrent = () => publish?.({
+		...coverage,
+		files: [...coverage.files]
+	});
+	publishCurrent();
+	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}`);
+		if (event.error) {
+			if (event.error.code !== "unsupported_file_type" && event.error.code !== "not_utf8") throw new Error(`Cannot count ${path}: ${event.error.message}`);
+		} 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)
+					}
+				};
 			}
-			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}`);
-			if (event.error) {
-				if (event.error.code !== "unsupported_file_type" && event.error.code !== "not_utf8") throw new Error(`Cannot count ${path}: ${event.error.message}`);
-				if (!remaining.size) break;
-				continue;
+			const baseIndex = event.file.lhs && baseFiles.get(event.file.lhs.path);
+			const index = (event.file.rhs && headFiles.get(event.file.rhs.path)) ?? 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 diff = event.diff;
-			const previousPath = event.file.lhs?.path !== path ? event.file.lhs?.path : void 0;
-			const base = diff.type === "text" ? diff.lhs?.text ?? null : null;
-			const head = diff.type === "text" ? diff.rhs?.text ?? null : null;
-			const fingerprint = hash([
-				pins.repositoryId,
-				base,
-				head,
-				...diff.type === "binary" ? [event.file.lhs?.oid ?? null, event.file.rhs?.oid ?? null] : []
-			]);
-			const sources = [];
-			if (base !== null) sources.push({
-				side: "base",
-				file: event.file.lhs.path,
-				fromLine: 1,
-				toLine: base.split("\n").length
-			});
-			if (head !== null) sources.push({
-				side: "head",
-				file: path,
-				fromLine: 1,
-				toLine: head.split("\n").length
-			});
-			fileSources.set(path, sources);
-			if (diff.type === "text") alignments.set(path, structuralRows(diff));
-			files.push({
-				path,
-				previousPath,
-				fingerprint,
-				changed: diff.type === "text" ? diff.structural_changes : emptyCoverage(),
-				folded: foldedChanges(diff),
-				viewed: emptyCoverage()
-			});
-			publish?.({
-				files: [...files],
-				fileSources: new Map(fileSources),
-				alignments: new Map(alignments)
-			});
-			if (!remaining.size) break;
+			publishCurrent();
 		}
-		if (remaining.size) throw new Error("Structural coverage is incomplete.");
+		if (!remaining.size) break;
 	}
-	return {
-		files,
-		fileSources,
-		alignments
-	};
+	if (remaining.size) throw new Error("Structural coverage is incomplete.");
+	return coverage;
 }
 /**
 * The changed lines diffr folds by default. A file it hides (lockfiles,
@@ -4221,6 +4213,7 @@ var LocalReviewData = class {
 		return {
 			pending: this.coveragePending(reviewId, pins, mode),
 			comparison: this.coverageCache.get(JSON.stringify([pins, mode])).partial ?? {
+				counted: false,
 				files: [],
 				fileSources: /* @__PURE__ */ new Map(),
 				alignments: /* @__PURE__ */ new Map()
@@ -4272,10 +4265,11 @@ var LocalReviewData = class {
 		}
 		return entry.promise;
 	}
-	async changes(pins, file) {
+	async changes(pins, file, previousPath) {
 		if (file === void 0) return this.summaries(pins);
 		checkRelativePath(file);
-		return this.rawPatch(pins, { paths: [file] });
+		if (previousPath) checkRelativePath(previousPath);
+		return this.rawPatch(pins, { paths: previousPath ? [previousPath, file] : [file] });
 	}
 	async summaries(pins) {
 		if (pins.worktreeRevision) return diffFileSummariesWorkingTree(await this.worktreeInput(pins));

package/dist/{mcp-BWp_kC0V.js → mcp-BDy9Aybd.js}

@@ -1,6 +1,6 @@
 import { a as toolResultText, s as ReviewApiError } from "./agent-client-B_b1GtUc.js";
 import { t as ReviewInputError } from "./input-error-OLgB_h31.js";
-import { n as REVIEW_STATUS_TOOL, t as authoringTools } from "./authoring-tools-CUJ_56bd.js";
+import { n as REVIEW_STATUS_TOOL, t as authoringTools } from "./authoring-tools-DvIO906l.js";
 import { i as publicTool, n as RECOVERY, r as callPublicTool, t as toolFailure } from "./tool-failure-DJmTieh5.js";
 import { Server } from "@modelcontextprotocol/sdk/server/index.js";
 import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";

package/dist/{process-error-telemetry-WjZ2FTnX.js → process-error-telemetry-Dzh5a3s_.js}

@@ -1,6 +1,6 @@
 import { c as jsonString, m as isStringValue, n as isJsonObject, u as parseJsonText } from "./src-X9phtB2j.js";
 import { a as sharedReviewId, c as verifyShareRepository, d as StreamLimitError, f as readBoundedStream, l as exportShare, m as shareIdSchema, n as ShareClient, p as parseShareLink, r as SharePreflightError, s as readShareRepository, t as ShareAuthError, u as validateShareSources } from "./client-BFClfQLO.js";
-import { _ as invalidateStructuralComparisons, a as inspectSnapshot, c as askAgentIds, d as askQuestionSchema, f as AgentSelectionSchema, i as commandSchema, l as askChoiceKinds, p as selectionMarkdown, r as SCRATCHPAD_ID, s as applyAskChange, u as askPicksSchema, v as diffrExecutable, y as diffrMissingError } from "./local-data-BfAaQmqC.js";
+import { _ as invalidateStructuralComparisons, a as inspectSnapshot, c as askAgentIds, d as askQuestionSchema, f as AgentSelectionSchema, i as commandSchema, l as askChoiceKinds, p as selectionMarkdown, r as SCRATCHPAD_ID, s as applyAskChange, u as askPicksSchema, v as diffrExecutable, y as diffrMissingError } from "./local-data-Bm84Xdck.js";
 import { d as REVIEW_CLIENT_REMOTE, i as parseReviewVerbRequest, r as parseReviewDesktopVerbResult, u as REVIEW_CLIENT_HEADER, v as reviewDiffrSummarizerInputSchema, y as decodeStructuralDiffEvent } from "./src-WCbabvdS.js";
 import { C as TraceStorageDeniedError, G as readStoreAuth, H as DEFAULT_STORE_ORIGIN, U as clearStoreAuth, W as openUrlInBrowser, _ as isTraceStorageConfigured, b as selectTraceStorage, o as listReviewTraceSessions, q as runStoreLogin, s as loadReviewAgentTrace, y as resolveTraceStorage } from "./review-agent-traces-DNJFYzc8.js";
 import { $ as unionIntervals, G as selectionKey, H as comparisonKey, I as fileLineRangeSchema, J as coverageSources, N as lensSelections, Q as subtractIntervals, T as selectionReferences, U as formatAnchor, V as anchorSelection, W as resolveDiffSelection, Y as emptyCoverage, Z as scopedCoverage, h as elements, k as UNCATEGORIZED_LENS_ID, u as anchorPins } from "./stored-document-migration-BxZWQQjv.js";
@@ -10,7 +10,7 @@ import { t as errorMessage } from "./error-message-OtiDonty.js";
 import { T as traceMachineEnabled } from "./review-home-paths-6zZH-1c9.js";
 import { i as readReviewPackageVersion, n as readBuildCommit } from "./package-paths-B6-zxvIO.js";
 import { a as REVIEW_APP_SESSION_ID_HEADER, c as sanitizeUiTelemetryEvent, i as BUNDLE_FRAME_PATTERN, l as cleanTelemetryText, r as reviewRequestOrigin, s as isReportableCleanedMessage } from "./request-origin-DA4K1qWY.js";
-import { a as scratchpadAvailable, c as readQuerySchemas, i as renderInstructions, o as inspectQuerySchema, r as instructionsQuerySchema, s as queryAnchor, t as authoringTools } from "./authoring-tools-CUJ_56bd.js";
+import { a as scratchpadAvailable, c as readQuerySchemas, i as renderInstructions, o as inspectQuerySchema, r as instructionsQuerySchema, s as queryAnchor, t as authoringTools } from "./authoring-tools-DvIO906l.js";
 import { t as fuzzyRank } from "./fuzzy-match-BoAmcyak.js";
 import { createRequire } from "node:module";
 import crypto$1, { createHash, randomUUID } from "node:crypto";
@@ -2909,10 +2909,12 @@ async function reviewProgress(store, data, snapshot, signal = new AbortControlle
 	const marks = snapshot.shared ? /* @__PURE__ */ new Map() : store.viewedCoverage(snapshot.reviewId);
 	const pins = snapshot.pins ? (await data.resolveSource(snapshot)).pins : void 0;
 	const comparison = pins ? partial ?? await data.coverage(snapshot.reviewId, pins, mode) : {
+		counted: true,
 		files: [],
 		fileSources: /* @__PURE__ */ new Map(),
 		alignments: /* @__PURE__ */ new Map()
 	};
+	const counting = !comparison.counted;
 	signal.throwIfAborted();
 	const documentKey = JSON.stringify(pins);
 	const comparisons = /* @__PURE__ */ new Map();
@@ -2922,7 +2924,8 @@ async function reviewProgress(store, data, snapshot, signal = new AbortControlle
 		let loading = comparisons.get(key);
 		if (loading) return loading;
 		if (partial) {
-			if (data.coverageSnapshot(snapshot.reviewId, own, mode).pending) return void 0;
+			const state = data.coverageSnapshot(snapshot.reviewId, own, mode);
+			if (state.pending) return state.comparison.counted ? Promise.resolve(state.comparison) : void 0;
 		}
 		loading = data.coverage(snapshot.reviewId, own, mode);
 		comparisons.set(key, loading);
@@ -2962,7 +2965,7 @@ async function reviewProgress(store, data, snapshot, signal = new AbortControlle
 			if (file) group[file.path] ??= withViewed(file);
 		}
 		let rows = file && ownAlignments.get(file.path);
-		if (!file && partial && !own) {
+		if (!file && counting && !own) {
 			pendingSources.add(key);
 			return [];
 		}
@@ -2995,8 +2998,8 @@ async function reviewProgress(store, data, snapshot, signal = new AbortControlle
 			return {
 				...lens,
 				...resolved,
-				pending: !!partial,
-				unavailable: partial || resolved.fileCount ? void 0 : "No files match these targets"
+				pending: counting,
+				unavailable: counting || resolved.fileCount ? void 0 : "No files match these targets"
 			};
 		} catch {
 			return {
@@ -3011,18 +3014,18 @@ async function reviewProgress(store, data, snapshot, signal = new AbortControlle
 			await resolveAvailable(source);
 		} catch {}
 	}));
-	const uncategorized = partial ? [] : uncategorizedSources(files, lenses.filter((lens) => !lens.unavailable).flatMap((lens) => lens.sources));
+	const uncategorized = counting ? [] : uncategorizedSources(files, lenses.filter((lens) => !lens.unavailable).flatMap((lens) => lens.sources));
 	lenses.push({
 		id: UNCATEGORIZED_LENS_ID,
 		title: "Uncategorized changes",
 		sources: uncategorized,
 		wholeFiles: false,
 		fileCount: new Set(uncategorized.map((source) => files.find((file) => source.file === (source.side === "base" ? file.previousPath ?? file.path : file.path))?.path ?? source.file)).size,
-		pending: !!partial,
-		unavailable: partial || uncategorized.length ? void 0 : "All changed lines are covered by lenses"
+		pending: counting,
+		unavailable: counting || uncategorized.length ? void 0 : "All changed lines are covered by lenses"
 	});
 	const progress = {
-		complete: !partial,
+		complete: !counting,
 		files,
 		referenceFiles,
 		lenses,

package/dist/profile-BQGaX8aX.js

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

package/dist/{profile-CviK33_F.js → profile-RDEioF8b.js}

@@ -1,4 +1,4 @@
-import { o as initializeReviewStoreSchema, t as openLocalReviewStore } from "./local-data-BfAaQmqC.js";
+import { o as initializeReviewStoreSchema, t as openLocalReviewStore } from "./local-data-Bm84Xdck.js";
 import { O as withFileLock } from "./s3-config-CAxhO9u_.js";
 import "./review-home-paths-6zZH-1c9.js";
 import { access, mkdir } from "node:fs/promises";

package/dist/server/desktop-host.js

@@ -9,12 +9,12 @@ import { A as devReviewHome, P as writePrivateJsonAtomic } from "../s3-config-CA
 import { U as shellQuote, i as reviewInstanceDiscoveryPath, o as reviewLegacyDiscoveryPath } from "../review-home-paths-6zZH-1c9.js";
 import { i as readReviewPackageVersion, t as findReviewPackageRoot } from "../package-paths-B6-zxvIO.js";
 import { c as reviewInstanceIdentity, t as REVIEW_INSTANCE_ENV } from "../desktop-discovery-C-Qz54wO.js";
-import { t as openReviewProfile } from "../profile-CviK33_F.js";
+import { t as openReviewProfile } from "../profile-RDEioF8b.js";
 import { r as reviewTelemetryChannel, t as ReviewTelemetry } from "../review-telemetry-DBy3eVCo.js";
 import { o as REVIEW_TELEMETRY_TABS } from "../request-origin-DA4K1qWY.js";
 import { c as removeLegacyReviewSkills, d as skipCliInstall, i as finishCliInstallUpdate, l as resetCliInstall, n as applyCliInstall, r as declineCliInstall, s as removeCliInstall, t as aliasInstallationToAccount, u as resolveCliInstallStatus } from "../account-alias-_1_7vROx.js";
-import { _ as setDiffrConfigValue, a as serverJson, c as recordClientError, d as HttpJsonError, f as ReviewServerError, g as saveDiffrSummarizer, h as readDiffrConfig, i as createWhiteboardCore, l as createNodeRequestListener, m as migrateDiffrConfig, n as installProcessErrorTelemetry, o as captureSanitizedUiTelemetry, p as GlobalReviewDesktopVerbRelay, r as serverCrashReportPath, s as clientErrorsForSession, t as drainServerCrashReport, u as readBoundedRequestJson, v as testDiffrSummarizer } from "../process-error-telemetry-WjZ2FTnX.js";
-import { c as readQuerySchemas } from "../authoring-tools-CUJ_56bd.js";
+import { _ as setDiffrConfigValue, a as serverJson, c as recordClientError, d as HttpJsonError, f as ReviewServerError, g as saveDiffrSummarizer, h as readDiffrConfig, i as createWhiteboardCore, l as createNodeRequestListener, m as migrateDiffrConfig, n as installProcessErrorTelemetry, o as captureSanitizedUiTelemetry, p as GlobalReviewDesktopVerbRelay, r as serverCrashReportPath, s as clientErrorsForSession, t as drainServerCrashReport, u as readBoundedRequestJson, v as testDiffrSummarizer } from "../process-error-telemetry-Dzh5a3s_.js";
+import { c as readQuerySchemas } from "../authoring-tools-DvIO906l.js";
 import crypto, { createHash, randomBytes, randomUUID } from "node:crypto";
 import { z } from "zod";
 import fs, { existsSync } from "node:fs";

package/dist/sharing/index.d.ts

@@ -312,7 +312,7 @@ declare const fileLineRangeSchema: z.ZodObject<{
 type FileLineRange = z.infer<typeof fileLineRangeSchema>;
 //#endregion
 //#region src/lens-selection.d.ts
-type AlignmentRow$1 = readonly [number | null, number | null];
+type AlignmentRow = readonly [number | null, number | null];
 //#endregion
 //#region src/viewed-coverage.d.ts
 /** Zero-based, half-open intervals; always normalized at the storage boundary. */
@@ -321,7 +321,7 @@ interface Coverage {
   base: LineInterval[];
   head: LineInterval[];
 }
-interface CoverageFile$1 {
+interface CoverageFile {
   path: string;
   previousPath?: string;
   fingerprint: string;
@@ -1040,9 +1040,11 @@ interface WrittenComponent {
 //#region src/review-api/comparison-coverage.d.ts
 type CoverageMode = "structural" | "textual";
 interface ComparisonCoverage {
-  files: CoverageFile$1[];
+  /** All Git changes are counted, even if folding is still loading. */
+  counted: boolean;
+  files: CoverageFile[];
   fileSources: Map<string, FileLineRange[]>;
-  alignments: Map<string, readonly AlignmentRow$1[]>;
+  alignments: Map<string, readonly AlignmentRow[]>;
 }
 //#endregion
 //#region src/review-api/pull-request.d.ts
@@ -1696,6 +1698,7 @@ declare class LocalReviewData {
   coverageSnapshot(reviewId: string, pins: Pins, mode: CoverageMode): {
     pending: boolean;
     comparison: ComparisonCoverage | {
+      counted: false;
       files: never[];
       fileSources: Map<any, any>;
       alignments: Map<any, any>;
@@ -1703,14 +1706,10 @@ declare class LocalReviewData {
   };
   private coverageNotification;
   private notifyCoverage;
-  coverage(reviewId: string, pins: Pins, mode: CoverageMode): Promise<{
-    files: CoverageFile[];
-    fileSources: Map<string, FileLineRange[]>;
-    alignments: Map<string, readonly AlignmentRow[]>;
-  }>;
+  coverage(reviewId: string, pins: Pins, mode: CoverageMode): Promise<ComparisonCoverage>;
   changes(pins: Pins): Promise<LocalVcsDiffFileSummary[]>;
-  changes(pins: Pins, file: string): Promise<string>;
-  changes(pins: Pins, file?: string): Promise<LocalVcsDiffFileSummary[] | string>;
+  changes(pins: Pins, file: string, previousPath?: string): Promise<string>;
+  changes(pins: Pins, file?: string, previousPath?: string): Promise<LocalVcsDiffFileSummary[] | string>;
   private summaries;
   /** Raw Git patch text for exact filenames, or every change. */
   private rawPatch;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dev.fast/whiteboard",
-  "version": "0.2.4-preview.20261009.108",
+  "version": "0.2.4",
   "private": false,
   "license": "MIT",
   "repository": {
@@ -71,10 +71,10 @@
     "tsx": "4.21.0",
     "typescript": "6.0.3",
     "vitest": "4.1.11",
+    "@dev.fast/json": "0.0.0",
     "@dev.fast/review-protocol": "0.0.0",
     "@dev.fast/review-share-protocol": "0.1.2",
     "@dev.fast/trace-core": "0.0.0",
-    "@dev.fast/json": "0.0.0",
     "@dev.fast/trace-protocol": "0.5.0"
   },
   "engines": {
@@ -85,7 +85,7 @@
     "repo": "devdotfast/review",
     "docs": "packages/review/instructions/authoring.md"
   },
-  "gitHead": "449d5ddd42bb5bd075544be06a5dd3ff5826ab52",
+  "gitHead": "5b00f0e66d034e187a29d270b3690a37b57ccf7b",
   "scripts": {
     "prebuild": "pnpm run build:workspace-deps",
     "build": "node --max-old-space-size=8192 node_modules/tsdown/dist/run.mjs",

package/src/review-api/comparison-coverage.ts

@@ -29,6 +29,8 @@ const hash = (parts: (string | null)[]) =>
   createHash("sha256").update(JSON.stringify(parts)).digest("hex");
 
 export interface ComparisonCoverage {
+  /** All Git changes are counted, even if folding is still loading. */
+  counted: boolean;
   files: CoverageFile[];
   fileSources: Map<string, FileLineRange[]>;
   alignments: Map<string, readonly AlignmentRow[]>;
@@ -42,201 +44,221 @@ export async function comparisonCoverage(
   mode: CoverageMode,
   signal: AbortSignal,
   publish?: (coverage: ComparisonCoverage) => void,
-) {
+): Promise<ComparisonCoverage> {
   const fileSources = new Map<string, FileLineRange[]>();
   const alignments = new Map<string, readonly AlignmentRow[]>();
 
   const files: CoverageFile[] = [];
 
-  if (mode === "textual") {
-    for (const file of await data.changes(pins)) {
-      signal.throwIfAborted();
-      const patch = await data.changes(pins, file.path);
-      const changed = emptyCoverage();
-
-      for (const hunk of parseUnifiedPatch(file.path, patch))
-        for (const line of hunk.lines) {
-          if (line.kind === "add")
-            changed.head.push([line.newLine! - 1, line.newLine!]);
-
-          if (line.kind === "remove")
-            changed.base.push([line.oldLine! - 1, line.oldLine!]);
-        }
-
-      changed.base = unionIntervals(changed.base);
-      changed.head = unionIntervals(changed.head);
-
-      const [base, head] = await Promise.all([
-        file.status === "added"
-          ? null
-          : data
-              .file(pins, "base", file.previousPath ?? file.path)
-              .then((value) => value.text)
-              .catch(() => null),
-        file.status === "deleted"
-          ? null
-          : data
-              .file(pins, "head", file.path)
-              .then((value) => value.text)
-              .catch(() => null),
-      ]);
-
-      alignments.set(
-        file.path,
-        textualRows(
-          file.path,
-          patch,
-          base === null ? 0 : base.split("\n").length,
-          head === null ? 0 : head.split("\n").length,
-        ),
-      );
+  for (const file of await data.changes(pins)) {
+    signal.throwIfAborted();
+    const patch = await data.changes(pins, file.path, file.previousPath);
+    const changed = emptyCoverage();
 
-      const readable =
-        (file.status === "added" || base !== null) &&
-        (file.status === "deleted" || head !== null);
-
-      const fingerprint = hash([
-        pins.repositoryId,
-        base,
-        head,
-        ...(readable ? [] : [pins.base, pins.head, patch]),
-      ]);
-
-      fileSources.set(file.path, [
-        ...(base !== null
-          ? [
-              {
-                side: "base" as const,
-                file: file.previousPath ?? file.path,
-                fromLine: 1,
-                toLine: base.split("\n").length,
-              },
-            ]
-          : []),
-        ...(head !== null
-          ? [
-              {
-                side: "head" as const,
-                file: file.path,
-                fromLine: 1,
-                toLine: head.split("\n").length,
-              },
-            ]
-          : []),
-      ]);
-      files.push({
-        path: file.path,
-        previousPath: file.previousPath,
-        fingerprint,
-        changed,
-        viewed: emptyCoverage(),
-      });
+    for (const hunk of parseUnifiedPatch(file.path, patch))
+      for (const line of hunk.lines) {
+        if (line.kind === "add")
+          changed.head.push([line.newLine! - 1, line.newLine!]);
+
+        if (line.kind === "remove")
+          changed.base.push([line.oldLine! - 1, line.oldLine!]);
+      }
+
+    changed.base = unionIntervals(changed.base);
+    changed.head = unionIntervals(changed.head);
+
+    const [base, head] = await Promise.all([
+      file.status === "added"
+        ? null
+        : data
+            .file(pins, "base", file.previousPath ?? file.path)
+            .then((value) => value.text)
+            .catch(() => null),
+      file.status === "deleted"
+        ? null
+        : data
+            .file(pins, "head", file.path)
+            .then((value) => value.text)
+            .catch(() => null),
+    ]);
+
+    alignments.set(
+      file.path,
+      textualRows(
+        file.path,
+        patch,
+        base === null ? 0 : base.split("\n").length,
+        head === null ? 0 : head.split("\n").length,
+      ),
+    );
+
+    const readable =
+      (file.status === "added" || base !== null) &&
+      (file.status === "deleted" || head !== null);
+
+    const fingerprint = hash([
+      pins.repositoryId,
+      base,
+      head,
+      ...(readable ? [] : [pins.base, pins.head, patch]),
+    ]);
+
+    fileSources.set(file.path, [
+      ...(base !== null
+        ? [
+            {
+              side: "base" as const,
+              file: file.previousPath ?? file.path,
+              fromLine: 1,
+              toLine: base.split("\n").length,
+            },
+          ]
+        : []),
+      ...(head !== null
+        ? [
+            {
+              side: "head" as const,
+              file: file.path,
+              fromLine: 1,
+              toLine: head.split("\n").length,
+            },
+          ]
+        : []),
+    ]);
+    files.push({
+      path: file.path,
+      previousPath: file.previousPath,
+      fingerprint,
+      changed,
+      viewed: emptyCoverage(),
+    });
+
+    if (mode === "textual")
       publish?.({
+        counted: false,
         files: [...files],
         fileSources: new Map(fileSources),
         alignments: new Map(alignments),
       });
+  }
+
+  const coverage: ComparisonCoverage = {
+    files,
+    fileSources,
+    alignments,
+    counted: true,
+  };
+
+  if (mode === "textual") return coverage;
+
+  // Keep Git's totals and file identities while structural results add folding.
+  const publishCurrent = () =>
+    publish?.({ ...coverage, files: [...coverage.files] });
+
+  publishCurrent();
+
+  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 = new Set<string>();
+
+  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;
     }
-  } else {
-    const remaining = new Set<string>();
-
-    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 === "complete" && (event.failed || event.aborted))
+      throw new Error(
+        event.aborted?.message ?? "Structural coverage is incomplete.",
+      );
 
-      if (event.type !== "file") continue;
+    if (event.type !== "file") continue;
 
-      const path = (event.file.rhs ?? event.file.lhs)!.path;
+    const path = (event.file.rhs ?? event.file.lhs)!.path;
 
-      if (!remaining.delete(path))
-        throw new Error(`Unexpected structural result: ${path}`);
+    if (!remaining.delete(path))
+      throw new Error(`Unexpected structural result: ${path}`);
 
-      if (event.error) {
-        // Neither kind has line ranges to count. Keep the rest of the review
-        // available when a repository contains non-UTF-8 fixtures or links.
-        if (
-          event.error.code !== "unsupported_file_type" &&
-          event.error.code !== "not_utf8"
-        )
-          throw new Error(`Cannot count ${path}: ${event.error.message}`);
+    if (event.error) {
+      if (
+        event.error.code !== "unsupported_file_type" &&
+        event.error.code !== "not_utf8"
+      )
+        throw new Error(`Cannot count ${path}: ${event.error.message}`);
+    } else if (event.diff.type === "text") {
+      const diff = event.diff;
+      const folded = foldedChanges(diff);
+
+      // Match each side separately: diffr can split a Git rename into two files.
+      for (const side of ["base", "head"] as const) {
+        const source = side === "base" ? event.file.lhs : event.file.rhs;
+
+        const index =
+          source && (side === "base" ? baseFiles : headFiles).get(source.path);
+
+        if (index === undefined) continue;
+        const file = coverage.files[index];
 
-        if (!remaining.size) break;
-        continue;
+        const visible = subtractIntervals(
+          diff.structural_changes[side],
+          folded[side],
+        );
+
+        coverage.files[index] = {
+          ...file,
+          folded: {
+            ...(file.folded ?? emptyCoverage()),
+            [side]: subtractIntervals(file.changed[side], visible),
+          },
+        };
       }
 
-      const diff = event.diff;
+      const baseIndex = event.file.lhs && baseFiles.get(event.file.lhs.path);
+      const headIndex = event.file.rhs && headFiles.get(event.file.rhs.path);
+      const index = headIndex ?? baseIndex;
 
-      const previousPath =
-        event.file.lhs?.path !== path ? event.file.lhs?.path : undefined;
-
-      const base = diff.type === "text" ? (diff.lhs?.text ?? null) : null;
-      const head = diff.type === "text" ? (diff.rhs?.text ?? null) : null;
-
-      const fingerprint = hash([
-        pins.repositoryId,
-        base,
-        head,
-        ...(diff.type === "binary"
-          ? [event.file.lhs?.oid ?? null, event.file.rhs?.oid ?? null]
-          : []),
-      ]);
-
-      const sources: FileLineRange[] = [];
-
-      if (base !== null)
-        sources.push({
-          side: "base",
-          file: event.file.lhs!.path,
-          fromLine: 1,
-          toLine: base.split("\n").length,
-        });
-
-      if (head !== null)
-        sources.push({
-          side: "head",
-          file: path,
-          fromLine: 1,
-          toLine: head.split("\n").length,
-        });
-      fileSources.set(path, sources);
-
-      if (diff.type === "text") alignments.set(path, structuralRows(diff));
-      files.push({
-        path,
-        previousPath,
-        fingerprint,
-        changed:
-          diff.type === "text" ? diff.structural_changes : emptyCoverage(),
-        folded: foldedChanges(diff),
-        viewed: emptyCoverage(),
-      });
-      publish?.({
-        files: [...files],
-        fileSources: new Map(fileSources),
-        alignments: new Map(alignments),
-      });
+      if (index !== undefined) {
+        const sources =
+          coverage.fileSources.get(coverage.files[index].path) ?? [];
 
-      if (!remaining.size) break;
+        if (
+          sources.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),
+          );
+      }
+
+      publishCurrent();
     }
 
-    if (remaining.size) throw new Error("Structural coverage is incomplete.");
+    if (!remaining.size) break;
   }
 
-  return { files, fileSources, alignments };
+  if (remaining.size) throw new Error("Structural coverage is incomplete.");
+
+  return coverage;
 }
 
 /**

package/src/review-api/local-data.ts

@@ -1409,6 +1409,7 @@ export class LocalReviewData {
     return {
       pending,
       comparison: entry.partial ?? {
+        counted: false,
         files: [],
         fileSources: new Map(),
         alignments: new Map(),
@@ -1496,16 +1497,21 @@ export class LocalReviewData {
   }
 
   changes(pins: Pins): Promise<LocalVcsDiffFileSummary[]>;
-  changes(pins: Pins, file: string): Promise<string>;
+  changes(pins: Pins, file: string, previousPath?: string): Promise<string>;
   changes(
     pins: Pins,
     file?: string,
+    previousPath?: string,
   ): Promise<LocalVcsDiffFileSummary[] | string>;
-  async changes(pins: Pins, file?: string) {
+  async changes(pins: Pins, file?: string, previousPath?: string) {
     if (file === undefined) return this.summaries(pins);
     checkRelativePath(file);
 
-    return this.rawPatch(pins, { paths: [file] });
+    if (previousPath) checkRelativePath(previousPath);
+
+    return this.rawPatch(pins, {
+      paths: previousPath ? [previousPath, file] : [file],
+    });
   }
   private async summaries(pins: Pins) {
     if (pins.worktreeRevision) {

package/src/review-api/review-progress.ts

@@ -79,7 +79,14 @@ export async function reviewProgress(
   // each resolve against the comparison their own pins name.
   const comparison: ComparisonCoverage = pins
     ? (partial ?? (await data.coverage(snapshot.reviewId, pins, mode)))
-    : { files: [], fileSources: new Map(), alignments: new Map() };
+    : {
+        counted: true,
+        files: [],
+        fileSources: new Map(),
+        alignments: new Map(),
+      };
+
+  const counting = !comparison.counted;
 
   signal.throwIfAborted();
 
@@ -98,7 +105,10 @@ export async function reviewProgress(
     if (partial) {
       const state = data.coverageSnapshot(snapshot.reviewId, own, mode);
 
-      if (state.pending) return undefined;
+      if (state.pending)
+        return state.comparison.counted
+          ? Promise.resolve(state.comparison)
+          : undefined;
     }
 
     loading = data.coverage(snapshot.reviewId, own, mode);
@@ -162,7 +172,7 @@ export async function reviewProgress(
 
     // Context-only files are absent from the changed-file stream. Read both pins
     // and require equality rather than inventing correspondence for missing diffs.
-    if (!file && partial && !own) {
+    if (!file && counting && !own) {
       pendingSources.add(key);
 
       return [];
@@ -228,9 +238,9 @@ export async function reviewProgress(
         return {
           ...lens,
           ...resolved,
-          pending: !!partial,
+          pending: counting,
           unavailable:
-            partial || resolved.fileCount
+            counting || resolved.fileCount
               ? undefined
               : "No files match these targets",
         };
@@ -257,7 +267,7 @@ export async function reviewProgress(
     ),
   );
 
-  const uncategorized = partial
+  const uncategorized = counting
     ? []
     : uncategorizedSources(
         files,
@@ -283,15 +293,15 @@ export async function reviewProgress(
           )?.path ?? source.file,
       ),
     ).size,
-    pending: !!partial,
+    pending: counting,
     unavailable:
-      partial || uncategorized.length
+      counting || uncategorized.length
         ? undefined
         : "All changed lines are covered by lenses",
   });
 
   const progress: ReviewProgress = {
-    complete: !partial,
+    complete: !counting,
     files,
     referenceFiles,
     lenses,

package/src/server/structural-diff.ts

@@ -63,9 +63,11 @@ export async function* structuralDiff(
 
   args.push("--", ...(input.paths ?? []));
 
-  const idleAbort = new AbortController();
-  const idle = setTimeout(() => idleAbort.abort(), 120_000);
-  const signal = AbortSignal.any([idleAbort.signal, input.signal]);
+  // Plugins can exceed two minutes between records. Apply this timeout only
+  // until the manifest arrives; plugins enforce their own request timeouts.
+  const startupAbort = new AbortController();
+  const startup = setTimeout(() => startupAbort.abort(), 120_000);
+  const signal = AbortSignal.any([startupAbort.signal, input.signal]);
 
   // The host inherits its own environment and runs from the repository, so
   // diffr reads the user's config and keys exactly as it would from a shell.
@@ -103,9 +105,6 @@ export async function* structuralDiff(
 
   try {
     for await (const line of lines) {
-      // Large comparisons may take minutes while continuing to make progress.
-      idle.refresh();
-
       // Bound individual records, not the entire streamed comparison: a
       // directory move can legitimately contain thousands of small files.
       // Generated parsers in diffr can exceed 64 MiB after JSON encoding.
@@ -126,6 +125,7 @@ export async function* structuralDiff(
         }
 
         started = true;
+        clearTimeout(startup);
       } else if (event.type === "complete") {
         completed = true;
 
@@ -151,7 +151,7 @@ export async function* structuralDiff(
     if (code !== 0 && !(code === 2 && (failed > 0 || aborted !== undefined)))
       throw exitError(code);
   } finally {
-    clearTimeout(idle);
+    clearTimeout(startup);
     lines.close();
 
     if (child.exitCode === null) child.kill();

package/dist/agent-cli-BJ_SxT1P.js

@@ -1,2 +0,0 @@
-import { n as runReviewAgentCli } from "./agent-cli-BYMMEOlO.js";
-export { runReviewAgentCli };

package/dist/profile-YLwWPRQp.js

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