@owlburtoe/pi-cc-tools 1.1.0 → 1.1.2

package/README.md

@@ -5,6 +5,8 @@ Claude Code inspired tool rendering for Pi — Shiki-powered diffs, status dots,
 ## Features
 
 - **Compact built-in tool rendering** for `read`, `bash`, `grep`, `find`, `ls`, `edit`, and `write`
+- **Semantic bash display** that renders common read-only shell one-liners like `nl -ba file | sed -n '1,200p'` as Claude Code-style `Read file (lines 1-200)` rows
+- **Read-only inspection grouping** that collapses consecutive `read`/`grep`/`find`/`ls` rows into one Claude-style `Inspect` block, capped to five visible entries by default
 - **Claude-style OpenAI tool rendering** for `apply_patch` plus common Pi/OpenAI-style tools like `webfetch`, `web_search`, `fetch_content`, task tools, and context tools
 - **`apply_patch` diff previews** that render parsed file patches in the call phase, similar to `edit`/`write`
 - **Adaptive edit/write diffs** with split or unified layouts, syntax highlighting, and inline word-level emphasis
@@ -33,6 +35,9 @@ Set in `.pi/settings.json` or `~/.pi/settings.json`:
   "bashOutputMode": "opencode",
   "bashCollapsedLines": 10,
   "bashStackConsecutive": true,
+  "bashSemanticDisplay": true,
+  "readOnlyToolGrouping": true,
+  "readOnlyToolGroupLimit": 5,
   "diffCollapsedLines": 24,
   "themeAdaptive": true,
   "diffTheme": "github-dark",
@@ -140,7 +145,7 @@ Color selections are persisted as `spinnerColor` / `spinnerStatusColor` in `~/.p
 
 | Value | Behavior |
 |-------|----------|
-| `opencode` | Compact status while collapsed, with `Ctrl+O` hint for output preview |
+| `opencode` | Compact status while collapsed, with `Ctrl+O` hint for output preview on raw shell commands. Semantic read-only bash rows omit the repeated hint to stay closer to Claude Code. |
 | `summary` | Status only; no output preview or expansion hint |
 | `preview` | Show a small output preview even while collapsed |
 
@@ -149,6 +154,8 @@ Color selections are persisted as `spinnerColor` / `spinnerStatusColor` in `~/.p
 | Setting | Default | Description |
 |---------|---------|-------------|
 | `bashStackConsecutive` | `true` | Remove the extra synthetic spacer between adjacent bash tool rows so command bursts render as a tight stack |
+| `bashSemanticDisplay` | `true` | Render common read-only shell file-inspection commands as semantic `Read` rows instead of raw `Bash` rows |
+| `readOnlyToolGrouping` | `true` | Collapse adjacent read-only inspection tools into one `Inspect` block; mutating `write`/`edit`/`apply_patch` rows stay independent |
 
 ### Numeric settings
 
@@ -158,6 +165,7 @@ Color selections are persisted as `spinnerColor` / `spinnerStatusColor` in `~/.p
 | `expandedPreviewMaxLines` | `4000` | Max lines when fully expanded |
 | `bashCollapsedLines` | `10` | Lines for collapsed bash output |
 | `diffCollapsedLines` | `24` | Diff lines before collapsing |
+| `readOnlyToolGroupLimit` | `5` | Max inspection entries shown inside a grouped read-only tool block |
 
 ## Notes
 

package/config/config.example.json

@@ -8,6 +8,9 @@
   "bashOutputMode": "opencode",
   "bashCollapsedLines": 10,
   "bashStackConsecutive": true,
+  "bashSemanticDisplay": true,
+  "readOnlyToolGrouping": true,
+  "readOnlyToolGroupLimit": 5,
   "diffCollapsedLines": 24,
   "showTruncationHints": false,
   "themeAdaptive": true,

package/extensions/index.ts

@@ -86,6 +86,10 @@ interface SettingsFile {
 	bashCollapsedLines?: number;
 	/** When true (default), consecutive bash tool rows render without the extra inter-tool spacer. */
 	bashStackConsecutive?: boolean;
+	/** When true (default), read-only shell file-inspection one-liners render as semantic Read rows instead of raw Bash rows. */
+	bashSemanticDisplay?: boolean;
+	readOnlyToolGrouping?: boolean;
+	readOnlyToolGroupLimit?: number;
 	showTruncationHints?: boolean;
 	diffCollapsedLines?: number;
 	diffTheme?: string;
@@ -339,6 +343,253 @@ function shouldStackConsecutiveBash(): boolean {
 	return readSettings().bashStackConsecutive !== false;
 }
 
+function readOnlyToolGroupingEnabled(): boolean {
+	return readSettings().readOnlyToolGrouping !== false;
+}
+
+function readOnlyToolGroupLimit(): number {
+	const value = readSettings().readOnlyToolGroupLimit;
+	return typeof value === "number" && Number.isFinite(value) && value > 0
+		? Math.max(1, Math.min(20, Math.floor(value)))
+		: 5;
+}
+
+function toolComponentRecord(value: unknown): Record<string, any> {
+	return value as Record<string, any>;
+}
+
+function componentCwd(value: unknown): string {
+	const cwd = toolComponentRecord(value).cwd;
+	return typeof cwd === "string" && cwd ? cwd : process.cwd();
+}
+
+function componentTextContent(value: unknown): string {
+	return getTextContent(toolComponentRecord(value).result);
+}
+
+function componentHasImageResult(value: unknown): boolean {
+	return !!getFirstImageBlock(toolComponentRecord(value).result);
+}
+
+function bashReadDisplayInfo(value: unknown): BashDisplayInfo | null {
+	const rec = toolComponentRecord(value);
+	if (rec.toolName !== "bash" || !bashSemanticDisplayEnabled()) return null;
+	return classifyBashCommandForDisplay(rec.args?.command ?? "");
+}
+
+function isReadOnlyInspectionToolExecution(value: unknown): boolean {
+	if (!readOnlyToolGroupingEnabled() || !isToolExecutionLike(value)) return false;
+	const rec = toolComponentRecord(value);
+	if (rec.expanded === true || componentHasImageResult(value)) return false;
+	if (rec.toolName === "read" || rec.toolName === "grep" || rec.toolName === "find" || rec.toolName === "ls") return true;
+	return !!bashReadDisplayInfo(value);
+}
+
+function hasConsecutiveReadOnlyInspectionToolChildren(children: unknown[]): boolean {
+	let previousWasInspect = false;
+	for (const child of children) {
+		const currentIsInspect = isReadOnlyInspectionToolExecution(child);
+		if (currentIsInspect && previousWasInspect) return true;
+		previousWasInspect = currentIsInspect;
+	}
+	return false;
+}
+
+function plural(count: number, noun: string): string {
+	if (count === 1) return `1 ${noun}`;
+	const suffix = /(?:s|x|z|ch|sh)$/i.test(noun) ? "es" : "s";
+	return `${count} ${noun}${suffix}`;
+}
+
+function textDataLines(text: string): string[] {
+	return text.split("\n").filter((line) => line.trim().length > 0 && !line.trimStart().startsWith("["));
+}
+
+function compactList(values: string[], limit: number): string {
+	if (values.length === 0) return "";
+	const shown = values.slice(0, limit);
+	const suffix = values.length > shown.length ? `, … +${values.length - shown.length}` : "";
+	return `${shown.join(", ")}${suffix}`;
+}
+
+function uniqueLeadingGrepFiles(text: string, limit: number): string[] {
+	const seen = new Set<string>();
+	const files: string[] = [];
+	for (const rawLine of text.split("\n")) {
+		const line = rawLine.trim();
+		if (!line || line.startsWith("[")) continue;
+		const match = /^(.+?)(?::\d+:|-\d+-)/.exec(line);
+		if (!match) continue;
+		const file = match[1];
+		if (seen.has(file)) continue;
+		seen.add(file);
+		files.push(file);
+		if (files.length >= limit + 1) break;
+	}
+	return files;
+}
+
+function formatOffsetLimit(args: any): string {
+	const parts: string[] = [];
+	if (args?.offset !== undefined && args?.offset !== null) parts.push(`offset=${args.offset}`);
+	if (args?.limit !== undefined && args?.limit !== null) parts.push(`limit=${args.limit}`);
+	return parts.length > 0 ? ` (${parts.join(", ")})` : "";
+}
+
+function readInspectionTarget(value: unknown): string {
+	const rec = toolComponentRecord(value);
+	return `${shortPath(componentCwd(value), rec.args?.path ?? "")}${formatOffsetLimit(rec.args)}`;
+}
+
+function grepInspectionTarget(value: unknown): string {
+	const rec = toolComponentRecord(value);
+	const pattern = summarizeText(rec.args?.pattern ?? "", 40);
+	const path = rec.args?.path ? ` in ${shortPath(componentCwd(value), rec.args.path)}` : "";
+	return `"${pattern}"${path}`;
+}
+
+function findInspectionTarget(value: unknown): string {
+	const rec = toolComponentRecord(value);
+	const pattern = summarizeText(rec.args?.pattern ?? "", 40);
+	const path = rec.args?.path ? ` in ${shortPath(componentCwd(value), rec.args.path)}` : "";
+	return `"${pattern}"${path}`;
+}
+
+function listInspectionTarget(value: unknown): string {
+	const rec = toolComponentRecord(value);
+	return shortPath(componentCwd(value), rec.args?.path ?? ".");
+}
+
+function readInspectionStatus(value: unknown): string {
+	const rec = toolComponentRecord(value);
+	if (!rec.result) return "Reading...";
+	if (rec.result?.isError) return "failed";
+	const text = componentTextContent(value);
+	const lines = text.length === 0 ? 0 : text.split("\n").length;
+	const suffix = rec.result?.details?.truncation?.truncated ? " (truncated)" : "";
+	return `${plural(lines, "line")} loaded${suffix}`;
+}
+
+function grepInspectionStatus(value: unknown): string {
+	const rec = toolComponentRecord(value);
+	if (!rec.result) return "Searching...";
+	if (rec.result?.isError) return "failed";
+	const text = componentTextContent(value).trim();
+	if (!text || /^No matches found\b/i.test(text)) return "no matches";
+	const lines = textDataLines(text);
+	let status = plural(lines.length, "match");
+	const files = uniqueLeadingGrepFiles(text, readOnlyToolGroupLimit());
+	if (files.length > 1) status += ` in ${compactList(files, readOnlyToolGroupLimit())}`;
+	if (rec.result?.details?.truncation?.truncated) status += " (truncated)";
+	return status;
+}
+
+function findInspectionStatus(value: unknown): string {
+	const rec = toolComponentRecord(value);
+	if (!rec.result) return "Finding...";
+	if (rec.result?.isError) return "failed";
+	const text = componentTextContent(value).trim();
+	if (!text || /^No files found\b/i.test(text)) return "no files";
+	const items = textDataLines(text);
+	let status = plural(items.length, "file");
+	if (items.length > 0) status += `: ${compactList(items, readOnlyToolGroupLimit())}`;
+	if (rec.result?.details?.truncation?.truncated) status += " (truncated)";
+	return status;
+}
+
+function listInspectionStatus(value: unknown): string {
+	const rec = toolComponentRecord(value);
+	if (!rec.result) return "Listing...";
+	if (rec.result?.isError) return "failed";
+	const text = componentTextContent(value).trim();
+	if (!text || /^empty directory\b/i.test(text)) return "empty directory";
+	const items = textDataLines(text);
+	let status = plural(items.length, "entry");
+	if (items.length > 0) status += `: ${compactList(items, readOnlyToolGroupLimit())}`;
+	return status;
+}
+
+function bashReadInspectionTarget(value: unknown): string {
+	const info = bashReadDisplayInfo(value);
+	if (!info) return "";
+	const range = info.rangeLabel ? ` (${info.rangeLabel})` : "";
+	return `${shortPath(componentCwd(value), info.path)}${range}`;
+}
+
+function bashReadInspectionStatus(value: unknown): string {
+	const rec = toolComponentRecord(value);
+	if (!rec.result) return "Reading...";
+	if (rec.result?.isError) return "failed";
+	const count = componentTextContent(value).split("\n").filter((line) => line.trim().length > 0).length;
+	return `${plural(count, "line")} read`;
+}
+
+function summarizeReadOnlyInspectionTool(value: unknown): string {
+	const rec = toolComponentRecord(value);
+	if (rec.toolName === "read") return `Read ${readInspectionTarget(value)} — ${readInspectionStatus(value)}`;
+	if (rec.toolName === "grep") return `Grep ${grepInspectionTarget(value)} — ${grepInspectionStatus(value)}`;
+	if (rec.toolName === "find") return `Find ${findInspectionTarget(value)} — ${findInspectionStatus(value)}`;
+	if (rec.toolName === "ls") return `List ${listInspectionTarget(value)} — ${listInspectionStatus(value)}`;
+	return `Read ${bashReadInspectionTarget(value)} — ${bashReadInspectionStatus(value)}`;
+}
+
+function renderInspectionGroup(group: unknown[], width: number): string[] {
+	syncToolBackgroundMode();
+	const limit = readOnlyToolGroupLimit();
+	const shown = group.slice(0, limit);
+	const remaining = group.length - shown.length;
+	const core: string[] = [` ${WRAP_MARK}● Inspect ${plural(group.length, "tool use")}`];
+	for (let index = 0; index < shown.length; index++) {
+		const isLast = index === shown.length - 1 && remaining === 0;
+		const branch = isLast ? "└─" : "├─";
+		core.push(` ${TOOL_RULE}${branch}${TRANSPARENT_RESET} ${WRAP_MARK}${summarizeReadOnlyInspectionTool(shown[index])}`);
+	}
+	if (remaining > 0) {
+		core.push(` ${TOOL_RULE}└─${TRANSPARENT_RESET} ${WRAP_MARK}… ${remaining} more inspection${remaining === 1 ? "" : "s"}`);
+	}
+	const renderedCore = core.flatMap((line) => wrapMarkedLine(line, width)).map((line) => padToWidth(line, width));
+	if (toolBackgroundMode === "outlines") return [" ".repeat(width), borderLine(width), ...renderedCore, borderLine(width)];
+	if (toolBackgroundMode === "transparent") return [" ".repeat(width), ...renderedCore];
+	return renderedCore;
+}
+
+function renderWithGroupedReadOnlyInspectionTools(container: any, width: number): string[] | null {
+	const children = Array.isArray(container?.children) ? container.children : null;
+	if (!children || !hasConsecutiveReadOnlyInspectionToolChildren(children)) return null;
+
+	const rendered: string[] = [];
+	let group: unknown[] = [];
+	let previousWasBash = false;
+	const flushGroup = () => {
+		if (group.length === 0) return;
+		if (group.length === 1) {
+			const child = group[0] as any;
+			const currentIsBash = isBashToolExecution(child);
+			const childLines = typeof child?.render === "function" ? child.render(width) : [];
+			rendered.push(...(currentIsBash && previousWasBash ? dropLeadingSpacerLine(childLines) : childLines));
+			previousWasBash = currentIsBash;
+		} else {
+			rendered.push(...renderInspectionGroup(group, width));
+			previousWasBash = false;
+		}
+		group = [];
+	};
+
+	for (const child of children) {
+		if (isReadOnlyInspectionToolExecution(child)) {
+			group.push(child);
+			continue;
+		}
+		flushGroup();
+		const currentIsBash = isBashToolExecution(child);
+		const childLines = typeof child?.render === "function" ? child.render(width) : [];
+		rendered.push(...(currentIsBash && previousWasBash ? dropLeadingSpacerLine(childLines) : childLines));
+		previousWasBash = currentIsBash;
+	}
+	flushGroup();
+	return rendered;
+}
+
 function hasConsecutiveBashToolChildren(children: unknown[]): boolean {
 	let previousWasBash = false;
 	for (const child of children) {
@@ -400,6 +651,8 @@ function patchGlobalToolBorders(): void {
 	const originalRender = proto.render;
 	proto.render = function patchedContainerRender(width: number): string[] {
 		if (!isToolExecutionLike(this)) {
+			const grouped = renderWithGroupedReadOnlyInspectionTools(this, width);
+			if (grouped) return grouped;
 			const stacked = renderWithStackedConsecutiveBash(this, width);
 			if (stacked) return stacked;
 		}
@@ -1373,6 +1626,191 @@ function bashOutputMode(): "opencode" | "summary" | "preview" {
 	return getMode(readSettings().bashOutputMode, ["opencode", "summary", "preview"] as const, "opencode");
 }
 
+function bashSemanticDisplayEnabled(): boolean {
+	return readSettings().bashSemanticDisplay !== false;
+}
+
+export interface BashDisplayInfo {
+	kind: "read";
+	label: "Read";
+	path: string;
+	rangeLabel?: string;
+	suppressCollapsedHint: boolean;
+}
+
+function tokenizeShellCommand(command: string): string[] | null {
+	const tokens: string[] = [];
+	let current = "";
+	let quote: "'" | '"' | null = null;
+	let escaping = false;
+
+	const push = () => {
+		if (current.length > 0) {
+			tokens.push(current);
+			current = "";
+		}
+	};
+
+	for (let i = 0; i < command.length; i++) {
+		const char = command[i];
+		if (escaping) {
+			current += char;
+			escaping = false;
+			continue;
+		}
+		if (quote) {
+			if (char === quote) {
+				quote = null;
+				continue;
+			}
+			if (quote === '"' && char === "\\") {
+				escaping = true;
+				continue;
+			}
+			current += char;
+			continue;
+		}
+		if (char === "\\") {
+			escaping = true;
+			continue;
+		}
+		if (char === "'" || char === '"') {
+			quote = char;
+			continue;
+		}
+		if (/\s/.test(char)) {
+			push();
+			continue;
+		}
+		if (char === "|" || char === ";") {
+			push();
+			if (char === "|" && command[i + 1] === "|") {
+				tokens.push("||");
+				i++;
+			} else {
+				tokens.push(char);
+			}
+			continue;
+		}
+		if (char === "&" && command[i + 1] === "&") {
+			push();
+			tokens.push("&&");
+			i++;
+			continue;
+		}
+		current += char;
+	}
+	if (escaping) current += "\\";
+	if (quote) return null;
+	push();
+	return tokens;
+}
+
+function isControlToken(token: string): boolean {
+	return token === "&&" || token === "||" || token === ";";
+}
+
+function isOptionToken(token: string): boolean {
+	return token.startsWith("-") && token !== "-" && token !== "--";
+}
+
+function singlePathArg(tokens: readonly string[]): string | null {
+	const paths = tokens.filter((token) => token !== "--" && token !== "-" && !isOptionToken(token));
+	return paths.length === 1 ? paths[0] : null;
+}
+
+function formatSedRangeLabel(script: string | undefined): string | undefined {
+	if (!script) return undefined;
+	const match = script.match(/^(\d+|\$)(?:,(\d+|\$))?p$/);
+	if (!match) return undefined;
+	const start = match[1];
+	const end = match[2];
+	if (!end || end === start) return `line ${start}`;
+	return `lines ${start}-${end}`;
+}
+
+function parseSedPrintRange(tokens: readonly string[]): string | undefined {
+	for (let i = 0; i < tokens.length; i++) {
+		if (tokens[i] === "-n" && tokens[i + 1]) {
+			const label = formatSedRangeLabel(tokens[i + 1]);
+			if (label) return label;
+		}
+		const direct = formatSedRangeLabel(tokens[i]);
+		if (direct) return direct;
+	}
+	return undefined;
+}
+
+function classifyNlRead(tokens: readonly string[]): BashDisplayInfo | null {
+	const pipeIndex = tokens.indexOf("|");
+	const left = pipeIndex === -1 ? tokens : tokens.slice(0, pipeIndex);
+	if (left[0] !== "nl") return null;
+	const path = singlePathArg(left.slice(1));
+	if (!path) return null;
+	let rangeLabel: string | undefined;
+	if (pipeIndex !== -1) {
+		const right = tokens.slice(pipeIndex + 1);
+		if (right[0] !== "sed" || right.includes("|")) return null;
+		rangeLabel = parseSedPrintRange(right);
+	}
+	return { kind: "read", label: "Read", path, rangeLabel, suppressCollapsedHint: true };
+}
+
+function classifySedRead(tokens: readonly string[]): BashDisplayInfo | null {
+	if (tokens[0] !== "sed") return null;
+	const rangeLabel = parseSedPrintRange(tokens);
+	if (!rangeLabel) return null;
+	const scriptIndex = tokens.findIndex((token) => formatSedRangeLabel(token) !== undefined);
+	const path = singlePathArg(tokens.slice(scriptIndex + 1));
+	if (!path) return null;
+	return { kind: "read", label: "Read", path, rangeLabel, suppressCollapsedHint: true };
+}
+
+function classifyCatRead(tokens: readonly string[]): BashDisplayInfo | null {
+	if (tokens[0] !== "cat") return null;
+	const path = singlePathArg(tokens.slice(1));
+	if (!path) return null;
+	return { kind: "read", label: "Read", path, suppressCollapsedHint: true };
+}
+
+function classifyHeadTailRead(tokens: readonly string[]): BashDisplayInfo | null {
+	const command = tokens[0];
+	if (command !== "head" && command !== "tail") return null;
+	const args = tokens.slice(1);
+	let count: string | undefined;
+	const pathCandidates: string[] = [];
+	for (let i = 0; i < args.length; i++) {
+		const token = args[i];
+		if (token === "--") continue;
+		if (token === "-n" && args[i + 1]) {
+			count = args[i + 1];
+			i++;
+			continue;
+		}
+		const compactCount = token.match(/^-([0-9]+)$/)?.[1];
+		if (compactCount) {
+			count = compactCount;
+			continue;
+		}
+		if (isOptionToken(token)) continue;
+		pathCandidates.push(token);
+	}
+	if (pathCandidates.length !== 1) return null;
+	const amount = count && /^\d+$/.test(count) ? count : "10";
+	const rangeLabel = `${command === "head" ? "first" : "last"} ${amount} lines`;
+	return { kind: "read", label: "Read", path: pathCandidates[0], rangeLabel, suppressCollapsedHint: true };
+}
+
+export function classifyBashCommandForDisplay(command: string): BashDisplayInfo | null {
+	const tokens = tokenizeShellCommand(command.trim());
+	if (!tokens || tokens.length === 0) return null;
+	if (tokens.some(isControlToken)) return null;
+	return classifyNlRead(tokens)
+		?? classifySedRead(tokens)
+		?? classifyCatRead(tokens)
+		?? classifyHeadTailRead(tokens);
+}
+
 function diffCollapsedLimit(): number {
 	const value = readSettings().diffCollapsedLines;
 	return typeof value === "number" && Number.isFinite(value) && value > 0 ? Math.floor(value) : 24;
@@ -4281,16 +4719,23 @@ export default function (pi: ExtensionAPI) {
 		},
 		renderCall(args, theme, ctx) {
 			syncToolCallStatus(ctx);
-			const summary = stableCallSummary(ctx, "_callSummary", () => summarizeText(args.command, 72));
-			return makeText(ctx.lastComponent, toolHeader("Bash", summary, theme, toolStatusDot(ctx, theme)));
+			const semantic = bashSemanticDisplayEnabled() ? classifyBashCommandForDisplay(args.command ?? "") : null;
+			const summary = stableCallSummary(ctx, "_callSummary", () => {
+				if (!semantic) return summarizeText(args.command, 72);
+				const path = sp(semantic.path);
+				return semantic.rangeLabel ? `${path} ${theme.fg("muted", `(${semantic.rangeLabel})`)}` : path;
+			});
+			return makeText(ctx.lastComponent, toolHeader(semantic?.label ?? "Bash", summary, theme, toolStatusDot(ctx, theme)));
 		},
 		renderResult(result, { expanded, isPartial }, theme, ctx) {
 			const details = result.details as BashToolDetails | undefined;
 			const output = result.content[0]?.type === "text" ? result.content[0].text : "";
 			const nonEmpty = output.split("\n").filter((line) => line.trim().length > 0);
+			const semantic = bashSemanticDisplayEnabled() ? classifyBashCommandForDisplay(ctx.args?.command ?? "") : null;
 			if (isPartial) {
 				setupBlinkTimer(ctx);
-				return makeText(ctx.lastComponent, withBranch(theme.fg("warning", `Running... (${nonEmpty.length} lines)`), theme));
+				const running = semantic?.kind === "read" ? "Reading" : "Running";
+				return makeText(ctx.lastComponent, withBranch(theme.fg("warning", `${running}... (${nonEmpty.length} lines)`), theme));
 			}
 			clearBlinkTimer(ctx);
 			setToolStatus(ctx, ctx.isError ? "error" : "success");
@@ -4299,8 +4744,9 @@ export default function (pi: ExtensionAPI) {
 			const isError = ctx.isError || (exitCode !== null && exitCode !== 0);
 			let text = isError
 				? theme.fg("error", exitCode !== null ? `Exit ${exitCode}` : "Failed")
-				: theme.fg("success", "Done");
-			text += theme.fg("muted", ` (${nonEmpty.length} lines)`);
+				: semantic?.kind === "read"
+					? `${theme.fg("success", "Read")} ${theme.fg("muted", `${nonEmpty.length} line${nonEmpty.length === 1 ? "" : "s"}`)}`
+					: `${theme.fg("success", "Done")}${theme.fg("muted", ` (${nonEmpty.length} lines)`)}`;
 			if (details?.truncation?.truncated) text += theme.fg("warning", " [truncated]");
 
 			const mode = bashOutputMode();
@@ -4311,7 +4757,10 @@ export default function (pi: ExtensionAPI) {
 				return makeText(ctx.lastComponent, withBranch(`${text}\n${preview}`, theme));
 			}
 
-			if (!expanded && nonEmpty.length > 0) return makeText(ctx.lastComponent, withBranch(`${text}${theme.fg("muted", " • Ctrl+O to expand")}`, theme));
+			if (!expanded && nonEmpty.length > 0) {
+				const hint = semantic?.suppressCollapsedHint ? "" : theme.fg("muted", " • Ctrl+O to expand");
+				return makeText(ctx.lastComponent, withBranch(`${text}${hint}`, theme));
+			}
 			if (!expanded) return makeText(ctx.lastComponent, withBranch(text, theme));
 			text += `\n${buildPreviewText(nonEmpty.map((line) => theme.fg(isError ? "error" : "dim", line)), true, theme, bashCollapsedLimit())}`;
 			return makeText(ctx.lastComponent, withBranch(text, theme));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@owlburtoe/pi-cc-tools",
-  "version": "1.1.0",
+  "version": "1.1.2",
   "description": "Claude Code-style tool rows for pi with Ctrl+O image previews and consistent built-in, MCP, and custom tool rendering",
   "keywords": [
     "pi-package",
@@ -32,6 +32,7 @@
   ],
   "scripts": {
     "bench:tools": "bun scripts/benchmark-tools.ts",
+    "test:bash-display": "bun scripts/test-bash-display.ts",
     "test:message-chrome": "bun scripts/test-message-chrome.ts"
   },
   "dependencies": {