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

package/dist/{thread-state-Do5LX10I.js → thread-state-5pYr6_zM.js}

@@ -1,4 +1,4 @@
-import { M as ReviewSelectedDiffSchema, T as ReviewApiSelectionSourceSchema } from "./src-BiWNbcF_.js";
+import { A as ReviewSelectedDiffSchema, C as ReviewApiSelectionSourceSchema } from "./src-DsM3jO8G.js";
 import { z } from "zod";
 /** A semantic selection, independent of comment/thread creation. */
 const AgentSelectionSchema = z.strictObject({
@@ -374,12 +374,7 @@ const askHistoryEntrySchema = z.object({
 	/** The commit the agent read. */
 	head: z.string(),
 	createdAt: z.string(),
-	updatedAt: z.string(),
-	/** The board the conversation started on, and the block if it was about one. */
-	anchor: z.object({
-		board: z.string(),
-		block: z.string().optional()
-	}).optional()
+	updatedAt: z.string()
 });
 //#endregion
 export { askHistoryEntrySchema as a, askPicksSchema as c, selectionMarkdown as d, askEntrySchema as i, askQuestionSchema as l, askAgentIds as n, askOfferSchema as o, askChoiceKinds as r, askPermissionsSchema as s, applyAskChange as t, AgentSelectionSchema as u };

package/dist/{tool-failure-DvI-gys2.js → tool-failure-C9MpbXJJ.js}

@@ -1,6 +1,5 @@
 import { m as isStringValue, n as isJsonObject } from "./src-X9phtB2j.js";
-import { l as boardTitle } from "./src-BiWNbcF_.js";
-import { n as callAuthoringTool, s as ReviewApiError, t as ToolText } from "./agent-client-Dtc-Kn_w.js";
+import { n as callAuthoringTool, s as ReviewApiError, t as ToolText } from "./agent-client-BOOi7EUR.js";
 import { t as ReviewInputError } from "./input-error-OLgB_h31.js";
 //#region src/review-api/public-tools.ts
 function publicDescription(text) {
@@ -39,7 +38,7 @@ function publicResult(value) {
 	if (Array.isArray(value)) return value.map(publicResult);
 	if (!isJsonObject(value)) return value;
 	return Object.fromEntries(Object.entries(value).map(([key, item]) => {
-		return [key === "reviewId" ? "sessionId" : key === "review" ? "session" : key === "reviews" ? "sessions" : key, key === "review" || key === "reviews" ? publicResult(item) : key === "title" && "reviewId" in value && isStringValue(item) ? boardTitle(item) : item];
+		return [key === "reviewId" ? "sessionId" : key === "review" ? "session" : key === "reviews" ? "sessions" : key, key === "review" || key === "reviews" ? publicResult(item) : item];
 	}));
 }
 async function callPublicTool(client, tool, input, signal) {
@@ -48,13 +47,7 @@ async function callPublicTool(client, tool, input, signal) {
 	const fields = { ...rest };
 	if (sessionId !== void 0) fields.reviewId = sessionId;
 	const result = await callAuthoringTool(client, tool, fields, signal);
-	return result instanceof ToolText ? result : withoutRepositoryIds(publicResult(result));
-}
-/** Agents never see repository ids. */
-function withoutRepositoryIds(value) {
-	if (Array.isArray(value)) return value.map(withoutRepositoryIds);
-	if (!isJsonObject(value)) return value;
-	return Object.fromEntries(Object.entries(value).filter(([key]) => key !== "repositoryId").map(([key, item]) => [key, withoutRepositoryIds(item)]));
+	return result instanceof ToolText ? result : publicResult(result);
 }
 //#endregion
 //#region src/review-api/recovery.ts

package/instructions/authoring.md

@@ -3,7 +3,7 @@ you are writing an interactive rfc-style whiteboard, for consumption by a staff
 **flow**
 follow these first five steps in order, with no tool calls beyond what they need: loading the tools they use (`session_create`, `session_activity_begin`, `session_edit`, `session_get`, and later `session_activity_update` and `session_activity_end`) and, for a PR, reading its description (`gh pr view`).
 
-- create the whiteboard with a `target` naming the checkout by its `repositoryPath`: `{kind:"commits", head, base?}` for commits, `{kind:"pullRequest", pullRequest}` for a PR by URL or number, or `{kind:"project"}` for a plan with no change yet. if they name none, use a `worktree` target with no `base`: it reviews the checkout, uncommitted work included, against the default branch. for uncommitted work only, pass `base: "HEAD"`. untracked files are left out: run `git add -N <file>` on new files you want reviewed
+- create the whiteboard, pinned to the commits/pr the user describes, naming the checkout by its `repositoryPath`. if they name none, use a `worktree` target with no `base`: it reviews the checkout, uncommitted work included, against the default branch. for uncommitted work only, pass `base: "HEAD"`. untracked files are left out: run `git add -N <file>` on new files you want reviewed
 - trigger a subagent with this exact instruction: "Call `session_get_instructions({topic:\"file-lenses\"})` and follow it for whiteboard <sessionId>, whose change is `<base>..<head>` in `<repositoryPath>`."
 - call `session_activity_begin` and pass its `activityId` on every `session_edit`
 - read the diff with git (or jj) in `repositoryPath`, at the commits `session_create` returned: `git diff <base> <head>`, or for a worktree target `git diff <base>` plus `git status` for untracked files. if it shows no changes, the target is wrong: fix it with `session_set_target` before writing
@@ -37,5 +37,4 @@ follow these first five steps in order, with no tool calls beyond what they need
 - use `session_activity_update` to provide regular status updates on your area of focus.
 - keep whiteboards short and sweet when possible (esp. for small changes.) feel free to omit sections.
 - when something (a phrase in the prose, diagram node, etc.) describes actual code in the codebase, always default to attaching/hyperlink code.
-- Link repository code as `[label](whiteboard:source/head/src/file.ts#L10-L24)`; use `base` for old code. To quote any other commit of the board's repository, as a plan does, put the commit in place of `head`: `[label](whiteboard:source/<commit>/src/file.ts#L10-L24)`. Use repository-relative paths and verified line numbers.
-- Link another whiteboard inline as `[label](whiteboard:board/<sessionId>)`, or one of its blocks as `[label](whiteboard:board/<sessionId>#<blockId>)`, wherever the prose refers to it. The board and block must exist.
+- Link repository code as `[label](review-source:head/src/file.ts#L10-L24)`; use `base` for old code. Use repository-relative paths and verified line numbers.

package/instructions/scratchpad.md

@@ -0,0 +1,23 @@
+# Scratchpad
+
+The scratchpad is one document in Whiteboard that is not a session for any change. It has no target, base, head or lifecycle. You draw on it the way you would sketch an explanation on a whiteboard: a short paragraph, a sequence or flow diagram, a call tree, a code peek, and the smallest view that makes the point. The user reads it in the app and copies blocks back to you.
+
+Its id is fixed: `scratchpad`. While it is turned on in Whiteboard Settings it always exists; never create, rename, trash, delete or share it.
+
+## When to draw here
+
+- The user asks to be shown how something works, wants a picture of a flow, a call path, a data shape or a layout, or is thinking out loud with you before there is a concrete plan.
+- The explanation spans more than one file or more than one repository, or would be an ASCII diagram in chat.
+
+Answer in chat when one sentence or one code line does it. Do not use the scratchpad to review a change; author a session instead (`session_get_instructions({})`).
+
+## How
+
+1. `session_capabilities({})`. Draw only when `desktopAvailable` and `scratchpadEnabled` are true. Otherwise answer in chat; if the pad is off, say once that it can be turned on in Whiteboard Settings.
+2. Every source reference needs pins. For each repository you will quote, `session_register_repository({path})` for its `repositoryId`, and `git rev-parse HEAD` (or the commit the user is looking at) for a commit id. Put `pins: {repositoryId, head}` on each block that quotes code (`code_peek`, `sequence`, `flow_diagram`, `call_stack_diff`, `database_lens` and `markdown` with `review-source:` links); a step, frame, attachment or operation that quotes other commits than its block takes its own `pins`. Add `base` only when an anchor reads the base side. A reference without pins is rejected, since the scratchpad has none to lend.
+3. Read what you cite at those commits, for example `git show <head>:<path>` in the repository, so line numbers match the pins.
+4. Call `session_activity_begin` and pass the `activityId` it returns on each `session_edit` as you append blocks. Omitted placement puts the block at the top of the pad, and the response's `targetId` names it. A thought that spans several blocks reads top-down only if you insert it bottom-up (last block first) or give each following block `afterId` of the block you just inserted. Draw a diagram whole in one insert; fix one you drew earlier by patching its nodes, edges or steps by ID, and `replace` a block only when the whole picture was wrong. Each block's fields are in the Components section below; read `session_get_instructions({topic:"authoring"})` for choosing components. End the activity when finished.
+5. After your first insert in a session, `session_open({sessionId: "scratchpad"})` once so the pad is showing. Do not call it again for later edits; they appear live.
+6. In chat, say in one line what you drew. Do not repeat the diagram there.
+
+Keep it a log, not a document: the newest thought sits at the top, above older ones. Do not build sections, outlines or status markers unless the user asks for structure. When the user asks to clear the pad, `replace` or `remove` its blocks; history keeps what was there.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dev.fast/whiteboard",
-  "version": "0.3.4-preview.20261019.127",
+  "version": "0.3.4",
   "private": false,
   "license": "MIT",
   "repository": {
@@ -74,11 +74,11 @@
     "tsx": "4.21.0",
     "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/json": "0.0.0"
+    "@dev.fast/review-protocol": "0.0.0"
   },
   "engines": {
     "node": ">=24 <25"
@@ -88,7 +88,7 @@
     "repo": "devdotfast/review",
     "docs": "packages/review/instructions/authoring.md"
   },
-  "gitHead": "5dd55db92af13eac9187d16cf07b4bd80c456724",
+  "gitHead": "f33bd6620742fe65ccc1d5fef416fbe8254dcf79",
   "scripts": {
     "prebuild": "pnpm run build:workspace-deps",
     "build": "node --max-old-space-size=8192 node_modules/tsdown/dist/run.mjs",

package/src/ask/agents.ts

@@ -19,7 +19,6 @@ import {
   type AskPermissions,
   askAgentIds,
 } from "@review/ask/thread-state.js";
-import { ASK_THREAD_ENV } from "@review/review-api/request-origin.js";
 
 /** How Whiteboard runs an agent that speaks ACP on stdio. */
 type AskAgentLaunch =
@@ -248,8 +247,6 @@ export type AskAgentLauncher = (
     /** The MCP servers its sessions are given. */
     mcpServers?: McpServer[];
     env?: NodeJS.ProcessEnv;
-    /** The Ask thread this agent serves. */
-    threadId?: string;
   },
 ) => Promise<AskAgentProcess>;
 
@@ -360,12 +357,7 @@ const STDERR_LIMIT = 8_000;
 export const launchAskAgent: AskAgentLauncher = async (
   agent,
   cwd,
-  {
-    permissions = "read-only",
-    mcpServers = [],
-    env: base = process.env,
-    threadId,
-  } = {},
+  { permissions = "read-only", mcpServers = [], env: base = process.env } = {},
 ) => {
   const spec = askAgents[agent];
   const executable = await findAgent(spec, base);
@@ -377,10 +369,6 @@ export const launchAskAgent: AskAgentLauncher = async (
     ...askAgentSettings(agent, permissions).env,
   };
 
-  // Lets `whiteboard api` calls from the agent's shell name the thread.
-  // The MCP server gets it in its own env, since MCP clients may not pass theirs on.
-  if (threadId) env[ASK_THREAD_ENV] = threadId;
-
   // A server started from inside a Claude Code session must not look nested.
   delete env.CLAUDECODE;
   delete env.CLAUDE_CODE_ENTRYPOINT;

package/src/ask/checkout-files.ts

@@ -1,10 +1,13 @@
-import { runVcs } from "@dev.fast/local-vcs";
+import { execFile } from "node:child_process";
+import { promisify } from "node:util";
+
+const execFileAsync = promisify(execFile);
 
 /** The checkout's files relative to its root: tracked ones, and new ones git
  * does not ignore. None where git cannot list them. */
 export async function checkoutFiles(root: string): Promise<string[]> {
   try {
-    const { stdout } = await runVcs(
+    const { stdout } = await execFileAsync(
       "git",
       [
         "-C",

package/src/ask/thread-state.ts

@@ -375,10 +375,6 @@ export const askHistoryEntrySchema = z.object({
   head: z.string(),
   createdAt: z.string(),
   updatedAt: z.string(),
-  /** The board the conversation started on, and the block if it was about one. */
-  anchor: z
-    .object({ board: z.string(), block: z.string().optional() })
-    .optional(),
 });
 
 export type AskHistoryEntry = z.infer<typeof askHistoryEntrySchema>;

package/src/ask/thread.ts

@@ -176,7 +176,6 @@ export class AskThread {
   /** The number of changes so far; a snapshot carries the one it includes. */
   private seq = 0;
   private readonly listeners = new Set<(update: AskUpdate) => void>();
-  private readonly statusListeners = new Set<() => void>();
   private readonly closers = new Set<() => void>();
   private readonly decisions = new Map<
     string,
@@ -289,10 +288,6 @@ export class AskThread {
     return { seq: this.seq, snapshot: this.state };
   }
 
-  get status() {
-    return this.state.status;
-  }
-
   /** Every change after the current `seq`, in order. */
   subscribe(listener: (update: AskUpdate) => void) {
     this.listeners.add(listener);
@@ -300,13 +295,6 @@ export class AskThread {
     return () => this.listeners.delete(listener);
   }
 
-  /** Calls back when the status changes, without following the thread. */
-  onStatusChange(listener: () => void) {
-    this.statusListeners.add(listener);
-
-    return () => this.statusListeners.delete(listener);
-  }
-
   /** Whether anything follows the thread, as an open panel does. */
   watched() {
     return this.listeners.size > 0;
@@ -642,7 +630,6 @@ export class AskThread {
     const launched = await this.launch(this.start.agent, this.start.cwd, {
       permissions: this.state.permissions,
       mcpServers: this.mcpServers,
-      threadId: this.id,
     });
 
     // Closed, stopped or given up on while the process started.
@@ -1546,8 +1533,5 @@ export class AskThread {
     const update = { seq: this.seq, change };
 
     for (const listener of this.listeners) listener(update);
-
-    if (change.type === "set" && change.status)
-      for (const listener of this.statusListeners) listener();
   }
 }

package/src/ask/threads.ts

@@ -18,7 +18,6 @@ import {
   type AskThreadStart,
   askThreadLimits,
 } from "@review/ask/thread.js";
-import { ASK_THREAD_ENV } from "@review/review-api/request-origin.js";
 
 /** How long an agent may take to say what it offers. */
 const OFFER_TIMEOUT_MS = 30_000;
@@ -164,7 +163,7 @@ export function cliAskTools(
   ];
 
   return {
-    mcpServers: (threadId) => {
+    mcpServers: () => {
       const cli = cliPath();
 
       return cli
@@ -173,9 +172,7 @@ export function cliAskTools(
               name: "whiteboard",
               command: process.execPath,
               args: [cli, "mcp"],
-              // MCP clients may not pass their own env on, so the server
-              // gets the thread it serves in its own.
-              env: [...env(), { name: ASK_THREAD_ENV, value: threadId }],
+              env: env(),
             },
           ]
         : [];
@@ -221,8 +218,6 @@ export class AskThreads {
 
   private readonly mcpServers: AskMcpServers;
 
-  private readonly statusListeners = new Set<() => void>();
-
   constructor(
     private readonly launch: AskAgentLauncher,
     private readonly tools: AskTools = {},
@@ -254,8 +249,6 @@ export class AskThreads {
     );
 
     this.threads.set(thread.id, thread);
-    thread.onStatusChange(() => this.statusChanged());
-    this.statusChanged();
     void thread.open();
     this.sweeper ??= setInterval(() => this.sweep(), SWEEP_MS);
     this.sweeper.unref?.();
@@ -298,19 +291,6 @@ export class AskThreads {
     }
   }
 
-  /** Calls back when a thread opens, closes or changes its status. */
-  subscribeStatus(listener: () => void) {
-    this.statusListeners.add(listener);
-
-    return () => {
-      this.statusListeners.delete(listener);
-    };
-  }
-
-  private statusChanged() {
-    for (const listener of this.statusListeners) listener();
-  }
-
   get(id: string) {
     return this.threads.get(id);
   }
@@ -333,10 +313,9 @@ export class AskThreads {
 
   close(id: string, save = true) {
     const thread = this.threads.get(id);
+    this.threads.delete(id);
     this.unwatchedSince.delete(id);
     thread?.close(save);
-
-    if (this.threads.delete(id)) this.statusChanged();
   }
 
   closeDeletedReviews(isDeleted: (reviewId: string) => boolean) {

package/src/cli-install.ts

@@ -699,16 +699,11 @@ export async function writePathShim(
   cliPath: string,
   runtimePath: string | undefined,
   devHome: string,
-  previewHome = devHome,
 ): Promise<void> {
   if (process.platform === "win32") {
     await writeFileAtomicAsync(
       shimPath,
-      windowsCliShim(
-        cliPath,
-        runtimePath ?? process.execPath,
-        devHome === previewHome ? devHome : undefined,
-      ),
+      windowsCliShim(cliPath, runtimePath ?? process.execPath, devHome),
       { replaceSymlink: true },
     );
 
@@ -721,8 +716,6 @@ export async function writePathShim(
 FALLBACK_CLI=${shSingleQuote(cliPath)}
 FALLBACK_RUNTIME=${shSingleQuote(runtimePath ?? "")}
 DEFAULT_HOME=${shSingleQuote(devHome)}
-channel="\${DEV_FAST_REVIEW_CHANNEL:-\${DEV_REVIEW_INSTANCE:-}}"
-[ "$channel" = preview ] && DEFAULT_HOME=${shSingleQuote(previewHome)}
 export DEV_REVIEW_HOME="\${DEV_REVIEW_HOME:-$DEFAULT_HOME}"
 DESKTOP="$DEV_REVIEW_HOME/review-desktop"
 
@@ -843,11 +836,7 @@ export async function installReviewCommand(input: {
     shimPath,
     input.cliPath,
     input.cliRuntimePath,
-    devReviewHome({ DEV_REVIEW_HOME: env.DEV_REVIEW_HOME }, homeDir),
-    devReviewHome(
-      { DEV_REVIEW_HOME: env.DEV_REVIEW_HOME, DEV_REVIEW_INSTANCE: "preview" },
-      homeDir,
-    ),
+    devReviewHome(env, homeDir),
   );
 
   if (await traceMachineEnabled({ homeDir, env })) {

package/src/cli-runner.ts

@@ -5,7 +5,6 @@ import { createInterface } from "node:readline/promises";
 import { Writable } from "node:stream";
 import { fileURLToPath } from "node:url";
 
-import { boardTitle } from "@dev.fast/review-protocol";
 import {
   REVIEW_CLI_INSTALL_TARGET_LABELS,
   REVIEW_REMOTE_AGENT_IDS,
@@ -837,9 +836,7 @@ export async function runReviewCli(input: ReviewCliInput): Promise<number> {
             : "Whiteboard Desktop is ready in the background. Pass --focus to bring it forward.\n",
       );
     } else {
-      input.stdout.write(
-        `Whiteboard Desktop is showing "${boardTitle(event.title)}".\n`,
-      );
+      input.stdout.write(`Whiteboard Desktop is showing "${event.title}".\n`);
     }
   };
 

package/src/cli.ts

@@ -84,12 +84,9 @@ async function maybeDelegateToDesktopCli(
   // This bootstrap runs before the Node floor check, so it cannot import
   // devReviewHome() from @dev.fast/trace-core: that module graph needs a modern
   // Node. Keep this copy in step with devReviewHome().
-  const channel =
-    env.DEV_FAST_REVIEW_CHANNEL?.trim() || env.DEV_REVIEW_INSTANCE?.trim();
-
   const devHome = env.DEV_REVIEW_HOME?.trim()
     ? path.resolve(env.DEV_REVIEW_HOME.trim())
-    : path.join(os.homedir(), channel === "preview" ? ".dev-preview" : ".dev");
+    : path.join(os.homedir(), ".dev");
 
   let cliPath: string;
   let runtimePath: string | undefined;

package/src/diff-selection-migration.ts

@@ -1,5 +1,4 @@
 import {
-  type JsonObject,
   type JsonValue,
   isJsonObject,
   isNumberValue,
@@ -15,63 +14,50 @@ export function migrateDiffSelections(input: unknown): JsonValue {
   return migrateNode(jsonValueSchema.parse(input));
 }
 
-/** Keys whose values held retired attachments. */
-export const ATTACHMENT_KEYS = [
-  "source",
-  "peek",
-  "sources",
-  "contextSources",
-  "callSite",
-];
-
-/** A retired attachment as a selection, or undefined for anything else. */
-export function retiredSelection(value: JsonObject): JsonObject | undefined {
-  if (!isStringValue(value.file)) return undefined;
-
-  if (isNumberValue(value.fromLine) && isNumberValue(value.toLine)) {
-    const side = value.side ?? value.graph ?? "head";
-
-    return {
-      file: value.file,
-      start: { side, line: value.fromLine },
-      end: { side, line: value.toLine },
-    };
-  }
-
-  if (
-    isJsonObject(value.start) &&
-    isJsonObject(value.end) &&
-    "baseLine" in value.start
-  ) {
-    const endpoint = (row: JsonObject) =>
-      isNumberValue(row.headLine)
-        ? { side: "head", line: row.headLine }
-        : { side: "base", line: row.baseLine ?? null };
-
-    return {
-      file: value.file,
-      start: endpoint(value.start),
-      end: endpoint(value.end),
-    };
-  }
-
-  return undefined;
-}
-
 function migrateNode(value: JsonValue, attachment = false): JsonValue {
   if (Array.isArray(value))
     return value.map((child) => migrateNode(child, attachment));
 
   if (!isJsonObject(value)) return value;
 
-  const selection = attachment ? retiredSelection(value) : undefined;
-
-  if (selection) return selection;
+  if (attachment && isStringValue(value.file)) {
+    if (isNumberValue(value.fromLine) && isNumberValue(value.toLine)) {
+      const side = value.side ?? value.graph ?? "head";
+
+      return {
+        file: value.file,
+        start: { side, line: value.fromLine },
+        end: { side, line: value.toLine },
+      };
+    }
+
+    if (
+      isJsonObject(value.start) &&
+      isJsonObject(value.end) &&
+      "baseLine" in value.start
+    ) {
+      const endpoint = (row: typeof value.start) =>
+        isNumberValue(row.headLine)
+          ? { side: "head", line: row.headLine }
+          : { side: "base", line: row.baseLine };
+
+      return {
+        file: value.file,
+        start: endpoint(value.start),
+        end: endpoint(value.end),
+      };
+    }
+  }
 
   return Object.fromEntries(
     Object.entries(value).map(([key, child]) => [
       key,
-      migrateNode(child, ATTACHMENT_KEYS.includes(key)),
+      migrateNode(
+        child,
+        ["source", "peek", "sources", "contextSources", "callSite"].includes(
+          key,
+        ),
+      ),
     ]),
   );
 }

package/src/lens-selection.ts

@@ -24,23 +24,8 @@ const sameSideAnchor = /^(head|base)\/(.+)#L(\d+)(?:-L(\d+))?$/;
 
 const crossSideAnchor = /^diff\/(.+)#([LR])(\d+)-([LR])(\d+)$/;
 
-const commitLinkAnchor = /^([0-9a-f]{7,40})\/(.+#L\d+(?:-L\d+)?)$/i;
-
-/** A source link's commit form, "<sha>/path#L10-L24" (without its
- * `whiteboard:source/` prefix): lines of that commit of the board's own
- * repository, read as their head side. Undefined for any other form. */
-export function commitLinkTarget(
-  target: string,
-): { commit: string; anchor: Anchor } | undefined {
-  const match = commitLinkAnchor.exec(target);
-
-  return match
-    ? { commit: match[1]!.toLowerCase(), anchor: `head/${match[2]!}` }
-    : undefined;
-}
-
 /** An anchor's selection, as markdown source links write it too (without
- * their `whiteboard:source/` prefix). Undefined when it isn't one. */
+ * their `review-source:` scheme). Undefined when it isn't one. */
 export function parseAnchor(text: string): DiffSelection | undefined {
   const same = sameSideAnchor.exec(text);
 
@@ -210,26 +195,6 @@ export function sourceAnchor(source: LensSource): FileLineRange {
   return sourceAnchors(source)[0]!;
 }
 
-/** Why a selection cannot be resolved at its pins, as telemetry reports it. */
-export const SELECTION_UNAVAILABLE_REASONS = [
-  "missing_file",
-  "no_alignment",
-  "outside_lines",
-  "other",
-] as const;
-
-export type SelectionUnavailableReason =
-  (typeof SELECTION_UNAVAILABLE_REASONS)[number];
-
-export class SelectionUnavailableError extends Error {
-  constructor(
-    readonly reason: SelectionUnavailableReason,
-    message: string,
-  ) {
-    super(message);
-  }
-}
-
 /** Resolve once against the renderer's rows. No matching or context expansion. */
 export function resolveDiffSelection(
   source: LensSource,
@@ -245,8 +210,7 @@ export function resolveDiffSelection(
   const end = locate(source.end);
 
   if (start < 0 || end < start)
-    throw new SelectionUnavailableError(
-      "outside_lines",
+    throw new Error(
       "Lens endpoints do not identify an ordered interval in this diff.",
     );
   const selected = rows.slice(start, end + 1);