feat(pptx): replace viewer with custom preview (#4536)

* feat(pptx): replace viewer with custom preview

* address comments

* address comments

* address bugbot

* address comment

* rm bun lock

* resolve conflict

* update credit comment

* address comments

* rm bun lock

* resolve conflict

* make previews consistent for zoom and scroll

* zoom fix

Author: icecrasher321

apps/sim/app/(auth)/signup/signup-form.tsx

@@ -271,7 +271,7 @@ function SignupFormContent({ githubAvailable, googleAvailable, isProduction }: S
             ...(token ? { 'x-captcha-response': token } : {}),
           },
           onError: (ctx) => {
-            logger.error('Signup error:', ctx.error)
+            logger.warn('Signup error:', ctx.error)
             const errorMessage: string[] = ['Failed to create account']
 
             let errorCode = 'unknown'

apps/sim/app/api/files/parse/route.test.ts

@@ -69,6 +69,7 @@ vi.mock('@/app/api/files/authorization', () => ({
 vi.mock('@/lib/uploads', () => ({
   getStorageProvider: mockGetStorageProvider,
   isUsingCloudStorage: mockIsUsingCloudStorage,
+  StorageService: storageServiceMock,
 }))
 
 vi.mock('@/lib/file-parsers', () => ({
@@ -172,6 +173,7 @@ describe('File Parse API Route', () => {
 
     permissionsMockFns.mockGetUserEntityPermissions.mockResolvedValue({ canView: true })
     storageServiceMockFns.mockHasCloudStorage.mockReturnValue(true)
+    storageServiceMockFns.mockDownloadFile.mockResolvedValue(Buffer.from('test file content'))
     mockIsSupportedFileType.mockReturnValue(true)
     mockParseFile.mockResolvedValue({
       content: 'parsed content',
@@ -245,6 +247,48 @@ describe('File Parse API Route', () => {
     }
   })
 
+  it('should keep known binary extensions as binary even when the bytes are valid UTF-8', async () => {
+    setupFileApiMocks({
+      cloudEnabled: true,
+      storageProvider: 's3',
+      authenticated: true,
+    })
+    mockIsSupportedFileType.mockReturnValue(false)
+    storageServiceMockFns.mockDownloadFile.mockResolvedValue(Buffer.from('valid utf8 bytes'))
+
+    const req = createMockRequest('POST', {
+      filePath: '/api/files/serve/execution/workspace-1/workflow-1/execution-1/image.png',
+    })
+
+    const response = await POST(req)
+    const data = await response.json()
+
+    expect(response.status).toBe(200)
+    expect(data.success).toBe(true)
+    expect(data.output.content).toBe('[Binary PNG file - 16 bytes]')
+  })
+
+  it('should parse unknown extensions as text when the bytes look like UTF-8 text', async () => {
+    setupFileApiMocks({
+      cloudEnabled: true,
+      storageProvider: 's3',
+      authenticated: true,
+    })
+    mockIsSupportedFileType.mockReturnValue(false)
+    storageServiceMockFns.mockDownloadFile.mockResolvedValue(Buffer.from('plain text content'))
+
+    const req = createMockRequest('POST', {
+      filePath: '/api/files/serve/execution/workspace-1/workflow-1/execution-1/readme.customtext',
+    })
+
+    const response = await POST(req)
+    const data = await response.json()
+
+    expect(response.status).toBe(200)
+    expect(data.success).toBe(true)
+    expect(data.output.content).toBe('plain text content')
+  })
+
   it('should handle multiple files', async () => {
     setupFileApiMocks({
       cloudEnabled: false,

apps/sim/app/api/files/parse/route.ts

@@ -1,4 +1,4 @@
-import { Buffer } from 'buffer'
+import { Buffer, isUtf8 } from 'buffer'
 import { createHash } from 'crypto'
 import fsPromises, { readFile } from 'fs/promises'
 import path from 'path'
@@ -39,6 +39,11 @@ const logger = createLogger('FilesParseAPI')
 
 const MAX_DOWNLOAD_SIZE_BYTES = 100 * 1024 * 1024 // 100 MB
 const DOWNLOAD_TIMEOUT_MS = 30000 // 30 seconds
+const BINARY_EXTENSIONS = new Set<string>(binaryExtensionsList)
+
+function isLikelyTextBuffer(fileBuffer: Buffer): boolean {
+  return isUtf8(fileBuffer) && !fileBuffer.includes(0)
+}
 
 interface ExecutionContext {
   workspaceId: string
@@ -863,10 +868,11 @@ function handleGenericBuffer(
   extension: string,
   fileType?: string
 ): ParseResult {
-  const isBinary = binaryExtensionsList.includes(extension)
-  const content = isBinary
-    ? `[Binary ${extension.toUpperCase()} file - ${fileBuffer.length} bytes]`
-    : fileBuffer.toString('utf-8')
+  const normalizedExtension = extension.toLowerCase()
+  const content =
+    !BINARY_EXTENSIONS.has(normalizedExtension) && isLikelyTextBuffer(fileBuffer)
+      ? fileBuffer.toString('utf-8')
+      : `[Binary ${normalizedExtension.toUpperCase()} file - ${fileBuffer.length} bytes]`
 
   return {
     success: true,