feat(pptx): replace viewer with custom preview

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.ts

@@ -1,9 +1,8 @@
-import { Buffer } from 'buffer'
+import { Buffer, isUtf8 } from 'buffer'
 import { createHash } from 'crypto'
 import fsPromises, { readFile } from 'fs/promises'
 import path from 'path'
 import { createLogger } from '@sim/logger'
-import binaryExtensionsList from 'binary-extensions'
 import { type NextRequest, NextResponse } from 'next/server'
 import { fileParseContract } from '@/lib/api/contracts/storage-transfer'
 import { getValidationErrorMessage, parseRequest } from '@/lib/api/server'
@@ -40,6 +39,10 @@ const logger = createLogger('FilesParseAPI')
 const MAX_DOWNLOAD_SIZE_BYTES = 100 * 1024 * 1024 // 100 MB
 const DOWNLOAD_TIMEOUT_MS = 30000 // 30 seconds
 
+function isLikelyTextBuffer(fileBuffer: Buffer): boolean {
+  return isUtf8(fileBuffer) && !fileBuffer.includes(0)
+}
+
 interface ExecutionContext {
   workspaceId: string
   workflowId: string
@@ -863,10 +866,9 @@ 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 content = isLikelyTextBuffer(fileBuffer)
+    ? fileBuffer.toString('utf-8')
+    : `[Binary ${extension.toUpperCase()} file - ${fileBuffer.length} bytes]`
 
   return {
     success: true,

apps/sim/app/workspace/[workspaceId]/files/components/file-viewer/pdf-viewer.tsx

@@ -2,10 +2,10 @@
 
 import { memo, useCallback, useEffect, useMemo, useRef, useState } from 'react'
 import { createLogger } from '@sim/logger'
-import { ChevronLeft, ChevronRight, ZoomIn, ZoomOut } from 'lucide-react'
 import { pdfjs, Document as ReactPdfDocument, Page as ReactPdfPage } from 'react-pdf'
 import 'react-pdf/dist/Page/TextLayer.css'
-import { Button, Skeleton } from '@/components/emcn'
+import { Skeleton } from '@/components/emcn'
+import { PreviewToolbar } from '@/app/workspace/[workspaceId]/files/components/file-viewer/preview-toolbar'
 
 pdfjs.GlobalWorkerOptions.workerSrc = new URL(
   'pdfjs-dist/build/pdf.worker.min.mjs',
@@ -172,81 +172,44 @@ export const PdfViewerCore = memo(function PdfViewerCore({ source, filename }: P
   return (
     <div className='flex flex-1 flex-col overflow-hidden'>
       {pageCount > 0 && !loadError && (
-        <div className='flex shrink-0 items-center justify-between border-[var(--border)] border-b bg-[var(--surface-1)] px-3 py-1.5'>
-          <div className='flex items-center gap-1'>
-            <Button
-              variant='ghost'
-              size='sm'
-              onClick={() => {
-                const prev = Math.max(1, currentPage - 1)
-                setCurrentPage(prev)
-                scrollToPage(prev)
-              }}
-              disabled={currentPage <= 1}
-              className='h-6 w-6 p-0 text-[var(--text-icon)]'
-              aria-label='Previous page'
-            >
-              <ChevronLeft className='h-[14px] w-[14px]' />
-            </Button>
-            <span className='min-w-[5rem] text-center text-[12px] text-[var(--text-secondary)]'>
-              {currentPage} / {pageCount}
-            </span>
-            <Button
-              variant='ghost'
-              size='sm'
-              onClick={() => {
-                const next = Math.min(pageCount, currentPage + 1)
-                setCurrentPage(next)
-                scrollToPage(next)
-              }}
-              disabled={currentPage >= pageCount}
-              className='h-6 w-6 p-0 text-[var(--text-icon)]'
-              aria-label='Next page'
-            >
-              <ChevronRight className='h-[14px] w-[14px]' />
-            </Button>
-          </div>
-
-          <div className='flex items-center gap-1'>
-            <Button
-              variant='ghost'
-              size='sm'
-              onClick={() => {
-                const c = containerRef.current
-                applyZoomAt(
-                  Math.max(PDF_ZOOM_MIN, zoomRef.current / PDF_ZOOM_STEP),
-                  c ? c.clientWidth / 2 : 0,
-                  c ? c.clientHeight / 2 : 0
-                )
-              }}
-              disabled={displayZoom <= PDF_ZOOM_MIN}
-              className='h-6 w-6 p-0 text-[var(--text-icon)]'
-              aria-label='Zoom out'
-            >
-              <ZoomOut className='h-[14px] w-[14px]' />
-            </Button>
-            <span className='min-w-[3rem] text-center text-[12px] text-[var(--text-secondary)]'>
-              {Math.round(displayZoom * 100)}%
-            </span>
-            <Button
-              variant='ghost'
-              size='sm'
-              onClick={() => {
-                const c = containerRef.current
-                applyZoomAt(
-                  Math.min(PDF_ZOOM_MAX, zoomRef.current * PDF_ZOOM_STEP),
-                  c ? c.clientWidth / 2 : 0,
-                  c ? c.clientHeight / 2 : 0
-                )
-              }}
-              disabled={displayZoom >= PDF_ZOOM_MAX}
-              className='h-6 w-6 p-0 text-[var(--text-icon)]'
-              aria-label='Zoom in'
-            >
-              <ZoomIn className='h-[14px] w-[14px]' />
-            </Button>
-          </div>
-        </div>
+        <PreviewToolbar
+          navigation={{
+            current: currentPage,
+            total: pageCount,
+            label: 'page',
+            onPrevious: () => {
+              const prev = Math.max(1, currentPage - 1)
+              setCurrentPage(prev)
+              scrollToPage(prev)
+            },
+            onNext: () => {
+              const next = Math.min(pageCount, currentPage + 1)
+              setCurrentPage(next)
+              scrollToPage(next)
+            },
+          }}
+          zoom={{
+            label: `${Math.round(displayZoom * 100)}%`,
+            canZoomOut: displayZoom > PDF_ZOOM_MIN,
+            canZoomIn: displayZoom < PDF_ZOOM_MAX,
+            onZoomOut: () => {
+              const c = containerRef.current
+              applyZoomAt(
+                Math.max(PDF_ZOOM_MIN, zoomRef.current / PDF_ZOOM_STEP),
+                c ? c.clientWidth / 2 : 0,
+                c ? c.clientHeight / 2 : 0
+              )
+            },
+            onZoomIn: () => {
+              const c = containerRef.current
+              applyZoomAt(
+                Math.min(PDF_ZOOM_MAX, zoomRef.current * PDF_ZOOM_STEP),
+                c ? c.clientWidth / 2 : 0,
+                c ? c.clientHeight / 2 : 0
+              )
+            },
+          }}
+        />
       )}
 
       <div