feat: update the Bash tool param to render on one line

Author: qorzj

src/tests/message-view.test.ts

@@ -3,6 +3,8 @@ import assert from "node:assert/strict";
 import { parseDiffPreview } from "../ui";
 import {
   buildThinkingSummary,
+  formatBashStatusParams,
+  formatToolStatusParams,
   renderMessageToStdout,
   getUpdatePlanPreviewLines,
   parseToolPayload,
@@ -60,6 +62,26 @@ test("MessageView shows full reasoning content in Normal/Raw mode", () => {
   );
 });
 
+test("formatBashStatusParams compacts multi-line commands and keeps the final description", () => {
+  assert.equal(
+    formatBashStatusParams('python3 -c "\nprint(1)\nprint(2)\n"  # Run inline script'),
+    'python3 -c " ... "  # Run inline script'
+  );
+});
+
+test("formatToolStatusParams preserves compacted Bash params but truncates other tools", () => {
+  assert.equal(
+    formatToolStatusParams({
+      name: "bash",
+      params: "cat <<'EOF'\nhello\nEOF  # Print heredoc",
+      ok: true,
+      metadata: null,
+    }),
+    "cat <<'EOF' ... EOF  # Print heredoc"
+  );
+  assert.equal(formatToolStatusParams({ name: "read", params: "first\nsecond", ok: true, metadata: null }), "first");
+});
+
 // --- renderMessageToStdout tests ---
 
 function makeSessionMessage(overrides: Partial<SessionMessage> & Pick<SessionMessage, "role">): SessionMessage {
@@ -139,6 +161,18 @@ test("renderMessageToStdout renders tool messages with resultMd output", () => {
   assert.ok(output.includes("line 1"));
 });
 
+test("renderMessageToStdout compacts multi-line Bash params", () => {
+  const payload = JSON.stringify({ name: "bash", ok: true });
+  const msg = makeSessionMessage({
+    role: "tool",
+    content: payload,
+    meta: { paramsMd: 'python3 -c "\nprint(1)\nprint(2)\n"  # Run inline script' },
+  });
+  const output = renderMessageToStdout(msg, RawMode.Raw);
+  assert.ok(output.includes('python3 -c " ... "  # Run inline script'));
+  assert.ok(!output.includes("print(1)"));
+});
+
 test("renderMessageToStdout renders UpdatePlan tool messages with Plan preview and resultMd", () => {
   const payload = JSON.stringify({
     name: "UpdatePlan",

src/ui/components/MessageView/utils.ts

@@ -54,10 +54,28 @@ export function buildThinkingSummary(content: string, messageParams: unknown | n
   return "";
 }
 
-/** Formats a tool's parameters for status display, preserving full bash commands but truncating others. */
+/** Formats multi-line Bash params as first line, a placeholder, and the final line. */
+export function formatBashStatusParams(params: string): string {
+  const value = params.trim();
+  if (!value) {
+    return "";
+  }
+
+  const lines = value.split(/\r?\n/);
+  if (lines.length <= 1) {
+    return value;
+  }
+
+  return `${lines[0]} ... ${lines[lines.length - 1].trimStart()}`;
+}
+
+/** Formats a tool's parameters for status display, compacting multi-line Bash commands and truncating others. */
 export function formatToolStatusParams(summary: ToolSummary): string {
+  if (summary.name.toLowerCase() === "bash") {
+    return formatBashStatusParams(summary.params);
+  }
   const params = firstNonEmptyLine(summary.params);
-  return summary.name.toLowerCase() === "bash" ? params : truncate(params, 120);
+  return truncate(params, 120);
 }
 
 /** Builds a structured summary (name, params, ok, metadata) from a tool session message. */
@@ -226,25 +244,13 @@ export function renderMessageToStdout(message: SessionMessage, mode: RawMode): s
   }
 
   if (message.role === "tool") {
-    const payload = parseToolPayload(message.content);
-    const metaFunctionName =
-      message.meta?.function && typeof (message.meta.function as { name?: unknown }).name === "string"
-        ? (message.meta.function as { name: string }).name
-        : null;
-    const name = payload.name || metaFunctionName || "tool";
-    const metaParams = typeof message.meta?.paramsMd === "string" ? message.meta.paramsMd.trim() : "";
-    const params = name.toLowerCase() === "bash" ? metaParams : truncate(metaParams, 120);
-    const statusLine = `${chalk("✧")} ${chalk(formatStatusName(name))}${params ? ` ${chalk(params)}` : ""}`;
+    const summary = buildToolSummary(message);
+    const params = formatToolStatusParams(summary);
+    const statusLine = `${chalk("✧")} ${chalk(formatStatusName(summary.name))}${params ? ` ${chalk(params)}` : ""}`;
 
     const metaResultMd = typeof message.meta?.resultMd === "string" ? message.meta.resultMd.trim() : "";
     const result = metaResultMd ? `\n${chalk.dim("  └ Result")}\n${metaResultMd}` : "";
 
-    const summary: ToolSummary = {
-      name,
-      params,
-      ok: payload.ok !== false,
-      metadata: payload.metadata,
-    };
     const planLines = getUpdatePlanPreviewLines(summary);
     if (planLines.length > 0) {
       const planText = planLines.map((line) => `  ${line}`).join("\n");