@oh-my-pi/pi-coding-agent 11.10.4 → 11.12.0

package/CHANGELOG.md

@@ -2,6 +2,79 @@
 
 ## [Unreleased]
 
+## [11.12.0] - 2026-02-11
+### Added
+
+- Added `resolveFileDisplayMode` utility to centralize file display mode resolution across tools (read, grep, file mentions)
+- Added automatic hashline formatting to @file mentions when hashline mode is active
+- Added `replace` hashline edit operation for substr-style fuzzy text replacement without line references, with optional `all` flag for replace-all behavior
+- Added `noopEdits` array to `applyHashlineEdits` return value to report edits that produced no changes, including edit index, location, and current content for diagnostics
+- Added validation to detect and reject hashline edits using wrong-format fields (`old_text`/`new_text` from replace mode, `diff` from patch mode) with helpful error messages
+- Added `additionalProperties: true` to all hashline edit schemas (`single`, `range`, `insertAfter`, and root) to tolerate extra fields from models
+- Added whitespace normalization in line reference parsing to tolerate spaces around colons (e.g., `5 : ab` now parses as `5:ab`)
+- Added `remaps` property to `HashlineMismatchError` providing quick-fix mapping of stale line references to corrected hashes
+- Added warnings detection in `applyHashlineEdits` to alert users when edits affect significantly more lines than expected, indicating possible unintended reformatting
+- Added diagnostic output showing target line content when an edit produces no changes, helping users identify hash mismatches or incorrect replacement content
+- Added `{{hashline}}` Handlebars helper to compute accurate `LINE:HASH` references for prompt examples and documentation
+- Added deduplication of identical hashline edits targeting the same line(s) in a single call
+- Added `replacement` as accepted alias for `content` in `insertAfter` operations
+- Added graceful degradation of `range` edits with missing `end` field to single-line edits
+- Added `additionalProperties: true` to hashline edit schemas to tolerate extra fields from models
+
+### Changed
+
+- Reverted hashline display format from `LINE:HASH  content` (two spaces) back to `LINE:HASH|content` (pipe separator) for consistency with legacy format
+- Changed hashline display format from `LINE:HASH| content` to `LINE:HASH  content` (two spaces instead of pipe separator) for improved readability
+- Removed `lines` and `hashes` parameters from `read` tool—file display mode (line numbers, hashlines) now determined automatically by settings and edit mode
+- Simplified `read` tool prompt to reflect automatic display mode detection based on configuration
+- Updated `grep` tool to respect file display mode settings, showing hashline-prefixed output when hashline mode is active
+- Renamed hashline edit operation keys from `single`/`range`/`insertAfter` to `set_line`/`replace_lines`/`insert_after` for clearer semantics
+- Renamed hashline edit fields: `loc` → `anchor`, `replacement` → `new_text`, `content` → `text` for consistency across all operation types
+- Separated hashline anchor-based edits (`set_line`, `replace_lines`, `insert_after`) from content-replace edits (`replace`) in application pipeline
+- Improved no-op edit diagnostics to use `noopEdits` array from `applyHashlineEdits`, providing precise line-by-line comparison when replacements match current content
+- Enhanced error messages for wrong-format hashline edits to guide users toward correct operation syntax
+- Strengthened hashline prompt guidance to emphasize that `replacement` must differ from current line content and clarify no-op error recovery procedures
+- Improved hashline prompt to clarify atomicity: all edits in one call are validated against the original file state, with line numbers and hashes referring to the pre-edit state
+- Added explicit instruction in hashline prompt to preserve exact whitespace and formatting when replacing lines, changing only the targeted token/expression
+- Added guidance in hashline prompt for swap operations: use two `single` operations in one call rather than attempting to account for line number shifts
+- Strengthened anti-reformatting instructions in hashline prompt to reduce formatting-only failures
+- Improved no-op error recovery guidance in hashline prompt to prevent infinite retry loops
+- Renamed hashline edit operation keys from `replaceLine`/`replaceLines` to `single`/`range` for clearer semantics
+- Renamed hashline edit field `content` to `replacement` in `single` and `range` operations to distinguish from `insertAfter.content`
+- Improved no-op edit diagnostics to show specific line-by-line comparisons when replacements match current content, helping users identify hash mismatches or formatting issues
+- Enhanced no-op error messages to distinguish between literally identical replacements and content normalized back by heuristics
+- Reverted hash algorithm from 3-character base-36 back to 2-character hexadecimal for line references
+- Enhanced range validation during hashline edits to detect and reject relocations that change the scope of affected lines
+- Improved wrapped-line restoration logic to only attempt merging when source lines exhibit continuation patterns
+- Updated hashline tool documentation to emphasize direction-locking mutations and clarify recovery procedures for hash mismatches
+- Changed `applyHashlineEdits` return type to include optional `warnings` array for reporting suspicious edit patterns
+- Improved hash relocation logic to recompute touched lines after hash-based line number adjustments, preventing incorrect merge heuristics
+- Enhanced error messages for no-op edits to include preview of target lines with their current hashes and content
+- Changed hashline edit format from `src`/`dst` object structure to direct operation schemas (`replaceLine`, `replaceLines`, `insertAfter`)
+- Changed hash algorithm from 2-character hexadecimal to 3-character base-36 alphanumeric for improved readability and collision resistance
+- Improved hash mismatch handling to automatically relocate stale line references when the hash uniquely identifies a moved line
+- Changed `HashlineEdit` from `src`/`dst` format to direct operation schemas: `replaceLine`, `replaceLines`, `insertAfter`
+- Changed hash algorithm from hexadecimal (base-16) to base-36 alphanumeric for shorter, more readable line references
+- Increased maximum wrapped-line restoration from 6 to 10 lines to handle longer reflowed statements
+- Updated prompt examples to use `{{hashline}}` Handlebars helper for generating correct line references in tool instructions
+
+### Removed
+
+- Removed `insertBefore` hashline edit operation for inserting content before a line
+- Removed `substr` hashline edit operation for substring-based line replacement
+- Removed `insertBefore` and `substr` hashline edit variants
+
+### Fixed
+
+- Fixed `parseLineRef` to handle both legacy pipe-separator format (`LINE:HASH| content`) and new two-space format (`LINE:HASH  content`) for backward compatibility
+- Fixed resource leak in browser query handler by properly disposing owned proxy elements for non-winning candidates
+- Fixed script evaluation to support async functions and await expressions in browser evaluate operations
+- Fixed `range` edits with missing `end` field to gracefully degrade to single-line edits instead of crashing
+- Fixed `insertAfter` operations to accept both `content` and `replacement` field names for consistency with other edit types
+- Fixed deduplication logic to correctly identify and remove identical hashline edits targeting the same line(s) in a single call
+- Fixed range-based edits to prevent invalid mutations when hash relocation changes the number of lines in the target range
+- Fixed multi-edit application to use original file state for all anchor references, preventing incorrect line numbers when earlier edits change file length
+
 ## [11.10.4] - 2026-02-10
 
 ### Added

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "11.10.4",
+	"version": "11.12.0",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"type": "module",
 	"ompConfig": {
@@ -84,12 +84,12 @@
 	},
 	"dependencies": {
 		"@mozilla/readability": "0.6.0",
-		"@oh-my-pi/omp-stats": "11.10.4",
-		"@oh-my-pi/pi-agent-core": "11.10.4",
-		"@oh-my-pi/pi-ai": "11.10.4",
-		"@oh-my-pi/pi-natives": "11.10.4",
-		"@oh-my-pi/pi-tui": "11.10.4",
-		"@oh-my-pi/pi-utils": "11.10.4",
+		"@oh-my-pi/omp-stats": "11.12.0",
+		"@oh-my-pi/pi-agent-core": "11.12.0",
+		"@oh-my-pi/pi-ai": "11.12.0",
+		"@oh-my-pi/pi-natives": "11.12.0",
+		"@oh-my-pi/pi-tui": "11.12.0",
+		"@oh-my-pi/pi-utils": "11.12.0",
 		"@sinclair/typebox": "^0.34.48",
 		"ajv": "^8.17.1",
 		"chalk": "^5.6.2",
@@ -107,7 +107,7 @@
 	},
 	"devDependencies": {
 		"@types/ms": "^2.1.0",
-		"@types/bun": "^1.3.8",
+		"@types/bun": "^1.3.9",
 		"ms": "^2.1.3",
 		"@types/jsdom": "27.0.0"
 	},

package/src/config/model-resolver.ts

@@ -29,6 +29,8 @@ export const defaultModelPerProvider: Record<KnownProvider, string> = {
 	zai: "glm-4.6",
 	mistral: "devstral-medium-latest",
 	minimax: "MiniMax-M2.1",
+	"minimax-code": "MiniMax-M2.1",
+	"minimax-code-cn": "MiniMax-M2.1",
 	opencode: "claude-opus-4-6",
 	"kimi-code": "kimi-k2.5",
 };

package/src/config/prompt-templates.ts

@@ -3,6 +3,7 @@ import * as path from "node:path";
 import { logger } from "@oh-my-pi/pi-utils";
 import Handlebars from "handlebars";
 import { CONFIG_DIR_NAME, getPromptsDir } from "../config";
+import { computeLineHash } from "../patch/hashline";
 import { jtdToTypeScript } from "../tools/jtd-to-typescript";
 import { parseFrontmatter } from "../utils/frontmatter";
 
@@ -228,6 +229,16 @@ handlebars.registerHelper("jtdToTypeScript", (schema: unknown): string => jtdToT
 
 handlebars.registerHelper("jsonStringify", (value: unknown): string => JSON.stringify(value));
 
+/**
+ * {{hashline lineNum "content"}} — compute a real hashline ref for prompt examples.
+ * Returns `"lineNum:hash"` using the actual hash algorithm.
+ */
+handlebars.registerHelper("hashline", (lineNum: unknown, content: unknown): string => {
+	const num = typeof lineNum === "number" ? lineNum : Number.parseInt(String(lineNum), 10);
+	const str = typeof content === "string" ? content : String(content ?? "");
+	return `${num}:${computeLineHash(num, str)}`;
+});
+
 export function renderPromptTemplate(template: string, context: TemplateContext = {}): string {
 	const compiled = handlebars.compile(template, { noEscape: true, strict: false });
 	const rendered = compiled(context ?? {});

package/src/config/settings-schema.ts

@@ -253,7 +253,7 @@ export const SETTINGS_SCHEMA = {
 		ui: {
 			tab: "config",
 			label: "Read hash lines",
-			description: "Include line hashes in read output for hashline edit mode (LINE:HASH| content)",
+			description: "Include line hashes in read output for hashline edit mode (LINE:HASH|content)",
 		},
 	},
 	showHardwareCursor: {

package/src/debug/system-info.ts

@@ -22,6 +22,21 @@ export interface SystemInfo {
 	terminal: string | undefined;
 }
 
+/** Map Darwin kernel major version to macOS marketing name. */
+function macosMarketingName(release: string): string | undefined {
+	const major = Number.parseInt(release.split(".")[0] ?? "", 10);
+	if (Number.isNaN(major)) return undefined;
+	const names: Record<number, string> = {
+		25: "Tahoe",
+		24: "Sequoia",
+		23: "Sonoma",
+		22: "Ventura",
+		21: "Monterey",
+		20: "Big Sur",
+	};
+	return names[major];
+}
+
 /** Collect system information */
 export async function collectSystemInfo(): Promise<SystemInfo> {
 	const cpus = os.cpus();
@@ -31,8 +46,14 @@ export async function collectSystemInfo(): Promise<SystemInfo> {
 	const shell = Bun.env.SHELL ?? Bun.env.ComSpec ?? "unknown";
 	const terminal = Bun.env.TERM_PROGRAM ?? Bun.env.TERM ?? undefined;
 
+	let osStr = `${os.type()} ${os.release()} (${os.platform()})`;
+	if (os.platform() === "darwin") {
+		const name = macosMarketingName(os.release());
+		if (name) osStr = `${osStr} ${name}`;
+	}
+
 	return {
-		os: `${os.type()} ${os.release()} (${os.platform()})`,
+		os: osStr,
 		arch: os.arch(),
 		cpu: cpuModel,
 		memory: {

package/src/mcp/tool-bridge.ts

@@ -53,10 +53,15 @@ export interface MCPToolDetails {
 /**
  * Convert JSON Schema from MCP to TypeBox-compatible schema.
  * MCP uses standard JSON Schema, TypeBox uses a compatible subset.
+ *
+ * Also normalizes schemas to work around common issues:
+ * - Adds `properties: {}` to object schemas missing it (some LLM providers require this)
  */
 function convertSchema(mcpSchema: MCPToolDefinition["inputSchema"]): TSchema {
-	// MCP schemas are JSON Schema objects, TypeBox can use them directly
-	// as long as we ensure the structure is correct
+	// Normalize: object schemas must have properties field for some providers
+	if (mcpSchema.type === "object" && !("properties" in mcpSchema)) {
+		return { ...mcpSchema, properties: {} } as unknown as TSchema;
+	}
 	return mcpSchema as unknown as TSchema;
 }
 

package/src/patch/diff.ts

@@ -9,8 +9,9 @@ import { resolveToCwd } from "../tools/path-utils";
 import { previewPatch } from "./applicator";
 import { DEFAULT_FUZZY_THRESHOLD, findMatch } from "./fuzzy";
 import { applyHashlineEdits } from "./hashline";
+import type { HashlineEdit } from "./index";
 import { adjustIndentation, normalizeToLF, stripBom } from "./normalize";
-import type { DiffError, DiffResult, HashlineEdit, PatchInput } from "./types";
+import type { DiffError, DiffResult, PatchInput } from "./types";
 import { EditMatchError } from "./types";
 
 // ═══════════════════════════════════════════════════════════════════════════