pi-readseek 0.3.4 → 0.3.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-readseek",
-  "version": "0.3.4",
+  "version": "0.3.6",
   "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-classify.ts

@@ -62,9 +62,6 @@ export function classifyEdit(oldContent: string, newContent: string): EditClassi
 
 let difftCachedResult: boolean | null = null;
 
-export function _resetDifftCache(): void {
-  difftCachedResult = null;
-}
 
 export async function isDifftAvailable(): Promise<boolean> {
   if (difftCachedResult !== null) return difftCachedResult;

package/src/hashline.ts

@@ -115,18 +115,6 @@ export function formatHashlineDisplay(lineNumber: number, content: string): stri
 	return `${lineNumber}:${computeLineHash(lineNumber, content)}|${escapeControlCharsForDisplay(content)}`;
 }
 
-export function hashLine(lineNumber: number, content: string): string {
-	return formatHashlineDisplay(lineNumber, content);
-}
-
-export function hashLines(content: string): string {
-	return content
-		.split("\n")
-		.map((line, i) => formatHashlineDisplay(i + 1, line))
-		.join("\n");
-}
-
-
 export function parseLineRef(ref: string): { line: number; hash: string; content?: string } {
 	const contentMatch = ref.match(/^[^|]*\|(.*)$/);
 	const contentAfterPipe = contentMatch ? contentMatch[1] : undefined;

package/src/map-cache.ts

@@ -133,20 +133,4 @@ export async function getOrGenerateMap(absPath: string): Promise<FileMap | null>
 	} catch {
 		return null;
 	}
-}
-export function setMapCacheMaxSize(size: number): void {
-	getMapCacheState().maxSize = size;
-}
-
-/**
- * Clear the map cache. Exported for testing.
- */
-export function clearMapCache(): void {
-	const state = getMapCacheState();
-	state.cache.clear();
-	state.maxSize = MAP_CACHE_MAX_SIZE;
-}
-
-export function __getInMemoryMapCacheSizeForTest(): number {
-	return getMapCacheState().cache.size;
 }

package/src/persistent-map-cache.ts

@@ -13,7 +13,7 @@ import { resolveReadseekJsonSettings } from "./readseek-settings.js";
  * 3. `$XDG_CACHE_HOME/pi-readseek/maps` (when non-empty).
  * 4. `~/.cache/pi-readseek/maps`.
  */
-export function resolveCacheDir(): string {
+function resolveCacheDir(): string {
   const explicit = process.env.PI_HASHLINE_MAP_CACHE_DIR;
   if (explicit && explicit.length > 0) return explicit;
 
@@ -170,39 +170,22 @@ async function tryWriteCachedRaw(key: string, map: FileMap): Promise<boolean> {
   }
 }
 
-/**
- * Low-level atomic write. Internal helper (exported so tests can seed fixtures
- * without going through the eviction counter).
- */
-export async function writeCachedRaw(key: string, map: FileMap): Promise<void> {
+/** Low-level atomic write. Internal helper. */
+async function writeCachedRaw(key: string, map: FileMap): Promise<void> {
   await tryWriteCachedRaw(key, map);
 }
 
-const DEFAULT_EVICTION_INTERVAL = 20;
-const DEFAULT_EVICTION_CAP = 5000;
+const EVICTION_INTERVAL = 20;
+const EVICTION_CAP = 5000;
 let writeCounter = 0;
-let evictionInterval = DEFAULT_EVICTION_INTERVAL;
-let evictionCap = DEFAULT_EVICTION_CAP;
-
-export function __setEvictionHooksForTest(
-  hooks: { interval?: number; cap?: number } | null,
-): void {
-  if (hooks === null) {
-    evictionInterval = DEFAULT_EVICTION_INTERVAL;
-    evictionCap = DEFAULT_EVICTION_CAP;
-    return;
-  }
 
-  if (typeof hooks.interval === "number") evictionInterval = hooks.interval;
-  if (typeof hooks.cap === "number") evictionCap = hooks.cap;
-}
 
 const CACHE_KEY_FILE = /^[0-9a-f]{32}\.json$/;
 async function maybeEvict(): Promise<void> {
   try {
     const dir = resolveCacheDir();
     const names = (await readdir(dir)).filter((n) => CACHE_KEY_FILE.test(n));
-    if (names.length <= evictionCap) return;
+    if (names.length <= EVICTION_CAP) return;
 
     const entries: Array<{ path: string; age: number }> = [];
     for (const name of names) {
@@ -217,7 +200,7 @@ async function maybeEvict(): Promise<void> {
     }
 
     entries.sort((a, b) => a.age - b.age);
-    const excess = entries.length - evictionCap;
+    const excess = entries.length - EVICTION_CAP;
     for (let i = 0; i < excess; i += 1) {
       try {
         await unlink(entries[i].path);
@@ -236,7 +219,7 @@ async function maybeEvict(): Promise<void> {
 export async function writeCached(key: string, map: FileMap): Promise<void> {
   if (!(await tryWriteCachedRaw(key, map))) return;
   writeCounter += 1;
-  if (writeCounter % evictionInterval === 0) {
+  if (writeCounter % EVICTION_INTERVAL === 0) {
     try {
       await maybeEvict();
     } catch {
@@ -245,7 +228,3 @@ export async function writeCached(key: string, map: FileMap): Promise<void> {
   }
 }
 
-// Test-only: reset the write counter between suites.
-export function __resetWriteCounter(): void {
-  writeCounter = 0;
-}

package/src/readseek/mapper.ts

@@ -1,7 +1,7 @@
 import { stat } from "node:fs/promises";
 
 import { readseekMap, readseekMapContent } from "../readseek-client.js";
-import { THRESHOLDS } from "./constants.js";
+import { throwIfAborted } from "../runtime.js";
 import type { FileMap, MapOptions } from "./types.js";
 
 export const READSEEK_MAPPER_NAME = "readseek";
@@ -20,17 +20,6 @@ export const READSEEK_MAPPER_IDENTITY: MapperIdentity = {
   mapperName: READSEEK_MAPPER_NAME,
   mapperVersion: READSEEK_MAPPER_VERSION,
 };
-
-export const ALL_MAPPER_IDENTITIES: Record<string, MapperIdentity> = {
-  readseek: READSEEK_MAPPER_IDENTITY,
-};
-
-function throwIfAborted(signal?: AbortSignal): void {
-  if (!signal?.aborted) return;
-  const reason = signal.reason;
-  throw reason instanceof Error ? reason : new Error("aborted");
-}
-
 export async function generateMapWithIdentity(
   filePath: string,
   options: MapOptions = {},
@@ -61,9 +50,3 @@ export async function generateMapFromContent(
   return map;
 }
 
-export function shouldGenerateMap(
-  totalLines: number,
-  totalBytes: number,
-): boolean {
-  return totalLines > THRESHOLDS.MAX_LINES || totalBytes > THRESHOLDS.MAX_BYTES;
-}

package/src/readseek/parser-errors.ts

@@ -17,6 +17,3 @@ export function reportParserError(
   console.error(`[readseek] ${context}: ${causeMessage(err)}`);
 }
 
-export function __resetParserErrorReporterForTests(): void {
-  emitted.clear();
-}

package/src/readseek/parser-loader.ts

@@ -76,8 +76,3 @@ export async function getWasmParser(langId: WasmLanguageId): Promise<WasmParser
   }
 }
 
-export function __resetWasmParserLoaderForTests(): void {
-  initPromise = null;
-  languages.clear();
-  languageLoads.clear();
-}

package/src/readseek-settings.ts

@@ -19,18 +19,6 @@ export interface ReadseekSettingsResult {
   warnings: ReadseekSettingsWarning[];
 }
 
-let pathOverride: { globalSettingsPath?: string; projectSettingsPath?: string } | null = null;
-
-export function __setReadseekSettingsPathsForTest(paths: {
-  globalSettingsPath?: string;
-  projectSettingsPath?: string;
-}): void {
-  pathOverride = { ...paths };
-}
-
-export function __resetReadseekSettingsPathsForTest(): void {
-  pathOverride = null;
-}
 
 function defaultGlobalSettingsPath(): string {
   return join(homedir(), ".pi/agent/readseek/settings.json");
@@ -137,10 +125,8 @@ function mergeSettings(base: ReadseekJsonSettings, override: ReadseekJsonSetting
 }
 
 export function resolveReadseekJsonSettings(): ReadseekSettingsResult {
-  const globalPath = pathOverride?.globalSettingsPath ?? defaultGlobalSettingsPath();
-  const projectPath = pathOverride?.projectSettingsPath ?? defaultProjectSettingsPath();
-  const globalResult = readSettingsFile(globalPath);
-  const projectResult = readSettingsFile(projectPath);
+  const globalResult = readSettingsFile(defaultGlobalSettingsPath());
+  const projectResult = readSettingsFile(defaultProjectSettingsPath());
   return {
     settings: mergeSettings(globalResult.settings, projectResult.settings),
     warnings: [...globalResult.warnings, ...projectResult.warnings],

package/src/readseek-value.ts

@@ -37,12 +37,6 @@ export interface ReadseekRange {
   totalLines?: number;
 }
 
-export interface ReadseekFileGroup {
-  path: string;
-  ranges: ReadseekRange[];
-  lines: ReadseekLine[];
-}
-
 export interface SemanticSummary {
   classification: "no-op" | "whitespace-only" | "semantic" | "mixed";
   difftasticAvailable: boolean;
@@ -76,7 +70,7 @@ export function buildReadseekLines(startLine: number, rawLines: string[]): Reads
   return rawLines.map((raw, index) => buildReadseekLine(startLine + index, raw));
 }
 
-export function renderReadseekLine(line: ReadseekLine): string {
+function renderReadseekLine(line: ReadseekLine): string {
   return `${line.anchor}|${line.display}`;
 }
 
@@ -106,18 +100,6 @@ export function buildReadseekError(
   };
 }
 
-export function buildReadseekRange(startLine: number, endLine: number, totalLines?: number): ReadseekRange {
-  return totalLines === undefined ? { startLine, endLine } : { startLine, endLine, totalLines };
-}
-
-export function buildReadseekFileGroup(path: string, ranges: ReadseekRange[], lines: ReadseekLine[]): ReadseekFileGroup {
-  return {
-    path,
-    ranges: ranges.map((range) => ({ ...range })),
-    lines: lines.map((line) => ({ ...line })),
-  };
-}
-
 export function buildReadseekEditResult(input: {
   ok?: boolean;
   path: string;

package/src/readseek-error-codes.ts

@@ -1,54 +0,0 @@
-/**
- * READSEEK error code taxonomy — single source of truth.
- *
- * Every error returned via `readseekValue.error.code` MUST be a key in this map.
- * To add a new error: extend this object with a kebab-case code, a one-line
- * description, and the trigger condition. Codes are stable: renaming is a
- * breaking change for downstream consumers.
- *
- * Distinction:
- * - errors (this file): fatal — the tool could not produce its primary result.
- * - warnings (ReadseekWarning): non-fatal — the tool produced a result but flagged
- *   something the agent should know.
- */
-export const READSEEK_ERROR_CODES = {
-  // read
-  "file-not-found": { description: "Target file does not exist", trigger: "fs ENOENT on read" },
-  "path-is-directory": { description: "Path resolves to a directory, not a file", trigger: "fs EISDIR on read" },
-  "permission-denied": { description: "Filesystem refused access", trigger: "fs EACCES or EPERM" },
-  "fs-error": { description: "Unexpected filesystem failure outside the specific classified cases", trigger: "non-ENOENT/non-EISDIR/non-EACCES/non-EPERM fs error while reading, writing, or stat'ing a path" },
-  "offset-past-end": { description: "Requested offset exceeds file length", trigger: "offset > total lines" },
-  "invalid-params-combo": { description: "Mutually exclusive parameters combined", trigger: "e.g. symbol + offset, bundle + map, map + symbol" },
-  "invalid-offset": { description: "offset is not a positive integer", trigger: "non-int or value < 1" },
-  "invalid-limit": { description: "limit is not a positive integer", trigger: "non-int or value < 1" },
-
-  // edit
-  "file-not-read": { description: "edit called on a path that was not read in this session", trigger: "wasReadInSession returned false" },
-  "hash-mismatch": { description: "edit anchors do not verify against current file contents", trigger: "applyHashlineEdits detected stale anchors" },
-  "no-op": { description: "edits produced identical content", trigger: "originalNormalized === result after applying edits" },
-  "text-not-found": { description: "replace.old_text not present in file", trigger: "replaceText returned 0 matches" },
-  "binary-file": { description: "edit refused because file is binary", trigger: "isBinaryBuffer detected NUL bytes" },
-  "invalid-edit-variant": { description: "edits[i] is not exactly one of set_line/replace_lines/insert_after/replace", trigger: "exactly-one variant check failed" },
-
-  // grep
-  "binary-file-target": { description: "grep target is a binary file", trigger: "explicit binary path supplied" },
-  "passthrough-unparsed": { description: "builtin grep result format unrecognized", trigger: "passthrough warning from underlying tool" },
-
-  // search
-  "readseek-not-installed": { description: "readseek CLI is not installed", trigger: "ENOENT on `readseek` invocation or bundled package resolution failure" },
-  "readseek-execution-error": { description: "readseek search exited with an error", trigger: "non-zero exit or invalid search output" },
-
-  // find / ls (shared shape)
-  "path-not-found": { description: "search/target path does not exist", trigger: "stat failed on path" },
-  "path-not-directory": { description: "path is a file, not a directory", trigger: "stat returned non-directory" },
-
-  // write
-  "binary-content": { description: "content written to write tool looks binary", trigger: "looksLikeBinary on supplied content" },
-
-  "syntax-regression": {
-    description: "edit introduced new tree-sitter syntax errors compared to pre-edit content",
-    trigger: "post-write validator detected net-new ERROR or MISSING nodes (block mode)",
-  },
-} as const;
-
-export type ReadseekErrorCode = keyof typeof READSEEK_ERROR_CODES;