pi-hashline-edit-pro 2.5.1 → 2.5.2

package/README.md

@@ -1,5 +1,7 @@
 # pi-hashline-edit-pro
 
+[![npm version](https://img.shields.io/npm/v/pi-hashline-edit-pro.svg)](https://www.npmjs.com/package/pi-hashline-edit-pro) [![npm downloads](https://img.shields.io/npm/dm/pi-hashline-edit-pro.svg)](https://www.npmjs.com/package/pi-hashline-edit-pro)
+
 Hash-anchored `read` and `replace` tools for [pi-coding-agent](https://github.com/badlogic/pi-mono/tree/main/packages/coding-agent). Every line of a file gets a unique 3-character hash, and you edit by hash. No line numbers, no fuzzy matching, no edits landing on the wrong line.
 
 Fork of [pi-hashline-edit](https://github.com/RimuruW/pi-hashline-edit) by RimuruW, extended with 3-character hashes and collision resolution.
@@ -68,7 +70,7 @@ Edge cases:
 - UTF-16 and UTF-32 text (detected via BOM) is rejected, since editing it would corrupt the file.
 - Empty files come back as a single empty-line hash (`HASH│`); use `replace` on that hash to insert content.
 - BOMs are stripped for display. Non-UTF-8 bytes are shown as `U+FFFD`; editing such a file rewrites it as UTF-8, with a warning.
-- Files over 238,328 lines are rejected with `[E_FILE_TOO_LARGE]`.
+- Files over 238,328 lines or 100MB are rejected with `[E_FILE_TOO_LARGE]`.
 
 ## The replace tool
 
@@ -169,7 +171,7 @@ A no-op replace never changes the file, so anchors remain valid. On first run af
 | `[E_UNDO_STALE]` | `undo_last_replace` refused: the file was modified or deleted after the last replace. |
 | `[E_UNDO_UNAVAILABLE]` | Undo history could not be persisted to the hash store; the `replace` was refused and the file was left unchanged. |
 | `[E_RANGE_STALE]` | A line in the replaced range no longer matches what was last shown (the file changed on disk, or the line was never shown). The edit was refused; the current range is returned with fresh anchors. |
-| `[E_FILE_TOO_LARGE]` | The file exceeds the 238,328-line hashline limit. |
+| `[E_FILE_TOO_LARGE]` | The file exceeds the 238,328-line hashline limit or the 100MB size limit. |
 
 ## Troubleshooting
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-hashline-edit-pro",
-  "version": "2.5.1",
+  "version": "2.5.2",
   "type": "module",
   "description": "Hash-anchored read/replace/undo tools for pi-coding-agent. Every line gets a unique 3-char hash (A-Za-z0-9) that stays stable across edits; stale or ambiguous anchors are rejected, never fuzzy-matched. Undo persists across restarts.",
   "main": "index.ts",

package/src/file-kind.ts

@@ -54,7 +54,8 @@ export type LFile =
   | { kind: "directory" }
   | { kind: "image"; mimeType: string }
   | { kind: "text"; text: string; hadUtf8DecodeErrors?: true }
-  | { kind: "binary"; description: string };
+  | { kind: "binary"; description: string }
+  | { kind: "too_large"; description: string };
 
 
 export interface LoadFileOptions {
@@ -78,8 +79,8 @@ export async function loadFileKindAndText(
   }
   if (pathStat.size > MAX_BYTES) {
     return {
-      kind: "binary",
-      description: `file exceeds ${MAX_BYTES} byte limit`
+      kind: "too_large",
+      description: `exceeds the ${MAX_BYTES / (1024 * 1024)}MB size limit`,
     };
   }
 

package/src/hash-store.ts

@@ -49,6 +49,21 @@ export function isValidHashList(value: unknown): value is string[] {
   return true;
 }
 
+export function parseHashList(raw: string, onInvalid: () => void): string[] | undefined {
+  let parsed: unknown;
+  try {
+    parsed = JSON.parse(raw);
+  } catch {
+    onInvalid();
+    return undefined;
+  }
+  if (!isValidHashList(parsed)) {
+    onInvalid();
+    return undefined;
+  }
+  return parsed;
+}
+
 function isValidSnapshot(value: unknown): value is LegacySnapshot {
   if (typeof value !== "object" || value === null) return false;
   const v = value as Record<string, unknown>;
@@ -418,21 +433,13 @@ export function getSnapshot(
   }
   const row = store.stmts.get(path, checksum, lineCount);
   if (!row) return undefined;
-  let parsed: unknown;
-  try {
-    parsed = JSON.parse(row.hashes as string);
-  } catch {
+  const parsed = parseHashList(row.hashes as string, () => {
     if (deleteCorrupt) store.stmts.deleteOne(path);
     snapshotCache.delete(path);
-    return undefined;
-  }
-  if (isValidHashList(parsed)) {
-    cacheSnapshot(path, checksum, lineCount, parsed);
-    return parsed;
-  }
-  if (deleteCorrupt) store.stmts.deleteOne(path);
-  snapshotCache.delete(path);
-  return undefined;
+  });
+  if (!parsed) return undefined;
+  cacheSnapshot(path, checksum, lineCount, parsed);
+  return parsed;
 }
 
 export function upsertSnapshot(
@@ -461,23 +468,15 @@ export function upsertUndo(store: HashStore, path: string, entry: UndoRecord): v
 export function getUndoEntry(store: HashStore, path: string): UndoRecord | undefined {
   const row = store.stmts.undoGet(path);
   if (!row) return undefined;
-  try {
-    const parsed = JSON.parse(row.hashes as string);
-    if (!isValidHashList(parsed)) {
-      store.stmts.undoDelete(path);
-      return undefined;
-    }
-    return {
-      content: row.content as string,
-      bom: row.bom as string,
-      ending: row.ending as string,
-      hashes: parsed as string[],
-      resultContent: row.result_content as string,
-    };
-  } catch {
-    store.stmts.undoDelete(path);
-    return undefined;
-  }
+  const parsed = parseHashList(row.hashes as string, () => store.stmts.undoDelete(path));
+  if (!parsed) return undefined;
+  return {
+    content: row.content as string,
+    bom: row.bom as string,
+    ending: row.ending as string,
+    hashes: parsed,
+    resultContent: row.result_content as string,
+  };
 }
 
 export function deleteUndo(store: HashStore, path: string): void {

package/src/served.ts

@@ -1,5 +1,5 @@
 import type { HashStore } from "./hash-store";
-import { isValidHashList } from "./hash-store";
+import { parseHashList } from "./hash-store";
 import { HASH_CLASS } from "./hashline/alphabet";
 
 const SERVED_DIFF_ROW_RE = new RegExp(`^[+ ](${HASH_CLASS})│`);
@@ -16,17 +16,9 @@ export function servedHashesFromDiff(diff: string): string[] {
 export function getServed(store: HashStore, path: string): Set<string> | undefined {
   const row = store.stmts.servedGet(path);
   if (!row) return undefined;
-  try {
-    const parsed = JSON.parse(row.hashes as string);
-    if (!isValidHashList(parsed)) {
-      store.stmts.servedDelete(path);
-      return undefined;
-    }
-    return new Set(parsed);
-  } catch {
-    store.stmts.servedDelete(path);
-    return undefined;
-  }
+  const parsed = parseHashList(row.hashes as string, () => store.stmts.servedDelete(path));
+  if (!parsed) return undefined;
+  return new Set(parsed);
 }
 
 export function recordServed(store: HashStore, path: string, hashes: string[]): void {

package/src/utils.ts

@@ -16,9 +16,7 @@ export function splitLines(text: string): string[] {
 }
 
 export function visLines(text: string): string[] {
-  if (text.length === 0) return [];
-  const lines = text.split("\n");
-  return text.endsWith("\n") ? lines.slice(0, -1) : lines;
+  return text.length === 0 ? [] : splitLines(text);
 }
 
 

package/src/validation.ts

@@ -36,5 +36,10 @@ export function valKind(file: LFile, path: string): asserts file is { kind: "tex
 	if (file.kind === "image") {
 		throw new Error(`[E_NOT_TEXT] Path is an image file: ${path}. Hashline edit only supports text files.`);
 	}
+	if (file.kind === "too_large") {
+		throw new Error(
+			`[E_FILE_TOO_LARGE] File is too large: ${path} (${file.description}). Hashline editing targets source-sized files; for very large files use write or a non-line-based approach.`,
+		);
+	}
 }