@oh-my-pi/pi-coding-agent 12.17.0 → 12.18.0

package/CHANGELOG.md

@@ -2,6 +2,48 @@
 
 ## [Unreleased]
 
+## [12.18.0] - 2026-02-21
+
+### Added
+
+- Added `overlay` option to custom UI hooks to display components as bottom-centered overlays instead of replacing the editor
+- Added automatic chat transcript rebuild when returning from custom or debug UI to prevent message duplication
+
+### Changed
+
+- Changed custom UI hook cleanup to conditionally restore editor state only when not using overlay mode
+- Extracted environment variable configuration for non-interactive bash execution into reusable `NO_PAGER_ENV` constant
+- Replaced custom timing instrumentation with logger.timeAsync() and logger.time() from pi-utils for consistent startup profiling
+- Removed PI_DEBUG_STARTUP environment variable in favor of logger.debug() for conditional debug output
+- Consolidated timing calls throughout initialization pipeline to use unified logger-based timing system
+
+### Removed
+
+- Deleted utils/timings.ts module - timing functionality now provided by pi-utils logger
+
+### Fixed
+
+- Fixed potential race condition in bash interactive component where output could be appended after the component was closed
+
+## [12.17.2] - 2026-02-21
+### Changed
+
+- Modified bash command normalization to only apply explicit head/tail parameters from tool input, removing automatic extraction from command pipes
+- Updated shell snapshot creation to use explicit timeout and kill signal configuration for more reliable process termination
+
+### Fixed
+
+- Fixed persistent shell session state not being reset after command abort or hard timeout, preventing stale environment variables from affecting subsequent commands
+- Fixed hard timeout handling to properly interrupt long-running commands that exceed the grace period beyond the configured timeout
+
+## [12.17.1] - 2026-02-21
+### Added
+
+- Added `filterBrowser` option to filter out browser automation MCP servers when builtin browser tool is enabled
+- Added `isBrowserMCPServer()` function to detect browser automation MCP servers by name, URL, or command patterns
+- Added `filterBrowserMCPServers()` function to remove browser MCP servers from loaded configurations
+- Added `BrowserFilterResult` type for browser MCP server filtering results
+
 ## [12.17.0] - 2026-02-21
 ### Added
 

package/package.json

@@ -1,50 +1,259 @@
 {
+	"type": "module",
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "12.17.0",
+	"version": "12.18.0",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
-	"type": "module",
-	"bin": {
-		"omp": "src/cli.ts"
+	"homepage": "https://github.com/can1357/oh-my-pi",
+	"author": "Can Bölük",
+	"contributors": [
+		"Mario Zechner"
+	],
+	"license": "MIT",
+	"repository": {
+		"type": "git",
+		"url": "git+https://github.com/can1357/oh-my-pi.git",
+		"directory": "packages/coding-agent"
 	},
+	"bugs": {
+		"url": "https://github.com/can1357/oh-my-pi/issues"
+	},
+	"keywords": [
+		"coding-agent",
+		"ai",
+		"llm",
+		"cli",
+		"tui",
+		"agent"
+	],
 	"main": "./src/index.ts",
 	"types": "./src/index.ts",
+	"bin": {
+		"omp": "src/cli.ts"
+	},
+	"scripts": {
+		"check": "tsgo -p tsconfig.json",
+		"format-prompts": "bun scripts/format-prompts.ts",
+		"generate-docs-index": "bun scripts/generate-docs-index.ts",
+		"prepack": "bun scripts/generate-docs-index.ts",
+		"build:binary": "cd ../.. && bun --cwd=packages/stats scripts/generate-client-bundle.ts && bun --cwd=packages/natives run embed:native && bun build --compile --define PI_COMPILED=true --root . ./packages/coding-agent/src/cli.ts --outfile packages/coding-agent/dist/omp && bun --cwd=packages/natives run embed:native --reset && bun --cwd=packages/stats scripts/generate-client-bundle.ts --reset",
+		"generate-template": "bun scripts/generate-template.ts",
+		"test": "bun test"
+	},
+	"dependencies": {
+		"@mozilla/readability": "0.6.0",
+		"@oh-my-pi/omp-stats": "12.18.0",
+		"@oh-my-pi/pi-agent-core": "12.18.0",
+		"@oh-my-pi/pi-ai": "12.18.0",
+		"@oh-my-pi/pi-natives": "12.18.0",
+		"@oh-my-pi/pi-tui": "12.18.0",
+		"@oh-my-pi/pi-utils": "12.18.0",
+		"@sinclair/typebox": "^0.34.48",
+		"@xterm/headless": "^6.0.0",
+		"ajv": "^8.18.0",
+		"chalk": "^5.6.2",
+		"diff": "^8.0.3",
+		"file-type": "^21.3.0",
+		"glob": "^13.0.3",
+		"handlebars": "^4.7.8",
+		"ignore": "^7.0.5",
+		"linkedom": "^0.18.12",
+		"marked": "^17.0.2",
+		"node-html-parser": "^7.0.2",
+		"puppeteer": "^24.37.3",
+		"smol-toml": "^1.6.0",
+		"zod": "^4.3.6"
+	},
+	"devDependencies": {
+		"@types/bun": "^1.3.9",
+		"@types/ms": "^2.1.0",
+		"ms": "^2.1.3"
+	},
+	"engines": {
+		"bun": ">=1.3.7"
+	},
+	"files": [
+		"src",
+		"scripts",
+		"examples",
+		"README.md",
+		"CHANGELOG.md"
+	],
 	"exports": {
 		".": {
 			"types": "./src/index.ts",
 			"import": "./src/index.ts"
 		},
-		"./hooks": {
-			"types": "./src/extensibility/hooks/index.ts",
-			"import": "./src/extensibility/hooks/index.ts"
+		"./*": {
+			"types": "./src/*.ts",
+			"import": "./src/*.ts"
 		},
-		"./sdk": {
-			"types": "./src/sdk.ts",
-			"import": "./src/sdk.ts"
+		"./capability": {
+			"types": "./src/capability/index.ts",
+			"import": "./src/capability/index.ts"
 		},
-		"./prompts/*": "./src/prompts/*.md",
 		"./capability/*": {
 			"types": "./src/capability/*.ts",
 			"import": "./src/capability/*.ts"
 		},
+		"./cli/*": {
+			"types": "./src/cli/*.ts",
+			"import": "./src/cli/*.ts"
+		},
+		"./commands/*": {
+			"types": "./src/commands/*.ts",
+			"import": "./src/commands/*.ts"
+		},
+		"./commit": {
+			"types": "./src/commit/index.ts",
+			"import": "./src/commit/index.ts"
+		},
+		"./commit/*": {
+			"types": "./src/commit/*.ts",
+			"import": "./src/commit/*.ts"
+		},
+		"./commit/agentic": {
+			"types": "./src/commit/agentic/index.ts",
+			"import": "./src/commit/agentic/index.ts"
+		},
+		"./commit/agentic/*": {
+			"types": "./src/commit/agentic/*.ts",
+			"import": "./src/commit/agentic/*.ts"
+		},
+		"./commit/agentic/tools": {
+			"types": "./src/commit/agentic/tools/index.ts",
+			"import": "./src/commit/agentic/tools/index.ts"
+		},
+		"./commit/agentic/tools/*": {
+			"types": "./src/commit/agentic/tools/*.ts",
+			"import": "./src/commit/agentic/tools/*.ts"
+		},
+		"./commit/analysis": {
+			"types": "./src/commit/analysis/index.ts",
+			"import": "./src/commit/analysis/index.ts"
+		},
+		"./commit/analysis/*": {
+			"types": "./src/commit/analysis/*.ts",
+			"import": "./src/commit/analysis/*.ts"
+		},
+		"./commit/changelog": {
+			"types": "./src/commit/changelog/index.ts",
+			"import": "./src/commit/changelog/index.ts"
+		},
+		"./commit/changelog/*": {
+			"types": "./src/commit/changelog/*.ts",
+			"import": "./src/commit/changelog/*.ts"
+		},
+		"./commit/git": {
+			"types": "./src/commit/git/index.ts",
+			"import": "./src/commit/git/index.ts"
+		},
+		"./commit/git/*": {
+			"types": "./src/commit/git/*.ts",
+			"import": "./src/commit/git/*.ts"
+		},
+		"./commit/map-reduce": {
+			"types": "./src/commit/map-reduce/index.ts",
+			"import": "./src/commit/map-reduce/index.ts"
+		},
+		"./commit/map-reduce/*": {
+			"types": "./src/commit/map-reduce/*.ts",
+			"import": "./src/commit/map-reduce/*.ts"
+		},
+		"./commit/utils/*": {
+			"types": "./src/commit/utils/*.ts",
+			"import": "./src/commit/utils/*.ts"
+		},
 		"./config/*": {
 			"types": "./src/config/*.ts",
 			"import": "./src/config/*.ts"
 		},
+		"./debug": {
+			"types": "./src/debug/index.ts",
+			"import": "./src/debug/index.ts"
+		},
+		"./debug/*": {
+			"types": "./src/debug/*.ts",
+			"import": "./src/debug/*.ts"
+		},
+		"./discovery": {
+			"types": "./src/discovery/index.ts",
+			"import": "./src/discovery/index.ts"
+		},
+		"./discovery/*": {
+			"types": "./src/discovery/*.ts",
+			"import": "./src/discovery/*.ts"
+		},
+		"./exa": {
+			"types": "./src/exa/index.ts",
+			"import": "./src/exa/index.ts"
+		},
+		"./exa/*": {
+			"types": "./src/exa/*.ts",
+			"import": "./src/exa/*.ts"
+		},
+		"./exec/*": {
+			"types": "./src/exec/*.ts",
+			"import": "./src/exec/*.ts"
+		},
+		"./export/*": {
+			"types": "./src/export/*.ts",
+			"import": "./src/export/*.ts"
+		},
+		"./export/html": {
+			"types": "./src/export/html/index.ts",
+			"import": "./src/export/html/index.ts"
+		},
+		"./export/html/*": {
+			"types": "./src/export/html/*.ts",
+			"import": "./src/export/html/*.ts"
+		},
 		"./extensibility/*": {
 			"types": "./src/extensibility/*.ts",
 			"import": "./src/extensibility/*.ts"
 		},
-		"./extensibility/hooks/*": {
-			"types": "./src/extensibility/hooks/*.ts",
-			"import": "./src/extensibility/hooks/*.ts"
+		"./extensibility/custom-commands": {
+			"types": "./src/extensibility/custom-commands/index.ts",
+			"import": "./src/extensibility/custom-commands/index.ts"
+		},
+		"./extensibility/custom-commands/*": {
+			"types": "./src/extensibility/custom-commands/*.ts",
+			"import": "./src/extensibility/custom-commands/*.ts"
+		},
+		"./extensibility/custom-commands/bundled/review": {
+			"types": "./src/extensibility/custom-commands/bundled/review/index.ts",
+			"import": "./src/extensibility/custom-commands/bundled/review/index.ts"
+		},
+		"./extensibility/custom-tools": {
+			"types": "./src/extensibility/custom-tools/index.ts",
+			"import": "./src/extensibility/custom-tools/index.ts"
+		},
+		"./extensibility/custom-tools/*": {
+			"types": "./src/extensibility/custom-tools/*.ts",
+			"import": "./src/extensibility/custom-tools/*.ts"
+		},
+		"./extensibility/extensions": {
+			"types": "./src/extensibility/extensions/index.ts",
+			"import": "./src/extensibility/extensions/index.ts"
 		},
 		"./extensibility/extensions/*": {
 			"types": "./src/extensibility/extensions/*.ts",
 			"import": "./src/extensibility/extensions/*.ts"
 		},
-		"./session/*": {
-			"types": "./src/session/*.ts",
-			"import": "./src/session/*.ts"
+		"./extensibility/hooks": {
+			"types": "./src/extensibility/hooks/index.ts",
+			"import": "./src/extensibility/hooks/index.ts"
+		},
+		"./extensibility/hooks/*": {
+			"types": "./src/extensibility/hooks/*.ts",
+			"import": "./src/extensibility/hooks/*.ts"
+		},
+		"./extensibility/plugins": {
+			"types": "./src/extensibility/plugins/index.ts",
+			"import": "./src/extensibility/plugins/index.ts"
+		},
+		"./extensibility/plugins/*": {
+			"types": "./src/extensibility/plugins/*.ts",
+			"import": "./src/extensibility/plugins/*.ts"
 		},
 		"./internal-urls": {
 			"types": "./src/internal-urls/index.ts",
@@ -54,85 +263,198 @@
 			"types": "./src/internal-urls/*.ts",
 			"import": "./src/internal-urls/*.ts"
 		},
+		"./ipy/*": {
+			"types": "./src/ipy/*.ts",
+			"import": "./src/ipy/*.ts"
+		},
+		"./lsp": {
+			"types": "./src/lsp/index.ts",
+			"import": "./src/lsp/index.ts"
+		},
+		"./lsp/*": {
+			"types": "./src/lsp/*.ts",
+			"import": "./src/lsp/*.ts"
+		},
+		"./lsp/clients": {
+			"types": "./src/lsp/clients/index.ts",
+			"import": "./src/lsp/clients/index.ts"
+		},
+		"./lsp/clients/*": {
+			"types": "./src/lsp/clients/*.ts",
+			"import": "./src/lsp/clients/*.ts"
+		},
+		"./mcp": {
+			"types": "./src/mcp/index.ts",
+			"import": "./src/mcp/index.ts"
+		},
+		"./mcp/*": {
+			"types": "./src/mcp/*.ts",
+			"import": "./src/mcp/*.ts"
+		},
+		"./mcp/transports": {
+			"types": "./src/mcp/transports/index.ts",
+			"import": "./src/mcp/transports/index.ts"
+		},
+		"./mcp/transports/*": {
+			"types": "./src/mcp/transports/*.ts",
+			"import": "./src/mcp/transports/*.ts"
+		},
+		"./memories": {
+			"types": "./src/memories/index.ts",
+			"import": "./src/memories/index.ts"
+		},
+		"./memories/*": {
+			"types": "./src/memories/*.ts",
+			"import": "./src/memories/*.ts"
+		},
+		"./modes": {
+			"types": "./src/modes/index.ts",
+			"import": "./src/modes/index.ts"
+		},
+		"./modes/*": {
+			"types": "./src/modes/*.ts",
+			"import": "./src/modes/*.ts"
+		},
+		"./modes/components": {
+			"types": "./src/modes/components/index.ts",
+			"import": "./src/modes/components/index.ts"
+		},
+		"./modes/components/*": {
+			"types": "./src/modes/components/*.ts",
+			"import": "./src/modes/components/*.ts"
+		},
+		"./modes/components/extensions": {
+			"types": "./src/modes/components/extensions/index.ts",
+			"import": "./src/modes/components/extensions/index.ts"
+		},
+		"./modes/components/extensions/*": {
+			"types": "./src/modes/components/extensions/*.ts",
+			"import": "./src/modes/components/extensions/*.ts"
+		},
+		"./modes/components/status-line": {
+			"types": "./src/modes/components/status-line/index.ts",
+			"import": "./src/modes/components/status-line/index.ts"
+		},
+		"./modes/components/status-line/*": {
+			"types": "./src/modes/components/status-line/*.ts",
+			"import": "./src/modes/components/status-line/*.ts"
+		},
+		"./modes/controllers/*": {
+			"types": "./src/modes/controllers/*.ts",
+			"import": "./src/modes/controllers/*.ts"
+		},
+		"./modes/rpc/*": {
+			"types": "./src/modes/rpc/*.ts",
+			"import": "./src/modes/rpc/*.ts"
+		},
+		"./modes/theme/*": {
+			"types": "./src/modes/theme/*.ts",
+			"import": "./src/modes/theme/*.ts"
+		},
+		"./modes/theme/defaults": {
+			"types": "./src/modes/theme/defaults/index.ts",
+			"import": "./src/modes/theme/defaults/index.ts"
+		},
+		"./modes/utils/*": {
+			"types": "./src/modes/utils/*.ts",
+			"import": "./src/modes/utils/*.ts"
+		},
+		"./patch": {
+			"types": "./src/patch/index.ts",
+			"import": "./src/patch/index.ts"
+		},
 		"./patch/*": {
 			"types": "./src/patch/*.ts",
 			"import": "./src/patch/*.ts"
 		},
-		"./cli": {
-			"types": "./src/cli.ts",
-			"import": "./src/cli.ts"
+		"./plan-mode/*": {
+			"types": "./src/plan-mode/*.ts",
+			"import": "./src/plan-mode/*.ts"
 		},
-		"./*": {
-			"types": "./src/*.ts",
-			"import": "./src/*.ts"
+		"./prompts/*": "./src/prompts/*.md",
+		"./secrets": {
+			"types": "./src/secrets/index.ts",
+			"import": "./src/secrets/index.ts"
+		},
+		"./secrets/*": {
+			"types": "./src/secrets/*.ts",
+			"import": "./src/secrets/*.ts"
+		},
+		"./session/*": {
+			"types": "./src/session/*.ts",
+			"import": "./src/session/*.ts"
+		},
+		"./session/compaction": {
+			"types": "./src/session/compaction/index.ts",
+			"import": "./src/session/compaction/index.ts"
+		},
+		"./session/compaction/*": {
+			"types": "./src/session/compaction/*.ts",
+			"import": "./src/session/compaction/*.ts"
+		},
+		"./slash-commands/*": {
+			"types": "./src/slash-commands/*.ts",
+			"import": "./src/slash-commands/*.ts"
+		},
+		"./ssh/*": {
+			"types": "./src/ssh/*.ts",
+			"import": "./src/ssh/*.ts"
+		},
+		"./stt": {
+			"types": "./src/stt/index.ts",
+			"import": "./src/stt/index.ts"
+		},
+		"./stt/*": {
+			"types": "./src/stt/*.ts",
+			"import": "./src/stt/*.ts"
+		},
+		"./task": {
+			"types": "./src/task/index.ts",
+			"import": "./src/task/index.ts"
+		},
+		"./task/*": {
+			"types": "./src/task/*.ts",
+			"import": "./src/task/*.ts"
+		},
+		"./tools": {
+			"types": "./src/tools/index.ts",
+			"import": "./src/tools/index.ts"
+		},
+		"./tools/*": {
+			"types": "./src/tools/*.ts",
+			"import": "./src/tools/*.ts"
+		},
+		"./tui": {
+			"types": "./src/tui/index.ts",
+			"import": "./src/tui/index.ts"
+		},
+		"./tui/*": {
+			"types": "./src/tui/*.ts",
+			"import": "./src/tui/*.ts"
+		},
+		"./utils/*": {
+			"types": "./src/utils/*.ts",
+			"import": "./src/utils/*.ts"
+		},
+		"./web/scrapers": {
+			"types": "./src/web/scrapers/index.ts",
+			"import": "./src/web/scrapers/index.ts"
+		},
+		"./web/scrapers/*": {
+			"types": "./src/web/scrapers/*.ts",
+			"import": "./src/web/scrapers/*.ts"
+		},
+		"./web/search": {
+			"types": "./src/web/search/index.ts",
+			"import": "./src/web/search/index.ts"
+		},
+		"./web/search/*": {
+			"types": "./src/web/search/*.ts",
+			"import": "./src/web/search/*.ts"
+		},
+		"./web/search/providers/*": {
+			"types": "./src/web/search/providers/*.ts",
+			"import": "./src/web/search/providers/*.ts"
 		}
-	},
-	"files": [
-		"src",
-		"scripts",
-		"examples",
-		"README.md",
-		"CHANGELOG.md"
-	],
-	"scripts": {
-		"check": "tsgo -p tsconfig.json",
-		"format-prompts": "bun scripts/format-prompts.ts",
-		"generate-docs-index": "bun scripts/generate-docs-index.ts",
-		"prepack": "bun scripts/generate-docs-index.ts",
-		"build:binary": "cd ../.. && bun --cwd=packages/stats scripts/generate-client-bundle.ts && bun --cwd=packages/natives run embed:native && bun build --compile --define PI_COMPILED=true --root . ./packages/coding-agent/src/cli.ts --outfile packages/coding-agent/dist/omp && bun --cwd=packages/natives run embed:native --reset && bun --cwd=packages/stats scripts/generate-client-bundle.ts --reset",
-		"generate-template": "bun scripts/generate-template.ts",
-		"test": "bun test"
-	},
-	"dependencies": {
-		"@mozilla/readability": "0.6.0",
-		"@oh-my-pi/omp-stats": "12.17.0",
-		"@oh-my-pi/pi-agent-core": "12.17.0",
-		"@oh-my-pi/pi-ai": "12.17.0",
-		"@oh-my-pi/pi-natives": "12.17.0",
-		"@oh-my-pi/pi-tui": "12.17.0",
-		"@oh-my-pi/pi-utils": "12.17.0",
-		"@sinclair/typebox": "^0.34.48",
-		"@xterm/headless": "^6.0.0",
-		"ajv": "^8.18.0",
-		"chalk": "^5.6.2",
-		"diff": "^8.0.3",
-		"file-type": "^21.3.0",
-		"glob": "^13.0.3",
-		"handlebars": "^4.7.8",
-		"ignore": "^7.0.5",
-		"linkedom": "^0.18.12",
-		"marked": "^17.0.2",
-		"node-html-parser": "^7.0.2",
-		"puppeteer": "^24.37.3",
-		"smol-toml": "^1.6.0",
-		"zod": "^4.3.6"
-	},
-	"devDependencies": {
-		"@types/bun": "^1.3.9",
-		"@types/ms": "^2.1.0",
-		"ms": "^2.1.3"
-	},
-	"keywords": [
-		"coding-agent",
-		"ai",
-		"llm",
-		"cli",
-		"tui",
-		"agent"
-	],
-	"author": "Can Bölük",
-	"contributors": ["Mario Zechner"],
-	"license": "MIT",
-	"repository": {
-		"type": "git",
-		"url": "git+https://github.com/can1357/oh-my-pi.git",
-		"directory": "packages/coding-agent"
-	},
-	"homepage": "https://github.com/can1357/oh-my-pi",
-	"bugs": {
-	"url": "https://github.com/can1357/oh-my-pi/issues"
-},
-"engines": {
-	"bun": ">=1.3.7"
-}
+	}
 }

package/src/capability/index.ts

@@ -8,12 +8,9 @@
  */
 import * as os from "node:os";
 import * as path from "node:path";
-import { $env } from "@oh-my-pi/pi-utils";
+import { logger } from "@oh-my-pi/pi-utils";
 import { getProjectDir } from "@oh-my-pi/pi-utils/dirs";
 
-/** Conditional startup debug prints (stderr) when PI_DEBUG_STARTUP is set */
-const debugStartup = $env.PI_DEBUG_STARTUP ? (stage: string) => process.stderr.write(`[startup] ${stage}\n`) : () => {};
-
 import type { Settings } from "../config/settings";
 import { clearCache as clearFsCache, cacheStats as fsCacheStats, invalidate as invalidateFs } from "./fs";
 import type {
@@ -115,12 +112,12 @@ async function loadImpl<T>(
 	const results = await Promise.all(
 		providers.map(async provider => {
 			try {
-				debugStartup(`capability:${capability.id}:${provider.id}:start`);
-				const result = await provider.load(ctx);
-				debugStartup(`capability:${capability.id}:${provider.id}:done`);
+				const result = await logger.timeAsync(`capability:${capability.id}:${provider.id}`, () =>
+					provider.load(ctx),
+				);
 				return { provider, result };
 			} catch (error) {
-				debugStartup(`capability:${capability.id}:${provider.id}:error`);
+				logger.debug(`capability:${capability.id}:${provider.id}:error`);
 				return { provider, error };
 			}
 		}),

package/src/cli/stats-cli.ts

@@ -85,9 +85,9 @@ function formatPercent(n: number): string {
 
 export async function runStatsCommand(cmd: StatsCommandArgs): Promise<void> {
 	// Lazy import to avoid loading stats module when not needed
-	const { getDashboardStats, syncAllSessions, getTotalMessageCount } = await import("@oh-my-pi/omp-stats");
-	const { startServer } = await import("@oh-my-pi/omp-stats/src/server");
-	const { closeDb } = await import("@oh-my-pi/omp-stats/src/db");
+	const { getDashboardStats, syncAllSessions, getTotalMessageCount, startServer, closeDb } = await import(
+		"@oh-my-pi/omp-stats"
+	);
 
 	// Sync session files first
 	console.log("Syncing session files...");