From 4e46d98156615a3a77f70e3330cebdbd08a98748 Mon Sep 17 00:00:00 2001
From: Kit Langton <kit.langton@gmail.com>
Date: Fri, 6 Mar 2026 16:15:01 -0500
Subject: [PATCH] fix(ui): shell-rolling-results formatting cleanup

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 .../src/components/shell-rolling-results.tsx  | 33 ++++++++-----------
 1 file changed, 13 insertions(+), 20 deletions(-)

diff --git a/packages/ui/src/components/shell-rolling-results.tsx b/packages/ui/src/components/shell-rolling-results.tsx
index bd37e3a6e3..19f98c6282 100644
--- a/packages/ui/src/components/shell-rolling-results.tsx
+++ b/packages/ui/src/components/shell-rolling-results.tsx
@@ -8,14 +8,7 @@ import { Icon } from "./icon"
 import { IconButton } from "./icon-button"
 import { TextShimmer } from "./text-shimmer"
 import { Tooltip } from "./tooltip"
-import {
-  animate,
-  clearFadeStyles,
-  clearMaskStyles,
-  FAST_SPRING,
-  GROW_SPRING,
-  WIPE_MASK,
-} from "./motion"
+import { animate, clearFadeStyles, clearMaskStyles, FAST_SPRING, GROW_SPRING, WIPE_MASK } from "./motion"
 import { useSpring } from "./motion-spring"
 import { busy, createThrottledValue, useToolFade } from "./tool-utils"
 
@@ -78,14 +71,16 @@ export function ShellRollingResults(props: { part: ToolPart; animate?: boolean }
   // a 2s timer sets it false to trigger the collapse. This avoids the flash caused by
   // holdOpen being set in a late-running effect.
   const [autoOpen, setAutoOpen] = createSignal(pending())
-  createEffect(on(pending, (isPending, wasPending) => {
-    if (isPending) {
-      setAutoOpen(true)
-    } else if (wasPending && !userToggled()) {
-      const timer = setTimeout(() => setAutoOpen(false), 2000)
-      onCleanup(() => clearTimeout(timer))
-    }
-  }))
+  createEffect(
+    on(pending, (isPending, wasPending) => {
+      if (isPending) {
+        setAutoOpen(true)
+      } else if (wasPending && !userToggled()) {
+        const timer = setTimeout(() => setAutoOpen(false), 2000)
+        onCleanup(() => clearTimeout(timer))
+      }
+    }),
+  )
   const effectiveOpen = createMemo(() => {
     if (pending()) return true
     if (userToggled()) return userOpen()
@@ -164,9 +159,7 @@ export function ShellRollingResults(props: { part: ToolPart; animate?: boolean }
           <span data-slot="shell-rolling-title">
             <TextShimmer text={i18n.t("ui.tool.shell")} active={pending()} />
           </span>
-          <Show when={subtitle()}>
-            {(text) => <ShellRollingSubtitle text={text()} animate={props.animate} />}
-          </Show>
+          <Show when={subtitle()}>{(text) => <ShellRollingSubtitle text={text()} animate={props.animate} />}</Show>
           <Show when={!pending()}>
             <span data-slot="shell-rolling-actions">
               <Tooltip
@@ -196,7 +189,7 @@ export function ShellRollingResults(props: { part: ToolPart; animate?: boolean }
         items={rows()}
         fixed={fixed()}
         fixedHeight={22}
-        rows={5}
+        rows={userToggled() && userOpen() ? 10 : 5}
         rowHeight={22}
         rowGap={0}
         open={effectiveOpen()}