pi-hashline-edit-pro 0.11.0 → 0.11.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-hashline-edit-pro",
-  "version": "0.11.0",
+  "version": "0.11.2",
   "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/prompts/replace-guidelines.md

@@ -1,3 +1,3 @@
 - Use `replace` with HASH anchors for all file changes; batch every change to one file into a single `replace` call.
 - After a successful `replace`, the response text is empty (warnings only). Call `read` to get fresh anchors for follow-up edits.
-- On `[E_STALE_ANCHOR]`, call `read` to get fresh anchors, copy the 3-character HASH from each line into `old_range`, and retry.
+- On `[E_STALE_ANCHOR]`, call `read` to get fresh anchors, copy the 3-character HASH from each line into `hash_range_incl`, and retry.

package/prompts/replace.md

@@ -13,10 +13,10 @@ read({ path: "src/main.ts" })
 // VRW│const z = 3;
 ```
 
-2. Copy the 3-character HASH (before `│`) into `old_range`:
+2. Copy the 3-character HASH (before `│`) into `hash_range_incl`:
 ```json
 { "path": "src/main.ts", "edits": [
-  { "old_range": ["MQX", "MQX"], "new_lines": ["const x = 99;"] }
+  { "hash_range_incl": ["MQX", "MQX"], "new_lines": ["const x = 99;"] }
 ] }
 ```
 
@@ -25,14 +25,14 @@ Examples:
 1. Single line replace:
 ```json
 { "path": "src/main.ts", "edits": [
-  { "old_range": ["MQX", "MQX"], "new_lines": ["const x = 1;"] }
+  { "hash_range_incl": ["MQX", "MQX"], "new_lines": ["const x = 1;"] }
 ] }
 ```
 
 2. Range replace (3 lines → 3 new lines):
 ```json
 { "path": "src/main.ts", "edits": [
-  { "old_range": ["ZPM", "VRW"], "new_lines": [
+  { "hash_range_incl": ["ZPM", "VRW"], "new_lines": [
     "function greet(name) {",
     "  return `Hello, ${name}`;",
     "}"
@@ -43,8 +43,8 @@ Examples:
 3. Multiple regions in one call (delete two non-adjacent ranges):
 ```json
 { "path": "src/server.ts", "edits": [
-  { "old_range": ["aB3", "xY7"], "new_lines": [] },
-  { "old_range": ["MQX", "ZPM"], "new_lines": [] }
+  { "hash_range_incl": ["aB3", "xY7"], "new_lines": [] },
+  { "hash_range_incl": ["MQX", "ZPM"], "new_lines": [] }
 ] }
 ```
 
@@ -52,38 +52,38 @@ Examples:
 
 Wrong:
 ```json
-{ "old_range": ["F4T", "F4T"], "new_lines": ["F4T│import { x } from \"./x\";"] }
+{ "hash_range_incl": ["F4T", "F4T"], "new_lines": ["F4T│import { x } from \"./x\";"] }
 ```
 
 Right:
 ```json
-{ "old_range": ["F4T", "F4T"], "new_lines": ["import { x } from \"./x\";"] }
+{ "hash_range_incl": ["F4T", "F4T"], "new_lines": ["import { x } from \"./x\";"] }
 ```
 
-`old_range` uses the hash anchor. `new_lines` uses literal file content only — the same text that appears after the `│` in `read` output.
+`hash_range_incl` uses the hash anchor. `new_lines` uses literal file content only — the same text that appears after the `│` in `read` output.
 
-⚠️ Common mistake: `old_range` is only the 3-character HASH, not the full `HASH│content` line.
+⚠️ Common mistake: `hash_range_incl` is only the 3-character HASH, not the full `HASH│content` line.
 
 Wrong:
 ```json
-{ "old_range": ["F4T│import { x } from \"./x\";", "F4T│import { x } from \"./x\";"], "new_lines": [...] }
+{ "hash_range_incl": ["F4T│import { x } from \"./x\";", "F4T│import { x } from \"./x\";"], "new_lines": [...] }
 ```
 
 Right:
 ```json
-{ "old_range": ["F4T", "F4T"], "new_lines": [...] }
+{ "hash_range_incl": ["F4T", "F4T"], "new_lines": [...] }
 ```
 
 Rules:
-- `old_range` is a pair `[start, end]`. A single-line replace is `old_range: ["X", "X"]`.
+- `hash_range_incl` is a pair `[start, end]`. A single-line replace is `hash_range_incl: ["X", "X"]`.
 - To delete a range, use `new_lines: []`.
-- `old_range` elements are HASH anchors only (e.g. `aB3`). Do not include `│` or line content.
+- `hash_range_incl` elements are HASH anchors only (e.g. `aB3`). Do not include `│` or line content.
 - `new_lines` is literal file content — each string becomes exactly one line in the file. No `HASH│` prefix, no `+`/`-` diff markers.
 - Don't add `""` for spacing unless you actually want a new blank line.
 - Copy anchors from the most recent `read` of the file. Do not guess or construct them.
 - All edits in one call must be non-conflicting. The runtime rejects with `[E_EDIT_CONFLICT]` if two ranges overlap.
 - If `new_lines` matches current content, the replace is classified as `noop` (file unchanged).
-- The `old_range` is inclusive — both anchors and every line between them are replaced. If your replacement content includes lines that already exist in the file (e.g. closing brackets), make sure those lines are within your range, otherwise they will appear twice.
+- The `hash_range_incl` is inclusive — both anchors and every line between them are replaced. If your replacement content includes lines that already exist in the file (e.g. closing brackets), make sure those lines are within your range, otherwise they will appear twice.
 
 On success, the response text is empty (or contains only warnings if present). Call `read` to get fresh anchors for follow-up edits.
 
@@ -92,9 +92,9 @@ Error recovery:
 - `[E_BAD_REF]` — malformed HASH. Re-read and try again.
 - `[E_BAD_OP]` — invalid operation (e.g. start line > end line).
 - `[E_BAD_SHAPE]` — malformed request or edit item (missing fields, wrong types, unknown fields).
-- `[E_LEGACY_SHAPE]` — old `oldText`/`newText` format detected. Use `{old_range, new_lines}` instead.
+- `[E_LEGACY_SHAPE]` — old `oldText`/`newText` or `old_range` format detected. Use `{hash_range_incl, new_lines}` instead.
 - `[E_EDIT_CONFLICT]` — two edits overlap on the same line range. Make edits non-overlapping.
 - `[E_AMBIGUOUS_ANCHOR]` — hash collision. Call `read` to get fresh anchors.
-- `[E_BARE_HASH_PREFIX]` — a `new_lines` entry starts with `HASH│`. Remove the hash prefix; keep only the literal line content that appears after `│` in `read` output. `old_range` uses hashes, `new_lines` does not.
+- `[E_BARE_HASH_PREFIX]` — a `new_lines` entry starts with `HASH│`. Remove the hash prefix; keep only the literal line content that appears after `│` in `read` output. `hash_range_incl` uses hashes, `new_lines` does not.
 - `[E_INVALID_PATCH]` — diff prefixes (`+`/`-`) in `new_lines`. Use literal content only.
-- `[E_WOULD_EMPTY]` — edit would empty a non-empty file.
+- `[E_WOULD_EMPTY]` — edit would empty a non-empty file.

package/src/hashline/apply.ts

@@ -76,8 +76,8 @@ function resToSpan(
 ): RESpan | null {
 	const { fileLines, lineStarts, hasTerminalNewline } = lineIndex;
 
-	const startLine = edit.old_range[0].line;
-	const endLine = edit.old_range[1].line;
+	const startLine = edit.hash_range_incl[0].line;
+	const endLine = edit.hash_range_incl[1].line;
 	const originalLines = fileLines.slice(startLine - 1, endLine);
 	if (
 		originalLines.length === edit.new_lines.length &&
@@ -87,7 +87,7 @@ function resToSpan(
 	) {
 		noopEdits.push({
 			editIndex: index,
-			loc: edit.old_range[0].hash,
+			loc: edit.hash_range_incl[0].hash,
 			currentContent: originalLines.join("\n"),
 		});
 		return null;
@@ -422,4 +422,4 @@ export function changedRange(
 	}
 
 	return { firstChangedLine, lastChangedLine };
-}
+}

package/src/hashline/parse.ts

@@ -20,7 +20,7 @@ function diagRef(ref: string): string {
 	}
 
 	if (trimmed.includes("│")) {
-		return `[E_BAD_REF] Invalid anchor "${trimmed}". old_range must contain the 3-char hash only — remove everything from "│" onward.`;
+		return `[E_BAD_REF] Invalid anchor "${trimmed}". hash_range_incl must contain the 3-char hash only — remove everything from "│" onward.`;
 	}
 
 	return `[E_BAD_REF] Invalid anchor "${trimmed}". Expected a 3-char base64 anchor (e.g. "aB3").`;

package/src/hashline/resolve.ts

@@ -9,9 +9,9 @@ export type RAnchor = {
 	hashMatched: boolean;
 };
 
-export type HEdit = { old_range: [Anchor, Anchor]; new_lines: string[] };
+export type HEdit = { hash_range_incl: [Anchor, Anchor]; new_lines: string[] };
 export type RHEdit = {
-	old_range: [RAnchor, RAnchor];
+	hash_range_incl: [RAnchor, RAnchor];
 	new_lines: string[];
 };
 
@@ -37,7 +37,7 @@ export interface NEdit {
 }
 
 export type HTEdit = {
-	old_range: [string, string];
+	hash_range_incl: [string, string];
 	new_lines: string[];
 };
 
@@ -118,7 +118,7 @@ export function fmtMismatch(
 	return out.join("\n");
 }
 
-const ITEM_KS = new Set(["old_range", "new_lines"]);
+const ITEM_KS = new Set(["hash_range_incl", "new_lines"]);
 
 function isStrArr(value: unknown): value is string[] {
 	return (
@@ -135,11 +135,11 @@ function isStrPair(value: unknown): value is [string, string] {
 }
 
 function assertItem(edit: Record<string, unknown>, index: number): void {
-	rejectUnknownFields(edit, ITEM_KS, `Edit ${index}`, "Each edit takes only { old_range, new_lines }.");
+	rejectUnknownFields(edit, ITEM_KS, `Edit ${index}`, "Each edit takes only { hash_range_incl, new_lines }.");
 
-	if ("old_range" in edit && !isStrPair(edit.old_range)) {
+	if ("hash_range_incl" in edit && !isStrPair(edit.hash_range_incl)) {
 		throw new Error(
-			`[E_BAD_SHAPE] Edit ${index} field "old_range" must be a pair of anchor strings [start, end].`,
+			`[E_BAD_SHAPE] Edit ${index} field "hash_range_incl" must be a pair of anchor strings [start, end].`,
 		);
 	}
 	if (!("new_lines" in edit)) {
@@ -148,9 +148,9 @@ function assertItem(edit: Record<string, unknown>, index: number): void {
 	if ("new_lines" in edit && !isStrArr(edit.new_lines)) {
 		throw new Error(`[E_BAD_SHAPE] Edit ${index} field "new_lines" must be a string array.`);
 	}
-	if (!isStrPair(edit.old_range)) {
+	if (!isStrPair(edit.hash_range_incl)) {
 		throw new Error(
-			`[E_BAD_SHAPE] Edit ${index} requires an "old_range" pair of anchor strings [start, end].`,
+			`[E_BAD_SHAPE] Edit ${index} requires an "hash_range_incl" pair of anchor strings [start, end].`,
 		);
 	}
 
@@ -163,7 +163,7 @@ export function resEdits(edits: HTEdit[]): HEdit[] {
 
 		const replaceLines = parseText(edit.new_lines);
 		result.push({
-			old_range: [parseHashRef(edit.old_range[0]), parseHashRef(edit.old_range[1])],
+			hash_range_incl: [parseHashRef(edit.hash_range_incl[0]), parseHashRef(edit.hash_range_incl[1])],
 			new_lines: replaceLines,
 		});
 	}
@@ -216,12 +216,12 @@ export function assertNoBarePrefix(
 			: `${matchedCount} match file line hashes — strong evidence the prefix was copied from read output.`;
 
 	throw new Error(
-		`[E_BARE_HASH_PREFIX] ${suspects.length} edit line(s) start with a hash-like prefix (${locations}). Example: ${JSON.stringify(exampleLine)}. ${linesHint} Remove the "HASH│" prefix from each affected new_lines entry; keep only the literal line content that appears after "│" in read output. Remember: old_range uses hash anchors, new_lines uses file content only.`
+		`[E_BARE_HASH_PREFIX] ${suspects.length} edit line(s) start with a hash-like prefix (${locations}). Example: ${JSON.stringify(exampleLine)}. ${linesHint} Remove the "HASH│" prefix from each affected new_lines entry; keep only the literal line content that appears after "│" in read output. Remember: hash_range_incl uses hash anchors, new_lines uses file content only.`
 	);
 }
 
 export function descEdit(edit: RHEdit): string {
-	return `replace ${edit.old_range[0].hash}-${edit.old_range[1].hash}`;
+	return `replace ${edit.hash_range_incl[0].hash}-${edit.hash_range_incl[1].hash}`;
 }
 
 export function valEdits(
@@ -247,14 +247,14 @@ export function valEdits(
 
 	for (const edit of edits) {
 		abortIf(signal);
-		const startResolved = tryResolve(edit.old_range[0]);
-		const endResolved = tryResolve(edit.old_range[1]);
+		const startResolved = tryResolve(edit.hash_range_incl[0]);
+		const endResolved = tryResolve(edit.hash_range_incl[1]);
 		if (!startResolved || !endResolved) {
 			continue;
 		}
 		if (startResolved.line > endResolved.line) {
 			throw new Error(
-				`[E_BAD_OP] Range start line ${startResolved.line} must be <= end line ${endResolved.line} (anchors ${edit.old_range[0].hash} and ${edit.old_range[1].hash}).`,
+				`[E_BAD_OP] Range start line ${startResolved.line} must be <= end line ${endResolved.line} (anchors ${edit.hash_range_incl[0].hash} and ${edit.hash_range_incl[1].hash}).`,
 			);
 		}
 		const endLine = endResolved.line;
@@ -293,7 +293,7 @@ export function valEdits(
 			});
 		}
 		resolved.push({
-			old_range: [startResolved, endResolved],
+			hash_range_incl: [startResolved, endResolved],
 			new_lines: edit.new_lines,
 		});
 	}
@@ -301,4 +301,4 @@ export function valEdits(
 	return { resolved, mismatches, boundaryWarnings };
 }
 
-export { warnUnicodeEsc };
+export { warnUnicodeEsc };

package/src/replace-normalize.ts

@@ -12,6 +12,16 @@ function coerceEditsArray(edits: unknown): unknown {
 	}
 }
 
+function normalizeEditItem(item: unknown): unknown {
+	if (!isRec(item)) return item;
+	const record: Record<string, unknown> = { ...item };
+	// Rename old_range to hash_range_incl for backward compatibility
+	if (has(record, "old_range") && !has(record, "hash_range_incl")) {
+		record.hash_range_incl = record.old_range;
+		delete record.old_range;
+	}
+	return record;
+}
 
 export function normReq(input: unknown): unknown {
 	if (!isRec(input)) {
@@ -31,5 +41,10 @@ export function normReq(input: unknown): unknown {
 		record.edits = coerceEditsArray(record.edits);
 	}
 
+	// Normalize each edit item: rename old_range to hash_range_incl
+	if (Array.isArray(record.edits)) {
+		record.edits = record.edits.map(normalizeEditItem);
+	}
+
 	return record;
 }

package/src/replace.ts

@@ -47,10 +47,10 @@ const newLinesSchema = Type.Array(Type.String(), {
 		"literal replacement file content, one string per line. Must not include the HASH│ prefix from read output.",
 });
 
-const oldRangeSchema = Type.Array(
+const hashRangeInclSchema = Type.Array(
 	Type.String({ description: "anchor (3-char HASH)" }),
 	{
-		description: "inclusive line range to replace [start_hash, end_hash]. Each element must be the 3-character hash anchor only; do not include the │ separator or line content.",
+		description: "inclusive hash range to replace [start_hash, end_hash]. Each element must be the 3-character hash anchor only; do not include the │ separator or line content.",
 		minItems: 2,
 		maxItems: 2,
 	},
@@ -58,7 +58,7 @@ const oldRangeSchema = Type.Array(
 
 const editItemSchema = Type.Object(
 	{
-		old_range: oldRangeSchema,
+		hash_range_incl: hashRangeInclSchema,
 		new_lines: newLinesSchema,
 	},
 	{ additionalProperties: false },
@@ -102,7 +102,7 @@ export function assertReq(
 	for (const legacyKey of ["oldText", "newText", "old_text", "new_text", "start", "end", "lines"]) {
 		if (has(request, legacyKey)) {
 			throw new Error(
-				`[E_LEGACY_SHAPE] "${legacyKey}" is not supported. Use {old_range: ["<START>", "<END>"], new_lines: [...]}.`
+				`[E_LEGACY_SHAPE] "${legacyKey}" is not supported. Use {hash_range_incl: ["<START>", "<END>"], new_lines: [...]}.`
 			);
 		}
 	}
@@ -114,7 +114,7 @@ export function assertReq(
 	}
 
 	if (has(request, "edits") && !Array.isArray(request.edits)) {
-		throw new Error('[E_BAD_SHAPE] Edit request requires an "edits" array when provided. Each edit is { old_range: ["<START>", "<END>"], new_lines: [...] }.');
+		throw new Error('[E_BAD_SHAPE] Edit request requires an "edits" array when provided. Each edit is { hash_range_incl: ["<START>", "<END>"], new_lines: [...] }.');
 	}
 }
 
@@ -415,4 +415,4 @@ const toolDef: ToolDef = {
 
 export function regReplace(pi: ExtensionAPI): void {
 	pi.registerTool(toolDef);
-}
+}