@oh-my-pi/pi-coding-agent 12.3.0 → 12.4.0

package/CHANGELOG.md

@@ -2,6 +2,21 @@
 
 ## [Unreleased]
 
+## [12.4.0] - 2026-02-14
+### Changed
+
+- Moved `sanitizeText` function from `@oh-my-pi/pi-utils` to `@oh-my-pi/pi-natives` for better code organization
+- Replaced internal `#normalizeOutput` methods with `sanitizeText` utility function in bash and Python execution components
+- Added line length clamping (4000 characters) to bash and Python execution output to prevent display of excessively long lines
+- Modified memory storage to isolate memories by project working directory, preventing cross-project memory contamination
+
+### Fixed
+
+- Fixed bash interactive tool to gracefully handle malformed output chunks by normalizing them before display
+- Fixed fetch tool incorrectly treating HTML content as plain text or markdown
+- Fixed output truncation notice displaying incorrect byte limit when maxBytes differs from outputBytes
+- Fixed Cloudflare returning corrupted bytes when compression is negotiated in web scraper requests
+
 ## [12.3.0] - 2026-02-14
 ### Added
 

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "12.3.0",
+	"version": "12.4.0",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"type": "module",
 	"bin": {
@@ -84,12 +84,12 @@
 	},
 	"dependencies": {
 		"@mozilla/readability": "0.6.0",
-		"@oh-my-pi/omp-stats": "12.3.0",
-		"@oh-my-pi/pi-agent-core": "12.3.0",
-		"@oh-my-pi/pi-ai": "12.3.0",
-		"@oh-my-pi/pi-natives": "12.3.0",
-		"@oh-my-pi/pi-tui": "12.3.0",
-		"@oh-my-pi/pi-utils": "12.3.0",
+		"@oh-my-pi/omp-stats": "12.4.0",
+		"@oh-my-pi/pi-agent-core": "12.4.0",
+		"@oh-my-pi/pi-ai": "12.4.0",
+		"@oh-my-pi/pi-natives": "12.4.0",
+		"@oh-my-pi/pi-tui": "12.4.0",
+		"@oh-my-pi/pi-utils": "12.4.0",
 		"@sinclair/typebox": "^0.34.48",
 		"@xterm/headless": "^6.0.0",
 		"ajv": "^8.17.1",

package/src/config/keybindings.ts

@@ -8,7 +8,7 @@ import {
 	matchesKey,
 	setEditorKeybindings,
 } from "@oh-my-pi/pi-tui";
-import { logger } from "@oh-my-pi/pi-utils";
+import { isEnoent, logger } from "@oh-my-pi/pi-utils";
 import { getAgentDir } from "@oh-my-pi/pi-utils/dirs";
 
 /**
@@ -204,6 +204,7 @@ export class KeybindingsManager {
 		try {
 			return await Bun.file(path).json();
 		} catch (error) {
+			if (isEnoent(error)) return {};
 			logger.warn("Failed to parse keybindings config", { path, error: String(error) });
 			return {};
 		}

package/src/lsp/render.ts

@@ -282,7 +282,7 @@ function renderHover(
 }
 
 /**
- * Syntax highlight code using native WASM highlighter.
+ * Syntax highlight code using native highlighter.
  */
 function highlightCode(codeText: string, language: string, theme: Theme): string[] {
 	const validLang = language && supportsLanguage(language) ? language : undefined;

package/src/memories/index.ts

@@ -152,7 +152,7 @@ export async function buildMemoryToolDeveloperInstructions(
 ): Promise<string | undefined> {
 	const cfg = loadMemoryConfig(settings);
 	if (!cfg.enabled) return undefined;
-	const memoryRoot = getMemoryRoot(agentDir);
+	const memoryRoot = getMemoryRoot(agentDir, settings.getCwd());
 	const summaryPath = path.join(memoryRoot, "memory_summary.md");
 
 	let text: string;
@@ -176,14 +176,14 @@ export async function buildMemoryToolDeveloperInstructions(
 /**
  * Clear all persisted memory state and generated artifacts.
  */
-export async function clearMemoryData(agentDir: string): Promise<void> {
+export async function clearMemoryData(agentDir: string, cwd: string): Promise<void> {
 	const db = openMemoryDb(getAgentDbPath(agentDir));
 	try {
 		clearMemoryDataInDb(db);
 	} finally {
 		closeMemoryDb(db);
 	}
-	await fs.rm(getMemoryRoot(agentDir), { recursive: true, force: true });
+	await fs.rm(getMemoryRoot(agentDir, cwd), { recursive: true, force: true });
 }
 
 /**
@@ -221,7 +221,7 @@ async function runPhase1(options: {
 	const db = openMemoryDb(getAgentDbPath(agentDir));
 	const nowSec = unixNow();
 	const workerId = `memory-${process.pid}`;
-	const memoryRoot = getMemoryRoot(agentDir);
+	const memoryRoot = getMemoryRoot(agentDir, session.sessionManager.getCwd());
 	const currentThreadId = session.sessionManager.getSessionId();
 
 	try {
@@ -345,7 +345,7 @@ async function runPhase2(options: {
 	const db = openMemoryDb(getAgentDbPath(agentDir));
 	const nowSec = unixNow();
 	const workerId = `memory-${process.pid}`;
-	const memoryRoot = getMemoryRoot(agentDir);
+	const memoryRoot = getMemoryRoot(agentDir, session.sessionManager.getCwd());
 
 	try {
 		const claimResult = tryClaimGlobalPhase2Job(db, {
@@ -1077,8 +1077,12 @@ function loadMemoryConfig(settings: Settings): MemoryRuntimeConfig {
 	};
 }
 
-function getMemoryRoot(agentDir: string): string {
-	return path.join(agentDir, "memories");
+function getMemoryRoot(agentDir: string, cwd: string): string {
+	return path.join(agentDir, "memories", encodeProjectPath(cwd));
+}
+
+function encodeProjectPath(cwd: string): string {
+	return `--${cwd.replace(/^[/\\]/, "").replace(/[/\\:]/g, "-")}--`;
 }
 
 function unixNow(): number {

package/src/modes/components/bash-execution.ts

@@ -1,6 +1,8 @@
 /**
  * Component for displaying bash command execution with streaming output.
  */
+
+import { sanitizeText } from "@oh-my-pi/pi-natives";
 import { Container, Loader, Spacer, Text, type TUI } from "@oh-my-pi/pi-tui";
 import { getSymbolTheme, theme } from "../../modes/theme/theme";
 import type { TruncationMeta } from "../../tools/output-meta";
@@ -10,6 +12,7 @@ import { truncateToVisualLines } from "./visual-truncate";
 
 // Preview line limit when not expanded (matches tool execution behavior)
 const PREVIEW_LINES = 20;
+const MAX_DISPLAY_LINE_CHARS = 4000;
 
 export class BashExecutionComponent extends Container {
 	#outputLines: string[] = [];
@@ -73,13 +76,15 @@ export class BashExecutionComponent extends Container {
 	}
 
 	appendOutput(chunk: string): void {
-		const clean = this.#normalizeOutput(chunk);
+		const clean = sanitizeText(chunk);
 
 		// Append to output lines
-		const newLines = clean.split("\n");
+		const newLines = clean.split("\n").map(line => this.#clampDisplayLine(line));
 		if (this.#outputLines.length > 0 && newLines.length > 0) {
 			// Append first chunk to last line (incomplete line continuation)
-			this.#outputLines[this.#outputLines.length - 1] += newLines[0];
+			this.#outputLines[this.#outputLines.length - 1] = this.#clampDisplayLine(
+				`${this.#outputLines[this.#outputLines.length - 1]}${newLines[0]}`,
+			);
 			this.#outputLines.push(...newLines.slice(1));
 		} else {
 			this.#outputLines.push(...newLines);
@@ -184,15 +189,17 @@ export class BashExecutionComponent extends Container {
 		}
 	}
 
-	#normalizeOutput(text: string): string {
-		// Strip ANSI codes and normalize line endings
-		// Note: binary data is already sanitized in tui-renderer.ts executeBashCommand
-		return Bun.stripANSI(text).replace(/\r\n/g, "\n").replace(/\r/g, "\n");
+	#clampDisplayLine(line: string): string {
+		if (line.length <= MAX_DISPLAY_LINE_CHARS) {
+			return line;
+		}
+		const omitted = line.length - MAX_DISPLAY_LINE_CHARS;
+		return `${line.slice(0, MAX_DISPLAY_LINE_CHARS)}… [${omitted} chars omitted]`;
 	}
 
 	#setOutput(output: string): void {
-		const clean = this.#normalizeOutput(output);
-		this.#outputLines = clean ? clean.split("\n") : [];
+		const clean = sanitizeText(output);
+		this.#outputLines = clean ? clean.split("\n").map(line => this.#clampDisplayLine(line)) : [];
 	}
 
 	/**

package/src/modes/components/python-execution.ts

@@ -2,6 +2,8 @@
  * Component for displaying user-initiated Python execution with streaming output.
  * Shares the same kernel session as the agent's Python tool.
  */
+
+import { sanitizeText } from "@oh-my-pi/pi-natives";
 import { Container, Loader, Spacer, Text, type TUI } from "@oh-my-pi/pi-tui";
 import { getSymbolTheme, highlightCode, theme } from "../../modes/theme/theme";
 import type { TruncationMeta } from "../../tools/output-meta";
@@ -10,6 +12,7 @@ import { DynamicBorder } from "./dynamic-border";
 import { truncateToVisualLines } from "./visual-truncate";
 
 const PREVIEW_LINES = 20;
+const MAX_DISPLAY_LINE_CHARS = 4000;
 
 export class PythonExecutionComponent extends Container {
 	#outputLines: string[] = [];
@@ -70,11 +73,13 @@ export class PythonExecutionComponent extends Container {
 	}
 
 	appendOutput(chunk: string): void {
-		const clean = this.#normalizeOutput(chunk);
+		const clean = sanitizeText(chunk);
 
-		const newLines = clean.split("\n");
+		const newLines = clean.split("\n").map(line => this.#clampDisplayLine(line));
 		if (this.#outputLines.length > 0 && newLines.length > 0) {
-			this.#outputLines[this.#outputLines.length - 1] += newLines[0];
+			this.#outputLines[this.#outputLines.length - 1] = this.#clampDisplayLine(
+				`${this.#outputLines[this.#outputLines.length - 1]}${newLines[0]}`,
+			);
 			this.#outputLines.push(...newLines.slice(1));
 		} else {
 			this.#outputLines.push(...newLines);
@@ -168,13 +173,17 @@ export class PythonExecutionComponent extends Container {
 		}
 	}
 
-	#normalizeOutput(text: string): string {
-		return Bun.stripANSI(text).replace(/\r\n/g, "\n").replace(/\r/g, "\n");
+	#clampDisplayLine(line: string): string {
+		if (line.length <= MAX_DISPLAY_LINE_CHARS) {
+			return line;
+		}
+		const omitted = line.length - MAX_DISPLAY_LINE_CHARS;
+		return `${line.slice(0, MAX_DISPLAY_LINE_CHARS)}… [${omitted} chars omitted]`;
 	}
 
 	#setOutput(output: string): void {
-		const clean = this.#normalizeOutput(output);
-		this.#outputLines = clean ? clean.split("\n") : [];
+		const clean = sanitizeText(output);
+		this.#outputLines = clean ? clean.split("\n").map(line => this.#clampDisplayLine(line)) : [];
 	}
 
 	getOutput(): string {

package/src/modes/components/tool-execution.ts

@@ -1,4 +1,5 @@
 import type { AgentTool } from "@oh-my-pi/pi-agent-core";
+import { sanitizeText } from "@oh-my-pi/pi-natives";
 import {
 	Box,
 	type Component,
@@ -12,7 +13,7 @@ import {
 	Text,
 	type TUI,
 } from "@oh-my-pi/pi-tui";
-import { logger, sanitizeText } from "@oh-my-pi/pi-utils";
+import { logger } from "@oh-my-pi/pi-utils";
 import { getProjectDir } from "@oh-my-pi/pi-utils/dirs";
 import type { Theme } from "../../modes/theme/theme";
 import { theme } from "../../modes/theme/theme";

package/src/modes/controllers/command-controller.ts

@@ -432,7 +432,7 @@ export class CommandController {
 
 		if (action === "reset" || action === "clear") {
 			try {
-				await clearMemoryData(agentDir);
+				await clearMemoryData(agentDir, this.ctx.sessionManager.getCwd());
 				await this.ctx.session.refreshBaseSystemPrompt();
 				this.ctx.showStatus("Memory data cleared and system prompt refreshed.");
 			} catch (error) {

package/src/session/streaming-output.ts

@@ -1,4 +1,4 @@
-import { sanitizeText } from "@oh-my-pi/pi-utils";
+import { sanitizeText } from "@oh-my-pi/pi-natives";
 import { DEFAULT_MAX_BYTES } from "../tools/truncate";
 
 export interface OutputSummary {

package/src/tools/bash-interactive.ts

@@ -1,5 +1,5 @@
 import type { AgentToolContext } from "@oh-my-pi/pi-agent-core";
-import { type PtyRunResult, PtySession } from "@oh-my-pi/pi-natives";
+import { type PtyRunResult, PtySession, sanitizeText } from "@oh-my-pi/pi-natives";
 import {
 	type Component,
 	matchesKey,
@@ -23,9 +23,8 @@ export interface BashInteractiveResult extends OutputSummary {
 }
 
 function normalizeCaptureChunk(chunk: string): string {
-	const noAnsi = Bun.stripANSI(chunk);
-	const normalized = noAnsi.replace(/\r\n/gu, "\n").replace(/\r/gu, "\n");
-	return normalized.replace(/[\x00-\x08\x0B-\x1F\x7F]/gu, "");
+	const normalized = chunk.replace(/\r\n/gu, "\n").replace(/\r/gu, "\n");
+	return sanitizeText(normalized);
 }
 
 const XtermTerminal = xterm.Terminal;
@@ -168,7 +167,7 @@ class BashInteractiveOverlayComponent implements Component {
 		const visibleLines: string[] = [];
 		for (let i = 0; i < maxContentRows; i++) {
 			const line = buffer.getLine(viewportY + i)?.translateToString(true) ?? "";
-			visibleLines.push(truncateToWidth(replaceTabs(line), innerWidth));
+			visibleLines.push(truncateToWidth(replaceTabs(sanitizeText(line)), innerWidth));
 		}
 		return visibleLines;
 	}
@@ -350,7 +349,12 @@ export async function runInteractiveBashPty(
 					},
 					(err, chunk) => {
 						if (err || !chunk) return;
-						component.appendOutput(chunk);
+						try {
+							component.appendOutput(chunk);
+						} catch {
+							const normalizedChunk = normalizeCaptureChunk(chunk);
+							component.appendOutput(normalizedChunk);
+						}
 						const normalizedChunk = normalizeCaptureChunk(chunk);
 						pendingChunks = pendingChunks.then(() => sink.push(normalizedChunk)).catch(() => {});
 						tui.requestRender();

package/src/tools/fetch.ts

@@ -241,7 +241,7 @@ async function tryContentNegotiation(
 	if (!result.ok) return null;
 
 	const mime = normalizeMime(result.contentType);
-	if (mime.includes("markdown") || mime === "text/plain") {
+	if ((mime.includes("markdown") || mime === "text/plain") && !looksLikeHtml(result.content)) {
 		return { content: result.content, type: result.contentType };
 	}
 

package/src/tools/output-meta.ts

@@ -21,6 +21,7 @@ export interface TruncationMeta {
 	totalBytes: number;
 	outputLines: number;
 	outputBytes: number;
+	maxBytes?: number;
 	/** Line range shown (1-indexed, inclusive) */
 	shownRange?: { start: number; end: number };
 	/** Artifact ID if full output was saved */
@@ -128,6 +129,7 @@ export class OutputMetaBuilder {
 			totalBytes: result.totalBytes,
 			outputLines: result.outputLines,
 			outputBytes: result.outputBytes,
+			maxBytes: result.maxBytes,
 			shownRange: { start: shownStart, end: shownEnd },
 			artifactId,
 			nextOffset: direction === "head" ? shownEnd + 1 : undefined,
@@ -212,6 +214,7 @@ export class OutputMetaBuilder {
 			totalBytes,
 			outputLines,
 			outputBytes,
+			maxBytes: options.maxBytes,
 			shownRange: { start: shownStart, end: shownEnd },
 			nextOffset: options.direction === "head" ? shownEnd + 1 : undefined,
 		};
@@ -319,14 +322,15 @@ export function formatOutputNotice(meta: OutputMeta | undefined): string {
 		const range = t.shownRange;
 		let notice: string;
 
-		if (range) {
+		if (range && range.end >= range.start) {
 			notice = `Showing lines ${range.start}-${range.end} of ${t.totalLines}`;
 		} else {
 			notice = `Showing ${t.outputLines} of ${t.totalLines} lines`;
 		}
 
 		if (t.truncatedBy === "bytes") {
-			notice += ` (${formatSize(t.outputBytes)} limit)`;
+			const maxBytes = t.maxBytes ?? t.outputBytes;
+			notice += ` (${formatSize(maxBytes)} limit)`;
 		}
 
 		if (t.nextOffset != null) {

package/src/web/scrapers/types.ts

@@ -92,6 +92,7 @@ export async function loadPage(url: string, options: LoadPageOptions = {}): Prom
 					"User-Agent": userAgent,
 					Accept: "text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8",
 					"Accept-Language": "en-US,en;q=0.5",
+					"Accept-Encoding": "identity", // Cloudflare Markdown-for-Agents returns corrupted bytes when compression is negotiated
 					...headers,
 				},
 				redirect: "follow",