@oh-my-pi/pi-coding-agent 14.5.2 → 14.5.5

package/CHANGELOG.md

@@ -2,6 +2,76 @@
 
 ## [Unreleased]
 
+## [14.5.5] - 2026-04-29
+### Breaking Changes
+
+- Rejected atom diffs with unrecognized operations (including lone '-' lines) by throwing parse errors instead of treating them as inserts
+
+### Added
+
+- Added duplicate-line post-edit detection that warns on newly introduced adjacent identical lines and auto-removes one duplicate when bracket-balance is restored
+- Added a warning when suspicious adjacent duplicates are introduced after edits so users can review potential stale-line issues
+
+### Changed
+
+- Changed anchor rebase handling to fail when multiple mutating anchors would need auto-rebase, preventing silent misapplied contiguous block rewrites
+
+### Fixed
+
+- Fixed bracket-corruption caused by botched block rewrites by automatically removing a newly introduced duplicate adjacent line when removing it restores the original `{}`, `()`, and `[]` balance and by warning when automatic removal is unsafe
+
+## [14.5.4] - 2026-04-28
+### Breaking Changes
+
+- Changed the `atom` edit mode from JSON `{ path, edits }` calls to the compact file-oriented `input` patch language that was previously exposed as `atomd`; `atomd` is no longer a separate edit variant
+- Renamed MCP tool identifiers from the `mcp_<server>_<tool>` format to `mcp__<server>_<tool>` so custom tool names, active tool lists, and persisted MCP selections must be updated to the new prefix
+- Renamed the built-in content-search tool from `grep` to `search`, including SDK/tool event names and settings keys (`search.enabled`, `search.contextBefore`, `search.contextAfter`), so integrations using `grep` and `grep.*` references must be updated
+
+### Added
+
+- Added internal URL support to the `search` tool, allowing `artifact://`-style paths that resolve to local files to be searched directly
+- Added IRC relay observation in the main agent UI so every IRC exchange between agents is rendered in the main transcript, even when the main agent is not a direct participant
+- Added stateful `href`/`hrefr` prompt helpers that can reuse anchors remembered from prior `hline` helper calls
+
+### Changed
+
+- Changed file-path rendering across search, find, AST, LSP, and related edit outputs to display targets as cwd-relative paths when they resolve inside the working directory and keep absolute paths for files outside the cwd
+- Changed system prompt guidance so in-cwd tool paths must be passed as cwd-relative paths and absolute paths only for out-of-cwd targets or `~` expansion
+- Updated `edit` streaming diff previews for `patch`, `replace`, and `hashline` to produce a single request-level preview for the new single-file `path` mode
+- Bumped default `read.defaultLimit` from 300 to 500 lines, and scaled the read tool's byte budget with the line limit (`max(50KB, lines * 512)`) so the configured line count is no longer truncated by the shared 50KB cap
+
+### Fixed
+
+- Fixed atom edit streaming previews to use atom headers for file names instead of apply_patch parsing errors.
+- Fixed collapsed search result rendering so summary and truncation rows stay within the collapsed output budget
+- Updated search path handling to support path lists and internal file paths while preserving previous search behavior
+
+## [14.5.3] - 2026-04-27
+
+### Added
+
+- Added bracketed `loc` forms `(anchor)`, `[anchor]`, `[anchor`, `(anchor`, `anchor]`, and `anchor)` to `atom` `splice` editing so a single anchor can target a block body, whole node, or partial node region
+- Added automatic block-delimiter inference for block splices using file extension, defaulting to `{` and using `(` for Lisp-family files
+- Added optional `pre`/`post` arguments to the `href` prompt helper so hashline references can be wrapped as bracketed or parenthesized anchors
+- Added destination-aware indent handling for block replacements by detecting file indent style and reapplying tabs/spaces to spliced body text
+
+### Changed
+
+- Changed bracketed atom locators to be `splice`-only and reject `pre`, `post`, or `sed` on region locators
+- Changed `applyAtomEdits` to forbid mixing `splice_block` with other anchor-scoped edit verbs in one call
+- Changed `splice_block` resolution behavior to include selected block range and enclosing-count context in warning output
+- Changed balanced-block parsing to support `kind` selection (`{`, `(`, `[`), nesting depth, and safer same-line enclosing selection
+
+### Removed
+
+- Removed the `sed` `F` option for literal matching; `sed` now accepts only `pat`, `rep`, and optional `g`, with `F`-style literal matching no longer supported
+
+### Fixed
+
+- Fixed `splice_block` multi-line replacements to replace the exact target region and avoid duplicate braces or duplicated signature lines from bare-anchor `splice` attempts
+- Fixed false-positive “unbalanced” replacement-body warnings caused by braces in regex/string/comment text by skipping those constructs during block scanning
+- Fixed `splice_block` for same-line `(` bodies so inline call sites like `int(port)` can be replaced correctly
+
 ## [14.5.2] - 2026-04-26
 ### Breaking Changes
 

package/examples/extensions/plan-mode.ts

@@ -22,7 +22,7 @@ import type { ExtensionAPI, ExtensionContext } from "@oh-my-pi/pi-coding-agent";
 import { Key } from "@oh-my-pi/pi-tui";
 
 // Read-only tools for plan mode
-const PLAN_MODE_TOOLS = ["read", "bash", "grep", "find", "ls"];
+const PLAN_MODE_TOOLS = ["read", "bash", "search", "find"];
 
 // Full set of tools for normal mode
 const NORMAL_MODE_TOOLS = ["read", "bash", "edit", "write"];

package/examples/sdk/README.md

@@ -69,7 +69,7 @@ const { session } = await createAgentSession({
 });
 
 // Read-only tools
-const { session } = await createAgentSession({ toolNames: ["read", "grep", "find", "ls"], authStorage, modelRegistry });
+const { session } = await createAgentSession({ toolNames: ["read", "search", "find"], authStorage, modelRegistry });
 
 // In-memory
 const { session } = await createAgentSession({

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "14.5.2",
+	"version": "14.5.5",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://github.com/can1357/oh-my-pi",
 	"author": "Can Boluk",
@@ -46,12 +46,12 @@
 	"dependencies": {
 		"@agentclientprotocol/sdk": "0.20.0",
 		"@mozilla/readability": "^0.6.0",
-		"@oh-my-pi/omp-stats": "14.5.2",
-		"@oh-my-pi/pi-agent-core": "14.5.2",
-		"@oh-my-pi/pi-ai": "14.5.2",
-		"@oh-my-pi/pi-natives": "14.5.2",
-		"@oh-my-pi/pi-tui": "14.5.2",
-		"@oh-my-pi/pi-utils": "14.5.2",
+		"@oh-my-pi/omp-stats": "14.5.5",
+		"@oh-my-pi/pi-agent-core": "14.5.5",
+		"@oh-my-pi/pi-ai": "14.5.5",
+		"@oh-my-pi/pi-natives": "14.5.5",
+		"@oh-my-pi/pi-tui": "14.5.5",
+		"@oh-my-pi/pi-utils": "14.5.5",
 		"@puppeteer/browsers": "^2.13.0",
 		"@sinclair/typebox": "^0.34.49",
 		"@xterm/headless": "^6.0.0",

package/src/config/prompt-templates.ts

@@ -43,21 +43,119 @@ function formatHashlineRef(lineNum: unknown, content: unknown): { num: number; t
 	return { num, text, ref };
 }
 
+interface HashlineHelperRef {
+	line: number;
+	ref: string;
+}
+
+interface HashlineHelperState {
+	last?: HashlineHelperRef;
+	byLine: Map<number, HashlineHelperRef>;
+}
+
+const HASHLINE_HELPER_STATE = Symbol("hashlineHelperState");
+
+interface HashlineHelperStateHolder {
+	[HASHLINE_HELPER_STATE]?: HashlineHelperState;
+}
+
+function isHelperOptions(value: unknown): value is prompt.HelperOptions {
+	return typeof value === "object" && value !== null && "hash" in value;
+}
+
+function splitHelperArgs(args: unknown[]): { positional: unknown[]; options?: prompt.HelperOptions } {
+	const maybeOptions = args.at(-1);
+	if (!isHelperOptions(maybeOptions)) return { positional: args };
+	return { positional: args.slice(0, -1), options: maybeOptions };
+}
+
+function getHashlineHelperState(context: unknown, options: prompt.HelperOptions | undefined): HashlineHelperState {
+	const data = options?.data;
+	const root = data?.root;
+	const holderTarget = data && typeof data === "object" ? data : root && typeof root === "object" ? root : context;
+	if (!holderTarget || typeof holderTarget !== "object") {
+		throw new Error("hashline prompt helpers require an object render context");
+	}
+
+	const holder = holderTarget as HashlineHelperStateHolder;
+	if (!holder[HASHLINE_HELPER_STATE]) {
+		holder[HASHLINE_HELPER_STATE] = { byLine: new Map() };
+	}
+	return holder[HASHLINE_HELPER_STATE];
+}
+
+function isLineNumberArg(value: unknown): boolean {
+	const num = typeof value === "number" ? value : Number.parseInt(String(value), 10);
+	return Number.isFinite(num);
+}
+
+function rememberHashlineRef(state: HashlineHelperState, line: number, ref: string): void {
+	const entry = { line, ref };
+	state.last = entry;
+	state.byLine.set(line, entry);
+}
+
+function requireStoredHashlineRef(state: HashlineHelperState, lineArg?: unknown): string {
+	if (lineArg === undefined) {
+		if (!state.last) {
+			throw new Error("{{href}} requires a previous {{hline}} call in the same prompt render");
+		}
+		return state.last.ref;
+	}
+
+	const line = typeof lineArg === "number" ? lineArg : Number.parseInt(String(lineArg), 10);
+	const entry = state.byLine.get(line);
+	if (!entry) {
+		throw new Error(`{{href ${line}}} requires a previous {{hline ${line} ...}} call in the same prompt render`);
+	}
+	return entry.ref;
+}
+
+function wrapHashlineRef(ref: string, args: unknown[]): string {
+	const preStr = typeof args[0] === "string" ? args[0] : "";
+	const postStr = typeof args[1] === "string" ? args[1] : "";
+	return `${preStr}${ref}${postStr}`;
+}
+
+function resolveHashlineRef(state: HashlineHelperState, args: unknown[]): string {
+	if (args.length === 0) return requireStoredHashlineRef(state);
+	const [first, second, ...rest] = args;
+	if (isLineNumberArg(first)) {
+		if (second === undefined) return requireStoredHashlineRef(state, first);
+		const { ref } = formatHashlineRef(first, second);
+		return wrapHashlineRef(ref, rest);
+	}
+	return wrapHashlineRef(requireStoredHashlineRef(state), args);
+}
+
 /**
  * {{href lineNum "content"}} — compute a real hashline ref for prompt examples.
- * Returns `"lineNumBIGRAM"` (e.g., `"42nd"`) using the actual hash algorithm.
+ * {{href lineNum}} — quote the ref remembered by the earlier {{hline lineNum "..."}}
+ * {{href}} — quote the ref from the previous {{hline}} call.
+ * {{href "[" "]"}} — wrap the previous {{hline}} ref with pre/post chars.
+ * Returns `"lineNumBIGRAM"` (e.g., `"42nd"`), or `"[42nd]"` when pre/post are supplied.
  */
-prompt.registerHelper("href", (lineNum: unknown, content: unknown): string => {
-	const { ref } = formatHashlineRef(lineNum, content);
-	return JSON.stringify(ref);
+prompt.registerHelper("href", function (this: unknown, ...args: unknown[]): string {
+	const { positional, options } = splitHelperArgs(args);
+	const state = getHashlineHelperState(this, options);
+	return JSON.stringify(resolveHashlineRef(state, positional));
+});
+prompt.registerHelper("hrefr", function (this: unknown, ...args: unknown[]): string {
+	const { positional, options } = splitHelperArgs(args);
+	const state = getHashlineHelperState(this, options);
+	return resolveHashlineRef(state, positional);
 });
 
 /**
  * {{hline lineNum "content"}} — format a full read-style line with prefix.
  * Returns `"lineNumBIGRAM|content"` (pipe between anchor and content).
  */
-prompt.registerHelper("hline", (lineNum: unknown, content: unknown): string => {
-	const { ref, text } = formatHashlineRef(lineNum, content);
+prompt.registerHelper("hline", function (this: unknown, ...args: unknown[]): string {
+	const { positional, options } = splitHelperArgs(args);
+	const [lineNum, content] = positional;
+	const { num, ref, text } = formatHashlineRef(lineNum, content);
+	const state = getHashlineHelperState(this, options);
+	rememberHashlineRef(state, num, ref);
 	return `${ref}${HASHLINE_CONTENT_SEPARATOR}${text}`;
 });
 

package/src/config/settings-schema.ts

@@ -1,5 +1,6 @@
 import { THINKING_EFFORTS } from "@oh-my-pi/pi-ai";
 import { TASK_SIMPLE_MODES } from "../task/simple-mode";
+import { EDIT_MODES } from "../utils/edit-mode";
 
 /** Unified settings schema - single source of truth for all settings.
  * Unified settings schema - single source of truth for all settings.
@@ -160,8 +161,8 @@ export const DEFAULT_BASH_INTERCEPTOR_RULES: BashInterceptorRule[] = [
 	},
 	{
 		pattern: "^\\s*(grep|rg|ripgrep|ag|ack)\\s+",
-		tool: "grep",
-		message: "Use the `grep` tool instead of grep/rg. It respects .gitignore and provides structured output.",
+		tool: "search",
+		message: "Use the `search` tool instead of grep/rg. It respects .gitignore and provides structured output.",
 	},
 	{
 		pattern: "^\\s*(find|fd|locate)\\s+.*(-name|-iname|-type|--type|-glob)",
@@ -955,12 +956,12 @@ export const SETTINGS_SCHEMA = {
 	// Edit tool
 	"edit.mode": {
 		type: "enum",
-		values: ["replace", "patch", "hashline", "vim", "apply_patch", "atom"] as const,
+		values: EDIT_MODES,
 		default: "hashline",
 		ui: {
 			tab: "editing",
 			label: "Edit Mode",
-			description: "Select the edit tool variant (replace, patch, hashline, vim, or apply_patch)",
+			description: "Select the edit tool variant (replace, patch, hashline, atom, vim, or apply_patch)",
 		},
 	},
 
@@ -1027,7 +1028,7 @@ export const SETTINGS_SCHEMA = {
 
 	"read.defaultLimit": {
 		type: "number",
-		default: 300,
+		default: 500,
 		ui: {
 			tab: "editing",
 			label: "Default Read Limit",
@@ -1198,30 +1199,30 @@ export const SETTINGS_SCHEMA = {
 		ui: { tab: "tools", label: "Find", description: "Enable the find tool for file searching" },
 	},
 
-	"grep.enabled": {
+	"search.enabled": {
 		type: "boolean",
 		default: true,
-		ui: { tab: "tools", label: "Grep", description: "Enable the grep tool for content searching" },
+		ui: { tab: "tools", label: "Search", description: "Enable the search tool for content searching" },
 	},
 
-	"grep.contextBefore": {
+	"search.contextBefore": {
 		type: "number",
 		default: 1,
 		ui: {
 			tab: "tools",
-			label: "Grep Context Before",
-			description: "Lines of context before each grep match",
+			label: "Search Context Before",
+			description: "Lines of context before each search match",
 			submenu: true,
 		},
 	},
 
-	"grep.contextAfter": {
+	"search.contextAfter": {
 		type: "number",
 		default: 3,
 		ui: {
 			tab: "tools",
-			label: "Grep Context After",
-			description: "Lines of context after each grep match",
+			label: "Search Context After",
+			description: "Lines of context after each search match",
 			submenu: true,
 		},
 	},

package/src/config/settings.ts

@@ -326,7 +326,7 @@ export class Settings {
 
 	/**
 	 * Get the edit variant for a specific model.
-	 * Returns "patch", "replace", "hashline", "vim", "apply_patch", or null (use global default).
+	 * Returns "patch", "replace", "hashline", "atom", "vim", "apply_patch", or null (use global default).
 	 */
 	getEditVariantForModel(model: string | undefined): EditMode | null {
 		if (!model) return null;

package/src/cursor.ts

@@ -177,10 +177,10 @@ export class CursorExecHandlers implements ICursorExecHandlers {
 
 	async grep(args: Parameters<NonNullable<ICursorExecHandlers["grep"]>>[0]) {
 		const toolCallId = decodeToolCallId(args.toolCallId);
-		const grepPath = args.glob ? `${args.path || "."}/${args.glob}` : args.path || ".";
-		const toolResultMessage = await executeTool(this.options, "grep", toolCallId, {
+		const searchPath = args.glob ? `${args.path || "."}/${args.glob}` : args.path || ".";
+		const toolResultMessage = await executeTool(this.options, "search", toolCallId, {
 			pattern: args.pattern,
-			path: grepPath,
+			path: searchPath,
 			i: args.caseInsensitive || undefined,
 		});
 		return toolResultMessage;
@@ -327,7 +327,7 @@ export class CursorExecHandlers implements ICursorExecHandlers {
 		const toolCallId = decodeToolCallId(call.toolCallId);
 		const tool = this.options.tools.get(toolName);
 		if (!tool) {
-			const availableTools = Array.from(this.options.tools.keys()).filter(name => name.startsWith("mcp_"));
+			const availableTools = Array.from(this.options.tools.keys()).filter(name => name.startsWith("mcp__"));
 			const message = formatMcpToolErrorMessage(toolName, availableTools);
 			const result = buildToolErrorResult(message);
 			return createToolResultMessage(toolCallId, toolName, result, true);