fix(integrations): address release review findings on the integration sweep (#10039)

* fix(integrations): validate Asana limits, Intercom snooze times, and Mailchimp template ids, clamp Airtable page size, accept Mailchimp batch arrays, and keep the Mailchimp email type option on update

* fix(integrations): resolve the tool-call operation in the QuickBooks, SAP Concur, Attio, Intercom, Notion, Box, and HubSpot mappers

* refactor(integrations): share the Intercom params mapper, reuse the boolean and JSON input helpers, and drop duplicated test helpers

* fix(mailchimp): leave the email type option unchanged when its stored value is null

* fix(integrations): seed the Mailchimp email type option per operation and tighten presence checks for Intercom, Airtable, Dropbox, and QuickBooks inputs

* fix(integrations): seed the Mailchimp email type option from the operation's first option and validate the Intercom ticket snooze on tool calls

* fix(mailchimp): seed the email type option per operation, keep Keep current outside create, and default an unset create value to true

* fix(integrations): lead the Intercom role and Pipedrive deal status with No change in agent tool rows

* fix(integrations): parse the Mailchimp update email type option like create, drop its block-level required, and keep a zero Asana limit for validation

* fix(asana): treat a whitespace-only limit field as blank on tool calls

Author: waleedlatif1

apps/docs/content/docs/integrations/mailchimp.mdx

@@ -87,7 +87,7 @@ Create a new audience (list) in Mailchimp
 | `contact` | string | Yes | JSON object of contact information \(e.g., \{"company": "Acme", "address1": "123 Main St", "city": "NYC", "state": "NY", "zip": "10001", "country": "US"\}\) |
 | `permissionReminder` | string | Yes | Permission reminder text shown to subscribers \(e.g., "You signed up for updates on our website"\) |
 | `campaignDefaults` | string | Yes | JSON object of default campaign settings \(e.g., \{"from_name": "Acme", "from_email": "news@acme.com", "subject": "", "language": "en"\}\) |
-| `emailTypeOption` | string | Yes | Support multiple email formats: "true" or "false" |
+| `emailTypeOption` | string | No | Support multiple email formats: "true" or "false" \(default "true"\) |
 
 #### Output
 
@@ -115,7 +115,7 @@ Update an existing audience (list) in Mailchimp
 | `audienceName` | string | No | The name of the audience/list \(e.g., "Newsletter Subscribers"\) |
 | `permissionReminder` | string | No | Permission reminder text shown to subscribers \(e.g., "You signed up for updates on our website"\) |
 | `campaignDefaults` | string | No | JSON object of default campaign settings \(e.g., \{"from_name": "Acme", "from_email": "news@acme.com"\}\) |
-| `emailTypeOption` | string | No | Support multiple email formats: "true" or "false". Omit to leave it unchanged |
+| `emailTypeOption` | string | No | Whether the audience supports multiple email formats \("true" or "false"\). Leave unset to keep the current setting |
 
 #### Output
 
@@ -1789,7 +1789,7 @@ Create a new batch operation in Mailchimp
 
 | Parameter | Type | Required | Description |
 | --------- | ---- | -------- | ----------- |
-| `operations` | string | Yes | JSON array of batch operations \(e.g., \[\{"method": "POST", "path": "/lists/\{list_id\}/members", "body": \{"email_address": "user@example.com", "status": "subscribed"\}\}\]\); an object body is sent as the JSON string Mailchimp expects |
+| `operations` | json | Yes | JSON array of batch operations \(e.g., \[\{"method": "POST", "path": "/lists/\{list_id\}/members", "body": \{"email_address": "user@example.com", "status": "subscribed"\}\}\]\); an object body is sent as the JSON string Mailchimp expects |
 
 #### Output
 

apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/sub-block.tsx

@@ -1190,7 +1190,7 @@ function SubBlockComponent({
             subBlockId={config.id}
             content={
               typeof config.value === 'function'
-                ? config.value(subBlockValues || {})
+                ? (config.value(subBlockValues || {}) ?? '')
                 : (config.defaultValue as string) || ''
             }
           />

apps/sim/blocks/blocks/asana.test.ts

@@ -90,3 +90,20 @@ describe('AsanaBlock agent tool calls with a configured field', () => {
     expect(mapped.workspace).toBe('ws-1')
   })
 })
+
+describe('AsanaBlock canvas limits', () => {
+  it('passes a numeric 0 limit through so validation can reject it', () => {
+    expect(
+      buildParams({ operation: 'get_task', workspace: 'ws-1', assignee: 'me', getTasks_limit: 0 })
+    ).toMatchObject({ limit: 0 })
+  })
+
+  it('ignores a whitespace-only limit field so the model limit applies on a tool call', () => {
+    expect(
+      buildParams(
+        { workspace: 'ws-1', assignee: 'me', getTasks_limit: '  ', limit: 25 },
+        { operation: 'get_task' }
+      )
+    ).toMatchObject({ limit: 25 })
+  })
+})

apps/sim/blocks/blocks/asana.ts

@@ -532,7 +532,14 @@ Return ONLY the date string in YYYY-MM-DD format - no explanations, no quotes, n
             .filter((gid) => gid.length > 0)
         }
         const toLimit = (value: unknown): number | undefined =>
-          value === undefined || value === null || value === '' ? undefined : Number(value)
+          value === undefined || value === null || String(value).trim() === ''
+            ? undefined
+            : Number(value)
+        /** A dedicated limit wins when present, so a reference resolving to 0 still reaches validation. */
+        const pickLimit = (dedicated: unknown, toolArg: unknown): unknown =>
+          dedicated === undefined || dedicated === null || String(dedicated).trim() === ''
+            ? toolCallArg(context, toolArg)
+            : dedicated
         const projectsArray = splitGids(params.projects)
 
         // Only send a completion value when the user actually set the toggle; an
@@ -553,7 +560,7 @@ Return ONLY the date string in YYYY-MM-DD format - no explanations, no quotes, n
               workspace: params.getTasks_workspace || toolCallArg(context, params.workspace),
               project: params.getTasks_project || toolCallArg(context, params.project),
               assignee: params.assignee,
-              limit: toLimit(params.getTasks_limit || toolCallArg(context, params.limit)),
+              limit: toLimit(pickLimit(params.getTasks_limit, params.limit)),
               offset: params.getTasks_offset || toolCallArg(context, params.offset),
             }
           case 'create_task':
@@ -588,7 +595,7 @@ Return ONLY the date string in YYYY-MM-DD format - no explanations, no quotes, n
               assignee: params.assignee,
               projects: projectsArray,
               completed: completedValue,
-              limit: toLimit(params.searchTasks_limit || toolCallArg(context, params.limit)),
+              limit: toLimit(pickLimit(params.searchTasks_limit, params.limit)),
             }
           case 'add_comment':
             return {

apps/sim/blocks/blocks/attio.test.ts

@@ -0,0 +1,20 @@
+import { triggersMock } from '@sim/testing/mocks/triggers.mock'
+import { describe, expect, it, vi } from 'vitest'
+
+vi.mock('@/triggers', () => triggersMock)
+
+import { AttioBlock } from '@/blocks/blocks/attio'
+
+describe('Attio agent operation resolution', () => {
+  it('maps the selected list when the operation arrives only in the tool-call context', () => {
+    const mapper = AttioBlock.tools.config?.params
+    if (!mapper) throw new Error('Attio block has no params mapper')
+
+    const mapped = mapper(
+      { oauthCredential: 'cred', listIdOrSlug: 'list-1' },
+      { operation: 'get_list' }
+    )
+
+    expect(mapped.list).toBe('list-1')
+  })
+})

apps/sim/blocks/blocks/attio.ts

@@ -1609,11 +1609,11 @@ Record reference: {"allowed_objects": ["people", "companies"]}`,
     ],
     config: {
       tool: (params) => `attio_${params.operation}`,
-      params: (params) => {
+      params: (params, context) => {
         const cleanParams: Record<string, unknown> = {
           oauthCredential: params.oauthCredential,
         }
-        const op = params.operation
+        const op = params.operation ?? context?.operation
 
         // Record params — each field is only sent for the operations whose subBlock condition
         // actually exposes it, so a stale value left over from switching operations never leaks

apps/sim/blocks/blocks/box.ts

@@ -534,7 +534,8 @@ export const BoxBlock = {
         if (normalizedFile) {
           params.file = normalizedFile
         }
-        const { credential, operation, ...rest } = params
+        const { credential, operation: storedOperation, ...rest } = params
+        const operation = storedOperation ?? context?.operation
 
         const baseParams: Record<string, unknown> = {
           accessToken: credential,

apps/sim/blocks/blocks/dropbox.test.ts

@@ -0,0 +1,38 @@
+import { triggersMock } from '@sim/testing/mocks/triggers.mock'
+import { describe, expect, it, vi } from 'vitest'
+
+vi.mock('@/triggers', () => triggersMock)
+
+import { DropboxBlock } from '@/blocks/blocks/dropbox'
+
+function requiredFor(subBlockId: string, values: Record<string, unknown>) {
+  const subBlock = DropboxBlock.subBlocks.find(
+    (candidate) => candidate.id === subBlockId && typeof candidate.required === 'function'
+  )
+  if (!subBlock || typeof subBlock.required !== 'function') {
+    throw new Error(`${subBlockId} has no dynamic required condition`)
+  }
+  return subBlock.required(values)
+}
+
+describe('Dropbox cursor continuation', () => {
+  it('requires a folder path only until a list folder cursor is set', () => {
+    expect(requiredFor('path', {})).toEqual({ field: 'operation', value: 'dropbox_list_folder' })
+    expect(requiredFor('path', { listFolderCursor: '   ' })).toEqual({
+      field: 'operation',
+      value: 'dropbox_list_folder',
+    })
+    expect(requiredFor('path', { listFolderCursor: 'cursor-1' })).toEqual({
+      field: 'operation',
+      value: [],
+    })
+  })
+
+  it('requires a search query only until a search cursor is set', () => {
+    expect(requiredFor('query', {})).toEqual({ field: 'operation', value: 'dropbox_search' })
+    expect(requiredFor('query', { searchCursor: 'cursor-1' })).toEqual({
+      field: 'operation',
+      value: [],
+    })
+  })
+})

apps/sim/blocks/blocks/dropbox.ts

@@ -175,7 +175,10 @@ export const DropboxBlock = {
       type: 'short-input',
       placeholder: '/ (root) or /folder',
       condition: { field: 'operation', value: 'dropbox_list_folder' },
-      required: true,
+      required: (values) =>
+        String(values?.listFolderCursor ?? '').trim()
+          ? { field: 'operation', value: [] }
+          : { field: 'operation', value: 'dropbox_list_folder' },
     },
     {
       id: 'recursive',
@@ -332,7 +335,10 @@ Return ONLY the timestamp string - no explanations, no quotes, no extra text.`,
       type: 'short-input',
       placeholder: 'Enter search term...',
       condition: { field: 'operation', value: 'dropbox_search' },
-      required: true,
+      required: (values) =>
+        String(values?.searchCursor ?? '').trim()
+          ? { field: 'operation', value: [] }
+          : { field: 'operation', value: 'dropbox_search' },
     },
     {
       id: 'path',

apps/sim/blocks/blocks/hubspot.ts

@@ -1635,7 +1635,7 @@ Return ONLY the JSON array of property names - no explanations, no markdown, no
       params: (params, context) => {
         const {
           oauthCredential,
-          operation,
+          operation: storedOperation,
           propertiesToSet,
           properties,
           searchProperties,
@@ -1647,6 +1647,7 @@ Return ONLY the JSON array of property names - no explanations, no markdown, no
           archived,
           ...rest
         } = params
+        const operation = storedOperation ?? context?.operation
 
         const cleanParams: Record<string, any> = {
           oauthCredential,