@oh-my-pi/pi-coding-agent 11.6.1 → 11.7.0

package/CHANGELOG.md

@@ -2,6 +2,17 @@
 
 ## [Unreleased]
 
+## [11.7.0] - 2026-02-07
+### Changed
+
+- Enhanced error messages for failed Python cells to include full combined output context instead of just the error message
+- Updated error cell output styling to use error color theme instead of standard tool output color for better visual distinction
+
+### Fixed
+
+- Improved error handling in Python cell execution to preserve and display combined output from previous cells when an error occurs
+- Fixed tab character rendering in Python tool output display to properly format whitespace in cell output and status events
+
 ## [11.6.1] - 2026-02-07
 ### Fixed
 

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "11.6.1",
+	"version": "11.7.0",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"type": "module",
 	"ompConfig": {
@@ -90,12 +90,12 @@
 		"@mozilla/readability": "0.6.0",
 		"@oclif/core": "^4.8.0",
 		"@oclif/plugin-autocomplete": "^3.2.40",
-		"@oh-my-pi/omp-stats": "11.6.1",
-		"@oh-my-pi/pi-agent-core": "11.6.1",
-		"@oh-my-pi/pi-ai": "11.6.1",
-		"@oh-my-pi/pi-natives": "11.6.1",
-		"@oh-my-pi/pi-tui": "11.6.1",
-		"@oh-my-pi/pi-utils": "11.6.1",
+		"@oh-my-pi/omp-stats": "11.7.0",
+		"@oh-my-pi/pi-agent-core": "11.7.0",
+		"@oh-my-pi/pi-ai": "11.7.0",
+		"@oh-my-pi/pi-natives": "11.7.0",
+		"@oh-my-pi/pi-tui": "11.7.0",
+		"@oh-my-pi/pi-utils": "11.7.0",
 		"@sinclair/typebox": "^0.34.48",
 		"ajv": "^8.17.1",
 		"chalk": "^5.6.2",

package/src/modes/interactive-mode.ts

@@ -454,6 +454,7 @@ export class InteractiveMode implements InteractiveModeContext {
 	private async loadTodoList(): Promise<void> {
 		const sessionFile = this.sessionManager.getSessionFile() ?? null;
 		if (!sessionFile) {
+			this.todoItems = [];
 			this.renderTodoList();
 			return;
 		}
@@ -463,9 +464,12 @@ export class InteractiveMode implements InteractiveModeContext {
 			const data = (await Bun.file(todoPath).json()) as { todos?: TodoItem[] };
 			if (data?.todos && Array.isArray(data.todos)) {
 				this.todoItems = data.todos;
+			} else {
+				this.todoItems = [];
 			}
 		} catch (error) {
 			if (isEnoent(error)) {
+				this.todoItems = [];
 				this.renderTodoList();
 				return;
 			}

package/src/modes/theme/theme.ts

@@ -184,242 +184,149 @@ export type SymbolKey =
 type SymbolMap = Record<SymbolKey, string>;
 
 const UNICODE_SYMBOLS: SymbolMap = {
-	// Status Indicators
-	// pick: ✓ | alt: ✔ ✅ ☑ ✚
-	"status.success": "✓",
-	// pick: ✗ | alt: ✘ ✖ ❌ ⨯
-	"status.error": "✗",
-	// pick: ⚠ | alt: ‼ ⁉ ▲ △
+	// Status
+	"status.success": "✔",
+	"status.error": "✘",
 	"status.warning": "⚠",
-	// pick: ℹ | alt: ⓘ 🛈 ⅈ
-	"status.info": "ℹ",
-	// pick: ◔ | alt: ● ◐ ◑ ◒ ◓ ⏳ …
-	"status.pending": "◔",
-	// pick: ○ | alt: ◌ ◯ ⃠
-	"status.disabled": "○",
-	// pick: ● | alt: ◉ ◎ ⬤
+	"status.info": "ⓘ",
+	"status.pending": "⏳",
+	"status.disabled": "⦸",
 	"status.enabled": "●",
-	// pick: ↻ | alt: ↺ ⟳ ⟲ ◐ ▶
-	"status.running": "↻",
-	// pick: ◐ | alt: ◑ ◒ ◓ ◔
-	"status.shadowed": "◐",
-	// pick: ⊗ | alt: ⊘ ⛔ ⏹ ⨂
-	"status.aborted": "⊗",
+	"status.running": "⟳",
+	"status.shadowed": "◌",
+	"status.aborted": "⏹",
 	// Navigation
-	// pick: ❯ | alt: › ▸ ▹
 	"nav.cursor": "❯",
-	// pick: ➜ | alt: → ➤ ➔ ⇒
-	"nav.selected": "➜",
-	// pick: ▸ | alt: ▶ ▹ ⯈
+	"nav.selected": "➤",
 	"nav.expand": "▸",
-	// pick: ▾ | alt: ▼ ▽ ⯆
 	"nav.collapse": "▾",
-	// pick: ← | alt: ↩ ↫ ⇦
-	"nav.back": "←",
-	// Tree Connectors
-	// pick: ├─ | alt: ├╴ ├╌ ├┄ ╠═
+	"nav.back": "⟵",
+	// Tree
 	"tree.branch": "├─",
-	// pick: └─ | alt: └╴ └╌ └┄ ╚═
 	"tree.last": "└─",
-	// pick: │ | alt: ┃ ║ ▏ ▕
 	"tree.vertical": "│",
-	// pick: ─ | alt: ━ ═ ╌ ┄
 	"tree.horizontal": "─",
-	// pick: └ | alt: ⎿ ╰ ↳
-	"tree.hook": "\u2514",
-	// Box Drawing - Rounded
-	// pick: ╭ | alt: ┌ ┏ ╔
+	"tree.hook": "└",
+	// Box (rounded)
 	"boxRound.topLeft": "╭",
-	// pick: ╮ | alt: ┐ ┓ ╗
 	"boxRound.topRight": "╮",
-	// pick: ╰ | alt: └ ┗ ╚
 	"boxRound.bottomLeft": "╰",
-	// pick: ╯ | alt: ┘ ┛ ╝
 	"boxRound.bottomRight": "╯",
-	// pick: ─ | alt: ━ ═ ╌
 	"boxRound.horizontal": "─",
-	// pick: │ | alt: ┃ ║ ▏
 	"boxRound.vertical": "│",
-	// Box Drawing - Sharp
-	// pick: ┌ | alt: ┏ ╭ ╔
+	// Box (sharp)
 	"boxSharp.topLeft": "┌",
-	// pick: ┐ | alt: ┓ ╮ ╗
 	"boxSharp.topRight": "┐",
-	// pick: └ | alt: ┗ ╰ ╚
 	"boxSharp.bottomLeft": "└",
-	// pick: ┘ | alt: ┛ ╯ ╝
 	"boxSharp.bottomRight": "┘",
-	// pick: ─ | alt: ━ ═ ╌
 	"boxSharp.horizontal": "─",
-	// pick: │ | alt: ┃ ║ ▏
 	"boxSharp.vertical": "│",
-	// pick: ┼ | alt: ╋ ╬ ┿
 	"boxSharp.cross": "┼",
-	// pick: ┬ | alt: ╦ ┯ ┳
 	"boxSharp.teeDown": "┬",
-	// pick: ┴ | alt: ╩ ┷ ┻
 	"boxSharp.teeUp": "┴",
-	// pick: ├ | alt: ╠ ┝ ┣
 	"boxSharp.teeRight": "├",
-	// pick: ┤ | alt: ╣ ┥ ┫
 	"boxSharp.teeLeft": "┤",
-	// Separators
-	// pick: │ | alt: ┃ ║ ▏
-	"sep.powerline": "│",
-	// pick: │ | alt: ┆ ┊
-	"sep.powerlineThin": "│",
-	// pick: > | alt: › ▸ »
-	"sep.powerlineLeft": ">",
-	// pick: < | alt: ‹ ◂ «
-	"sep.powerlineRight": "<",
-	// pick: > | alt: › ▸
+	// Separators (powerline-ish, but pure Unicode)
+	"sep.powerline": "▕",
+	"sep.powerlineThin": "┆",
+	"sep.powerlineLeft": "▶",
+	"sep.powerlineRight": "◀",
 	"sep.powerlineThinLeft": ">",
-	// pick: < | alt: ‹ ◂
 	"sep.powerlineThinRight": "<",
-	// pick: █ | alt: ▓ ▒ ░ ▉ ▌
-	"sep.block": "█",
-	// pick: space | alt: ␠ ·
+	"sep.block": "▌",
 	"sep.space": " ",
-	// pick: > | alt: › » ▸
 	"sep.asciiLeft": ">",
-	// pick: < | alt: ‹ « ◂
 	"sep.asciiRight": "<",
-	// pick: · | alt: • ⋅
 	"sep.dot": " · ",
-	// pick: / | alt: ／ ∕ ⁄
 	"sep.slash": " / ",
-	// pick: | | alt: │ ┃ ║
-	"sep.pipe": " | ",
+	"sep.pipe": " │ ",
 	// Icons
-	// pick: ◈ | alt: ◆ ⬢ ◇
-	"icon.model": "◈",
-	// pick: 📋 | alt: 🗒 📝
-	"icon.plan": "📋",
-	// pick: 📁 | alt: 📂 🗂 🗃
+	"icon.model": "⬢",
+	"icon.plan": "🗺",
 	"icon.folder": "📁",
-	// pick: 📄 | alt: 📃 📝
 	"icon.file": "📄",
-	// pick: ⎇ | alt: 🔀 ⑂
 	"icon.git": "⎇",
-	// pick: ⎇ | alt: 🌿 ⑂
-	"icon.branch": "⎇",
-	// pick: ⊛ | alt: ◎ ◍ ⊙
-	"icon.tokens": "⊛",
-	// pick: ◫ | alt: ◧ ▣ ▦
+	"icon.branch": "⑂",
+	"icon.tokens": "🪙",
 	"icon.context": "◫",
-	// pick: $ | alt: 💲 💰
-	"icon.cost": "$",
-	// pick: ◷ | alt: ⏱ ⏲ ⌛
-	"icon.time": "◷",
-	// pick: π | alt: ∏ ∑
+	"icon.cost": "💲",
+	"icon.time": "⏱",
 	"icon.pi": "π",
-	// pick: AG | alt: 👥 👤
-	"icon.agents": "AG",
-	// pick: cache | alt: 💾 🗄
-	"icon.cache": "cache",
-	// pick: in: | alt: ⤵ ↲
-	"icon.input": "in:",
-	// pick: out: | alt: ⤴ ↱
-	"icon.output": "out:",
-	// pick: host | alt: 🖥 💻
-	"icon.host": "host",
-	// pick: id | alt: 🧭 🧩
-	"icon.session": "id",
-	// pick: 📦 | alt: 🧰
+	"icon.agents": "👥",
+	"icon.cache": "💾",
+	"icon.input": "⤵",
+	"icon.output": "⤴",
+	"icon.host": "🖥",
+	"icon.session": "🆔",
 	"icon.package": "📦",
-	// pick: ⚠ | alt: ❗
 	"icon.warning": "⚠",
-	// pick: ↩ | alt: ↺ ⟲
-	"icon.rewind": "↩",
-	// pick: ⚡ | alt: ✨ ✦
-	"icon.auto": "⚡",
-	// pick: ✧ | alt: ⚙ SK 🧠
-	"icon.extensionSkill": "✧",
-	// pick: ⚒ | alt: ⛭ TL 🛠
-	"icon.extensionTool": "⚒",
-	// pick: / | alt: ⌘ ⌥
-	"icon.extensionSlashCommand": "/",
-	// pick: ◈ | alt: ⧫ MCP 🔌
-	"icon.extensionMcp": "◈",
-	// pick: § | alt: ⚖ RL 📏
-	"icon.extensionRule": "§",
-	// pick: ↪ | alt: ⚓ HK 🪝
-	"icon.extensionHook": "↪",
-	// pick: PR | alt: 💬 ✎
-	"icon.extensionPrompt": "PR",
-	// pick: CF | alt: 📄 📎
-	"icon.extensionContextFile": "CF",
-	// pick: IN | alt: 📘 ℹ
-	"icon.extensionInstruction": "IN",
-	// Thinking Levels
-	// pick: [min] | alt: · ◔ min
-	"thinking.minimal": "[min]",
-	// pick: [low] | alt: ◑ low ▪ low
-	"thinking.low": "[low]",
-	// pick: [med] | alt: ◒ med ▪ med
-	"thinking.medium": "[med]",
-	// pick: [high] | alt: ◕ high ▪ high
-	"thinking.high": "[high]",
-	// pick: [xhi] | alt: ◉ xhi ▪ xhi
-	"thinking.xhigh": "[xhi]",
+	"icon.rewind": "↶",
+	"icon.auto": "⟲",
+	"icon.extensionSkill": "✦",
+	"icon.extensionTool": "🛠",
+	"icon.extensionSlashCommand": "⌘",
+	"icon.extensionMcp": "🔌",
+	"icon.extensionRule": "⚖",
+	"icon.extensionHook": "🪝",
+	"icon.extensionPrompt": "✎",
+	"icon.extensionContextFile": "📎",
+	"icon.extensionInstruction": "📘",
+	// Thinking levels
+	"thinking.minimal": "◔ min",
+	"thinking.low": "◑ low",
+	"thinking.medium": "◒ med",
+	"thinking.high": "◕ high",
+	"thinking.xhigh": "◉ xhi",
 	// Checkboxes
-	// pick: ☑ | alt: ✓ ✔ ✅
 	"checkbox.checked": "☑",
-	// pick: ☐ | alt: □ ▢
 	"checkbox.unchecked": "☐",
-	// pick: • | alt: · ▪ ◦
+	// Formatting
 	"format.bullet": "•",
-	// pick: – | alt: — ― -
-	"format.dash": "–",
-	// pick: ⟨ | alt: [ ⟦
-	"format.bracketLeft": "⟨",
-	// pick: ⟩ | alt: ] ⟧
-	"format.bracketRight": "⟩",
-	// Markdown-specific
-	// pick: │ | alt: ┃ ║
-	"md.quoteBorder": "│",
-	// pick: ─ | alt: ━ ═
+	"format.dash": "—",
+	"format.bracketLeft": "⟦",
+	"format.bracketRight": "⟧",
+	// Markdown
+	"md.quoteBorder": "▏",
 	"md.hrChar": "─",
-	// pick: • | alt: · ▪ ◦
 	"md.bullet": "•",
-	// Language icons (unicode uses code symbol prefix)
-	"lang.default": "❖",
-	"lang.typescript": "❖ ts",
-	"lang.javascript": "❖ js",
-	"lang.python": "❖ py",
-	"lang.rust": "❖ rs",
-	"lang.go": "❖ go",
-	"lang.java": "❖ java",
-	"lang.c": "❖ c",
-	"lang.cpp": "❖ c++",
-	"lang.csharp": "❖ c#",
-	"lang.ruby": "❖ rb",
-	"lang.php": "❖ php",
-	"lang.swift": "❖ swift",
-	"lang.kotlin": "❖ kt",
-	"lang.shell": "❖ sh",
-	"lang.html": "❖ html",
-	"lang.css": "❖ css",
-	"lang.json": "❖ json",
-	"lang.yaml": "❖ yaml",
-	"lang.markdown": "❖ md",
-	"lang.sql": "❖ sql",
-	"lang.docker": "❖ docker",
-	"lang.lua": "❖ lua",
-	"lang.text": "❖ txt",
-	"lang.env": "❖ env",
-	"lang.toml": "❖ toml",
-	"lang.xml": "❖ xml",
-	"lang.ini": "❖ ini",
-	"lang.conf": "❖ conf",
-	"lang.log": "❖ log",
-	"lang.csv": "❖ csv",
-	"lang.tsv": "❖ tsv",
-	"lang.image": "❖ img",
-	"lang.pdf": "❖ pdf",
-	"lang.archive": "❖ zip",
-	"lang.binary": "❖ bin",
-	// Settings tab icons
+	// Language/file icons (emoji-centric, no Nerd Font required)
+	"lang.default": "⌘",
+	"lang.typescript": "🟦",
+	"lang.javascript": "🟨",
+	"lang.python": "🐍",
+	"lang.rust": "🦀",
+	"lang.go": "🐹",
+	"lang.java": "☕",
+	"lang.c": "Ⓒ",
+	"lang.cpp": "➕",
+	"lang.csharp": "♯",
+	"lang.ruby": "💎",
+	"lang.php": "🐘",
+	"lang.swift": "🕊",
+	"lang.kotlin": "🅺",
+	"lang.shell": "💻",
+	"lang.html": "🌐",
+	"lang.css": "🎨",
+	"lang.json": "🧾",
+	"lang.yaml": "📋",
+	"lang.markdown": "📝",
+	"lang.sql": "🗄",
+	"lang.docker": "🐳",
+	"lang.lua": "🌙",
+	"lang.text": "🗒",
+	"lang.env": "🔧",
+	"lang.toml": "🧾",
+	"lang.xml": "⟨⟩",
+	"lang.ini": "⚙",
+	"lang.conf": "⚙",
+	"lang.log": "📜",
+	"lang.csv": "📑",
+	"lang.tsv": "📑",
+	"lang.image": "🖼",
+	"lang.pdf": "📕",
+	"lang.archive": "🗜",
+	"lang.binary": "⚙",
+	// Settings tabs
 	"tab.display": "🎨",
 	"tab.agent": "🤖",
 	"tab.input": "⌨",
@@ -846,7 +753,7 @@ export type SpinnerType = "status" | "activity";
 
 const SPINNER_FRAMES: Record<SymbolPreset, Record<SpinnerType, string[]>> = {
 	unicode: {
-		status: ["·", "•", "●", "•"],
+		status: ["⣾", "⣽", "⣻", "⢿", "⡿", "⣟", "⣯", "⣷"],
 		activity: ["⠋", "⠙", "⠹", "⠸", "⠼", "⠴", "⠦", "⠧", "⠇", "⠏"],
 	},
 	nerd: {

package/src/prompts/system/system-prompt.md

@@ -179,6 +179,7 @@ Continue non-destructively; someone's work may live there.
 - Ask for parameters only when required; otherwise choose safe defaults, state them.
 - Non-trivial logic: define test first when feasible.
 - Algorithmic work: start naive correct version before optimizing.
+- **Formatting is a batch operation, not per-edit cleanup.** Never fix indentation or style issues one edit at a time. Make all semantic changes first, then run the project's formatter once at the end. One command beats twenty whitespace edits.
 
 ## Integration
 - AGENTS.md defines local law; nearest wins, deeper overrides higher.

package/src/prompts/tools/patch.md

@@ -48,7 +48,7 @@ Returns success/failure; on failure, error message indicates:
 - Never use anchors as comments (no line numbers, location labels, placeholders like `@@ @@`)
 - Do not place new lines outside intended block
 - If edit fails or breaks structure, re-read file and produce new patch from current content—do not retry same diff
-- **NEVER** use edit to fix indentation or reformat code—run the project's formatter instead
+- **NEVER** use edit to fix indentation, whitespace, or reformat code. Formatting is a single command run once at the end (`bun fmt`, `cargo fmt`, `prettier --write`, etc.)—not N individual edits. If you see inconsistent indentation after an edit, leave it; the formatter will fix all of it in one pass.
 </critical>
 
 <example name="create">

package/src/tools/python.ts

@@ -17,7 +17,7 @@ import type { ToolSession } from ".";
 import type { OutputMeta } from "./output-meta";
 import { allocateOutputArtifact, createTailBuffer } from "./output-utils";
 import { resolveToCwd } from "./path-utils";
-import { shortenPath, ToolUIKit, truncateToWidth } from "./render-utils";
+import { replaceTabs, shortenPath, ToolUIKit, truncateToWidth } from "./render-utils";
 import { ToolAbortError, ToolError } from "./tool-errors";
 import { toolResult } from "./tool-result";
 import { DEFAULT_MAX_BYTES } from "./truncate";
@@ -341,18 +341,93 @@ export class PythonTool implements AgentTool<typeof pythonSchema> {
 					cellResult.status = "error";
 					pushUpdate();
 					const errorMsg = result.output || "Command aborted";
-					throw new ToolError(cells.length > 1 ? `Cell ${i + 1} aborted: ${errorMsg}` : errorMsg);
+					const combinedOutput = cellOutputs.join("\n\n");
+					const outputText =
+						cells.length > 1
+							? `${combinedOutput}\n\nCell ${i + 1} aborted: ${errorMsg}`
+							: combinedOutput || errorMsg;
+
+					const rawSummary = (await finalizeOutput()) ?? {
+						output: "",
+						truncated: false,
+						totalLines: 0,
+						totalBytes: 0,
+						outputLines: 0,
+						outputBytes: 0,
+					};
+					const outputLines = combinedOutput.length > 0 ? combinedOutput.split("\n").length : 0;
+					const outputBytes = Buffer.byteLength(combinedOutput, "utf-8");
+					const missingLines = Math.max(0, rawSummary.totalLines - rawSummary.outputLines);
+					const missingBytes = Math.max(0, rawSummary.totalBytes - rawSummary.outputBytes);
+					const summaryForMeta: OutputSummary = {
+						output: combinedOutput,
+						truncated: rawSummary.truncated,
+						totalLines: outputLines + missingLines,
+						totalBytes: outputBytes + missingBytes,
+						outputLines,
+						outputBytes,
+						artifactId: rawSummary.artifactId,
+					};
+
+					const details: PythonToolDetails = {
+						cells: cellResults,
+						jsonOutputs: jsonOutputs.length > 0 ? jsonOutputs : undefined,
+						images: images.length > 0 ? images : undefined,
+						statusEvents: statusEvents.length > 0 ? statusEvents : undefined,
+						isError: true,
+					};
+
+					return toolResult(details)
+						.text(outputText)
+						.truncationFromSummary(summaryForMeta, { direction: "tail" })
+						.done();
 				}
 
 				if (result.exitCode !== 0 && result.exitCode !== undefined) {
 					cellResult.status = "error";
 					pushUpdate();
 					const combinedOutput = cellOutputs.join("\n\n");
-					throw new ToolError(
+					const outputText =
 						cells.length > 1
 							? `${combinedOutput}\n\nCell ${i + 1} failed (exit code ${result.exitCode}). Earlier cells succeeded—their state persists. Fix only cell ${i + 1}.`
-							: `${combinedOutput}\n\nCommand exited with code ${result.exitCode}`,
-					);
+							: combinedOutput
+								? `${combinedOutput}\n\nCommand exited with code ${result.exitCode}`
+								: `Command exited with code ${result.exitCode}`;
+
+					const rawSummary = (await finalizeOutput()) ?? {
+						output: "",
+						truncated: false,
+						totalLines: 0,
+						totalBytes: 0,
+						outputLines: 0,
+						outputBytes: 0,
+					};
+					const outputLines = combinedOutput.length > 0 ? combinedOutput.split("\n").length : 0;
+					const outputBytes = Buffer.byteLength(combinedOutput, "utf-8");
+					const missingLines = Math.max(0, rawSummary.totalLines - rawSummary.outputLines);
+					const missingBytes = Math.max(0, rawSummary.totalBytes - rawSummary.outputBytes);
+					const summaryForMeta: OutputSummary = {
+						output: combinedOutput,
+						truncated: rawSummary.truncated,
+						totalLines: outputLines + missingLines,
+						totalBytes: outputBytes + missingBytes,
+						outputLines,
+						outputBytes,
+						artifactId: rawSummary.artifactId,
+					};
+
+					const details: PythonToolDetails = {
+						cells: cellResults,
+						jsonOutputs: jsonOutputs.length > 0 ? jsonOutputs : undefined,
+						images: images.length > 0 ? images : undefined,
+						statusEvents: statusEvents.length > 0 ? statusEvents : undefined,
+						isError: true,
+					};
+
+					return toolResult(details)
+						.text(outputText)
+						.truncationFromSummary(summaryForMeta, { direction: "tail" })
+						.done();
 				}
 
 				cellResult.status = "complete";
@@ -627,7 +702,7 @@ function formatStatusEventExpanded(event: PythonStatusEvent, theme: Theme): stri
 	const addPreview = (preview: string, maxLines = 3) => {
 		const previewLines = String(preview).split("\n").slice(0, maxLines);
 		for (const line of previewLines) {
-			lines.push(`   ${theme.fg("toolOutput", truncateToWidth(line, 80))}`);
+			lines.push(`   ${theme.fg("toolOutput", truncateToWidth(replaceTabs(line), 80))}`);
 		}
 		const totalLines = String(preview).split("\n").length;
 		if (totalLines > maxLines) {
@@ -744,7 +819,10 @@ function formatCellOutputLines(
 	const rawLines = cell.output ? cell.output.split("\n") : [];
 	const displayLines = expanded ? rawLines : rawLines.slice(-previewLines);
 	const hiddenCount = rawLines.length - displayLines.length;
-	const outputLines = displayLines.map(line => theme.fg("toolOutput", line));
+	const outputLines = displayLines.map(line => {
+		const cleaned = replaceTabs(line);
+		return cell.status === "error" ? theme.fg("error", cleaned) : theme.fg("toolOutput", cleaned);
+	});
 
 	if (outputLines.length === 0) {
 		return { lines: [], hiddenCount: 0 };