pi-readseek 0.4.7 → 0.4.9

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-readseek",
-  "version": "0.4.7",
+  "version": "0.4.9",
   "description": "Pi extension for readseek-backed hash-anchored read/edit/grep, structural code maps, structural search, and file exploration",
   "type": "module",
   "exports": {

package/src/edit-diff.ts

@@ -14,6 +14,7 @@ export function detectLineEnding(content: string): "\r\n" | "\n" {
 			lf++;
 		}
 	}
+	if (lf === 0 && crlf === 0) return "\n";
 	if (lf === 0) return "\r\n";
 	if (crlf === 0) return "\n";
 	return crlf > lf ? "\r\n" : "\n";

package/src/edit.ts

@@ -277,6 +277,7 @@ export async function executeEdit(opts: ExecuteEditOptions): Promise<any> {
 	const replaceSymbolRanges: { start: number; end: number }[] = [];
 	const rsProbeResults: { type: "ok"; content: string; replacement: string; warnings: string[]; range: { start: number; end: number } }[] = [];
 	for (const rs of replaceSymbolEdits) {
+		throwIfAborted(signal);
 		const probe = await replaceSymbol({
 			filePath: absolutePath,
 			content: originalNormalized,

package/src/grep.ts

@@ -315,6 +315,24 @@ export async function executeGrep(opts: ExecuteGrepOptions): Promise<any> {
 	let parsedCount = 0;
 	let candidateUnparsedCount = 0;
 	const candidateLinePattern = /^.+(?::|-)\d+(?::|-)\s/;
+	// Pre-read all matched files with bounded concurrency so the processing
+	// loop below hits the cache on every getFileLines call instead of
+	// serialising disk I/O.
+	if (!p.summary) {
+		const pathsToRead = new Set<string>();
+		for (const line of textBlock.text.split("\n")) {
+			throwIfAborted(signal);
+			const parsed = parseGrepOutputLine(line);
+			if (parsed && Number.isFinite(parsed.lineNumber) && parsed.lineNumber >= 1) {
+				pathsToRead.add(toAbsolutePath(parsed.displayPath));
+			}
+		}
+		const CONCURRENCY = 8;
+		const pathList = [...pathsToRead];
+		for (let i = 0; i < pathList.length; i += CONCURRENCY) {
+			await Promise.all(pathList.slice(i, i + CONCURRENCY).map((p) => getFileLines(p)));
+		}
+	}
 
 	const addSummaryMatch = (displayPath: string, absolutePath: string) => {
 		let group = groupsByPath.get(displayPath);

package/src/map-cache.ts

@@ -29,11 +29,15 @@ function getMapCacheState(): MapCacheGlobalState {
 	globalObject[MAP_CACHE_STATE_KEY] = state;
 	return state;
 }
+/** Move an existing entry to the most-recently-used position (Map insertion-order tail). */
+function touchMapEntry<K, V>(map: Map<K, V>, key: K, value: V): void {
+	map.delete(key);
+	map.set(key, value);
+}
 
 function rememberInMemory(absPath: string, entry: CacheEntry): void {
 	const state = getMapCacheState();
-	if (state.cache.has(absPath)) state.cache.delete(absPath);
-	state.cache.set(absPath, entry);
+	touchMapEntry(state.cache, absPath, entry);
 	if (state.cache.size > state.maxSize) {
 		const oldestKey = state.cache.keys().next().value;
 		if (oldestKey !== undefined) state.cache.delete(oldestKey);
@@ -55,8 +59,7 @@ export async function getOrGenerateMap(absPath: string): Promise<FileMap | null>
 		const state = getMapCacheState();
 		const cached = state.cache.get(absPath);
 		if (cached && cached.mtimeMs === mtimeMs) {
-			state.cache.delete(absPath);
-			state.cache.set(absPath, cached);
+			touchMapEntry(state.cache, absPath, cached);
 			return cached.map;
 		}
 		const inflight = state.inflight.get(absPath);

package/src/read.ts

@@ -91,7 +91,7 @@ export async function executeRead(opts: ExecuteReadOptions): Promise<AgentToolRe
 	const rawBundle = typeof rawParams.bundle === "string" ? rawParams.bundle.trim() : undefined;
 	const requestedMapViaBundle =
 		rawBundle === "map" ||
-		(rawBundle === "local" && rawParams.symbol === undefined && rawParams.map !== false);
+		(rawBundle === "local" && rawParams.symbol === undefined && (rawParams.map ?? true));
 	const p = {
 		...rawParams,
 		offset: offset.value,
@@ -174,7 +174,8 @@ export async function executeRead(opts: ExecuteReadOptions): Promise<AgentToolRe
 	}
 	const hasBinaryContent = looksLikeBinary(rawBuffer);
 	throwIfAborted(signal);
-	const normalized = normalizeToLF(stripBom(rawBuffer.toString("utf-8")).text);
+	const rawText = rawBuffer.toString("utf-8");
+	const normalized = normalizeToLF(stripBom(rawText).text);
 	const allLines = splitReadseekLines(normalized);
 	const total = allLines.length;
 	const structuredWarnings: ReadseekWarning[] = [];
@@ -370,7 +371,7 @@ export async function executeRead(opts: ExecuteReadOptions): Promise<AgentToolRe
 		structuredWarnings.push(buildReadseekWarning("binary-content", warning));
 	}
 
-	if (hasBareCarriageReturn(rawBuffer.toString("utf-8"))) {
+	if (hasBareCarriageReturn(rawText)) {
 		const warning = "[Warning: file contains bare CR (\\r) line endings — line numbering may be inconsistent with grep and other tools]";
 		structuredWarnings.push(buildReadseekWarning("bare-cr", warning));
 	}

package/src/write.ts

@@ -1,4 +1,5 @@
-import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
+import { existsSync, mkdirSync, writeFileSync } from "node:fs";
+import { readFile } from "node:fs/promises";
 import { dirname, relative } from "node:path";
 
 import { withFileMutationQueue, type ExtensionAPI } from "@earendil-works/pi-coding-agent";
@@ -92,10 +93,11 @@ export interface WriteResult extends WriteDiffFields {
   };
 }
 
-function readPreviousTextForDiff(filePath: string): string {
+
+async function readPreviousTextForDiff(filePath: string): Promise<string> {
   try {
     if (!existsSync(filePath)) return "";
-    const previous = readFileSync(filePath);
+    const previous = await readFile(filePath);
     if (looksLikeBinary(previous)) return "";
     return previous.toString("utf-8");
   } catch {
@@ -223,7 +225,7 @@ export async function executeWrite(opts: {
       },
     };
   }
-  const previousContent = readPreviousTextForDiff(filePath);
+  const previousContent = await readPreviousTextForDiff(filePath);
   const existedBeforeWrite = existsSync(filePath);
 
   // Create parent directories