@oh-my-pi/pi-coding-agent 6.8.2 → 6.8.4

package/CHANGELOG.md

@@ -2,6 +2,36 @@
 
 ## [Unreleased]
 
+## [6.8.4] - 2026-01-21
+### Changed
+
+- Updated output sink to properly handle large outputs
+- Improved error message formatting in SSH executor
+- Updated web fetch timeout bounds and conversion
+
+### Fixed
+
+- Fixed output truncation handling in streaming output
+- Fixed timeout handling in web fetch tool
+- Fixed async stream dumping in executors
+
+## [6.8.3] - 2026-01-21
+
+### Changed
+
+- Updated keybinding system to normalize key IDs to lowercase
+- Changed label edit shortcut from 'l' to 'Shift+L' in tree selector
+- Changed output file extension from `.out.md` to `.md` for artifacts
+
+### Removed
+
+- Removed bundled worktree command from custom commands loader
+
+### Fixed
+
+- Fixed keybinding case sensitivity issues by normalizing all key IDs
+- Fixed task artifact path handling and simplified file structure
+
 ## [6.8.2] - 2026-01-21
 
 ### Fixed

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "6.8.2",
+	"version": "6.8.4",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"type": "module",
 	"ompConfig": {
@@ -40,11 +40,11 @@
 		"prepublishOnly": "bun run generate-template && bun run clean && bun run build"
 	},
 	"dependencies": {
-		"@oh-my-pi/pi-agent-core": "6.8.2",
-		"@oh-my-pi/pi-ai": "6.8.2",
-		"@oh-my-pi/pi-git-tool": "6.8.2",
-		"@oh-my-pi/pi-tui": "6.8.2",
-		"@oh-my-pi/pi-utils": "6.8.2",
+		"@oh-my-pi/pi-agent-core": "6.8.4",
+		"@oh-my-pi/pi-ai": "6.8.4",
+		"@oh-my-pi/pi-git-tool": "6.8.4",
+		"@oh-my-pi/pi-tui": "6.8.4",
+		"@oh-my-pi/pi-utils": "6.8.4",
 		"@openai/agents": "^0.3.7",
 		"@sinclair/typebox": "^0.34.46",
 		"ajv": "^8.17.1",

package/src/core/agent-session.ts

@@ -22,7 +22,7 @@ import type { Rule } from "../capability/rule";
 import { getAgentDbPath } from "../config";
 import { theme } from "../modes/interactive/theme/theme";
 import ttsrInterruptTemplate from "../prompts/system/ttsr-interrupt.md" with { type: "text" };
-import { type BashResult, executeBash as executeBashCommand, executeBashWithOperations } from "./bash-executor";
+import { type BashResult, executeBash as executeBashCommand } from "./bash-executor";
 import {
 	type CompactionResult,
 	calculateContextTokens,
@@ -60,10 +60,8 @@ import type { Skill, SkillWarning } from "./skills";
 import { expandSlashCommand, type FileSlashCommand } from "./slash-commands";
 import { closeAllConnections } from "./ssh/connection-manager";
 import { unmountAll } from "./ssh/sshfs-mount";
-import type { BashOperations } from "./tools/bash";
 import { normalizeDiff, normalizeToLF, ParseError, previewPatch, stripBom } from "./tools/patch";
 import { resolveToCwd } from "./tools/path-utils";
-import { getArtifactsDir } from "./tools/task/artifacts";
 import type { TodoItem } from "./tools/todo-write";
 import type { TtsrManager } from "./ttsr";
 
@@ -1991,10 +1989,7 @@ export class AgentSession {
 		const sessionFile = this.sessionManager.getSessionFile();
 		if (!sessionFile) return;
 
-		const artifactsDir = getArtifactsDir(sessionFile);
-		if (!artifactsDir) return;
-
-		const todoPath = `${artifactsDir}/todos.json`;
+		const todoPath = `${sessionFile.slice(0, -6)}/todos.json`;
 		const file = Bun.file(todoPath);
 		if (!(await file.exists())) {
 			this._todoReminderCount = 0;
@@ -2550,25 +2545,19 @@ export class AgentSession {
 	 * @param command The bash command to execute
 	 * @param onChunk Optional streaming callback for output
 	 * @param options.excludeFromContext If true, command output won't be sent to LLM (!! prefix)
-	 * @param options.operations Custom BashOperations for remote execution
 	 */
 	async executeBash(
 		command: string,
 		onChunk?: (chunk: string) => void,
-		options?: { excludeFromContext?: boolean; operations?: BashOperations },
+		options?: { excludeFromContext?: boolean },
 	): Promise<BashResult> {
 		this._bashAbortController = new AbortController();
 
 		try {
-			const result = options?.operations
-				? await executeBashWithOperations(command, process.cwd(), options.operations, {
-						onChunk,
-						signal: this._bashAbortController.signal,
-					})
-				: await executeBashCommand(command, {
-						onChunk,
-						signal: this._bashAbortController.signal,
-					});
+			const result = await executeBashCommand(command, {
+				onChunk,
+				signal: this._bashAbortController.signal,
+			});
 
 			this.recordBashResult(command, result, options);
 			return result;

package/src/core/bash-executor.ts

@@ -8,7 +8,6 @@ import { cspawn, Exception, ptree } from "@oh-my-pi/pi-utils";
 import { getShellConfig } from "../utils/shell";
 import { getOrCreateSnapshot, getSnapshotSourceCommand } from "../utils/shell-snapshot";
 import { OutputSink } from "./streaming-output";
-import type { BashOperations } from "./tools/bash";
 
 export interface BashExecutorOptions {
 	cwd?: string;
@@ -34,7 +33,7 @@ export async function executeBash(command: string, options?: BashExecutorOptions
 	const prefixedCommand = prefix ? `${prefix} ${command}` : command;
 	const finalCommand = `${snapshotPrefix}${prefixedCommand}`;
 
-	const stream = new OutputSink({ onChunk: options?.onChunk });
+	const sink = new OutputSink({ onChunk: options?.onChunk });
 
 	const child = cspawn([shell, ...args, finalCommand], {
 		cwd: options?.cwd,
@@ -45,12 +44,9 @@ export async function executeBash(command: string, options?: BashExecutorOptions
 
 	// Pump streams - errors during abort/timeout are expected
 	// Use preventClose to avoid closing the shared sink when either stream finishes
-	await Promise.allSettled([
-		child.stdout.pipeTo(stream.createWritable()),
-		child.stderr.pipeTo(stream.createWritable()),
-	])
-		.then(() => stream.close())
-		.catch(() => {});
+	await Promise.allSettled([child.stdout.pipeTo(sink.createInput()), child.stderr.pipeTo(sink.createInput())]).catch(
+		() => {},
+	);
 
 	// Wait for process exit
 	try {
@@ -58,7 +54,7 @@ export async function executeBash(command: string, options?: BashExecutorOptions
 		return {
 			exitCode: child.exitCode ?? 0,
 			cancelled: false,
-			...stream.dump(),
+			...(await sink.dump()),
 		};
 	} catch (err) {
 		// Exception covers NonZeroExitError, AbortError, TimeoutError
@@ -71,7 +67,7 @@ export async function executeBash(command: string, options?: BashExecutorOptions
 				return {
 					exitCode: undefined,
 					cancelled: true,
-					...stream.dump(annotation),
+					...(await sink.dump(annotation)),
 				};
 			}
 
@@ -79,60 +75,7 @@ export async function executeBash(command: string, options?: BashExecutorOptions
 			return {
 				exitCode: err.exitCode,
 				cancelled: false,
-				...stream.dump(),
-			};
-		}
-
-		throw err;
-	}
-}
-
-export async function executeBashWithOperations(
-	command: string,
-	cwd: string,
-	operations: BashOperations,
-	options?: BashExecutorOptions,
-): Promise<BashResult> {
-	const stream = new OutputSink({ onChunk: options?.onChunk });
-	const writable = stream.createWritable();
-	const writer = writable.getWriter();
-
-	const closeStreams = async () => {
-		try {
-			await writer.close();
-		} catch {}
-		try {
-			await writable.close();
-		} catch {}
-		try {
-			await stream.close();
-		} catch {}
-	};
-
-	try {
-		const result = await operations.exec(command, cwd, {
-			onData: (data) => writer.write(data),
-			signal: options?.signal,
-			timeout: options?.timeout,
-		});
-
-		await closeStreams();
-
-		const cancelled = options?.signal?.aborted ?? false;
-
-		return {
-			exitCode: cancelled ? undefined : (result.exitCode ?? undefined),
-			cancelled,
-			...stream.dump(),
-		};
-	} catch (err) {
-		await closeStreams();
-
-		if (options?.signal?.aborted) {
-			return {
-				exitCode: undefined,
-				cancelled: true,
-				...stream.dump(),
+				...(await sink.dump()),
 			};
 		}
 

package/src/core/custom-commands/loader.ts

@@ -13,7 +13,6 @@ import { getAgentDir, getConfigDirs } from "../../config";
 import * as piCodingAgent from "../../index";
 import { execCommand } from "../exec";
 import { ReviewCommand } from "./bundled/review";
-import { WorktreeCommand } from "./bundled/wt";
 import type {
 	CustomCommand,
 	CustomCommandAPI,
@@ -153,13 +152,6 @@ function loadBundledCommands(sharedApi: CustomCommandAPI): LoadedCustomCommand[]
 		source: "bundled",
 	});
 
-	bundled.push({
-		path: "bundled:wt",
-		resolvedPath: "bundled:wt",
-		command: new WorktreeCommand(sharedApi),
-		source: "bundled",
-	});
-
 	return bundled;
 }
 

package/src/core/extensions/types.ts

@@ -29,7 +29,6 @@ import type {
 	SessionManager,
 } from "../session-manager";
 import type { BashToolDetails, FindToolDetails, GrepToolDetails, LsToolDetails, ReadToolDetails } from "../tools";
-import type { BashOperations } from "../tools/bash";
 import type { EditToolDetails } from "../tools/patch";
 
 export type { ExecOptions, ExecResult } from "../exec";
@@ -551,8 +550,6 @@ export interface InputEventResult {
 
 /** Result from user_bash event handler */
 export interface UserBashEventResult {
-	/** Custom operations to use for execution */
-	operations?: BashOperations;
 	/** Full replacement: extension handled execution, use this result */
 	result?: BashResult;
 }

package/src/core/index.ts

@@ -11,7 +11,7 @@ export {
 	type PromptOptions,
 	type SessionStats,
 } from "./agent-session";
-export { type BashExecutorOptions, type BashResult, executeBash, executeBashWithOperations } from "./bash-executor";
+export { type BashExecutorOptions, type BashResult, executeBash } from "./bash-executor";
 export type { CompactionResult } from "./compaction/index";
 export {
 	discoverAndLoadExtensions,

package/src/core/keybindings.ts

@@ -124,6 +124,8 @@ const KEY_LABELS: Record<string, string> = {
 	right: "Right",
 };
 
+const normalizeKeyId = (key: KeyId): KeyId => key.toLowerCase() as KeyId;
+
 function formatKeyPart(part: string): string {
 	const lower = part.toLowerCase();
 	const modifier = MODIFIER_LABELS[lower];
@@ -199,14 +201,20 @@ export class KeybindingsManager {
 		// Set defaults for app actions
 		for (const [action, keys] of Object.entries(DEFAULT_APP_KEYBINDINGS)) {
 			const keyArray = Array.isArray(keys) ? keys : [keys];
-			this.appActionToKeys.set(action as AppAction, [...keyArray]);
+			this.appActionToKeys.set(
+				action as AppAction,
+				keyArray.map((key) => normalizeKeyId(key as KeyId)),
+			);
 		}
 
 		// Override with user config (app actions only)
 		for (const [action, keys] of Object.entries(this.config)) {
 			if (keys === undefined || !isAppAction(action)) continue;
 			const keyArray = Array.isArray(keys) ? keys : [keys];
-			this.appActionToKeys.set(action, keyArray);
+			this.appActionToKeys.set(
+				action,
+				keyArray.map((key) => normalizeKeyId(key as KeyId)),
+			);
 		}
 	}
 

package/src/core/python-executor.ts

@@ -1,4 +1,4 @@
-import { logger, sanitizeText } from "@oh-my-pi/pi-utils";
+import { logger } from "@oh-my-pi/pi-utils";
 import {
 	checkPythonKernelAvailability,
 	type KernelDisplayOutput,
@@ -210,30 +210,20 @@ async function executeWithKernel(
 	code: string,
 	options: PythonExecutorOptions | undefined,
 ): Promise<PythonResult> {
-	const sink = new OutputSink({ onLine: options?.onChunk });
+	const sink = new OutputSink({ onChunk: options?.onChunk });
 	const displayOutputs: KernelDisplayOutput[] = [];
 
 	try {
-		const writable = sink.createStringWritable();
-		const writer = writable.getWriter();
-		let result: KernelExecuteResult;
-		try {
-			result = await kernel.execute(code, {
-				signal: options?.signal,
-				timeoutMs: options?.timeout,
-				onChunk: (text) => {
-					writer.write(sanitizeText(text));
-				},
-				onDisplay: (output) => {
-					displayOutputs.push(output);
-				},
-			});
-		} catch (err) {
-			await writer.abort(err);
-			throw err;
-		} finally {
-			await writer.close().catch(() => {});
-		}
+		const result = await kernel.execute(code, {
+			signal: options?.signal,
+			timeoutMs: options?.timeout,
+			onChunk: (text) => {
+				sink.push(text);
+			},
+			onDisplay: (output) => {
+				displayOutputs.push(output);
+			},
+		});
 
 		if (result.cancelled) {
 			const secs = options?.timeout ? Math.round(options.timeout / 1000) : undefined;
@@ -244,7 +234,7 @@ async function executeWithKernel(
 				cancelled: true,
 				displayOutputs,
 				stdinRequested: result.stdinRequested,
-				...sink.dump(annotation),
+				...(await sink.dump(annotation)),
 			};
 		}
 
@@ -254,7 +244,7 @@ async function executeWithKernel(
 				cancelled: false,
 				displayOutputs,
 				stdinRequested: true,
-				...sink.dump("Kernel requested stdin; interactive input is not supported."),
+				...(await sink.dump("Kernel requested stdin; interactive input is not supported.")),
 			};
 		}
 
@@ -264,7 +254,7 @@ async function executeWithKernel(
 			cancelled: false,
 			displayOutputs,
 			stdinRequested: false,
-			...sink.dump(),
+			...(await sink.dump()),
 		};
 	} catch (err) {
 		const error = err instanceof Error ? err : new Error(String(err));

package/src/core/ssh/ssh-executor.ts

@@ -70,16 +70,11 @@ export async function executeSSH(
 		timeout: options?.timeout,
 	});
 
-	const sink = new OutputSink({ onLine: options?.onChunk });
+	const sink = new OutputSink({ onChunk: options?.onChunk });
 
-	try {
-		await Promise.allSettled([
-			child.stdout.pipeTo(sink.createWritable()),
-			child.stderr.pipeTo(sink.createWritable()),
-		]);
-	} finally {
-		await sink.close();
-	}
+	await Promise.allSettled([child.stdout.pipeTo(sink.createInput()), child.stderr.pipeTo(sink.createInput())]).catch(
+		() => {},
+	);
 
 	try {
 		await child.exited;
@@ -87,7 +82,7 @@ export async function executeSSH(
 		return {
 			exitCode,
 			cancelled: false,
-			...sink.dump(),
+			...(await sink.dump()),
 		};
 	} catch (err) {
 		if (err instanceof ptree.Exception) {
@@ -95,20 +90,20 @@ export async function executeSSH(
 				return {
 					exitCode: undefined,
 					cancelled: true,
-					...sink.dump(`SSH: ${err.message}`),
+					...(await sink.dump(`SSH: ${err.message}`)),
 				};
 			}
 			if (err.aborted) {
 				return {
 					exitCode: undefined,
 					cancelled: true,
-					...sink.dump(`SSH command aborted: ${err.message}`),
+					...(await sink.dump(`Command aborted: ${err.message}`)),
 				};
 			}
 			return {
 				exitCode: err.exitCode,
 				cancelled: false,
-				...sink.dump(`Unexpected error: ${err.message}`),
+				...(await sink.dump(`Unexpected error: ${err.message}`)),
 			};
 		}
 		throw err;