fix(chat): remove special handling for non-streaming

[icecrasher321]

Summary

Remove special handling in client chat for non streaming block. Was appending additional output.

Type of Change

• Bug fix

Testing

Tested manually

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Review	Updated (UTC)
docs	Skipped		Jan 14, 2026 4:38am

[greptile-apps]

Greptile Summary

Removed special handling code in the streaming response processor that was appending additional selected outputs (like model, tokens, toolCalls) to streaming messages. This code was causing duplicate output to be displayed in the chat interface.

Key changes:

• Removed 29 lines from processStreamingResponse function that extracted and appended non-content outputs from logs after the final streaming event
• The code would skip 'content' outputs but append other selected outputs as formatted text to the streaming message
• Non-streaming responses already handle selected outputs correctly by creating separate messages (lines 582-596), so this special case handling was redundant for streaming

Impact:

• Fixes bug where additional outputs were being duplicated or incorrectly appended during streaming
• Simplifies the streaming logic by removing special case handling
• Maintains existing functionality for non-streaming responses

Confidence Score: 5/5

• This PR is safe to merge with minimal risk
• The change is a straightforward removal of problematic code that was causing duplicate outputs. The removed logic was special-case handling in streaming responses that was redundant with existing non-streaming output handling. No new logic is introduced, only deletion of 29 lines. The function still properly handles errors and finalizes message streams. The fix directly addresses the stated issue without introducing side effects.
• No files require special attention

Important Files Changed

Filename	Overview
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/chat/chat.tsx	Removed 29 lines of code that was appending additional selected outputs to streaming messages, fixing duplicate output issue

Sequence Diagram

sequenceDiagram
    participant User
    participant Chat as Chat Component
    participant Stream as processStreamingResponse
    participant Store as Chat Store

    User->>Chat: Send message
    Chat->>Chat: handleSendMessage()
    Chat->>Chat: handleRunWorkflow()
    
    alt Streaming Response
        Chat->>Stream: processStreamingResponse(stream, messageId)
        loop Read stream chunks
            Stream->>Stream: Parse SSE events
            alt Content chunk
                Stream->>Store: appendMessageContent(messageId, chunk)
            else Final event (success=false)
                Stream->>Store: appendMessageContent(messageId, error)
                Stream->>Store: finalizeMessageStream(messageId)
            else Final event (success=true)
                Note over Stream: Removed: extracting additional outputs
                Stream->>Store: finalizeMessageStream(messageId)
            end
        end
    else Non-Streaming Response
        Chat->>Chat: handleWorkflowResponse(result)
        loop For each selected output
            Chat->>Store: addMessage(output)
        end
    end

Loading