fix(input-format): require run-ready file shape, don't clobber named files input

Addresses review (Greptile P1s):
- parseInputFormatFiles now requires the full run-ready shape (id/name/url +
  finite size + type), so a partial file can't open in uploader mode or reach
  workflowInput.files only to be rejected by normalizeStartFile (which would
  silently drop every file).
- buildInputFormatInput no longer overwrites a user field literally named
  'files' with the upload channel; that reserved-name collision is left for the
  executor to surface.

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/w/[workflowId]/hooks/use-workflow-execution.ts

@@ -163,8 +163,11 @@ function buildInputFormatInput(inputFormatValue: unknown): Record<string, any> |
     }
   }
 
+  // Route file[] fields to the dedicated `files` channel, but never clobber a
+  // user field that is itself named `files` (a reserved-name config error the
+  // executor surfaces on its own).
   const files = collectInputFormatFiles(inputFormatValue)
-  if (files.length > 0) testInput.files = files
+  if (files.length > 0 && !('files' in testInput)) testInput.files = files
 
   return Object.keys(testInput).length > 0 ? testInput : undefined
 }

apps/sim/lib/workflows/input-format.test.ts

@@ -288,6 +288,17 @@ describe('parseInputFormatFiles', () => {
   it.concurrent('keeps only the valid files in a mixed array', () => {
     expect(parseInputFormatFiles(JSON.stringify([file, { name: 'bad' }]))).toEqual([file])
   })
+
+  it.concurrent('rejects partial files missing the run-ready size/type', () => {
+    expect(parseInputFormatFiles(JSON.stringify([{ id: 'x', name: 'a.pdf', url: '/u' }]))).toEqual(
+      []
+    )
+    expect(
+      parseInputFormatFiles(
+        JSON.stringify([{ id: 'x', name: 'a.pdf', url: '/u', size: Number.NaN, type: 'x' }])
+      )
+    ).toEqual([])
+  })
 })
 
 describe('collectInputFormatFiles', () => {

apps/sim/lib/workflows/input-format.ts

@@ -81,14 +81,21 @@ export function parseInputFormatFiles(value: unknown): InputFormatFile[] {
 
   if (!Array.isArray(raw)) return []
 
-  return raw.filter(
-    (file): file is InputFormatFile =>
-      file !== null &&
-      typeof file === 'object' &&
-      typeof (file as InputFormatFile).name === 'string' &&
-      typeof (file as InputFormatFile).url === 'string' &&
-      typeof (file as InputFormatFile).id === 'string'
-  )
+  return raw.filter((file): file is InputFormatFile => {
+    if (file === null || typeof file !== 'object') return false
+    const f = file as InputFormatFile
+    // Require the full run-ready shape the executor's normalizeStartFile needs,
+    // so a partial object never opens in uploader mode or reaches the files
+    // channel only to be rejected (which would silently drop every file).
+    return (
+      typeof f.id === 'string' &&
+      typeof f.name === 'string' &&
+      typeof f.url === 'string' &&
+      typeof f.size === 'number' &&
+      Number.isFinite(f.size) &&
+      typeof f.type === 'string'
+    )
+  })
 }
 
 /**