refactor: consolidate duplicate JSX in ternary expressions

Simplifies ternary expressions by extracting duplicate JSX elements and
consolidating conditional logic.

Author: brandonkachen

cli/src/chat.tsx

@@ -1207,21 +1207,21 @@ export const App = ({
             }}
           >
             <text wrap={false}>
-              {hasStatus ? statusIndicatorNode : null}
-              {hasStatus && (exitWarning || shouldShowQueuePreview) ? '  ' : ''}
-              {exitWarning ? (
+              {hasStatus && statusIndicatorNode}
+              {hasStatus && (exitWarning || shouldShowQueuePreview) && '  '}
+              {exitWarning && (
                 <span fg={theme.statusSecondary}>{exitWarning}</span>
-              ) : null}
-              {exitWarning && shouldShowQueuePreview ? '  ' : ''}
-              {shouldShowQueuePreview ? (
+              )}
+              {exitWarning && shouldShowQueuePreview && '  '}
+              {shouldShowQueuePreview && (
                 <span fg={theme.statusSecondary} bg={theme.inputFocusedBg}>
                   {' '}
                   {formatQueuedPreview(
                     queuedMessages,
                     Math.max(30, terminalWidth - 25),
                   )}{' '}
                 </span>
-              ) : null}
+              )}
             </text>
           </box>
         )}

cli/src/components/branch-item.tsx

@@ -170,24 +170,14 @@ export const BranchItem = ({
           </text>
         </box>
         <box style={{ flexShrink: 1, marginBottom: 0 }}>
-          {isStreaming && isCollapsed && streamingPreview && (
+          {isCollapsed && (isStreaming ? streamingPreview : finishedPreview) && (
             <text
-              key="streaming-preview"
+              key={isStreaming ? 'streaming-preview' : 'finished-preview'}
               wrap
-              fg={theme.agentText}
+              fg={isStreaming ? theme.agentText : theme.agentResponseCount}
               attributes={TextAttributes.ITALIC}
             >
-              {streamingPreview}
-            </text>
-          )}
-          {!isStreaming && isCollapsed && finishedPreview && (
-            <text
-              key="finished-preview"
-              wrap
-              fg={theme.agentResponseCount}
-              attributes={TextAttributes.ITALIC}
-            >
-              {finishedPreview}
+              {isStreaming ? streamingPreview : finishedPreview}
             </text>
           )}
           {!isCollapsed && (

cli/src/components/terminal-link.tsx

@@ -55,15 +55,6 @@ export const TerminalLink: React.FC<TerminalLinkProps> = ({
     return <span fg={displayColor}>{text}</span>
   }
 
-  const renderLine = (line: string, index: number) => {
-    const coloredText = <span fg={displayColor}>{line}</span>
-    return (
-      <text key={index} wrap={lineWrap}>
-        {shouldUnderline ? <u>{coloredText}</u> : coloredText}
-      </text>
-    )
-  }
-
   return (
     <box
       style={{
@@ -77,7 +68,14 @@ export const TerminalLink: React.FC<TerminalLinkProps> = ({
       onMouseOut={() => setIsHovered(false)}
       onMouseDown={handleActivate}
     >
-      {displayLines.map(renderLine)}
+      {displayLines.map((line: string, index: number) => {
+        const coloredText = <span fg={displayColor}>{line}</span>
+        return (
+          <text key={index} wrap={lineWrap}>
+            {shouldUnderline ? <u>{coloredText}</u> : coloredText}
+          </text>
+        )
+      })}
     </box>
   )
 }