@dev.fast/whiteboard 0.3.4-preview.20261019.127 → 0.3.4

package/dist/{mcp-ovmiUE_x.js → mcp-D1hziANa.js}

@@ -1,7 +1,7 @@
-import { a as toolResultText, s as ReviewApiError } from "./agent-client-Dtc-Kn_w.js";
+import { a as toolResultText, s as ReviewApiError } from "./agent-client-BOOi7EUR.js";
 import { t as ReviewInputError } from "./input-error-OLgB_h31.js";
-import { n as REVIEW_STATUS_TOOL, t as authoringTools } from "./authoring-tools-B7KU04f7.js";
-import { i as publicTool, n as RECOVERY, r as callPublicTool, t as toolFailure } from "./tool-failure-DvI-gys2.js";
+import { n as REVIEW_STATUS_TOOL, t as authoringTools } from "./authoring-tools-BNwuMnz6.js";
+import { i as publicTool, n as RECOVERY, r as callPublicTool, t as toolFailure } from "./tool-failure-C9MpbXJJ.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";
@@ -31,13 +31,14 @@ const RELOAD_TOOLS = "Whiteboard is running now, but this session listed Whitebo
 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 review a branch, a change or a pull request, or to explain a system in Whiteboard), author a Whiteboard with the default topic.",
+		...context.scratchpadAvailable ? ["When the user asks in conversation to be shown how code works or wants a diagram, without asking for a Whiteboard, call session_capabilities; if it reports scratchpadEnabled and desktopAvailable, draw on the Whiteboard scratchpad rather than answering only in chat, starting with session_get_instructions({topic:\"scratchpad\"})."] : [],
 		...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\"})."] : [],
 		"Never read or write Whiteboard files or SQL."
 	].join(" ");
 }
 async function serveReviewMcp(connect, stdin, stdout, stderr = process.stderr, traceEnabled = false, offlineStatus, onToolCall, environmentAgent) {
 	const instructionsTool = {
-		...authoringTools(traceEnabled).find((tool) => tool.name === "review_get_instructions"),
+		...authoringTools(false, traceEnabled).find((tool) => tool.name === "review_get_instructions"),
 		name: "session_get_instructions"
 	};
 	const server = new Server({
@@ -75,7 +76,10 @@ async function serveReviewMcp(connect, stdin, stdout, stderr = process.stderr, t
 		const always = [instructionsTool, REVIEW_STATUS_TOOL].map((fallback) => tools.find((tool) => tool.name === fallback.name) ?? fallback);
 		return { tools: [...always, ...tools.filter((tool) => !always.includes(tool))].map(({ name, description, inputSchema }) => ({
 			name,
-			description: name === instructionsTool.name ? `${mcpAuthoringGuidance({ traceEnabled })}\n\n${description}` : description,
+			description: name === instructionsTool.name ? `${mcpAuthoringGuidance({
+				scratchpadAvailable: true,
+				traceEnabled
+			})}\n\n${description}` : description,
 			inputSchema
 		})) };
 	});

package/dist/{package-paths-BWxO-UEL.js → package-paths-C--__jEv.js}

@@ -1,5 +1,5 @@
 import { c as jsonString, o as jsonObject, u as parseJsonText } from "./src-X9phtB2j.js";
-import "./review-home-paths-D8AoCdd2.js";
+import "./review-home-paths-j2BRAaNv.js";
 import { readFileSync } from "node:fs";
 import path from "node:path";
 import { fileURLToPath } from "node:url";

package/dist/profile-CYPmvBoA.js

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

package/dist/{profile-BadSkCkI.js → profile-DokFw782.js}

@@ -1,6 +1,6 @@
-import { a as projectOfCheckout, g as CheckoutGroups, o as groupCheckoutsAhead, s as initializeReviewStoreSchema, t as openLocalReviewStore } from "./local-data-Cpl4ENbJ.js";
-import "./review-home-paths-D8AoCdd2.js";
-import { j as withFileLock } from "./s3-config-CjcLyFV5.js";
+import { o as initializeReviewStoreSchema, t as openLocalReviewStore } from "./local-data-D_9tvdsp.js";
+import "./review-home-paths-j2BRAaNv.js";
+import { j as withFileLock } from "./s3-config-BTO12tCC.js";
 import { access, mkdir } from "node:fs/promises";
 import path from "node:path";
 import { DatabaseSync } from "node:sqlite";
@@ -18,11 +18,7 @@ async function openReviewProfile(home, options) {
 	});
 	const outcome = await withFileLock(path.join(home, ".review-profile-startup"), lockOptions, async () => {
 		for (const source of [path.join(home, "review-server", "reviews.db"), path.join(home, "reviews.db")]) await importHeadlessStore(home, source);
-		const database = path.join(home, "review-api.db");
-		return openLocalReviewStore(database, {
-			...options,
-			checkoutOwners: await groupCheckoutsAhead(database)
-		});
+		return openLocalReviewStore(path.join(home, "review-api.db"), options);
 	});
 	if (!outcome.acquired) throw new Error("Another Review process is initializing this profile. Retry shortly.");
 	return outcome.result;
@@ -49,27 +45,19 @@ async function importHeadlessStore(home, source) {
 			try {
 				if (database.prepare("SELECT 1 FROM headless.reviews s JOIN reviews t ON s.id=t.id LIMIT 1").get()) throw new Error(`Cannot merge ${source}: the shared store already contains one of its review IDs. Both databases are unchanged.`);
 				database.exec(`
-          INSERT INTO repositories(id,path,name) SELECT s.id,s.path,s.name FROM headless.repositories s WHERE NOT EXISTS (SELECT 1 FROM repositories t WHERE t.path=s.path);
+          INSERT INTO repositories SELECT s.* FROM headless.repositories s WHERE NOT EXISTS (SELECT 1 FROM repositories t WHERE t.path=s.path);
           CREATE TEMP TABLE repository_ids AS SELECT s.id old_id,t.id new_id FROM headless.repositories s JOIN repositories t ON s.path=t.path;
         `);
 				if (database.prepare(`SELECT 1 FROM headless.versions s LEFT JOIN repository_ids r ON json_extract(s.snapshot,'$.pins.repositoryId')=r.old_id WHERE r.old_id IS NULL LIMIT 1`).get()) throw new Error(`Cannot merge ${source}: a version refers to an unregistered repository. Both databases are unchanged.`);
 				if (database.prepare(`SELECT 1 FROM headless.resources s JOIN resources t ON s.id=t.id JOIN repository_ids r ON s.repository_id=r.old_id
           WHERE t.repository_id!=r.new_id OR t.kind!=s.kind OR t.mime_type!=s.mime_type OR t.data IS NOT s.data LIMIT 1`).get()) throw new Error(`Cannot merge ${source}: a resource ID has different content. Both databases are unchanged.`);
-				if (database.prepare("SELECT 1 FROM pragma_table_info('reviews') WHERE name='project_id'").get()) {
-					const groups = new CheckoutGroups(database);
-					for (const { new_id: repositoryId } of database.prepare("SELECT new_id FROM repository_ids").all()) projectOfCheckout(database, String(repositoryId), groups);
-					database.exec(`INSERT INTO reviews(id,version,next_id,project_id) SELECT s.id,s.version,s.next_id,p.project_id
-              FROM headless.reviews s JOIN headless.versions v ON v.review_id=s.id AND v.version=s.version
-              JOIN repository_ids r ON json_extract(v.snapshot,'$.pins.repositoryId')=r.old_id
-              JOIN repositories p ON p.id=r.new_id;`);
-				} else database.exec("INSERT INTO reviews(id,version,next_id) SELECT id,version,next_id FROM headless.reviews;");
 				database.exec(`
           INSERT OR IGNORE INTO resources SELECT s.id,r.new_id,s.kind,s.mime_type,s.data FROM headless.resources s JOIN repository_ids r ON s.repository_id=r.old_id;
-          INSERT INTO versions(review_id,version,snapshot,thread_id,author_kind) SELECT s.review_id,s.version,
+          INSERT INTO reviews SELECT * FROM headless.reviews;
+          INSERT INTO versions SELECT s.review_id,s.version,
             CASE WHEN json_type(s.snapshot,'$.target')='object'
               THEN json_set(s.snapshot,'$.pins.repositoryId',r.new_id,'$.target.repositoryId',r.new_id)
-              ELSE json_set(s.snapshot,'$.pins.repositoryId',r.new_id) END,
-            NULL, NULL
+              ELSE json_set(s.snapshot,'$.pins.repositoryId',r.new_id) END
             FROM headless.versions s JOIN repository_ids r ON json_extract(s.snapshot,'$.pins.repositoryId')=r.old_id;
           INSERT INTO review_attention SELECT * FROM headless.review_attention;
         `);

package/dist/{remote-attach-Dd-YobM6.js → remote-attach-CEJ-kvDY.js}

@@ -1,7 +1,7 @@
-import { i as readReviewPackageVersion } from "./package-paths-BWxO-UEL.js";
-import { a as readReviewServerHealth, l as serverNotReady } from "./server-discovery-CKjw3-6h.js";
-import { c as stopBackgroundServer, o as recordedBackgroundServer, r as ensureBackgroundServer } from "./background-server-C03RKxKS.js";
-import { t as ensureRemoteLanguageServer } from "./remote-language-server-BXk87KCr.js";
+import { i as readReviewPackageVersion } from "./package-paths-C--__jEv.js";
+import { a as readReviewServerHealth, l as serverNotReady } from "./server-discovery-Bey8IqXH.js";
+import { c as stopBackgroundServer, o as recordedBackgroundServer, r as ensureBackgroundServer } from "./background-server-CkvxA-6O.js";
+import { t as ensureRemoteLanguageServer } from "./remote-language-server-BTKBQ7x4.js";
 import path from "node:path";
 import { spawn } from "node:child_process";
 import { gt, valid } from "semver";

package/dist/{remote-extensions-CNXwqK0D.js → remote-extensions-Blcjr9Ze.js}

@@ -1,7 +1,7 @@
 import { u as parseJsonText } from "./src-X9phtB2j.js";
-import "./review-home-paths-D8AoCdd2.js";
-import { N as devReviewHome } from "./s3-config-CjcLyFV5.js";
-import { t as findReviewPackageRoot } from "./package-paths-BWxO-UEL.js";
+import "./review-home-paths-j2BRAaNv.js";
+import { N as devReviewHome } from "./s3-config-BTO12tCC.js";
+import { t as findReviewPackageRoot } from "./package-paths-C--__jEv.js";
 import { createHash } from "node:crypto";
 import { z } from "zod";
 import { createReadStream, createWriteStream } from "node:fs";

package/dist/{remote-extensions-VP-kfq0C.js → remote-extensions-GN8Gf_qG.js}

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

package/dist/{remote-language-server-BXk87KCr.js → remote-language-server-BTKBQ7x4.js}

@@ -1,8 +1,8 @@
-import "./review-home-paths-D8AoCdd2.js";
-import { A as processStartIdentity, j as withFileLock } from "./s3-config-CjcLyFV5.js";
-import { t as findReviewPackageRoot } from "./package-paths-BWxO-UEL.js";
-import { n as cliSpawn, s as spawnDetached } from "./background-server-C03RKxKS.js";
-import { r as remoteServerPaths, t as ensureRemoteExtensions } from "./remote-extensions-CNXwqK0D.js";
+import "./review-home-paths-j2BRAaNv.js";
+import { A as processStartIdentity, j as withFileLock } from "./s3-config-BTO12tCC.js";
+import { t as findReviewPackageRoot } from "./package-paths-C--__jEv.js";
+import { n as cliSpawn, s as spawnDetached } from "./background-server-CkvxA-6O.js";
+import { r as remoteServerPaths, t as ensureRemoteExtensions } from "./remote-extensions-Blcjr9Ze.js";
 import { randomBytes } from "node:crypto";
 import { z } from "zod";
 import { mkdir, readFile, rename, stat, writeFile } from "node:fs/promises";

package/dist/{remote-uninstall-DuOzsvaD.js → remote-uninstall-DW7Gdryh.js}

@@ -1,9 +1,9 @@
-import { C as REVIEW_REMOTE_WRAPPER_MARK, S as REVIEW_REMOTE_INSTALL_LOCK } from "./src-BiWNbcF_.js";
-import "./review-home-paths-D8AoCdd2.js";
-import { M as DEV_REVIEW_HOME_ENV } from "./s3-config-CjcLyFV5.js";
-import { c as stopBackgroundServer, o as recordedBackgroundServer } from "./background-server-C03RKxKS.js";
-import { i as whiteboardRemoteHome } from "./remote-extensions-CNXwqK0D.js";
-import { n as remoteLanguageServerGroups, r as stopProcessGroup } from "./remote-language-server-BXk87KCr.js";
+import { b as REVIEW_REMOTE_INSTALL_LOCK, x as REVIEW_REMOTE_WRAPPER_MARK } from "./src-DsM3jO8G.js";
+import "./review-home-paths-j2BRAaNv.js";
+import { M as DEV_REVIEW_HOME_ENV } from "./s3-config-BTO12tCC.js";
+import { c as stopBackgroundServer, o as recordedBackgroundServer } from "./background-server-CkvxA-6O.js";
+import { i as whiteboardRemoteHome } from "./remote-extensions-Blcjr9Ze.js";
+import { n as remoteLanguageServerGroups, r as stopProcessGroup } from "./remote-language-server-BTKBQ7x4.js";
 import { existsSync } from "node:fs";
 import { lstat, mkdir, readFile, readdir, rm, stat } from "node:fs/promises";
 import path from "node:path";

package/dist/{request-origin-B9qCPiVB.js → request-origin-Zcn_CjX4.js}

@@ -1,7 +1,6 @@
 import { a as jsonNumber, c as jsonString, i as jsonBoolean, n as isJsonObject } from "./src-X9phtB2j.js";
-import { _ as REVIEW_DIFF_LAYOUT_SCOPES, g as REVIEW_DIFF_LAYOUTS, w as REVIEW_THEME_CHOICES } from "./src-BiWNbcF_.js";
-import { t as SELECTION_UNAVAILABLE_REASONS } from "./lens-selection-CyJapcsl.js";
-import { n as askAgentIds } from "./thread-state-Do5LX10I.js";
+import { S as REVIEW_THEME_CHOICES, h as REVIEW_DIFF_LAYOUT_SCOPES, m as REVIEW_DIFF_LAYOUTS } from "./src-DsM3jO8G.js";
+import { n as askAgentIds } from "./thread-state-5pYr6_zM.js";
 import { z } from "zod";
 //#region src/telemetry-clean-text.ts
 const USER_DATA_REGEXES = [
@@ -156,12 +155,6 @@ const PEEK_VIA = [
 	"diff_definition"
 ];
 const PEEK_ROOT_KIND = ["range"];
-/** `unresolved`: progress finished without this selection; `error`: progress failed. */
-const PEEK_FAILURE_REASON = [
-	...SELECTION_UNAVAILABLE_REASONS,
-	"error",
-	"unresolved"
-];
 const LSP_FEATURE = [
 	"hover",
 	"goto_definition",
@@ -231,6 +224,7 @@ const SETTING_NAME = [
 	"telemetry_enabled",
 	"keymap",
 	"software_map_enabled",
+	"scratchpad_enabled",
 	"diffr_config",
 	"theme",
 	"ctrl_tab",
@@ -244,11 +238,11 @@ const REVIEW_OPENED_VIA = [
 	"cli",
 	"other"
 ];
-/** Where a review's source comes from: its target's kind. */
+/** Where a review's source comes from: its target's kind, or the one scratchpad. */
 const SESSION_SOURCE_KIND = [
 	"worktree",
 	"commits",
-	"project"
+	"scratchpad"
 ];
 /**
 * The authoring agent a review session is attributed to. This is the one
@@ -330,21 +324,12 @@ const INSTALL_LOCATION = [
 ];
 const DIFF_OPENED_VIA = [
 	"topbar",
-	"sidebar",
 	"lens",
 	"locate",
 	"file_picker"
 ];
 /** Which diff opened: a commit's, or a file diff drawn plain or structural. */
 const DIFF_OPENED_KIND = ["commit", "structural"];
-const DIFF_FAILED_REASON = [
-	"missing_binary",
-	"process_exit",
-	"stream_ended",
-	"aborted",
-	"file_error",
-	"other"
-];
 /** GitHub Enterprise is any GitHub host but github.com. */
 const PR_PROVIDER = [
 	"github",
@@ -371,12 +356,7 @@ const ONBOARDING_STEP = [
 	"run"
 ];
 const RUN_MODE = ["draw", "copy"];
-const NEW_WHITEBOARD_VIA = [
-	"button",
-	"command",
-	"sidebar"
-];
-const NEW_WHITEBOARD_MODE = [...RUN_MODE, "empty"];
+const NEW_WHITEBOARD_VIA = ["button", "command"];
 const FIND_SURFACE = ["review", "diff"];
 const DISCORD_VIA = [
 	"topbar",
@@ -697,12 +677,9 @@ const UI_TELEMETRY_EVENTS = {
 			via: DIFF_OPENED_VIA
 		}
 	},
-	diff_failed: {
-		event: "review_diff_failed",
-		properties: {
-			reason: DIFF_FAILED_REASON,
-			files: "number"
-		}
+	scratchpad_opened: {
+		event: "review_scratchpad_opened",
+		properties: {}
 	},
 	discord_clicked: {
 		event: "review_discord_clicked",
@@ -722,10 +699,7 @@ const UI_TELEMETRY_EVENTS = {
 	},
 	peek_resolve_failed: {
 		event: "review_peek_resolve_failed",
-		properties: {
-			root_kind: PEEK_ROOT_KIND,
-			reason: PEEK_FAILURE_REASON
-		}
+		properties: { root_kind: PEEK_ROOT_KIND }
 	},
 	review_deleted: {
 		event: "review_review_deleted",
@@ -770,7 +744,7 @@ const UI_TELEMETRY_EVENTS = {
 	new_whiteboard_started: {
 		event: "review_new_whiteboard_started",
 		properties: {
-			mode: NEW_WHITEBOARD_MODE,
+			mode: RUN_MODE,
 			embedded: "boolean",
 			agent: askAgentIds
 		}
@@ -908,13 +882,6 @@ function isValidReviewAppSessionId(value) {
 */
 const REVIEW_VIA_HEADER = "x-review-via";
 const REVIEW_AGENT_HEADER = "x-review-agent";
-/** The Ask thread a request comes from. The CLI reads it from ASK_THREAD_ENV,
-* which Desktop sets for the thread's agent. */
-const REVIEW_THREAD_HEADER = "x-review-thread";
-const ASK_THREAD_ENV = "DEV_REVIEW_ASK_THREAD";
-/** Set by Desktop and the canvas on a command a person gave, such as a new
-* board from the sidebar; its version is credited to them. */
-const REVIEW_PERSON_HEADER = "x-review-person";
 const viaSchema = z.enum([
 	"api",
 	"mcp",
@@ -930,4 +897,4 @@ function reviewRequestOrigin(headers) {
 	} : { via };
 }
 //#endregion
-export { REVIEW_VIA_HEADER as a, REVIEW_APP_SESSION_ID_HEADER as c, sanitizeUiTelemetryEvent as d, cleanTelemetryText as f, REVIEW_THREAD_HEADER as i, REVIEW_TELEMETRY_TABS as l, REVIEW_AGENT_HEADER as n, reviewRequestOrigin as o, REVIEW_PERSON_HEADER as r, BUNDLE_FRAME_PATTERN as s, ASK_THREAD_ENV as t, isReportableCleanedMessage as u };
+export { REVIEW_APP_SESSION_ID_HEADER as a, sanitizeUiTelemetryEvent as c, BUNDLE_FRAME_PATTERN as i, cleanTelemetryText as l, REVIEW_VIA_HEADER as n, REVIEW_TELEMETRY_TABS as o, reviewRequestOrigin as r, isReportableCleanedMessage as s, REVIEW_AGENT_HEADER as t };

package/dist/{review-app-launcher-C7el2Uoy.js → review-app-launcher-CkwLclzR.js}

@@ -1,5 +1,5 @@
-import { o as parseReviewVerbResponse } from "./src-BiWNbcF_.js";
-import { b as selectReviewInstance, m as healthyReviewInstance, v as reviewInstanceStartHint, y as reviewInstanceUnavailable } from "./server-discovery-CKjw3-6h.js";
+import { o as parseReviewVerbResponse } from "./src-DsM3jO8G.js";
+import { b as selectReviewInstance, m as healthyReviewInstance, v as reviewInstanceStartHint, y as reviewInstanceUnavailable } from "./server-discovery-Bey8IqXH.js";
 import { closeSync, existsSync, fstatSync, mkdtempSync, openSync, readSync } from "node:fs";
 import path from "node:path";
 import { homedir, tmpdir } from "node:os";

package/dist/{review-home-paths-D8AoCdd2.js → review-home-paths-j2BRAaNv.js}

@@ -1,7 +1,7 @@
-import { E as runVcs, d as git, f as gitAt, x as resolveRepoContext } from "./src-WPaQn75m.js";
+import { d as git, f as gitAt, y as resolveRepoContext } from "./src-BMPuxWv4.js";
 import { a as jsonNumber, c as jsonString, i as jsonBoolean, m as isStringValue, n as isJsonObject, o as jsonObject, r as jsonArray, t as isJsonArray, u as parseJsonText } from "./src-X9phtB2j.js";
 import { a as TRACE_STORE_CLIENT_ID, b as commitShaSchema, c as deleteStoreResponseSchema, d as listUploadsResponseSchema, f as storeErrorCodeSchema, g as uploadManifestMismatch, h as traceObjectNameSchema, i as storeRoutes, l as listSessionsResponseSchema, m as storeResponseSchema, n as DEVICE_TOKEN_PATH, o as beginUploadResponseSchema, p as storeErrorEnvelopeSchema, r as SESSION_PATH, s as completeUploadResponseSchema, t as DEVICE_CODE_PATH, u as listUploadsQuerySchema, x as sessionIdSchema } from "./src-CGP5ytbV.js";
-import { E as writeTraceConfigFile, F as writeFileAtomic, I as writeFileAtomicAsync, L as writePrivateJsonAtomic, N as devReviewHome, P as normalizeStoreOrigin, S as readTraceConfigFile, _ as emptyTraceConfig, b as hostedCaptureEnabled, c as resolveS3Setup, d as traceEnvPath, f as traceEnvValue$1, g as currentStore, h as TraceConfigurationError, i as isS3MockMode, l as s3CacheIdentity, m as DEFAULT_HOSTED_ORIGIN, n as clearTraceEnvCache$1, p as traceSettingsPath, u as s3MockRoot, v as enabledOriginsOf, w as s3Store, x as hostedOrigin, y as hasHostedStore } from "./s3-config-CjcLyFV5.js";
+import { E as writeTraceConfigFile, F as writeFileAtomic, I as writeFileAtomicAsync, L as writePrivateJsonAtomic, N as devReviewHome, P as normalizeStoreOrigin, S as readTraceConfigFile, _ as emptyTraceConfig, b as hostedCaptureEnabled, c as resolveS3Setup, d as traceEnvPath, f as traceEnvValue$1, g as currentStore, h as TraceConfigurationError, i as isS3MockMode, l as s3CacheIdentity, m as DEFAULT_HOSTED_ORIGIN, n as clearTraceEnvCache$1, p as traceSettingsPath, u as s3MockRoot, v as enabledOriginsOf, w as s3Store, x as hostedOrigin, y as hasHostedStore } from "./s3-config-BTO12tCC.js";
 import { t as errorMessage } from "./error-message-OtiDonty.js";
 import crypto, { createHash, randomBytes } from "node:crypto";
 import { z } from "zod";
@@ -9,7 +9,8 @@ import { accessSync, closeSync, constants, createReadStream, createWriteStream,
 import { chmod, mkdir, mkdtemp, readFile, readdir, rename, rm, stat, writeFile } from "node:fs/promises";
 import path from "node:path";
 import os, { homedir, tmpdir } from "node:os";
-import { spawn } from "node:child_process";
+import { execFile, spawn } from "node:child_process";
+import { promisify } from "node:util";
 import { Readable, Transform } from "node:stream";
 import { AsyncLocalStorage } from "node:async_hooks";
 import { pipeline } from "node:stream/promises";
@@ -400,18 +401,14 @@ async function runStoreLogin(input) {
 	} catch (error) {
 		return failWithJsonError(output, "login", error instanceof StoreApiError ? error.message : "Could not start the login.");
 	}
-	const pending = {
-		url: device.verification_uri_complete,
-		userCode: device.user_code
-	};
 	humanStream(output).write(`Open ${device.verification_uri_complete}\n`);
 	humanStream(output).write(`Code: ${device.user_code}\n`);
 	emitJsonEvent(output, {
 		event: "login",
 		status: "pending",
-		...pending
+		url: device.verification_uri_complete,
+		userCode: device.user_code
 	});
-	input.onDeviceCode?.(pending);
 	if (!input.noBrowser) await openUrl(device.verification_uri_complete);
 	let intervalMs = Math.max(1, device.interval) * 1e3;
 	const expiresAt = Date.now() + device.expires_in * 1e3;
@@ -2540,7 +2537,7 @@ function isTraceStorageConfigured(scope = {}) {
 }
 /** Loads the direct-store implementation on demand, without opening a store. */
 async function loadS3TraceStorage() {
-	return (await import("./s3-CsG2EWED.js")).S3TraceStorage;
+	return (await import("./s3-DtJFWC0G.js")).S3TraceStorage;
 }
 //#endregion
 //#region ../trace-core/src/trace-upload-status.ts
@@ -5266,7 +5263,7 @@ async function configureTraceMachine(input) {
 	if (input.verify !== false) {
 		if (env.TRACE_R2_MODE === "mock") verifiedAt = (/* @__PURE__ */ new Date()).toISOString();
 		else {
-			const { S3TraceStorage } = await import("./s3-CsG2EWED.js");
+			const { S3TraceStorage } = await import("./s3-DtJFWC0G.js");
 			const readiness = await S3TraceStorage.fromCredentials({
 				endpoint: credentials.endpoint,
 				bucket: credentials.bucket,
@@ -5400,6 +5397,7 @@ async function writeSettings(filePath, settings) {
 }
 //#endregion
 //#region ../trace-core/src/trace-hook-runner.ts
+const execFileAsync = promisify(execFile);
 const SESSION_ID_REGEX = /^[A-Za-z0-9][A-Za-z0-9._-]{7,127}$/;
 /**
 * Starts `whiteboard trace sync` detached for one session. The attempt names
@@ -5481,7 +5479,7 @@ async function runTraceHook(input) {
 	], { allowFailure: true });
 	let sessionFilePath = null;
 	if (gitPathResult.ok && gitPathResult.stdout.trim()) sessionFilePath = gitPathResult.stdout.trim();
-	const jjRootResult = await runVcs("jj", ["root"], { cwd: input.cwd }).catch(() => null);
+	const jjRootResult = await execFileAsync("jj", ["root"], { cwd: input.cwd }).catch(() => null);
 	if (!sessionFilePath && jjRootResult?.stdout.trim()) sessionFilePath = path.join(jjRootResult.stdout.trim(), ".jj", "agent-session");
 	let remainingSessions = [];
 	if (sessionFilePath) {
@@ -5500,24 +5498,24 @@ async function runTraceHook(input) {
 	if (jjRootResult?.stdout.trim()) {
 		if ((isStart || isHeartbeat) && remainingSessions.length > 0) {
 			const templateVal = jjCommitTrailersConfigValue(remainingSessions);
-			await runVcs("jj", [
+			await execFileAsync("jj", [
 				"config",
 				"set",
 				"--repo",
 				"templates.commit_trailers",
-				{ value: templateVal }
+				templateVal
 			], { cwd: input.cwd }).catch(() => void 0);
 		} else if (isEnd) {
 			if (remainingSessions.length > 0) {
 				const templateVal = jjCommitTrailersConfigValue(remainingSessions);
-				await runVcs("jj", [
+				await execFileAsync("jj", [
 					"config",
 					"set",
 					"--repo",
 					"templates.commit_trailers",
-					{ value: templateVal }
+					templateVal
 				], { cwd: input.cwd }).catch(() => void 0);
-			} else await runVcs("jj", [
+			} else await execFileAsync("jj", [
 				"config",
 				"unset",
 				"--repo",
@@ -5736,7 +5734,7 @@ async function runTraceStatus(input) {
 		input.stderr.write("Upload status filters require hosted storage.\n");
 		return 1;
 	}
-	const [{ S3TraceStorage }, { describeS3Setup, noTraceConfigurationMessage }] = await Promise.all([import("./s3-CsG2EWED.js"), import("./s3-config-zz4GWh2T.js")]);
+	const [{ S3TraceStorage }, { describeS3Setup, noTraceConfigurationMessage }] = await Promise.all([import("./s3-DtJFWC0G.js"), import("./s3-config-CSy3fx8H.js")]);
 	const setup = describeS3Setup(input.scope.env);
 	input.stdout.write(`Checking trace configuration (${setup.envPath})…\n`);
 	for (const failure of await listTraceSyncFailures(input.scope.devHome)) input.stdout.write(describeTraceSyncFailure(failure));
@@ -6403,4 +6401,4 @@ function reviewDesktopStateDir(env = process.env) {
 	return path.join(reviewDesktopRoot(env), "state");
 }
 //#endregion
-export { resolveTraceCommand as $, runTraceDeny as A, TraceStorageDeniedError as B, disableTraceMachine as C, listReviewTraceSessions as D, traceMachineStatus as E, describeSelection as F, listTraceRepositoryRoots as G, resolveTraceRepositoryTarget as H, isTraceStorageConfigured as I, HOSTED_CAPTURE_SCOPE_DESCRIPTION as J, traceRepositoryStatus as K, loadS3TraceStorage as L, runTraceSessions as M, runTraceStoreDelete as N, loadReviewAgentTrace as O, runTraceStoreInfo as P, removeAgentTraceHook as Q, resolveTraceStorage as R, configureTraceMachine as S, traceMachineEnabled as T, disableAllTraceRepositories as U, requireTraceConsent as V, enableTraceRepository as W, describeTraceHookOwners as X, AGENT_TRACE_HOOK_AGENTS as Y, installHarnessHooks as Z, runTraceRepair as _, reviewInstancesDir as a, readStoreAuth as at, runTraceGitHook as b, registerTraceCommands as c, runStoreWhoami as ct, runTraceList as d, withStoreAuthorization as dt, shellQuote as et, runTracePull as f, emitJsonEvent as ft, runTraceInstallMachine as g, runTraceEnable as h, jsonRequestedInArgv as ht, reviewInstanceDiscoveryPath as i, clearStoreAuth as it, runTraceOnboard as j, runTraceAllow as k, resolveTraceReadStorage as l, StoreApiError as lt, runTraceDisable as m, humanStream as mt, reviewDesktopStateDir as n, traceScope as nt, reviewLegacyDiscoveryPath as o, runStoreLogin as ot, runTraceShow as p, failWithJsonError as pt, readTraceUserConfig as q, reviewDevInstanceKey as r, DEFAULT_STORE_ORIGIN as rt, safeStorageSegment as s, runStoreLogout as st, reviewDefaultInstancePath as t, traceHomeDir as tt, runTraceBlame as u, StoreClient as ut, runTraceStatus as v, readLegacyCaptureSettings as w, runTraceHook as x, runTraceSync as y, selectTraceStorage as z };
+export { resolveTraceCommand as $, runTraceDeny as A, TraceStorageDeniedError as B, disableTraceMachine as C, listReviewTraceSessions as D, traceMachineStatus as E, describeSelection as F, listTraceRepositoryRoots as G, resolveTraceRepositoryTarget as H, isTraceStorageConfigured as I, HOSTED_CAPTURE_SCOPE_DESCRIPTION as J, traceRepositoryStatus as K, loadS3TraceStorage as L, runTraceSessions as M, runTraceStoreDelete as N, loadReviewAgentTrace as O, runTraceStoreInfo as P, removeAgentTraceHook as Q, resolveTraceStorage as R, configureTraceMachine as S, traceMachineEnabled as T, disableAllTraceRepositories as U, requireTraceConsent as V, enableTraceRepository as W, describeTraceHookOwners as X, AGENT_TRACE_HOOK_AGENTS as Y, installHarnessHooks as Z, runTraceRepair as _, reviewInstancesDir as a, openUrlInBrowser as at, runTraceGitHook as b, registerTraceCommands as c, runStoreLogout as ct, runTraceList as d, StoreClient as dt, shellQuote as et, runTracePull as f, withStoreAuthorization as ft, runTraceInstallMachine as g, jsonRequestedInArgv as gt, runTraceEnable as h, humanStream as ht, reviewInstanceDiscoveryPath as i, clearStoreAuth as it, runTraceOnboard as j, runTraceAllow as k, resolveTraceReadStorage as l, runStoreWhoami as lt, runTraceDisable as m, failWithJsonError as mt, reviewDesktopStateDir as n, traceScope as nt, reviewLegacyDiscoveryPath as o, readStoreAuth as ot, runTraceShow as p, emitJsonEvent as pt, readTraceUserConfig as q, reviewDevInstanceKey as r, DEFAULT_STORE_ORIGIN as rt, safeStorageSegment as s, runStoreLogin as st, reviewDefaultInstancePath as t, traceHomeDir as tt, runTraceBlame as u, StoreApiError as ut, runTraceStatus as v, readLegacyCaptureSettings as w, runTraceHook as x, runTraceSync as y, selectTraceStorage as z };

package/dist/{review-telemetry-BsoRMC2w.js → review-telemetry-BLNwYnLV.js}

@@ -1,9 +1,9 @@
 import { c as jsonString, l as jsonValueSchema, o as jsonObject, u as parseJsonText } from "./src-X9phtB2j.js";
-import "./src-BiWNbcF_.js";
-import "./review-home-paths-D8AoCdd2.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-BWxO-UEL.js";
-import "./request-origin-B9qCPiVB.js";
+import "./src-DsM3jO8G.js";
+import "./review-home-paths-j2BRAaNv.js";
+import { F as writeFileAtomic, N as devReviewHome, j as withFileLock, k as processIsAlive } from "./s3-config-BTO12tCC.js";
+import { i as readReviewPackageVersion, t as findReviewPackageRoot } from "./package-paths-C--__jEv.js";
+import "./request-origin-Zcn_CjX4.js";
 import { createHmac, randomUUID } from "node:crypto";
 import { z } from "zod";
 import { existsSync, readFileSync, rmSync } from "node:fs";

package/dist/runtime-CKiWhGrf.js

@@ -0,0 +1,52 @@
+import path from "node:path";
+import { spawn } from "node:child_process";
+//#region src/runtime.ts
+async function resolveReviewRoot(cwd, execFile = execFilePromise) {
+	const initialCwd = path.resolve(cwd);
+	try {
+		const { stdout } = await execFile("jj", ["root"], { cwd: initialCwd });
+		const root = stdout.trim();
+		if (root) return path.resolve(root);
+	} catch {}
+	try {
+		const { stdout } = await execFile("git", [
+			"-C",
+			initialCwd,
+			"rev-parse",
+			"--show-toplevel"
+		]);
+		const root = stdout.trim();
+		if (root) return path.resolve(root);
+	} catch {}
+	return initialCwd;
+}
+function execFilePromise(file, args, options = {}) {
+	return execFileUntraced(file, args, options);
+}
+function execFileUntraced(file, args, options = {}) {
+	return new Promise((resolve, reject) => {
+		const child = spawn(file, args, {
+			cwd: options.cwd,
+			stdio: [
+				"ignore",
+				"pipe",
+				"pipe"
+			]
+		});
+		let stdout = "";
+		let stderr = "";
+		child.stdout?.on("data", (chunk) => {
+			stdout += String(chunk);
+		});
+		child.stderr?.on("data", (chunk) => {
+			stderr += String(chunk);
+		});
+		child.on("error", reject);
+		child.on("exit", (code) => {
+			if (code === 0) resolve({ stdout });
+			else reject(new Error(stderr.trim() || `${file} exited with ${code}`));
+		});
+	});
+}
+//#endregion
+export { resolveReviewRoot as t };

package/dist/runtime.d.ts

@@ -1,6 +1,6 @@
 //#region src/runtime.d.ts
 declare function resolveReviewRoot(cwd: string, execFile?: typeof execFilePromise): Promise<string>;
-declare function execFilePromise(file: "git" | "jj", args: string[], options?: {
+declare function execFilePromise(file: string, args: string[], options?: {
   cwd?: string;
 }): Promise<{
   stdout: string;

package/dist/runtime.js

@@ -1,2 +1,2 @@
-import { t as resolveReviewRoot } from "./runtime-Bj2tD33R.js";
+import { t as resolveReviewRoot } from "./runtime-CKiWhGrf.js";
 export { resolveReviewRoot };

package/dist/{s3-CsG2EWED.js → s3-DtJFWC0G.js}

@@ -1,6 +1,6 @@
 import { a as jsonNumber, c as jsonString, o as jsonObject, r as jsonArray, u as parseJsonText } from "./src-X9phtB2j.js";
 import { S as sessionMetaSchema, b as commitShaSchema, y as byCommitSchema } from "./src-CGP5ytbV.js";
-import { i as isS3MockMode, l as s3CacheIdentity, s as resolveS3Credentials, u as s3MockRoot } from "./s3-config-CjcLyFV5.js";
+import { i as isS3MockMode, l as s3CacheIdentity, s as resolveS3Credentials, u as s3MockRoot } from "./s3-config-BTO12tCC.js";
 import { t as errorMessage } from "./error-message-OtiDonty.js";
 import { existsSync, mkdirSync, readFileSync, readdirSync, rmSync, statSync, writeFileSync } from "node:fs";
 import path from "node:path";

package/dist/{s3-config-CjcLyFV5.js → s3-config-BTO12tCC.js}

@@ -169,13 +169,11 @@ const DEV_REVIEW_HOME_ENV = "DEV_REVIEW_HOME";
 * The one resolver for the Review home directory. Every reader of
 * DEV_REVIEW_HOME calls this, so an untrimmed or empty value cannot make two
 * modules disagree about the directory. An empty value means "use the
-* default", which is ~/.dev-preview for preview.
+* default".
 */
 function devReviewHome(env = process.env, homeDir = os.homedir()) {
 	const override = env[DEV_REVIEW_HOME_ENV]?.trim();
-	if (override) return path.resolve(override);
-	const channel = env.DEV_FAST_REVIEW_CHANNEL?.trim() || env.DEV_REVIEW_INSTANCE?.trim();
-	return path.join(homeDir, channel === "preview" ? ".dev-preview" : ".dev");
+	return override ? path.resolve(override) : path.join(homeDir, ".dev");
 }
 //#endregion
 //#region ../trace-core/src/with-file-lock.ts

package/dist/{s3-config-zz4GWh2T.js → s3-config-CSy3fx8H.js}

@@ -1,2 +1,2 @@
-import { a as noTraceConfigurationMessage, c as resolveS3Setup, d as traceEnvPath, f as traceEnvValue, i as isS3MockMode, l as s3CacheIdentity, n as clearTraceEnvCache, o as readTraceEnvFile, p as traceSettingsPath, r as describeS3Setup, s as resolveS3Credentials, t as S3_DEFAULT_REGION, u as s3MockRoot } from "./s3-config-CjcLyFV5.js";
+import { a as noTraceConfigurationMessage, c as resolveS3Setup, d as traceEnvPath, f as traceEnvValue, i as isS3MockMode, l as s3CacheIdentity, n as clearTraceEnvCache, o as readTraceEnvFile, p as traceSettingsPath, r as describeS3Setup, s as resolveS3Credentials, t as S3_DEFAULT_REGION, u as s3MockRoot } from "./s3-config-BTO12tCC.js";
 export { S3_DEFAULT_REGION, clearTraceEnvCache, describeS3Setup, isS3MockMode, noTraceConfigurationMessage, readTraceEnvFile, resolveS3Credentials, resolveS3Setup, s3CacheIdentity, s3MockRoot, traceEnvPath, traceEnvValue, traceSettingsPath };