@oh-my-pi/pi-coding-agent 12.19.2 → 12.19.3

package/CHANGELOG.md

@@ -2,6 +2,19 @@
 
 ## [Unreleased]
 
+## [12.19.3] - 2026-02-22
+### Added
+
+- Added `pty` parameter to bash tool to enable PTY mode for commands requiring a real terminal (e.g., sudo, ssh, top, less)
+
+### Changed
+
+- Changed bash tool to use per-command PTY control instead of global virtual terminal setting
+
+### Removed
+
+- Removed `bash.virtualTerminal` setting; use the `pty` parameter on individual bash commands instead
+
 ## [12.19.1] - 2026-02-22
 ### Removed
 

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "12.19.2",
+	"version": "12.19.3",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://github.com/can1357/oh-my-pi",
 	"author": "Can Boluk",
@@ -41,12 +41,12 @@
 	},
 	"dependencies": {
 		"@mozilla/readability": "^0.6",
-		"@oh-my-pi/omp-stats": "12.19.2",
-		"@oh-my-pi/pi-agent-core": "12.19.2",
-		"@oh-my-pi/pi-ai": "12.19.2",
-		"@oh-my-pi/pi-natives": "12.19.2",
-		"@oh-my-pi/pi-tui": "12.19.2",
-		"@oh-my-pi/pi-utils": "12.19.2",
+		"@oh-my-pi/omp-stats": "12.19.3",
+		"@oh-my-pi/pi-agent-core": "12.19.3",
+		"@oh-my-pi/pi-ai": "12.19.3",
+		"@oh-my-pi/pi-natives": "12.19.3",
+		"@oh-my-pi/pi-tui": "12.19.3",
+		"@oh-my-pi/pi-utils": "12.19.3",
 		"@sinclair/typebox": "^0.34",
 		"@xterm/headless": "^6.0",
 		"ajv": "^8.18",

package/src/config/settings-schema.ts

@@ -756,17 +756,6 @@ export const SETTINGS_SCHEMA = {
 	// ─────────────────────────────────────────────────────────────────────────
 	// Bash interceptor settings
 	// ─────────────────────────────────────────────────────────────────────────
-	"bash.virtualTerminal": {
-		type: "enum",
-		values: ["on", "off"] as const,
-		default: "off",
-		ui: {
-			tab: "bash",
-			label: "Virtual terminal",
-			description: "Use PTY-backed interactive execution for bash",
-			submenu: true,
-		},
-	},
 	"bashInterceptor.enabled": {
 		type: "boolean",
 		default: false,

package/src/main.ts

@@ -538,7 +538,6 @@ export async function runRootCommand(parsed: Args, rawArgs: string[]): Promise<v
 	const cwd = getProjectDir();
 	await logger.timeAsync("settings:init", () => Settings.init({ cwd }));
 	if (parsedArgs.noPty) {
-		settings.override("bash.virtualTerminal", "off");
 		Bun.env.PI_NO_PTY = "1";
 	}
 	const {

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

@@ -154,11 +154,6 @@ const OPTION_PROVIDERS: Partial<Record<SettingPath, OptionProvider>> = {
 		{ value: "tool-only", label: "tool-only", description: "Interrupt only on tool-call argument matches" },
 		{ value: "never", label: "never", description: "Never interrupt; inject warning after completion" },
 	],
-	// Virtual terminal
-	"bash.virtualTerminal": [
-		{ value: "on", label: "On", description: "PTY-backed interactive execution" },
-		{ value: "off", label: "Off", description: "Standard non-interactive execution" },
-	],
 	// Provider options
 	"providers.webSearch": [
 		{

package/src/tools/bash.ts

@@ -34,6 +34,11 @@ const bashSchemaBase = Type.Object({
 	cwd: Type.Optional(Type.String({ description: "Working directory (default: cwd)" })),
 	head: Type.Optional(Type.Number({ description: "Return only first N lines of output" })),
 	tail: Type.Optional(Type.Number({ description: "Return only last N lines of output" })),
+	pty: Type.Optional(
+		Type.Boolean({
+			description: "Run in PTY mode when command needs a real terminal (e.g. sudo/ssh/top/less); default: false",
+		}),
+	),
 });
 
 const bashSchemaWithAsync = Type.Object({
@@ -54,6 +59,7 @@ export interface BashToolInput {
 	head?: number;
 	tail?: number;
 	async?: boolean;
+	pty?: boolean;
 }
 
 export interface BashToolDetails {
@@ -123,7 +129,15 @@ export class BashTool implements AgentTool<BashToolSchema, BashToolDetails> {
 
 	async execute(
 		_toolCallId: string,
-		{ command: rawCommand, timeout: rawTimeout = 300, cwd, head, tail, async: asyncRequested = false }: BashToolInput,
+		{
+			command: rawCommand,
+			timeout: rawTimeout = 300,
+			cwd,
+			head,
+			tail,
+			async: asyncRequested = false,
+			pty = false,
+		}: BashToolInput,
 		signal?: AbortSignal,
 		onUpdate?: AgentToolUpdateCallback<BashToolDetails>,
 		ctx?: AgentToolContext,
@@ -188,7 +202,7 @@ export class BashTool implements AgentTool<BashToolSchema, BashToolDetails> {
 					try {
 						const result = await executeBash(command, {
 							cwd: commandCwd,
-							sessionKey: this.session.getSessionId?.() ?? undefined,
+							sessionKey: `${this.session.getSessionId?.() ?? ""}:async:${jobId}`,
 							timeout: timeoutMs,
 							signal: runSignal,
 							env: extraEnv,
@@ -229,11 +243,7 @@ export class BashTool implements AgentTool<BashToolSchema, BashToolDetails> {
 		const extraEnv = artifactsDir ? { ARTIFACTS: artifactsDir } : undefined;
 		const { path: artifactPath, id: artifactId } = (await this.session.allocateOutputArtifact?.("bash")) ?? {};
 
-		const usePty =
-			this.session.settings.get("bash.virtualTerminal") === "on" &&
-			$env.PI_NO_PTY !== "1" &&
-			ctx?.hasUI === true &&
-			ctx.ui !== undefined;
+		const usePty = pty && $env.PI_NO_PTY !== "1" && ctx?.hasUI === true && ctx.ui !== undefined;
 		const result: BashResult | BashInteractiveResult = usePty
 			? await runInteractiveBashPty(ctx.ui!, {
 					command,