@oh-my-pi/pi-coding-agent 13.0.0 → 13.0.1

package/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 ## [Unreleased]
 
+## [13.0.1] - 2026-02-22
+### Changed
+
+- Simplified hashline edit schema to use unified `first`/`last` anchor fields instead of operation-specific field names (`tag`, `before`, `after`)
+- Improved resilience of anchor resolution to degrade gracefully when anchors are missing or invalid, allowing edits to proceed with available anchors
+- Updated hashline tool documentation to reflect new unified anchor syntax across all operations (replace, append, prepend, insert)
+
 ## [13.0.0] - 2026-02-22
 ### Added
 

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "13.0.0",
+	"version": "13.0.1",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://github.com/can1357/oh-my-pi",
 	"author": "Can Boluk",
@@ -41,12 +41,12 @@
 	},
 	"dependencies": {
 		"@mozilla/readability": "^0.6",
-		"@oh-my-pi/omp-stats": "13.0.0",
-		"@oh-my-pi/pi-agent-core": "13.0.0",
-		"@oh-my-pi/pi-ai": "13.0.0",
-		"@oh-my-pi/pi-natives": "13.0.0",
-		"@oh-my-pi/pi-tui": "13.0.0",
-		"@oh-my-pi/pi-utils": "13.0.0",
+		"@oh-my-pi/omp-stats": "13.0.1",
+		"@oh-my-pi/pi-agent-core": "13.0.1",
+		"@oh-my-pi/pi-ai": "13.0.1",
+		"@oh-my-pi/pi-natives": "13.0.1",
+		"@oh-my-pi/pi-tui": "13.0.1",
+		"@oh-my-pi/pi-utils": "13.0.1",
 		"@sinclair/typebox": "^0.34",
 		"@xterm/headless": "^6.0",
 		"ajv": "^8.18",

package/src/patch/index.ts

@@ -168,24 +168,6 @@ export function stripNewLinePrefixes(lines: string[]): string[] {
 	});
 }
 
-const hashlineReplaceContentFormat = (kind: string) =>
-	Type.Union([
-		Type.Null(),
-		Type.Array(Type.String(), { description: `${kind} lines` }),
-		Type.String({ description: `${kind} line` }),
-	]);
-
-const hashlineInsertContentFormat = (kind: string) =>
-	Type.Union([
-		Type.Array(Type.String(), { description: `${kind} lines`, minItems: 1 }),
-		Type.String({ description: `${kind} line`, minLength: 1 }),
-	]);
-
-const hashlineTagFormat = (what: string) =>
-	Type.String({
-		description: `Tag identifying the ${what} in "LINE#ID" format`,
-	});
-
 export function hashlineParseContent(edit: string | string[] | null): string[] {
 	if (edit === null) return [];
 	if (Array.isArray(edit)) return edit;
@@ -194,61 +176,23 @@ export function hashlineParseContent(edit: string | string[] | null): string[] {
 	if (lines[lines.length - 1].trim() === "") return lines.slice(0, -1);
 	return lines;
 }
-const hashlineReplaceTagEditSchema = Type.Object(
-	{
-		op: Type.Literal("replace"),
-		tag: hashlineTagFormat("line being replaced"),
-		content: hashlineReplaceContentFormat("Replacement"),
-	},
-	{ additionalProperties: false },
-);
 
-const hashlineAppendEditSchema = Type.Object(
+const hashlineEditSpecSchema = Type.Object(
 	{
-		op: Type.Literal("append"),
-		after: Type.Optional(hashlineTagFormat("line after which to append")),
-		content: hashlineInsertContentFormat("Appended"),
-	},
-	{ additionalProperties: false },
-);
-
-const hashlinePrependEditSchema = Type.Object(
-	{
-		op: Type.Literal("prepend"),
-		before: Type.Optional(hashlineTagFormat("line before which to prepend")),
-		content: hashlineInsertContentFormat("Prepended"),
-	},
-	{ additionalProperties: false },
-);
-
-const hashlineReplaceRangeEditSchema = Type.Object(
-	{
-		op: Type.Literal("replace"),
-		first: hashlineTagFormat("first line"),
-		last: hashlineTagFormat("last line"),
-		content: hashlineReplaceContentFormat("Replacement"),
-	},
-	{ additionalProperties: false },
-);
-
-const hashlineInsertEditSchema = Type.Object(
-	{
-		op: Type.Literal("insert"),
-		before: Type.Optional(hashlineTagFormat("line before which to insert")),
-		after: Type.Optional(hashlineTagFormat("line after which to insert")),
-		content: hashlineInsertContentFormat("Inserted"),
+		op: StringEnum(["replace", "append", "prepend", "insert"], {
+			description: "Operation type",
+		}),
+		first: Type.Optional(Type.String({ description: 'First/start anchor tag in "LINE#ID" format' })),
+		last: Type.Optional(Type.String({ description: 'Last/end anchor tag in "LINE#ID" format' })),
+		content: Type.Union([
+			Type.Null(),
+			Type.Array(Type.String(), { description: "Content lines" }),
+			Type.String({ description: "Content line" }),
+		]),
 	},
 	{ additionalProperties: false },
 );
 
-const hashlineEditSpecSchema = Type.Union([
-	hashlineReplaceTagEditSchema,
-	hashlineReplaceRangeEditSchema,
-	hashlineAppendEditSchema,
-	hashlinePrependEditSchema,
-	hashlineInsertEditSchema,
-]);
-
 const hashlineEditSchema = Type.Object(
 	{
 		path: Type.String({ description: "File path (relative or absolute)" }),
@@ -265,6 +209,88 @@ const hashlineEditSchema = Type.Object(
 export type HashlineToolEdit = Static<typeof hashlineEditSpecSchema>;
 export type HashlineParams = Static<typeof hashlineEditSchema>;
 
+// ═══════════════════════════════════════════════════════════════════════════
+// Resilient anchor resolution
+// ═══════════════════════════════════════════════════════════════════════════
+
+/**
+ * Map flat tool-schema edits (first/last) into typed HashlineEdit objects.
+ *
+ * Resilient: as long as at least one anchor exists, we execute.
+ * - replace + first only → single-line replace (tag = first)
+ * - replace + first + last → range replace
+ * - append + first or last → append after that anchor
+ * - prepend + first or last → prepend before that anchor
+ * - insert + first + last → insert between them
+ * - insert + one anchor → degrade to append/prepend
+ * - no anchors → file-level append/prepend (only for those ops)
+ *
+ * Unknown ops default to "replace".
+ */
+function resolveEditAnchors(edits: HashlineToolEdit[]): HashlineEdit[] {
+	const result: HashlineEdit[] = [];
+	for (const edit of edits) {
+		const content = hashlineParseContent(edit.content);
+		const first = edit.first ? tryParseTag(edit.first) : undefined;
+		const last = edit.last ? tryParseTag(edit.last) : undefined;
+
+		// Normalize op — default unknown values to "replace"
+		const op = edit.op === "append" || edit.op === "prepend" || edit.op === "insert" ? edit.op : "replace";
+
+		switch (op) {
+			case "replace": {
+				if (first && last) {
+					result.push({ op: "replace", first, last, content });
+				} else if (first) {
+					result.push({ op: "replace", tag: first, content });
+				} else if (last) {
+					result.push({ op: "replace", tag: last, content });
+				} else {
+					throw new Error("Replace requires at least one anchor (first or last).");
+				}
+				break;
+			}
+			case "append": {
+				// Prefer first as the "after" anchor; fall back to last
+				const anchor = first ?? last;
+				result.push({ op: "append", ...(anchor ? { after: anchor } : {}), content });
+				break;
+			}
+			case "prepend": {
+				// Prefer last as the "before" anchor; fall back to first
+				const anchor = last ?? first;
+				result.push({ op: "prepend", ...(anchor ? { before: anchor } : {}), content });
+				break;
+			}
+			case "insert": {
+				if (first && last) {
+					result.push({ op: "insert", after: first, before: last, content });
+				} else if (first) {
+					// Degrade: insert after first
+					result.push({ op: "append", after: first, content });
+				} else if (last) {
+					// Degrade: insert before last
+					result.push({ op: "prepend", before: last, content });
+				} else {
+					// No anchors — append to end
+					result.push({ op: "append", content });
+				}
+				break;
+			}
+		}
+	}
+	return result;
+}
+
+/** Parse a tag, returning undefined instead of throwing on garbage. */
+function tryParseTag(raw: string): LineTag | undefined {
+	try {
+		return parseTag(raw);
+	} catch {
+		return undefined;
+	}
+}
+
 // ═══════════════════════════════════════════════════════════════════════════
 // LSP FileSystem for patch mode
 // ═══════════════════════════════════════════════════════════════════════════
@@ -522,24 +548,15 @@ export class EditTool implements AgentTool<TInput> {
 			if (!(await file.exists())) {
 				const content: string[] = [];
 				for (const edit of edits) {
-					switch (edit.op) {
-						case "append": {
-							if (edit.after) {
-								throw new Error(`File not found: ${path}`);
-							}
-							content.push(...hashlineParseContent(edit.content));
-							break;
-						}
-						case "prepend": {
-							if (edit.before) {
-								throw new Error(`File not found: ${path}`);
-							}
+					// For file creation, only anchorless appends/prepends are valid
+					if ((edit.op === "append" || edit.op === "prepend") && !edit.first && !edit.last) {
+						if (edit.op === "prepend") {
 							content.unshift(...hashlineParseContent(edit.content));
-							break;
-						}
-						default: {
-							throw new Error(`File not found: ${path}`);
+						} else {
+							content.push(...hashlineParseContent(edit.content));
 						}
+					} else {
+						throw new Error(`File not found: ${path}`);
 					}
 				}
 				await file.write(content.join("\n"));
@@ -553,74 +570,7 @@ export class EditTool implements AgentTool<TInput> {
 				};
 			}
 
-			const anchorEdits: HashlineEdit[] = [];
-			for (const edit of edits) {
-				switch (edit.op) {
-					case "replace": {
-						if ("tag" in edit) {
-							anchorEdits.push({
-								op: "replace",
-								tag: parseTag(edit.tag),
-								content: hashlineParseContent(edit.content),
-							});
-						} else {
-							anchorEdits.push({
-								op: "replace",
-								first: parseTag(edit.first),
-								last: parseTag(edit.last),
-								content: hashlineParseContent(edit.content),
-							});
-						}
-						break;
-					}
-					case "append": {
-						const { after, content } = edit;
-						anchorEdits.push({
-							op: "append",
-							...(after ? { after: parseTag(after) } : {}),
-							content: hashlineParseContent(content),
-						});
-						break;
-					}
-					case "prepend": {
-						const { before, content } = edit;
-						anchorEdits.push({
-							op: "prepend",
-							...(before ? { before: parseTag(before) } : {}),
-							content: hashlineParseContent(content),
-						});
-						break;
-					}
-					case "insert": {
-						const { before, after, content } = edit;
-						if (before && !after) {
-							anchorEdits.push({
-								op: "prepend",
-								before: parseTag(before),
-								content: hashlineParseContent(content),
-							});
-						} else if (after && !before) {
-							anchorEdits.push({
-								op: "append",
-								after: parseTag(after),
-								content: hashlineParseContent(content),
-							});
-						} else if (before && after) {
-							anchorEdits.push({
-								op: "insert",
-								before: parseTag(before),
-								after: parseTag(after),
-								content: hashlineParseContent(content),
-							});
-						} else {
-							throw new Error(`Insert must have both before and after tags.`);
-						}
-						break;
-					}
-					default:
-						throw new Error(`Invalid edit operation: ${JSON.stringify(edit)}`);
-				}
-			}
+			const anchorEdits = resolveEditAnchors(edits);
 
 			const rawContent = await file.text();
 			const { bom, text: content } = stripBom(rawContent);

package/src/patch/shared.ts

@@ -86,12 +86,12 @@ interface EditRenderArgs {
 	edits?: HashlineEditPreview[];
 }
 
-type HashlineEditPreview =
-	| { op: "replace"; tag: string; content: string[] }
-	| { op: "replace"; first: string; last: string; content: string[] }
-	| { op: "append"; after?: string; content: string[] }
-	| { op: "prepend"; before?: string; content: string[] }
-	| { op: "insert"; before: string; after: string; content: string[] };
+type HashlineEditPreview = {
+	op: string;
+	first?: string;
+	last?: string;
+	content: string | string[] | null;
+};
 
 /** Extended context for edit tool rendering */
 export interface EditRenderContext {
@@ -166,26 +166,18 @@ function formatStreamingHashlineEdits(edits: unknown[], uiTheme: Theme): string
 
 		const contentLines = Array.isArray(editRecord.content) ? (editRecord.content as string[]).join("\n") : "";
 
-		// replace with tag (single line)
-		if ("tag" in editRecord && !("first" in editRecord)) {
-			const tag = typeof editRecord.tag === "string" ? editRecord.tag : "…";
-			return { srcLabel: `• line ${tag}`, dst: contentLines };
-		}
-		// replace with first..last (range)
-		if ("first" in editRecord || "last" in editRecord) {
-			const first = typeof editRecord.first === "string" ? editRecord.first : "…";
-			const last = typeof editRecord.last === "string" ? editRecord.last : "…";
-			return { srcLabel: `• range ${first}..${last}`, dst: contentLines };
+		const op = typeof editRecord.op === "string" ? editRecord.op : "?";
+		const first = typeof editRecord.first === "string" ? editRecord.first : undefined;
+		const last = typeof editRecord.last === "string" ? editRecord.last : undefined;
+
+		if (first && last && first !== last) {
+			return { srcLabel: `\u2022 ${op} ${first}..${last}`, dst: contentLines };
 		}
-		// append/prepend/insert
-		if ("before" in editRecord || "after" in editRecord) {
-			const after = typeof editRecord.after === "string" ? editRecord.after : undefined;
-			const before = typeof editRecord.before === "string" ? editRecord.before : undefined;
-			const refs = [after, before].filter(Boolean).join("..") || "…";
-			return { srcLabel: `• insert ${refs}`, dst: contentLines };
+		const anchor = first ?? last;
+		if (anchor) {
+			return { srcLabel: `\u2022 ${op} ${anchor}`, dst: contentLines };
 		}
-
-		return { srcLabel: "• (incomplete edit)", dst: "" };
+		return { srcLabel: `\u2022 ${op} (file-level)`, dst: contentLines };
 	}
 }
 function formatMetadataLine(lineCount: number | null, language: string | undefined, uiTheme: Theme): string {

package/src/prompts/compaction/compaction-short-summary.md

@@ -5,5 +5,5 @@ Rules:
 - MUST describe the changes made, not the process
 - MUST NOT mention running tests, builds, or other validation steps
 - MUST NOT explain what the user asked for
-- MUST write in first person (I added..., I fixed...)
+- MUST write in first person (I added…, I fixed…)
 - MUST NOT ask questions

package/src/prompts/system/agent-creation-architect.md

@@ -33,17 +33,17 @@ When a user describes what they want an agent to do, you will:
       <function call omitted for brevity only for this example>
       <commentary>
       Since a significant piece of code was written, use the {{TASK_TOOL_NAME}} tool to launch the test-runner agent to run the tests.
-</commentary>
+      </commentary>
       assistant: "Now let me use the test-runner agent to run the tests"
-</example>
+      </example>
     - <example>
       Context: User is creating an agent to respond to the word "hello" with a friendly jok.
       user: "Hello"
       assistant: "I'm going to use the {{TASK_TOOL_NAME}} tool to launch the greeting-responder agent to respond with a friendly joke"
       <commentary>
       Since the user is greeting, use the greeting-responder agent to respond with a friendly joke.
-</commentary>
-</example>
+      </commentary>
+      </example>
   - If the user mentioned or implied that the agent should be used proactively, you SHOULD include examples of this.
 - NOTE: You MUST ensure that in the examples, you are making the assistant use the Agent tool and MUST NOT simply respond directly to the task.
 

package/src/prompts/tools/hashline.md

@@ -12,14 +12,15 @@ Apply precise file edits using `LINE#ID` tags, anchoring to the file content.
 </workflow>
 
 <operations>
+Every edit has `op`, `first` (start anchor), `last` (end anchor), and `content`. All anchors use `"LINE#ID"` format.
 - **Line or range replace/delete**
-  - `{ op: "replace", tag: "N#ID", content: […] }`
-  - `{ op: "replace", first: "N#ID", last: "N#ID", content: […] }`
+  - `{ op: "replace", first: "N#ID", content: […] }` (single line)
+  - `{ op: "replace", first: "N#ID", last: "N#ID", content: […] }` (range)
   - Use for swaps, block rewrites, or deleting a full span (`content: null`).
 - **Insert** (new content)
-  - `{ op: "prepend", before: "N#ID", content: […] }` or `{ op: "prepend", content: […] }` (no `before` = insert at beginning of file)
-  - `{ op: "append", after: "N#ID", content: […] }` or `{ op: "append", content: […] }` (no `after` = insert at end of file)
-  - `{ op: "insert", after: "N#ID", before: "N#ID", content: […] }` (between adjacent anchors; safest for blocks)
+  - `{ op: "prepend", last: "N#ID", content: […] }` or `{ op: "prepend", content: […] }` (no anchor = insert at beginning of file)
+  - `{ op: "append", first: "N#ID", content: […] }` or `{ op: "append", content: […] }` (no anchor = insert at end of file)
+  - `{ op: "insert", first: "N#ID", last: "N#ID", content: […] }` (between adjacent anchors; safest for blocks)
 - **File-level controls**
   - `{ delete: true, edits: [] }` deletes the file (cannot be combined with `rename`).
   - `{ rename: "new/path.ts", edits: […] }` writes result to new path and removes old path.
@@ -62,7 +63,7 @@ Apply precise file edits using `LINE#ID` tags, anchoring to the file content.
 ```
 ```
 op: "replace"
-tag: "{{hlineref 23 "  const timeout: number = 5000;"}}"
+first: "{{hlineref 23 "  const timeout: number = 5000;"}}"
 content: ["  const timeout: number = 30_000;"]
 ```
 </example>
@@ -74,7 +75,7 @@ content: ["  const timeout: number = 30_000;"]
 ```
 ```
 op: "replace"
-tag: "{{hlineref 7 "// @ts-ignore"}}"
+first: "{{hlineref 7 "// @ts-ignore"}}"
 content: null
 ```
 </example>
@@ -85,7 +86,7 @@ content: null
 ```
 ```
 op: "replace"
-tag: "{{hlineref 14 "  placeholder: \"DO NOT SHIP\","}}"
+first: "{{hlineref 14 "  placeholder: \"DO NOT SHIP\","}}"
 content: [""]
 ```
 </example>
@@ -127,10 +128,10 @@ content: null
 ```
 ```
 op: "prepend"
-before: "{{hlineref 1 "import * as fs from \"node:fs/promises\";"}}"
+last: "{{hlineref 1 "import * as fs from \"node:fs/promises\";"}}"
 content: ["import * as os from \"node:os\";"]
 ```
-Use `before` for anchored insertion before a specific line. Omit `before` to prepend at BOF.
+Use `last` for anchored insertion before a specific line. Omit anchor to prepend at BOF.
 </example>
 
 <example name="append at end of file">
@@ -139,10 +140,10 @@ Use `before` for anchored insertion before a specific line. Omit `before` to pre
 ```
 ```
 op: "append"
-after: "{{hlineref 260 "export { serialize, deserialize };"}}"
+first: "{{hlineref 260 "export { serialize, deserialize };"}}"
 content: ["export { validate };"]
 ```
-Use `after` for anchored insertion after a specific line. Omit `after` to append at EOF.
+Use `first` for anchored insertion after a specific line. Omit anchor to append at EOF.
 </example>
 
 <example name="add an entry between known siblings">
@@ -152,8 +153,8 @@ Use `after` for anchored insertion after a specific line. Omit `after` to append
 ```
 ```
 op: "insert"
-after: "{{hlineref 44 "  \"build\": \"bun run compile\","}}"
-before: "{{hlineref 45 "  \"test\": \"bun test\""}}"
+first: "{{hlineref 44 "  \"build\": \"bun run compile\","}}"
+last: "{{hlineref 45 "  \"test\": \"bun test\""}}"
 content: ["  \"lint\": \"biome check\","]
 ```
 Dual anchors pin the insert to exactly one gap, preventing drift from edits elsewhere in the file. **Always prefer dual anchors when both boundaries are content lines.**
@@ -168,10 +169,10 @@ Dual anchors pin the insert to exactly one gap, preventing drift from edits else
 ```
 ```
 op: "insert"
-before: "{{hlineref 103 "export function serialize(data: unknown): string {"}}"
+last: "{{hlineref 103 "export function serialize(data: unknown): string {"}}"
 content: ["function validate(data: unknown): boolean {", "  return data != null && typeof data === \"object\";", "}", ""]
 ```
-The trailing `""` in `content` preserves the blank-line separator. **Anchor to the structural line (`export function ...`), not the blank line above it** — blank lines are ambiguous and may be added or removed by other edits.
+The trailing `""` in `content` preserves the blank-line separator. **Anchor to the structural line (`export function …`), not the blank line above it** — blank lines are ambiguous and may be added or removed by other edits.
 </example>
 
 <example name="file delete">
@@ -192,14 +193,14 @@ edits: […]
 <example name="anti-pattern: anchoring to whitespace">
 Bad — tags to a blank line; fragile if blank lines shift:
 ```
-after: "{{hlineref 102 ""}}"
+first: "{{hlineref 102 ""}}"
 content: ["function validate() {", …, "}"]
 ```
 
 Good — anchors to the structural target:
 
 ```
-before: "{{hlineref 103 "export function serialize(data: unknown): string {"}}"
+last: "{{hlineref 103 "export function serialize(data: unknown): string {"}}"
 content: ["function validate() {", …, "}"]
 ```
 </example>
@@ -207,7 +208,7 @@ content: ["function validate() {", …, "}"]
 <critical>
 You MUST ensure:
 - Payload shape is `{ "path": string, "edits": [operation, …], "delete"?: boolean, "rename"?: string }`
-- Every edit MUST match exactly one variant
+- Each edit has `op`, optional `first`/`last` anchors, and `content`
 - Every tag MUST be copied EXACTLY from a tool result as `N#ID`
 - Scope MUST be minimal and formatting MUST be preserved except targeted token changes
 </critical>

package/src/prompts/tools/patch.md

@@ -60,7 +60,7 @@ edit {"path":"src/app.py","op":"update","diff":"@@ def greet():\n def greet():\n
 </example>
 
 <example name="rename">
-edit {"path":"src/app.py","op":"update","rename":"src/main.py","diff":"@@\n ...\n"}
+edit {"path":"src/app.py","op":"update","rename":"src/main.py","diff":"@@\n …\n"}
 </example>
 
 <example name="delete">

package/src/prompts/tools/python.md

@@ -36,8 +36,8 @@ All helpers auto-print results and return values for chaining.
 <output>
 User sees output like Jupyter notebook; rich displays render fully:
 - `display(JSON(data))` → interactive JSON tree
-- `display(HTML(...))` → rendered HTML
-- `display(Markdown(...))` → formatted markdown
+- `display(HTML(…))` → rendered HTML
+- `display(Markdown(…))` → formatted markdown
 - `plt.show()` → inline figures
   **You will see object repr** (e.g., `<IPython.core.display.JSON object>`). Trust `display()`; you MUST NOT assume user sees only repr.
 </output>

package/src/prompts/tools/replace.md

@@ -24,7 +24,7 @@ For position-addressed or pattern-addressed changes, bash more efficient:
 
 |Operation|Command|
 |---|---|
-|Append to file|`cat >> file <<'EOF'`...`EOF`|
+|Append to file|`cat >> file <<'EOF'`…`EOF`|
 |Prepend to file|`{ cat - file; } <<'EOF' > tmp && mv tmp file`|
 |Delete lines N-M|`sed -i 'N,Md' file`|
 |Insert after line N|`sed -i 'Na\text' file`|

package/src/prompts/tools/task.md

@@ -19,7 +19,6 @@ Subagents have no access to your conversation history. They don't know:
 Subagents CAN grep the parent conversation file for supplementary details.
 
 For large intermediate outputs (long traces, JSON payloads, temporary analysis snapshots), you SHOULD write them to `local://<path>` and pass the path in task context instead of inlining bulky text.
----
 
 ## Parameters
 
@@ -128,9 +127,9 @@ Use structure every assignment:
 - Patterns/APIs to use; reference files if applicable
 
 ## Edge Cases / Don't Break
-- Tricky case 1: ...
-- Tricky case 2: ...
-- Existing behavior must survive: ...
+- Tricky case 1: …
+- Tricky case 2: …
+- Existing behavior must survive: …
 
 ## Acceptance (task-local)
 - Expected behavior or observable result
@@ -145,11 +144,11 @@ Use structure every assignment:
 - "Migrate to N-API."
 - "Fix the bug in streaming."
 - "Update all constructors in `src/**/*.ts`."
-**Vague context** — forces agent invent conventions:
+  **Vague context** — forces agent invent conventions:
 - "Use existing patterns."
 - "Follow conventions."
 - "No WASM."
-**Redundant context** — wastes tokens repeating what subagents already have:
+  **Redundant context** — wastes tokens repeating what subagents already have:
 - Restating AGENTS.md rules (coding style, import conventions, formatting commands, logger usage, etc.)
 - Repeating project constraints from context files
 - Listing tool/framework preferences already documented in the repo
@@ -192,9 +191,9 @@ First style wastes your time, brittle if code shifts. Second gives agent room to
 <example type="bad" label="Duplicated context inflates tokens">
 <tasks>
   <task name="Grep">
-    <description>Port grep module from WASM to N-API...</description>
-    <assignment>Port grep module from WASM to N-API... (same blob repeated)</assignment>
-</task>
+    <description>Port grep module from WASM to N-API…</description>
+    <assignment>Port grep module from WASM to N-API… (same blob repeated)</assignment>
+  </task>
 </tasks>
 </example>
 
@@ -210,7 +209,7 @@ Do not touch TS bindings or downstream consumers — separate phase.
 - MUST use `#[napi]` attribute macro on all exports
 - MUST return `napi::Result<T>` for fallible ops; never panic
 - MUST use `spawn_blocking` for filesystem I/O or >1ms work
-...
+  …
 
 ## Acceptance (global)
 - Caller verifies after all tasks: `cargo test -p pi-natives` and `cargo build -p pi-natives` with no warnings
@@ -232,17 +231,17 @@ Do not touch TS bindings or downstream consumers — separate phase.
 
 ## Acceptance (task-local)
 - Three functions exported with correct signatures (caller verifies build after all tasks)
-</assignment>
-</task>
+    </assignment>
+  </task>
 
   <task name="PortHighlight">
     <description>Port highlight module to N-API</description>
     <assignment>
 ## Target
 - Files: `src/highlight.rs`, `src/lib.rs` (registration only)
-...
-</assignment>
-</task>
+  …
+    </assignment>
+  </task>
 </tasks>
 </example>
 ---
@@ -254,7 +253,7 @@ Each task MUST have small, well-defined scope — **at most 3–5 files**.
 - File paths use globs (`src/**/*.ts`) instead of explicit names
 - Assignment says "update all" / "migrate everything" / "refactor across"
 - Scope covers entire package or directory tree
-**Fix:** You MUST enumerate files first (grep/glob discovery), then fan out one task per file or small cluster.
+  **Fix:** You MUST enumerate files first (grep/glob discovery), then fan out one task per file or small cluster.
 ---
 
 ## Parallelization