fixed supabase tools

Author: aadamgough

apps/sim/blocks/blocks/supabase.ts

@@ -596,13 +596,20 @@ Return ONLY the PostgREST filter expression - no explanations, no markdown, no e
     },
     // Storage Upload fields
     {
-      id: 'path',
-      title: 'File Path',
+      id: 'fileName',
+      title: 'File Name',
       type: 'short-input',
-      placeholder: 'folder/file.jpg',
+      placeholder: 'myfile.pdf',
       condition: { field: 'operation', value: 'storage_upload' },
       required: true,
     },
+    {
+      id: 'path',
+      title: 'Folder Path (optional)',
+      type: 'short-input',
+      placeholder: 'folder/subfolder/',
+      condition: { field: 'operation', value: 'storage_upload' },
+    },
     {
       id: 'fileContent',
       title: 'File Content',
@@ -1065,10 +1072,10 @@ Return ONLY the PostgREST filter expression - no explanations, no markdown, no e
     countType: { type: 'string', description: 'Count type: exact, planned, or estimated' },
     // Storage operation inputs
     bucket: { type: 'string', description: 'Storage bucket name' },
-    path: { type: 'string', description: 'File path in storage' },
+    path: { type: 'string', description: 'File or folder path in storage' },
     fileContent: { type: 'string', description: 'File content (base64 for binary)' },
     contentType: { type: 'string', description: 'MIME type of the file' },
-    fileName: { type: 'string', description: 'Optional filename override for downloaded file' },
+    fileName: { type: 'string', description: 'File name for upload or download override' },
     upsert: { type: 'boolean', description: 'Whether to overwrite existing file' },
     download: { type: 'boolean', description: 'Whether to force download' },
     paths: { type: 'array', description: 'Array of file paths' },

apps/sim/tools/index.ts

@@ -747,6 +747,8 @@ async function handleInternalRequest(
           url: fullUrl,
           json: () => response.json(),
           text: () => response.text(),
+          arrayBuffer: () => response.arrayBuffer(),
+          blob: () => response.blob(),
         } as Response
 
         const data = await tool.transformResponse(mockResponse, params)

apps/sim/tools/supabase/storage_upload.ts

@@ -26,11 +26,17 @@ export const storageUploadTool: ToolConfig<
       visibility: 'user-or-llm',
       description: 'The name of the storage bucket',
     },
-    path: {
+    fileName: {
       type: 'string',
       required: true,
       visibility: 'user-or-llm',
-      description: 'The path where the file will be stored (e.g., "folder/file.jpg")',
+      description: 'The name of the file (e.g., "document.pdf", "image.jpg")',
+    },
+    path: {
+      type: 'string',
+      required: false,
+      visibility: 'user-or-llm',
+      description: 'Optional folder path (e.g., "folder/subfolder/")',
     },
     fileContent: {
       type: 'string',
@@ -60,7 +66,14 @@ export const storageUploadTool: ToolConfig<
 
   request: {
     url: (params) => {
-      return `https://${params.projectId}.supabase.co/storage/v1/object/${params.bucket}/${params.path}`
+      // Combine folder path and fileName, ensuring proper formatting
+      let fullPath = params.fileName
+      if (params.path) {
+        // Ensure path ends with / and doesn't have double slashes
+        const folderPath = params.path.endsWith('/') ? params.path : `${params.path}/`
+        fullPath = `${folderPath}${params.fileName}`
+      }
+      return `https://${params.projectId}.supabase.co/storage/v1/object/${params.bucket}/${fullPath}`
     },
     method: 'POST',
     headers: (params) => {

apps/sim/tools/supabase/types.ts

@@ -132,7 +132,8 @@ export interface SupabaseStorageUploadParams {
   apiKey: string
   projectId: string
   bucket: string
-  path: string
+  fileName: string
+  path?: string
   fileContent: string
   contentType?: string
   upsert?: boolean