fix: show oversized upload error when mime type is missing (#7757)

jobenjada · cursoragent · web-flow · commit cefc2bdf609c · 2026-04-20T07:00:41.000Z
Co-authored-by: Cursor Agent &lt;cursoragent@cursor.com&gt;
diff --git a/apps/web/modules/ui/components/file-input/lib/utils.test.ts b/apps/web/modules/ui/components/file-input/lib/utils.test.ts
@@ -67,6 +67,15 @@ describe("File Input Utils", () => {
       expect(toast.error).toHaveBeenCalledWith(expect.stringContaining("File exceeds 5 MB size limit."));
     });
 
+    test("should show size error for oversized files even when mime type is empty", async () => {
+      const files = [new File(["x".repeat(101000)], "large.ico", { type: "" })];
+
+      const result = await getAllowedFiles(files, ["ico"], 0.1);
+
+      expect(result).toHaveLength(0);
+      expect(toast.error).toHaveBeenCalledWith(expect.stringContaining("File exceeds 0.1 MB size limit."));
+    });
+
     test("should convert HEIC files to JPEG", async () => {
       const heicFile = new File(["test"], "test.heic", { type: "image/heic" });
       const mockConvertedFile = new File(["converted"], "test.jpg", { type: "image/jpeg" });
diff --git a/apps/web/modules/ui/components/file-input/lib/utils.ts b/apps/web/modules/ui/components/file-input/lib/utils.ts
@@ -21,7 +21,7 @@ export const getAllowedFiles = async (
   const convertedFiles: File[] = [];
 
   for (const file of files) {
-    if (!file || !file.type) {
+    if (!file) {
       continue;
     }
 

Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@ export const getAllowedFiles = async (`
`21`	`21`	`const convertedFiles: File[] = [];`
`22`	`22`
`23`	`23`	`for (const file of files) {`
`24`		`- if (!file \|\| !file.type) {`
	`24`	`+ if (!file) {`
`25`	`25`	`continue;`
`26`	`26`	`}`
`27`	`27`