pi-readseek 0.4.30 → 0.5.0

package/src/edit.ts

@@ -29,8 +29,6 @@ import { upsertDiffComponent, upsertTextComponent } from "./tui-diff-component.j
 import type { FreshAnchorsPredicate } from "./tool-types.js";
 import { filePathParam, registerReadSeekTool } from "./register-tool.js";
 
-import { resolveEditDiffDisplay } from "./readseek-settings.js";
-
 const EDIT_PENDING_PREVIEW_STATE_KEY = "hashline-edit-pending-preview";
 
 function pendingPreviewLines(summary: string, preview: PendingDiffPreviewResult | undefined, expanded: boolean): { lines: string[]; diffData?: ReturnType<typeof buildDiffData>; headerLabel?: string } {
@@ -73,7 +71,7 @@ type HashlineParams = Static<typeof hashlineEditSchema>;
 
 const EDIT_PROMPT_METADATA = defineToolPromptMetadata({
 	promptUrl: new URL("../prompts/edit.md", import.meta.url),
-	promptSnippet: "Edit files using hash-verified anchors from read/grep/search/write",
+	promptSnippet: "Edit files using hash-verified anchors from readSeek_read/readSeek_grep/readSeek_search/readSeek_write",
 });
 
 function buildEditError(
@@ -85,7 +83,7 @@ function buildEditError(
 ): {
 	content: [{ type: "text"; text: string }];
 	isError: true;
-	details: EditToolDetails & { readseekValue: any };
+	details: EditToolDetails & { readSeekValue: any };
 } {
 	return {
 		content: [{ type: "text", text: message }],
@@ -94,13 +92,13 @@ function buildEditError(
 			diff: "",
 			patch: "",
 			firstChangedLine: undefined,
-			readseekValue: {
+			readSeekValue: {
 				tool: "edit",
 				ok: false,
 				path,
 				error: buildReadSeekError(code, message, hint, errorDetails),
 			},
-		} as EditToolDetails & { readseekValue: any },
+		} as EditToolDetails & { readSeekValue: any },
 	};
 }
 
@@ -137,14 +135,14 @@ export async function executeEdit(opts: ExecuteEditOptions): Promise<any> {
 	if (wasReadInSession && !wasReadInSession(absolutePath)) {
 		const message = [
 			`You must get fresh anchors for ${absolutePath} before editing it.`,
-			`Call read(${JSON.stringify(rawPath)}) first, or use grep, search, or write to produce fresh anchors for this file.`,
-			"edit requires fresh LINE:HASH anchors from read, grep, search, or write so the hashes match the current file contents.",
+			`Call readSeek_read(${JSON.stringify(rawPath)}) first, or use readSeek_grep, readSeek_search, or readSeek_write to produce fresh anchors for this file.`,
+			"readSeek_edit requires fresh LINE:HASH anchors from readSeek_read, readSeek_grep, readSeek_search, or readSeek_write so the hashes match the current file contents.",
 		].join(" ");
 		return buildEditError(
 			absolutePath,
 			"file-not-read",
 			message,
-			`Call read(${JSON.stringify(rawPath)}) first, or use grep, search, or write to produce fresh anchors for this file.`,
+			`Call readSeek_read(${JSON.stringify(rawPath)}) first, or use readSeek_grep, readSeek_search, or readSeek_write to produce fresh anchors for this file.`,
 		);
 	}
 	const hasTopLevelReplaceInput =
@@ -253,7 +251,7 @@ export async function executeEdit(opts: ExecuteEditOptions): Promise<any> {
 	// Error-precedence order: replace_symbol resolution > anchor-overlap > anchored-edit.
 	//
 	// store successful probe results and reuse them in the apply loop so
-	// readseekMapContent is invoked at most once per replace_symbol edit.
+	// readSeekMapContent is invoked at most once per replace_symbol edit.
 	const replaceSymbolRanges: { start: number; end: number }[] = [];
 	const rsProbeResults: { type: "ok"; content: string; replacement: string; warnings: string[]; range: { start: number; end: number } }[] = [];
 	try {
@@ -365,7 +363,7 @@ export async function executeEdit(opts: ExecuteEditOptions): Promise<any> {
 	// Apply pass: reuse all probe results. The probe pass resolved every
 	// replace_symbol against originalNormalized; apply those replacements in
 	// reverse source order so original line ranges stay valid and no second
-	// replaceSymbol/readseekMapContent call is needed.
+	// replaceSymbol/readSeekMapContent call is needed.
 	const replaceSymbolWarnings: string[] = [];
 	if (rsProbeResults.length > 0) {
 		const lines = originalNormalized.split("\n");
@@ -539,10 +537,10 @@ export async function executeEdit(opts: ExecuteEditOptions): Promise<any> {
 			patch: builtOutput.patch,
 			diffData,
 			firstChangedLine: anchorResult.firstChangedLine ?? diffResult.firstChangedLine,
-			readseekValue: builtOutput.readseekValue,
+			readSeekValue: builtOutput.readSeekValue,
 		} as EditToolDetails & {
 			diffData: typeof diffData;
-			readseekValue: {
+			readSeekValue: {
 				tool: string;
 				ok: boolean;
 				path: string;
@@ -567,11 +565,7 @@ export async function executeEdit(opts: ExecuteEditOptions): Promise<any> {
 
 export function registerEditTool(pi: ExtensionAPI, options: EditToolOptions = {}) {
 	const tool = registerReadSeekTool(pi, {
-		policy: "mutating",
-		pythonName: "edit",
-		defaultExposure: "not-safe-by-default",
-	}, {
-		name: "edit",
+		name: "readSeek_edit",
 		label: "Edit",
 		description: EDIT_PROMPT_METADATA.description,
 		promptSnippet: EDIT_PROMPT_METADATA.promptSnippet,
@@ -617,7 +611,7 @@ export function registerEditTool(pi: ExtensionAPI, options: EditToolOptions = {}
 				previewKey,
 				() => buildPendingEditPreviewData(args ?? {}, context.cwd ?? process.cwd()),
 			);
-			const expanded = !!context.expanded || resolveEditDiffDisplay() === "expanded";
+			const expanded = !!context.expanded;
 			const preview2 = pendingPreviewLines(text, preview, expanded);
 			if (preview2.diffData) {
 				return upsertDiffComponent(context.lastComponent, { prefixLines: preview2.lines, diffData: preview2.diffData, theme, expanded: true });
@@ -625,7 +619,7 @@ export function registerEditTool(pi: ExtensionAPI, options: EditToolOptions = {}
 			return upsertTextComponent(context.lastComponent, clampLinesToWidth(preview2.lines, context.width).join("\n"));
 		},
 			renderResult(result: any, options: ToolRenderResultOptions, theme: any, ...rest: any[]) {
-			const { isPartial, isError, expanded: baseExpanded, width, context } = resolveRenderResultContext(options, rest);
+			const { isPartial, isError, expanded, width, context } = resolveRenderResultContext(options, rest);
 
 			if (isPartial) {
 				return renderPendingResult("pending edit", width);
@@ -638,13 +632,13 @@ export function registerEditTool(pi: ExtensionAPI, options: EditToolOptions = {}
 				.join("\n") ?? "";
 			const details = result.details ?? {};
 			const diff: string = details.diff ?? "";
-			const readseekValue = details.readseekValue as {
+			const readSeekValue = details.readSeekValue as {
 				warnings?: string[];
 				noopEdits?: unknown[];
 			} | undefined;
-			const warnings = readseekValue?.warnings ?? [];
-			const noopEdits = readseekValue?.noopEdits ?? [];
-			const semanticClassification = (readseekValue as any)?.semanticSummary?.classification as string | undefined;
+			const warnings = readSeekValue?.warnings ?? [];
+			const noopEdits = readSeekValue?.noopEdits ?? [];
+			const semanticClassification = (readSeekValue as any)?.semanticSummary?.classification as string | undefined;
 
 			const info = formatEditResultText({
 				isError: isError || !!result.isError,
@@ -655,7 +649,6 @@ export function registerEditTool(pi: ExtensionAPI, options: EditToolOptions = {}
 				semanticClassification: semanticClassification as any,
 			});
 
-			const expanded = baseExpanded || resolveEditDiffDisplay() === "expanded";
 			const diffData = (details as any).diffData;
 			const stats = diffData?.stats ?? { added: 0, removed: 0 };
 			let text = "";

package/src/file-map.ts

@@ -1,7 +1,7 @@
 import { stat } from "node:fs/promises";
 
 import type { FileMap } from "./readseek/types.js";
-import { readseekMap } from "./readseek-client.js";
+import { readSeekMap } from "./readseek-client.js";
 
 /**
  * Fetch a structural file map from readseek, which maintains its own on-disk
@@ -10,7 +10,7 @@ import { readseekMap } from "./readseek-client.js";
 export async function getOrGenerateMap(absPath: string): Promise<FileMap | null> {
 	try {
 		const { size } = await stat(absPath);
-		return await readseekMap(absPath, size);
+		return await readSeekMap(absPath, size);
 	} catch {
 		return null;
 	}

package/src/grep-budget.ts

@@ -1,29 +1,6 @@
 import { DEFAULT_MAX_BYTES, DEFAULT_MAX_LINES } from "@earendil-works/pi-coding-agent";
 import { resolveReadSeekJsonSettings } from "./readseek-settings.js";
 
-const POSITIVE_BASE10_INT = /^[1-9][0-9]*$/;
-
-/**
- * Strict positive base-10 integer parser used by readseek env knobs.
- *
- * Accepts: trimmed strings matching /^[1-9][0-9]*$/ that parse to a finite
- * positive integer.
- *
- * Rejects: undefined, empty, whitespace-only, "0", negative, signed, hex
- * ("0x10"), exponent ("1e3"), decimal ("3.14"), separators ("1,000" /
- * "1_000"), embedded whitespace ("5 5").
- *
- * Returns `undefined` on rejection; never throws.
- */
-function parsePositiveBase10Int(raw: string | undefined | null): number | undefined {
-	if (raw === undefined || raw === null) return undefined;
-	const trimmed = String(raw).trim();
-	if (!POSITIVE_BASE10_INT.test(trimmed)) return undefined;
-	const parsed = Number.parseInt(trimmed, 10);
-	if (!Number.isFinite(parsed) || parsed <= 0) return undefined;
-	return parsed;
-}
-
 export interface GrepOutputBudget {
 	maxLines: number;
 	maxBytes: number;
@@ -31,7 +8,7 @@ export interface GrepOutputBudget {
 
 /**
  * Effective grep-output ceilings used as clamp upper bounds and as the
- * fallback defaults when env vars are unset/invalid.
+ * fallback defaults when the settings are unset.
  *
  * The bytes ceiling is the already-tightened 50 KiB used by `buildGrepOutput`
  * today, NOT the unclamped `DEFAULT_MAX_BYTES`.
@@ -39,41 +16,20 @@ export interface GrepOutputBudget {
 export const GREP_OUTPUT_DEFAULT_MAX_LINES = DEFAULT_MAX_LINES;
 export const GREP_OUTPUT_DEFAULT_MAX_BYTES = Math.min(DEFAULT_MAX_BYTES, 50 * 1024);
 
-function resolveEnvDimension(rawEnvValue: string | undefined, ceiling: number): number | undefined {
-	const parsed = parsePositiveBase10Int(rawEnvValue);
-	return parsed === undefined ? undefined : Math.min(parsed, ceiling);
-}
-
-function resolveDimension(rawEnvValue: string | undefined, jsonValue: number | undefined, ceiling: number): number {
-	if (rawEnvValue !== undefined) {
-		const envValue = resolveEnvDimension(rawEnvValue, ceiling);
-		if (envValue !== undefined) return envValue;
-	}
+function resolveDimension(jsonValue: number | undefined, ceiling: number): number {
 	if (jsonValue !== undefined) return Math.min(jsonValue, ceiling);
 	return ceiling;
 }
 
 /**
- * Resolve the effective grep visible-output budget. Re-reads `process.env`
- * on every call (no memoization) so tests and long-lived agent sessions can
- * change the env vars dynamically.
- *
- * Invalid / zero / negative env values fall back to the current defaults.
- * Above-default values clamp to the current defaults. Below-default values
- * are used as-is.
+ * Resolve the effective grep visible-output budget from the readseek
+ * settings. Below-default values are used as-is; above-default values clamp
+ * to the current defaults.
  */
 export function resolveGrepOutputBudget(): GrepOutputBudget {
 	const settings = resolveReadSeekJsonSettings().settings.grep;
 	return {
-		maxLines: resolveDimension(
-			process.env.READSEEK_GREP_MAX_LINES,
-			settings?.maxLines,
-			GREP_OUTPUT_DEFAULT_MAX_LINES,
-		),
-		maxBytes: resolveDimension(
-			process.env.READSEEK_GREP_MAX_BYTES,
-			settings?.maxBytes,
-			GREP_OUTPUT_DEFAULT_MAX_BYTES,
-		),
+		maxLines: resolveDimension(settings?.maxLines, GREP_OUTPUT_DEFAULT_MAX_LINES),
+		maxBytes: resolveDimension(settings?.maxBytes, GREP_OUTPUT_DEFAULT_MAX_BYTES),
 	};
 }

package/src/grep-output.ts

@@ -68,7 +68,7 @@ interface BuildGrepOutputInput {
 
 interface GrepOutputResult {
   text: string;
-  readseekValue: {
+  readSeekValue: {
     tool: "grep";
     summary: boolean;
     totalMatches: number;
@@ -159,7 +159,7 @@ export function buildGrepOutput(input: BuildGrepOutputInput): GrepOutputResult {
   if (truncated.truncated) {
     text = `${truncated.content}\n\n[Output truncated: showing ${truncated.outputLines} of ${truncated.totalLines} lines (${formatSize(truncated.outputBytes)} of ${formatSize(truncated.totalBytes)}). Refine pattern or increase limit.]`;
   }
-  const readseekValue: GrepOutputResult["readseekValue"] = {
+  const readSeekValue: GrepOutputResult["readSeekValue"] = {
     tool: "grep",
     summary: input.summary,
     totalMatches: input.totalMatches,
@@ -171,11 +171,11 @@ export function buildGrepOutput(input: BuildGrepOutputInput): GrepOutputResult {
     })),
   };
   if (!input.summary && input.scopeMode === "symbol") {
-    readseekValue.scopes = buildScopeMetadata(input.groups, scopeWarnings);
+    readSeekValue.scopes = buildScopeMetadata(input.groups, scopeWarnings);
   }
 
   return {
     text,
-    readseekValue,
+    readSeekValue,
   };
 }

package/src/grep.ts

@@ -168,7 +168,6 @@ function escapeForRegex(s: string): string {
 }
 
 interface GrepToolOptions {
-	searchGuideline?: string;
 	onFileAnchored?: FileAnchoredCallback;
 }
 
@@ -253,7 +252,7 @@ export async function executeGrep(opts: ExecuteGrepOptions): Promise<any> {
 					),
 					details: {
 						...(typeof result.details === "object" && result.details !== null ? result.details : {}),
-						readseekValue: {
+						readSeekValue: {
 							tool: "grep",
 							summary: !!p.summary,
 							totalMatches: 0,
@@ -410,7 +409,7 @@ export async function executeGrep(opts: ExecuteGrepOptions): Promise<any> {
 			...result,
 			details: {
 				...passthroughDetails,
-				readseekValue: {
+				readSeekValue: {
 					tool: "grep",
 					summary: true,
 					totalMatches: 0,
@@ -436,7 +435,7 @@ export async function executeGrep(opts: ExecuteGrepOptions): Promise<any> {
 				...passthroughDetails,
 				hashlinePassthrough: true,
 				hashlineWarning: warning,
-				readseekValue: {
+				readSeekValue: {
 					tool: "grep",
 					summary: !!p.summary,
 					totalMatches: 0,
@@ -477,20 +476,20 @@ export async function executeGrep(opts: ExecuteGrepOptions): Promise<any> {
 		renderedGroups = scoped.groups;
 		scopeWarnings = scoped.warnings;
 	}
-	const readseekRecords = recordsFromGroups(renderedGroups);
+	const readSeekRecords = recordsFromGroups(renderedGroups);
 	const builtOutput = buildGrepOutput({
 		summary: !!summary,
 		totalMatches,
 		groups: renderedGroups,
 		limit: effectiveLimit,
-		records: readseekRecords,
+		records: readSeekRecords,
 		scopeMode: p.scope === "symbol" && !summary ? "symbol" : undefined,
 		scopeWarnings,
 		passthroughLines,
 	});
 
-	if (!summary && readseekRecords.length > 0) {
-		const anchoredPaths = new Set(readseekRecords.map((record) => record.path));
+	if (!summary && readSeekRecords.length > 0) {
+		const anchoredPaths = new Set(readSeekRecords.map((record) => record.path));
 		for (const absolutePath of anchoredPaths) {
 			onFileAnchored?.(absolutePath);
 		}
@@ -508,25 +507,19 @@ export async function executeGrep(opts: ExecuteGrepOptions): Promise<any> {
 		),
 		details: {
 			...compactDetails,
-			readseekValue: builtOutput.readseekValue,
+			readSeekValue: builtOutput.readSeekValue,
 		},
 	};
 }
 
 export function registerGrepTool(pi: ExtensionAPI, options: GrepToolOptions = {}) {
 	const tool = registerReadSeekTool(pi, {
-		policy: "read-only",
-		pythonName: "grep",
-		defaultExposure: "safe-by-default",
-	}, {
-		name: "grep",
+		name: "readSeek_grep",
 		label: "grep",
 		description: GREP_PROMPT_METADATA.description,
 		parameters: grepSchema,
 		promptSnippet: GREP_PROMPT_METADATA.promptSnippet,
-		promptGuidelines: options.searchGuideline
-			? [GREP_PROMPT_METADATA.promptGuidelines[0], options.searchGuideline]
-			: GREP_PROMPT_METADATA.promptGuidelines,
+		promptGuidelines: GREP_PROMPT_METADATA.promptGuidelines,
 		async execute(toolCallId, params, signal, onUpdate, ctx) {
 			return executeGrep({
 				toolCallId,
@@ -558,14 +551,14 @@ export function registerGrepTool(pi: ExtensionAPI, options: GrepToolOptions = {}
 		renderResult(result: any, options: ToolRenderResultOptions, theme: any, ...rest: any[]) {
 			const { isPartial, isError, expanded, cwd, width } = resolveRenderResultContext(options, rest);
 
-			if (isPartial) return renderPendingResult("pending search", width);
+			if (isPartial) return renderPendingResult("pending grep", width);
 
 			const content = result.content?.[0];
 			const textContent = content?.type === "text" ? content.text : "";
 
 			if (isError || result.isError) return renderErrorResult(textContent, { expanded, width });
 
-			const readseekValue = (result.details as any)?.readseekValue as {
+			const readSeekValue = (result.details as any)?.readSeekValue as {
 				tool: "grep";
 				summary: boolean;
 				totalMatches: number;
@@ -575,26 +568,26 @@ export function registerGrepTool(pi: ExtensionAPI, options: GrepToolOptions = {}
 			const hasBinaryWarning = textContent.includes("appears to be a binary file");
 
 			const fileSet = new Set<string>();
-			for (const r of readseekValue?.records ?? []) {
+			for (const r of readSeekValue?.records ?? []) {
 				if (r.path) fileSet.add(r.path);
 			}
 
 			const info = formatGrepResultText({
-				totalMatches: readseekValue?.totalMatches ?? 0,
-				summary: readseekValue?.summary ?? false,
-				records: readseekValue?.records ?? [],
+				totalMatches: readSeekValue?.totalMatches ?? 0,
+				summary: readSeekValue?.summary ?? false,
+				records: readSeekValue?.records ?? [],
 				fileCount: fileSet.size,
 				hasBinaryWarning,
 			});
 
 			if (info.noMatches && !hasBinaryWarning) return new Text(summaryLine("no matches"), 0, 0);
-			const matchCount = readseekValue?.totalMatches ?? 0;
+			const matchCount = readSeekValue?.totalMatches ?? 0;
 			const matchWord = matchCount === 1 ? "match" : "matches";
 			let text = summaryLine(`${matchCount} ${matchWord} returned`, { hidden: !!textContent && !expanded });
 			for (const badge of info.badges) text += theme.fg(badge.startsWith("⚠") ? "warning" : "dim", `  ${badge}`);
-			if (expanded && readseekValue?.records) {
+			if (expanded && readSeekValue?.records) {
 				const fileCounts = new Map<string, number>();
-				for (const r of readseekValue.records) if (r.path && r.kind === "match") fileCounts.set(r.path, (fileCounts.get(r.path) ?? 0) + 1);
+				for (const r of readSeekValue.records) if (r.path && r.kind === "match") fileCounts.set(r.path, (fileCounts.get(r.path) ?? 0) + 1);
 				for (const [filePath, count] of [...fileCounts.entries()].sort((a, b) => b[1] - a[1]).slice(0, 20)) {
 					const display = path.relative(cwd, filePath) || filePath;
 					text += "\n" + theme.fg("dim", `  ${display} (${count})`);

package/src/hover.ts

@@ -8,7 +8,7 @@ import { defineToolPromptMetadata } from "./tool-prompt-metadata.js";
 import { buildToolErrorResult } from "./readseek-value.js";
 import { resolveToCwd } from "./path-utils.js";
 import { formatFsError } from "./fs-error.js";
-import { classifyReadSeekFailure, readseekIdentify } from "./readseek-client.js";
+import { classifyReadSeekFailure, readSeekIdentify } from "./readseek-client.js";
 import { filePathParam, registerReadSeekTool } from "./register-tool.js";
 
 import { clampLinesToWidth, linkToolPath, renderPendingResult, resolveRenderResultContext, summaryLine } from "./tui-render-utils.js";
@@ -55,7 +55,7 @@ export async function executeHover(opts: ExecuteHoverOptions): Promise<any> {
 	}
 
 	try {
-		const output = await readseekIdentify(filePath, content, {
+		const output = await readSeekIdentify(filePath, content, {
 			line: p.line,
 			column: p.column,
 			signal,
@@ -77,7 +77,7 @@ export async function executeHover(opts: ExecuteHoverOptions): Promise<any> {
 		return {
 			content: [{ type: "text", text: lines.join("\n") }],
 			details: {
-				readseekValue: {
+				readSeekValue: {
 					tool: "hover",
 					ok: true,
 					path: filePath,
@@ -93,11 +93,7 @@ export async function executeHover(opts: ExecuteHoverOptions): Promise<any> {
 
 export function registerHoverTool(pi: ExtensionAPI) {
 	registerReadSeekTool(pi, {
-		policy: "read-only",
-		pythonName: "hover",
-		defaultExposure: "opt-in",
-	}, {
-		name: "hover",
+		name: "readSeek_hover",
 		label: "Hover",
 		description: HOVER_PROMPT_METADATA.description,
 		promptSnippet: HOVER_PROMPT_METADATA.promptSnippet,

package/src/read-output.ts

@@ -63,7 +63,7 @@ interface ReadOutputInput {
 interface ReadOutputResult {
   text: string;
   lines: ReadSeekLine[];
-  readseekValue: {
+  readSeekValue: {
     tool: "read";
     path: string;
     range: {
@@ -141,7 +141,7 @@ export function buildReadOutput(input: ReadOutputInput): ReadOutputResult {
     text = `${warnings.map((warning) => warning.message).join("\n\n")}\n\n${text}`;
   }
 
-  const readseekValue: ReadOutputResult["readseekValue"] = {
+  const readSeekValue: ReadOutputResult["readSeekValue"] = {
     tool: "read",
     path: input.path,
     range: {
@@ -160,7 +160,7 @@ export function buildReadOutput(input: ReadOutputInput): ReadOutputResult {
   };
 
   if (input.bundle) {
-    readseekValue.bundle = {
+    readSeekValue.bundle = {
       mode: input.bundle.mode,
       applied: input.bundle.applied,
       localSupport: input.bundle.localSupport.map((item) => {
@@ -181,6 +181,6 @@ export function buildReadOutput(input: ReadOutputInput): ReadOutputResult {
   return {
     text,
     lines,
-    readseekValue,
+    readSeekValue,
   };
 }

package/src/read.ts

@@ -26,7 +26,7 @@ import { buildReadOutput } from "./read-output.js";
 
 import { buildLocalBundle } from "./read-local-bundle.js";
 import { coerceObviousBase10Int } from "./coerce-obvious-int.js";
-import { readseekRead, readseekDetect, type ReadSeekDetection } from "./readseek-client.js";
+import { readSeekRead, readSeekDetect, readSeekImage, type ReadSeekDetection } from "./readseek-client.js";
 import { formatReadCallText, formatReadResultText } from "./read-render-helpers.js";
 import { resolveReadSeekOcrMode } from "./readseek-settings.js";
 import { clampLineToWidth, clampLinesToWidth, linkToolPath, renderPendingResult, renderToolLabel, resolveRenderResultContext, summaryLine, wrapReadHashlinesForWidthCached } from "./tui-render-utils.js";
@@ -65,9 +65,9 @@ export interface ExecuteReadOptions {
 function hasReadAnchors(result: AgentToolResult<any>): boolean {
 	const details = (result as { details?: unknown }).details;
 	if (!details || typeof details !== "object") return false;
-	const readseekValue = (details as { readseekValue?: unknown }).readseekValue;
-	if (!readseekValue || typeof readseekValue !== "object") return false;
-	const lines = (readseekValue as { lines?: unknown }).lines;
+	const readSeekValue = (details as { readSeekValue?: unknown }).readSeekValue;
+	if (!readSeekValue || typeof readSeekValue !== "object") return false;
+	const lines = (readSeekValue as { lines?: unknown }).lines;
 	return Array.isArray(lines) && lines.length > 0;
 }
 
@@ -171,24 +171,19 @@ export async function executeRead(opts: ExecuteReadOptions): Promise<AgentToolRe
 	if (hasBinaryContent) {
 		let detection: ReadSeekDetection | undefined;
 		try {
-			detection = await readseekDetect(absolutePath, { signal });
+			detection = await readSeekDetect(absolutePath, { signal });
 		} catch {
 		}
 		if (detection?.kind === "image") {
 			const builtinRead = createReadTool(cwd);
 			const builtinResult = await builtinRead.execute(toolCallId, p, signal, onUpdate);
 			const ocrMode = resolveReadSeekOcrMode();
-			const shouldRunVision = ocrMode === "on" || (ocrMode === "auto" && !opts.modelSupportsImages);
+			const shouldRunVision = ocrMode === "force" || (ocrMode === "auto" && !opts.modelSupportsImages);
 			if (!shouldRunVision) return succeed(builtinResult);
 
 			let ocrFailed = false;
 			try {
-				const ocrDetection = await readseekDetect(absolutePath, {
-					ocr: true,
-					caption: true,
-					objects: true,
-					signal,
-				});
+				const ocrDetection = await readSeekImage(absolutePath, ["ocr", "caption", "objects"], { signal });
 				const imageAnalysis = formatImageAnalysis(ocrDetection);
 				if (imageAnalysis) {
 					return succeed({
@@ -356,33 +351,33 @@ export async function executeRead(opts: ExecuteReadOptions): Promise<AgentToolRe
 	}
 
 	throwIfAborted(signal);
-	let readseekOutput: Awaited<ReturnType<typeof readseekRead>>;
+	let readSeekOutput: Awaited<ReturnType<typeof readSeekRead>>;
 	try {
-		readseekOutput = total === 0
-			? await readseekRead(absolutePath, undefined, undefined, { signal })
-			: await readseekRead(absolutePath, startLine, endIdx, { signal });
+		readSeekOutput = total === 0
+			? await readSeekRead(absolutePath, undefined, undefined, { signal })
+			: await readSeekRead(absolutePath, startLine, endIdx, { signal });
 	} catch (err: any) {
 		const detail = err?.message ? ` — ${err.message}` : "";
 		const message = `readseek failed while reading ${rawPath}${detail}`;
 		return buildToolErrorResult("read", "readseek-error", message, { path: rawParams.path, hint: "Ensure @jarkkojs/readseek and its npm platform package are installed.", details: { message: err?.message } });
 	}
 	const expectedLineCount = Math.max(0, endIdx - startLine + 1);
-	const invalidLine = readseekOutput.hashlines.find((line, index) => line.line !== startLine + index);
-	if (readseekOutput.hashlines.length !== expectedLineCount || invalidLine) {
+	const invalidLine = readSeekOutput.hashlines.find((line, index) => line.line !== startLine + index);
+	if (readSeekOutput.hashlines.length !== expectedLineCount || invalidLine) {
 		const message = invalidLine
 			? `readseek returned non-sequential line ${invalidLine.line} for requested range ${startLine}-${endIdx}`
-			: `readseek returned ${readseekOutput.hashlines.length} lines for requested range ${startLine}-${endIdx} (${expectedLineCount} expected)`;
+			: `readseek returned ${readSeekOutput.hashlines.length} lines for requested range ${startLine}-${endIdx} (${expectedLineCount} expected)`;
 		return buildToolErrorResult("read", "readseek-output-mismatch", message, { path: rawParams.path });
 	}
-	const readseekLines: ReadSeekLine[] = readseekOutput.hashlines.map((line) => ({
+	const readSeekLines: ReadSeekLine[] = readSeekOutput.hashlines.map((line) => ({
 		line: line.line,
 		hash: line.hash,
 		anchor: `${line.line}:${line.hash}`,
 		raw: line.text,
 		display: escapeControlCharsForDisplay(line.text),
 	}));
-	const selected = readseekLines.map((line) => line.raw);
-	const formatted = renderReadSeekLines(readseekLines);
+	const selected = readSeekLines.map((line) => line.raw);
+	const formatted = renderReadSeekLines(readSeekLines);
 
 	const truncation = truncateHead(formatted, { maxLines: DEFAULT_MAX_LINES, maxBytes: DEFAULT_MAX_BYTES });
 
@@ -423,7 +418,7 @@ export async function executeRead(opts: ExecuteReadOptions): Promise<AgentToolRe
 		endLine: endIdx,
 		totalLines: total,
 		selectedLines: selected,
-		lines: readseekLines,
+		lines: readSeekLines,
 		warnings: structuredWarnings,
 		truncation: truncation.truncated
 			? {
@@ -456,7 +451,7 @@ export async function executeRead(opts: ExecuteReadOptions): Promise<AgentToolRe
 		content: [{ type: "text", text: readOutput.text }],
 		details: {
 			truncation: truncation.truncated ? truncation : undefined,
-			readseekValue: readOutput.readseekValue,
+			readSeekValue: readOutput.readSeekValue,
 		},
 	});
 }
@@ -469,11 +464,7 @@ function splitReadSeekLines(text: string): string[] {
 
 export function registerReadTool(pi: ExtensionAPI, options: ReadToolOptions = {}) {
 	const tool = registerReadSeekTool(pi, {
-		policy: "read-only",
-		pythonName: "read",
-		defaultExposure: "safe-by-default",
-	}, {
-		name: "read",
+		name: "readSeek_read",
 		label: "Read",
 		description: READ_PROMPT_METADATA.description,
 		promptSnippet: READ_PROMPT_METADATA.promptSnippet,
@@ -529,16 +520,16 @@ export function registerReadTool(pi: ExtensionAPI, options: ReadToolOptions = {}
 				return new Text(clampLinesToWidth([summaryLine(errorText)], width).join("\n"), 0, 0);
 			}
 
-			const readseekValue = (result.details as any)?.readseekValue as { range: { startLine: number; endLine: number; totalLines: number }; truncation: any; symbol: any; map: any; warnings: ReadSeekWarning[] } | undefined;
-			if (!readseekValue) {
+			const readSeekValue = (result.details as any)?.readSeekValue as { range: { startLine: number; endLine: number; totalLines: number }; truncation: any; symbol: any; map: any; warnings: ReadSeekWarning[] } | undefined;
+			if (!readSeekValue) {
 				const lines = textContent.split("\n").filter(Boolean).length || textContent.split("\n").length;
 				return new Text(summaryLine(`loaded ${lines} ${lines === 1 ? "line" : "lines"}`, { hidden: !!textContent && !expanded }), 0, 0);
 			}
 
-			const info = formatReadResultText({ range: readseekValue.range, truncation: readseekValue.truncation, symbol: readseekValue.symbol, map: readseekValue.map, warnings: readseekValue.warnings });
-			const visibleLines = info.truncated && readseekValue.truncation ? readseekValue.truncation.outputLines : readseekValue.range.endLine - readseekValue.range.startLine + 1;
+			const info = formatReadResultText({ range: readSeekValue.range, truncation: readSeekValue.truncation, symbol: readSeekValue.symbol, map: readSeekValue.map, warnings: readSeekValue.warnings });
+			const visibleLines = info.truncated && readSeekValue.truncation ? readSeekValue.truncation.outputLines : readSeekValue.range.endLine - readSeekValue.range.startLine + 1;
 			const loadedWord = visibleLines === 1 ? "line" : "lines";
-			const summaryParts: string[] = [info.truncated ? `loaded ${visibleLines} of ${readseekValue.truncation?.totalLines ?? readseekValue.range.totalLines} ${loadedWord} (truncated)` : `loaded ${visibleLines} ${loadedWord}`];
+			const summaryParts: string[] = [info.truncated ? `loaded ${visibleLines} of ${readSeekValue.truncation?.totalLines ?? readSeekValue.range.totalLines} ${loadedWord} (truncated)` : `loaded ${visibleLines} ${loadedWord}`];
 			if (info.symbolBadge) summaryParts.push(info.symbolBadge);
 			for (const badge of info.badges) summaryParts.push(badge);
 			const summary = summaryParts.join(" • ");