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

package/CHANGELOG.md

@@ -2,6 +2,25 @@
 
 ## [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
+
+- Removed `replaceText` edit operation from hashline mode (substring-based text replacement)
+- Removed autocorrect heuristics that attempted to detect and fix line merges and formatting rewrites in hashline edits
+
 ## [12.19.0] - 2026-02-22
 ### Added
 

package/package.json

@@ -1,10 +1,10 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "12.19.0",
+	"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 Bölük",
+	"author": "Can Boluk",
 	"contributors": [
 		"Mario Zechner"
 	],
@@ -40,33 +40,33 @@
 		"test": "bun test"
 	},
 	"dependencies": {
-		"@mozilla/readability": "0.6.0",
-		"@oh-my-pi/omp-stats": "12.19.0",
-		"@oh-my-pi/pi-agent-core": "12.19.0",
-		"@oh-my-pi/pi-ai": "12.19.0",
-		"@oh-my-pi/pi-natives": "12.19.0",
-		"@oh-my-pi/pi-tui": "12.19.0",
-		"@oh-my-pi/pi-utils": "12.19.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"
+		"@mozilla/readability": "^0.6",
+		"@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",
+		"chalk": "^5.6",
+		"diff": "^8.0",
+		"file-type": "^21.3",
+		"glob": "^13.0",
+		"handlebars": "^4.7",
+		"ignore": "^7.0",
+		"linkedom": "^0.18",
+		"marked": "^17.0",
+		"node-html-parser": "^7.0",
+		"puppeteer": "^24.37",
+		"smol-toml": "^1.6",
+		"zod": "^4.3"
 	},
 	"devDependencies": {
-		"@types/bun": "^1.3.9",
-		"@types/ms": "^2.1.0",
-		"ms": "^2.1.3"
+		"@types/bun": "^1.3",
+		"@types/ms": "^2.1",
+		"ms": "^2.1"
 	},
 	"engines": {
 		"bun": ">=1.3.7"
@@ -87,6 +87,14 @@
 			"types": "./src/*.ts",
 			"import": "./src/*.ts"
 		},
+		"./async": {
+			"types": "./src/async/index.ts",
+			"import": "./src/async/index.ts"
+		},
+		"./async/*": {
+			"types": "./src/async/*.ts",
+			"import": "./src/async/*.ts"
+		},
 		"./capability": {
 			"types": "./src/capability/index.ts",
 			"import": "./src/capability/index.ts"

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/patch/hashline.ts

@@ -21,159 +21,6 @@ export type HashlineEdit =
 	| { op: "append"; after?: LineTag; content: string[] }
 	| { op: "prepend"; before?: LineTag; content: string[] }
 	| { op: "insert"; after: LineTag; before: LineTag; content: string[] };
-export type ReplaceTextEdit = { op: "replaceText"; old_text: string; new_text: string; all?: boolean };
-export type EditSpec = HashlineEdit | ReplaceTextEdit;
-
-/**
- * Compare two strings ignoring all whitespace differences.
- *
- * Returns true when the non-whitespace characters are identical — meaning
- * the only differences are in spaces, tabs, or other whitespace.
- */
-function equalsIgnoringWhitespace(a: string, b: string): boolean {
-	// Fast path: identical strings
-	if (a === b) return true;
-	// Compare with all whitespace removed
-	return a.replace(/\s+/g, "") === b.replace(/\s+/g, "");
-}
-
-function stripAllWhitespace(s: string): string {
-	return s.replace(/\s+/g, "");
-}
-
-function stripTrailingContinuationTokens(s: string): string {
-	// Heuristic: models often merge a continuation line into the prior line
-	// while also changing the trailing operator (e.g. `&&` → `||`).
-	// Strip common trailing continuation tokens so we can still detect merges.
-	return s.replace(/(?:&&|\|\||\?\?|\?|:|=|,|\+|-|\*|\/|\.|\()\s*$/u, "");
-}
-
-function stripMergeOperatorChars(s: string): string {
-	// Used for merge detection when the model changes a logical operator like
-	// `||` → `??` while also merging adjacent lines.
-	return s.replace(/[|&?]/g, "");
-}
-
-function leadingWhitespace(s: string): string {
-	const match = s.match(/^\s*/);
-	return match ? match[0] : "";
-}
-
-function restoreLeadingIndent(templateLine: string, line: string): string {
-	if (line.length === 0) return line;
-	const templateIndent = leadingWhitespace(templateLine);
-	if (templateIndent.length === 0) return line;
-	const indent = leadingWhitespace(line);
-	if (indent.length > 0) return line;
-	return templateIndent + line;
-}
-
-function restoreIndentForPairedReplacement(oldLines: string[], newLines: string[]): string[] {
-	if (oldLines.length !== newLines.length) return newLines;
-	let changed = false;
-	const out = new Array<string>(newLines.length);
-	for (let i = 0; i < newLines.length; i++) {
-		const restored = restoreLeadingIndent(oldLines[i], newLines[i]);
-		out[i] = restored;
-		if (restored !== newLines[i]) changed = true;
-	}
-	return changed ? out : newLines;
-}
-
-/**
- * Undo pure formatting rewrites where the model reflows a single logical line
- * into multiple lines (or similar), but the token stream is identical.
- */
-function restoreOldWrappedLines(oldLines: string[], newLines: string[]): string[] {
-	if (oldLines.length === 0 || newLines.length < 2) return newLines;
-
-	const canonToOld = new Map<string, { line: string; count: number }>();
-	for (const line of oldLines) {
-		const canon = stripAllWhitespace(line);
-		const bucket = canonToOld.get(canon);
-		if (bucket) bucket.count++;
-		else canonToOld.set(canon, { line, count: 1 });
-	}
-
-	const candidates: { start: number; len: number; replacement: string; canon: string }[] = [];
-	for (let start = 0; start < newLines.length; start++) {
-		for (let len = 2; len <= 10 && start + len <= newLines.length; len++) {
-			const canonSpan = stripAllWhitespace(newLines.slice(start, start + len).join(""));
-			const old = canonToOld.get(canonSpan);
-			if (old && old.count === 1 && canonSpan.length >= 6) {
-				candidates.push({ start, len, replacement: old.line, canon: canonSpan });
-			}
-		}
-	}
-	if (candidates.length === 0) return newLines;
-
-	// Keep only spans whose canonical match is unique in the new output.
-	const canonCounts = new Map<string, number>();
-	for (const c of candidates) {
-		canonCounts.set(c.canon, (canonCounts.get(c.canon) ?? 0) + 1);
-	}
-	const uniqueCandidates = candidates.filter(c => (canonCounts.get(c.canon) ?? 0) === 1);
-	if (uniqueCandidates.length === 0) return newLines;
-
-	// Apply replacements back-to-front so indices remain stable.
-	uniqueCandidates.sort((a, b) => b.start - a.start);
-	const out = [...newLines];
-	for (const c of uniqueCandidates) {
-		out.splice(c.start, c.len, c.replacement);
-	}
-	return out;
-}
-
-function stripInsertAnchorEchoAfter(anchorLine: string, dstLines: string[]): string[] {
-	if (dstLines.length <= 1) return dstLines;
-	if (equalsIgnoringWhitespace(dstLines[0], anchorLine)) {
-		return dstLines.slice(1);
-	}
-	return dstLines;
-}
-
-function stripInsertAnchorEchoBefore(anchorLine: string, dstLines: string[]): string[] {
-	if (dstLines.length <= 1) return dstLines;
-	if (equalsIgnoringWhitespace(dstLines[dstLines.length - 1], anchorLine)) {
-		return dstLines.slice(0, -1);
-	}
-	return dstLines;
-}
-
-function stripInsertBoundaryEcho(afterLine: string, beforeLine: string, dstLines: string[]): string[] {
-	let out = dstLines;
-	if (out.length > 1 && equalsIgnoringWhitespace(out[0], afterLine)) {
-		out = out.slice(1);
-	}
-	if (out.length > 1 && equalsIgnoringWhitespace(out[out.length - 1], beforeLine)) {
-		out = out.slice(0, -1);
-	}
-	return out;
-}
-
-function stripRangeBoundaryEcho(fileLines: string[], startLine: number, endLine: number, dstLines: string[]): string[] {
-	// Only strip when the model replaced with multiple lines and grew the edit.
-	// This avoids turning a single-line replacement into a deletion.
-	const count = endLine - startLine + 1;
-	if (dstLines.length <= 1 || dstLines.length <= count) return dstLines;
-
-	let out = dstLines;
-	const beforeIdx = startLine - 2;
-	if (beforeIdx >= 0 && equalsIgnoringWhitespace(out[0], fileLines[beforeIdx])) {
-		out = out.slice(1);
-	}
-
-	const afterIdx = endLine;
-	if (
-		afterIdx < fileLines.length &&
-		out.length > 0 &&
-		equalsIgnoringWhitespace(out[out.length - 1], fileLines[afterIdx])
-	) {
-		out = out.slice(0, -1);
-	}
-
-	return out;
-}
 
 const NIBBLE_STR = "ZPMQVRWSNKTXJBYH";
 
@@ -594,38 +441,6 @@ export function applyHashlineEdits(
 	let firstChangedLine: number | undefined;
 	const noopEdits: Array<{ editIndex: number; loc: string; currentContent: string }> = [];
 
-	const autocorrect = Bun.env.PI_HL_AUTOCORRECT === "1";
-
-	function collectExplicitlyTouchedLines(): Set<number> {
-		const touched = new Set<number>();
-		for (const edit of edits) {
-			switch (edit.op) {
-				case "set":
-					touched.add(edit.tag.line);
-					break;
-				case "replace":
-					for (let ln = edit.first.line; ln <= edit.last.line; ln++) touched.add(ln);
-					break;
-				case "append":
-					if (edit.after) {
-						touched.add(edit.after.line);
-					}
-					break;
-				case "prepend":
-					if (edit.before) {
-						touched.add(edit.before.line);
-					}
-					break;
-				case "insert":
-					touched.add(edit.after.line);
-					touched.add(edit.before.line);
-					break;
-			}
-		}
-		return touched;
-	}
-
-	const explicitlyTouchedLines = collectExplicitlyTouchedLines();
 	// Pre-validate: collect all hash mismatches before mutating
 	const mismatches: HashMismatch[] = [];
 	function validateRef(ref: { line: number; hash: string }): boolean {
@@ -765,35 +580,8 @@ export function applyHashlineEdits(
 	for (const { edit, idx } of annotated) {
 		switch (edit.op) {
 			case "set": {
-				const merged = autocorrect ? maybeExpandSingleLineMerge(edit.tag.line, edit.content) : null;
-				if (merged) {
-					const origLines = originalFileLines.slice(
-						merged.startLine - 1,
-						merged.startLine - 1 + merged.deleteCount,
-					);
-					let nextLines = merged.newLines;
-					nextLines = restoreIndentForPairedReplacement([origLines[0] ?? ""], nextLines);
-
-					if (origLines.every((line, i) => line === nextLines[i])) {
-						noopEdits.push({
-							editIndex: idx,
-							loc: `${edit.tag.line}#${edit.tag.hash}`,
-							currentContent: origLines.join("\n"),
-						});
-						break;
-					}
-					fileLines.splice(merged.startLine - 1, merged.deleteCount, ...nextLines);
-					trackFirstChanged(merged.startLine);
-					break;
-				}
-
-				const count = 1;
 				const origLines = originalFileLines.slice(edit.tag.line - 1, edit.tag.line);
-				let stripped = autocorrect
-					? stripRangeBoundaryEcho(originalFileLines, edit.tag.line, edit.tag.line, edit.content)
-					: edit.content;
-				stripped = autocorrect ? restoreOldWrappedLines(origLines, stripped) : stripped;
-				const newLines = autocorrect ? restoreIndentForPairedReplacement(origLines, stripped) : stripped;
+				const newLines = edit.content;
 				if (origLines.every((line, i) => line === newLines[i])) {
 					noopEdits.push({
 						editIndex: idx,
@@ -802,36 +590,19 @@ export function applyHashlineEdits(
 					});
 					break;
 				}
-				fileLines.splice(edit.tag.line - 1, count, ...newLines);
+				fileLines.splice(edit.tag.line - 1, 1, ...newLines);
 				trackFirstChanged(edit.tag.line);
 				break;
 			}
 			case "replace": {
 				const count = edit.last.line - edit.first.line + 1;
-				const origLines = originalFileLines.slice(edit.first.line - 1, edit.first.line - 1 + count);
-				let stripped = autocorrect
-					? stripRangeBoundaryEcho(originalFileLines, edit.first.line, edit.last.line, edit.content)
-					: edit.content;
-				stripped = autocorrect ? restoreOldWrappedLines(origLines, stripped) : stripped;
-				const newLines = autocorrect ? restoreIndentForPairedReplacement(origLines, stripped) : stripped;
-				if (autocorrect && origLines.every((line, i) => line === newLines[i])) {
-					noopEdits.push({
-						editIndex: idx,
-						loc: `${edit.first.line}#${edit.first.hash}`,
-						currentContent: origLines.join("\n"),
-					});
-					break;
-				}
+				const newLines = edit.content;
 				fileLines.splice(edit.first.line - 1, count, ...newLines);
 				trackFirstChanged(edit.first.line);
 				break;
 			}
 			case "append": {
-				const inserted = edit.after
-					? autocorrect
-						? stripInsertAnchorEchoAfter(originalFileLines[edit.after.line - 1], edit.content)
-						: edit.content
-					: edit.content;
+				const inserted = edit.content;
 				if (inserted.length === 0) {
 					noopEdits.push({
 						editIndex: idx,
@@ -855,11 +626,7 @@ export function applyHashlineEdits(
 				break;
 			}
 			case "prepend": {
-				const inserted = edit.before
-					? autocorrect
-						? stripInsertAnchorEchoBefore(originalFileLines[edit.before.line - 1], edit.content)
-						: edit.content
-					: edit.content;
+				const inserted = edit.content;
 				if (inserted.length === 0) {
 					noopEdits.push({
 						editIndex: idx,
@@ -884,7 +651,7 @@ export function applyHashlineEdits(
 			case "insert": {
 				const afterLine = originalFileLines[edit.after.line - 1];
 				const beforeLine = originalFileLines[edit.before.line - 1];
-				const inserted = autocorrect ? stripInsertBoundaryEcho(afterLine, beforeLine, edit.content) : edit.content;
+				const inserted = edit.content;
 				if (inserted.length === 0) {
 					noopEdits.push({
 						editIndex: idx,
@@ -911,51 +678,4 @@ export function applyHashlineEdits(
 			firstChangedLine = line;
 		}
 	}
-
-	function maybeExpandSingleLineMerge(
-		line: number,
-		content: string[],
-	): { startLine: number; deleteCount: number; newLines: string[] } | null {
-		if (content.length !== 1) return null;
-		if (line < 1 || line > fileLines.length) return null;
-
-		const newLine = content[0];
-		const newCanon = stripAllWhitespace(newLine);
-		const newCanonForMergeOps = stripMergeOperatorChars(newCanon);
-		if (newCanon.length === 0) return null;
-
-		const orig = fileLines[line - 1];
-		const origCanon = stripAllWhitespace(orig);
-		const origCanonForMatch = stripTrailingContinuationTokens(origCanon);
-		const origCanonForMergeOps = stripMergeOperatorChars(origCanon);
-		const origLooksLikeContinuation = origCanonForMatch.length < origCanon.length;
-		if (origCanon.length === 0) return null;
-		const nextIdx = line;
-		const prevIdx = line - 2;
-		// Case A: dst absorbed the next continuation line.
-		if (origLooksLikeContinuation && nextIdx < fileLines.length && !explicitlyTouchedLines.has(line + 1)) {
-			const next = fileLines[nextIdx];
-			const nextCanon = stripAllWhitespace(next);
-			const a = newCanon.indexOf(origCanonForMatch);
-			const b = newCanon.indexOf(nextCanon);
-			if (a !== -1 && b !== -1 && a < b && newCanon.length <= origCanon.length + nextCanon.length + 32) {
-				return { startLine: line, deleteCount: 2, newLines: [newLine] };
-			}
-		}
-		// Case B: dst absorbed the previous declaration/continuation line.
-		if (prevIdx >= 0 && !explicitlyTouchedLines.has(line - 1)) {
-			const prev = fileLines[prevIdx];
-			const prevCanon = stripAllWhitespace(prev);
-			const prevCanonForMatch = stripTrailingContinuationTokens(prevCanon);
-			const prevLooksLikeContinuation = prevCanonForMatch.length < prevCanon.length;
-			if (!prevLooksLikeContinuation) return null;
-			const a = newCanonForMergeOps.indexOf(stripMergeOperatorChars(prevCanonForMatch));
-			const b = newCanonForMergeOps.indexOf(origCanonForMergeOps);
-			if (a !== -1 && b !== -1 && a < b && newCanon.length <= prevCanon.length + origCanon.length + 32) {
-				return { startLine: line - 1, deleteCount: 2, newLines: [newLine] };
-			}
-		}
-
-		return null;
-	}
 }

package/src/patch/index.ts

@@ -34,14 +34,7 @@ import { enforcePlanModeWrite, resolvePlanPath } from "../tools/plan-mode-guard"
 import { applyPatch } from "./applicator";
 import { generateDiffString, generateUnifiedDiffString, replaceText } from "./diff";
 import { findMatch } from "./fuzzy";
-import {
-	applyHashlineEdits,
-	computeLineHash,
-	type HashlineEdit,
-	type LineTag,
-	parseTag,
-	type ReplaceTextEdit,
-} from "./hashline";
+import { applyHashlineEdits, computeLineHash, type HashlineEdit, type LineTag, parseTag } from "./hashline";
 import { detectLineEnding, normalizeToLF, restoreLineEndings, stripBom } from "./normalize";
 import { buildNormativeUpdateInput } from "./normative";
 import { type EditToolDetails, getLspBatchRequest } from "./shared";
@@ -201,13 +194,6 @@ export function hashlineParseContent(edit: string | string[] | null): string[] {
 	if (lines[lines.length - 1].trim() === "") return lines.slice(0, -1);
 	return lines;
 }
-
-function hashlineParseContentString(edit: string | string[] | null): string {
-	if (edit === null) return "";
-	if (Array.isArray(edit)) return edit.join("\n");
-	return edit;
-}
-
 const hashlineTargetEditSchema = Type.Object(
 	{
 		op: Type.Literal("set"),
@@ -255,25 +241,12 @@ const hashlineInsertEditSchema = Type.Object(
 	{ additionalProperties: false },
 );
 
-const hashlineReplaceTextEditSchema = Type.Object(
-	{
-		op: Type.Literal("replaceText"),
-		old_text: Type.String({ description: "Text to find", minLength: 1 }),
-		new_text: hashlineReplaceContentFormat("Replacement"),
-		all: Type.Optional(Type.Boolean({ description: "Replace all occurrences" })),
-	},
-	{ additionalProperties: false },
-);
-
-const HL_REPLACE_ENABLED = Bun.env.PI_HL_REPLACETXT === "1";
-
 const hashlineEditSpecSchema = Type.Union([
 	hashlineTargetEditSchema,
 	hashlineRangeEditSchema,
 	hashlineAppendEditSchema,
 	hashlinePrependEditSchema,
 	hashlineInsertEditSchema,
-	...(HL_REPLACE_ENABLED ? [hashlineReplaceTextEditSchema] : []),
 ]);
 
 const hashlineEditSchema = Type.Object(
@@ -486,7 +459,7 @@ export class EditTool implements AgentTool<TInput> {
 			case "patch":
 				return renderPromptTemplate(patchDescription);
 			case "hashline":
-				return renderPromptTemplate(hashlineDescription, { allowReplaceText: HL_REPLACE_ENABLED });
+				return renderPromptTemplate(hashlineDescription);
 			default:
 				return renderPromptTemplate(replaceDescription);
 		}
@@ -581,7 +554,6 @@ export class EditTool implements AgentTool<TInput> {
 			}
 
 			const anchorEdits: HashlineEdit[] = [];
-			const replaceEdits: ReplaceTextEdit[] = [];
 			for (const edit of edits) {
 				switch (edit.op) {
 					case "set": {
@@ -643,16 +615,6 @@ export class EditTool implements AgentTool<TInput> {
 						}
 						break;
 					}
-					case "replaceText": {
-						const { old_text, new_text, all } = edit;
-						replaceEdits.push({
-							op: "replaceText",
-							old_text: old_text,
-							new_text: hashlineParseContentString(new_text),
-							all: all ?? false,
-						});
-						break;
-					}
 					default:
 						throw new Error(`Invalid edit operation: ${JSON.stringify(edit)}`);
 				}
@@ -668,19 +630,6 @@ export class EditTool implements AgentTool<TInput> {
 			const anchorResult = applyHashlineEdits(normalizedContent, anchorEdits);
 			normalizedContent = anchorResult.content;
 
-			// Apply content-replace edits (substr-style fuzzy replace)
-			for (const r of replaceEdits) {
-				if (r.old_text.length === 0) {
-					throw new Error("old_text must not be empty.");
-				}
-				const rep = replaceText(normalizedContent, r.old_text, r.new_text, {
-					fuzzy: this.#allowFuzzy,
-					all: r.all ?? false,
-					threshold: this.#fuzzyThreshold,
-				});
-				normalizedContent = rep.content;
-			}
-
 			const result = {
 				content: normalizedContent,
 				firstChangedLine: anchorResult.firstChangedLine,

package/src/prompts/tools/hashline.md

@@ -22,10 +22,6 @@ Apply precise file edits using `LINE#ID` tags, anchoring to the file content.
   - `{ op: "prepend", before: "N#ID", content: […] }` or `{ op: "prepend", content: […] }` (no `before` = insert at beginning of file)
   - `{ op: "append", after: "N#ID", content: […] }` or `{ op: "append", content: […] }` (no `after` = insert at end of file)
   - `{ op: "insert", after: "N#ID", before: "N#ID", content: […] }` (between adjacent anchors; safest for blocks)
-{{#if allowReplaceText}}
-- **Content replace**
-  - `{ op: "replaceText", old_text: "…", new_text: "…", all?: boolean }`
-{{/if}}
 - **File-level controls**
   - `{ delete: true, edits: [] }` deletes the file (cannot be combined with `rename`).
   - `{ rename: "new/path.ts", edits: […] }` writes result to new path and removes old path.
@@ -180,18 +176,6 @@ content: ["function validate(data: unknown): boolean {", "  return data != null
 The trailing `""` in `content` preserves the blank-line separator. **Anchor to the structural line (`export function ...`), not the blank line above it** — blank lines are ambiguous and may be added or removed by other edits.
 </example>
 
-{{#if allowReplaceText}}
-<example name="content replace (rare)">
-```
-op: "replaceText"
-old_text: "x = 42"
-new_text: "x = 99"
-```
-
-Use only when line anchors aren't available. `old_text` must match exactly one location in the file (or set `"all": true` for all occurrences).
-</example>
-{{/if}}
-
 <example name="file delete">
 ```
 path: "src/deprecated/legacy.ts"

package/src/session/agent-session.ts

@@ -2224,6 +2224,7 @@ export class AgentSession {
 
 		this.#disconnectFromAgent();
 		await this.abort();
+		this.#asyncJobManager?.cancelAll();
 		this.agent.reset();
 		await this.sessionManager.flush();
 		await this.sessionManager.newSession(options);
@@ -2933,6 +2934,7 @@ Be thorough - include exact file paths, function names, error messages, and tech
 
 			// Start a new session
 			await this.sessionManager.flush();
+			this.#asyncJobManager?.cancelAll();
 			await this.sessionManager.newSession();
 			this.agent.reset();
 			this.agent.sessionId = this.sessionManager.getSessionId();
@@ -4126,6 +4128,7 @@ Be thorough - include exact file paths, function names, error messages, and tech
 
 		// Flush pending writes before branching
 		await this.sessionManager.flush();
+		this.#asyncJobManager?.cancelAll();
 
 		if (!selectedEntry.parentId) {
 			await this.sessionManager.newSession({ parentSession: previousSessionFile });

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,