From d96bdb012640fcf3fcdb93355049924566c9fd81 Mon Sep 17 00:00:00 2001
From: Naiyuan Qing <145280634+NevilleQingNY@users.noreply.github.com>
Date: Thu, 26 Feb 2026 16:18:29 +0800
Subject: [PATCH] fix(ui): prevent empty div flicker between tool transitions

The text wrapper div was rendering when `isStreaming` was true but
`text` was empty, causing a brief empty flex container to appear
between tool executions. Changed condition from `(text || isStreaming)`
to `text` only, so the div only renders when there's actual content.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
---
 packages/ui/src/components/message-list.tsx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/packages/ui/src/components/message-list.tsx b/packages/ui/src/components/message-list.tsx
index 84e712a5..bc53818b 100644
--- a/packages/ui/src/components/message-list.tsx
+++ b/packages/ui/src/components/message-list.tsx
@@ -119,8 +119,8 @@ export const MessageList = memo(function MessageList({
               <ThinkingItem thinking={thinking} isStreaming={isStreaming} />
             )}
 
-            {/* Render text content (if any) */}
-            {(text || isStreaming) && (
+            {/* Render text content (if any) - require actual text, not just streaming state */}
+            {text && (
               <div
                 className={cn(
                   "flex items-center gap-1.5",