Fix

Author: Sg312

apps/sim/app/api/tools/file/manage/route.ts

@@ -43,23 +43,33 @@ export const POST = withRouteHandler(async (request: NextRequest) => {
     switch (body.operation) {
       case 'get': {
         const { fileId, fileInput } = body
-
-        if (fileInput && typeof fileInput === 'object' && !Array.isArray(fileInput)) {
-          return NextResponse.json({ success: true, data: { file: fileInput } })
-        }
-
-        if (!fileId) {
+        const selectedFileId =
+          fileId ||
+          (fileInput && typeof fileInput === 'object' && !Array.isArray(fileInput)
+            ? typeof fileInput.id === 'string'
+              ? fileInput.id
+              : typeof fileInput.fileId === 'string'
+                ? fileInput.fileId
+                : ''
+            : '')
+
+        if (!selectedFileId) {
           return NextResponse.json({ success: false, error: 'File is required' }, { status: 400 })
         }
 
-        const file = await getWorkspaceFile(workspaceId, fileId)
+        const file = await getWorkspaceFile(workspaceId, selectedFileId)
         if (!file) {
           return NextResponse.json(
-            { success: false, error: `File not found: "${fileId}"` },
+            { success: false, error: `File not found: "${selectedFileId}"` },
             { status: 404 }
           )
         }
 
+        logger.info('File retrieved', {
+          fileId: file.id,
+          name: file.name,
+        })
+
         return NextResponse.json({
           success: true,
           data: {

apps/sim/lib/api/contracts/tools/file.ts

@@ -22,14 +22,18 @@ export const fileManageAppendBodySchema = z.object({
   content: z.string({ error: 'content is required for append operation' }),
 })
 
-export const fileManageGetBodySchema = z.object({
-  operation: z.literal('get'),
-  workspaceId: z.string().min(1).optional(),
-  fileId: z.string().min(1).optional(),
-  fileInput: z.any().optional(),
-})
+export const fileManageGetBodySchema = z
+  .object({
+    operation: z.literal('get'),
+    workspaceId: z.string().min(1).optional(),
+    fileId: z.string().min(1).optional(),
+    fileInput: z.any().optional(),
+  })
+  .refine((data) => data.fileId !== undefined || data.fileInput !== undefined, {
+    message: 'Either fileId or fileInput is required for get operation',
+  })
 
-export const fileManageBodySchema = z.discriminatedUnion('operation', [
+export const fileManageBodySchema = z.union([
   fileManageWriteBodySchema,
   fileManageAppendBodySchema,
   fileManageGetBodySchema,