@oh-my-pi/pi-coding-agent 1.341.0 → 2.1.1337

package/CHANGELOG.md

@@ -2,6 +2,92 @@
 
 ## [Unreleased]
 
+## [2.1.1337] - 2026-01-03
+
+### Added
+
+- Added `pi update` command to check for and install updates from GitHub releases or via bun
+
+### Changed
+
+- Changed HTML export to use compile-time bundled templates via Bun macros for improved performance
+- Changed `exportToHtml` and `exportFromFile` functions to be async
+- Simplified build process by embedding assets (themes, templates, agents, commands) directly into the binary at compile time
+- Removed separate asset copying steps from build scripts
+
+## [2.0.1337] - 2026-01-03
+### Added
+
+- Added shell environment snapshot to preserve user aliases, functions, and shell options when executing bash commands
+- Added support for `PI_BASH_NO_CI`, `PI_BASH_NO_LOGIN`, and `PI_SHELL_PREFIX` environment variables for shell customization
+- Added zsh support alongside bash for shell detection and configuration
+
+### Changed
+
+- Changed shell detection to prefer user's `$SHELL` when it's bash or zsh, with improved fallback path resolution
+- Changed Edit tool to reject `.ipynb` files with guidance to use NotebookEdit tool instead
+
+## [1.500.0] - 2026-01-03
+### Added
+
+- Added provider tabs to model selector with Tab/Arrow navigation for filtering models by provider
+- Added context menu to model selector for choosing model role (Default, Smol, Slow) instead of keyboard shortcuts
+- Added LSP diagnostics display in tool execution output showing errors and warnings after file edits
+- Added centralized file logger with daily rotation to `~/.pi/logs/` for debugging production issues
+- Added `logger` property to hook and custom tool APIs for error/warning/debug logging
+- Added `output` tool to read full agent/task outputs by ID when truncated previews are insufficient
+- Added `task` tool to reviewer agent, enabling parallel exploration of large codebases during reviews
+- Added subprocess tool registry for extracting and rendering tool data from subprocess agents in real-time
+- Added combined review result rendering showing verdict and findings in a tree structure
+- Auto-read file mentions: Reference files with `@path/to/file.ext` syntax in prompts to automatically inject their contents, eliminating manual Read tool calls
+- Added `hidden` property for custom tools to exclude them from default tool list unless explicitly requested
+- Added `explicitTools` option to `createAgentSession` for enabling hidden tools by name
+- Added example review tools (`report_finding`, `submit_review`) with structured findings accumulation and verdict rendering
+- Added `/review` example command for interactive code review with branch comparison, uncommitted changes, and commit review modes
+- Custom TypeScript slash commands: Create programmable commands at `~/.pi/agent/commands/[name]/index.ts` or `.pi/commands/[name]/index.ts`. Commands export a factory returning `{ name, description, execute(args, ctx) }`. Return a string to send as LLM prompt, or void for fire-and-forget actions. Full access to `HookCommandContext` for UI dialogs, session control, and shell execution.
+- Claude command directories: Markdown slash commands now also load from `~/.claude/commands/` and `.claude/commands/` (parallel to existing `.pi/commands/` support)
+- `commands.enableClaudeUser` and `commands.enableClaudeProject` settings to disable Claude command directory loading
+- `/export --copy` option to copy entire session as formatted text to clipboard
+
+### Changed
+
+- Changed model selector keyboard shortcuts from S/L keys to a context menu opened with Enter
+- Changed model role indicators from symbols (✓ ⚡ 🧠) to labeled badges ([ DEFAULT ] [ SMOL ] [ SLOW ])
+- Changed model list sorting to include secondary sort by model ID within each provider
+- Changed silent error suppression to log warnings and debug info for tool errors, theme loading, and command loading failures
+- Changed Task tool progress display to show agent index (e.g., `reviewer(0)`) for easier Output tool ID derivation
+- Changed Task tool output to only include file paths when Output tool is unavailable, providing Read tool fallback
+- Changed Task tool output references to use simpler ID format (e.g., `reviewer_0`) with line/char counts for Output tool integration
+- Changed subagent recursion prevention from blanket blocking to same-agent blocking. Non-recursive agents can now spawn other agent types (e.g., reviewer can spawn explore agents) but cannot spawn themselves.
+- Changed `/review` command from markdown to interactive TypeScript with mode selection menu (branch comparison, uncommitted changes, commit review, custom)
+- Changed bundled commands to be overridable by user/project commands with same name
+- Changed subprocess termination to wait for message_end event to capture accurate token counts
+- Changed token counting in subprocess to accumulate across messages instead of overwriting
+- Updated bundled `reviewer` agent to use structured review tools with priority-based findings (P0-P3) and formal verdict submission
+- Task tool now streams artifacts in real-time: input written before spawn, session jsonl written by subprocess, output written at completion
+
+### Removed
+
+- Removed separate Exa error logger in favor of centralized logging system
+- Removed `findings_count` parameter from `submit_review` tool - findings are now counted automatically
+- Removed artifacts location display from task tool output
+
+### Fixed
+
+- Fixed race condition in event listener iteration by copying array before iteration to prevent mutation during callbacks
+- Fixed potential memory leak from orphaned abort controllers by properly aborting existing controllers before replacement
+- Fixed stream reader resource leak by adding proper `releaseLock()` calls in finally blocks
+- Fixed hook API methods throwing clear errors when handlers are not initialized instead of silently failing
+- Fixed LSP client race conditions with concurrent client creation and file operations using proper locking
+- Fixed Task tool progress display showing stale data by cloning progress objects before passing to callbacks
+- Fixed Task tool missing final progress events by waiting for readline to close before resolving
+- Fixed RPC mode race condition with concurrent prompt commands by serializing execution
+- Fixed pre-commit hook race condition causing `index.lock` errors when GitKraken/IDE git integrations detect file changes during formatting
+- Fixed Task tool output artifacts (`out.md`) containing duplicated text from streaming updates
+- Fixed Task tool progress display showing repeated nearly-identical lines during streaming
+- Fixed Task tool subprocess model selection ignoring agent's configured model and falling back to settings default. The `--model` flag now accepts `provider/model` format directly.
+- Fixed Task tool showing "done + succeeded" when aborted; now correctly displays "⊘ aborted" status
+
 ## [1.341.0] - 2026-01-03
 ### Added
 

package/README.md

@@ -192,7 +192,7 @@ The agent reads, writes, and edits files, and executes commands via bash.
 | ------------------------- | --------------------------------------------------------------------------- |
 | `/settings`               | Open settings menu (thinking, theme, queue mode, toggles)                   |
 | `/model`                  | Switch models mid-session (fuzzy search, arrow keys, Enter to select)       |
-| `/export [file]`          | Export session to self-contained HTML                                       |
+| `/export [file\|--copy]`  | Export session to HTML file or copy to clipboard                            |
 | `/share`                  | Upload session as secret GitHub gist, get shareable URL (requires `gh` CLI) |
 | `/session`                | Show session info: path, message counts, token usage, cost                  |
 | `/hotkeys`                | Show all keyboard shortcuts                                                 |

package/examples/custom-tools/subagent/index.ts

@@ -18,7 +18,7 @@ import * as path from "node:path";
 import type { AgentToolResult } from "@oh-my-pi/pi-agent-core";
 import type { Message } from "@oh-my-pi/pi-ai";
 import type { CustomTool, CustomToolAPI, CustomToolFactory } from "@oh-my-pi/pi-coding-agent";
-import { type AgentConfig, type AgentScope, discoverAgents, formatAgentList } from "./agents.js";
+import { type AgentConfig, type AgentScope, discoverAgents, formatAgentList } from "./agents";
 
 const MAX_PARALLEL_TASKS = 8;
 const MAX_CONCURRENCY = 4;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "1.341.0",
+	"version": "2.1.1337",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"type": "module",
 	"piConfig": {
@@ -32,16 +32,15 @@
 		"check": "tsgo --noEmit",
 		"clean": "rm -rf dist",
 		"build": "tsgo -p tsconfig.build.json && chmod +x dist/cli.js && npm run copy-assets",
-		"build:binary": "npm run build && bun build --compile ./dist/cli.js --outfile dist/pi && npm run copy-binary-assets",
-		"copy-assets": "mkdir -p dist/modes/interactive/theme && cp src/modes/interactive/theme/*.json dist/modes/interactive/theme/ && mkdir -p dist/core/export-html && cp src/core/export-html/template.html src/core/export-html/template.css src/core/export-html/template.js dist/core/export-html/",
-		"copy-binary-assets": "cp package.json dist/ && cp README.md dist/ && cp CHANGELOG.md dist/ && mkdir -p dist/theme && cp src/modes/interactive/theme/*.json dist/theme/ && mkdir -p dist/export-html && cp src/core/export-html/template.html src/core/export-html/template.css src/core/export-html/template.js dist/export-html/ && cp -r docs dist/ && cp -r examples dist/",
-		"test": "vitest --run",
+		"copy-assets": "cp -r src/core/tools/task/bundled-agents dist/core/tools/task/ && cp -r src/core/tools/task/bundled-commands dist/core/tools/task/ && cp src/core/export-html/template.* dist/core/export-html/",
+		"build:binary": "bun build --compile ./src/cli.ts --outfile dist/pi",
+		"test": "bun test",
 		"prepublishOnly": "npm run clean && npm run build"
 	},
 	"dependencies": {
-		"@oh-my-pi/pi-agent-core": "workspace:*",
-		"@oh-my-pi/pi-ai": "workspace:*",
-		"@oh-my-pi/pi-tui": "workspace:*",
+		"@oh-my-pi/pi-agent-core": "1.5.0",
+		"@oh-my-pi/pi-ai": "1.5.0",
+		"@oh-my-pi/pi-tui": "1.5.0",
 		"@sinclair/typebox": "^0.34.46",
 		"ajv": "^8.17.1",
 		"chalk": "^5.5.0",
@@ -53,7 +52,9 @@
 		"highlight.js": "^11.11.1",
 		"marked": "^15.0.12",
 		"minimatch": "^10.1.1",
-		"strip-ansi": "^7.1.2"
+		"strip-ansi": "^7.1.2",
+		"winston": "^3.17.0",
+		"winston-daily-rotate-file": "^5.0.0"
 	},
 	"devDependencies": {
 		"@types/diff": "^7.0.2",

package/src/bun-imports.d.ts

@@ -0,0 +1,16 @@
+/**
+ * Type declarations for Bun's import attributes.
+ * These allow importing non-JS files as text at build time.
+ */
+
+// Markdown files imported as text
+declare module "*.md" {
+	const content: string;
+	export default content;
+}
+
+// Text files imported as text
+declare module "*.txt" {
+	const content: string;
+	export default content;
+}

package/src/cli/args.ts

@@ -4,8 +4,8 @@
 
 import type { ThinkingLevel } from "@oh-my-pi/pi-agent-core";
 import chalk from "chalk";
-import { APP_NAME, CONFIG_DIR_NAME, ENV_AGENT_DIR } from "../config.js";
-import { allTools, type ToolName } from "../core/tools/index.js";
+import { APP_NAME, CONFIG_DIR_NAME, ENV_AGENT_DIR } from "../config";
+import { allTools, type ToolName } from "../core/tools/index";
 
 export type Mode = "text" | "json" | "rpc";
 
@@ -152,8 +152,7 @@ ${chalk.bold("Usage:")}
   ${APP_NAME} [options] [@files...] [messages...]
 
 ${chalk.bold("Options:")}
-  --provider <name>              Provider name (default: google)
-  --model <id>                   Model ID (default: gemini-2.5-flash)
+  --model <pattern>              Model to use (fuzzy match: "opus", "gpt-5.2", or "p-openai/gpt-5.2")
   --smol <id>                    Smol/fast model for lightweight tasks (or PI_SMOL_MODEL env)
   --slow <id>                    Slow/reasoning model for thorough analysis (or PI_SLOW_MODEL env)
   --api-key <key>                API key (defaults to env vars)
@@ -199,8 +198,8 @@ ${chalk.bold("Examples:")}
   # Continue previous session
   ${APP_NAME} --continue "What did we discuss?"
 
-  # Use different model
-  ${APP_NAME} --provider openai --model gpt-4o-mini "Help me refactor this code"
+  # Use different model (fuzzy matching)
+  ${APP_NAME} --model opus "Help me refactor this code"
 
   # Limit model cycling to specific models
   ${APP_NAME} --models claude-sonnet,claude-haiku,gpt-4o

package/src/cli/file-processor.ts

@@ -3,11 +3,11 @@
  */
 
 import { access, readFile, stat } from "node:fs/promises";
+import { resolve } from "node:path";
 import type { ImageContent } from "@oh-my-pi/pi-ai";
 import chalk from "chalk";
-import { resolve } from "path";
-import { resolveReadPath } from "../core/tools/path-utils.js";
-import { detectSupportedImageMimeTypeFromFile } from "../utils/mime.js";
+import { resolveReadPath } from "../core/tools/path-utils";
+import { detectSupportedImageMimeTypeFromFile } from "../utils/mime";
 
 export interface ProcessedFiles {
 	text: string;

package/src/cli/list-models.ts

@@ -3,8 +3,8 @@
  */
 
 import type { Api, Model } from "@oh-my-pi/pi-ai";
-import type { ModelRegistry } from "../core/model-registry.js";
-import { fuzzyFilter } from "../utils/fuzzy.js";
+import type { ModelRegistry } from "../core/model-registry";
+import { fuzzyFilter } from "../utils/fuzzy";
 
 /**
  * Format a number as human-readable (e.g., 200000 -> "200K", 1000000 -> "1M")

package/src/cli/plugin-cli.ts

@@ -5,7 +5,7 @@
  */
 
 import chalk from "chalk";
-import { PluginManager, parseSettingValue, validateSetting } from "../core/plugins/index.js";
+import { PluginManager, parseSettingValue, validateSetting } from "../core/plugins/index";
 
 // =============================================================================
 // Types

package/src/cli/session-picker.ts

@@ -3,8 +3,8 @@
  */
 
 import { ProcessTerminal, TUI } from "@oh-my-pi/pi-tui";
-import type { SessionInfo } from "../core/session-manager.js";
-import { SessionSelectorComponent } from "../modes/interactive/components/session-selector.js";
+import type { SessionInfo } from "../core/session-manager";
+import { SessionSelectorComponent } from "../modes/interactive/components/session-selector";
 
 /** Show TUI session selector and return selected session path or null if cancelled */
 export async function selectSession(sessions: SessionInfo[]): Promise<string | null> {

package/src/cli/update-cli.ts

@@ -0,0 +1,273 @@
+/**
+ * Update CLI command handler.
+ *
+ * Handles `pi update` to check for and install updates.
+ * Uses bun if available, otherwise downloads binary from GitHub releases.
+ */
+
+import { execSync, spawnSync } from "node:child_process";
+import { createWriteStream, existsSync, renameSync, unlinkSync } from "node:fs";
+import { dirname } from "node:path";
+import { Readable } from "node:stream";
+import { pipeline } from "node:stream/promises";
+import chalk from "chalk";
+import { VERSION } from "../config";
+
+/**
+ * Detect if we're running as a Bun compiled binary.
+ */
+const isBunBinary =
+	import.meta.url.includes("$bunfs") || import.meta.url.includes("~BUN") || import.meta.url.includes("%7EBUN");
+
+const REPO = "can1357/oh-my-pi";
+const PACKAGE = "@oh-my-pi/pi-coding-agent";
+
+interface ReleaseInfo {
+	tag: string;
+	version: string;
+	assets: Array<{ name: string; url: string }>;
+}
+
+/**
+ * Parse update subcommand arguments.
+ * Returns undefined if not an update command.
+ */
+export function parseUpdateArgs(args: string[]): { force: boolean; check: boolean } | undefined {
+	if (args.length === 0 || args[0] !== "update") {
+		return undefined;
+	}
+
+	return {
+		force: args.includes("--force") || args.includes("-f"),
+		check: args.includes("--check") || args.includes("-c"),
+	};
+}
+
+/**
+ * Check if bun is available in PATH.
+ */
+function hasBun(): boolean {
+	try {
+		const result = spawnSync("bun", ["--version"], { encoding: "utf-8", stdio: "pipe" });
+		return result.status === 0;
+	} catch {
+		return false;
+	}
+}
+
+/**
+ * Get the latest release info from GitHub.
+ */
+async function getLatestRelease(): Promise<ReleaseInfo> {
+	const response = await fetch(`https://api.github.com/repos/${REPO}/releases/latest`);
+	if (!response.ok) {
+		throw new Error(`Failed to fetch release info: ${response.statusText}`);
+	}
+
+	const data = (await response.json()) as {
+		tag_name: string;
+		assets: Array<{ name: string; browser_download_url: string }>;
+	};
+
+	return {
+		tag: data.tag_name,
+		version: data.tag_name.replace(/^v/, ""),
+		assets: data.assets.map((a) => ({ name: a.name, url: a.browser_download_url })),
+	};
+}
+
+/**
+ * Compare semver versions. Returns:
+ * - negative if a < b
+ * - 0 if a == b
+ * - positive if a > b
+ */
+function compareVersions(a: string, b: string): number {
+	const pa = a.split(".").map(Number);
+	const pb = b.split(".").map(Number);
+
+	for (let i = 0; i < Math.max(pa.length, pb.length); i++) {
+		const na = pa[i] || 0;
+		const nb = pb[i] || 0;
+		if (na !== nb) return na - nb;
+	}
+	return 0;
+}
+
+/**
+ * Get the appropriate binary name for this platform.
+ */
+function getBinaryName(): string {
+	const platform = process.platform;
+	const arch = process.arch;
+
+	let os: string;
+	switch (platform) {
+		case "linux":
+			os = "linux";
+			break;
+		case "darwin":
+			os = "darwin";
+			break;
+		case "win32":
+			os = "windows";
+			break;
+		default:
+			throw new Error(`Unsupported platform: ${platform}`);
+	}
+
+	let archName: string;
+	switch (arch) {
+		case "x64":
+			archName = "x64";
+			break;
+		case "arm64":
+			archName = "arm64";
+			break;
+		default:
+			throw new Error(`Unsupported architecture: ${arch}`);
+	}
+
+	if (os === "windows") {
+		return `pi-${os}-${archName}.exe`;
+	}
+	return `pi-${os}-${archName}`;
+}
+
+/**
+ * Update via bun package manager.
+ */
+async function updateViaBun(): Promise<void> {
+	console.log(chalk.dim("Updating via bun..."));
+
+	try {
+		execSync(`bun update -g ${PACKAGE}`, { stdio: "inherit" });
+		console.log(chalk.green("\n✓ Update complete"));
+	} catch {
+		throw new Error("bun update failed");
+	}
+}
+
+/**
+ * Update by downloading binary from GitHub releases.
+ */
+async function updateViaBinary(release: ReleaseInfo): Promise<void> {
+	const binaryName = getBinaryName();
+	const asset = release.assets.find((a) => a.name === binaryName);
+
+	if (!asset) {
+		throw new Error(`No binary found for ${binaryName}`);
+	}
+
+	const execPath = process.execPath;
+	const _execDir = dirname(execPath);
+	const tempPath = `${execPath}.new`;
+	const backupPath = `${execPath}.bak`;
+
+	console.log(chalk.dim(`Downloading ${binaryName}...`));
+
+	// Download to temp file
+	const response = await fetch(asset.url, { redirect: "follow" });
+	if (!response.ok || !response.body) {
+		throw new Error(`Download failed: ${response.statusText}`);
+	}
+
+	const fileStream = createWriteStream(tempPath, { mode: 0o755 });
+	const nodeStream = Readable.fromWeb(response.body as import("stream/web").ReadableStream);
+	await pipeline(nodeStream, fileStream);
+
+	// Replace current binary
+	console.log(chalk.dim("Installing update..."));
+
+	try {
+		// Backup current binary
+		if (existsSync(backupPath)) {
+			unlinkSync(backupPath);
+		}
+		renameSync(execPath, backupPath);
+
+		// Move new binary into place
+		renameSync(tempPath, execPath);
+
+		// Clean up backup
+		unlinkSync(backupPath);
+
+		console.log(chalk.green(`\n✓ Updated to ${release.version}`));
+		console.log(chalk.dim("Restart pi to use the new version"));
+	} catch (err) {
+		// Restore from backup if possible
+		if (existsSync(backupPath) && !existsSync(execPath)) {
+			renameSync(backupPath, execPath);
+		}
+		if (existsSync(tempPath)) {
+			unlinkSync(tempPath);
+		}
+		throw err;
+	}
+}
+
+/**
+ * Run the update command.
+ */
+export async function runUpdateCommand(opts: { force: boolean; check: boolean }): Promise<void> {
+	console.log(chalk.dim(`Current version: ${VERSION}`));
+
+	// Check for updates
+	let release: ReleaseInfo;
+	try {
+		release = await getLatestRelease();
+	} catch (err) {
+		console.error(chalk.red(`Failed to check for updates: ${err}`));
+		process.exit(1);
+	}
+
+	const comparison = compareVersions(release.version, VERSION);
+
+	if (comparison <= 0 && !opts.force) {
+		console.log(chalk.green("✓ Already up to date"));
+		return;
+	}
+
+	if (comparison > 0) {
+		console.log(chalk.cyan(`New version available: ${release.version}`));
+	} else {
+		console.log(chalk.yellow(`Forcing reinstall of ${release.version}`));
+	}
+
+	if (opts.check) {
+		// Just check, don't install
+		return;
+	}
+
+	// Choose update method
+	try {
+		if (!isBunBinary && hasBun()) {
+			await updateViaBun();
+		} else {
+			await updateViaBinary(release);
+		}
+	} catch (err) {
+		console.error(chalk.red(`Update failed: ${err}`));
+		process.exit(1);
+	}
+}
+
+/**
+ * Print update command help.
+ */
+export function printUpdateHelp(): void {
+	console.log(`${chalk.bold("pi update")} - Check for and install updates
+
+${chalk.bold("Usage:")}
+  pi update [options]
+
+${chalk.bold("Options:")}
+  -c, --check   Check for updates without installing
+  -f, --force   Force reinstall even if up to date
+
+${chalk.bold("Examples:")}
+  pi update           Update to latest version
+  pi update --check   Check if updates are available
+  pi update --force   Force reinstall
+`);
+}

package/src/cli.ts

@@ -5,6 +5,6 @@
  *
  * Test with: npx tsx src/cli-new.ts [args...]
  */
-import { main } from "./main.js";
+import { main } from "./main";
 
 main(process.argv.slice(2));