@dev.fast/whiteboard 0.3.3 → 0.3.4-preview.20261019.125

package/src/review-api/workspaces.ts

@@ -5,7 +5,12 @@ import path from "node:path";
 import { DatabaseSync } from "node:sqlite";
 
 import { git, gitCommonDir } from "@dev.fast/local-vcs";
-import { errorMessage, processIsAlive } from "@dev.fast/trace-core";
+import {
+  errorMessage,
+  pidReused,
+  processIsAlive,
+  processStartIdentity,
+} from "@dev.fast/trace-core";
 import { reviewManagedCheckoutRoot } from "@review/review-checkout-paths.js";
 import {
   ensureReviewPinnedCheckout,
@@ -48,9 +53,6 @@ interface Environment extends WorkspaceStatus {
   role: "base" | "head";
 }
 
-const OWNED_ELSEWHERE =
-  "Another Whiteboard server is managing this review's language workspaces. Retry once it stops.";
-
 /** Local lifecycle only: source and authored history never depend on preparation.
  * Status/queue/process lifecycle follows #334, retaining the legacy prepare config
  * and per-review managed checkout layout instead of a new settings system.
@@ -75,11 +77,21 @@ export class ReviewWorkspaces {
     this.db.exec(
       "PRAGMA journal_mode=WAL; CREATE TABLE IF NOT EXISTS pinned_environments(id TEXT PRIMARY KEY, value TEXT NOT NULL)",
     );
+
     // Desktops sharing a profile each own the reviews they prepare; the
     // lease stops a second process from preparing or collecting them. The
     // profile-wide lock it replaces only ever shipped in preview builds.
+    // Older leases have no start identity and may name a reused pid.
+    if (
+      !this.db
+        .prepare(
+          "SELECT 1 FROM pragma_table_info('workspace_leases') WHERE name='started'",
+        )
+        .get()
+    )
+      this.db.exec("DROP TABLE IF EXISTS workspace_leases");
     this.db.exec(
-      "DROP TABLE IF EXISTS workspace_owner; CREATE TABLE IF NOT EXISTS workspace_leases(review_id TEXT PRIMARY KEY,owner TEXT NOT NULL,pid INTEGER NOT NULL)",
+      "DROP TABLE IF EXISTS workspace_owner; CREATE TABLE IF NOT EXISTS workspace_leases(review_id TEXT PRIMARY KEY,owner TEXT NOT NULL,pid INTEGER NOT NULL,started TEXT)",
     );
     this.db.exec("BEGIN IMMEDIATE");
 
@@ -284,7 +296,8 @@ export class ReviewWorkspaces {
   }
 
   async remove(reviewId: string) {
-    if (!this.claim(reviewId)) throw new ReviewInputError(OWNED_ELSEWHERE, 409);
+    if (!this.claim(reviewId))
+      throw new ReviewInputError(this.ownedElsewhere(reviewId), 409);
     await Promise.all(this.requests.values());
     this.collect(undefined, reviewId);
     await this.cleanup;
@@ -297,9 +310,19 @@ export class ReviewWorkspaces {
   }
 
   private lease(reviewId: string) {
-    return this.db
-      .prepare("SELECT owner,pid FROM workspace_leases WHERE review_id=?")
+    const row = this.db
+      .prepare(
+        "SELECT owner,pid,started FROM workspace_leases WHERE review_id=?",
+      )
       .get(reviewId);
+
+    return (
+      row && {
+        owner: String(row.owner),
+        pid: Number(row.pid),
+        started: String(row.started ?? ""),
+      }
+    );
   }
 
   private leasedElsewhere(reviewId: string): boolean {
@@ -308,10 +331,20 @@ export class ReviewWorkspaces {
     return Boolean(
       lease &&
       lease.owner !== this.ownerId &&
-      processIsAlive(Number(lease.pid)),
+      processIsAlive(lease.pid) &&
+      !pidReused(lease.pid, lease.started),
     );
   }
 
+  /** The holder may release its lease after `claim` fails. */
+  private ownedElsewhere(reviewId: string) {
+    const pid = this.lease(reviewId)?.pid;
+
+    return pid
+      ? `Whiteboard process ${pid} is managing this review's language workspaces. Quit that Whiteboard Desktop or run \`whiteboard server stop\`, then retry.`
+      : "Another Whiteboard process was managing this review's language workspaces. Retry.";
+  }
+
   /** Takes the review's lease unless another live Desktop holds it. */
   private claim(reviewId: string): boolean {
     // Status reads claim on every poll; holding the lease needs no write lock.
@@ -323,8 +356,13 @@ export class ReviewWorkspaces {
 
       if (free)
         this.db
-          .prepare("INSERT OR REPLACE INTO workspace_leases VALUES(?,?,?)")
-          .run(reviewId, this.ownerId, process.pid);
+          .prepare("INSERT OR REPLACE INTO workspace_leases VALUES(?,?,?,?)")
+          .run(
+            reviewId,
+            this.ownerId,
+            process.pid,
+            processStartIdentity(process.pid),
+          );
       this.db.exec("COMMIT");
 
       return free;
@@ -437,7 +475,7 @@ export class ReviewWorkspaces {
               rootPath: null,
               generation: "",
               state: "pending",
-              log: OWNED_ELSEWHERE,
+              log: this.ownedElsewhere(reviewId),
             },
       );
     }
@@ -607,7 +645,8 @@ export class ReviewWorkspaces {
 
     if (this.jobs.has(id)) return this.status(environment);
 
-    if (!this.claim(reviewId)) throw new ReviewInputError(OWNED_ELSEWHERE, 409);
+    if (!this.claim(reviewId))
+      throw new ReviewInputError(this.ownedElsewhere(reviewId), 409);
 
     if (environment.state === "cleanup-failed") {
       this.collect();

package/src/review-api/worktree-events.ts

@@ -1,8 +1,6 @@
-import { execFile } from "node:child_process";
 import { isAbsolute, relative } from "node:path";
-import { promisify } from "node:util";
 
-const exec = promisify(execFile);
+import { runVcs } from "@dev.fast/local-vcs";
 
 const normalize = (name: string) => name.replaceAll("\\", "/").normalize("NFC");
 
@@ -53,7 +51,7 @@ export async function ignoredDirectories(
   gitDirectory?: string,
 ): Promise<string[]> {
   try {
-    const { stdout } = await exec(
+    const { stdout } = await runVcs(
       "git",
       [
         "-C",
@@ -105,7 +103,7 @@ export async function gitDirectoryPrefix(
   common: string,
 ): Promise<string | null> {
   try {
-    const { stdout } = await exec("git", [
+    const { stdout } = await runVcs("git", [
       "-C",
       rootPath,
       "rev-parse",

package/src/review-api/worktree-source.ts

@@ -1,9 +1,7 @@
-import { execFile } from "node:child_process";
 import { createHash } from "node:crypto";
 import { dirname, isAbsolute, relative, resolve, sep } from "node:path";
-import { promisify } from "node:util";
 
-import { type LocalVcs, gitCommonDir } from "@dev.fast/local-vcs";
+import { type LocalVcs, gitCommonDir, runVcs } from "@dev.fast/local-vcs";
 import { isMissingFileError } from "@review/fs-utils.js";
 
 import { checkoutFs } from "./checkout-fs.js";
@@ -11,8 +9,6 @@ import { ReviewInputError } from "./document.js";
 
 const { lstat, readFile, readlink, realpath } = checkoutFs;
 
-const exec = promisify(execFile);
-
 export const EMPTY_SOURCE = "empty-worktree-baseline";
 
 /** Never follow a working-copy link outside the selected checkout. */
@@ -49,7 +45,7 @@ export async function workingFiles(vcs: LocalVcs): Promise<string[]> {
       "Working source requires a Git-backed jj repository.",
     );
 
-  const { stdout } = await exec(
+  const { stdout } = await runVcs(
     "git",
     [
       "-C",
@@ -100,7 +96,7 @@ export async function untrackedFileCount(
   if (vcs.kind !== "git") return undefined;
 
   try {
-    const { stdout } = await exec(
+    const { stdout } = await runVcs(
       "git",
       [
         "-C",

package/src/review-api/worktree-structural.integration.ts

@@ -318,3 +318,75 @@ it("compares a staged file in an unborn repository with empty source", async ()
     rmSync(root, { recursive: true, force: true });
   }
 }, 30_000);
+
+it("shows a peek on an untracked file as head lines", async () => {
+  const root = mkdtempSync(path.join(tmpdir(), "worktree-untracked-"));
+  const repository = path.join(root, "repository");
+  mkdirSync(repository);
+
+  const git = (...args: string[]) =>
+    execFileSync("git", ["-C", repository, ...args], {
+      encoding: "utf8",
+    }).trim();
+
+  git("init", "-q", "-b", "main");
+  writeFileSync(path.join(repository, "value.ts"), "export const value = 1;\n");
+  git("add", "value.ts");
+  git(
+    "-c",
+    "user.name=Test",
+    "-c",
+    "user.email=test@example.invalid",
+    "commit",
+    "-qm",
+    "Base",
+  );
+  writeFileSync(path.join(repository, "new.ts"), "export const a = 1;\n");
+
+  const local = openLocalReviewStore(path.join(root, "reviews.db"));
+
+  const selection = {
+    file: "new.ts",
+    start: { side: "head" as const, line: 1 },
+    end: { side: "head" as const, line: 1 },
+  };
+
+  try {
+    const { id } = await local.data.register(repository);
+
+    const { reviewId } = await local.store.execute({
+      operation: {
+        type: "create",
+        title: "Live",
+        target: { kind: "worktree", repositoryId: id },
+      },
+    });
+
+    await local.store.execute({
+      operation: {
+        type: "edit",
+        reviewId,
+        edit: {
+          type: "insert",
+          content: { type: "code_peek", source: formatAnchor(selection) },
+        },
+      },
+    });
+
+    const progress = await (
+      await createReviewApi(local.store, local.data).request(
+        `/${reviewId}/progress`,
+      )
+    ).json();
+
+    expect(progress.untrackedFiles).toBe(1);
+    expect(progress.unavailableSelections).toEqual({});
+    expect(progress.resolvedSelections[selectionKey(selection)]).toEqual([
+      { file: "new.ts", side: "head", fromLine: 1, toLine: 1 },
+    ]);
+  } finally {
+    await local.data.close();
+    await local.store.close();
+    rmSync(root, { recursive: true, force: true });
+  }
+});

package/src/review-stack.ts

@@ -1,15 +1,10 @@
-import { execFile } from "node:child_process";
-import { promisify } from "node:util";
-
-import { defaultBranch, git } from "@dev.fast/local-vcs";
+import { defaultBranch, git, runVcs } from "@dev.fast/local-vcs";
 import type {
   ReviewApiSummary,
   ReviewStackLayer,
 } from "@dev.fast/review-protocol";
 import { z } from "zod";
 
-const execFileAsync = promisify(execFile);
-
 const GitHubStacksSchema = z.array(
   z.object({
     pull_requests: z.array(
@@ -27,7 +22,7 @@ type Subject = Pick<ReviewApiSummary, "origin"> & { reviewId?: string };
 
 type Candidate = Pick<
   ReviewApiSummary,
-  "reviewId" | "title" | "origin" | "pins" | "trashedAt" | "repositoryGroup"
+  "reviewId" | "title" | "origin" | "pins" | "trashedAt" | "project"
 >;
 
 interface StackEntry {
@@ -140,7 +135,7 @@ async function localStack(
   repositoryPath: string,
 ): Promise<StackEntry[]> {
   const group = reviews.find((review) => review.reviewId === subject.reviewId)
-    ?.repositoryGroup?.key;
+    ?.project?.id;
 
   const current = subject.origin?.branch;
 
@@ -152,9 +147,7 @@ async function localStack(
 
   const name = (ref: string) => ref.replace(/^origin\//, "");
 
-  const inGroup = reviews.filter(
-    (review) => review.repositoryGroup?.key === group,
-  );
+  const inGroup = reviews.filter((review) => review.project?.id === group);
 
   const latest = (field: "branch" | "baseBranch", branch: string) => {
     const matches = inGroup.filter(
@@ -208,9 +201,13 @@ async function localStack(
 async function forkPoint(repositoryPath: string, main: string, branch: string) {
   const ref = `refs/heads/${branch}`;
 
-  const fork = await git(repositoryPath, ["merge-base", main, ref], {
-    allowFailure: true,
-  });
+  const fork = await git(
+    repositoryPath,
+    ["merge-base", { rev: main }, { rev: ref }],
+    {
+      allowFailure: true,
+    },
+  );
 
   if (!fork.ok) return null;
 
@@ -218,7 +215,7 @@ async function forkPoint(repositoryPath: string, main: string, branch: string) {
 
   const log = await git(
     repositoryPath,
-    ["log", "--reverse", "--format=%s", `${base}..${ref}`],
+    ["log", "--reverse", "--format=%s", { rev: `${base}..${ref}` }],
     { allowFailure: true },
   );
 
@@ -229,15 +226,10 @@ async function defaultRunGitHubApi(
   host: string,
   endpoint: string,
 ): Promise<string> {
-  const { stdout } = await execFileAsync(
+  const { stdout } = await runVcs(
     "gh",
-    ["api", "--hostname", host, endpoint],
-    {
-      env: { ...process.env, GH_PROMPT_DISABLED: "1" },
-      encoding: "utf8",
-      timeout: 10_000,
-      maxBuffer: 2 * 1024 * 1024,
-    },
+    ["api", "--hostname", { value: host }, { value: endpoint }],
+    { timeout: 10_000, maxBuffer: 2 * 1024 * 1024 },
   );
 
   return stdout;

package/src/runtime.ts

@@ -1,6 +1,7 @@
-import { spawn } from "node:child_process";
 import path from "node:path";
 
+import { runVcs } from "@dev.fast/local-vcs";
+
 export async function resolveReviewRoot(
   cwd: string,
   execFile: typeof execFilePromise = execFilePromise,
@@ -39,36 +40,9 @@ export async function resolveReviewRoot(
 }
 
 function execFilePromise(
-  file: string,
+  file: "git" | "jj",
   args: string[],
   options: { cwd?: string } = {},
 ): Promise<{ stdout: string }> {
-  return execFileUntraced(file, args, options);
-}
-
-function execFileUntraced(
-  file: string,
-  args: string[],
-  options: { cwd?: string } = {},
-): Promise<{ stdout: string }> {
-  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}`));
-    });
-  });
+  return runVcs(file, args, options);
 }

package/src/server/desktop-host.ts

@@ -84,6 +84,8 @@ export async function runDesktopHost(
   const { local, shared, server } = await takeOverServerHome(home, async () => {
     const local = await openReviewProfile(home, {
       manageWorkspaces: true,
+      // Desktop shows each PR's state beside its board.
+      pullRequestStates: true,
     });
 
     // JSON is the sole user-review store. A failure is surfaced, never replaced
@@ -197,9 +199,12 @@ function requiredPositiveInteger(
 
 if (process.env.DEV_FAST_REVIEW_DESKTOP_HOST_AUTOSTART !== "0") {
   void runDesktopHost().catch((error) => {
+    // Exit now: a host that couldn't start (a store that couldn't be
+    // upgraded) must not linger with timers alive, or Desktop waits out its
+    // ready timeout instead of reporting why.
     process.stderr.write(
       `${error instanceof Error ? (error.stack ?? error.message) : String(error)}\n`,
+      () => process.exit(1),
     );
-    process.exitCode = 1;
   });
 }

package/src/server/desktop-server.ts

@@ -47,10 +47,6 @@ import {
   reviewInstanceDiscoveryPath,
   reviewLegacyDiscoveryPath,
 } from "@review/review-home-paths";
-import {
-  readScratchpadEnabled,
-  writeScratchpadEnabled,
-} from "@review/review-preferences";
 import {
   ReviewTelemetry,
   type ReviewTelemetryContext,
@@ -176,10 +172,6 @@ export function createGlobalReviewServer(
   let closing = false;
   const cliPath = path.join(input.packageRoot, "dist", "cli.js");
 
-  // The scratchpad preference, read once at listen and kept current by the
-  // settings endpoint below: this server is the only writer while it runs.
-  let scratchpadEnabled = false;
-
   const discovery: ReviewDesktopDiscovery = {
     version: REVIEW_DESKTOP_DISCOVERY_VERSION,
     instanceId,
@@ -212,7 +204,6 @@ export function createGlobalReviewServer(
     relay,
     token,
     instanceId,
-    scratchpad: () => scratchpadEnabled,
     status: () => {
       const { key, channel, checkout, appVersion, cliVersion, instanceId } =
         discovery;
@@ -265,25 +256,6 @@ export function createGlobalReviewServer(
   app.all("/reviews-api", (context) => gateway.fetch(context.req.raw));
   app.all("/reviews-api/*", (context) => gateway.fetch(context.req.raw));
   app.get("/remote-hosts", () => serverJson(200, gateway.hosts()));
-  app.get("/preferences/scratchpad", () =>
-    serverJson(200, { enabled: scratchpadEnabled }),
-  );
-  app.put("/preferences/scratchpad", async (context) => {
-    const request = z
-      .object({ enabled: z.boolean() })
-      .safeParse(await readBoundedRequestJson(context.req.raw));
-
-    if (!request.success)
-      throw new ReviewServerError("enabled must be a boolean.", 400);
-
-    scratchpadEnabled = await writeScratchpadEnabled(request.data.enabled);
-
-    // Home watches the catalog; the pad appears or goes without a store write.
-    if (scratchpadEnabled) await reviewStore.ensureScratchpad();
-    reviewStore.invalidateCatalog();
-
-    return serverJson(200, { enabled: scratchpadEnabled });
-  });
   app.post("/app/focus", async () => {
     const result = await relay.dispatch({
       name: "focusWindow",
@@ -525,7 +497,6 @@ export function createGlobalReviewServer(
         store: reviewStore,
         data: input.reviewData,
       });
-      scratchpadEnabled = await readScratchpadEnabled();
       boundPort = await listen(httpServer, input.port);
       discovery.url = urlForBoundPort();
 
@@ -583,8 +554,7 @@ export function sessionStartedSource(
   try {
     const snapshot = reviewStore.read(reviewUuid);
 
-    const sourceKind =
-      snapshot.kind === "scratchpad" ? snapshot.kind : snapshot.target?.kind;
+    const sourceKind = snapshot.target?.kind;
 
     const pullRequestUrl = snapshot.origin?.pullRequestUrl;
 

package/src/server/headless-host.ts

@@ -212,8 +212,6 @@ async function serve(input: HeadlessServerInput) {
     token: discovery.token,
     instanceId: discovery.instanceId,
     softwareMapEnabled: input.softwareMapEnabled,
-    // The scratchpad is the laptop's alone, even with a Desktop attached.
-    scratchpad: () => false,
     status: () => ({ key: "headless", home: input.stateDir }),
     ask: {
       tools: cliAskTools(() => (existsSync(cliPath) ? cliPath : undefined), {

package/src/server/hono-http.ts

@@ -7,6 +7,7 @@ import {
   parseJsonText,
 } from "@dev.fast/review-protocol";
 import { type HttpBindings, getRequestListener } from "@hono/node-server";
+import { REVIEW_PERSON_HEADER } from "@review/review-api/request-origin";
 import { REVIEW_APP_SESSION_ID_HEADER } from "@review/ui-telemetry-events";
 import type { Hono } from "hono";
 import type { ContentfulStatusCode } from "hono/utils/http-status";
@@ -73,7 +74,7 @@ export function applyCorsHeaders(
 
   response.headers.set(
     "access-control-allow-headers",
-    `content-type, x-review-token, ${REVIEW_APP_SESSION_ID_HEADER}`,
+    `content-type, x-review-token, ${REVIEW_PERSON_HEADER}, ${REVIEW_APP_SESSION_ID_HEADER}`,
   );
   response.headers.set(
     "access-control-allow-methods",

package/src/server/legacy-tutorial-cleanup.ts

@@ -13,14 +13,13 @@ import { devReviewHome } from "@review/review-home-paths";
  */
 export async function removeLegacyTutorial(input: {
   store: ReviewStore;
-  data: LocalReviewData;
+  data: Pick<LocalReviewData, "forgetRepository">;
   root?: string;
 }): Promise<void> {
   const tutorialRoot = input.root ?? path.join(devReviewHome(), "tutorial");
 
   try {
-    for (const reviewId of input.store.legacyTutorialIds())
-      await input.store.execute({ operation: { type: "delete", reviewId } });
+    await input.store.removeLegacyTutorials();
 
     for (const repository of input.store.repositories()) {
       const relative = path.relative(tutorialRoot, repository.path);

package/src/server/review-gateway-list.ts

@@ -32,11 +32,9 @@ function decorate(
 
   return {
     ...entry,
-    ...(entry.repositoryGroup && {
-      repositoryGroup: {
-        key: `${alias}:${entry.repositoryGroup.key}`,
-        label: entry.repositoryGroup.label,
-      },
+    // A project on another machine is that machine's.
+    ...(entry.project && {
+      project: { id: `${alias}:${entry.project.id}`, name: entry.project.name },
     }),
     host: alias,
     hostState,

package/src/server/review-gateway-memory.ts

@@ -16,7 +16,7 @@ const listEntrySchema = z.looseObject({
   repositoryName: z.string(),
   viewedAt: z.string().nullable(),
   trashedAt: z.string().nullable(),
-  repositoryGroup: z.object({ key: z.string(), label: z.string() }).optional(),
+  project: z.object({ id: z.string(), name: z.string() }).optional(),
 });
 
 export const listEntriesSchema = z.array(z.unknown()).transform(

package/src/server/review-lifecycle-telemetry.ts

@@ -34,12 +34,12 @@ export function reviewLifecycleTelemetry(
     );
 
   return {
-    onReviewCreated: ({ reviewId, kind, blocks, via, agentKind }) => {
+    onReviewCreated: ({ reviewId, blocks, via, agentKind }) => {
       if (reported.has(reviewId)) return;
       reported.add(reviewId);
 
       if (via === "api" || via === "mcp") authoring.set(reviewId, agentKind);
-      const properties: PostHogCaptureProperties = { kind, blocks, via };
+      const properties: PostHogCaptureProperties = { blocks, via };
 
       if (agentKind) properties.agent_kind = agentKind;
       capture("review_review_created", properties, reviewId);

package/src/server/review-server-core.ts

@@ -132,7 +132,6 @@ export interface WhiteboardCoreInput {
   token: string;
   instanceId: string;
   softwareMapEnabled?: boolean;
-  scratchpad: () => boolean;
   status: () => JsonObject;
   hooks?: ReviewApiHooks;
   ask?: {
@@ -192,7 +191,6 @@ export function createWhiteboardCore(input: WhiteboardCoreInput) {
     callbacks.open,
     shared,
     callbacks.capabilities,
-    input.scratchpad,
     () => traceMachineEnabled(),
     input.status,
     input.hooks,
@@ -236,9 +234,7 @@ export function relayReviewCallbacks(
 
       return z.object({ softwareMapEnabled: z.boolean() }).parse(result.result);
     },
-    async capabilities(): Promise<
-      Omit<AuthoringCapabilities, "scratchpadEnabled">
-    > {
+    async capabilities(): Promise<AuthoringCapabilities> {
       if (!relay.attached)
         return { desktopAvailable: false, softwareMapEnabled };