pi-readseek 0.4.12 → 0.4.14

package/src/readseek-value.ts

@@ -1,7 +1,7 @@
 import { computeLineHash, escapeControlCharsForDisplay } from "./hashline.js";
 import type { DiffData } from "./diff-data.js";
 
-export interface ReadseekLine {
+export interface ReadSeekLine {
   line: number;
   hash: string;
   anchor: string;
@@ -9,29 +9,29 @@ export interface ReadseekLine {
   display: string;
 }
 
-export interface ReadseekWarningSymbol {
+export interface ReadSeekWarningSymbol {
   name: string;
   kind: string;
   startLine: number;
   endLine: number;
   parentName?: string;
 }
-export interface ReadseekWarning {
+export interface ReadSeekWarning {
   code: string;
   message: string;
   tier?: "camelCase" | "substring";
-  symbol?: ReadseekWarningSymbol;
-  otherCandidates?: ReadseekWarningSymbol[];
+  symbol?: ReadSeekWarningSymbol;
+  otherCandidates?: ReadSeekWarningSymbol[];
 }
 
-export interface ReadseekError {
+export interface ReadSeekError {
   code: string;
   message: string;
   hint?: string;
   details?: unknown;
 }
 
-export interface ReadseekRange {
+export interface ReadSeekRange {
   startLine: number;
   endLine: number;
   totalLines?: number;
@@ -42,7 +42,7 @@ export interface SemanticSummary {
   difftasticAvailable: boolean;
   movedBlocks?: number;
 }
-export interface ReadseekEditResult {
+export interface ReadSeekEditResult {
   tool: "edit";
   ok: boolean;
   path: string;
@@ -56,11 +56,11 @@ export interface ReadseekEditResult {
 }
 
 /**
- * Build a {@link ReadseekLine} from an already-known hash. Use this when the
+ * Build a {@link ReadSeekLine} from an already-known hash. Use this when the
  * hash is supplied by readseek (search, refs) rather than computed from `raw`;
- * {@link buildReadseekLine} delegates here after hashing.
+ * {@link buildReadSeekLine} delegates here after hashing.
  */
-export function buildReadseekLineWithHash(line: number, hash: string, raw: string): ReadseekLine {
+export function buildReadSeekLineWithHash(line: number, hash: string, raw: string): ReadSeekLine {
   return {
     line,
     hash,
@@ -70,36 +70,36 @@ export function buildReadseekLineWithHash(line: number, hash: string, raw: strin
   };
 }
 
-export function buildReadseekLine(line: number, raw: string): ReadseekLine {
-  return buildReadseekLineWithHash(line, computeLineHash(raw), raw);
+export function buildReadSeekLine(line: number, raw: string): ReadSeekLine {
+  return buildReadSeekLineWithHash(line, computeLineHash(raw), raw);
 }
 
-export function buildReadseekLines(startLine: number, rawLines: string[]): ReadseekLine[] {
-  return rawLines.map((raw, index) => buildReadseekLine(startLine + index, raw));
+export function buildReadSeekLines(startLine: number, rawLines: string[]): ReadSeekLine[] {
+  return rawLines.map((raw, index) => buildReadSeekLine(startLine + index, raw));
 }
 
-function renderReadseekLine(line: ReadseekLine): string {
+function renderReadSeekLine(line: ReadSeekLine): string {
   return `${line.anchor}|${line.display}`;
 }
 
-export function renderReadseekLines(lines: ReadseekLine[]): string {
-  return lines.map(renderReadseekLine).join("\n");
+export function renderReadSeekLines(lines: ReadSeekLine[]): string {
+  return lines.map(renderReadSeekLine).join("\n");
 }
 
-export function buildReadseekWarning(
+export function buildReadSeekWarning(
   code: string,
   message: string,
-  metadata: Omit<ReadseekWarning, "code" | "message"> = {},
-): ReadseekWarning {
+  metadata: Omit<ReadSeekWarning, "code" | "message"> = {},
+): ReadSeekWarning {
   return { code, message, ...metadata };
 }
 
-export function buildReadseekError(
+export function buildReadSeekError(
   code: string,
   message: string,
   hint?: string,
   details?: unknown,
-): ReadseekError {
+): ReadSeekError {
   return {
     code,
     message,
@@ -117,7 +117,7 @@ export interface ToolErrorResult {
 /**
  * Build the standard failure envelope shared by every read-family tool: a text
  * content block plus a `readseekValue` carrying `ok: false` and a
- * {@link buildReadseekError} payload.
+ * {@link buildReadSeekError} payload.
  *
  * `path` is included only when provided, and `extra` is merged into
  * `readseekValue` so callers can attach tool-specific fields (e.g. write's
@@ -138,13 +138,13 @@ export function buildToolErrorResult(
         ...(opts.extra ?? {}),
         ok: false,
         ...(opts.path !== undefined ? { path: opts.path } : {}),
-        error: buildReadseekError(code, message, opts.hint, opts.details),
+        error: buildReadSeekError(code, message, opts.hint, opts.details),
       },
     },
   };
 }
 
-export function buildReadseekEditResult(input: {
+export function buildReadSeekEditResult(input: {
   ok?: boolean;
   path: string;
   summary: string;
@@ -154,7 +154,7 @@ export function buildReadseekEditResult(input: {
   warnings: string[];
   noopEdits: unknown[];
   semanticSummary?: SemanticSummary;
-}): ReadseekEditResult {
+}): ReadSeekEditResult {
   return {
     tool: "edit",
     ok: input.ok ?? true,

package/src/refs-output.ts

@@ -1,6 +1,6 @@
-import type { ReadseekLine } from "./readseek-value.js";
+import type { ReadSeekLine } from "./readseek-value.js";
 
-export interface RefsOutputLine extends ReadseekLine {
+export interface RefsOutputLine extends ReadSeekLine {
   enclosingSymbol?: string;
 }
 
@@ -10,12 +10,12 @@ export interface RefsOutputFile {
   lines: RefsOutputLine[];
 }
 
-export interface BuildRefsOutputInput {
+interface BuildRefsOutputInput {
   name: string;
   files: RefsOutputFile[];
 }
 
-export interface RefsOutputResult {
+interface RefsOutputResult {
   text: string;
   readseekValue: {
     tool: "refs";

package/src/refs.ts

@@ -4,12 +4,12 @@ import { Type } from "@sinclair/typebox";
 import path from "node:path";
 import { defineToolPromptMetadata } from "./tool-prompt-metadata.js";
 import { statSearchPathOrError } from "./stat-search-path.js";
-import { buildReadseekLineWithHash, buildToolErrorResult } from "./readseek-value.js";
+import { buildReadSeekLineWithHash, buildToolErrorResult } from "./readseek-value.js";
 import { resolveToCwd } from "./path-utils.js";
-import { classifyReadseekFailure, readseekRefs, type ReadseekReference } from "./readseek-client.js";
+import { classifyReadSeekFailure, readseekRefs, type ReadSeekReference } from "./readseek-client.js";
 import { buildRefsOutput, type RefsOutputFile, type RefsOutputLine } from "./refs-output.js";
 import type { FileAnchoredCallback } from "./tool-types.js";
-import { registerReadseekTool } from "./register-tool.js";
+import { registerReadSeekTool } from "./register-tool.js";
 
 import { clampLineToWidth, renderAnchoredFilesResult, renderToolLabel } from "./tui-render-utils.js";
 
@@ -44,14 +44,14 @@ export interface ExecuteRefsOptions {
   onFileAnchored?: FileAnchoredCallback;
 }
 
-function refsLine(reference: ReadseekReference): RefsOutputLine {
+function refsLine(reference: ReadSeekReference): RefsOutputLine {
   return {
-    ...buildReadseekLineWithHash(reference.line, reference.line_hash, reference.text),
+    ...buildReadSeekLineWithHash(reference.line, reference.line_hash, reference.text),
     enclosingSymbol: reference.enclosingSymbol,
   };
 }
 
-function groupReferences(references: ReadseekReference[], cwd: string): RefsOutputFile[] {
+function groupReferences(references: ReadSeekReference[], cwd: string): RefsOutputFile[] {
   const files = new Map<string, RefsOutputFile>();
   const seen = new Set<string>();
   for (const reference of references) {
@@ -69,7 +69,7 @@ function groupReferences(references: ReadseekReference[], cwd: string): RefsOutp
   return [...files.values()];
 }
 
-function isReadseekCursorValidationFailure(message: string): boolean {
+function isReadSeekCursorValidationFailure(message: string): boolean {
   return (
     /line and column must be greater than zero/i.test(message) ||
     /line \d+ not found/i.test(message) ||
@@ -115,8 +115,8 @@ export async function executeRefs(opts: ExecuteRefsOptions): Promise<any> {
       details: { readseekValue: builtOutput.readseekValue },
     };
   } catch (err: any) {
-    const failure = classifyReadseekFailure(err);
-    if (p.scope && isReadseekCursorValidationFailure(failure.message)) {
+    const failure = classifyReadSeekFailure(err);
+    if (p.scope && isReadSeekCursorValidationFailure(failure.message)) {
       return buildToolErrorResult("refs", "invalid-parameter", failure.message);
     }
     return buildToolErrorResult("refs", failure.code, failure.message, failure.hint ? { hint: failure.hint } : {});
@@ -124,7 +124,7 @@ export async function executeRefs(opts: ExecuteRefsOptions): Promise<any> {
 }
 
 export function registerRefsTool(pi: ExtensionAPI, options: RefsToolOptions = {}) {
-  const tool = registerReadseekTool(pi, {
+  const tool = registerReadSeekTool(pi, {
     policy: "read-only",
     pythonName: "refs",
     defaultExposure: "opt-in",

package/src/register-tool.ts

@@ -1,20 +1,20 @@
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 
-export type ReadseekToolPolicy = "read-only" | "mutating";
+export type ReadSeekToolPolicy = "read-only" | "mutating";
 
-export type ReadseekToolExposure = "safe-by-default" | "opt-in" | "not-safe-by-default";
+export type ReadSeekToolExposure = "safe-by-default" | "opt-in" | "not-safe-by-default";
 
 /**
  * Per-tool fields of the pi tool config (`ptc`) that genuinely differ between
  * readseek tools. The remaining fields are constant or derived.
  */
-export interface ReadseekToolConfig {
-  policy: ReadseekToolPolicy;
+export interface ReadSeekToolConfig {
+  policy: ReadSeekToolPolicy;
   pythonName: string;
-  defaultExposure: ReadseekToolExposure;
+  defaultExposure: ReadSeekToolExposure;
 }
 
-export interface ReadseekToolPtc extends ReadseekToolConfig {
+export interface ReadSeekToolPtc extends ReadSeekToolConfig {
   callable: true;
   enabled: true;
   readOnly: boolean;
@@ -28,12 +28,12 @@ type ToolSpec = Parameters<ExtensionAPI["registerTool"]>[0];
  * `callable` and `enabled` are always true and `readOnly` is derived from
  * `policy`, so callers supply only the fields that vary between tools.
  */
-export function registerReadseekTool<T extends ToolSpec>(
+export function registerReadSeekTool<T extends ToolSpec>(
   pi: ExtensionAPI,
-  config: ReadseekToolConfig,
+  config: ReadSeekToolConfig,
   tool: T,
-): T & { ptc: ReadseekToolPtc } {
-  const ptc: ReadseekToolPtc = {
+): T & { ptc: ReadSeekToolPtc } {
+  const ptc: ReadSeekToolPtc = {
     callable: true,
     enabled: true,
     policy: config.policy,

package/src/replace-symbol.ts

@@ -1,15 +1,16 @@
 import { readseekMapContent } from "./readseek-client.js";
 import { findSymbol } from "./readseek/symbol-lookup.js";
 import { formatAmbiguous, formatNotFound } from "./readseek/symbol-error-format.js";
+import { normalizeToLF } from "./edit-diff.js";
 
-export interface ReplaceSymbolInput {
+interface ReplaceSymbolInput {
 	filePath: string;
 	content: string;
 	symbol: string;
 	newBody: string;
 }
 
-export type ReplaceSymbolResult =
+type ReplaceSymbolResult =
 	| { type: "ok"; content: string; replacement: string; warnings: string[]; range: { start: number; end: number } }
 	| { type: "not-found"; message: string }
 	| { type: "ambiguous"; message: string }
@@ -50,7 +51,7 @@ export async function replaceSymbol(input: ReplaceSymbolInput): Promise<ReplaceS
 	const lines = input.content.split("\n");
 	const sigLine = lines[sym.startLine - 1] ?? "";
 	const indent = detectIndent(sigLine);
-	const reindented = reindent(dedent(input.newBody), indent);
+	const reindented = reindent(dedent(normalizeToLF(input.newBody)), indent);
 	const warnings: string[] = [];
 	const leaf = input.symbol.replace(/@\d+$/, "").split(".").pop() ?? "";
 	const declNameRe =

package/src/sg-output.ts

@@ -1,10 +1,10 @@
-import type { ReadseekLine, ReadseekRange } from "./readseek-value.js";
+import type { ReadSeekLine, ReadSeekRange } from "./readseek-value.js";
 
 export interface SgOutputFile {
   displayPath: string;
   path: string;
-  ranges: ReadseekRange[];
-  lines: ReadseekLine[];
+  ranges: ReadSeekRange[];
+  lines: ReadSeekLine[];
   symbols?: Array<{ name: string; kind?: string }>;
 }
 
@@ -19,8 +19,8 @@ export interface SgOutputResult {
     tool: "search";
     files: Array<{
       path: string;
-      ranges: ReadseekRange[];
-      lines: ReadseekLine[];
+      ranges: ReadSeekRange[];
+      lines: ReadSeekLine[];
     }>;
   };
 }

package/src/sg.ts

@@ -5,13 +5,13 @@ import { Text } from "@earendil-works/pi-tui";
 import { Type } from "@sinclair/typebox";
 
 import { defineToolPromptMetadata } from "./tool-prompt-metadata.js";
-import { buildReadseekLineWithHash, buildToolErrorResult, type ReadseekLine } from "./readseek-value.js";
+import { buildReadSeekLineWithHash, buildToolErrorResult, type ReadSeekLine } from "./readseek-value.js";
 import { resolveToCwd } from "./path-utils.js";
 import { statSearchPathOrError } from "./stat-search-path.js";
-import { classifyReadseekFailure, isReadseekAvailable, readseekSearch, type ReadseekHashline, type ReadseekSearchFileOutput } from "./readseek-client.js";
+import { classifyReadSeekFailure, isReadSeekAvailable, readseekSearch, type ReadSeekHashline, type ReadSeekSearchFileOutput } from "./readseek-client.js";
 import { buildSgOutput } from "./sg-output.js";
 import type { FileAnchoredCallback } from "./tool-types.js";
-import { registerReadseekTool } from "./register-tool.js";
+import { registerReadSeekTool } from "./register-tool.js";
 
 import { clampLineToWidth, renderAnchoredFilesResult, renderToolLabel } from "./tui-render-utils.js";
 
@@ -53,7 +53,7 @@ const SG_PROMPT_METADATA = defineToolPromptMetadata({
 });
 
 export function isSgAvailable(): boolean {
-  return isReadseekAvailable();
+  return isReadSeekAvailable();
 }
 
 interface SgToolOptions {
@@ -70,19 +70,19 @@ export interface ExecuteSgOptions {
   onFileAnchored?: FileAnchoredCallback;
 }
 
-function readseekLineFromSearch(line: ReadseekHashline): ReadseekLine {
-  return buildReadseekLineWithHash(line.line, line.hash, line.text);
+function readseekLineFromSearch(line: ReadSeekHashline): ReadSeekLine {
+  return buildReadSeekLineWithHash(line.line, line.hash, line.text);
 }
 
-function linesFromSearchResult(result: ReadseekSearchFileOutput, ranges: SgRange[]): ReadseekLine[] {
-  const lineMap = new Map<number, ReadseekLine>();
+function linesFromSearchResult(result: ReadSeekSearchFileOutput, ranges: SgRange[]): ReadSeekLine[] {
+  const lineMap = new Map<number, ReadSeekLine>();
   for (const match of result.matches) {
     for (const line of match.hashlines) {
       lineMap.set(line.line, readseekLineFromSearch(line));
     }
   }
 
-  const lines: ReadseekLine[] = [];
+  const lines: ReadSeekLine[] = [];
   const seen = new Set<number>();
   for (const range of ranges) {
     for (let line = range.startLine; line <= range.endLine; line++) {
@@ -140,7 +140,7 @@ export async function executeSg(opts: ExecuteSgOptions): Promise<any> {
       displayPath: string;
       path: string;
       ranges: SgRange[];
-      lines: ReadseekLine[];
+      lines: ReadSeekLine[];
       symbols?: SgEnclosingSymbol[];
     }> = [];
 
@@ -183,13 +183,13 @@ export async function executeSg(opts: ExecuteSgOptions): Promise<any> {
       },
     };
   } catch (err: any) {
-    const failure = classifyReadseekFailure(err);
+    const failure = classifyReadSeekFailure(err);
     return buildToolErrorResult("search", failure.code, failure.message, failure.hint ? { hint: failure.hint } : {});
   }
 }
 
 export function registerSgTool(pi: ExtensionAPI, options: SgToolOptions = {}) {
-  const tool = registerReadseekTool(pi, {
+  const tool = registerReadSeekTool(pi, {
     policy: "read-only",
     pythonName: "search",
     defaultExposure: "opt-in",

package/src/tool-prompt-metadata.ts

@@ -3,7 +3,7 @@ import { readFileSync } from "node:fs";
 import { DEFAULT_MAX_BYTES, DEFAULT_MAX_LINES, formatSize } from "@earendil-works/pi-coding-agent";
 
 const COMPACT_DESCRIPTIONS: Record<string, string> = {
-  "read.md": "Read text files/images by path; text has LINE:HASH anchors, images return attachments.",
+  "read.md": "Read text files/images by path; text has LINE:HASH anchors, images return the attachment plus OCR-extracted text.",
   "edit.md": "Edit existing text files using fresh LINE:HASH anchors from read, grep, search, or write.",
   "grep.md": "Search file contents; non-summary results include LINE:HASH anchors for edits.",
 
@@ -16,7 +16,7 @@ const COMPACT_GUIDELINES: Record<string, string[]> = {
   "read.md": [
     "Use read for file contents, images/screenshots, ranges, symbols, and edit anchors.",
     "Use map or symbol mode before pulling large code files into context.",
-    "Use read for images; it returns attachments, so avoid OCR tools unless explicitly needed.",
+    "Use read for images; it returns the image attachment plus OCR-extracted text, so you don't need separate OCR tools.",
   ],
   "edit.md": [
     "Use edit with fresh LINE:HASH anchors for existing files.",
@@ -41,20 +41,20 @@ const COMPACT_GUIDELINES: Record<string, string[]> = {
   ],
 };
 
-export interface ToolPromptMetadata {
+interface ToolPromptMetadata {
   description: string;
   promptSnippet: string;
   promptGuidelines: string[];
 }
 
-export function loadPrompt(promptUrl: URL): string {
+function loadPrompt(promptUrl: URL): string {
   return readFileSync(promptUrl, "utf-8")
     .replaceAll("{{DEFAULT_MAX_LINES}}", String(DEFAULT_MAX_LINES))
     .replaceAll("{{DEFAULT_MAX_BYTES}}", formatSize(DEFAULT_MAX_BYTES))
     .trim();
 }
 
-export function firstPromptParagraph(prompt: string): string {
+function firstPromptParagraph(prompt: string): string {
   return prompt.split(/\n\s*\n/, 1)[0]?.trim() ?? prompt;
 }
 

package/src/write.ts

@@ -7,9 +7,9 @@ import { Type } from "@sinclair/typebox";
 
 import { resolveToCwd } from "./path-utils.js";
 import { ensureHashInit, formatHashlineDisplay } from "./hashline.js";
-import { buildReadseekError, buildReadseekLine, buildReadseekWarning, type ReadseekLine, type ReadseekWarning } from "./readseek-value.js";
+import { buildReadSeekError, buildReadSeekLine, buildReadSeekWarning, buildToolErrorResult, type ReadSeekLine, type ReadSeekWarning } from "./readseek-value.js";
 import { looksLikeBinary } from "./binary-detect.js";
-import { getOrGenerateMap } from "./map-cache.js";
+import { getOrGenerateMap } from "./file-map.js";
 import { formatFileMapWithBudget } from "./readseek/formatter.js";
 
 import { defineToolPromptMetadata } from "./tool-prompt-metadata.js";
@@ -19,7 +19,7 @@ import { buildDiffData, type DiffData } from "./diff-data.js";
 import { clampLineToWidth, clampLinesToWidth, isRendererExpanded, linkToolPath, renderErrorResult, renderToolLabel, summaryLine } from "./tui-render-utils.js";
 import { DiffPreviewComponent } from "./tui-diff-component.js";
 import type { FileAnchoredCallback } from "./tool-types.js";
-import { registerReadseekTool } from "./register-tool.js";
+import { registerReadSeekTool } from "./register-tool.js";
 
 const WRITE_PENDING_PREVIEW_STATE_KEY = "hashline-write-pending-preview";
 
@@ -84,8 +84,8 @@ export interface WriteResult extends WriteDiffFields {
   readseekValue: {
     tool: "write";
     path: string;
-    lines: ReadseekLine[];
-    warnings: ReadseekWarning[];
+    lines: ReadSeekLine[];
+    warnings: ReadSeekWarning[];
     diff?: string;
     diffData?: DiffData;
     map?: { appended: boolean };
@@ -173,26 +173,11 @@ function mapFsWriteError(err: any, path: string): MappedFsError {
 
 function buildWriteFsErrorResult(err: any, absolutePath: string) {
   const mapped = mapFsWriteError(err, absolutePath);
-  return {
-    content: [{ type: "text" as const, text: mapped.message }],
-    isError: true,
-    details: {
-      readseekValue: {
-        tool: "write" as const,
-        path: absolutePath,
-        lines: [] as ReadseekLine[],
-        warnings: [] as ReadseekWarning[],
-        ok: false,
-        error: buildReadseekError(
-          mapped.code,
-          mapped.message,
-          undefined,
-          mapped.includeMeta ? { fsCode: err?.code, fsMessage: err?.message } : undefined,
-        ),
-      },
-      warnings: [] as string[],
-    },
-  };
+  return buildToolErrorResult("write", mapped.code, mapped.message, {
+    path: absolutePath,
+    extra: { lines: [], warnings: [] },
+    details: mapped.includeMeta ? { fsCode: err?.code, fsMessage: err?.message } : undefined,
+  });
 }
 
 export async function executeWrite(opts: {
@@ -205,12 +190,12 @@ export async function executeWrite(opts: {
 
   const { path: filePath, content, map: requestMap, cwd } = opts;
   const warnings: string[] = [];
-  const readseekWarnings: ReadseekWarning[] = [];
+  const readseekWarnings: ReadSeekWarning[] = [];
 
   if (hasBareCarriageReturn(content)) {
     const message = "File content contains bare CR (\\r) line endings; write refuses to emit anchors that read/edit would normalize differently.";
     warnings.push(message);
-    readseekWarnings.push(buildReadseekWarning("bare-cr", message));
+    readseekWarnings.push(buildReadSeekWarning("bare-cr", message));
     return {
       text: `Cannot write ${filePath}\n⚠️ ${message}`,
       warnings,
@@ -224,7 +209,7 @@ export async function executeWrite(opts: {
   }
   if (looksLikeBinary(Buffer.from(content, "utf-8"))) {
     warnings.push("File content appears to be binary.");
-    readseekWarnings.push(buildReadseekWarning("binary-content", "File content appears to be binary."));
+    readseekWarnings.push(buildReadSeekWarning("binary-content", "File content appears to be binary."));
     return {
       text: `Cannot write ${filePath}\n⚠️ File content appears to be binary — refusing to write.`,
       warnings,
@@ -257,12 +242,12 @@ export async function executeWrite(opts: {
 
   // Compute hashlines
   const rawLines = content.split("\n");
-  const readseekLines: ReadseekLine[] = [];
+  const readseekLines: ReadSeekLine[] = [];
   const displayLines: string[] = [];
 
   for (let i = 0; i < rawLines.length; i++) {
     const lineNum = i + 1;
-    const readseekLine = buildReadseekLine(lineNum, rawLines[i]);
+    const readseekLine = buildReadSeekLine(lineNum, rawLines[i]);
     readseekLines.push(readseekLine);
     displayLines.push(formatHashlineDisplay(lineNum, rawLines[i]));
   }
@@ -325,7 +310,7 @@ export async function executeWrite(opts: {
 }
 
 export function registerWriteTool(pi: ExtensionAPI, options: WriteToolOptions = {}) {
-  const tool = registerReadseekTool(pi, {
+  const tool = registerReadSeekTool(pi, {
     policy: "mutating",
     pythonName: "write",
     defaultExposure: "not-safe-by-default",
@@ -363,7 +348,7 @@ export function registerWriteTool(pi: ExtensionAPI, options: WriteToolOptions =
 
           // Lift binary-content signal into a fatal readseekValue.error envelope so
           // downstream consumers get the same taxonomy shape as every other tool.
-          // The existing ReadseekWarning entry is preserved on readseekValue.warnings for
+          // The existing ReadSeekWarning entry is preserved on readseekValue.warnings for
           // backward compatibility.
           for (const code of ["binary-content", "bare-cr"] as const) {
             const warning = result.readseekValue.warnings.find((w) => w.code === code);
@@ -375,7 +360,7 @@ export function registerWriteTool(pi: ExtensionAPI, options: WriteToolOptions =
                   readseekValue: {
                     ...result.readseekValue,
                     ok: false,
-                    error: buildReadseekError(code, warning.message),
+                    error: buildReadSeekError(code, warning.message),
                   },
                   warnings: result.warnings,
                 },

package/src/image-detect.ts

@@ -1,47 +0,0 @@
-const PNG_SIGNATURE = [0x89, 0x50, 0x4e, 0x47, 0x0d, 0x0a, 0x1a, 0x0a];
-
-function startsWithBytes(buffer: Buffer, bytes: number[]): boolean {
-	return buffer.length >= bytes.length && bytes.every((byte, index) => buffer[index] === byte);
-}
-
-function startsWithAscii(buffer: Buffer, offset: number, text: string): boolean {
-	if (buffer.length < offset + text.length) return false;
-	for (let index = 0; index < text.length; index++) {
-		if (buffer[offset + index] !== text.charCodeAt(index)) return false;
-	}
-	return true;
-}
-
-function readUint32BE(buffer: Buffer, offset: number): number {
-	return (
-		((buffer[offset] ?? 0) * 0x1000000) +
-		((buffer[offset + 1] ?? 0) << 16) +
-		((buffer[offset + 2] ?? 0) << 8) +
-		(buffer[offset + 3] ?? 0)
-	);
-}
-
-function isPng(buffer: Buffer): boolean {
-	return buffer.length >= 16 && readUint32BE(buffer, PNG_SIGNATURE.length) === 13 && startsWithAscii(buffer, 12, "IHDR");
-}
-
-function isAnimatedPng(buffer: Buffer): boolean {
-	let offset = PNG_SIGNATURE.length;
-	while (offset + 8 <= buffer.length) {
-		const chunkLength = readUint32BE(buffer, offset);
-		const chunkTypeOffset = offset + 4;
-		if (startsWithAscii(buffer, chunkTypeOffset, "acTL")) return true;
-		if (startsWithAscii(buffer, chunkTypeOffset, "IDAT")) return false;
-		const nextOffset = offset + 8 + chunkLength + 4;
-		if (nextOffset <= offset || nextOffset > buffer.length) return false;
-		offset = nextOffset;
-	}
-	return false;
-}
-
-export function isSupportedImageBuffer(buffer: Buffer): boolean {
-	if (startsWithBytes(buffer, [0xff, 0xd8, 0xff])) return buffer[3] !== 0xf7;
-	if (startsWithBytes(buffer, PNG_SIGNATURE)) return isPng(buffer) && !isAnimatedPng(buffer);
-	if (startsWithAscii(buffer, 0, "GIF")) return true;
-	return startsWithAscii(buffer, 0, "RIFF") && startsWithAscii(buffer, 8, "WEBP");
-}