From 1ae94a9ef866bfcb1441c447b12bc285ba3023e0 Mon Sep 17 00:00:00 2001
From: Louis Knight-Webb <louis@bloop.ai>
Date: Thu, 23 Oct 2025 14:56:31 +0100
Subject: [PATCH] Prevent blank tool calls when output is multi-line
 (vibe-kanban) (#1077)

* Done! The Oracle tool will now display "Oracle" next to its icon instead of rendering blank. All tool calls now consistently show: icon + tool name + optional inline summary.

* Done! Removed the colon separator to fix the i18n CI error. The Oracle tool will now display cleanly as "Oracle [summary]" with the existing gap spacing providing visual separation.

* Done! Now it shows either the inline summary (if available) or the tool name, not both. Oracle will display "Oracle" when there's no summary.

* tsc
---
 .../DisplayConversationEntry.tsx                   | 14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/frontend/src/components/NormalizedConversation/DisplayConversationEntry.tsx b/frontend/src/components/NormalizedConversation/DisplayConversationEntry.tsx
index 63c72434..13cad66a 100644
--- a/frontend/src/components/NormalizedConversation/DisplayConversationEntry.tsx
+++ b/frontend/src/components/NormalizedConversation/DisplayConversationEntry.tsx
@@ -497,16 +497,12 @@ const ToolCallCard: React.FC<{
     <div className="inline-block w-full flex flex-col gap-4">
       <HeaderWrapper {...headerProps} className={headerClassName}>
         <span className=" min-w-0 flex items-center gap-1.5">
-          {entryType ? (
-            <span>
-              {getStatusIndicator(entryType)}
-              {getEntryIcon(entryType)}
-            </span>
-          ) : (
-            <span className="font-normal flex">{label}</span>
-          )}
-          {showInlineSummary && (
+          {entryType && getStatusIndicator(entryType)}
+          {entryType && getEntryIcon(entryType)}
+          {showInlineSummary ? (
             <span className="font-light">{inlineText}</span>
+          ) : (
+            <span className="font-normal">{label}</span>
           )}
         </span>
       </HeaderWrapper>