Skip to content

feat(files): expand file editor to support more formats, add docx/xlsx preview #3971

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
waleedlatif1 merged 11 commits into from
Apr 5, 2026
Merged

feat(files): expand file editor to support more formats, add docx/xlsx preview #3971

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
6fc7ab4
feat(files): expand file editor to support more formats, add docx/xls…
waleedlatif1 Apr 5, 2026
646f8d1
lint
waleedlatif1 Apr 5, 2026
edc103d
fix(files): narrow fileData type for closure in docx/xlsx preview eff…
waleedlatif1 Apr 5, 2026
13c4919
fix(files): address PR review — fix xlsx type, simplify error helper,…
waleedlatif1 Apr 5, 2026
0cd5e46
add mothership read externsions
icecrasher321 Apr 5, 2026
71b20bf
fix(files): update upload test — js is now a supported extension
waleedlatif1 Apr 5, 2026
846214f
fix(files): deduplicate code extensions, handle dotless filenames
waleedlatif1 Apr 5, 2026
a284499
fix(files): lower xlsx preview row cap to 1k and type workbookRef pro…
waleedlatif1 Apr 5, 2026
a7b732b
refactor(files): extract shared DataTable, isolate client-safe constants
waleedlatif1 Apr 5, 2026
e31cf4d
refactor(validation): remove Node path import, use plain string extra…
waleedlatif1 Apr 5, 2026
6c4b02e
lint
waleedlatif1 Apr 5, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix(files): deduplicate code extensions, handle dotless filenames
  • Loading branch information
@waleedlatif1
waleedlatif1 committed Apr 5, 2026
commit 846214f85e7c803e791ba0ce7628ff3985ed5f2b
48 changes: 4 additions & 44 deletions apps/sim/app/workspace/[workspaceId]/files/components/file-viewer/file-viewer.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ import { Skeleton } from '@/components/emcn'
import { cn } from '@/lib/core/utils/cn'
import type { WorkspaceFileRecord } from '@/lib/uploads/contexts/workspace'
import { getFileExtension } from '@/lib/uploads/utils/file-utils'
import { SUPPORTED_CODE_EXTENSIONS } from '@/lib/uploads/utils/validation'
import {
useUpdateWorkspaceFileContent,
useWorkspaceFileBinary,
Expand Down Expand Up @@ -44,7 +45,6 @@ const TEXT_EDITABLE_MIME_TYPES = new Set([

const TEXT_EDITABLE_EXTENSIONS = new Set([
'md',
'mdx',
'txt',
'json',
'yaml',
Expand All @@ -53,48 +53,7 @@ const TEXT_EDITABLE_EXTENSIONS = new Set([
'html',
'htm',
'svg',
'xml',
'css',
'scss',
'less',
'js',
'jsx',
'ts',
'tsx',
'py',
'rb',
'go',
'rs',
'java',
'kt',
'swift',
'c',
'cpp',
'h',
'hpp',
'cs',
'php',
'sh',
'bash',
'zsh',
'fish',
'sql',
'graphql',
'gql',
'toml',
'ini',
'conf',
'cfg',
'env',
'log',
'diff',
'patch',
'dockerfile',
'makefile',
'gitignore',
'editorconfig',
'prettierrc',
'eslintrc',
...SUPPORTED_CODE_EXTENSIONS,
])

const IFRAME_PREVIEWABLE_MIME_TYPES = new Set(['application/pdf'])
Expand Down Expand Up @@ -136,7 +95,8 @@ function resolveFileCategory(mimeType: string | null, filename: string): FileCat
if (mimeType && XLSX_PREVIEWABLE_MIME_TYPES.has(mimeType)) return 'xlsx-previewable'

const ext = getFileExtension(filename)
if (TEXT_EDITABLE_EXTENSIONS.has(ext)) return 'text-editable'
const nameKey = ext || filename.toLowerCase()
if (TEXT_EDITABLE_EXTENSIONS.has(nameKey)) return 'text-editable'
if (IFRAME_PREVIEWABLE_EXTENSIONS.has(ext)) return 'iframe-previewable'
if (IMAGE_PREVIEWABLE_EXTENSIONS.has(ext)) return 'image-previewable'
if (PPTX_PREVIEWABLE_EXTENSIONS.has(ext)) return 'pptx-previewable'
Expand Down
Loading