fix(dropbox): read the nested existing shared link and page list folder and search with cursors

Author: waleedlatif1

apps/sim/blocks/blocks/dropbox.ts

@@ -3,7 +3,7 @@ import { DropboxIcon } from '@/components/icons'
 import { getScopesForService } from '@/lib/oauth/utils'
 import type { BlockConfig, BlockMeta } from '@/blocks/types'
 import { AuthMode, IntegrationType } from '@/blocks/types'
-import { normalizeFileInput } from '@/blocks/utils'
+import { normalizeFileInput, toolCallArg } from '@/blocks/utils'
 
 /*
  * Canonical basic/advanced pair for the upload source, shared by the card
@@ -190,6 +190,14 @@ export const DropboxBlock = {
       placeholder: '500',
       condition: { field: 'operation', value: 'dropbox_list_folder' },
     },
+    {
+      id: 'listFolderCursor',
+      title: 'Cursor',
+      type: 'short-input',
+      placeholder: 'Cursor from a previous call, to fetch the next page',
+      condition: { field: 'operation', value: 'dropbox_list_folder' },
+      mode: 'advanced',
+    },
     {
       id: 'path',
       title: 'Folder Path',
@@ -347,6 +355,14 @@ Return ONLY the timestamp string - no explanations, no quotes, no extra text.`,
       placeholder: '100',
       condition: { field: 'operation', value: 'dropbox_search' },
     },
+    {
+      id: 'searchCursor',
+      title: 'Cursor',
+      type: 'short-input',
+      placeholder: 'Cursor from a previous call, to fetch the next page',
+      condition: { field: 'operation', value: 'dropbox_search' },
+      mode: 'advanced',
+    },
     {
       id: 'path',
       title: 'Path',
@@ -459,8 +475,15 @@ Return ONLY the timestamp string - no explanations, no quotes, no extra text.`,
             return 'dropbox_upload'
         }
       },
-      params: (params) => {
+      params: (params, context) => {
         const result: Record<string, unknown> = {}
+        const operation = params.operation ?? context?.operation
+        // `cursor` is List Shared Links' field; List Folder and Search page with their own
+        if (operation === 'dropbox_list_folder') {
+          result.cursor = params.listFolderCursor || toolCallArg(context, params.cursor)
+        } else if (operation === 'dropbox_search') {
+          result.cursor = params.searchCursor || toolCallArg(context, params.cursor)
+        }
         if (params.limit) result.limit = Number(params.limit)
         if (params.maxResults) result.maxResults = Number(params.maxResults)
         const normalizedFile = normalizeFileInput(params.file, { single: true })
@@ -494,6 +517,11 @@ Return ONLY the timestamp string - no explanations, no quotes, no extra text.`,
     maxResults: { type: 'number', description: 'Maximum search results' },
     directOnly: { type: 'boolean', description: 'Only return direct links, not parent folders' },
     cursor: { type: 'string', description: 'Fetch the next page of shared links (pagination)' },
+    listFolderCursor: {
+      type: 'string',
+      description: 'Fetch the next page of folder entries (pagination)',
+    },
+    searchCursor: { type: 'string', description: 'Fetch the next page of search results' },
     beforeRev: { type: 'string', description: 'Fetch revisions before this one (pagination)' },
     rev: { type: 'string', description: 'Revision identifier to restore' },
   },

apps/sim/tools/dropbox/create_shared_link.ts

@@ -90,10 +90,10 @@ export const dropboxCreateSharedLinkTool: ToolConfig<
     if (!response.ok) {
       // A link may already exist for this path - Dropbox includes its metadata in the error
       // when the requested settings match the existing link, so surface it as a success.
-      // Per Stone's JSON rules, a union member whose payload is a struct (SharedLinkMetadata)
-      // flattens that struct's fields alongside ".tag" - there is no nested "metadata" key.
-      const existingLink = data.error?.shared_link_already_exists
-      if (existingLink) {
+      // SharedLinkMetadata has enumerated subtypes (file/folder), so Stone nests it under the
+      // union tag: error.shared_link_already_exists = { ".tag": "metadata", metadata: {...} }.
+      const existingLink = data.error?.shared_link_already_exists?.metadata
+      if (existingLink?.url) {
         return {
           success: true,
           output: {

apps/sim/tools/dropbox/dropbox.test.ts

@@ -0,0 +1,67 @@
+/**
+ * @vitest-environment node
+ */
+import { describe, expect, it } from 'vitest'
+import { dropboxCreateSharedLinkTool } from '@/tools/dropbox/create_shared_link'
+import { dropboxListFolderTool } from '@/tools/dropbox/list_folder'
+import { dropboxSearchTool } from '@/tools/dropbox/search'
+import type { ToolConfig } from '@/tools/types'
+
+function resolveRequest<P>(tool: ToolConfig<P>, params: P) {
+  const { url, body } = tool.request
+  return {
+    url: typeof url === 'function' ? url(params) : url,
+    body: body?.(params),
+  }
+}
+
+describe('dropbox_create_shared_link', () => {
+  it('returns the existing link that Dropbox nests under the shared_link_already_exists union', async () => {
+    const existing = {
+      '.tag': 'file',
+      url: 'https://www.dropbox.com/s/abc/report.pdf?dl=0',
+      name: 'report.pdf',
+      path_lower: '/report.pdf',
+      link_permissions: { can_revoke: true, resolved_visibility: { '.tag': 'public' } },
+    }
+    const response = Response.json(
+      {
+        error_summary: 'shared_link_already_exists/metadata/..',
+        error: {
+          '.tag': 'shared_link_already_exists',
+          shared_link_already_exists: { '.tag': 'metadata', metadata: existing },
+        },
+      },
+      { status: 409 }
+    )
+
+    const result = await dropboxCreateSharedLinkTool.transformResponse?.(response)
+
+    expect(result?.success).toBe(true)
+    expect(result?.output.sharedLink?.url).toBe(existing.url)
+  })
+})
+
+describe('cursor pagination', () => {
+  it('continues a folder listing from a cursor', () => {
+    expect(
+      resolveRequest(dropboxListFolderTool, { path: '/docs', recursive: true, cursor: ' abc ' })
+    ).toEqual({
+      url: 'https://api.dropboxapi.com/2/files/list_folder/continue',
+      body: { cursor: 'abc' },
+    })
+    expect(resolveRequest(dropboxListFolderTool, { path: '/docs' }).url).toBe(
+      'https://api.dropboxapi.com/2/files/list_folder'
+    )
+  })
+
+  it('continues a search from a cursor', () => {
+    expect(resolveRequest(dropboxSearchTool, { query: 'invoice', cursor: 'xyz' })).toEqual({
+      url: 'https://api.dropboxapi.com/2/files/search/continue_v2',
+      body: { cursor: 'xyz' },
+    })
+    expect(resolveRequest(dropboxSearchTool, { query: 'invoice' }).url).toBe(
+      'https://api.dropboxapi.com/2/files/search_v2'
+    )
+  })
+})

apps/sim/tools/dropbox/list_folder.ts

@@ -42,12 +42,22 @@ export const dropboxListFolderTool: ToolConfig<DropboxListFolderParams, DropboxL
         type: 'number',
         required: false,
         visibility: 'user-or-llm',
-        description: 'Maximum number of results to return (default: 500)',
+        description: 'Approximate maximum number of entries to return per page (1-2000)',
+      },
+      cursor: {
+        type: 'string',
+        required: false,
+        visibility: 'user-or-llm',
+        description:
+          'Cursor from a previous List Folder call to fetch the next page via list_folder/continue (other options are ignored when set)',
       },
     },
 
     request: {
-      url: 'https://api.dropboxapi.com/2/files/list_folder',
+      url: (params) =>
+        params.cursor?.trim()
+          ? 'https://api.dropboxapi.com/2/files/list_folder/continue'
+          : 'https://api.dropboxapi.com/2/files/list_folder',
       method: 'POST',
       headers: (params) => {
         if (!params.accessToken) {
@@ -58,13 +68,17 @@ export const dropboxListFolderTool: ToolConfig<DropboxListFolderParams, DropboxL
           'Content-Type': 'application/json',
         }
       },
-      body: (params) => ({
-        path: params.path.trim() === '/' ? '' : params.path.trim(),
-        recursive: params.recursive ?? false,
-        include_deleted: params.includeDeleted ?? false,
-        include_media_info: params.includeMediaInfo ?? false,
-        limit: params.limit,
-      }),
+      body: (params) => {
+        const cursor = params.cursor?.trim()
+        if (cursor) return { cursor }
+        return {
+          path: params.path.trim() === '/' ? '' : params.path.trim(),
+          recursive: params.recursive ?? false,
+          include_deleted: params.includeDeleted ?? false,
+          include_media_info: params.includeMediaInfo ?? false,
+          limit: params.limit,
+        }
+      },
     },
 
     transformResponse: async (response) => {
@@ -105,7 +119,7 @@ export const dropboxListFolderTool: ToolConfig<DropboxListFolderParams, DropboxL
       },
       cursor: {
         type: 'string',
-        description: 'Cursor for pagination',
+        description: 'Cursor to pass back as the cursor input to fetch the next page',
       },
       hasMore: {
         type: 'boolean',

apps/sim/tools/dropbox/search.ts

@@ -37,10 +37,20 @@ export const dropboxSearchTool: ToolConfig<DropboxSearchParams, DropboxSearchRes
       visibility: 'user-or-llm',
       description: 'Maximum number of results to return (default: 100)',
     },
+    cursor: {
+      type: 'string',
+      required: false,
+      visibility: 'user-or-llm',
+      description:
+        'Cursor from a previous Search call to fetch the next page via search/continue_v2 (other options are ignored when set)',
+    },
   },
 
   request: {
-    url: 'https://api.dropboxapi.com/2/files/search_v2',
+    url: (params) =>
+      params.cursor?.trim()
+        ? 'https://api.dropboxapi.com/2/files/search/continue_v2'
+        : 'https://api.dropboxapi.com/2/files/search_v2',
     method: 'POST',
     headers: (params) => {
       if (!params.accessToken) {
@@ -52,6 +62,9 @@ export const dropboxSearchTool: ToolConfig<DropboxSearchParams, DropboxSearchRes
       }
     },
     body: (params) => {
+      const cursor = params.cursor?.trim()
+      if (cursor) return { cursor }
+
       const body: Record<string, any> = {
         query: params.query,
       }
@@ -130,7 +143,8 @@ export const dropboxSearchTool: ToolConfig<DropboxSearchParams, DropboxSearchRes
     },
     cursor: {
       type: 'string',
-      description: 'Cursor for pagination',
+      description: 'Cursor to pass back as the cursor input to fetch the next page',
+      optional: true,
     },
   },
 }

apps/sim/tools/dropbox/types.ts

@@ -106,6 +106,7 @@ export interface DropboxListFolderParams extends DropboxBaseParams {
   includeDeleted?: boolean
   includeMediaInfo?: boolean
   limit?: number
+  cursor?: string
 }
 
 export interface DropboxListFolderResponse extends ToolResponse {
@@ -192,6 +193,7 @@ export interface DropboxSearchParams extends DropboxBaseParams {
   path?: string
   fileExtensions?: string
   maxResults?: number
+  cursor?: string
 }
 
 export interface DropboxSearchResponse extends ToolResponse {