refactor: convert Maps to objects, remove verbose logs, and cleanup comments

- Convert all Map data structures to plain objects in use-send-message.ts
- Remove [SEND-MESSAGE], [SDK-ERROR], [SEND-MESSAGE-ERROR], [BATCH-RETRY], and [RETRY-PENDING] log prefixes
- Remove verbose logs like 'Creating AI message for response', 'Adding AI message to UI', 'Calling client.run()'
- Reduce comments in use-auth-state.ts
- Remove [CONNECTION-CHECK] logs from use-connection-status.ts
- Remove warning emoji from error messages
- Fix type error in use-network-status.test.ts

Author: brandonkachen

cli/src/components/__tests__/message-block.completion.test.tsx

@@ -28,7 +28,6 @@ const baseProps = {
   isAi: true,
   isLoading: false,
   timestamp: '12:00',
-  timestampNote: undefined,
   isComplete: false,
   completionTime: undefined,
   credits: undefined,

cli/src/components/__tests__/message-block.streaming.test.tsx

@@ -26,7 +26,6 @@ const baseProps = {
   isAi: true,
   isComplete: false,
   timestamp: '12:00',
-  timestampNote: undefined,
   completionTime: undefined,
   credits: undefined,
   textColor: theme.foreground,

cli/src/components/message-block.tsx

@@ -38,7 +38,6 @@ interface MessageBlockProps {
   isAi: boolean
   isLoading: boolean
   timestamp: string
-  timestampNote?: string
   isComplete?: boolean
   completionTime?: string
   credits?: number
@@ -65,7 +64,6 @@ export const MessageBlock = memo((props: MessageBlockProps): ReactNode => {
     isAi,
     isLoading,
     timestamp,
-    timestampNote,
     isComplete,
     completionTime,
     credits,
@@ -93,9 +91,7 @@ export const MessageBlock = memo((props: MessageBlockProps): ReactNode => {
   const selectedFeedbackCategory = useFeedbackStore(selectMessageFeedbackCategory(messageId))
 
   const resolvedTextColor = textColor ?? theme.foreground
-  const timestampDisplay = timestampNote
-    ? `[${timestamp} · ${timestampNote}]`
-    : `[${timestamp}]`
+  const timestampDisplay = `[${timestamp}]`
 
   const shouldShowLoadingTimer = isAi && isLoading && !isComplete
   const shouldShowCompletionFooter = isAi && isComplete

cli/src/components/message-with-agents.tsx

@@ -182,7 +182,6 @@ export const MessageWithAgents = memo(
                   isAi={isAi}
                   isLoading={isLoading}
                   timestamp={message.timestamp}
-                  timestampNote={message.timestampNote}
                   isComplete={message.isComplete}
                   completionTime={message.completionTime}
                   credits={message.credits}
@@ -224,7 +223,6 @@ export const MessageWithAgents = memo(
                 isAi={isAi}
                 isLoading={isLoading}
                 timestamp={message.timestamp}
-                timestampNote={message.timestampNote}
                 isComplete={message.isComplete}
                 completionTime={message.completionTime}
                 credits={message.credits}

cli/src/components/shimmer-text.tsx

@@ -190,18 +190,18 @@ export const ShimmerText = ({
   }, [colors, primaryColor, numChars, theme.info, theme.muted])
 
   const generateAttributes = (length: number): number[] => {
-    const attrs: number[] = []
+    const attributes: number[] = []
     for (let i = 0; i < length; i++) {
       const ratio = length <= 1 ? 0 : i / (length - 1)
       if (ratio < 0.23) {
-        attrs.push(TextAttributes.BOLD)
+        attributes.push(TextAttributes.BOLD)
       } else if (ratio < 0.69) {
-        attrs.push(TextAttributes.NONE)
+        attributes.push(TextAttributes.NONE)
       } else {
-        attrs.push(TextAttributes.DIM)
+        attributes.push(TextAttributes.DIM)
       }
     }
-    return attrs
+    return attributes
   }
 
   const generatedColors = useMemo(

cli/src/hooks/__tests__/use-network-status.test.ts

@@ -159,7 +159,7 @@ describe('useNetworkStatus', () => {
       ]
 
       sources.forEach((error) => {
-        expect(['auth', 'validation', 'unknown']).toContain(error?.source)
+        expect(['auth', 'validation', 'unknown']).toContain(error!.source)
       })
     })
   })

cli/src/hooks/use-auth-state.ts

@@ -41,15 +41,13 @@ export const useAuthState = ({
   const [user, setUser] = useState<User | null>(null)
   const [authError, setAuthError] = useState<AuthError | null>(null)
 
-  // Update authentication state when requireAuth changes
   useEffect(() => {
     if (requireAuth === null) {
       return
     }
     setIsAuthenticated(!requireAuth)
   }, [requireAuth])
 
-  // Update authentication state based on query results
   useEffect(() => {
     if (authQuery.isSuccess && authQuery.data) {
       setIsAuthenticated(true)
@@ -67,25 +65,19 @@ export const useAuthState = ({
     } else if (authQuery.isError) {
       const error = authQuery.error
 
-      // Check if this is a network error
       if (isNetworkError(error)) {
-        // For network errors, don't force login - allow access to the main app
         setAuthError({
           type: 'network',
           message: 'Unable to reach server. Please check your connection.',
         })
-        // Don't set isAuthenticated to false for network errors
-        // This allows the app to continue working offline
       } else if (isAuthenticationError(error)) {
-        // For authentication errors, require login
         setAuthError({
           type: 'authentication',
           message: 'Invalid API key. Please log in again.',
         })
         setIsAuthenticated(false)
         setUser(null)
       } else {
-        // Unknown error - treat as authentication failure for safety
         setAuthError({
           type: 'unknown',
           message: error?.message || 'Authentication check failed',
@@ -96,10 +88,8 @@ export const useAuthState = ({
     }
   }, [authQuery.isSuccess, authQuery.isError, authQuery.data, authQuery.error, user])
 
-  // Handle successful login
   const handleLoginSuccess = useCallback(
     (loggedInUser: User) => {
-      // Reset the SDK client to pick up new credentials
       resetCodebuffClient()
       resetChatStore()
       resetLoginState()
@@ -110,7 +100,6 @@ export const useAuthState = ({
     [resetChatStore, resetLoginState, setInputFocused],
   )
 
-  // Auto-focus input after authentication
   useEffect(() => {
     if (isAuthenticated !== true) return
 

cli/src/hooks/use-connection-status.ts

@@ -82,26 +82,9 @@ export const useConnectionStatus = (
         // Also handle first connection (for pending messages from previous session)
         if (connected && (previousConnected === false || previousConnected === null)) {
           const isInitialConnection = previousConnected === null
-          logger.info(
-            {
-              previousConnected,
-              connected,
-              isInitialConnection,
-              hasCallback: !!onReconnected
-            },
-            isInitialConnection
-              ? '[CONNECTION-CHECK] Initial connection established - checking for pending messages'
-              : '[CONNECTION-CHECK] Connection restored to backend - triggering reconnection handler'
-          )
           if (onReconnected) {
-            logger.info(
-              { isInitialConnection },
-              '[CONNECTION-CHECK] Calling onReconnected callback'
-            )
             onReconnected(isInitialConnection)
           }
-        } else if (!connected && previousConnected === true) {
-          logger.info('[CONNECTION-CHECK] Connection lost!')
         }
 
         previousConnected = connected