From 88bebef3a99b23abeef7b7beb8329dd20333640f Mon Sep 17 00:00:00 2001 From: Michael Bleigh Date: Fri, 10 Apr 2026 20:50:04 -0700 Subject: [PATCH] fix(ui): hide summary in header when displayed in box --- packages/cli/src/ui/components/messages/ToolGroupMessage.tsx | 4 +++- packages/cli/src/ui/components/messages/ToolMessage.tsx | 1 + packages/cli/src/ui/components/messages/ToolShared.tsx | 4 +++- packages/core/src/agent/tool-display-utils.ts | 2 +- 4 files changed, 8 insertions(+), 3 deletions(-) diff --git a/packages/cli/src/ui/components/messages/ToolGroupMessage.tsx b/packages/cli/src/ui/components/messages/ToolGroupMessage.tsx index 3a37f3ff5e..cfec822f9d 100644 --- a/packages/cli/src/ui/components/messages/ToolGroupMessage.tsx +++ b/packages/cli/src/ui/components/messages/ToolGroupMessage.tsx @@ -61,7 +61,9 @@ export const isCompactTool = ( tool: IndividualToolCallDisplay, isCompactModeEnabled: boolean, ): boolean => { - const hasCompactOutputSupport = COMPACT_OUTPUT_ALLOWLIST.has(tool.name); + const hasCompactOutputSupport = COMPACT_OUTPUT_ALLOWLIST.has( + tool.originalRequestName || tool.name, + ); const displayStatus = mapCoreStatusToDisplayStatus(tool.status); return ( isCompactModeEnabled && diff --git a/packages/cli/src/ui/components/messages/ToolMessage.tsx b/packages/cli/src/ui/components/messages/ToolMessage.tsx index d70c470408..25cf476512 100644 --- a/packages/cli/src/ui/components/messages/ToolMessage.tsx +++ b/packages/cli/src/ui/components/messages/ToolMessage.tsx @@ -116,6 +116,7 @@ export const ToolMessage: React.FC = ({ emphasis={emphasis} progressMessage={progressMessage} originalRequestName={originalRequestName} + hideSummary={!display?.result && !!display?.resultSummary} /> = ({ @@ -206,6 +207,7 @@ export const ToolInfo: React.FC = ({ display, progressMessage: _progressMessage, originalRequestName, + hideSummary, }) => { const status = mapCoreStatusToDisplayStatus(coreStatus); const nameColor = React.useMemo(() => { @@ -228,7 +230,7 @@ export const ToolInfo: React.FC = ({ const displayName = display?.name || name; const displayDescription = display?.description || description; - const displaySummary = display?.resultSummary; + const displaySummary = hideSummary ? undefined : display?.resultSummary; return ( diff --git a/packages/core/src/agent/tool-display-utils.ts b/packages/core/src/agent/tool-display-utils.ts index ed0e6f6b76..53ca5eaf75 100644 --- a/packages/core/src/agent/tool-display-utils.ts +++ b/packages/core/src/agent/tool-display-utils.ts @@ -35,7 +35,7 @@ export function populateToolDisplay({ description: invocation?.getDescription?.(), }; - if (resultDisplay) { + if (resultDisplay !== undefined && display.result === undefined) { display.result = toolResultDisplayToDisplayContent(resultDisplay); }