fix: formatting and imports

Author: Patel230

apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/chat/chat.tsx

@@ -2,7 +2,6 @@
 
 import { type KeyboardEvent, useCallback, useEffect, useMemo, useRef, useState } from 'react'
 import { createLogger } from '@sim/logger'
-import { formatOutputForWorkflow } from '@/lib/core/utils/format-output'
 import {
   AlertCircle,
   ArrowDownToLine,
@@ -25,6 +24,7 @@ import {
 } from '@/components/emcn'
 import { useSession } from '@/lib/auth/auth-client'
 import { cn } from '@/lib/core/utils/cn'
+import { formatOutputForWorkflow } from '@/lib/core/utils/format-output'
 import {
   extractBlockIdFromOutputId,
   extractPathFromOutputId,

apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/chat/components/chat-message/chat-message.tsx

@@ -1,5 +1,4 @@
 import { useMemo } from 'react'
-
 import { formatOutputForChat } from '@/lib/core/utils/format-output'
 import { StreamingIndicator } from '@/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/copilot/components/copilot-message/components/smooth-streaming'
 
@@ -95,10 +94,7 @@ const WordWrap = ({ text }: { text: string }) => {
  * Renders a chat message with optional file attachments
  */
 export function ChatMessage({ message }: ChatMessageProps) {
-  const formattedContent = useMemo(
-    () => formatOutputForChat(message.content),
-    [message.content]
-  )
+  const formattedContent = useMemo(() => formatOutputForChat(message.content), [message.content])
 
   const handleAttachmentClick = (attachment: ChatAttachment) => {
     const validDataUrl = attachment.dataUrl?.trim()

apps/sim/lib/core/utils/format-output.test.ts

@@ -1,11 +1,11 @@
 import { describe, expect, it } from 'vitest'
 import {
-  formatOutputForDisplay,
   formatOutputForChat,
+  formatOutputForDisplay,
   formatOutputForWorkflow,
   formatOutputRaw,
   formatOutputSafe,
-  isOutputSafe
+  isOutputSafe,
 } from './format-output'
 
 describe('format-output utilities', () => {
@@ -37,32 +37,22 @@ describe('format-output utilities', () => {
         data: {
           response: {
             message: {
-              content: 'Deep text'
-            }
-          }
-        }
+              content: 'Deep text',
+            },
+          },
+        },
       }
       expect(formatOutputForDisplay(nested)).toBe('Deep text')
     })
 
     // Arrays
     it('handles arrays of text objects', () => {
-      const arr = [
-        { text: 'Line 1' },
-        { text: 'Line 2' },
-        { content: 'Line 3' }
-      ]
+      const arr = [{ text: 'Line 1' }, { text: 'Line 2' }, { content: 'Line 3' }]
       expect(formatOutputForDisplay(arr)).toBe('Line 1 Line 2 Line 3')
     })
 
     it('handles mixed arrays', () => {
-      const mixed = [
-        'String',
-        { text: 'Object text' },
-        123,
-        null,
-        { message: 'Message text' }
-      ]
+      const mixed = ['String', { text: 'Object text' }, 123, null, { message: 'Message text' }]
       expect(formatOutputForDisplay(mixed)).toBe('String Object text 123 Message text')
     })
 
@@ -103,7 +93,7 @@ describe('format-output utilities', () => {
       const buffer = Buffer.from('Hello Buffer')
       expect(formatOutputForDisplay(buffer)).toBe('Hello Buffer')
 
-      const binaryBuffer = Buffer.from([0xFF, 0xFE, 0x00, 0x01])
+      const binaryBuffer = Buffer.from([0xff, 0xfe, 0x00, 0x01])
       expect(formatOutputForDisplay(binaryBuffer)).toBe('[Binary Data]')
     })
 
@@ -118,10 +108,12 @@ describe('format-output utilities', () => {
     // Whitespace handling
     it('preserves whitespace when requested', () => {
       const spaced = 'Line 1\n\nLine 2\t\tTabbed'
-      expect(formatOutputForDisplay(spaced, { preserveWhitespace: true }))
-        .toBe('Line 1\n\nLine 2\t\tTabbed')
-      expect(formatOutputForDisplay(spaced, { preserveWhitespace: false }))
-        .toBe('Line 1 Line 2 Tabbed')
+      expect(formatOutputForDisplay(spaced, { preserveWhitespace: true })).toBe(
+        'Line 1\n\nLine 2\t\tTabbed'
+      )
+      expect(formatOutputForDisplay(spaced, { preserveWhitespace: false })).toBe(
+        'Line 1 Line 2 Tabbed'
+      )
     })
 
     // Mode-specific formatting
@@ -148,7 +140,7 @@ describe('format-output utilities', () => {
       const customObj = {
         toString() {
           return 'Custom String'
-        }
+        },
       }
       expect(formatOutputForDisplay(customObj)).toBe('Custom String')
     })
@@ -157,7 +149,7 @@ describe('format-output utilities', () => {
       const obj = {
         func: () => console.log('test'),
         undef: undefined,
-        sym: Symbol('test')
+        sym: Symbol('test'),
       }
       const result = formatOutputForDisplay(obj, { mode: 'raw' })
       expect(result).toContain('[Function]')
@@ -214,11 +206,14 @@ describe('format-output utilities', () => {
   describe('error handling', () => {
     it('handles errors gracefully', () => {
       // Create object that throws on property access
-      const evil = new Proxy({}, {
-        get() {
-          throw new Error('Evil object!')
+      const evil = new Proxy(
+        {},
+        {
+          get() {
+            throw new Error('Evil object!')
+          },
         }
-      })
+      )
 
       const result = formatOutputForDisplay(evil)
       expect(result).toContain('[')
@@ -241,29 +236,33 @@ describe('format-output utilities', () => {
   describe('real-world LLM outputs', () => {
     it('handles OpenAI format', () => {
       const openAIResponse = {
-        choices: [{
-          message: {
-            content: 'AI response here'
-          }
-        }]
+        choices: [
+          {
+            message: {
+              content: 'AI response here',
+            },
+          },
+        ],
       }
       expect(formatOutputForDisplay(openAIResponse)).toBe('AI response here')
     })
 
     it('handles Anthropic format', () => {
       const anthropicResponse = {
-        content: [{
-          text: 'Claude response'
-        }]
+        content: [
+          {
+            text: 'Claude response',
+          },
+        ],
       }
       expect(formatOutputForDisplay(anthropicResponse)).toBe('Claude response')
     })
 
     it('handles streaming chunks', () => {
       const chunk = {
         delta: {
-          content: 'Streaming text'
-        }
+          content: 'Streaming text',
+        },
       }
       expect(formatOutputForDisplay(chunk)).toBe('Streaming text')
     })
@@ -272,11 +271,11 @@ describe('format-output utilities', () => {
       const toolOutput = {
         result: {
           data: {
-            output: 'Tool execution result'
-          }
-        }
+            output: 'Tool execution result',
+          },
+        },
       }
       expect(formatOutputForDisplay(toolOutput)).toBe('Tool execution result')
     })
   })
-})
+})

apps/sim/lib/core/utils/format-output.ts

@@ -3,7 +3,18 @@ import { createLogger } from '@sim/logger'
 const logger = createLogger('FormatOutput')
 
 // Type definitions for better type safety
-type TextFieldName = 'text' | 'content' | 'message' | 'body' | 'value' | 'result' | 'output' | 'response' | 'answer' | 'reply' | 'data'
+type TextFieldName =
+  | 'text'
+  | 'content'
+  | 'message'
+  | 'body'
+  | 'value'
+  | 'result'
+  | 'output'
+  | 'response'
+  | 'answer'
+  | 'reply'
+  | 'data'
 type FormatMode = 'chat' | 'workflow' | 'raw'
 
 interface FormatOptions {
@@ -30,8 +41,17 @@ interface TextExtractable {
 
 // Constants
 const TEXT_FIELDS: readonly TextFieldName[] = [
-  'text', 'content', 'message', 'body', 'value', 'result',
-  'output', 'response', 'answer', 'reply', 'data',
+  'text',
+  'content',
+  'message',
+  'body',
+  'value',
+  'result',
+  'output',
+  'response',
+  'answer',
+  'reply',
+  'data',
 ] as const
 
 const MAX_STRING_LENGTH = 50000
@@ -76,9 +96,7 @@ function extractText(value: unknown, depth = 0): string | null {
     }
 
     // For arrays, concatenate all text content
-    const texts = value
-      .map(item => extractText(item, depth + 1))
-      .filter(Boolean)
+    const texts = value.map((item) => extractText(item, depth + 1)).filter(Boolean)
 
     return texts.length > 0 ? texts.join(' ') : null
   }
@@ -140,19 +158,23 @@ function extractText(value: unknown, depth = 0): string | null {
 function safeStringify(value: unknown): string {
   try {
     const seen = new WeakSet()
-    return JSON.stringify(value, (_, val) => {
-      if (val === undefined) return '[undefined]'
-      if (typeof val === 'function') return '[Function]'
-      if (typeof val === 'symbol') return '[Symbol]'
-      if (typeof val === 'bigint') return val.toString()
-
-      if (typeof val === 'object' && val !== null) {
-        if (seen.has(val)) return '[Circular]'
-        seen.add(val)
-      }
+    return JSON.stringify(
+      value,
+      (_, val) => {
+        if (val === undefined) return '[undefined]'
+        if (typeof val === 'function') return '[Function]'
+        if (typeof val === 'symbol') return '[Symbol]'
+        if (typeof val === 'bigint') return val.toString()
+
+        if (typeof val === 'object' && val !== null) {
+          if (seen.has(val)) return '[Circular]'
+          seen.add(val)
+        }
 
-      return val
-    }, 2)
+        return val
+      },
+      2
+    )
   } catch (error) {
     logger.debug('Stringify failed', { error })
     return '[Unable to display]'
@@ -163,10 +185,7 @@ function safeStringify(value: unknown): string {
  * Format output for display with smart text extraction
  * Optimized for performance and common use cases
  */
-export function formatOutputForDisplay(
-  output: unknown,
-  options: FormatOptions = {}
-): string {
+export function formatOutputForDisplay(output: unknown, options: FormatOptions = {}): string {
   const {
     mode = 'chat',
     maxLength = MAX_STRING_LENGTH,
@@ -285,4 +304,4 @@ export function formatOutputSafe(output: unknown): string {
   }
 
   return formatted
-}
+}