fix(files): detect extensions without a length cap

Author: waleedlatif1

apps/sim/lib/uploads/utils/file-utils.test.ts

@@ -329,6 +329,8 @@ describe('ensureFileNameExtension', () => {
       'download (641).jpg'
     )
     expect(ensureFileNameExtension('hero.png', 'image/jpeg')).toBe('hero.png')
+    expect(ensureFileNameExtension('site.webmanifest', 'application/json')).toBe('site.webmanifest')
+    expect(ensureFileNameExtension('Sim.ai <> RVTech', 'text/html')).toBe('Sim.ai <> RVTech.html')
     expect(ensureFileNameExtension('blob', 'application/octet-stream')).toBe('blob')
     expect(ensureFileNameExtension('blob', null)).toBe('blob')
   })

apps/sim/lib/uploads/utils/file-utils.ts

@@ -3,6 +3,7 @@ import { omit } from '@sim/utils/object'
 import type { StorageContext } from '@/lib/uploads'
 import {
   ACCEPTED_FILE_TYPES,
+  isAlphanumericExtension,
   SUPPORTED_ARCHIVE_EXTENSIONS,
   SUPPORTED_DOCUMENT_EXTENSIONS,
 } from '@/lib/uploads/utils/validation'
@@ -617,8 +618,6 @@ export function getExtensionFromMimeType(mimeType: string): string | null {
   return MIME_TO_EXTENSION[mimeType.split(';')[0].trim().toLowerCase()] || null
 }
 
-const FILE_NAME_EXTENSION_RE = /\.[A-Za-z0-9]{1,8}$/
-
 /**
  * Appends the extension the content type implies when a file name carries none, so a
  * saved copy opens in the right application.
@@ -627,7 +626,7 @@ export function ensureFileNameExtension(
   fileName: string,
   contentType: string | null | undefined
 ): string {
-  if (!contentType || FILE_NAME_EXTENSION_RE.test(fileName)) return fileName
+  if (!contentType || isAlphanumericExtension(getFileExtension(fileName))) return fileName
   const extension = getExtensionFromMimeType(contentType)
   return extension ? `${fileName}.${extension}` : fileName
 }