@oh-my-pi/pi-coding-agent 14.2.0 → 14.3.0

package/CHANGELOG.md

@@ -2,6 +2,65 @@
 
 ## [Unreleased]
 
+## [14.3.0] - 2026-04-25
+
+### Added
+
+- Added Markdown pipe-table `row_N` chunk selectors for row-level table edits.
+- Added `resolveToolAlias` export so tool names in CLI and session setup are normalized to canonical names, including mapping legacy `read` references to `open`
+- Added new `open` and `open-chunk` tool prompt documentation pages to describe canonical `open` usage for local files/directories, chunk reads, and URLs
+- Added full-output retrieval metadata to minimized shell command output by appending an `artifact://<id>` footer with byte counts, allowing users to open the original unminimized command output
+- Added streaming preview API exports from the package (`resolveEditMode`, `EDIT_MODE_STRATEGIES`, and chunk preview helpers) so editors can reuse mode-aware edit preview logic programmatically
+- Added `shellMinimizer` configuration options (`enabled`, `settingsPath`, `only`, `except`, and `maxCaptureBytes`) so users can control shell output minimization behavior
+
+### Changed
+
+- Changed the canonical file/URL reader tool from `read` to `open` across default tool lists and routing, including system prompts, plan mode, cursor handlers, and runtime tool registration
+- Changed runtime and UI handling to render and track `open` tool calls as first-class (with `read` accepted as legacy alias), including ACP mapping, session observers, and streaming message groups
+- Changed chunk edit guidance to document parser-specific region behavior, including TypeScript decorator/JSDoc sibling chunks, Python docstrings as body content, Python opaque nested chunks, Markdown whole-chunk fallbacks, ID volatility, and indentation display differences
+- Changed chunk deletion in chunk edit mode to require explicit `delete: true`; `write: null` and bare `{ path }` entries now fail with guidance instead of deleting content.
+- Changed chunk edit validation to reject entries with multiple operation fields instead of choosing one and ignoring the rest.
+- Changed chunk edit validation to reject `write: ""` as an accidental destructive empty replacement; use the open tool for inspection or `delete: true` for deletion.
+- Changed chunk edit responses to warn when appending or prepending to a container without `~`, since that inserts outside the container rather than inside its body.
+- Changed fetch output logging so URL-fetch artifacts now use `.open.log` naming instead of `.read.log`
+- Changed Bash interception guidance and errors to recommend `open` in place of `read` for cat/head/tail-style commands
+- Changed exported SDK tool surface to expose `OpenTool` as canonical and keep `ReadTool` as a compatibility alias
+- Changed session list loading to use parallel workers and fixed-size prefix reads per session file, reducing latency when loading many or large sessions
+- Changed edit call rendering to use mode-aware streaming diff previews, including multi-file chunk edit previews grouped by file path while arguments are still streaming
+- Changed shell execution in both interactive and non-interactive modes to route command output through the configured shell output minimizer
+- Changed default behavior so shell output minimization can now be toggled from settings without code changes
+- Changed shell output minimization to leave compound and piped commands unchanged; only a single eligible whole command is captured and minimized after it exits
+
+### Removed
+
+- Removed the chunk edit `read: true` operation; use the open tool to inspect chunks without modifying files.
+- Removed the `replace: { old, new }` chunk edit operation. Use `write` or `insert` for chunk edits instead.
+
+### Fixed
+
+- Fixed startup crashes on Linux systems where Bun's `os.cpus()` fails on non-contiguous CPU numbering ([#779](https://github.com/can1357/oh-my-pi/issues/779))
+- Fixed `gh_pr_push` so branches without `gh_pr_checkout` metadata fail instead of falling back to the tracked merge branch, and updated the GitHub tool setting copy to stop calling the tool group read-only ([#778](https://github.com/can1357/oh-my-pi/issues/778))
+- Fixed session list metadata extraction to better populate session titles and first-user summaries from partial session data when full JSONL parsing is unavailable
+- Fixed shell execution output to replace raw streamed bash output with the minimizer’s rewritten text before final output while still preserving the full original output as artifact metadata
+- Fixed bash command minimization to save the full unminimized output as a `bash-original` artifact during AgentSession shell execution, enabling `artifact://` access to complete command output
+- Fixed streaming chunk previews that could display an incomplete trailing edit as a deletion when partial JSON temporarily converted in-flight values to `null`
+- Fixed edit streaming preview updates to cancel obsolete in-flight computations and avoid rendering stale previews as args change
+- Fixed chunk edits to reject unsafe `^`/`~` writes on code leaf chunks instead of falling back to whole-chunk replacement and risking structural indentation corruption
+- Fixed chunk `replace` operations to dedent multiline replacement snippets before reapplying the matched source indentation, preventing Python nested replacements from compounding indentation on repeated edits.
+- Fixed Go chunk trees to classify `package` clauses separately from imports and to avoid duplicating method receivers in method summaries.
+- Fixed chunk path-not-found guidance so it recommends `sel="?"` without claiming the already-shown listing must be re-read.
+- Fixed Markdown chunk appends to preserve blank-line separators after line-oriented inserts such as table rows
+- Fixed Markdown section region-fallback warnings to call out child chunks that will be replaced by whole-section edits.
+- Fixed rejected chunk-edit errors to distinguish current file content from hypothetical post-edit parse-error previews and to state when a same-file batch was rolled back.
+- Fixed unsafe Python head-region edits by rejecting decorated Python `^` writes and Python `^` deletes that can orphan indented bodies while still parsing.
+- Fixed Markdown table-row appends so row-shaped content lands inside the table block instead of after the trailing blank-line separator.
+- Fixed Markdown root writes to preserve fenced-code indentation verbatim.
+- Fixed Rust enum-variant replacement matching so trailing commas are included consistently with whole-variant writes.
+- Fixed streaming edit call headers to keep showing the target file path while the edit arguments are still arriving
+- Fixed Mermaid fenced markdown rendering in assistant messages on terminals without image protocol support ([#650](https://github.com/can1357/oh-my-pi/issues/650))
+- Fixed chunk edit path parsing so plan-mode edits to section-addressed `local://PLAN.md:<selector>` paths are classified as writes to the plan file
+- Fixed SQLite `read` helper queries to reject `where=` clauses with SQL control syntax that could override the structured selector's pagination; raw SQL remains available through `q=SELECT ...`
+
 ## [14.2.0] - 2026-04-23
 
 ### Added

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "14.2.0",
+	"version": "14.3.0",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://github.com/can1357/oh-my-pi",
 	"author": "Can Boluk",
@@ -44,31 +44,31 @@
 		"generate-template": "bun scripts/generate-template.ts"
 	},
 	"dependencies": {
-		"@agentclientprotocol/sdk": "0.16.1",
-		"@mozilla/readability": "^0.6",
-		"@oh-my-pi/omp-stats": "14.2.0",
-		"@oh-my-pi/pi-agent-core": "14.2.0",
-		"@oh-my-pi/pi-ai": "14.2.0",
-		"@oh-my-pi/pi-natives": "14.2.0",
-		"@oh-my-pi/pi-tui": "14.2.0",
-		"@oh-my-pi/pi-utils": "14.2.0",
-		"@sinclair/typebox": "^0.34",
-		"@xterm/headless": "^6.0",
-		"ajv": "^8.18",
-		"chalk": "^5.6",
-		"diff": "^8.0",
+		"@agentclientprotocol/sdk": "0.20.0",
+		"@mozilla/readability": "^0.6.0",
+		"@oh-my-pi/omp-stats": "14.3.0",
+		"@oh-my-pi/pi-agent-core": "14.3.0",
+		"@oh-my-pi/pi-ai": "14.3.0",
+		"@oh-my-pi/pi-natives": "14.3.0",
+		"@oh-my-pi/pi-tui": "14.3.0",
+		"@oh-my-pi/pi-utils": "14.3.0",
+		"@sinclair/typebox": "^0.34.49",
+		"@xterm/headless": "^6.0.0",
+		"ajv": "^8.20.0",
+		"chalk": "^5.6.2",
+		"diff": "^9.0.0",
 		"fflate": "0.8.2",
 		"handlebars": "^4.7.9",
-		"linkedom": "^0.18",
-		"lru-cache": "11.3.1",
-		"markit-ai": "0.5.0",
-		"puppeteer": "^24.37",
+		"linkedom": "^0.18.12",
+		"lru-cache": "11.3.5",
+		"markit-ai": "0.5.3",
+		"puppeteer": "^24.42.0",
 		"turndown": "7.2.4",
 		"turndown-plugin-gfm": "1.0.2",
 		"zod": "4.3.6"
 	},
 	"devDependencies": {
-		"@types/bun": "^1.3",
+		"@types/bun": "^1.3.13",
 		"@types/turndown": "5.0.6"
 	},
 	"engines": {

package/src/cli/args.ts

@@ -60,7 +60,16 @@ export function parseArgs(args: string[], extensionFlags?: Map<string, { type: "
 	};
 
 	for (let i = 0; i < args.length; i++) {
-		const arg = args[i];
+		let arg = args[i];
+
+		// Support --flag=value syntax (e.g. --tools=ask,read)
+		if (arg.startsWith("--") && arg.includes("=")) {
+			const eqIdx = arg.indexOf("=");
+			const value = arg.slice(eqIdx + 1);
+			arg = arg.slice(0, eqIdx);
+			// Insert the value so the existing "args[++i]" logic picks it up
+			args.splice(i + 1, 0, value);
+		}
 
 		if (arg === "--help" || arg === "-h") {
 			result.help = true;

package/src/cli/shell-cli.ts

@@ -5,10 +5,10 @@
  */
 import * as path from "node:path";
 import { createInterface } from "node:readline/promises";
-import { Shell } from "@oh-my-pi/pi-natives";
+import { type MinimizerOptions, Shell } from "@oh-my-pi/pi-natives";
 import { APP_NAME, getProjectDir } from "@oh-my-pi/pi-utils";
 import chalk from "chalk";
-import { Settings } from "../config/settings";
+import { Settings, type ShellMinimizerSettings } from "../config/settings";
 import { getOrCreateSnapshot } from "../utils/shell-snapshot";
 
 export interface ShellCommandArgs {
@@ -41,6 +41,17 @@ export function parseShellArgs(args: string[]): ShellCommandArgs | undefined {
 	return result;
 }
 
+function buildMinimizerOptions(group: ShellMinimizerSettings): MinimizerOptions | undefined {
+	if (!group.enabled) return undefined;
+	return {
+		enabled: true,
+		settingsPath: group.settingsPath || undefined,
+		only: group.only.length > 0 ? group.only : undefined,
+		except: group.except.length > 0 ? group.except : undefined,
+		maxCaptureBytes: group.maxCaptureBytes,
+	};
+}
+
 export async function runShellCommand(cmd: ShellCommandArgs): Promise<void> {
 	if (!process.stdin.isTTY) {
 		process.stderr.write("Error: shell console requires an interactive TTY.\n");
@@ -51,7 +62,8 @@ export async function runShellCommand(cmd: ShellCommandArgs): Promise<void> {
 	const settings = await Settings.init({ cwd });
 	const { shell, env: shellEnv } = settings.getShellConfig();
 	const snapshotPath = cmd.noSnapshot || !shell.includes("bash") ? null : await getOrCreateSnapshot(shell, shellEnv);
-	const shellSession = new Shell({ sessionEnv: shellEnv, snapshotPath: snapshotPath ?? undefined });
+	const minimizer = buildMinimizerOptions(settings.getGroup("shellMinimizer"));
+	const shellSession = new Shell({ sessionEnv: shellEnv, snapshotPath: snapshotPath ?? undefined, minimizer });
 
 	let active = false;
 	let lastChar: string | null = null;

package/src/config/settings-schema.ts

@@ -1123,6 +1123,39 @@ export const SETTINGS_SCHEMA = {
 	},
 	"bashInterceptor.patterns": { type: "array", default: DEFAULT_BASH_INTERCEPTOR_RULES },
 
+	// Shell output minimizer
+	"shellMinimizer.enabled": {
+		type: "boolean",
+		default: true,
+		ui: {
+			tab: "editing",
+			label: "Shell Minimizer",
+			description: "Compress verbose shell output (git, npm, cargo, etc.) before returning it to the agent",
+		},
+	},
+	"shellMinimizer.settingsPath": {
+		type: "string",
+		default: undefined,
+		ui: {
+			tab: "editing",
+			label: "Minimizer Settings Path",
+			description: "Optional TOML file with per-command minimizer overrides",
+			submenu: true,
+		},
+	},
+	"shellMinimizer.only": { type: "array", default: EMPTY_STRING_ARRAY },
+	"shellMinimizer.except": { type: "array", default: EMPTY_STRING_ARRAY },
+	"shellMinimizer.maxCaptureBytes": {
+		type: "number",
+		default: 4 * 1024 * 1024,
+		ui: {
+			tab: "editing",
+			label: "Minimizer Capture Limit",
+			description: "Maximum captured output bytes before falling back to raw streaming",
+			submenu: true,
+		},
+	},
+
 	// Python
 	"python.toolMode": {
 		type: "enum",
@@ -1315,7 +1348,8 @@ export const SETTINGS_SCHEMA = {
 		ui: {
 			tab: "tools",
 			label: "GitHub CLI",
-			description: "Enable read-only gh_* tools for GitHub repository, issue, pull request, diff, and search access",
+			description:
+				"Enable gh_* tools for GitHub repository, issue, pull request, diff, search, checkout, and PR push workflows",
 		},
 	},
 
@@ -1612,6 +1646,22 @@ export const SETTINGS_SCHEMA = {
 		ui: { tab: "tasks", label: "Claude Project Commands", description: "Load commands from .claude/commands/" },
 	},
 
+	"commands.enableOpencodeUser": {
+		type: "boolean",
+		default: true,
+		ui: {
+			tab: "tasks",
+			label: "OpenCode User Commands",
+			description: "Load commands from ~/.config/opencode/commands/",
+		},
+	},
+
+	"commands.enableOpencodeProject": {
+		type: "boolean",
+		default: true,
+		ui: { tab: "tasks", label: "OpenCode Project Commands", description: "Load commands from .opencode/commands/" },
+	},
+
 	// ────────────────────────────────────────────────────────────────────────
 	// Providers
 	// ────────────────────────────────────────────────────────────────────────
@@ -1956,6 +2006,14 @@ export interface BashInterceptorRule {
 	allowSubcommands?: string[];
 }
 
+export interface ShellMinimizerSettings {
+	enabled: boolean;
+	settingsPath: string | undefined;
+	only: string[];
+	except: string[];
+	maxCaptureBytes: number;
+}
+
 /** Map group prefix -> typed settings interface */
 export interface GroupTypeMap {
 	compaction: CompactionSettings;
@@ -1973,6 +2031,7 @@ export interface GroupTypeMap {
 	modelRoles: Record<string, string>;
 	modelTags: ModelTagsSettings;
 	cycleOrder: string[];
+	shellMinimizer: ShellMinimizerSettings;
 }
 
 export type GroupPrefix = keyof GroupTypeMap;

package/src/dap/session.ts

@@ -1075,11 +1075,17 @@ export class DapSessionManager {
 	 * MUST be called before the command that triggers the event.
 	 */
 	#prepareStopOutcome(session: DapSession, signal?: AbortSignal, timeoutMs: number = 30_000): Promise<unknown> {
-		return Promise.race([
+		const promises = [
 			session.client.waitForEvent("stopped", undefined, signal, timeoutMs),
 			session.client.waitForEvent("terminated", undefined, signal, timeoutMs),
 			session.client.waitForEvent("exited", undefined, signal, timeoutMs),
-		]);
+		];
+		// Promise.race leaves the losing waiters pending; their timeouts would
+		// otherwise surface as unhandled rejections once they fire.
+		for (const p of promises) {
+			p.catch(() => {});
+		}
+		return Promise.race(promises);
 	}
 
 	/**

package/src/debug/system-info.ts

@@ -40,8 +40,12 @@ function macosMarketingName(release: string): string | undefined {
 
 /** Collect system information */
 export async function collectSystemInfo(): Promise<SystemInfo> {
-	const cpus = os.cpus();
-	const cpuModel = cpus[0]?.model ?? "Unknown CPU";
+	let cpuModel = "Unknown CPU";
+	try {
+		cpuModel = os.cpus()[0]?.model ?? cpuModel;
+	} catch {
+		// Keep debug report collection best-effort when CPU probing fails.
+	}
 
 	// Try to get shell from environment
 	const shell = Bun.env.SHELL ?? Bun.env.ComSpec ?? "unknown";

package/src/discovery/claude.ts

@@ -18,6 +18,7 @@ import { type SlashCommand, slashCommandCapability } from "../capability/slash-c
 import { type SystemPrompt, systemPromptCapability } from "../capability/system-prompt";
 import { type CustomTool, toolCapability } from "../capability/tool";
 import type { LoadContext, LoadResult } from "../capability/types";
+import { settings } from "../config/settings";
 import {
 	calculateDepth,
 	createSourceMeta,
@@ -252,48 +253,69 @@ async function loadExtensionModules(ctx: LoadContext): Promise<LoadResult<Extens
 // Slash Commands
 // =============================================================================
 
+/**
+ * Read the Claude command-loading toggles from settings.
+ * Falls back to true (current behavior) when settings are not initialized,
+ * e.g. inside discovery unit tests that run without Settings.init().
+ */
+function readClaudeCommandToggles(): { enableUser: boolean; enableProject: boolean } {
+	try {
+		return {
+			enableUser: settings.get("commands.enableClaudeUser") ?? true,
+			enableProject: settings.get("commands.enableClaudeProject") ?? true,
+		};
+	} catch {
+		return { enableUser: true, enableProject: true };
+	}
+}
+
 async function loadSlashCommands(ctx: LoadContext): Promise<LoadResult<SlashCommand>> {
 	const items: SlashCommand[] = [];
 	const warnings: string[] = [];
+	const { enableUser, enableProject } = readClaudeCommandToggles();
+
+	if (enableUser) {
+		const userBase = getUserClaude(ctx);
+		const userCommandsDir = path.join(userBase, "commands");
+
+		const userResult = await loadFilesFromDir<SlashCommand>(ctx, userCommandsDir, PROVIDER_ID, "user", {
+			extensions: ["md"],
+			transform: (name, content, path, source) => {
+				const cmdName = name.replace(/\.md$/, "");
+				return {
+					name: cmdName,
+					path,
+					content,
+					level: "user",
+					_source: source,
+				};
+			},
+		});
 
-	const userBase = getUserClaude(ctx);
-	const userCommandsDir = path.join(userBase, "commands");
-
-	const userResult = await loadFilesFromDir<SlashCommand>(ctx, userCommandsDir, PROVIDER_ID, "user", {
-		extensions: ["md"],
-		transform: (name, content, path, source) => {
-			const cmdName = name.replace(/\.md$/, "");
-			return {
-				name: cmdName,
-				path,
-				content,
-				level: "user",
-				_source: source,
-			};
-		},
-	});
-
-	items.push(...userResult.items);
-	if (userResult.warnings) warnings.push(...userResult.warnings);
-
-	const projectCommandsDir = path.join(ctx.cwd, CONFIG_DIR, "commands");
+		items.push(...userResult.items);
+		if (userResult.warnings) warnings.push(...userResult.warnings);
+	}
 
-	const projectResult = await loadFilesFromDir<SlashCommand>(ctx, projectCommandsDir, PROVIDER_ID, "project", {
-		extensions: ["md"],
-		transform: (name, content, path, source) => {
-			const cmdName = name.replace(/\.md$/, "");
-			return {
-				name: cmdName,
-				path,
-				content,
-				level: "project",
-				_source: source,
-			};
-		},
-	});
+	if (enableProject) {
+		const projectCommandsDir = path.join(ctx.cwd, CONFIG_DIR, "commands");
+
+		const projectResult = await loadFilesFromDir<SlashCommand>(ctx, projectCommandsDir, PROVIDER_ID, "project", {
+			extensions: ["md"],
+			transform: (name, content, path, source) => {
+				const cmdName = name.replace(/\.md$/, "");
+				return {
+					name: cmdName,
+					path,
+					content,
+					level: "project",
+					_source: source,
+				};
+			},
+		});
 
-	items.push(...projectResult.items);
-	if (projectResult.warnings) warnings.push(...projectResult.warnings);
+		items.push(...projectResult.items);
+		if (projectResult.warnings) warnings.push(...projectResult.warnings);
+	}
 
 	return { items, warnings };
 }

package/src/discovery/opencode.ts

@@ -26,6 +26,7 @@ import { type Settings, settingsCapability } from "../capability/settings";
 import { type Skill, skillCapability } from "../capability/skill";
 import { type SlashCommand, slashCommandCapability } from "../capability/slash-command";
 import type { LoadContext, LoadResult, SourceMeta } from "../capability/types";
+import { settings } from "../config/settings";
 
 import {
 	buildExtensionModuleItems,
@@ -236,9 +237,26 @@ async function loadExtensionModules(ctx: LoadContext): Promise<LoadResult<Extens
 // Slash Commands (commands/)
 // =============================================================================
 
+/**
+ * Read the OpenCode command-loading toggles from settings.
+ * Falls back to true (current behavior) when settings are not initialized,
+ * e.g. inside discovery unit tests that run without Settings.init().
+ */
+function readOpencodeCommandToggles(): { enableUser: boolean; enableProject: boolean } {
+	try {
+		return {
+			enableUser: settings.get("commands.enableOpencodeUser") ?? true,
+			enableProject: settings.get("commands.enableOpencodeProject") ?? true,
+		};
+	} catch {
+		return { enableUser: true, enableProject: true };
+	}
+}
+
 async function loadSlashCommands(ctx: LoadContext): Promise<LoadResult<SlashCommand>> {
-	const userCommandsDir = getUserPath(ctx, "opencode", "commands");
-	const projectCommandsDir = getProjectPath(ctx, "opencode", "commands");
+	const { enableUser, enableProject } = readOpencodeCommandToggles();
+	const userCommandsDir = enableUser ? getUserPath(ctx, "opencode", "commands") : null;
+	const projectCommandsDir = enableProject ? getProjectPath(ctx, "opencode", "commands") : null;
 
 	const transformCommand =
 		(level: "user" | "project") => (name: string, content: string, filePath: string, source: SourceMeta) => {

package/src/edit/index.ts

@@ -68,6 +68,7 @@ export * from "./modes/patch";
 export * from "./modes/replace";
 export * from "./normalize";
 export * from "./renderer";
+export * from "./streaming";
 
 type TInput =
 	| typeof replaceEditSchema
@@ -322,7 +323,8 @@ export class EditTool implements AgentTool<TInput> {
 						chunkAutoIndent: resolveChunkAutoIndent(),
 					}),
 				parameters: chunkEditParamsSchema,
-				invalidParamsMessage: "Invalid edit parameters for chunk mode.",
+				invalidParamsMessage:
+					"Invalid edit parameters for chunk mode. Expected `{ edits: [{ path: 'file:selector', ...op }, ...] }` with at least one edit. Each edit needs a `path`; supply exactly one of `write: 'content'`, `insert: { loc, body }`, or `delete: true`.",
 				validate: isChunkParams,
 				execute: (
 					tool: EditTool,