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

package/CHANGELOG.md

@@ -2,6 +2,27 @@
 
 ## [Unreleased]
 
+## [6.9.0] - 2026-01-21
+### Removed
+
+- Removed Git tool and all related functionality
+- Removed voice control and TTS features
+- Removed worktree management system
+- Removed bundled wt custom command
+- Removed voice-related settings and configuration options
+- Removed @oh-my-pi/pi-git-tool dependency
+
+## [6.8.5] - 2026-01-21
+### Breaking Changes
+
+- Changed timeout parameter from seconds to milliseconds in Python tool
+- Updated PythonExecutorOptions interface to use timeoutMs instead of timeout
+
+### Changed
+
+- Updated default timeout to 30000ms (30 seconds) for Python tool
+- Improved streaming output handling and buffer management
+
 ## [6.8.4] - 2026-01-21
 ### Changed
 

package/package.json

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

package/src/core/python-executor.ts

@@ -14,9 +14,9 @@ export interface PythonExecutorOptions {
 	/** Working directory for command execution */
 	cwd?: string;
 	/** Timeout in milliseconds */
-	timeout?: number;
+	timeoutMs?: number;
 	/** Callback for streaming output chunks (already sanitized) */
-	onChunk?: (chunk: string) => void;
+	onChunk?: (chunk: string) => Promise<void> | void;
 	/** AbortSignal for cancellation */
 	signal?: AbortSignal;
 	/** Session identifier for kernel reuse */
@@ -216,17 +216,13 @@ async function executeWithKernel(
 	try {
 		const result = await kernel.execute(code, {
 			signal: options?.signal,
-			timeoutMs: options?.timeout,
-			onChunk: (text) => {
-				sink.push(text);
-			},
-			onDisplay: (output) => {
-				displayOutputs.push(output);
-			},
+			timeoutMs: options?.timeoutMs,
+			onChunk: (text) => sink.push(text),
+			onDisplay: (output) => void displayOutputs.push(output),
 		});
 
 		if (result.cancelled) {
-			const secs = options?.timeout ? Math.round(options.timeout / 1000) : undefined;
+			const secs = options?.timeoutMs ? Math.round(options.timeoutMs / 1000) : undefined;
 			const annotation =
 				result.timedOut && secs !== undefined ? `Command timed out after ${secs} seconds` : undefined;
 			return {

package/src/core/sdk.ts

@@ -86,7 +86,6 @@ import {
 	createTools,
 	EditTool,
 	FindTool,
-	GitTool,
 	GrepTool,
 	getWebSearchTools,
 	LsTool,
@@ -218,7 +217,6 @@ export {
 	createTools,
 	EditTool,
 	FindTool,
-	GitTool,
 	GrepTool,
 	loadSshTool,
 	LsTool,

package/src/core/settings-manager.ts

@@ -98,10 +98,6 @@ export interface BashInterceptorSettings {
 	patterns?: BashInterceptorRule[]; // default: built-in rules
 }
 
-export interface GitSettings {
-	enabled?: boolean; // default: false (structured git tool; use bash for git commands when disabled)
-}
-
 export interface MCPSettings {
 	enableProjectConfig?: boolean; // default: true (load .mcp.json from project root)
 }
@@ -145,15 +141,6 @@ export interface TodoCompletionSettings {
 	maxReminders?: number; // default: 3 - maximum reminders before giving up
 }
 
-export interface VoiceSettings {
-	enabled?: boolean; // default: false
-	transcriptionModel?: string; // default: "whisper-1"
-	transcriptionLanguage?: string; // optional language hint (e.g., "en")
-	ttsModel?: string; // default: "gpt-4o-mini-tts"
-	ttsVoice?: string; // default: "alloy"
-	ttsFormat?: "wav" | "mp3" | "opus" | "aac" | "flac"; // default: "wav"
-}
-
 export type StatusLineSegmentId =
 	| "pi"
 	| "model"
@@ -224,14 +211,12 @@ export interface Settings {
 	enabledModels?: string[]; // Model patterns for cycling (same format as --models CLI flag)
 	exa?: ExaSettings;
 	bashInterceptor?: BashInterceptorSettings;
-	git?: GitSettings;
 	mcp?: MCPSettings;
 	lsp?: LspSettings;
 	python?: PythonSettings;
 	edit?: EditSettings;
 	ttsr?: TtsrSettings;
 	todoCompletion?: TodoCompletionSettings;
-	voice?: VoiceSettings;
 	providers?: ProviderSettings;
 	disabledProviders?: string[]; // Discovery provider IDs that are disabled
 	disabledExtensions?: string[]; // Individual extension IDs that are disabled (e.g., "skill:commit")
@@ -252,12 +237,6 @@ export const DEFAULT_BASH_INTERCEPTOR_RULES: BashInterceptorRule[] = [
 		tool: "grep",
 		message: "Use the `grep` tool instead of grep/rg. It respects .gitignore and provides structured output.",
 	},
-	{
-		pattern: "^\\s*git(\\s+|$)",
-		tool: "git",
-		message:
-			"Use the `git` tool instead of running git in bash. It provides structured output and safety confirmations.",
-	},
 	{
 		pattern: "^\\s*(find|fd|locate)\\s+.*(-name|-iname|-type|--type|-glob)",
 		tool: "find",
@@ -319,19 +298,11 @@ const DEFAULT_SETTINGS: Settings = {
 		enableWebsets: false,
 	},
 	bashInterceptor: DEFAULT_BASH_INTERCEPTOR_SETTINGS,
-	git: { enabled: false },
 	mcp: { enableProjectConfig: true },
 	lsp: { formatOnWrite: false, diagnosticsOnWrite: true, diagnosticsOnEdit: false },
 	python: { toolMode: "both", kernelMode: "session", sharedGateway: true },
 	edit: { fuzzyMatch: true, fuzzyThreshold: 0.95, streamingAbort: false },
 	ttsr: { enabled: true, contextMode: "discard", repeatMode: "once", repeatGap: 10 },
-	voice: {
-		enabled: false,
-		transcriptionModel: "whisper-1",
-		ttsModel: "gpt-4o-mini-tts",
-		ttsVoice: "alloy",
-		ttsFormat: "wav",
-	},
 	providers: { webSearch: "auto", image: "auto" },
 } satisfies Settings;
 
@@ -1170,10 +1141,6 @@ export class SettingsManager {
 		await this.save();
 	}
 
-	getGitToolEnabled(): boolean {
-		return this.settings.git?.enabled ?? false;
-	}
-
 	getPythonToolMode(): PythonToolMode {
 		return this.settings.python?.toolMode ?? "both";
 	}
@@ -1210,14 +1177,6 @@ export class SettingsManager {
 		await this.save();
 	}
 
-	async setGitToolEnabled(enabled: boolean): Promise<void> {
-		if (!this.globalSettings.git) {
-			this.globalSettings.git = {};
-		}
-		this.globalSettings.git.enabled = enabled;
-		await this.save();
-	}
-
 	getMCPProjectConfigEnabled(): boolean {
 		return this.settings.mcp?.enableProjectConfig ?? true;
 	}
@@ -1430,70 +1389,6 @@ export class SettingsManager {
 		await this.save();
 	}
 
-	getVoiceSettings(): Required<VoiceSettings> {
-		return {
-			enabled: this.settings.voice?.enabled ?? false,
-			transcriptionModel: this.settings.voice?.transcriptionModel ?? "whisper-1",
-			transcriptionLanguage: this.settings.voice?.transcriptionLanguage ?? "",
-			ttsModel: this.settings.voice?.ttsModel ?? "tts-1",
-			ttsVoice: this.settings.voice?.ttsVoice ?? "alloy",
-			ttsFormat: this.settings.voice?.ttsFormat ?? "wav",
-		};
-	}
-
-	async setVoiceSettings(settings: VoiceSettings): Promise<void> {
-		this.globalSettings.voice = { ...this.globalSettings.voice, ...settings };
-		await this.save();
-	}
-
-	getVoiceEnabled(): boolean {
-		return this.settings.voice?.enabled ?? false;
-	}
-
-	async setVoiceEnabled(enabled: boolean): Promise<void> {
-		if (!this.globalSettings.voice) {
-			this.globalSettings.voice = {};
-		}
-		this.globalSettings.voice.enabled = enabled;
-		await this.save();
-	}
-
-	getVoiceTtsModel(): string {
-		return this.settings.voice?.ttsModel ?? "gpt-4o-mini-tts";
-	}
-
-	async setVoiceTtsModel(model: string): Promise<void> {
-		if (!this.globalSettings.voice) {
-			this.globalSettings.voice = {};
-		}
-		this.globalSettings.voice.ttsModel = model;
-		await this.save();
-	}
-
-	getVoiceTtsVoice(): string {
-		return this.settings.voice?.ttsVoice ?? "alloy";
-	}
-
-	async setVoiceTtsVoice(voice: string): Promise<void> {
-		if (!this.globalSettings.voice) {
-			this.globalSettings.voice = {};
-		}
-		this.globalSettings.voice.ttsVoice = voice;
-		await this.save();
-	}
-
-	getVoiceTtsFormat(): "wav" | "mp3" | "opus" | "aac" | "flac" {
-		return this.settings.voice?.ttsFormat ?? "wav";
-	}
-
-	async setVoiceTtsFormat(format: "wav" | "mp3" | "opus" | "aac" | "flac"): Promise<void> {
-		if (!this.globalSettings.voice) {
-			this.globalSettings.voice = {};
-		}
-		this.globalSettings.voice.ttsFormat = format;
-		await this.save();
-	}
-
 	// ═══════════════════════════════════════════════════════════════════════════
 	// Status Line Settings
 	// ═══════════════════════════════════════════════════════════════════════════

package/src/core/streaming-output.ts

@@ -33,8 +33,6 @@ export class OutputSink {
 		path: string;
 		sink: Bun.FileSink;
 	};
-	#bytesWritten: number = 0;
-	#pending: Promise<void> = Promise.resolve();
 
 	readonly #allocateFilePath: () => string;
 	readonly #spillThreshold: number;
@@ -54,15 +52,16 @@ export class OutputSink {
 
 	async #pushSanitized(data: string): Promise<void> {
 		this.#onChunk?.(data);
-		const dataBytes = Buffer.byteLength(data);
-		const overflow = dataBytes + this.#bytesWritten > this.#spillThreshold || this.#file != null;
+
+		const bufferOverflow = data.length + this.#buffer.length > this.#spillThreshold;
+		const overflow = this.#file || bufferOverflow;
 
 		const sink = overflow ? await this.#fileSink() : null;
 
 		this.#buffer += data;
 		await sink?.write(data);
 
-		if (this.#buffer.length > this.#spillThreshold) {
+		if (bufferOverflow) {
 			this.#buffer = this.#buffer.slice(-this.#spillThreshold);
 		}
 	}
@@ -81,28 +80,21 @@ export class OutputSink {
 
 	async push(chunk: string): Promise<void> {
 		chunk = sanitizeText(chunk);
-		const op = this.#pending.then(() => this.#pushSanitized(chunk));
-		this.#pending = op.catch(() => {});
-		await op;
+		await this.#pushSanitized(chunk);
 	}
 
 	createInput(): WritableStream<Uint8Array | string> {
-		let decoder: TextDecoder | undefined;
-		let finalize = async () => {};
+		const dec = new TextDecoder("utf-8", { ignoreBOM: true });
+		const finalize = async () => {
+			await this.push(dec.decode());
+		};
 
 		return new WritableStream<Uint8Array | string>({
 			write: async (chunk) => {
 				if (typeof chunk === "string") {
 					await this.push(chunk);
 				} else {
-					if (!decoder) {
-						const dec = new TextDecoder("utf-8", { ignoreBOM: true });
-						decoder = dec;
-						finalize = async () => {
-							await this.push(dec.decode());
-						};
-					}
-					await this.push(decoder.decode(chunk, { stream: true }));
+					await this.push(dec.decode(chunk, { stream: true }));
 				}
 			},
 			close: finalize,
@@ -111,7 +103,6 @@ export class OutputSink {
 	}
 
 	async dump(notice?: string): Promise<OutputResult> {
-		await this.#pending;
 		const noticeLine = notice ? `[${notice}]\n` : "";
 
 		if (this.#file) {

package/src/core/tools/index.ts

@@ -7,7 +7,6 @@ export { exaTools } from "./exa/index";
 export type { ExaRenderDetails, ExaSearchResponse, ExaSearchResult } from "./exa/types";
 export { type FindOperations, FindTool, type FindToolDetails, type FindToolOptions } from "./find";
 export { setPreferredImageProvider } from "./gemini-image";
-export { GitTool, type GitToolDetails } from "./git";
 export { type GrepOperations, GrepTool, type GrepToolDetails, type GrepToolOptions } from "./grep";
 export { type LsOperations, LsTool, type LsToolDetails, type LsToolOptions } from "./ls";
 export {
@@ -72,7 +71,6 @@ import { BashTool } from "./bash";
 import { CalculatorTool } from "./calculator";
 import { CompleteTool } from "./complete";
 import { FindTool } from "./find";
-import { GitTool } from "./git";
 import { GrepTool } from "./grep";
 import { LsTool } from "./ls";
 import { LspTool } from "./lsp/index";
@@ -132,7 +130,6 @@ export interface ToolSession {
 		getEditFuzzyMatch(): boolean;
 		getEditFuzzyThreshold?(): number;
 		getEditPatchMode?(): boolean;
-		getGitToolEnabled(): boolean;
 		getBashInterceptorEnabled(): boolean;
 		getBashInterceptorSimpleLsEnabled(): boolean;
 		getBashInterceptorRules(): BashInterceptorRule[];
@@ -152,7 +149,6 @@ export const BUILTIN_TOOLS: Record<string, ToolFactory> = {
 	ssh: loadSshTool,
 	edit: (s) => new EditTool(s),
 	find: (s) => new FindTool(s),
-	git: GitTool.createIf,
 	grep: (s) => new GrepTool(s),
 	ls: (s) => new LsTool(s),
 	lsp: LspTool.createIf,

package/src/core/tools/python.ts

@@ -40,7 +40,7 @@ function groupPreludeHelpers(helpers: PreludeHelper[]): PreludeCategory[] {
 
 export const pythonSchema = Type.Object({
 	code: Type.String({ description: "Python code to execute" }),
-	timeout: Type.Optional(Type.Number({ description: "Timeout in seconds (optional, no default timeout)" })),
+	timeoutMs: Type.Optional(Type.Number({ description: "Timeout in milliseconds (default: 30000)" })),
 	workdir: Type.Optional(
 		Type.String({ description: "Working directory for the command (default: current directory)" }),
 	),
@@ -151,7 +151,7 @@ export class PythonTool implements AgentTool<typeof pythonSchema> {
 			throw new Error("Python tool requires a session when not using proxy executor");
 		}
 
-		const { code, timeout, workdir, reset } = params;
+		const { code, timeoutMs = 30000, workdir, reset } = params;
 		const controller = new AbortController();
 		const onAbort = () => controller.abort();
 		signal?.addEventListener("abort", onAbort, { once: true });
@@ -182,7 +182,7 @@ export class PythonTool implements AgentTool<typeof pythonSchema> {
 			const sessionId = sessionFile ? `session:${sessionFile}:workdir:${commandCwd}` : `cwd:${commandCwd}`;
 			const executorOptions: PythonExecutorOptions = {
 				cwd: commandCwd,
-				timeout: timeout ? timeout * 1000 : undefined,
+				timeoutMs,
 				signal: controller.signal,
 				sessionId,
 				kernelMode: this.session.settings?.getPythonKernelMode?.() ?? "session",

package/src/index.ts

@@ -201,8 +201,6 @@ export {
 	type FindToolDetails,
 	type FindToolOptions,
 	formatSize,
-	GitTool,
-	type GitToolDetails,
 	type GrepOperations,
 	type GrepToolDetails,
 	type GrepToolOptions,

package/src/modes/interactive/components/settings-defs.ts

@@ -83,7 +83,6 @@ const THINKING_DESCRIPTIONS: Record<ThinkingLevel, string> = {
  * - behavior: Core agent behavior (compaction, modes, retries, notifications)
  * - tools: Tool-specific settings (bash, git, python, edit, MCP, skills)
  * - display: Visual/UI settings (theme, images, thinking)
- * - voice: Voice mode and TTS settings
  * - ttsr: Time Traveling Stream Rules settings
  * - status: Status line configuration
  * - lsp: LSP integration settings
@@ -250,15 +249,6 @@ export const SETTINGS_DEFS: SettingDef[] = [
 		get: (sm) => sm.getBashInterceptorSimpleLsEnabled(),
 		set: (sm, v) => sm.setBashInterceptorSimpleLsEnabled(v),
 	},
-	{
-		id: "gitTool",
-		tab: "tools",
-		type: "boolean",
-		label: "Git tool",
-		description: "Enable structured Git tool",
-		get: (sm) => sm.getGitToolEnabled(),
-		set: (sm, v) => sm.setGitToolEnabled(v),
-	},
 	{
 		id: "pythonToolMode",
 		tab: "tools",
@@ -494,66 +484,6 @@ export const SETTINGS_DEFS: SettingDef[] = [
 		set: (sm, v) => sm.setShowHardwareCursor(v),
 	},
 
-	// ═══════════════════════════════════════════════════════════════════════════
-	// Voice tab - Voice mode and TTS settings
-	// ═══════════════════════════════════════════════════════════════════════════
-	{
-		id: "voiceEnabled",
-		tab: "voice",
-		type: "boolean",
-		label: "Voice mode",
-		description: "Enable realtime voice input/output (Ctrl+Y toggle, auto-send on silence)",
-		get: (sm) => sm.getVoiceEnabled(),
-		set: (sm, v) => sm.setVoiceEnabled(v),
-	},
-	{
-		id: "voiceTtsModel",
-		tab: "voice",
-		type: "submenu",
-		label: "TTS model",
-		description: "Text-to-speech model for voice output",
-		get: (sm) => sm.getVoiceTtsModel(),
-		set: (sm, v) => sm.setVoiceTtsModel(v),
-		getOptions: () => [
-			{ value: "gpt-4o-mini-tts", label: "GPT-4o Mini TTS", description: "Fast and efficient" },
-			{ value: "tts-1", label: "TTS-1", description: "Standard quality" },
-			{ value: "tts-1-hd", label: "TTS-1 HD", description: "Higher quality" },
-		],
-	},
-	{
-		id: "voiceTtsVoice",
-		tab: "voice",
-		type: "submenu",
-		label: "TTS voice",
-		description: "Voice for text-to-speech output",
-		get: (sm) => sm.getVoiceTtsVoice(),
-		set: (sm, v) => sm.setVoiceTtsVoice(v),
-		getOptions: () => [
-			{ value: "alloy", label: "Alloy", description: "Neutral" },
-			{ value: "echo", label: "Echo", description: "Male" },
-			{ value: "fable", label: "Fable", description: "British" },
-			{ value: "onyx", label: "Onyx", description: "Deep male" },
-			{ value: "nova", label: "Nova", description: "Female" },
-			{ value: "shimmer", label: "Shimmer", description: "Female" },
-		],
-	},
-	{
-		id: "voiceTtsFormat",
-		tab: "voice",
-		type: "submenu",
-		label: "TTS format",
-		description: "Audio format for voice output",
-		get: (sm) => sm.getVoiceTtsFormat(),
-		set: (sm, v) => sm.setVoiceTtsFormat(v as "wav" | "mp3" | "opus" | "aac" | "flac"),
-		getOptions: () => [
-			{ value: "wav", label: "WAV", description: "Uncompressed, best quality" },
-			{ value: "mp3", label: "MP3", description: "Compressed, widely compatible" },
-			{ value: "opus", label: "Opus", description: "Efficient compression" },
-			{ value: "aac", label: "AAC", description: "Apple-friendly" },
-			{ value: "flac", label: "FLAC", description: "Lossless compression" },
-		],
-	},
-
 	// ═══════════════════════════════════════════════════════════════════════════
 	// TTSR tab - Time Traveling Stream Rules
 	// ═══════════════════════════════════════════════════════════════════════════

package/src/modes/interactive/components/settings-selector.ts

@@ -117,7 +117,6 @@ const SETTINGS_TABS: Tab[] = [
 	{ id: "behavior", label: "Behavior" },
 	{ id: "tools", label: "Tools" },
 	{ id: "display", label: "Display" },
-	{ id: "voice", label: "Voice" },
 	{ id: "ttsr", label: "TTSR" },
 	{ id: "status", label: "Status" },
 	{ id: "lsp", label: "LSP" },

package/src/modes/interactive/controllers/event-controller.ts

@@ -68,7 +68,6 @@ export class EventController {
 					getSymbolTheme().spinnerFrames,
 				);
 				this.ctx.statusContainer.addChild(this.ctx.loadingAnimation);
-				this.ctx.startVoiceProgressTimer();
 				this.ctx.ui.requestRender();
 				break;
 
@@ -250,7 +249,6 @@ export class EventController {
 			}
 
 			case "agent_end":
-				this.ctx.stopVoiceProgressTimer();
 				if (this.ctx.loadingAnimation) {
 					this.ctx.loadingAnimation.stop();
 					this.ctx.loadingAnimation = undefined;
@@ -262,15 +260,6 @@ export class EventController {
 					this.ctx.streamingMessage = undefined;
 				}
 				this.ctx.pendingTools.clear();
-				if (this.ctx.settingsManager.getVoiceEnabled() && this.ctx.voiceAutoModeEnabled) {
-					const lastAssistant = this.ctx.findLastAssistantMessage();
-					if (lastAssistant && lastAssistant.stopReason !== "aborted" && lastAssistant.stopReason !== "error") {
-						const text = this.ctx.extractAssistantText(lastAssistant);
-						if (text) {
-							this.ctx.voiceSupervisor.notifyResult(text);
-						}
-					}
-				}
 				this.ctx.ui.requestRender();
 				this.sendCompletionNotification();
 				break;

package/src/modes/interactive/controllers/selector-controller.ts

@@ -199,15 +199,6 @@ export class SelectorController {
 				this.ctx.ui.invalidate();
 				break;
 			}
-			case "voiceEnabled": {
-				if (!value) {
-					this.ctx.voiceAutoModeEnabled = false;
-					this.ctx.stopVoiceProgressTimer();
-					void this.ctx.voiceSupervisor.stop();
-					this.ctx.setVoiceStatus(undefined);
-				}
-				break;
-			}
 			case "statusLinePreset":
 			case "statusLineSeparator":
 			case "statusLineShowHooks":