pi-readseek 0.6.6 → 0.6.8

package/README.md

@@ -64,7 +64,7 @@ optional (defaults shown):
 - **replacedTools:** built-in tool names to replace with their `readSeek_*` equivalents.
   Valid values are `"read"`, `"edit"`, `"write"`, and `"grep"`. For a
   readseek-only file surface, use `["read", "edit", "write", "grep"]`.
-- **imageMode:** `"auto"` exposes `none`, `ocr`, `caption`, and `objects`;
+- **imageMode:** `"auto"` exposes `none`, `all`, `ocr`, `caption`, and `objects`;
   `"on"` omits `none`; `"off"` removes the `image` parameter and always skips
   image/PDF files. Omitting `image` also skips visual files.
 - **syntaxValidation:** pre-write syntax-regression check in `readSeek_edit`:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-readseek",
-  "version": "0.6.6",
+  "version": "0.6.8",
   "description": "Pi extension for readseek-backed hash-anchored read/edit/grep, structural code maps, structural search, and file exploration",
   "type": "module",
   "exports": {
@@ -39,7 +39,7 @@
     "node": ">=20.0.0"
   },
   "dependencies": {
-    "@jarkkojs/readseek": "^0.6.6",
+    "@jarkkojs/readseek": "^0.6.8",
     "diff": "^8.0.3",
     "xxhash-wasm": "^1.1.0"
   },

package/src/edit-output.ts

@@ -76,9 +76,6 @@ function formatWhitespaceOnlyWarning(semanticSummary: SemanticSummary | undefine
   if (!extractNewTextValues(edits).some((text) => /\S/.test(text))) return undefined;
   return "⚠ Edit classified as whitespace-only — if you intended a behavior change, re-read to verify.";
 }
-function formatSemanticSuffix(_semanticSummary: SemanticSummary | undefined): string {
-  return "";
-}
 function formatReplaceHint(edits: unknown[] | undefined, noopEdits: unknown[]): string | undefined {
   if ((noopEdits ?? []).length > 0) return undefined;
   const counts = countEditTypes(edits);
@@ -88,7 +85,7 @@ function formatReplaceHint(edits: unknown[] | undefined, noopEdits: unknown[]):
 }
 export function buildEditOutput(input: BuildEditOutputInput): EditOutputResult {
   const summary = `Updated ${input.displayPath}`;
-  const visibleSummary = `${buildVisibleSummary(input.displayPath, input.diff, input.edits)}${formatSemanticSuffix(input.semanticSummary)}`;
+  const visibleSummary = buildVisibleSummary(input.displayPath, input.diff, input.edits);
   const semanticWarning = formatWhitespaceOnlyWarning(input.semanticSummary, input.edits);
   const warningText = input.warnings.length ? `\n\nWarnings:\n${input.warnings.join("\n")}` : "";
   const replaceHint = formatReplaceHint(input.edits, input.noopEdits);

package/src/read.ts

@@ -66,7 +66,7 @@ function hasReadAnchors(result: AgentToolResult<any>): boolean {
 	return Array.isArray(lines) && lines.length > 0;
 }
 
-function formatImageAnalysis(detection: ReadSeekDetection, mode: "ocr" | "caption" | "objects"): string | undefined {
+function formatImageAnalysis(detection: ReadSeekDetection, mode: "all" | "ocr" | "caption" | "objects"): string | undefined {
 	if (detection.kind !== "image") return undefined;
 	const sections: string[] = [];
 	const ocr = detection.ocr?.trim();
@@ -80,6 +80,7 @@ function formatImageAnalysis(detection: ReadSeekDetection, mode: "ocr" | "captio
 	if (sections.length > 0) return sections.join("\n\n");
 	if (mode === "ocr") return "No OCR text detected in image.";
 	if (mode === "objects") return "No objects detected in image.";
+	if (mode === "all") return "No image analysis returned.";
 	return "Image caption returned no text.";
 }
 
@@ -524,7 +525,7 @@ function splitReadSeekLines(text: string): string[] {
 export function registerReadTool(pi: ExtensionAPI, options: ReadToolOptions = {}) {
 	const name = options.name ?? "readSeek_read";
 	const imageMode = resolveReadSeekImageMode();
-	const imageModes = imageMode === "auto" ? ["none", "ocr", "caption", "objects"] as const : ["ocr", "caption", "objects"] as const;
+	const imageModes = imageMode === "auto" ? ["none", "all", "ocr", "caption", "objects"] as const : ["all", "ocr", "caption", "objects"] as const;
 	const promptMetadata = defineToolPromptMetadata({
 		promptUrl: new URL("../prompts/read.md", import.meta.url),
 		promptSnippet: "Read anchored text, symbols, maps, images, or PDFs",

package/src/readseek-client.ts

@@ -116,7 +116,7 @@ export interface ReadSeekDetectedObject {
 	bbox: [number, number, number, number];
 }
 
-export type ReadSeekImageMode = "ocr" | "caption" | "objects";
+export type ReadSeekImageMode = "all" | "ocr" | "caption" | "objects";
 
 export interface ReadSeekPreparedImage {
 	mime: string;
@@ -851,8 +851,9 @@ export async function readSeekImage(
 	modes: ReadSeekImageMode[],
 	options: { signal?: AbortSignal } = {},
 ): Promise<ReadSeekDetection> {
+	const requestedModes = modes.length > 1 ? ["all"] : modes;
 	const results = await Promise.allSettled(
-		modes.map(async (mode) =>
+		requestedModes.map(async (mode) =>
 			parseDetectOutput(await runReadSeekVision(["read", "--image", mode, filePath], { signal: options.signal })),
 		),
 	);
@@ -892,7 +893,7 @@ function parsePdfImage(value: unknown): ReadSeekPdfImage {
 	if (!value || typeof value !== "object") throw new Error("invalid readseek PDF image");
 	const image = value as Record<string, unknown>;
 	const mode = requireString(image.mode, "PDF image.mode");
-	if (mode !== "none" && mode !== "ocr" && mode !== "caption" && mode !== "objects") {
+	if (mode !== "none" && mode !== "all" && mode !== "ocr" && mode !== "caption" && mode !== "objects") {
 		throw new Error("invalid readseek PDF image.mode");
 	}
 	return {

package/src/readseek-value.ts

@@ -76,12 +76,8 @@ export function buildReadSeekLines(startLine: number, rawLines: string[]): ReadS
   return rawLines.map((raw, index) => buildReadSeekLine(startLine + index, raw));
 }
 
-function renderReadSeekLine(line: ReadSeekLine): string {
-  return `${line.anchor}|${line.display}`;
-}
-
 export function renderReadSeekLines(lines: ReadSeekLine[]): string {
-  return lines.map(renderReadSeekLine).join("\n");
+  return lines.map((line) => `${line.anchor}|${line.display}`).join("\n");
 }
 
 /**

package/src/search.ts

@@ -7,7 +7,7 @@ import { defineToolPromptMetadata } from "./tool-prompt-metadata.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, readSeekSearch, type ReadSeekHashline, type ReadSeekSearchFileOutput } from "./readseek-client.js";
+import { classifyReadSeekFailure, readSeekSearch, type ReadSeekSearchFileOutput } from "./readseek-client.js";
 import { buildSearchOutput } from "./search-output.js";
 import type { FileAnchoredCallback } from "./tool-types.js";
 import { langParam, readSeekGitSearchParams, searchPathParam, validateIgnoredRequiresOthers } from "./readseek-params.js";
@@ -66,15 +66,11 @@ export interface ExecuteSearchOptions {
   onFileAnchored?: FileAnchoredCallback;
 }
 
-function readSeekLineFromSearch(line: ReadSeekHashline): ReadSeekLine {
-  return buildReadSeekLineWithHash(line.line, line.hash, line.text);
-}
-
 function linesFromSearchResult(result: ReadSeekSearchFileOutput, ranges: SearchRange[]): 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));
+      lineMap.set(line.line, buildReadSeekLineWithHash(line.line, line.hash, line.text));
     }
   }