@oh-my-pi/pi-coding-agent 17.3.3 → 17.3.5

package/src/tts/speech-enhancer.ts

@@ -15,7 +15,7 @@
  *   mechanical {@link SpeakableStream} cleanup — speech never blocks on the
  *   model.
  */
-import { type AssistantMessage, completeSimple } from "@oh-my-pi/pi-ai";
+import { type AssistantMessage, completeSimple, retryTransientCompletion } from "@oh-my-pi/pi-ai";
 import { logger, prompt } from "@oh-my-pi/pi-utils";
 import type { ModelRegistry } from "../config/model-registry";
 import { getModelMatchPreferences, resolveModelRoleValue } from "../config/model-resolver";
@@ -82,20 +82,25 @@ export class SpeechEnhancer {
 			if (!apiKey) return null;
 			// Resolve metadata after getApiKey so the session-sticky credential is recorded first.
 			const metadata = this.#deps.metadataResolver?.(model.provider);
-			const timeout = AbortSignal.timeout(REWRITE_TIMEOUT_MS);
-			const response = await completeSimple(
-				model,
-				{
-					systemPrompt: [SYSTEM_PROMPT],
-					messages: [{ role: "user", content: boundBlock(block), timestamp: Date.now() }],
-				},
-				{
-					apiKey: registry.resolver(model, sessionId),
-					maxTokens: ANSWER_MAX_TOKENS,
-					disableReasoning: true,
-					metadata,
-					signal: signal ? AbortSignal.any([signal, timeout]) : timeout,
+			const response = await retryTransientCompletion(
+				() => {
+					const timeout = AbortSignal.timeout(REWRITE_TIMEOUT_MS);
+					return completeSimple(
+						model,
+						{
+							systemPrompt: [SYSTEM_PROMPT],
+							messages: [{ role: "user", content: boundBlock(block), timestamp: Date.now() }],
+						},
+						{
+							apiKey: registry.resolver(model, sessionId),
+							maxTokens: ANSWER_MAX_TOKENS,
+							disableReasoning: true,
+							metadata,
+							signal: signal ? AbortSignal.any([signal, timeout]) : timeout,
+						},
+					);
 				},
+				{ signal },
 			);
 			if (response.stopReason === "error") {
 				logger.debug("speech-enhancer: rewrite errored", { error: response.errorMessage });

package/src/utils/commit-message-generator.ts

@@ -4,7 +4,7 @@
  */
 import type { ThinkingLevel } from "@oh-my-pi/pi-agent-core";
 import type { Api, Model } from "@oh-my-pi/pi-ai";
-import { completeSimple } from "@oh-my-pi/pi-ai";
+import { completeSimple, retryTransientCompletion } from "@oh-my-pi/pi-ai";
 import { logger, prompt } from "@oh-my-pi/pi-utils";
 
 import type { ModelRegistry } from "../config/model-registry";
@@ -106,17 +106,19 @@ export async function generateCommitMessage(
 
 		try {
 			const maxTokens = COMMIT_MAX_TOKENS;
-			const response = await completeSimple(
-				candidate.model,
-				{
-					systemPrompt: [COMMIT_SYSTEM_PROMPT],
-					messages: [{ role: "user", content: userMessage, timestamp: Date.now() }],
-				},
-				{
-					apiKey: registry.resolver(candidate.model, sessionId),
-					maxTokens,
-					reasoning: toReasoningEffort(candidate.thinkingLevel),
-				},
+			const response = await retryTransientCompletion(() =>
+				completeSimple(
+					candidate.model,
+					{
+						systemPrompt: [COMMIT_SYSTEM_PROMPT],
+						messages: [{ role: "user", content: userMessage, timestamp: Date.now() }],
+					},
+					{
+						apiKey: registry.resolver(candidate.model, sessionId),
+						maxTokens,
+						reasoning: toReasoningEffort(candidate.thinkingLevel),
+					},
+				),
 			);
 
 			if (response.stopReason === "error") {

package/src/utils/external-editor.ts

@@ -33,6 +33,27 @@ export interface OpenInEditorOptions {
 	trimTrailingNewline?: boolean;
 }
 
+/** Subprocess argv and Windows quoting mode used to launch an external editor. */
+export interface EditorSpawnCommand {
+	cmd: string[];
+	windowsVerbatimArguments: boolean;
+}
+
+/** Resolves shell argv without letting the host runtime re-quote the editor command. */
+export function resolveEditorSpawnCommand(
+	editorCmd: string,
+	tmpFile: string,
+	platform: NodeJS.Platform = process.platform,
+): EditorSpawnCommand {
+	const windows = platform === "win32";
+	// cmd.exe strips the outer /s /c quote pair; Bun must pass the embedded
+	// editor/path quotes verbatim instead of applying argv escaping to them.
+	const cmd = windows
+		? ["cmd.exe", "/d", "/s", "/c", `"${editorCmd} "${tmpFile}""`]
+		: [$which("sh") ?? "sh", "-c", `${editorCmd} "$1"`, "sh", tmpFile];
+	return { cmd, windowsVerbatimArguments: windows };
+}
+
 /**
  * Opens `content` in the user's external editor and returns the edited text.
  * Returns `null` if the editor exits with a non-zero code.
@@ -50,15 +71,13 @@ export async function openInEditor(
 	try {
 		await Bun.write(tmpFile, content);
 
+		const spawnCommand = resolveEditorSpawnCommand(editorCmd, tmpFile);
 		const [stdin, stdout, stderr] = options?.stdio ?? ["inherit", "inherit", "inherit"];
-		const cmd =
-			process.platform === "win32"
-				? ["cmd", "/c", `${editorCmd} "${tmpFile}"`]
-				: [$which("sh") ?? "sh", "-c", `${editorCmd} "$1"`, "sh", tmpFile];
-		const child = Bun.spawn(cmd, {
+		const child = Bun.spawn(spawnCommand.cmd, {
 			stdin,
 			stdout,
 			stderr,
+			windowsVerbatimArguments: spawnCommand.windowsVerbatimArguments,
 		});
 		const exitCode = await child.exited;
 		if (exitCode === 0) {

package/src/utils/markit.ts

@@ -1,5 +1,5 @@
 import * as path from "node:path";
-import { logger, untilAborted } from "@oh-my-pi/pi-utils";
+import { untilAborted } from "@oh-my-pi/pi-utils";
 import type { ConversionResult, Markit, StreamInfo } from "../markit";
 import { ToolAbortError } from "../tools/tool-errors";
 import {
@@ -8,7 +8,6 @@ import {
 	readMarkitConversionCache,
 	writeMarkitConversionCache,
 } from "./markit-cache";
-import { loadEmbeddedMupdfWasm } from "./mupdf-wasm-embed";
 
 /**
  * File extensions markit can actually convert to markdown — one per registered
@@ -31,53 +30,16 @@ export interface MarkitConversionResult {
 
 export interface MarkitFileConversionOptions {
 	/**
-	 * Directory the PDF converter writes extracted images/diagrams into. When
-	 * set, each embedded image is rendered to `<id>.png` and referenced by path
-	 * in the markdown; when unset, markit emits an `<!-- image: <id> ... -->`
-	 * placeholder comment instead.
+	 * Directory converters may use for extracted image or diagram files. Since
+	 * those files are conversion side effects, conversions using this option
+	 * bypass the markdown cache.
 	 */
 	imageDir?: string;
 }
 
-interface MuPdfWasmModuleConfig {
-	print?: (...values: unknown[]) => void;
-	printErr?: (...values: unknown[]) => void;
-	wasmBinary?: Uint8Array;
-}
-
-function logMuPdfWasmOutput(stream: "stdout" | "stderr", values: unknown[]): void {
-	const message = values.length === 1 && typeof values[0] === "string" ? values[0] : values.map(String).join(" ");
-	logger.debug("mupdf wasm output", { stream, message });
-}
-
-// `$libmupdf_wasm_Module` is declared globally (as `any`) by the mupdf package.
-// Install print hooks before the WASM module initializes so its stdout/stderr
-// route to the file logger instead of corrupting the TUI.
-function installMuPdfWasmLogger(): void {
-	const moduleConfig: MuPdfWasmModuleConfig = globalThis.$libmupdf_wasm_Module ?? {};
-	moduleConfig.print = (...values: unknown[]) => logMuPdfWasmOutput("stdout", values);
-	moduleConfig.printErr = (...values: unknown[]) => logMuPdfWasmOutput("stderr", values);
-	globalThis.$libmupdf_wasm_Module = moduleConfig;
-}
-
-// Hand the WASM module its bytes directly when the compiled binary embedded them
-// (scripts/embed-mupdf-wasm.ts); a single-file binary has no node_modules for
-// mupdf to read `mupdf-wasm.wasm` from. Source/npm builds get undefined here and
-// mupdf loads its own wasm. Must run before the mupdf module evaluates.
-function installEmbeddedMupdfWasm(): void {
-	const wasmBinary = loadEmbeddedMupdfWasm();
-	if (!wasmBinary) return;
-	const moduleConfig: MuPdfWasmModuleConfig = globalThis.$libmupdf_wasm_Module ?? {};
-	moduleConfig.wasmBinary = wasmBinary;
-	globalThis.$libmupdf_wasm_Module = moduleConfig;
-}
-
-installMuPdfWasmLogger();
-
 let markit: () => Markit | Promise<Markit> = async () => {
-	// Lazy: keep the document engine (mammoth/mupdf) off the startup
-	// import graph — it loads only when a document is first converted.
-	installEmbeddedMupdfWasm();
+	// Lazy: keep the document engine off the startup import graph — it loads
+	// only when a document is first converted.
 	const promise = import("../markit").then(({ Markit }) => {
 		const instance = new Markit();
 		markit = () => instance;

package/src/utils/title-generator.ts

@@ -4,7 +4,7 @@
 import { dlopen, FFIType, ptr } from "bun:ffi";
 import * as path from "node:path";
 
-import { type Api, type AssistantMessage, completeSimple, type Model } from "@oh-my-pi/pi-ai";
+import { type Api, type AssistantMessage, completeSimple, type Model, retryTransientCompletion } from "@oh-my-pi/pi-ai";
 import { StreamMarkupHealing } from "@oh-my-pi/pi-ai/utils/stream-markup-healing";
 import { isConPTYHosted } from "@oh-my-pi/pi-tui";
 import { isTerminalHeadless, logger, prompt } from "@oh-my-pi/pi-utils";
@@ -254,24 +254,28 @@ export async function generateTitleOnline(
 		const maxTokens = TITLE_MAX_TOKENS;
 		logger.debug("title-generator: request", { ...modelContext, maxTokens });
 
-		const response = await completeSimple(
-			model,
-			{
-				systemPrompt,
-				messages: [{ role: "user", content: userMessage, timestamp: Date.now() }],
-			},
-			{
-				apiKey: registry.resolver(model, sessionId),
-				maxTokens,
-				disableReasoning: true,
-				// Greedy decode: titling is extraction, not generation. Backends that
-				// default temperature high (e.g. Ollama's 0.8) otherwise garble names
-				// from the message ("hashline" → "HasHroshi"). Providers whose models
-				// reject sampling params drop this via `supportsSamplingParams`.
-				temperature: 0,
-				metadata,
-				signal,
-			},
+		const response = await retryTransientCompletion(
+			() =>
+				completeSimple(
+					model,
+					{
+						systemPrompt,
+						messages: [{ role: "user", content: userMessage, timestamp: Date.now() }],
+					},
+					{
+						apiKey: registry.resolver(model, sessionId),
+						maxTokens,
+						disableReasoning: true,
+						// Greedy decode: titling is extraction, not generation. Backends that
+						// default temperature high (e.g. Ollama's 0.8) otherwise garble names
+						// from the message ("hashline" → "HasHroshi"). Providers whose models
+						// reject sampling params drop this via `supportsSamplingParams`.
+						temperature: 0,
+						metadata,
+						signal,
+					},
+				),
+			{ signal },
 		);
 
 		if (response.stopReason === "error") {

package/src/vibe/runtime.ts

@@ -421,11 +421,17 @@ export class VibeSessionRegistry {
 	}
 
 	/**
-	 * Insert a bare worker record without the spawn/job machinery. Test-only —
-	 * lets {@link aggregateVibeWorkerTokensPerSecond} be exercised against a
-	 * fake roster + AgentRegistry session without driving a real turn.
+	 * Insert a bare worker record without the spawn machinery. Test-only —
+	 * lets focused runtime tests attach an optional synthetic in-flight job.
 	 */
-	registerRecordForTests(record: { id: string; cli?: VibeCli; ownerId: string; state?: VibeSessionState }): void {
+	registerRecordForTests(record: {
+		id: string;
+		cli?: VibeCli;
+		ownerId: string;
+		state?: VibeSessionState;
+		jobId?: string;
+	}): void {
+		const now = Date.now();
 		this.#records.set(record.id, {
 			id: record.id,
 			cli: record.cli ?? "fast",
@@ -434,8 +440,11 @@ export class VibeSessionRegistry {
 			parentSessionFile: null,
 			agent: getBundledAgent("sonic")!,
 			state: record.state ?? "running",
-			createdAt: Date.now(),
-			lastActivityAt: Date.now(),
+			createdAt: now,
+			lastActivityAt: now,
+			turn: record.jobId
+				? { jobId: record.jobId, message: "test turn", startedAt: now, trace: [], toolCount: 0 }
+				: undefined,
 			queue: [],
 			turnCount: 0,
 			killed: false,
@@ -1113,25 +1122,31 @@ export class VibeSessionRegistry {
 			if (job?.status === "running") runningJobs.push(job);
 		}
 
-		let waited = false;
+		let waitEndedByTimeout = false;
 		if (runningJobs.length > 0 && collectSettled().length === 0) {
-			waited = true;
 			const timeoutMs = Math.max(1, Math.trunc(args.timeoutMs ?? DEFAULT_WAIT_TIMEOUT_MS));
 			const watchedJobIds = runningJobs.map(job => job.id);
 			manager.watchJobs(watchedJobIds);
-			const { promise: timeoutPromise, resolve: timeoutResolve } = Promise.withResolvers<void>();
-			const timeoutHandle = setTimeout(() => timeoutResolve(), timeoutMs);
-			const racePromises: Promise<unknown>[] = [...runningJobs.map(job => job.promise), timeoutPromise];
+			const { promise: timeoutPromise, resolve: timeoutResolve } = Promise.withResolvers<"timeout">();
+			const timeoutHandle = setTimeout(() => timeoutResolve("timeout"), timeoutMs);
+			const racePromises: Array<Promise<"settled" | "timeout" | "aborted">> = [
+				...runningJobs.map(job => job.promise.then(() => "settled" as const)),
+				timeoutPromise,
+			];
 			let abortCleanup: (() => void) | undefined;
 			if (args.signal) {
-				const { promise: abortPromise, resolve: abortResolve } = Promise.withResolvers<void>();
-				const onAbort = () => abortResolve();
-				args.signal.addEventListener("abort", onAbort, { once: true });
-				abortCleanup = () => args.signal?.removeEventListener("abort", onAbort);
+				const { promise: abortPromise, resolve: abortResolve } = Promise.withResolvers<"aborted">();
+				const onAbort = () => abortResolve("aborted");
+				if (args.signal.aborted) {
+					onAbort();
+				} else {
+					args.signal.addEventListener("abort", onAbort, { once: true });
+					abortCleanup = () => args.signal?.removeEventListener("abort", onAbort);
+				}
 				racePromises.push(abortPromise);
 			}
 			try {
-				await Promise.race(racePromises);
+				waitEndedByTimeout = (await Promise.race(racePromises)) === "timeout";
 			} finally {
 				manager.unwatchJobs(watchedJobIds);
 				clearTimeout(timeoutHandle);
@@ -1144,7 +1159,7 @@ export class VibeSessionRegistry {
 		// Current in-flight state, independent of the snapshot: a session whose
 		// watched turn settled may already be mid queued follow-up.
 		const stillRunning = watched.filter(record => record.turn !== undefined).map(record => record.id);
-		return { settled, stillRunning, timedOut: waited && settled.length === 0 };
+		return { settled, stillRunning, timedOut: waitEndedByTimeout && settled.length === 0 };
 	}
 
 	/** Detach one parent's process-local workers without tombstoning their persisted conversations. */

package/dist/types/markit/converters/pdf/columns.d.ts

@@ -1,35 +0,0 @@
-/**
- * Multi-column layout detection and text box reordering.
- *
- * Many PDFs (legal documents, datasheets, academic papers) use two-column
- * layouts. Without column detection, text boxes are ordered by Y position
- * only, interleaving left and right column content.
- *
- * Algorithm:
- *   1. Collect left edges of all text boxes on the page
- *   2. Find the largest horizontal gap between consecutive left edges
- *   3. If gap > MIN_GAP_RATIO of the text width and both sides have
- *      enough boxes → multi-column detected
- *   4. Assign each text box to a column based on its center X
- *   5. Return columns in reading order (left-to-right, top-to-bottom)
- *
- * This only detects the column structure. The caller is responsible for
- * processing each column's text boxes independently (table detection,
- * rendering, etc.).
- */
-import type { TextBox } from "./types.js";
-export interface ColumnLayout {
-    /** Number of columns detected (1 = single column, 2+ = multi-column). */
-    columnCount: number;
-    /** Text boxes grouped by column, in reading order (left to right). */
-    columns: TextBox[][];
-    /** X positions of column boundaries (between columns). */
-    boundaries: number[];
-}
-/**
- * Detect column layout and return text boxes grouped by column.
- *
- * For single-column pages, returns all boxes in one group.
- * For multi-column pages, returns boxes split by column in reading order.
- */
-export declare function detectColumns(textBoxes: TextBox[]): ColumnLayout;

package/dist/types/markit/converters/pdf/extract.d.ts

@@ -1,10 +0,0 @@
-import type { ImageRegion, PageContent } from "./types.js";
-/**
- * Render an image region from a PDF page as a PNG buffer.
- * Uses mupdf's DrawDevice to render just the cropped area at 2x resolution.
- */
-export declare function renderImageRegion(input: Uint8Array, region: ImageRegion): Promise<Uint8Array>;
-/**
- * Extract text boxes and vector segments from all pages of a PDF buffer.
- */
-export declare function extractPages(input: Uint8Array): Promise<PageContent[]>;

package/dist/types/markit/converters/pdf/grid.d.ts

@@ -1,25 +0,0 @@
-/**
- * Table grid detection from vector segments and text boxes.
- *
- * Ported from @oharato/pdf2md-ts with TypeScript types and without
- * CJK-specific borderless table heuristics. The core algorithm:
- *
- * 1. Classify segments as horizontal or vertical lines
- * 2. Group horizontal Y-lines into table groups (split by vertical gaps)
- * 3. For each group:
- *    a. Full grid (H+V lines): build cells from grid intersections,
- *       place text via raycasting
- *    b. H-line only (no V lines): infer columns from text X positions
- * 4. Prune empty rows/cols
- *
- * Coordinate system: PDF native (bottom-left origin, Y increases upward).
- */
-import type { Segment, TableGrid, TextBox } from "./types.js";
-export interface GridResult {
-    grids: TableGrid[];
-    consumedIds: string[];
-}
-/**
- * Detect all table grids on a single page from its text boxes and segments.
- */
-export declare function resolveTableGrids(pageNumber: number, textBoxes: TextBox[], segments: Segment[]): GridResult;

package/dist/types/markit/converters/pdf/headers.d.ts

@@ -1,24 +0,0 @@
-/**
- * Running header/footer detection and removal.
- *
- * Many PDFs have repeated text at the top or bottom of every page:
- * document titles, chapter names, page numbers, copyright notices.
- * These pollute the markdown output as false headings or noise.
- *
- * Algorithm:
- *   1. For each page, bucket text boxes by Y position (top/bottom zones)
- *   2. Collect the text content at each zone across all pages
- *   3. Text appearing on >20% of pages OR 8+ consecutive pages is a
- *      running header/footer
- *   4. Remove matching text boxes before further processing
- */
-import type { PageContent } from "./types.js";
-/**
- * Detect and remove running headers and footers from all pages.
- * Mutates the pages array in place, removing header/footer text boxes.
- *
- * Uses two strategies:
- *   1. Global frequency: text appearing on > 20% of all pages
- *   2. Consecutive runs: text appearing on 8+ consecutive pages
- */
-export declare function stripHeadersFooters(pages: PageContent[]): void;

package/dist/types/markit/converters/pdf/render.d.ts

@@ -1,24 +0,0 @@
-/**
- * Markdown rendering for PDF pages.
- *
- * Converts table grids and free text boxes into markdown, handling:
- * - Table grid → markdown table (`| col | col |`)
- * - Free text → paragraphs with heading detection (by font size)
- * - Content ordering (top-to-bottom via Y coordinate)
- * - Paragraph wrap merging (lines broken across PDF line boundaries)
- * - Page number removal
- *
- * Ported from @oharato/pdf2md-ts, stripped of CJK/TDnet-specific logic.
- */
-import type { TableGrid, TextBox } from "./types.js";
-/**
- * Render a TableGrid as a markdown table.
- */
-export declare function renderTableToMarkdown(table: TableGrid): string;
-/**
- * Render one page's content: free text and tables interleaved top-to-bottom.
- */
-export declare function renderPageContent(freeTextBoxes: TextBox[], tables: TableGrid[], imageBlocks?: Array<{
-    topY: number;
-    markdown: string;
-}>, allTextBoxes?: TextBox[]): string;

package/dist/types/markit/converters/pdf/types.d.ts

@@ -1,75 +0,0 @@
-/** Bounding box in PDF coordinate space (origin = bottom-left). */
-export type Bounds = {
-    left: number;
-    right: number;
-    /** Higher value = higher on the page. */
-    top: number;
-    bottom: number;
-};
-/** A text fragment with position and font metadata. */
-export type TextBox = {
-    id: string;
-    text: string;
-    bounds: Bounds;
-    pageNumber: number;
-    /** Dominant font size in points. */
-    fontSize: number;
-    /** True if rendered bold (font name or rendering mode). */
-    isBold: boolean;
-};
-/** A horizontal or vertical line segment extracted from vector graphics. */
-export type Segment = {
-    id: string;
-    x1: number;
-    y1: number;
-    x2: number;
-    y2: number;
-};
-/** A single cell in a resolved table grid. */
-export type TableCell = {
-    row: number;
-    col: number;
-    text: string;
-    rowSpan: number;
-    colSpan: number;
-};
-/** A resolved table grid ready for markdown rendering. */
-export type TableGrid = {
-    pageNumber: number;
-    rows: number;
-    cols: number;
-    cells: TableCell[];
-    warnings: string[];
-    /** Top Y coordinate (PDF space: larger = higher on page). */
-    topY: number;
-    /** True for tables detected without vector borders. */
-    isBorderless: boolean;
-};
-/** An image/diagram region detected on a page. */
-export type ImageRegion = {
-    id: string;
-    pageNumber: number;
-    /** Bounding box in mupdf coordinates (top-left origin). */
-    bbox: {
-        x: number;
-        y: number;
-        w: number;
-        h: number;
-    };
-    /** Y position in PDF coordinates (bottom-left) for ordering. */
-    topY: number;
-};
-/** Result of extracting content from a single PDF page. */
-export type PageContent = {
-    pageNumber: number;
-    textBoxes: TextBox[];
-    segments: Segment[];
-    images: ImageRegion[];
-};
-/** A block of rendered content (text paragraph or table). */
-export type ContentBlock = {
-    topY: number;
-    content: string;
-    /** True if this line has wide gaps between text boxes (column headers). */
-    isTabular?: boolean;
-};

package/dist/types/tools/read-pdf-images.d.ts

@@ -1,12 +0,0 @@
-import type { AgentToolResult } from "@oh-my-pi/pi-agent-core";
-import type { ToolSession } from "../sdk.js";
-import type { ReadToolDetails } from "./read.js";
-export declare function rewritePdfImagePlaceholders(markdown: string, pdfPath: string): string;
-export declare function splitPdfImageMemberReadPath(readPath: string): {
-    pdfPath: string;
-    member: string;
-} | null;
-export declare function readPdfImageMember(session: ToolSession, autoResizeImages: boolean, absolutePdfPath: string, pdfDisplayPath: string, member: string, suffixResolution: {
-    from: string;
-    to: string;
-} | undefined, signal?: AbortSignal): Promise<AgentToolResult<ReadToolDetails>>;

package/dist/types/utils/mupdf-wasm-embed.d.ts

@@ -1 +0,0 @@
-export declare function loadEmbeddedMupdfWasm(): Uint8Array | undefined;

package/scripts/embed-mupdf-wasm.ts

@@ -1,67 +0,0 @@
-#!/usr/bin/env bun
-
-// Embeds mupdf's `mupdf-wasm.wasm` into the compiled single-file binary.
-//
-// mupdf loads its wasm by reading the `mupdf-wasm.wasm` sibling of its own
-// module via `new URL(..., import.meta.url)` + `readFileSync`. A `bun --compile`
-// binary has no node_modules, so that read fails (`ENOENT .../mupdf-wasm.wasm`),
-// and marking mupdf `--external` instead makes `bun --compile` eagerly fail to
-// resolve the package at startup (the static `import * as mupdf` lives in a lazy
-// chunk but is hoisted). So the binary build bundles mupdf and embeds the wasm
-// bytes here, handing them to the WASM module as `$libmupdf_wasm_Module.wasmBinary`
-// (see src/utils/markit.ts).
-//
-// `--generate` copies the wasm next to src/utils/mupdf-wasm-embed.ts and rewrites
-// that module to import it via `with { type: "file" }`; `--reset` restores the
-// checked-in placeholder and removes the copy. The npm `dist/cli.js` bundle never
-// runs this — it keeps mupdf external and loads the wasm from node_modules.
-
-import * as fs from "node:fs/promises";
-import { createRequire } from "node:module";
-import * as path from "node:path";
-
-const utilsDir = path.join(import.meta.dir, "..", "src", "utils");
-const helperPath = path.join(utilsDir, "mupdf-wasm-embed.ts");
-const wasmCopyPath = path.join(utilsDir, "mupdf-wasm.wasm");
-
-const placeholder = `// AUTOGENERATED -- managed by scripts/embed-mupdf-wasm.ts. Do not edit by hand.
-//
-// Compiled single-file binaries cannot let mupdf resolve its \`mupdf-wasm.wasm\`
-// sibling from the read-only bunfs, so the binary build (scripts/build-binary.ts
-// and scripts/ci-release-build-binaries.ts) regenerates this module to embed the
-// wasm bytes via \`with { type: "file" }\` and copies the wasm next to it. Source
-// checkouts, \`bun test\`, and the npm \`dist/cli.js\` bundle keep mupdf external and
-// load the wasm from node_modules, so this placeholder returns undefined and the
-// build resets back to it afterward.
-export function loadEmbeddedMupdfWasm(): Uint8Array | undefined {
-\treturn undefined;
-}
-`;
-
-const generated = `// AUTOGENERATED -- managed by scripts/embed-mupdf-wasm.ts. Do not edit or commit.
-import { readFileSync } from "node:fs";
-import wasmPath from "./mupdf-wasm.wasm" with { type: "file" };
-
-export function loadEmbeddedMupdfWasm(): Uint8Array | undefined {
-\treturn readFileSync(wasmPath);
-}
-`;
-
-if (process.argv.includes("--reset")) {
-	await Bun.write(helperPath, placeholder);
-	try {
-		await fs.unlink(wasmCopyPath);
-	} catch (err) {
-		if ((err as NodeJS.ErrnoException).code !== "ENOENT") throw err;
-	}
-	process.exit(0);
-}
-
-const wasmSource = path.join(path.dirname(createRequire(import.meta.url).resolve("mupdf")), "mupdf-wasm.wasm");
-const wasmFile = Bun.file(wasmSource);
-if (!(await wasmFile.exists())) {
-	throw new Error(`mupdf wasm not found at ${wasmSource}; run \`bun install\` first.`);
-}
-await Bun.write(wasmCopyPath, wasmFile);
-await Bun.write(helperPath, generated);
-console.log(`Embedded mupdf wasm (${wasmFile.size} bytes) into ${path.relative(process.cwd(), wasmCopyPath)}`);