@esso0428/pi-subagents 0.15.14 → 0.15.15

package/src/ui/agent-widget.ts

@@ -14,7 +14,19 @@ import { getLifetimeTotal, getSessionContextPercent, type LifetimeUsage, type Se
 // ---- Constants ----
 
 /** Maximum number of rendered lines before overflow collapse kicks in. */
-const MAX_WIDGET_LINES = 12;
+export const MAX_WIDGET_LINES = 12;
+/** Keep a small editor/input area visible below the above-editor widget. */
+const MIN_EDITOR_LINES = 4;
+
+/**
+ * Derive the widget ceiling from the terminal height while retaining the
+ * historical MAX_WIDGET_LINES ceiling. Very short terminals may have no room
+ * for the widget; that is preferable to consuming the editor/input area.
+ */
+export function getWidgetLineBudget(rows: number): number {
+  if (!Number.isFinite(rows)) return MAX_WIDGET_LINES;
+  return Math.min(MAX_WIDGET_LINES, Math.max(0, Math.floor(rows) - MIN_EDITOR_LINES));
+}
 
 /** Braille spinner frames for animated running indicator. */
 export const SPINNER = ["⠋", "⠙", "⠹", "⠸", "⠼", "⠴", "⠦", "⠧", "⠇", "⠏"];
@@ -176,9 +188,9 @@ export function buildInvocationTags(
   return { modelName: invocation.modelName, tags };
 }
 
-/** Truncate text to a single line, max `len` chars. */
+/** Normalize and truncate text so it can never add physical widget rows. */
 function truncateLine(text: string, len = 60): string {
-  const line = text.split("\n").find(l => l.trim())?.trim() ?? "";
+  const line = text.replace(/[\r\n]+/g, " ").replace(/\s+/g, " ").trim();
   if (line.length <= len) return line;
   return line.slice(0, len) + "…";
 }
@@ -325,7 +337,7 @@ export class AgentWidget {
       statusText = theme.fg("dim", " stopped");
     } else if (a.status === "error") {
       icon = theme.fg("error", "✗");
-      const errMsg = a.error ? `: ${a.error.slice(0, 60)}` : "";
+      const errMsg = a.error ? `: ${truncateLine(a.error)}` : "";
       statusText = theme.fg("error", ` error${errMsg}`);
     } else {
       // aborted
@@ -340,7 +352,7 @@ export class AgentWidget {
     parts.push(duration);
 
     const modeTag = modeLabel ? ` ${theme.fg("dim", `(${modeLabel})`)}` : "";
-    return `${icon} ${theme.fg("dim", name)}${modeTag}  ${theme.fg("dim", a.description)} ${theme.fg("dim", "·")} ${theme.fg("dim", parts.join(" · "))}${statusText}`;
+    return `${icon} ${theme.fg("dim", name)}${modeTag}  ${theme.fg("dim", truncateLine(a.description))} ${theme.fg("dim", "·")} ${theme.fg("dim", parts.join(" · "))}${statusText}`;
   }
 
   /**
@@ -363,6 +375,8 @@ export class AgentWidget {
     if (!hasActive && !hasFinished) return [];
 
     const w = tui.terminal.columns;
+    const maxLines = getWidgetLineBudget(tui.terminal.rows);
+    if (maxLines === 0) return [];
     const truncate = (line: string) => truncateToWidth(line, w);
     const headingColor = hasActive ? "accent" : "dim";
     const headingIcon = hasActive ? "●" : "○";
@@ -399,7 +413,7 @@ export class AgentWidget {
       const activity = bg ? describeActivity(bg.activeTools, bg.responseText) : "thinking…";
 
       runningLines.push([
-        truncate(theme.fg("dim", "├─") + ` ${theme.fg("accent", frame)} ${theme.bold(name)}${modeTag}  ${theme.fg("muted", a.description)} ${theme.fg("dim", "·")} ${fgPreservingNestedStyles(theme, "dim", statsText)}`),
+        truncate(theme.fg("dim", "├─") + ` ${theme.fg("accent", frame)} ${theme.bold(name)}${modeTag}  ${theme.fg("muted", truncateLine(a.description))} ${theme.fg("dim", "·")} ${fgPreservingNestedStyles(theme, "dim", statsText)}`),
         truncate(theme.fg("dim", "│  ") + theme.fg("dim", `  ⎿  ${activity}`)),
       ]);
     }
@@ -408,12 +422,18 @@ export class AgentWidget {
       ? truncate(theme.fg("dim", "├─") + ` ${theme.fg("muted", "◦")} ${theme.fg("dim", `${queued.length} queued`)}`)
       : undefined;
 
-    // Assemble with overflow cap (heading + overflow indicator = 2 reserved lines).
-    const maxBody = MAX_WIDGET_LINES - 1; // heading takes 1 line
+    // Assemble with a responsive cap (heading + overflow indicator = 2
+    // reserved lines when content exceeds the available body budget).
+    const maxBody = maxLines - 1; // heading takes 1 line
     const totalBody = finishedLines.length + runningLines.length * 2 + (queuedLine ? 1 : 0);
 
     const lines: string[] = [truncate(theme.fg(headingColor, headingIcon) + " " + theme.fg(headingColor, "Agents"))];
 
+    if (maxLines === 1) {
+      // There is room only for the heading; do not consume the editor/input row.
+      return lines;
+    }
+
     if (totalBody <= maxBody) {
       // Everything fits — add all lines and fix up connectors for the last item.
       lines.push(...finishedLines);
@@ -439,6 +459,7 @@ export class AgentWidget {
       // Reserve 1 line for overflow indicator.
       let budget = maxBody - 1;
       let hiddenRunning = 0;
+      let hiddenQueued = 0;
       let hiddenFinished = 0;
 
       // 1. Running agents (2 lines each)
@@ -452,9 +473,13 @@ export class AgentWidget {
       }
 
       // 2. Queued line
-      if (queuedLine && budget >= 1) {
-        lines.push(queuedLine);
-        budget--;
+      if (queuedLine) {
+        if (budget >= 1) {
+          lines.push(queuedLine);
+          budget--;
+        } else {
+          hiddenQueued = 1;
+        }
       }
 
       // 3. Finished agents
@@ -470,9 +495,10 @@ export class AgentWidget {
       // Overflow summary
       const overflowParts: string[] = [];
       if (hiddenRunning > 0) overflowParts.push(`${hiddenRunning} running`);
+      if (hiddenQueued > 0) overflowParts.push(`${hiddenQueued} queued`);
       if (hiddenFinished > 0) overflowParts.push(`${hiddenFinished} finished`);
       const overflowText = overflowParts.join(", ");
-      lines.push(truncate(theme.fg("dim", "└─") + ` ${theme.fg("dim", `+${hiddenRunning + hiddenFinished} more (${overflowText})`)}`)
+      lines.push(truncate(theme.fg("dim", "└─") + ` ${theme.fg("dim", `+${hiddenRunning + hiddenQueued + hiddenFinished} more (${overflowText})`)}`)
       );
     }
 

package/src/ui/ccstyle/diff/diff-renderer.ts

@@ -2951,11 +2951,6 @@ export function renderWriteDiffResult(
 			return new Text(theme.fg("warning", "↳ diff unavailable: durable diff has no displayable lines"), 0, 0);
 		}
 	}
-	const persistedWasOverwrite = persistedParsed?.entries.some(
-		(entry) => entry.kind === "line" && entry.lineKind === "remove",
-	) ?? false;
-	const wasOverwrite = options.fileExistedBeforeWrite === true || persistedWasOverwrite;
-
 	if (typeof content !== "string" && !persistedParsed) {
 		if (!safeFallbackText.trim()) {
 			return new Text(theme.fg("muted", "↳ write completed"), 0, 0);
@@ -3036,7 +3031,7 @@ export function renderWriteDiffResult(
 					hovered,
 					clampDiffLinesToWidth(
 						renderWriteCollapsedHintLine(
-							wasOverwrite,
+							options.fileExistedBeforeWrite === true,
 							safeWidth,
 							theme,
 							hovered,
@@ -3048,7 +3043,7 @@ export function renderWriteDiffResult(
 			}
 
 			const header = renderWriteHeader(
-				wasOverwrite,
+				options.fileExistedBeforeWrite === true,
 				safeWidth,
 				theme,
 				options.headerLabel,

package/src/ui/ccstyle/tool-renderer.ts

@@ -31,11 +31,8 @@ function summaryForCall(toolName: string, args: any): { main: string; detail: st
       detail: details.length ? ` (${details.join(", ")})` : "",
     };
   }
-  const value =
-    args.path ?? args.file_path ?? args.command ?? args.query ?? args.pattern ?? args.url ?? args.name;
-  return value === undefined
-    ? { main: title, detail: "" }
-    : { main: `${title} ${oneLine(value)}`, detail: "" };
+  const value = args.path ?? args.file_path ?? args.command ?? args.query ?? args.pattern ?? args.url ?? args.name;
+  return value === undefined ? { main: title, detail: "" } : { main: `${title} ${oneLine(value)}`, detail: "" };
 }
 
 function renderCallComponent(toolName: string, args: any, theme: any, context: any): Component {
@@ -45,11 +42,9 @@ function renderCallComponent(toolName: string, args: any, theme: any, context: a
   return {
     render(width: number): string[] {
       const safeWidth = Math.max(1, Math.floor(Number.isFinite(width) ? width : 1));
-      const summary = summaryForCall(toolName, args);
       const prefix = ` ${theme.fg(color, icon)} `;
-      const title = theme.fg("toolTitle", summary.main);
-      const detail = summary.detail ? theme.fg("dim", summary.detail) : "";
-      return [truncateToWidth(prefix + title + detail, safeWidth, "")];
+      const summary = summaryForCall(toolName, args);
+      return [truncateToWidth(prefix + theme.fg("toolTitle", summary.main) + theme.fg("dim", summary.detail), safeWidth, "")];
     },
     invalidate() {},
   };
@@ -78,19 +73,11 @@ export function createViewerCcstyleTool(toolName: string, isHovered: () => boole
     },
     renderResult(result, options, theme, context) {
       settled = true;
-      return createViewerCcstyleResult(
-        name,
-        result,
-        { ...options, isHovered },
-        theme,
-        {
-          ...context,
-          isHovered,
-          diffConfig: {
-            ...DEFAULT_TOOL_DISPLAY_CONFIG,
-          },
-        },
-      );
+      return createViewerCcstyleResult(name, result, { ...options, isHovered }, theme, {
+        ...context,
+        isHovered,
+        diffConfig: { ...DEFAULT_TOOL_DISPLAY_CONFIG },
+      });
     },
   };
 }

package/src/ui/ccstyle/tool-result.ts

@@ -51,21 +51,16 @@ export function outputLineCount(result: any): number {
 }
 
 export function countLines(text: string): number {
-  return text.trim() ? text.trim().split("\n").filter((line) => line.trim().length > 0).length : 0;
+  return text.trim() ? text.trim().split("\n").filter((line) => line.trim()).length : 0;
 }
 
 /** Format arguments as stable, readable `key: value` rows for the Input rail. */
 export function formatToolInputArgs(args: unknown, maxChars = 8_000): string {
   if (args === undefined || args === null) return "";
-  if (typeof args !== "object") {
-    const text = sanitizeToolResultText(String(args));
-    return text.length > maxChars ? `${text.slice(0, maxChars)}…` : text;
+  if (typeof args !== "object" || Array.isArray(args)) {
+    const rendered = typeof args === "string" ? sanitizeToolResultText(args) : safeJson(args);
+    return rendered.length > maxChars ? `${rendered.slice(0, maxChars)}…` : rendered;
   }
-  if (Array.isArray(args)) {
-    const text = safeJson(args);
-    return text.length > maxChars ? `${text.slice(0, maxChars)}…` : text;
-  }
-
   const preferred = [
     "path", "file_path", "command", "query", "pattern", "url", "name",
     "message", "content", "old_string", "new_string",
@@ -81,32 +76,24 @@ export function formatToolInputArgs(args: unknown, maxChars = 8_000): string {
       return li - ri;
     });
   const lines: string[] = [];
-  for (const [rawKey, value] of entries) {
-    const key = sanitizeToolResultText(rawKey);
+  for (const [key, value] of entries) {
+    const safeKey = sanitizeToolResultText(key);
     if (typeof value === "string") {
       const safeValue = sanitizeToolResultText(value);
       if (safeValue.includes("\n")) {
-        lines.push(`${key}:`);
-        for (const line of safeValue.split("\n")) lines.push(`  ${line}`);
-      } else {
-        lines.push(`${key}: ${safeValue}`);
-      }
-      continue;
-    }
-    if (typeof value === "number" || typeof value === "boolean" || value === null) {
-      lines.push(`${key}: ${String(value)}`);
-      continue;
-    }
-    const json = safeJson(value);
-    if (json.includes("\n")) {
-      lines.push(`${key}:`);
-      for (const line of json.split("\n")) lines.push(`  ${line}`);
+        lines.push(`${safeKey}:`);
+        lines.push(...safeValue.split("\n").map((line) => `  ${line}`));
+      } else lines.push(`${safeKey}: ${safeValue}`);
     } else {
-      lines.push(`${key}: ${json}`);
+      const rendered = safeJson(value);
+      if (rendered.includes("\n")) {
+        lines.push(`${safeKey}:`);
+        lines.push(...rendered.split("\n").map((line) => `  ${line}`));
+      } else lines.push(`${safeKey}: ${rendered}`);
     }
   }
-  const text = lines.join("\n");
-  return text.length > maxChars ? `${text.slice(0, maxChars)}…` : text;
+  const rendered = lines.join("\n");
+  return rendered.length > maxChars ? `${rendered.slice(0, maxChars)}…` : rendered;
 }
 
 function hasExpandableDetail(text: string, args: unknown): boolean {
@@ -115,18 +102,8 @@ function hasExpandableDetail(text: string, args: unknown): boolean {
 
 export type ToolIoSection = "input" | "output";
 
-/**
- * Viewer-local Input/Output card. The rail and show-more rows intentionally
- * mirror ccstyle, but state is kept on this component so history remains
- * deterministic and no global terminal mouse handler is needed.
- */
-class ExpandedToolIoView implements Component {
-  private readonly isError: boolean;
-  private input: string;
-  private output: string;
-  private maxOutputLines: number;
-  private maxInputLines: number;
-  private theme: any;
+/** Viewer-local adaptation of pi-cc's structured Input/Output tool card. */
+export class ExpandedToolIoView implements Component {
   private cachedWidth: number | undefined;
   private cachedLines: string[] | undefined;
   private lineSections: Array<ToolIoSection | null> = [];
@@ -134,28 +111,16 @@ class ExpandedToolIoView implements Component {
   private showMoreRows: Partial<Record<ToolIoSection, number>> = {};
 
   constructor(
-    input: string,
-    output: string,
-    isError: boolean,
-    maxOutputLines = DEFAULT_TOOL_DISPLAY_CONFIG.expandedPreviewMaxLines,
-    maxInputLines = DEFAULT_TOOL_DISPLAY_CONFIG.expandedPreviewMaxLines,
-    theme?: any,
-  ) {
-    this.input = input;
-    this.output = output;
-    this.isError = isError;
-    this.maxOutputLines = Math.max(1, maxOutputLines);
-    this.maxInputLines = Math.max(1, maxInputLines);
-    this.theme = theme;
-  }
+    private input: string,
+    private output: string,
+    private readonly isError: boolean,
+    private readonly maxOutputLines = DEFAULT_TOOL_DISPLAY_CONFIG.expandedPreviewMaxLines,
+    private readonly maxInputLines = DEFAULT_TOOL_DISPLAY_CONFIG.expandedPreviewMaxLines,
+    private readonly theme?: any,
+  ) {}
 
-  getInputBody(): string {
-    return this.input;
-  }
-
-  getOutputBody(): string {
-    return this.output.trim() ? this.output : "Done";
-  }
+  getInputBody(): string { return this.input; }
+  getOutputBody(): string { return this.output.trim() ? this.output : "Done"; }
 
   setHoveredSection(section: ToolIoSection | null): void {
     if (this.hoveredSection === section) return;
@@ -163,47 +128,30 @@ class ExpandedToolIoView implements Component {
     this.invalidate();
   }
 
-  sectionAtLine(line: number): ToolIoSection | null {
-    return this.lineSections[line] ?? null;
-  }
-
-  showMoreLine(section: ToolIoSection): number | undefined {
-    return this.showMoreRows[section];
-  }
+  sectionAtLine(line: number): ToolIoSection | null { return this.lineSections[line] ?? null; }
+  showMoreLine(section: ToolIoSection): number | undefined { return this.showMoreRows[section]; }
 
   render(width: number): string[] {
     const safeWidth = toolViewportWidth(width);
     if (this.cachedLines && this.cachedWidth === safeWidth) return this.cachedLines;
-
-    const bodyColor = this.isError ? "error" : "toolOutput";
+    const theme = this.theme;
+    const color = (name: string, value: string): string => typeof theme?.fg === "function" ? theme.fg(name, value) : value;
+    const bold = (value: string): string => typeof theme?.bold === "function" ? theme.bold(value) : value;
     const rail = " │ ";
     const contentWidth = Math.max(1, safeWidth - visibleWidth(rail));
+    const bodyColor = this.isError ? "error" : "toolOutput";
     const rows: string[] = [];
     this.lineSections = [];
     this.showMoreRows = {};
-
     const push = (line: string, section: ToolIoSection | null) => {
       rows.push(truncateToWidth(line, safeWidth, ""));
       this.lineSections.push(section);
     };
-    // The theme is optional so the component remains safe in small/test hosts.
-    const theme = this.theme;
-    const color = (name: string, value: string): string => typeof theme?.fg === "function" ? theme.fg(name, value) : value;
-    const bold = (value: string): string => typeof theme?.bold === "function" ? theme.bold(value) : value;
-    const pushHeader = (corner: "├" | "└", title: string) => push(
-      color("dim", ` ${corner} `) + color("accent", bold(title)),
-      null,
-    );
-    const pushRail = (value: string, section: ToolIoSection, continued: boolean) => push(
-      color("dim", continued ? rail : "   ") + value,
-      section,
-    );
-    const pushBlank = () => push(color("dim", " │"), null);
-    const styleInput = (line: string): string => {
-      const match = line.match(/^([A-Za-z_][\w.-]*)(:\s*)(.*)$/);
-      return match
-        ? color("dim", `${match[1]}${match[2]}`) + color("muted", match[3] ?? "")
-        : color("muted", line);
+    const pushHeader = (corner: "├" | "└", title: string) => {
+      push(color("dim", ` ${corner} `) + color("accent", bold(title)), null);
+    };
+    const pushRail = (line: string, section: ToolIoSection, continued: boolean) => {
+      push(color("dim", continued ? rail : "   ") + line, section);
     };
     const pushBody = (body: string, section: ToolIoSection, limit: number, continued: boolean, input: boolean) => {
       const raw = body.replace(/\t/g, "   ").replace(/\n+$/, "");
@@ -213,20 +161,25 @@ class ExpandedToolIoView implements Component {
       }
       const sources = raw.split("\n");
       const wrapped: string[] = [];
+      const styleInput = (line: string): string => {
+        const match = line.match(/^([A-Za-z_][\w.-]*)(:\s*)(.*)$/);
+        return match
+          ? color("dim", `${match[1]}${match[2]}`) + color("muted", match[3] ?? "")
+          : color("muted", line);
+      };
       for (const source of sources) {
         const styled = input ? styleInput(source) : color(bodyColor, source);
-        const parts = wrapTextWithAnsi(styled, contentWidth);
-        wrapped.push(...(parts.length ? parts : [styled]));
+        const lines = wrapTextWithAnsi(styled, contentWidth);
+        wrapped.push(...(lines.length ? lines : [styled]));
       }
       const truncated = sources.length > limit || wrapped.length > limit;
       const visible = truncated ? wrapped.slice(0, limit) : wrapped;
       for (const line of visible) pushRail(line, section, continued);
-      if (truncated) {
-        const hidden = Math.max(1, wrapped.length - visible.length);
+      if (truncated && wrapped.length > visible.length) {
+        const hidden = wrapped.length - visible.length;
         const hintColor = this.hoveredSection === section ? "text" : "dim";
         pushRail(
-          color("dim", `… +${hidden} more lines`) +
-            color("dim", " •") + color(hintColor, ` ${showMoreHintText()}`),
+          color("dim", `… +${hidden} more lines`) + color("dim", " •") + color(hintColor, ` ${showMoreHintText()}`),
           section,
           continued,
         );
@@ -235,15 +188,13 @@ class ExpandedToolIoView implements Component {
     };
 
     const hasInput = this.input.trim().length > 0;
-    const output = this.getOutputBody();
     if (hasInput) {
       pushHeader("├", "Input");
-      pushBody(this.input, "input", this.maxInputLines, true, true);
-      pushBlank();
+      pushBody(this.input, "input", Math.max(1, this.maxInputLines), true, true);
+      push(color("dim", " │"), null);
     }
     pushHeader("└", "Output");
-    pushBody(output, "output", this.maxOutputLines, false, false);
-
+    pushBody(this.getOutputBody(), "output", Math.max(1, this.maxOutputLines), false, false);
     this.cachedWidth = safeWidth;
     this.cachedLines = rows;
     return rows;
@@ -253,7 +204,6 @@ class ExpandedToolIoView implements Component {
     this.cachedWidth = undefined;
     this.cachedLines = undefined;
   }
-
 }
 
 function unavailableComponent(reason: string, theme: any): Component {
@@ -265,34 +215,24 @@ function unavailableComponent(reason: string, theme: any): Component {
   };
 }
 
-function resultText(result: any): string {
-  return rawTextFromResult(result);
-}
+function resultText(result: any): string { return rawTextFromResult(result); }
 
 function persistedFromDetails(details: any): PersistedDiff | undefined {
   if (!details || typeof details !== "object") return undefined;
   const persisted: PersistedDiff = {};
   if (typeof details.diff === "string") persisted.diff = details.diff;
   if (typeof details.patch === "string") persisted.patch = details.patch;
-  if (typeof details.diffUnavailableReason === "string") {
-    persisted.diffUnavailableReason = details.diffUnavailableReason;
-  }
+  if (typeof details.diffUnavailableReason === "string") persisted.diffUnavailableReason = details.diffUnavailableReason;
   return Object.keys(persisted).length ? persisted : undefined;
 }
 
 function viewerConfig(context: any): ViewerDiffConfig {
   const candidate = context?.diffConfig;
-  if (!candidate || typeof candidate !== "object") return DEFAULT_TOOL_DISPLAY_CONFIG;
-  return {
-    ...DEFAULT_TOOL_DISPLAY_CONFIG,
-    ...candidate,
-  };
+  return candidate && typeof candidate === "object"
+    ? { ...DEFAULT_TOOL_DISPLAY_CONFIG, ...candidate }
+    : DEFAULT_TOOL_DISPLAY_CONFIG;
 }
 
-/**
- * Rich edit/write rendering for the conversation viewer. Durable details are
- * passed through before any live/ephemeral context so history is deterministic.
- */
 export function renderRichToolResult(
   toolName: string,
   result: any,
@@ -303,61 +243,35 @@ export function renderRichToolResult(
   if (options?.isPartial || options?.isError || context?.isError) return undefined;
   const expanded = options?.expanded === true || context?.expanded === true;
   const args = context?.args ?? options?.args;
-  const filePath = args?.file_path ?? args?.path;
   const details = result?.details;
   const persistedDiff = context?.persistedDiff ?? persistedFromDetails(details);
   const fileExistedBeforeWrite = typeof context?.fileExistedBeforeWrite === "boolean"
     ? context.fileExistedBeforeWrite
-    : typeof details?.fileExistedBeforeWrite === "boolean"
-      ? details.fileExistedBeforeWrite
-      : undefined;
+    : typeof details?.fileExistedBeforeWrite === "boolean" ? details.fileExistedBeforeWrite : undefined;
 
   if (toolName === "edit") {
-    return renderEditDiffResult(
-      details,
-      { expanded, filePath, persistedDiff, isHovered: options?.isHovered, invalidate: context?.invalidate },
-      viewerConfig(context),
-      theme,
-      resultText(result),
-    );
+    return renderEditDiffResult(details, {
+      expanded, filePath: args?.file_path ?? args?.path, persistedDiff,
+      isHovered: options?.isHovered, invalidate: context?.invalidate,
+    }, viewerConfig(context), theme, resultText(result));
   }
   if (toolName !== "write") return undefined;
   if (persistedDiff?.diffUnavailableReason && !persistedDiff.diff && !persistedDiff.patch) {
     return unavailableComponent(persistedDiff.diffUnavailableReason, theme);
   }
-
   const content = typeof args?.content === "string" ? args.content : undefined;
-  if (!content && !persistedDiff?.diff && !persistedDiff?.patch) {
+  if (typeof content !== "string" && !persistedDiff?.diff && !persistedDiff?.patch) {
     return unavailableComponent("execution metadata is unavailable", theme);
   }
-  return renderWriteDiffResult(
-    content,
-    {
-      expanded,
-      filePath,
-      persistedDiff,
-      previousContent: context?.previousContent,
-      fileExistedBeforeWrite,
-      isHovered: options?.isHovered,
-      invalidate: context?.invalidate,
-    },
-    viewerConfig(context),
-    theme,
-    resultText(result),
-  );
-}
-
-function renderCallSummary(toolName: string, args: any, theme: any): string {
-  const title = toolName.replace(/([a-z0-9])([A-Z])/g, "$1 $2").replace(/[_-]+/g, " ");
-  const value = args?.path ?? args?.file_path ?? args?.command ?? args?.query ?? args?.name;
-  const summary = value === undefined ? title : `${title} ${oneLine(value)}`;
-  return theme.fg("toolTitle", summary);
-}
-
-function settledIcon(context: any): string {
-  if (context?.isError) return "✗";
-  if (context?.isPartial || context?.executionStarted) return "●";
-  return "✓";
+  return renderWriteDiffResult(content, {
+    expanded,
+    filePath: args?.file_path ?? args?.path,
+    persistedDiff,
+    previousContent: context?.previousContent,
+    fileExistedBeforeWrite,
+    isHovered: options?.isHovered,
+    invalidate: context?.invalidate,
+  }, viewerConfig(context), theme, resultText(result));
 }
 
 function renderFallbackResult(result: any, options: any, theme: any, context: any): Component {
@@ -365,49 +279,31 @@ function renderFallbackResult(result: any, options: any, theme: any, context: an
   const expanded = options?.expanded === true || context?.expanded === true;
   const maxChars = Number.isFinite(context?.maxChars) ? Math.max(0, Math.floor(context.maxChars)) : 16_384;
   const text = textFromResult(result, expanded, maxChars);
-  if (expanded && (text.trim() || formatToolInputArgs(context?.args, maxChars).trim())) {
+  const input = formatToolInputArgs(context?.args, maxChars);
+  if (expanded && (text.trim() || input.trim())) {
     const maxLines = Number.isFinite(context?.diffConfig?.expandedPreviewMaxLines)
       ? Math.max(1, Math.floor(context.diffConfig.expandedPreviewMaxLines))
       : DEFAULT_TOOL_DISPLAY_CONFIG.expandedPreviewMaxLines;
-    return new ExpandedToolIoView(
-      formatToolInputArgs(context?.args, maxChars),
-      text,
-      Boolean(error),
-      maxLines,
-      maxLines,
-      theme,
-    );
+    return new ExpandedToolIoView(input, text, Boolean(error), maxLines, maxLines, theme);
   }
-  const lines = text ? text.split("\n") : [];
-  const count = lines.length;
+  const count = text ? text.split("\n").length : 0;
   const summary = error ? oneLine(text) || "Failed" : count ? `${count} ${count === 1 ? "line" : "lines"} returned` : "Done";
   const expandable = !expanded && hasExpandableDetail(text, context?.args);
   return {
     render(width: number): string[] {
       const hovered = options?.isHovered?.() ?? context?.isHovered?.() ?? false;
-      const clickLabel = showMoreHintText();
-      const hint = expandable ? ` • ${clickLabel}` : "";
-      const color = error ? "error" : "muted";
+      const hint = expandable ? ` • ${showMoreHintText()}` : "";
       const line = expandable && hovered
-        ? theme.fg(color, `   ↳ ${summary} • `) + theme.fg("text", clickLabel)
-        : theme.fg(color, `   ↳ ${summary}${hint}`);
+        ? theme.fg(error ? "error" : "muted", `   ↳ ${summary} • `) + theme.fg("text", showMoreHintText())
+        : theme.fg(error ? "error" : "muted", `   ↳ ${summary}${hint}`);
       return [truncateToWidth(line, toolViewportWidth(width), "")];
     },
     invalidate() {},
   };
 }
 
-export function createViewerCcstyleResult(
-  toolName: string,
-  result: any,
-  options: any,
-  theme: any,
-  context: any,
-): Component {
-  return (
-    renderRichToolResult(toolName, result, options, theme, context) ??
-    renderFallbackResult(result, options, theme, context)
-  );
+export function createViewerCcstyleResult(toolName: string, result: any, options: any, theme: any, context: any): Component {
+  return renderRichToolResult(toolName, result, options, theme, context) ?? renderFallbackResult(result, options, theme, context);
 }
 
-export { ExpandedToolIoView };
+export type { DisplayConfigInput };