pi-hashline-edit-pro 0.13.1 → 0.13.3

package/index.ts

@@ -1,9 +1,9 @@
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { readFile } from "fs/promises";
 import { join, isAbsolute } from "path";
-import { lineHashes, initHasher, fmtRegion } from "./src/hashline";
+import { initHasher } from "./src/hashline";
 import { regReplace } from "./src/replace";
-import { regRead, formatPaginationHint } from "./src/read";
+import { regRead, fmtReadPreview } from "./src/read";
 import { toLF, stripBOM } from "./src/replace-diff";
 import { visLines } from "./src/utils";
 import { AUTO_READ_MAX } from "./src/constants";
@@ -47,29 +47,18 @@ export default function (pi: ExtensionAPI): void {
       const content = await readFile(absolutePath, "utf-8");
       const { text: rawContent } = stripBOM(content);
       const normalized = toLF(rawContent);
-      const visibleLines = visLines(normalized);
 
-      if (visibleLines.length === 0) return;
+      if (visLines(normalized).length === 0) return;
 
-      const truncated = visibleLines.length > AUTO_READ_MAX;
-      const displayLines = truncated ? visibleLines.slice(0, AUTO_READ_MAX) : visibleLines;
+      const preview = fmtReadPreview(normalized, { limit: AUTO_READ_MAX });
+      if (!preview.text) return;
 
-      const hashes = lineHashes(normalized);
-      const selectedHashes = hashes.slice(0, displayLines.length);
-      const hashlineOutput = fmtRegion(selectedHashes, displayLines);
-
-      const paginationHint = truncated
-        ? `\n\n${formatPaginationHint(1, AUTO_READ_MAX, visibleLines.length, AUTO_READ_MAX + 1)}`
-        : "";
-
-      if (hashlineOutput) {
-        return {
-          content: [
-            ...(event.content ?? []),
-            { type: "text", text: `\n\n--- Auto-read (hashline anchors) ---\n${hashlineOutput}${paginationHint}` },
-          ],
-        };
-      }
+      return {
+        content: [
+          ...(event.content ?? []),
+          { type: "text", text: `\n\n--- Auto-read (hashline anchors) ---\n${preview.text}` },
+        ],
+      };
     } catch (error) {
       console.error("Auto-read after write failed:", error);
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-hashline-edit-pro",
-  "version": "0.13.1",
+  "version": "0.13.3",
   "description": "Strict hashline read/replace tool for pi-coding-agent with hash-anchored edits (3-char, 18-bit, perfect hashing)",
   "main": "index.ts",
   "repository": {

package/src/constants.ts

@@ -1,16 +1,4 @@
 export const AUTO_READ_MAX = 2000;
-export const ANCHOR_BUDGET = 50 * 1024;
-
-/**
- * Context lines of post-edit anchor surfacing. Intentionally 0: after a
- * successful `replace` the response text is empty by design (the model calls
- * `read` for fresh anchors — see "Chained edits" in the README). With 0,
- * `affRange` returns null, so the anchor-block branch in `buildChanged`
- * (`src/replace-response.ts`) is dormant. Set this > 0 (and likely bump
- * MAX_OUT) to revive surfacing a small anchor window after each edit.
- */
-export const CTX_LINES = 0;
-export const MAX_OUT = 12;
 export const SNIFF_BYTES = 8192;
 export const MAX_BYTES = 100 * 1024 * 1024;
 

package/src/hashline/apply.ts

@@ -12,7 +12,6 @@ import {
 	type BDupWarn,
 } from "./resolve";
 import { cntLines } from "../utils";
-import { CTX_LINES, MAX_OUT } from "../constants";
 
 type LIdx = {
 	fileLines: string[];
@@ -271,6 +270,7 @@ export function applyEdits(
 	content: string;
 	firstChangedLine: number | undefined;
 	lastChangedLine: number | undefined;
+	resultHashes?: string[];
 	warnings?: string[];
 	noopEdits?: NEdit[];
 } {
@@ -350,52 +350,12 @@ export function applyEdits(
 		content: result,
 		firstChangedLine: range?.firstChangedLine,
 		lastChangedLine: range?.lastChangedLine,
+		resultHashes,
 		...(warnings.length ? { warnings } : {}),
 		...(noopEdits.length ? { noopEdits } : {}),
 	};
 }
 
-export function affRange(params: {
-	firstChangedLine: number | undefined;
-	lastChangedLine: number | undefined;
-	resultLineCount: number;
-	contextLines?: number;
-	maxOutputLines?: number;
-}): { start: number; end: number } | null {
-	const {
-		firstChangedLine,
-		lastChangedLine,
-		resultLineCount,
-		contextLines = CTX_LINES,
-		maxOutputLines = MAX_OUT,
-	} = params;
-
-	if (firstChangedLine === undefined || lastChangedLine === undefined) {
-		return null;
-	}
-
-	if (contextLines === 0) {
-		return null;
-	}
-
-	if (resultLineCount === 0) {
-		return null;
-	}
-
-	const start = Math.max(1, firstChangedLine - contextLines);
-	const end = Math.min(resultLineCount, lastChangedLine + contextLines);
-
-	if (end < start) {
-		return null;
-	}
-
-	if (end - start + 1 > maxOutputLines) {
-		return null;
-	}
-
-	return { start, end };
-}
-
 export function fmtRegion(
 	hashes: string[],
 	lines: string[],

package/src/hashline/hash.ts

@@ -55,6 +55,9 @@ function getH(): Hasher {
 hasherP = xxhash().then((h) => {
 	hasher = h;
 	return h;
+}).catch((err) => {
+	console.error("xxhash-wasm initialization failed:", err);
+	throw err;
 });
 
 export function initHasher(): Promise<Hasher> {

package/src/hashline/index.ts

@@ -34,7 +34,6 @@ export {
 export {
 	buildIdx,
 	applyEdits,
-	affRange,
 	fmtRegion,
 	changedRange,
 } from "./apply";

package/src/replace-normalize.ts

@@ -9,6 +9,24 @@ function tryParseJSON<T>(value: unknown, guard: (v: unknown) => v is T): T | und
   return undefined;
 }
 
+/**
+ * Coerces an array of edit items: JSON-string items → objects,
+ * JSON-string content_lines → string arrays. Shared by the `changes`
+ * and `edits` normalization branches.
+ */
+function coerceEditArray(items: unknown[]): unknown[] {
+  return items
+    .map((item: unknown) => tryParseJSON(item, isRec) ?? item)
+    .map((change: unknown) => {
+      if (!isRec(change)) return change;
+      if (typeof change.content_lines !== "string") return change;
+      const parsed = tryParseJSON(change.content_lines, (v): v is string[] =>
+        Array.isArray(v) && v.every((i) => typeof i === "string"),
+      );
+      return parsed ? { ...change, content_lines: parsed } : change;
+    });
+}
+
 export function normReq(input: unknown): unknown {
   if (!isRec(input)) {
     return input;
@@ -24,36 +42,30 @@ export function normReq(input: unknown): unknown {
   const hasChangesField = has(record, "changes");
   const hasEditsField = has(record, "edits");
 
-  if (hasChangesField) {
+if (hasChangesField) {
     const raw = tryParseJSON(record.changes, Array.isArray) ?? record.changes;
     if (Array.isArray(raw)) {
-      record.changes = raw
-        .map((item: unknown) => tryParseJSON(item, isRec) ?? item)
-        .map((change: unknown) => {
-          if (!isRec(change)) return change;
-          if (typeof change.content_lines !== "string") return change;
-          const parsed = tryParseJSON(change.content_lines, (v): v is string[] =>
-            Array.isArray(v) && v.every((i) => typeof i === "string"),
-          );
-          return parsed ? { ...change, content_lines: parsed } : change;
-        });
+        record.changes = coerceEditArray(raw);
+    } else {
+        // Single object (JSON string or literal) → wrap in array
+        const single = typeof raw === "string"
+            ? tryParseJSON(raw, isRec)
+            : isRec(raw) ? raw : undefined;
+        if (single) record.changes = coerceEditArray([single]);
     }
-  } else if (hasEditsField) {
+} else if (hasEditsField) {
     const raw = tryParseJSON(record.edits, Array.isArray) ?? record.edits;
     if (Array.isArray(raw)) {
-      record.changes = raw
-        .map((item: unknown) => tryParseJSON(item, isRec) ?? item)
-        .map((change: unknown) => {
-          if (!isRec(change)) return change;
-          if (typeof change.content_lines !== "string") return change;
-          const parsed = tryParseJSON(change.content_lines, (v): v is string[] =>
-            Array.isArray(v) && v.every((i) => typeof i === "string"),
-          );
-          return parsed ? { ...change, content_lines: parsed } : change;
-        });
+        record.changes = coerceEditArray(raw);
+    } else {
+        const single = typeof raw === "string"
+            ? tryParseJSON(raw, isRec)
+            : isRec(raw) ? raw : undefined;
+        if (single) record.changes = coerceEditArray([single]);
     }
     delete record.edits;
-  }
+}
 
   return record;
 }
+

package/src/replace-response.ts

@@ -1,12 +1,9 @@
 import type { ReplaceDetails } from "./replace";
 import { genDiff } from "./replace-diff";
 import {
-	affRange,
 	lineHashes,
-	fmtRegion,
 } from "./hashline";
 import { visLines } from "./utils";
-import { ANCHOR_BUDGET } from "./constants";
 
 type TResult = {
 	content: Array<{ type: "text"; text: string }>;
@@ -153,34 +150,9 @@ export function buildChanged(input: SuccessInput): TResult {
 	const addedLines = cntDiff(diffResult.diff, "+");
 	const removedLines = cntDiff(diffResult.diff, "-");
 	const warningsBlock = warnBlock(warnings);
-	const anchorRange = affRange({
-		firstChangedLine: editMeta.firstChangedLine,
-		lastChangedLine: editMeta.lastChangedLine,
-		resultLineCount: resultLines.length,
-	});
-	const anchorsBlock = anchorRange
-		? (() => {
-				const region = resultLines.slice(
-					anchorRange.start - 1,
-					anchorRange.end,
-				);
-				const regionHashes = resultHashes.slice(
-					anchorRange.start - 1,
-					anchorRange.end,
-				);
-				const formatted = fmtRegion(regionHashes, region);
-				const block = `--- Anchors ---\n${formatted}`;
-				return Buffer.byteLength(block, "utf8") <=
-					ANCHOR_BUDGET
-					? block
-					: "Anchors omitted; use read for subsequent edits.";
-			})()
-		: resultLines.length === 0
-			? "File is empty. Use replace to insert content."
-			: "";
-	const text = [anchorsBlock, warningsBlock.trimStart()]
-		.filter((section) => section.length > 0)
-		.join("\n\n");
+	const text = resultLines.length === 0
+		? "File is empty. Use replace to insert content."
+		: warningsBlock.trimStart();
 
 	const metrics = buildM({
 		classification: "applied",

package/src/replace.ts

@@ -99,6 +99,7 @@ interface PipelineResult {
   firstChangedLine?: number;
   lastChangedLine?: number;
   originalHashes?: string[];
+  resultHashes?: string[];
 }
 
 const E_DESC = loadP("../prompts/replace.md");
@@ -173,6 +174,7 @@ async function execPipeline(
     noopEdits: anchorResult.noopEdits,
     firstChangedLine: anchorResult.firstChangedLine,
     lastChangedLine: anchorResult.lastChangedLine,
+    resultHashes: anchorResult.resultHashes,
     originalHashes,
   };
 }
@@ -184,7 +186,7 @@ export async function compPreview(
 	try {
 		const normalized = normReq(request);
 		assertReq(normalized);
-		const { path, originalNormalized, result } = await execPipeline(
+		const { path, originalNormalized, result, resultHashes } = await execPipeline(
 			normalized,
 			cwd,
 			constants.R_OK,
@@ -196,7 +198,7 @@ export async function compPreview(
 			};
 		}
 
-		return { diff: genDiff(originalNormalized, result, 4, lineHashes(result)).diff };
+		return { diff: genDiff(originalNormalized, result, 4, resultHashes).diff };
 	} catch (error: unknown) {
 		return { error: error instanceof Error ? error.message : String(error) };
 	}
@@ -340,6 +342,7 @@ const toolDef: ToolDef = {
 				noopEdits,
 				firstChangedLine,
 				lastChangedLine,
+				resultHashes,
 			} = await execPipeline(
 				normalizedParams,
 				ctx.cwd,
@@ -390,7 +393,7 @@ const toolDef: ToolDef = {
 				path,
 				originalNormalized,
 				result,
-				resultHashes: lineHashes(result),
+				resultHashes,
 				warnings,
 				snapshotId: updatedSnapshotId,
 				editMeta,