pi-read-map 1.2.2 → 1.2.4

package/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 All notable changes to this project will be documented in this file.
 
+## [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.2",
+  "version": "1.2.4",
   "description": "Pi extension that adds structural file maps for large files",
   "type": "module",
   "pi": {

package/src/constants.ts

@@ -7,11 +7,11 @@ export const THRESHOLDS = {
   /** Maximum bytes before truncation */
   MAX_BYTES: 50 * 1024,
   /** Maximum map size in bytes */
-  MAX_MAP_BYTES: 20 * 1024,
+  MAX_MAP_BYTES: 25 * 1024,
   /** Target size for full detail */
   FULL_TARGET_BYTES: 10 * 1024,
   /** Target size for compact detail */
-  COMPACT_TARGET_BYTES: 15 * 1024,
+  COMPACT_TARGET_BYTES: 20 * 1024,
   /** Maximum size for outline level */
   MAX_OUTLINE_BYTES: 50 * 1024,
   /** Maximum size for truncated level (hard cap) */

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 }>();
 
@@ -257,6 +311,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);