pi-readseek 0.12.3 → 0.12.5

package/README.md

@@ -27,6 +27,10 @@ attachments without a separate extraction call. Reading never runs local inferen
 captioning, detection, and OCR require their explicit tools. Legacy tool names and
 selectable inspection facets are removed.
 
+Image attachments share a 32 MiB base64-data budget per tool result. Images that
+would exceed the remaining budget are omitted with a notice and their file path;
+smaller images later in the result can still be attached.
+
 The four `show` tools display readable summaries and grouped details rather than
 JSON dumps. Expand with Ctrl+O to reveal content; long lines wrap on resize.
 Source anchors, selected document nodes, asset paths, and continuation notices

package/dist/index.ts

@@ -5374,7 +5374,9 @@ function registerDefTool(pi, options = {}) {
 // src/access.ts
 import { Text as Text7, wrapTextWithAnsi as wrapTextWithAnsi2 } from "@earendil-works/pi-tui";
 import { Type as Type11 } from "@sinclair/typebox";
-import { readFile as readFile2 } from "node:fs/promises";
+import { createReadStream } from "node:fs";
+import { stat } from "node:fs/promises";
+var MAX_IMAGE_ATTACHMENT_BYTES = 32 * 1024 * 1024;
 var operations = {
   show: readSeekShow,
   tree: readSeekTree,
@@ -5386,22 +5388,25 @@ var operations = {
   objects: readSeekObjects,
   ocr: readSeekOcr
 };
-function resolveNonShowSummary(tool, envelope) {
-  if (tool === "syntax") {
-    const errorCount = envelope?.diagnostics?.error_count ?? 0;
+function resolveNonShowSummary(value) {
+  if (!value)
+    return "complete";
+  if (value.tool === "syntax") {
+    const errorCount = value.envelope.errorCount + value.envelope.missingCount;
     return errorCount === 0 ? "clean" : `${errorCount} syntax error${errorCount === 1 ? "" : "s"}`;
   }
-  if (tool === "check") {
-    const violations = Array.isArray(envelope?.violations) ? envelope.violations.length : 0;
+  if (value.tool === "check") {
+    const violations = value.envelope.violation_count;
     return violations === 0 ? "clean" : `${violations} rule violation${violations === 1 ? "" : "s"}`;
   }
-  if (tool === "objects") {
-    const objects = Array.isArray(envelope?.objects) ? envelope.objects.length : 0;
+  if (value.tool === "objects") {
+    const envelope = value.envelope;
+    const objects = "vision" in envelope ? envelope.vision.reduce((count, vision) => count + (vision.objects?.length ?? 0), 0) : ("objects" in envelope.content) ? envelope.content.objects?.length ?? 0 : 0;
     return `${objects} object${objects === 1 ? "" : "s"}`;
   }
-  if (tool === "caption")
+  if (value.tool === "caption")
     return "caption generated";
-  if (tool === "ocr")
+  if (value.tool === "ocr")
     return "text recognized";
   return "complete";
 }
@@ -5492,13 +5497,38 @@ async function executeAccess(opts) {
       { type: "text", text: presentation ? formatShowResult(presentation) : JSON.stringify(envelope) }
     ];
     if (opts.tool === "show" || opts.tool === "assets") {
+      let remainingImageBytes = MAX_IMAGE_ATTACHMENT_BYTES;
       for (const [imagePath, mimeType] of imagePaths(envelope)) {
+        opts.signal?.throwIfAborted();
         if (opts.imageMode === "off")
           content.push({ type: "text", text: `Image available at ${imagePath}; image attachments are disabled by readseek.imageMode.` });
-        else if (opts.modelSupportsImages)
-          content.push({ type: "image", data: (await readFile2(imagePath)).toString("base64"), mimeType });
-        else
+        else if (!opts.modelSupportsImages)
           content.push({ type: "text", text: `Image available at ${imagePath}; this host model does not support image attachments.` });
+        else {
+          const maxSourceBytes = Math.floor(remainingImageBytes / 4) * 3;
+          let data;
+          if (maxSourceBytes > 0 && (await stat(imagePath)).size <= maxSourceBytes) {
+            const chunks = [];
+            let sourceBytes = 0;
+            const stream = createReadStream(imagePath, { end: maxSourceBytes, signal: opts.signal });
+            for await (const chunk of stream) {
+              sourceBytes += chunk.length;
+              if (sourceBytes > maxSourceBytes)
+                break;
+              chunks.push(chunk);
+            }
+            opts.signal?.throwIfAborted();
+            if (sourceBytes <= maxSourceBytes)
+              data = Buffer.concat(chunks, sourceBytes).toString("base64");
+          }
+          opts.signal?.throwIfAborted();
+          if (data === undefined)
+            content.push({ type: "text", text: `Image available at ${imagePath}; attachment omitted because it would exceed the 32 MiB image attachment budget for this result.` });
+          else {
+            remainingImageBytes -= data.length;
+            content.push({ type: "image", data, mimeType });
+          }
+        }
       }
     }
     return { content, details: { readSeekValue: { tool: opts.tool, ok: true, path: filePath, envelope, presentation } } };
@@ -5566,7 +5596,7 @@ function registerAccessTools(pi, options = {}) {
           const summary = summaryLine(presentation.summary, { hidden: hasExpandable && !expanded, theme, style: hasWarning ? "warning" : "success" });
           return renderShowPresentation(summary, presentation.body, extras, expanded, theme, width);
         }
-        const nonShowSummary = resolveNonShowSummary(name, envelope);
+        const nonShowSummary = resolveNonShowSummary(result.details?.readSeekValue);
         const summary = summaryLine(nonShowSummary, { hidden: !!text && !expanded, theme, style: "success" });
         return new Text7(clampLinesToWidth((expanded ? `${summary}
 ${text}` : summary).split(`
@@ -5650,7 +5680,7 @@ function piReadSeekExtension(pi) {
     onFileAnchored: (absolutePath) => sessionAnchors.markAnchored(absolutePath),
     name: readName,
     toolAliases,
-    imageMode: replacedBuiltIns.has("read") ? resolveReadSeekImageMode() : undefined
+    imageMode: resolveReadSeekImageMode()
   });
   registerWriteTool(pi, {
     onFileAnchored: (absolutePath) => sessionAnchors.markAnchored(absolutePath),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-readseek",
-  "version": "0.12.3",
+  "version": "0.12.5",
   "description": "Pi extension for LINE:HASH-anchored file operations and structural code navigation",
   "type": "module",
   "exports": {
@@ -39,8 +39,8 @@
     "node": ">=20.0.0"
   },
   "dependencies": {
-    "@jarkkojs/readseek": "^0.12.3",
-    "@jarkkojs/readseek-api": "^0.12.3",
+    "@jarkkojs/readseek": "^0.12.5",
+    "@jarkkojs/readseek-api": "^0.12.5",
     "diff": "^9.0.0",
     "xxhash-wasm": "^1.1.0"
   },