@wierdbytes/pi-facelift 0.3.1 → 0.3.2

package/CHANGELOG.md

@@ -1,5 +1,18 @@
 # Changelog
 
+## 0.3.2
+
+- **Change**: `bash` tool titles always render the full command, even
+  in collapsed view. Previously the multi-line command string was
+  sliced at 80 chars (with a trailing `…`) before being split into
+  sub-tree continuation rows, which clobbered the second line of
+  short multi-step commands like
+  `tmux kill-session -t poc 2>/dev/null\nrm -f /Users/.../poc/long-path`.
+  Per-line right-truncation inside `frameTop` is unchanged — lines
+  that genuinely overflow the terminal width are still clipped — so
+  the only behavior change is that long commands no longer hide
+  content behind the expand toggle.
+
 ## 0.3.1
 
 - **Fix**: `read` bodies rendered as plain uncolored text whenever

package/bash-rendering.test.ts

@@ -72,21 +72,24 @@ function loadBashTool() {
 }
 
 describe("bash renderCall expansion", () => {
-	it("truncates long commands when collapsed", () => {
-		const bashTool = loadBashTool();
-		const command = `printf '${"x".repeat(120)}'`;
+	it("shows the full command even when collapsed (no length-based truncation)", () => {
+		withStdoutColumns(200, () => {
+			const bashTool = loadBashTool();
+			// 120-char command fits inside a 200-col frame, so the only previous
+			// reason it would have been clipped was the old 80-char compact cap.
+			const command = `printf '${"x".repeat(120)}'`;
 
-		const rendered = bashTool.renderCall({ command }, mockTheme, {
-			lastComponent: new MockText(),
-			isError: false,
-			state: {},
-			expanded: false,
-			invalidate: () => {},
-		});
+			const rendered = bashTool.renderCall({ command }, mockTheme, {
+				lastComponent: new MockText(),
+				isError: false,
+				state: {},
+				expanded: false,
+				invalidate: () => {},
+			});
 
-		expect(rendered.getText()).toContain("bash");
-		expect(rendered.getText()).toContain("…");
-		expect(rendered.getText()).not.toContain(command);
+			expect(rendered.getText()).toContain("bash");
+			expect(rendered.getText()).toContain(command);
+		});
 	});
 
 	it("shows the full command when expanded", () => {

package/index.ts

@@ -1214,7 +1214,6 @@ export default function piFaceliftExtension(pi: PiFaceliftApi, deps?: PiFacelift
 					state.endedAt = undefined;
 				}
 				const timeout = args.timeout ? ` ${theme.fg("muted", `(${args.timeout}s timeout)`)}` : "";
-				const displayCmd = ctx.expanded || cmd.length <= 80 ? cmd : `${cmd.slice(0, 77)}…`;
 
 				// Multi-line bash commands (shell line continuations with `\`, here-docs,
 				// or embedded newlines) need each line wrapped in the accent color
@@ -1222,7 +1221,12 @@ export default function piFaceliftExtension(pi: PiFaceliftApi, deps?: PiFacelift
 				// whitespace on continuation lines is dropped — the sub-tree connector
 				// already provides visual indentation, so the user's heredoc indent
 				// would just push content right and misalign the tree.
-				const cmdLines = displayCmd.split("\n");
+				//
+				// The full command is always rendered in the title — no length-based
+				// truncation in compact mode. `frameTop` still right-truncates any
+				// individual line that exceeds the frame width, which is a separate
+				// (display-fit) concern from hiding command content.
+				const cmdLines = cmd.split("\n");
 				const firstCmd = cmdLines[0];
 				const restCmd = cmdLines.slice(1).map((line) => line.replace(/^\s+/, ""));
 				const firstTitle = `${theme.fg("toolTitle", theme.bold("bash"))} ${theme.fg("accent", firstCmd)}${timeout}`;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wierdbytes/pi-facelift",
-  "version": "0.3.1",
+  "version": "0.3.2",
   "description": "Cosmetic facelift for built-in pi tool output — open-right rounded frames, syntax-highlighted reads, status-aware bash summaries, find/grep.",
   "type": "module",
   "files": [