pi-hashline-edit-pro 0.16.12 → 0.16.13

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-hashline-edit-pro",
-  "version": "0.16.12",
+  "version": "0.16.13",
   "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/hashline/apply.ts

@@ -14,6 +14,20 @@ import {
 } from "./resolve";
 import { visLines } from "../utils";
 
+function lastNonEmptyIndex(lines: string[]): number {
+	for (let i = lines.length - 1; i >= 0; i--) {
+		if (lines[i]!.length > 0) return i;
+	}
+	return -1;
+}
+
+function firstNonEmptyIndex(lines: string[]): number {
+	for (let i = 0; i < lines.length; i++) {
+		if (lines[i]!.length > 0) return i;
+	}
+	return -1;
+}
+
 type LIdx = {
 	fileLines: string[];
 	lineStarts: number[];
@@ -306,13 +320,15 @@ export function applyEdits(
 			const edit = correctedEdits[bw.editIndex];
 			if (!edit) continue;
 			if (bw.kind === "trailing") {
-				const removed = edit.content_lines.pop();
-				if (removed !== undefined) {
+				const idx = lastNonEmptyIndex(edit.content_lines);
+				if (idx >= 0) {
+					const removed = edit.content_lines.splice(idx, 1)[0];
 					autoFixes.push({ kind: "trailing", editIndex: bw.editIndex, removedLine: removed });
 				}
 			} else {
-				const removed = edit.content_lines.shift();
-				if (removed !== undefined) {
+				const idx = firstNonEmptyIndex(edit.content_lines);
+				if (idx >= 0) {
+					const removed = edit.content_lines.splice(idx, 1)[0];
 					autoFixes.push({ kind: "leading", editIndex: bw.editIndex, removedLine: removed });
 				}
 			}

package/src/hashline/resolve.ts

@@ -94,7 +94,7 @@ export function fmtMismatch(
   const refList = notFound.map((m) => `"${m.ref.hash}"`).join(", ");
   if (notFound.length > 0) {
     out.push(
-      `[E_STALE_ANCHOR] ${notFound.length} stale anchor${notFound.length > 1 ? "s" : ""}${filePath ? ` in ${filePath}` : ""}: ${refList}. Call read() to get fresh anchors, then copy the 3-char HASH of the start and end of the range you are replacing into hash_range_inclusive of your next replace call.`
+      `[E_STALE_ANCHOR] ${notFound.length} stale anchor${notFound.length > 1 ? "s" : ""}${filePath ? ` in ${filePath}` : ""}: ${refList}. The file content has changed since those anchors were read. Call read() to get fresh anchors, then copy the 3-char HASH of the start and end of the range you are replacing into hash_range_inclusive of your next replace call.`
     );
   }
   if (ambiguous.length > 0) {
@@ -153,9 +153,19 @@ function assertItem(edit: Record<string, unknown>, index: number): void {
   if ("content_lines" in edit && !isStrArr(edit.content_lines)) {
     const val = edit.content_lines;
     if (typeof val === "string") {
-      throw new Error(CONTENT_LINES_NOT_STRING_MSG);
+      try {
+        const parsed = JSON.parse(val);
+        if (Array.isArray(parsed)) {
+          edit.content_lines = parsed;
+        } else {
+          throw new Error(CONTENT_LINES_NOT_STRING_MSG);
+        }
+      } catch {
+        throw new Error(CONTENT_LINES_NOT_STRING_MSG);
+      }
+    } else {
+      throw new Error(`[E_BAD_SHAPE] Edit ${index} field "content_lines" must be a string array.`);
     }
-    throw new Error(`[E_BAD_SHAPE] Edit ${index} field "content_lines" must be a string array.`);
   }
   if (!isStrPair(edit.hash_range_inclusive)) {
     throw new Error(
@@ -230,6 +240,20 @@ export function descEdit(edit: RHEdit): string {
 	return `replace ${edit.hash_range_inclusive[0].hash}-${edit.hash_range_inclusive[1].hash}`;
 }
 
+function lastNonEmpty(lines: string[]): string | undefined {
+	for (let i = lines.length - 1; i >= 0; i--) {
+		if (lines[i]!.length > 0) return lines[i]!;
+	}
+	return undefined;
+}
+
+function firstNonEmpty(lines: string[]): string | undefined {
+	for (let i = 0; i < lines.length; i++) {
+		if (lines[i]!.length > 0) return lines[i]!;
+	}
+	return undefined;
+}
+
 function checkBoundaryDup(
 	adjacentLine: string | undefined,
 	replacementEdge: string | undefined,
@@ -296,11 +320,11 @@ export function valEdits(
 		}
 		const endLine = endResolved.line;
 		const nextLine = fileLines[endLine];
-		const replacementLastLine = edit.content_lines.at(-1);
+		const replacementLastLine = lastNonEmpty(edit.content_lines);
 		const trailing = checkBoundaryDup(nextLine, replacementLastLine, "trailing", endLine, resolved.length);
 		if (trailing) boundaryWarnings.push(trailing);
 		const prevLine = fileLines[startResolved.line - 2];
-		const replacementFirstLine = edit.content_lines[0];
+		const replacementFirstLine = firstNonEmpty(edit.content_lines);
 		const leading = checkBoundaryDup(prevLine, replacementFirstLine, "leading", startResolved.line - 2, resolved.length);
 		if (leading) boundaryWarnings.push(leading);
 		resolved.push({

package/src/replace-normalize.ts

@@ -1,12 +1,19 @@
 import { isRec, has } from "./utils";
 import { CONTENT_LINES_NOT_STRING_MSG } from "./constants";
-function assertContentLinesNotString(
-	value: unknown,
-	label: string,
-): void {
-	if (typeof value === "string") {
-		throw new Error(CONTENT_LINES_NOT_STRING_MSG);
-	}
+
+function tryParseContentLines(record: Record<string, unknown>, key: string, label: string): void {
+  const val = record[key];
+  if (typeof val !== "string") return;
+  try {
+    const parsed = JSON.parse(val);
+    if (Array.isArray(parsed)) {
+      record[key] = parsed;
+      return;
+    }
+  } catch {
+    // fall through to error
+  }
+  throw new Error(CONTENT_LINES_NOT_STRING_MSG);
 }
 
 export function normalizeFilePath(record: Record<string, unknown>): void {
@@ -27,6 +34,17 @@ function normalizeField(
     record[to] = raw;
   } else if (isRec(raw)) {
     record[to] = [raw];
+  } else if (typeof raw === "string") {
+    try {
+      const parsed = JSON.parse(raw);
+      if (Array.isArray(parsed)) {
+        record[to] = parsed;
+      } else if (isRec(parsed)) {
+        record[to] = [parsed];
+      }
+    } catch {
+      // not valid JSON, leave as-is for downstream validation
+    }
   }
   if (from !== to) delete record[from];
 }
@@ -41,7 +59,7 @@ export function normReq(input: unknown): unknown {
   normalizeFilePath(record);
 
   if (has(record, "content_lines") && typeof record.content_lines === "string") {
-    assertContentLinesNotString(record.content_lines, "Top-level");
+    tryParseContentLines(record, "content_lines", "Top-level");
   }
 
   normalizeField(record, "changes", "changes");
@@ -51,7 +69,7 @@ export function normReq(input: unknown): unknown {
     for (let i = 0; i < record.changes.length; i++) {
       const item = record.changes[i];
       if (isRec(item) && has(item, "content_lines") && typeof item.content_lines === "string") {
-        assertContentLinesNotString(item.content_lines, `changes[${i}]`);
+        tryParseContentLines(item, "content_lines", `changes[${i}]`);
       }
     }
   }

package/src/replace.ts

@@ -113,10 +113,11 @@ interface PipelineResult {
   resultHashes: string[];
 }
 
-const ROOT_KS = new Set(["path", "changes"]);
+const ROOT_KS = new Set(["path", "changes", "content_lines", "hash_range_inclusive"]);
 
 export function assertReq(
   request: unknown,
+  flat?: boolean
 ): asserts request is ReqParams {
   if (!isRec(request)) {
     throw new Error("[E_BAD_SHAPE] Edit request must be an object.");
@@ -137,10 +138,14 @@ export function assertReq(
   }
 
   if (!Array.isArray(request.changes)) {
+    if (flat) {
+      throw new Error(
+        '[E_BAD_SHAPE] Edit request requires both "content_lines" and "hash_range_inclusive" at the top level.',
+      );
+    }
     throw new Error('[E_BAD_SHAPE] Edit request requires a "changes" array. Each change is { content_lines: [...], hash_range_inclusive: ["<START>", "<END>"] }.');
   }
 }
-
 export async function execPipeline(
   params: ReqParams,
   cwd: string,
@@ -217,10 +222,11 @@ export async function execPipeline(
 export async function compPreview(
   request: unknown,
   cwd: string,
+  flat?: boolean
 ): Promise<RPreview> {
   try {
     const normalized = normReq(request);
-    assertReq(normalized);
+    assertReq(normalized, flat);
     const { path, originalNormalized, originalHashes, result, resultHashes } = await execPipeline(
       normalized,
       cwd,
@@ -352,7 +358,7 @@ export function buildToolDef(opts: { flat: boolean; autoRead?: boolean }): ToolD
           context.state.preview = undefined;
           const previewGeneration = (context.state.previewGeneration ?? 0) + 1;
           context.state.previewGeneration = previewGeneration;
-          compPreview(previewInput, context.cwd)
+          compPreview(previewInput, context.cwd, opts.flat)
             .then((preview) => {
               if (
                 context.state.argsKey === argsKey &&