@oh-my-pi/pi-coding-agent 4.3.1 → 4.3.2

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 ## [Unreleased]
 
+## [4.3.2] - 2026-01-11
+### Changed
+
+- Increased default bash output preview from 5 to 10 lines when collapsed
+- Updated expanded bash output view to show full untruncated output when available
+
 ## [4.3.1] - 2026-01-11
 
 ### Changed

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "4.3.1",
+	"version": "4.3.2",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"type": "module",
 	"ompConfig": {
@@ -39,10 +39,10 @@
 		"prepublishOnly": "bun run generate-template && bun run clean && bun run build"
 	},
 	"dependencies": {
-		"@oh-my-pi/pi-ai": "4.3.1",
-		"@oh-my-pi/pi-agent-core": "4.3.1",
-		"@oh-my-pi/pi-git-tool": "4.3.1",
-		"@oh-my-pi/pi-tui": "4.3.1",
+		"@oh-my-pi/pi-ai": "4.3.2",
+		"@oh-my-pi/pi-agent-core": "4.3.2",
+		"@oh-my-pi/pi-git-tool": "4.3.2",
+		"@oh-my-pi/pi-tui": "4.3.2",
 		"@openai/agents": "^0.3.7",
 		"@sinclair/typebox": "^0.34.46",
 		"ajv": "^8.17.1",

package/src/core/tools/bash.ts

@@ -15,6 +15,8 @@ import { resolveToCwd } from "./path-utils";
 import { createToolUIKit } from "./render-utils";
 import { DEFAULT_MAX_BYTES, formatSize, type TruncationResult, truncateTail } from "./truncate";
 
+export const BASH_DEFAULT_PREVIEW_LINES = 10;
+
 const bashSchema = Type.Object({
 	command: Type.String({ description: "Bash command to execute" }),
 	timeout: Type.Optional(Type.Number({ description: "Timeout in seconds (optional, no default timeout)" })),
@@ -26,6 +28,7 @@ const bashSchema = Type.Object({
 export interface BashToolDetails {
 	truncation?: TruncationResult;
 	fullOutputPath?: string;
+	fullOutput?: string;
 }
 
 /**
@@ -101,9 +104,12 @@ export function createBashTool(session: ToolSession, options?: BashToolOptions):
 						const truncation = truncateTail(currentOutput);
 						onUpdate({
 							content: [{ type: "text", text: truncation.content || "" }],
-							details: {
-								truncation: truncation.truncated ? truncation : undefined,
-							},
+							details: truncation.truncated
+								? {
+										truncation,
+										fullOutput: currentOutput,
+									}
+								: undefined,
 						});
 					}
 				},
@@ -129,6 +135,7 @@ export function createBashTool(session: ToolSession, options?: BashToolOptions):
 				details = {
 					truncation,
 					fullOutputPath: result.fullOutputPath,
+					fullOutput: currentOutput,
 				};
 
 				const startLine = truncation.totalLines - truncation.outputLines + 1;
@@ -173,6 +180,9 @@ interface BashRenderContext {
 	previewLines?: number;
 }
 
+// Preview line limit when not expanded (matches tool-execution behavior)
+export const BASH_PREVIEW_LINES = 10;
+
 export const bashToolRenderer = {
 	renderCall(args: BashRenderArgs, uiTheme: Theme): Component {
 		const ui = createToolUIKit(uiTheme);
@@ -214,21 +224,25 @@ export const bashToolRenderer = {
 		const { renderContext } = options;
 		const details = result.details;
 
+		const expanded = renderContext?.expanded ?? options.expanded;
+		const previewLines = renderContext?.previewLines ?? BASH_DEFAULT_PREVIEW_LINES;
+
 		// Get output from context (preferred) or fall back to result content
 		const output = renderContext?.output ?? (result.content?.find((c) => c.type === "text")?.text ?? "").trim();
-		const expanded = renderContext?.expanded ?? options.expanded;
-		const previewLines = renderContext?.previewLines ?? 5;
+		const fullOutput = details?.fullOutput;
+		const displayOutput = expanded ? (fullOutput ?? output) : output;
+		const showingFullOutput = expanded && fullOutput !== undefined;
 
 		// Build truncation warning lines (static, doesn't depend on width)
 		const truncation = details?.truncation;
 		const fullOutputPath = details?.fullOutputPath;
 		let warningLine: string | undefined;
-		if (truncation?.truncated || fullOutputPath) {
+		if (fullOutputPath || (truncation?.truncated && !showingFullOutput)) {
 			const warnings: string[] = [];
 			if (fullOutputPath) {
 				warnings.push(`Full output: ${fullOutputPath}`);
 			}
-			if (truncation?.truncated) {
+			if (truncation?.truncated && !showingFullOutput) {
 				if (truncation.truncatedBy === "lines") {
 					warnings.push(`Truncated: showing ${truncation.outputLines} of ${truncation.totalLines} lines`);
 				} else {
@@ -237,17 +251,19 @@ export const bashToolRenderer = {
 					);
 				}
 			}
-			warningLine = uiTheme.fg("warning", ui.wrapBrackets(warnings.join(". ")));
+			if (warnings.length > 0) {
+				warningLine = uiTheme.fg("warning", ui.wrapBrackets(warnings.join(". ")));
+			}
 		}
 
-		if (!output) {
+		if (!displayOutput) {
 			// No output - just show warning if any
 			return new Text(warningLine ?? "", 0, 0);
 		}
 
 		if (expanded) {
 			// Show all lines when expanded
-			const styledOutput = output
+			const styledOutput = displayOutput
 				.split("\n")
 				.map((line) => uiTheme.fg("toolOutput", line))
 				.join("\n");
@@ -256,7 +272,7 @@ export const bashToolRenderer = {
 		}
 
 		// Collapsed: use width-aware caching component
-		const styledOutput = output
+		const styledOutput = displayOutput
 			.split("\n")
 			.map((line) => uiTheme.fg("toolOutput", line))
 			.join("\n");

package/src/core/tools/render-utils.ts

@@ -359,11 +359,11 @@ export function formatDiagnostics(
 			const isLastDiag = di === diagnostics.length - 1;
 			const diagBranch = isLastFile
 				? isLastDiag
-					? `   ${theme.tree.last}`
-					: `   ${theme.tree.branch}`
+					? `  ${theme.tree.last}`
+					: `  ${theme.tree.branch}`
 				: isLastDiag
-					? ` ${theme.tree.vertical} ${theme.tree.last}`
-					: ` ${theme.tree.vertical} ${theme.tree.branch}`;
+					? `${theme.tree.vertical} ${theme.tree.last}`
+					: `${theme.tree.vertical} ${theme.tree.branch}`;
 
 			const sevIcon =
 				d.severity === "error"

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

@@ -11,6 +11,7 @@ import {
 	type TUI,
 } from "@oh-my-pi/pi-tui";
 import stripAnsi from "strip-ansi";
+import { BASH_DEFAULT_PREVIEW_LINES } from "../../../core/tools/bash";
 import { computeEditDiff, type EditDiffError, type EditDiffResult } from "../../../core/tools/edit-diff";
 import { toolRenderers } from "../../../core/tools/renderers";
 import { convertToPng } from "../../../utils/image-convert";
@@ -19,7 +20,6 @@ import { theme } from "../theme/theme";
 import { renderDiff } from "./diff";
 
 // Preview line limit for bash when not expanded
-const BASH_PREVIEW_LINES = 5;
 const GENERIC_PREVIEW_LINES = 6;
 const GENERIC_ARG_PREVIEW = 6;
 const GENERIC_VALUE_MAX = 80;
@@ -485,7 +485,7 @@ export class ToolExecutionComponent extends Container {
 			const output = this.getTextOutput().trim();
 			context.output = output;
 			context.expanded = this.expanded;
-			context.previewLines = BASH_PREVIEW_LINES;
+			context.previewLines = BASH_DEFAULT_PREVIEW_LINES;
 		} else if (this.toolName === "edit") {
 			// Edit needs diff preview and renderDiff function
 			context.editDiffPreview = this.editDiffPreview;