@dev.fast/whiteboard 0.2.1-preview.20261006.99 → 0.2.1-preview.20261007.102

package/dist/server/desktop-host.js

@@ -1,28 +1,25 @@
 #!/usr/bin/env node
 import { a as diffFileSummaries, b as resolveRevision, i as diff, x as splitGitPatchFiles } from "../src-CmdiBL20.js";
-import { c as jsonString, i as jsonBoolean, m as isStringValue, n as isJsonObject, p as isObjectValue, s as jsonProperty, u as parseJsonText } from "../src-X9phtB2j.js";
-import { i as SharedReviewStore } from "../client-DFoh50W3.js";
-import { c as ReviewBugReportRequestSchema, l as parseReviewBugReportResponse, o as parseZod, s as ReviewBugReportMetaV2Schema, t as parseReviewCliInstallApplyRequest, v as reviewDiffrSummarizerInputSchema, y as decodeStructuralDiffEvent } from "../src-DnwdaQ2r.js";
-import { _ as invalidateStructuralComparisons, b as installFullDiffr, v as diffrExecutable, y as diffrMissingError } from "../local-data-CxLk75rx.js";
-import { C as resourceReferences, p as documentSchema } from "../stored-document-migration-DHeMpLMb.js";
+import { c as jsonString, i as jsonBoolean, n as isJsonObject, p as isObjectValue, s as jsonProperty, u as parseJsonText } from "../src-X9phtB2j.js";
+import { i as SharedReviewStore } from "../client-D01H2KOV.js";
+import { c as ReviewBugReportRequestSchema, l as parseReviewBugReportResponse, o as parseZod, s as ReviewBugReportMetaV2Schema, t as parseReviewCliInstallApplyRequest, v as reviewDiffrSummarizerInputSchema } from "../src-DHiDJQDy.js";
+import { C as resourceReferences, p as documentSchema } from "../stored-document-migration-T9Uj5MkN.js";
 import { t as ReviewInputError } from "../input-error-OLgB_h31.js";
 import { A as devReviewHome, P as writePrivateJsonAtomic } from "../s3-config-CAxhO9u_.js";
 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-DBO-PM2V.js";
-import { t as openReviewProfile } from "../profile-4ry2f3AM.js";
-import { r as reviewTelemetryChannel, t as ReviewTelemetry } from "../review-telemetry-DEFnQsQ2.js";
-import { o as REVIEW_TELEMETRY_TABS } from "../request-origin-D_4QIdoN.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-DOHN1RSH.js";
-import { a as serverJson, c as recordClientError, d as HttpJsonError, f as ReviewServerError, i as createWhiteboardCore, l as createNodeRequestListener, n as installProcessErrorTelemetry, o as captureSanitizedUiTelemetry, p as GlobalReviewDesktopVerbRelay, r as serverCrashReportPath, s as clientErrorsForSession, t as drainServerCrashReport, u as readBoundedRequestJson } from "../process-error-telemetry-CkiiadgB.js";
-import { c as readQuerySchemas } from "../authoring-tools-Cr6Kpjsx.js";
+import { c as reviewInstanceIdentity, t as REVIEW_INSTANCE_ENV } from "../desktop-discovery-DK2HoBy5.js";
+import { t as openReviewProfile } from "../profile-B6jrb2QN.js";
+import { r as reviewTelemetryChannel, t as ReviewTelemetry } from "../review-telemetry-2UlsdE4D.js";
+import { o as REVIEW_TELEMETRY_TABS } from "../request-origin-BB2TENZ5.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-DLE9XZhi.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-DN5BpZnc.js";
+import { c as readQuerySchemas } from "../authoring-tools-Bvre8hN2.js";
 import crypto, { createHash, randomBytes, randomUUID } from "node:crypto";
 import { z } from "zod";
 import fs, { existsSync } from "node:fs";
-import { chmod, copyFile, cp, mkdir, mkdtemp, readFile, realpath, rename, rm, stat, writeFile } from "node:fs/promises";
-import path, { join } from "node:path";
-import { tmpdir } from "node:os";
-import { execFile } from "node:child_process";
+import { chmod, copyFile, cp, mkdir, readFile, realpath, rename, rm, stat, writeFile } from "node:fs/promises";
+import path from "node:path";
 import { promisify } from "node:util";
 import { setTimeout as setTimeout$1 } from "node:timers/promises";
 import { pathToFileURL } from "node:url";
@@ -30,7 +27,6 @@ import { Writable } from "node:stream";
 import { pipeline } from "node:stream/promises";
 import { gzip, gzipSync } from "node:zlib";
 import { createServer } from "node:http";
-import { stringify } from "smol-toml";
 import { Hono } from "hono";
 //#region src/review-bundled-tools.ts
 function reviewToolsRoot(env = process.env) {
@@ -426,283 +422,6 @@ async function dumpInside(file, directory) {
 	}
 }
 //#endregion
-//#region src/server/diffr-config.ts
-const execFileAsync = promisify(execFile);
-let writes = Promise.resolve();
-let testing = false;
-function serialized(operation) {
-	const next = writes.then(operation, operation);
-	writes = next.catch(() => {});
-	return next;
-}
-async function diffr(args, rootPath, options = {}) {
-	try {
-		const { stdout } = await execFileAsync(diffrExecutable(), args, {
-			cwd: rootPath,
-			maxBuffer: 16777216,
-			signal: options.signal ?? AbortSignal.timeout(3e4),
-			env: options.env ?? process.env
-		});
-		return stdout;
-	} catch (error) {
-		const failure = error;
-		if (failure.code === "ENOENT") throw diffrMissingError();
-		if (failure.name === "AbortError") throw new Error("diffr timed out or was cancelled.");
-		throw new Error("diffr could not complete the operation. Check its configuration and credentials.");
-	}
-}
-async function values(rootPath, reveal = false, signal) {
-	return json(await diffr([
-		"config",
-		"show",
-		"--json",
-		...reveal ? ["--reveal"] : []
-	], rootPath, { signal }));
-}
-function json(output) {
-	try {
-		const parsed = parseJsonText(output);
-		if (isJsonObject(parsed)) return parsed;
-	} catch {}
-	throw new Error("diffr configuration response is malformed.");
-}
-function valueAt(object, key) {
-	let value = object;
-	for (const segment of key.split(".")) value = isJsonObject(value) ? value[segment] : void 0;
-	return value;
-}
-const prefix = "plugins.bundled.summarize";
-function savedProvider(config) {
-	const value = valueAt(config, `${prefix}.provider`);
-	return isStringValue(value) ? value : "";
-}
-function savedEndpoint(config) {
-	const value = valueAt(config, `${prefix}.endpoint`);
-	return isStringValue(value) ? value : "";
-}
-/** Whether a saved key belongs to a different destination than the draft's. */
-function movesKey(config, draft) {
-	return draft.provider !== savedProvider(config) || draft.endpoint !== savedEndpoint(config);
-}
-function environmentKey(providers, id) {
-	return providers.find((provider) => provider.id === id)?.keyVariables.map((name) => process.env[name]).find((key) => !!key);
-}
-function keyOptional(providers, id, endpoint) {
-	return endpoint !== "" && !!providers.find((provider) => provider.id === id)?.keylessCustomEndpoint;
-}
-function strings(value) {
-	return Array.isArray(value) ? value.filter(isStringValue) : [];
-}
-/**
-* What diffr's schema says about summaries: each provider's defaults, and
-* the default prompt with the link its description gives. Nothing when
-* diffr cannot describe its schema: the rest of Settings still works.
-*/
-async function summaryDefaults(rootPath) {
-	let schema;
-	try {
-		schema = json(await diffr(["config", "schema"], rootPath));
-	} catch {
-		return {};
-	}
-	const options = valueAt(schema, "properties.plugins.properties.bundled.properties.summarize.properties");
-	if (!isJsonObject(options)) return {};
-	const option = (name) => isJsonObject(options[name]) ? options[name] : {};
-	const provider = option("provider");
-	const titles = strings(provider["x-enum-titles"]);
-	const models = valueAt(option("model"), "x-default-by.values");
-	const details = valueAt(option("provider_details"), "x-default-by.values");
-	const providers = strings(provider.enum).map((id, index) => {
-		const detail = isJsonObject(details) ? details[id] : void 0;
-		const model = isJsonObject(models) ? models[id] : void 0;
-		const endpoint = isJsonObject(detail) ? detail.endpoint : void 0;
-		return {
-			id,
-			title: titles[index] ?? id,
-			model: isStringValue(model) ? model : "",
-			endpoint: isStringValue(endpoint) ? endpoint : "",
-			keyVariables: isJsonObject(detail) ? strings(detail.key_variables) : [],
-			keylessCustomEndpoint: isJsonObject(detail) && detail.keyless_custom_endpoint === true
-		};
-	});
-	const prompt = option("system_prompt");
-	return {
-		providers,
-		defaultPrompt: isStringValue(prompt.default) ? prompt.default : void 0
-	};
-}
-async function read(rootPath) {
-	const config = await values(rootPath);
-	const defaults = await summaryDefaults(rootPath);
-	const saved = valueAt(config, `${prefix}.api_key`);
-	const credentialSource = isStringValue(saved) && saved.length > 0 ? "config" : environmentKey(defaults.providers ?? [], savedProvider(config)) ? "environment" : "missing";
-	const plugins = config.plugins;
-	if (isJsonObject(plugins)) for (const namespace of Object.values(plugins)) {
-		if (!isJsonObject(namespace)) continue;
-		for (const plugin of Object.values(namespace)) if (isJsonObject(plugin)) delete plugin.api_key;
-	}
-	return {
-		values: config,
-		credentialSource,
-		...defaults
-	};
-}
-function readDiffrConfig(rootPath) {
-	return serialized(() => read(rootPath));
-}
-function diffrConfigValueText(value) {
-	return isStringValue(value) ? value : JSON.stringify(value);
-}
-async function writeSettings(entries, rootPath) {
-	let changed = false;
-	let error;
-	let current = await values(rootPath);
-	try {
-		for (const [key, value] of entries) {
-			if (valueAt(current, key) === value) continue;
-			await diffr([
-				"config",
-				"set",
-				key,
-				diffrConfigValueText(value)
-			], rootPath);
-			changed = true;
-			current = await values(rootPath);
-		}
-	} catch {
-		error = changed ? "Some settings were saved before diffr rejected a change. Check the current values and try again." : "diffr could not save the setting. Check its configuration and the entered value.";
-	} finally {
-		if (changed) invalidateStructuralComparisons();
-	}
-	return {
-		...await read(rootPath),
-		changed,
-		error
-	};
-}
-function setDiffrConfigValue(key, value, rootPath) {
-	if (!/^[A-Za-z0-9_][A-Za-z0-9_-]*(\.[A-Za-z0-9_][A-Za-z0-9_-]*)*$/.test(key)) throw new Error("Invalid diffr config key.");
-	return serialized(() => writeSettings([[key, value]], rootPath));
-}
-function saveDiffrSummarizer(input, rootPath) {
-	const parsed = reviewDiffrSummarizerInputSchema.safeParse(input);
-	if (!parsed.success) throw new Error("Invalid summary settings.");
-	const draft = parsed.data;
-	return serialized(async () => {
-		const current = await read(rootPath);
-		const providers = current.providers ?? [];
-		const moving = movesKey(current.values, draft);
-		const savedKey = current.credentialSource === "config";
-		if (draft.enabled && !draft.apiKey && !(savedKey && !moving) && !environmentKey(providers, draft.provider) && !keyOptional(providers, draft.provider, draft.endpoint)) return {
-			...current,
-			changed: false,
-			error: "Add an API key before enabling summaries."
-		};
-		const entries = [];
-		if (!draft.enabled) entries.push([`${prefix}.enabled`, false]);
-		if (moving && savedKey) entries.push([`${prefix}.api_key`, ""]);
-		entries.push([`${prefix}.provider`, draft.provider]);
-		if (draft.endpoint !== savedEndpoint(current.values) || draft.endpoint && draft.provider !== savedProvider(current.values)) entries.push([`${prefix}.endpoint`, draft.endpoint]);
-		if (draft.apiKey) entries.push([`${prefix}.api_key`, draft.apiKey]);
-		if (draft.systemPrompt.trim() && draft.systemPrompt !== valueAt(current.values, `${prefix}.system_prompt`)) entries.push([`${prefix}.system_prompt`, draft.systemPrompt]);
-		entries.push([`${prefix}.model`, draft.model], [`${prefix}.tests`, draft.tests]);
-		if (draft.enabled) entries.push([`${prefix}.enabled`, true]);
-		return writeSettings(entries, rootPath);
-	});
-}
-async function testDiffrSummarizer(input, rootPath, signal) {
-	const parsed = reviewDiffrSummarizerInputSchema.safeParse(input);
-	if (!parsed.success) throw new Error("Invalid summary settings.");
-	if (testing) throw new Error("A summary test is already running.");
-	testing = true;
-	const timeout = AbortSignal.timeout(6e4);
-	const deadline = signal ? AbortSignal.any([signal, timeout]) : timeout;
-	let directory;
-	try {
-		const config = await serialized(() => values(rootPath, true, deadline));
-		const saved = valueAt(config, prefix);
-		if (!isJsonObject(saved)) throw new Error("The summarizer is not available in this configuration.");
-		const { provider, endpoint, model, systemPrompt } = parsed.data;
-		const providers = (await summaryDefaults(rootPath)).providers ?? [];
-		const variables = providers.find((known) => known.id === provider)?.keyVariables ?? [];
-		const apiKey = parsed.data.apiKey || (!movesKey(config, parsed.data) && isStringValue(saved.api_key) ? saved.api_key : "") || environmentKey(providers, provider);
-		if (!apiKey && !keyOptional(providers, provider, endpoint)) throw new Error("Add an API key to test summaries.");
-		directory = await mkdtemp(join(tmpdir(), "review-summary-"));
-		const options = {
-			...saved,
-			enabled: true,
-			provider,
-			model,
-			...systemPrompt.trim() && { system_prompt: systemPrompt },
-			min_lines: 1,
-			retries: 0,
-			request_timeout_ms: 45e3
-		};
-		delete options.api_key;
-		delete options.provider_details;
-		if (endpoint) options.endpoint = endpoint;
-		else delete options.endpoint;
-		await mkdir(join(directory, "diffr"));
-		await writeFile(join(directory, "diffr", "config.toml"), stringify({
-			version: 1,
-			plugins: {
-				order: ["bundled.summarize"],
-				bundled: { summarize: options }
-			}
-		}), { mode: 384 });
-		const before = join(directory, "before.rs"), after = join(directory, "after.rs");
-		await writeFile(before, "");
-		await writeFile(after, SUMMARY_SAMPLE);
-		const output = await diffr([
-			"--no-index",
-			"--format",
-			"ndjson",
-			"--stream-annotations",
-			"--",
-			before,
-			after
-		], directory, {
-			env: {
-				...process.env,
-				...Object.fromEntries(providers.flatMap((known) => known.keyVariables).map((name) => [name, ""])),
-				...apiKey && variables[0] && { [variables[0]]: apiKey },
-				XDG_CONFIG_HOME: directory
-			},
-			signal: deadline
-		});
-		try {
-			const events = output.trim().split("\n").map(decodeStructuralDiffEvent);
-			const complete = events.at(-1);
-			const annotation = events.flatMap((event) => event.type === "annotations" ? event.annotations : []).find((item) => item.label.trim());
-			if (complete?.type === "complete" && complete.failed === 0 && !complete.aborted && !events.some((event) => (event.type === "annotations" || event.type === "file") && event.error) && annotation) return apiKey ? annotation.label.split(apiKey).join("[redacted]") : annotation.label;
-		} catch {}
-		throw new Error("No summary was produced. Check the API key, model, and network connection.");
-	} finally {
-		testing = false;
-		if (directory) await rm(directory, {
-			recursive: true,
-			force: true
-		});
-	}
-}
-const SUMMARY_SAMPLE = `pub fn count_values(values: &[i32]) -> (i32, i32) {
-    let mut total = 0;
-    let mut count = 0;
-    for value in values {
-        if *value > 0 {
-            total += value;
-            count += 1;
-        }
-    }
-    let average = if count > 0 {
-        total / count
-    } else {
-        0
-    };
-    (count, average)
-}
-`;
-//#endregion
 //#region src/review-diff-files.ts
 async function resolveReviewDiffFiles(input) {
 	const baseRef = input.baseRef?.trim();
@@ -1407,15 +1126,6 @@ function createGlobalReviewServer(input) {
 		await withReviewLock(TUTORIAL_LIFECYCLE_LOCK_KEY, deleteTutorialLocked);
 		return serverJson(200, { ok: true });
 	});
-	app.post("/diffr-languages/install", async () => {
-		try {
-			await installFullDiffr();
-			invalidateStructuralComparisons();
-			return serverJson(200, { installed: true });
-		} catch (error) {
-			return serverJson(503, { error: error instanceof Error ? error.message : String(error) });
-		}
-	});
 	app.get("/diffr-config", async () => serverJson(200, await readDiffrConfig()));
 	app.put("/diffr-config", async (context) => {
 		const body = await readBoundedRequestJson(context.req.raw);
@@ -1522,6 +1232,7 @@ function createGlobalReviewServer(input) {
 			return urlForBoundPort();
 		},
 		listen: async () => {
+			await migrateDiffrConfig();
 			scratchpadEnabled = await readScratchpadEnabled();
 			boundPort = await listen(httpServer, input.port);
 			discovery.url = urlForBoundPort();

package/dist/sharing/index.d.ts

@@ -88,16 +88,7 @@ type StructuralDiffEvent = {
 } | ({
   type: "file";
   file: StructuralPairing<StructuralFileRef>;
-  visibility?: StructuralVisibility;
 } & StructuralOutcome) | {
-  type: "annotations";
-  file: StructuralPairing<StructuralFileRef>;
-  annotations: {
-    region_id: number;
-    label: string;
-  }[];
-  error?: StructuralProblem;
-} | {
   type: "complete";
   succeeded: number;
   failed: number;
@@ -158,7 +149,8 @@ type StructuralDiff = ({
 type StructuralSource = {
   text: string;
   syntax?: StructuralSyntaxSpan[];
-  regions?: StructuralRegion[];
+  /** The whole file, as one fold. Both sides' roots share a fold_state_id; its visibility is the file's. */
+  root: StructuralRegion;
 };
 type StructuralBinaryRef = {
   size: number;
@@ -183,6 +175,10 @@ type StructuralNode = {
 } | {
   kind: "fold";
   children: StructuralRegion[];
+  /** Where the fold's content starts. Collapsed rows sit at this column. */
+  indent: StructuralPos;
+  /** From the opener to the closer, before rounding to lines. Absent without an opener. */
+  syntax?: StructuralSourceRange;
 };
 type StructuralSpan = {
   line: number;

package/dist/sharing/index.js

@@ -1,2 +1,2 @@
-import { a as sharedReviewId, c as verifyShareRepository, i as SharedReviewStore, l as exportShare, n as ShareClient, o as validateShareBundle } from "../client-DFoh50W3.js";
+import { a as sharedReviewId, c as verifyShareRepository, i as SharedReviewStore, l as exportShare, n as ShareClient, o as validateShareBundle } from "../client-D01H2KOV.js";
 export { ShareClient, SharedReviewStore, exportShare, sharedReviewId, validateShareBundle, verifyShareRepository };

package/dist/{src-DnwdaQ2r.js → src-DHiDJQDy.js}

@@ -3,7 +3,7 @@ import { _ as ReviewAgentTraceEventSchema, v as ReviewAgentTraceSessionSchema }
 import { z } from "zod";
 //#region ../../node_modules/@dev.fast/diffr/dist/contract.js
 /**
-* Mirrors src/protocol/mod.rs and src/pairing.rs. See ../README.md for sync checks.
+* Mirrors crates/diffr-core/src/protocol/mod.rs and crates/diffr-core/src/pairing.rs. See ../README.md for sync checks.
 * Import only zod: Review embeds this file in generated source.
 * Defaults are omitted, not null. Lines are zero-based, columns are UTF-8 bytes,
 * and ranges are half-open.
@@ -35,18 +35,8 @@ const StructuralDiffEventSchema = z.lazy(() => z.union([
 	}),
 	z.object({
 		type: z.literal("file"),
-		file: structuralPairingSchema(StructuralFileRefSchema),
-		visibility: StructuralVisibilitySchema.optional()
+		file: structuralPairingSchema(StructuralFileRefSchema)
 	}).and(StructuralOutcomeSchema),
-	z.object({
-		type: z.literal("annotations"),
-		file: structuralPairingSchema(StructuralFileRefSchema),
-		annotations: z.array(z.object({
-			region_id: structuralU32,
-			label: z.string()
-		})),
-		error: StructuralProblemSchema.optional()
-	}),
 	z.object({
 		type: z.literal("complete"),
 		succeeded: structuralU32,
@@ -120,7 +110,7 @@ const StructuralDiffSchema = z.lazy(() => z.union([z.object({
 const StructuralSourceSchema = z.lazy(() => z.object({
 	text: z.string(),
 	syntax: z.array(StructuralSyntaxSpanSchema).optional(),
-	regions: z.array(StructuralRegionSchema).optional()
+	root: StructuralRegionSchema
 }));
 const StructuralBinaryRefSchema = z.lazy(() => z.object({ size: z.number().int().nonnegative() }));
 const StructuralSyntaxSpanSchema = z.lazy(() => z.object({
@@ -141,7 +131,9 @@ const StructuralNodeSchema = z.lazy(() => z.union([z.object({
 	changed: z.array(StructuralSpanSchema).optional()
 }), z.object({
 	kind: z.literal("fold"),
-	children: z.array(StructuralRegionSchema)
+	children: z.array(StructuralRegionSchema),
+	indent: StructuralPosSchema,
+	syntax: StructuralSourceRangeSchema.optional()
 })]));
 const StructuralSpanSchema = z.lazy(() => z.object({
 	line: structuralU32,
@@ -262,6 +254,7 @@ z.object({
 		keylessCustomEndpoint: z.boolean()
 	})).optional(),
 	defaultPrompt: z.string().optional(),
+	defaultHiddenTags: z.array(z.string()).optional(),
 	changed: z.boolean().optional(),
 	error: z.string().optional()
 });

package/dist/{stored-document-migration-DHeMpLMb.js → stored-document-migration-T9Uj5MkN.js}

@@ -1,5 +1,5 @@
 import { f as isNumberValue, l as jsonValueSchema, m as isStringValue, n as isJsonObject } from "./src-X9phtB2j.js";
-import "./src-DnwdaQ2r.js";
+import "./src-DHiDJQDy.js";
 import { t as ReviewInputError } from "./input-error-OLgB_h31.js";
 import { z } from "zod";
 import { fromMarkdown } from "mdast-util-from-markdown";

package/dist/{tool-failure-C8zB73HV.js → tool-failure-Djeeax6q.js}

@@ -1,5 +1,5 @@
 import { m as isStringValue, n as isJsonObject } from "./src-X9phtB2j.js";
-import { n as callAuthoringTool, s as ReviewApiError, t as ToolText } from "./agent-client-CPQo7iTI.js";
+import { n as callAuthoringTool, s as ReviewApiError, t as ToolText } from "./agent-client-DoaODH6I.js";
 import { t as ReviewInputError } from "./input-error-OLgB_h31.js";
 //#region src/review-api/public-tools.ts
 function publicDescription(text) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dev.fast/whiteboard",
-  "version": "0.2.1-preview.20261006.99",
+  "version": "0.2.1-preview.20261007.102",
   "private": false,
   "license": "MIT",
   "repository": {
@@ -43,6 +43,7 @@
     "@agentclientprotocol/claude-agent-acp": "0.84.0",
     "@agentclientprotocol/codex-acp": "2.0.0",
     "@agentclientprotocol/sdk": "1.5.1",
+    "@dev.fast/diffr": "0.1.14",
     "@hono/node-server": "2.0.12",
     "@img/sharp-wasm32": "0.35.4",
     "@modelcontextprotocol/sdk": "1.30.0",
@@ -61,7 +62,6 @@
     "zod": "4.6.5"
   },
   "devDependencies": {
-    "@dev.fast/diffr": "0.1.10",
     "@dev.fast/local-vcs": "^0.1.0",
     "@types/node": "24.12.2",
     "@types/semver": "7.7.1",
@@ -72,10 +72,10 @@
     "typescript": "6.0.3",
     "vitest": "4.1.11",
     "@dev.fast/json": "0.0.0",
-    "@dev.fast/review-share-protocol": "0.1.2",
-    "@dev.fast/review-protocol": "0.0.0",
+    "@dev.fast/trace-core": "0.0.0",
     "@dev.fast/trace-protocol": "0.5.0",
-    "@dev.fast/trace-core": "0.0.0"
+    "@dev.fast/review-share-protocol": "0.1.2",
+    "@dev.fast/review-protocol": "0.0.0"
   },
   "engines": {
     "node": ">=24 <25"
@@ -85,10 +85,9 @@
     "repo": "devdotfast/review",
     "docs": "packages/review/instructions/authoring.md"
   },
-  "gitHead": "b8634e9e5ae077e753c5bdd743c9702d31fb9efe",
+  "gitHead": "912c9427199c112f1f66e09b6a9ca86cc6797968",
   "scripts": {
     "prebuild": "pnpm run build:workspace-deps",
-    "ensure:diffr": "diffr-fetch --into bin",
     "build": "node --max-old-space-size=8192 node_modules/tsdown/dist/run.mjs",
     "build:tutorial-assets": "tsx scripts/build-tutorial-assets.ts",
     "ensure:tutorial-assets": "tsx scripts/ensure-tutorial-assets.ts",
@@ -103,7 +102,7 @@
     "test:node": "vitest run --config vitest.config.ts --project shared-module-graph",
     "test:telemetry-privacy": "vitest run --config vitest.config.ts src/ui-telemetry-events.test.ts src/error-telemetry.test.ts",
     "typecheck": "pnpm -w exec tsc --noEmit -p packages/review",
-    "test:integration:diffr": "pnpm run build:workspace-deps && pnpm run ensure:diffr --required && vitest run --config vitest.integration.config.ts",
+    "test:integration:diffr": "pnpm run build:workspace-deps && vitest run --config vitest.integration.config.ts",
     "whiteboard": "tsx src/cli.ts"
   }
 }

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

@@ -4,7 +4,6 @@ import {
   type StructuralDiff,
   type StructuralRegion,
   type StructuralSource,
-  type StructuralVisibility,
   structuralRows,
 } from "@dev.fast/review-protocol";
 import type { AlignmentRow } from "@review/lens-selection.js";
@@ -217,7 +216,7 @@ export async function comparisonCoverage(
         fingerprint,
         changed:
           diff.type === "text" ? diff.structural_changes : emptyCoverage(),
-        folded: foldedChanges(event.visibility, diff),
+        folded: foldedChanges(diff),
         viewed: emptyCoverage(),
       });
       publish?.({
@@ -244,21 +243,17 @@ export async function comparisonCoverage(
  * the lines, not just the counts, are known. A paired leaf contributes its
  * changed spans' lines; an unpaired leaf, all of its lines.
  */
-export function foldedChanges(
-  visibility: StructuralVisibility | undefined,
-  diff: StructuralDiff,
-): Coverage {
+export function foldedChanges(diff: StructuralDiff): Coverage {
   if (diff.type !== "text") return emptyCoverage();
   const all = diff.structural_changes;
 
-  if (visibility?.collapsed)
-    return { base: unionIntervals(all.base), head: unionIntervals(all.head) };
-
   const side = (
     changed: readonly LineInterval[],
     source: StructuralSource | undefined,
     other: StructuralSource | undefined,
   ): LineInterval[] => {
+    if (source?.root.visibility?.collapsed) return unionIntervals(changed);
+
     const paired = new Set<number>();
 
     const pair = (region: StructuralRegion) => {
@@ -266,7 +261,7 @@ export function foldedChanges(
       else region.children.forEach(pair);
     };
 
-    other?.regions?.forEach(pair);
+    if (other) pair(other.root);
 
     const hidden: LineInterval[] = [],
       visible: LineInterval[] = [];
@@ -292,7 +287,7 @@ export function foldedChanges(
         ]);
     };
 
-    for (const region of source?.regions ?? []) collect(region, false);
+    if (source) collect(source.root, false);
 
     return subtractIntervals(intersectIntervals(changed, hidden), visible);
   };

package/src/review-api/http.ts

@@ -12,11 +12,12 @@ import {
   AgentSelectionSchema,
   selectionMarkdown,
 } from "@review/agent-selection.js";
-import type { AskAgentStatus } from "@review/ask/agents.js";
+import { type AskAgentStatus, askAgents } from "@review/ask/agents.js";
 import { checkoutFiles, mentionableFiles } from "@review/ask/checkout-files.js";
 import { parseFileRef, resolveFileRefs } from "@review/ask/file-refs.js";
 import {
   type AskAgentId,
+  type AskThreadState,
   askAgentIds,
   askChoiceKinds,
   askPicksSchema,
@@ -1154,15 +1155,15 @@ export function createReviewApi(
         checkout.live
           ? "Your working directory is the repository the review describes."
           : `Your working directory is a checkout of the review's head commit, ${checkout.head}. Answer from this code, not from other branches.`,
-        "Answer the question.",
+        "Answer the question for a staff engineer: lead with the answer and keep it short. Explain at the level of components and data flow before functions, and check each claim about the code against code you have read.",
         "Name files by their path from the checkout root, with a line where it helps, as in `src/app.ts:42`; the reviewer can open them from your answer.",
         ...(reach?.kind === "mcp"
           ? [
-              `The whiteboard MCP tools read and change this review: its sessionId is "${reviewId}". Read it with session_get. If the reviewer asks you to change the review, edit it with session_edit; do not write files to do it.`,
+              `The whiteboard MCP tools read and change this review: its sessionId is "${reviewId}". Read it with session_get. If the reviewer asks you to change the review, first read session_get_instructions({}) for its guidelines and each component's fields (the review exists, so skip creating one), then edit it with session_edit; do not write files to do it.`,
             ]
           : reach?.kind === "cli"
             ? [
-                `Whiteboard's CLI reads and changes this review from your shell: its sessionId is "${reviewId}". Read it with \`${reach.command} api session_get '{"sessionId":"${reviewId}"}'\`. If the reviewer asks you to change the review, edit it with \`${reach.command} api session_edit '<json>'\`; do not write files to do it. \`${reach.command} api tools\` lists each tool's input.`,
+                `Whiteboard's CLI reads and changes this review from your shell: its sessionId is "${reviewId}". Read it with \`${reach.command} api session_get '{"sessionId":"${reviewId}"}'\`. If the reviewer asks you to change the review, first read \`${reach.command} api session_get_instructions '{}'\` for its guidelines and each component's fields (the review exists, so skip creating one), then edit it with \`${reach.command} api session_edit '<json>'\`; do not write files to do it. \`${reach.command} api tools\` lists each tool's input.`,
               ]
             : []),
         "",
@@ -1309,6 +1310,32 @@ export function createReviewApi(
       return context.json({ threads: store.askHistory.list(reviewId) });
     });
 
+    // A saved conversation, without starting its agent.
+    app.get("/:id/ask/:threadId", (context) => {
+      const record = store.askHistory.get(context.req.param("threadId"));
+
+      if (record?.reviewId !== context.req.param("id"))
+        throw new ReviewInputError("This conversation was not found.", 404);
+      const target = record.selection.target;
+
+      return context.json({
+        id: record.id,
+        agent: record.agent,
+        agentName: askAgents[record.agent].name,
+        status: "idle",
+        readOnly: true,
+        bypass: false,
+        head: record.head,
+        cwd: record.cwd,
+        title: record.title,
+        selection: {
+          title: record.selection.title,
+          quote: target.kind === "text" ? target.quote : undefined,
+        },
+        entries: record.entries ?? [],
+      } satisfies AskThreadState);
+    });
+
     // A saved conversation: attach to it if it is still running, else start
     // the agent and load it, at the commit it was asked about.
     app.post("/:id/ask/:threadId/open", async (context) => {

package/src/review-api/store.ts

@@ -1237,11 +1237,6 @@ export class ReviewStore {
         ...(applied?.children && { children: applied.children }),
       };
 
-      if (snapshot.staleSources?.length)
-        warnings.push(
-          "Some authored source ranges changed. Update their references before presenting this review.",
-        );
-
       if (warnings.length) result.warnings = warnings;
 
       this.commitCommand(