pi-hashline-edit-pro 2.1.0 → 2.1.1

package/README.md

@@ -91,7 +91,7 @@ One edit per call, with `hash_bounds` and `new_content` at the top level:
 Notes:
 
 - The request is checked before any file I/O, so a bad request never touches the file.
-- Common copy-paste slips are fixed automatically and reported: a leftover `HASH│` prefix, diff-preview rows pasted into the replacement, a reversed range, or a boundary line pasted twice. `file_path` works as an alias for `path`.
+- Common copy-paste slips are fixed automatically and reported: a leftover `HASH│` prefix in `new_content` or `hash_bounds`, diff-preview rows pasted into the replacement, a reversed range, or a boundary line pasted twice. `file_path` works as an alias for `path` in all three tools.
 - An edit that produces identical content reports `No changes made` and leaves the anchors alone.
 - After a successful edit you get the post-edit diff with fresh anchors, so you can keep editing without re-reading.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-hashline-edit-pro",
-  "version": "2.1.0",
+  "version": "2.1.1",
   "type": "module",
   "description": "Strict hashline read/replace tool for pi-coding-agent with hash-anchored edits (3-char, 62-symbol, perfect hashing)",
   "main": "index.ts",
@@ -49,7 +49,8 @@
     "test:watch": "vitest",
     "test:coverage": "vitest run --coverage --coverage.thresholds.lines=90 --coverage.thresholds.statements=90 --coverage.thresholds.functions=85 --coverage.thresholds.branches=80",
     "lint": "eslint \"src/**/*.ts\" \"index.ts\" \"test/**/*.ts\"",
-    "typecheck": "tsc --noEmit"
+    "typecheck": "tsc --noEmit",
+    "prepublishOnly": "npm run typecheck && npm run lint && npm test"
   },
   "devDependencies": {
     "@earendil-works/pi-coding-agent": "^0.84.0",

package/src/hashline/resolve.ts

@@ -1,5 +1,5 @@
 import { abortIf, rejectUnknownFields, lastNonEmpty, firstNonEmpty, clipLine } from "../utils";
-import { HL_BARE_PREFIX_RE, HL_PREFIX_PLUS_RE, HL_PREFIX_MINUS_RE } from "./hash";
+import { HASH_CLASS, HL_BARE_PREFIX_RE, HL_PREFIX_PLUS_RE, HL_PREFIX_MINUS_RE } from "./hash";
 import { parseHashRef, parseText, type Anchor } from "./parse";
 import { NEW_CONTENT_NOT_STRING_MSG } from "../constants";
 
@@ -161,13 +161,32 @@ function assertItem(edit: Record<string, unknown>): void {
   }
 }
 
-export function resEdit(edit: HTEdit): HEdit {
+const ANCHOR_ROW_RE = new RegExp(`^([+-]?)(${HASH_CLASS})│`);
+
+export function resEdit(edit: HTEdit, warnings?: string[]): HEdit {
   assertItem(edit as Record<string, unknown>);
 
   const replaceLines = parseText(edit.new_content);
+  const bounds = edit.hash_bounds.map((ref) => {
+    const trimmed = ref.trim();
+    const match = trimmed.match(ANCHOR_ROW_RE);
+    if (match) {
+      let message: string;
+      if (match[1] === "+") {
+        message = `[E_BAD_REF] Autocorrected: stripped diff-preview marker copied from the diff preview in hash_bounds entry "${trimmed}".`;
+      } else if (match[1] === "-") {
+        message = `[E_BAD_REF] Autocorrected: stripped leading "-" marker in hash_bounds entry "${trimmed}".`;
+      } else {
+        message = `[E_BAD_REF] Autocorrected: stripped "HASH│" prefix copied from read output in hash_bounds entry "${trimmed}".`;
+      }
+      warnings?.push(message);
+      return match[2]!;
+    }
+    return ref;
+  }) as [string, string];
   return {
     content_lines: replaceLines,
-    hash_bounds: [parseHashRef(edit.hash_bounds[0]), parseHashRef(edit.hash_bounds[1])],
+    hash_bounds: [parseHashRef(bounds[0]), parseHashRef(bounds[1])],
   };
 }
 

package/src/read.ts

@@ -12,7 +12,7 @@ import { loadFileKindAndText } from "./file-kind";
 import { readNormFile } from "./file-reader";
 import { lineHashes, fmtRegion, HASH_SEP, MAX_HASH_LINES } from "./hashline";
 import { toCwd } from "./paths";
-import { abortIf } from "./utils";
+import { abortIf, isRec, normalizeFilePath } from "./utils";
 import { fileSnap } from "./file-reader";
 import { visLines } from "./utils";
 import { loadP, loadGuide } from "./prompts";
@@ -154,6 +154,12 @@ export function regRead(pi: ExtensionAPI): void {
 		description: R_DESC,
 		promptSnippet: R_SNIPPET,
 		promptGuidelines: readGuide(),
+		prepareArguments: (args: unknown) => {
+			if (!isRec(args)) return args as any;
+			const record = { ...args };
+			normalizeFilePath(record);
+			return record;
+		},
 		parameters: Type.Object({
 			path: Type.String({
 				description: "Path to the file to read (relative or absolute)",

package/src/replace-normalize.ts

@@ -1,11 +1,4 @@
-import { isRec } from "./utils";
-
-export function normalizeFilePath(record: Record<string, unknown>): void {
-  if (typeof record.path !== "string" && typeof record.file_path === "string") {
-    record.path = record.file_path;
-    delete record.file_path;
-  }
-}
+import { isRec, normalizeFilePath } from "./utils";
 
 export function normReq(input: unknown): unknown {
   if (!isRec(input)) {

package/src/replace-render.ts

@@ -163,62 +163,8 @@ function trimEmpty(lines: string[]): string[] {
 	return lines.slice(start, end);
 }
 
-function isSectionBoundary(line: string): boolean {
-	return (
-		line === "--- Anchors ---" ||
-		line === "Warnings:" ||
-		line === "Structure outline:" ||
-		/^--- Range \d+ ---$/.test(line)
-	);
-}
-
 export function fmtResultMd(text: string): string {
-	const lines = text.split("\n");
-	const sections: string[] = [];
-	let plainLines: string[] = [];
-
-	const flush = () => {
-		const trimmed = trimEmpty(plainLines);
-		if (trimmed.length > 0) {
-			sections.push(trimmed.join("\n"));
-		}
-		plainLines = [];
-	};
-
-	let index = 0;
-	while (index < lines.length) {
-		const line = lines[index]!;
-
-		if (line.startsWith("--- Anchors ")) {
-			flush();
-			const title = line.replace(/^---\s*/, "").replace(/\s*---$/, "");
-			index++;
-			const bodyLines: string[] = [];
-			while (
-				index < lines.length &&
-				!isSectionBoundary(lines[index]!)
-			) {
-				bodyLines.push(lines[index]!);
-				index++;
-			}
-			sections.push(
-				[
-					`#### ${title}`,
-					"```text",
-					...trimEmpty(bodyLines),
-					"```",
-				].join("\n"),
-			);
-			continue;
-		}
-
-		plainLines.push(line);
-		index++;
-	}
-
-	flush();
-
-	return sections.join("\n\n");
+	return trimEmpty(text.split("\n")).join("\n");
 }
 
 export function mkMdTheme(theme: MdTheme) {

package/src/replace-undo.ts

@@ -7,7 +7,7 @@ import { contentChecksum } from "./hashline/hasher";
 import { resolveTarget, writeAtomic } from "./fs-write";
 import { toCwd } from "./paths";
 import { toLF, stripBOM, genDiff, restoreEndings, type LineEnding } from "./replace-diff";
-import { cntDiff, splitLines, errCode } from "./utils";
+import { cntDiff, splitLines, errCode, isRec, normalizeFilePath } from "./utils";
 import { loadP, loadGuide } from "./prompts";
 import { buildMetrics } from "./replace-response";
 import { changedRange } from "./hashline";
@@ -91,6 +91,12 @@ export function regReplaceUndo(pi: ExtensionAPI): void {
     description: loadP("../prompts/undo-last-replace.md"),
     promptSnippet: loadP("../prompts/undo-last-replace-snippet.md"),
     promptGuidelines: loadGuide("../prompts/undo-last-replace-guidelines.md"),
+    prepareArguments: (args: unknown) => {
+      if (!isRec(args)) return args as any;
+      const record = { ...args };
+      normalizeFilePath(record);
+      return record;
+    },
     parameters: Type.Object({
       path: Type.String({
         description: "Path to the file to undo",

package/src/replace.ts

@@ -12,8 +12,8 @@ import {
   type LineEnding,
 } from "./replace-diff";
 import { readNormFile } from "./file-reader";
-import { normReq, normalizeFilePath } from "./replace-normalize";
-import { isRec, rejectUnknownFields, abortIf } from "./utils";
+import { normReq } from "./replace-normalize";
+import { isRec, rejectUnknownFields, abortIf, normalizeFilePath } from "./utils";
 import { resolveTarget, writeAtomic } from "./fs-write";
 import { applyEdit,
   lineHashes,
@@ -175,10 +175,14 @@ export async function execPipeline(
 
   const path = params.path;
 
-  const edit = resEdit({
-    hash_bounds: params.hash_bounds,
-    new_content: params.new_content,
-  });
+  const editWarnings: string[] = [];
+  const edit = resEdit(
+    {
+      hash_bounds: params.hash_bounds,
+      new_content: params.new_content,
+    },
+    editWarnings,
+  );
 
   const hashStore = options?.store ?? await loadHashStore();
   const { normalized: originalNormalized, bom, originalEnding, fileHashes: originalHashes, hadUtf8DecodeErrors, absolutePath } = await readNormFile(
@@ -207,8 +211,7 @@ export async function execPipeline(
         hashes: originalHashes,
         removedHashes,
       }, hashStore, noPersist !== true);
-  const warnings = [...(anchorResult.warnings ?? [])];
-
+  const warnings = [...editWarnings, ...(anchorResult.warnings ?? [])];
   const { totalAddedLines, totalRemovedLines } = countLineChanges(
     edit, originalHashes, isNoop, anchorResult.autoFixes?.length ?? 0,
   );

package/src/utils.ts

@@ -2,8 +2,11 @@ export function isRec(value: unknown): value is Record<string, unknown> {
 	return typeof value === "object" && value !== null && !Array.isArray(value);
 }
 
-export function has(record: Record<string, unknown>, key: string): boolean {
-	return Object.hasOwn(record, key);
+export function normalizeFilePath(record: Record<string, unknown>): void {
+  if (typeof record.path !== "string" && typeof record.file_path === "string") {
+    record.path = record.file_path;
+    delete record.file_path;
+  }
 }
 
 export function splitLines(text: string): string[] {