pi-read-map 1.2.3 → 1.2.5

package/CHANGELOG.md

@@ -2,6 +2,19 @@
 
 All notable changes to this project will be documented in this file.
 
+## [1.2.5] - 2026-02-15
+
+### Fixed
+
+- `sendMessage` calls in `tool_result` handler now use `deliverAs: "followUp"` instead of the default `"steer"` mode. The default `"steer"` mode interrupts streaming and skips remaining parallel tools — when multiple reads ran concurrently and one triggered a file-map or directory-listing message, the remaining tool calls were skipped, leaving `tool_use` blocks without matching `tool_result` blocks. This caused a 400 error from the Claude API on the next request.
+
+## [1.2.4] - 2026-02-15
+
+### Fixed
+
+- Binary/image files (`.jpg`, `.png`, `.gif`, `.webp`, etc.) no longer enter the map generation pipeline — they are delegated directly to the built-in read tool. Previously, a >50 KB image would pass the size threshold, run `wc -l` on binary data, and trigger ctags/grep fallback. When multiple image reads ran in parallel, the resulting `sendMessage` calls broke the Claude API's `tool_use`/`tool_result` pairing requirement.
+- Fallback mapper now returns `null` when grep finds zero symbols instead of returning an empty `FileMap`. This prevents unnecessary `sendMessage` calls for unmappable files.
+
 ## [1.2.2] - 2026-02-14
 
 ### Changed

package/README.md

@@ -157,9 +157,10 @@ tests/
 
 The extension intercepts `read` calls and decides:
 
-1. **Small files** (≤2,000 lines, ≤50 KB): Delegate to built-in read tool
-2. **Targeted reads** (offset or limit provided): Delegate to built-in read tool
-3. **Large files:**
+1. **Binary files** (images, audio, video, archives, etc.): Delegate to built-in read tool
+2. **Small files** (≤2,000 lines, ≤50 KB): Delegate to built-in read tool
+3. **Targeted reads** (offset or limit provided): Delegate to built-in read tool
+4. **Large files:**
    - Call built-in read for the first chunk
    - Detect language from file extension
    - Dispatch to a mapper (language-specific → ctags → grep fallback)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-read-map",
-  "version": "1.2.3",
+  "version": "1.2.5",
   "description": "Pi extension that adds structural file maps for large files",
   "type": "module",
   "pi": {

package/src/index.ts

@@ -10,7 +10,7 @@ import { Text } from "@mariozechner/pi-tui";
 import { Type } from "@sinclair/typebox";
 import { exec } from "node:child_process";
 import { stat } from "node:fs/promises";
-import { basename, resolve } from "node:path";
+import { basename, extname, resolve } from "node:path";
 import { promisify } from "node:util";
 
 import type { FileMapMessageDetails } from "./types.js";
@@ -22,6 +22,60 @@ export type { FileMapMessageDetails } from "./types.js";
 
 const execAsync = promisify(exec);
 
+/**
+ * File extensions that are binary/image files and should be
+ * delegated directly to the built-in read tool without map generation.
+ */
+const BINARY_EXTENSIONS = new Set([
+  ".jpg",
+  ".jpeg",
+  ".png",
+  ".gif",
+  ".webp",
+  ".bmp",
+  ".ico",
+  ".tiff",
+  ".tif",
+  ".svg",
+  ".avif",
+  ".heic",
+  ".heif",
+  // Audio/video
+  ".mp3",
+  ".mp4",
+  ".wav",
+  ".avi",
+  ".mov",
+  ".mkv",
+  ".flac",
+  ".ogg",
+  ".webm",
+  // Archives
+  ".zip",
+  ".tar",
+  ".gz",
+  ".bz2",
+  ".xz",
+  ".7z",
+  ".rar",
+  // Binary data
+  ".bin",
+  ".exe",
+  ".dll",
+  ".so",
+  ".dylib",
+  ".o",
+  ".a",
+  ".wasm",
+  ".pdf",
+  ".doc",
+  ".docx",
+  ".xls",
+  ".xlsx",
+  ".ppt",
+  ".pptx",
+]);
+
 // In-memory cache for maps
 const mapCache = new Map<string, { mtime: number; map: string }>();
 
@@ -82,11 +136,14 @@ export default function piReadMapExtension(pi: ExtensionAPI): void {
     // Send pending directory listing after read-on-directory error
     const pendingLs = pendingDirectoryLs.get(event.toolCallId);
     if (pendingLs) {
-      pi.sendMessage({
-        customType: "directory-listing",
-        content: `${pendingLs.path} is a directory. Here is ls:\n${pendingLs.listing}`,
-        display: true,
-      });
+      pi.sendMessage(
+        {
+          customType: "directory-listing",
+          content: `${pendingLs.path} is a directory. Here is ls:\n${pendingLs.listing}`,
+          display: true,
+        },
+        { deliverAs: "followUp" }
+      );
       pendingDirectoryLs.delete(event.toolCallId);
     }
 
@@ -96,12 +153,15 @@ export default function piReadMapExtension(pi: ExtensionAPI): void {
     }
 
     // Send the map as a custom message
-    pi.sendMessage({
-      customType: "file-map",
-      content: pending.map,
-      display: true,
-      details: pending.details,
-    });
+    pi.sendMessage(
+      {
+        customType: "file-map",
+        content: pending.map,
+        display: true,
+        details: pending.details,
+      },
+      { deliverAs: "followUp" }
+    );
 
     // Clean up
     pendingMaps.delete(event.toolCallId);
@@ -257,6 +317,11 @@ export default function piReadMapExtension(pi: ExtensionAPI): void {
       // Resolve path
       const absPath = resolve(cwd, inputPath.replace(/^@/, ""));
 
+      // Skip binary/image files — delegate directly without map generation
+      if (BINARY_EXTENSIONS.has(extname(absPath).toLowerCase())) {
+        return builtInRead.execute(toolCallId, params, signal, onUpdate);
+      }
+
       // Check file size and line count
       let stats;
       try {

package/src/mappers/fallback.ts

@@ -141,6 +141,11 @@ export async function fallbackMapper(
       };
     });
 
+    // No symbols found — nothing useful to map
+    if (symbols.length === 0) {
+      return null;
+    }
+
     // Get language info
     const langInfo = detectLanguage(filePath);