fix(integrations): fail loudly on partial Notion reads, validate Asana limits and assignee, accept cursor-only Dropbox paging, normalize Supabase order modifiers, and mark nullable pagination outputs

Author: waleedlatif1

apps/docs/content/docs/integrations/box.mdx

@@ -156,7 +156,7 @@ List files and folders in a Box folder
 |   ↳ `size` | number | Item size in bytes |
 |   ↳ `createdAt` | string | Creation timestamp |
 |   ↳ `modifiedAt` | string | Last modified timestamp |
-| `totalCount` | number | Total number of items in the folder \(omitted by Box for marker pagination\) |
+| `totalCount` | number | Total number of items in the folder; null with marker pagination, where Box omits it |
 | `offset` | number | Current pagination offset |
 | `limit` | number | Current pagination limit |
 | `nextMarker` | string | Marker for the next page when using marker-based pagination |

apps/docs/content/docs/integrations/dropbox.mdx

@@ -104,7 +104,7 @@ List the contents of a folder in Dropbox
 
 | Parameter | Type | Required | Description |
 | --------- | ---- | -------- | ----------- |
-| `path` | string | Yes | The path of the folder to list \(use "" for root\) |
+| `path` | string | No | The path of the folder to list \(use "" for root\). Required unless cursor is set |
 | `recursive` | boolean | No | If true, list contents recursively |
 | `includeDeleted` | boolean | No | If true, include deleted files/folders |
 | `includeMediaInfo` | boolean | No | If true, include media info for photos/videos |
@@ -322,7 +322,7 @@ Search for files and folders in Dropbox
 
 | Parameter | Type | Required | Description |
 | --------- | ---- | -------- | ----------- |
-| `query` | string | Yes | The search query |
+| `query` | string | No | The search query. Required unless cursor is set |
 | `path` | string | No | Dropbox folder path to limit search scope \(e.g., /folder/subfolder\) |
 | `fileExtensions` | string | No | Comma-separated list of file extensions to filter by \(e.g., pdf,xlsx\) |
 | `maxResults` | number | No | Maximum number of results to return \(default: 100\) |

apps/sim/lib/internal/asana/operations.test.ts

@@ -81,6 +81,20 @@ describe('Asana operations', () => {
     expect(parsedUrl.searchParams.get('offset')).toBe('tok')
   })
 
+  it('rejects a workspace task list without an assignee and out-of-range page sizes', async () => {
+    await expect(executeAsanaGetTask({ ...AUTH, workspace: 'workspace1' })).rejects.toMatchObject({
+      status: 400,
+      body: { error: 'assignee is required when listing tasks by workspace' },
+    })
+    await expect(
+      executeAsanaSearchTasks({ ...AUTH, workspace: 'workspace1', limit: 500 })
+    ).rejects.toMatchObject({ status: 400 })
+    await expect(
+      executeAsanaGetTask({ ...AUTH, project: 'project1', limit: '2.5' })
+    ).rejects.toMatchObject({ status: 400 })
+    expect(fetchMock).not.toHaveBeenCalled()
+  })
+
   it('sends the search page size', async () => {
     fetchMock.mockResolvedValueOnce(new Response(JSON.stringify({ data: [] })))
     await executeAsanaSearchTasks({ ...AUTH, workspace: 'workspace1', limit: 25 })

apps/sim/lib/internal/asana/operations.ts

@@ -35,6 +35,16 @@ function validateId(value: string, name: string): void {
   }
 }
 
+/** Asana accepts page sizes from 1 to 100; reject anything else before calling the API. */
+function pageLimit(value: string | number): string {
+  const limit = Number(value)
+  if (!Number.isInteger(limit) || limit < 1 || limit > 100) {
+    const error = 'limit must be a whole number from 1 to 100'
+    throw new AsanaOperationError(error, 400, { error })
+  }
+  return String(limit)
+}
+
 function dataObject(result: AsanaJsonObject): AsanaJsonObject {
   return toRecord(result.data)
 }
@@ -289,10 +299,14 @@ export async function executeAsanaGetTask(input: AsanaGetTaskBody, signal?: Abor
     params.append('project', input.project)
   } else if (input.workspace) {
     validateId(input.workspace, 'workspace')
+    if (!input.assignee) {
+      const error = 'assignee is required when listing tasks by workspace'
+      throw new AsanaOperationError(error, 400, { error })
+    }
     params.append('workspace', input.workspace)
-    if (input.assignee) params.append('assignee', input.assignee)
+    params.append('assignee', input.assignee)
   }
-  params.append('limit', input.limit ? String(input.limit) : '50')
+  params.append('limit', input.limit ? pageLimit(input.limit) : '50')
   if (input.offset) params.append('offset', input.offset)
   params.append('opt_fields', TASK_OPT_FIELDS)
   const result = await client.json(`/tasks?${params.toString()}`, { method: 'GET' }, signal)
@@ -357,7 +371,7 @@ export async function executeAsanaSearchTasks(input: AsanaSearchTasksBody, signa
   if (input.completed !== undefined) {
     params.append('completed', String(input.completed))
   }
-  if (input.limit) params.append('limit', String(input.limit))
+  if (input.limit) params.append('limit', pageLimit(input.limit))
   params.append('opt_fields', TASK_OPT_FIELDS)
   const result = await new AsanaClient(input.accessToken).json(
     `/workspaces/${input.workspace}/tasks/search?${params.toString()}`,

apps/sim/tools/airtable/types.ts

@@ -10,7 +10,14 @@ interface AirtableRecord {
 interface AirtableBase {
   id: string
   name: string
-  permissionLevel: 'none' | 'read' | 'comment' | 'edit' | 'create'
+  permissionLevel:
+    | 'none'
+    | 'read'
+    | 'comment'
+    | 'edit'
+    | 'create'
+    | 'interfaceOnly'
+    | 'permissionSetOnly'
 }
 
 interface AirtableFieldOption {

apps/sim/tools/asana/get_task.ts

@@ -134,6 +134,7 @@ export const asanaGetTaskTool: InternalToolConfig<AsanaGetTaskParams, AsanaGetTa
     },
     next_page: {
       type: 'object',
+      nullable: true,
       description: 'Pagination info when fetching multiple tasks; null on the last page',
       optional: true,
       properties: {

apps/sim/tools/box/list_folder_items.ts

@@ -106,7 +106,7 @@ export const boxListFolderItemsTool: ToolConfig<BoxListFolderItemsParams, BoxFol
             createdAt: item.created_at ?? null,
             modifiedAt: item.modified_at ?? null,
           })),
-          totalCount: data.total_count ?? 0,
+          totalCount: data.total_count ?? null,
           offset: data.offset ?? 0,
           limit: data.limit ?? 0,
           nextMarker: data.next_marker ?? null,

apps/sim/tools/box/types.ts

@@ -127,7 +127,7 @@ export interface BoxFileInfoResponse extends ToolResponse {
 export interface BoxFolderItemsResponse extends ToolResponse {
   output: {
     entries: Array<Record<string, unknown>>
-    totalCount: number
+    totalCount: number | null
     offset: number
     limit: number
     nextMarker: string | null
@@ -241,7 +241,9 @@ export const FOLDER_ITEMS_OUTPUT_PROPERTIES = {
   },
   totalCount: {
     type: 'number',
-    description: 'Total number of items in the folder (omitted by Box for marker pagination)',
+    description:
+      'Total number of items in the folder; null with marker pagination, where Box omits it',
+    nullable: true,
   },
   offset: { type: 'number', description: 'Current pagination offset' },
   limit: { type: 'number', description: 'Current pagination limit' },

apps/sim/tools/clickup/get_tasks.ts

@@ -229,6 +229,7 @@ export const clickupGetTasksTool: ToolConfig<ClickUpGetTasksParams, ClickUpTaskL
       type: 'boolean',
       description: 'Whether this is the last page of results',
       optional: true,
+      nullable: true,
     },
   },
 }

apps/sim/tools/docusign/types.ts

@@ -58,16 +58,19 @@ export const LIST_PAGINATION_OUTPUTS = {
     type: 'number',
     description: 'Zero-based index of the first result in this page',
     optional: true,
+    nullable: true,
   },
   endPosition: {
     type: 'number',
     description: 'Zero-based index of the last result in this page',
     optional: true,
+    nullable: true,
   },
   nextUri: {
     type: 'string',
     description: 'URI of the next page of results; null on the last page',
     optional: true,
+    nullable: true,
   },
 } as const satisfies Record<string, OutputProperty>