fix(clickup): integer-only maxDocs and location filters, depth-aware doc headings, most-specific-location hints

Author: waleedlatif1

apps/sim/connectors/clickup/clickup.ts

@@ -96,14 +96,15 @@ function parsePage(value: unknown): ClickUpDocPage | null {
  * Flattens a Doc's page tree depth-first into markdown sections, skipping
  * deleted and archived pages (their subpages are skipped with them).
  */
-function flattenPages(pages: ClickUpDocPage[], sections: string[]): void {
+function flattenPages(pages: ClickUpDocPage[], sections: string[], depth = 0): void {
+  const heading = '#'.repeat(Math.min(depth + 1, 6))
   for (const page of pages) {
     if (page.deleted || page.archived) continue
     const parts: string[] = []
-    if (page.name.trim()) parts.push(`# ${page.name.trim()}`)
+    if (page.name.trim()) parts.push(`${heading} ${page.name.trim()}`)
     if (page.content.trim()) parts.push(page.content.trim())
     if (parts.length > 0) sections.push(parts.join('\n\n'))
-    flattenPages(page.pages, sections)
+    flattenPages(page.pages, sections, depth + 1)
   }
 }
 
@@ -169,7 +170,7 @@ export const clickupConnector: ConnectorConfig = {
       typeof sourceConfig.spaceId === 'string' && sourceConfig.spaceId.trim()
         ? sourceConfig.spaceId.trim()
         : undefined
-    const maxDocs = sourceConfig.maxDocs ? Number(sourceConfig.maxDocs) : 0
+    const maxDocs = sourceConfig.maxDocs ? Math.floor(Number(sourceConfig.maxDocs)) : 0
 
     const url = buildSearchDocsUrl(workspaceId, { spaceId, limit: LIST_PAGE_SIZE, cursor })
     const response = await fetchWithRetry(url, {
@@ -271,8 +272,8 @@ export const clickupConnector: ConnectorConfig = {
     }
 
     const maxDocs = sourceConfig.maxDocs as string | undefined
-    if (maxDocs && (Number.isNaN(Number(maxDocs)) || Number(maxDocs) <= 0)) {
-      return { valid: false, error: 'Max docs must be a positive number' }
+    if (maxDocs && (!Number.isInteger(Number(maxDocs)) || Number(maxDocs) <= 0)) {
+      return { valid: false, error: 'Max docs must be a positive whole number' }
     }
 
     const spaceId =

apps/sim/lib/webhooks/providers/clickup.test.ts

@@ -315,12 +315,17 @@ describe('ClickUp webhook provider', () => {
       expect(deleteInit.method).toBe('DELETE')
     })
 
-    it('rejects non-numeric location filters before calling ClickUp', async () => {
+    it('rejects non-integer location filters before calling ClickUp', async () => {
       await expect(
         clickupHandler.createSubscription!(
           createContext({ ...validConfig, triggerSpaceId: 'not-a-number' })
         )
-      ).rejects.toThrow(/Space ID must be numeric/)
+      ).rejects.toThrow(/Space ID must be a whole number/)
+      await expect(
+        clickupHandler.createSubscription!(
+          createContext({ ...validConfig, triggerSpaceId: '12.5' })
+        )
+      ).rejects.toThrow(/Space ID must be a whole number/)
       expect(fetchMock).not.toHaveBeenCalled()
     })
   })

apps/sim/lib/webhooks/providers/clickup.ts

@@ -74,8 +74,8 @@ function parseOptionalNumericId(value: unknown, label: string): number | undefin
   const raw = String(value).trim()
   if (!raw) return undefined
   const parsed = Number(raw)
-  if (!Number.isFinite(parsed)) {
-    throw new Error(`ClickUp ${label} must be numeric. Received: ${raw}`)
+  if (!Number.isInteger(parsed)) {
+    throw new Error(`ClickUp ${label} must be a whole number. Received: ${raw}`)
   }
   return parsed
 }

apps/sim/triggers/clickup/subblocks.ts

@@ -69,7 +69,8 @@ export function buildClickUpTriggerSubBlocks(triggerId: string): SubBlockConfig[
       title: 'Space ID (Optional)',
       type: 'short-input',
       placeholder: 'Leave empty for the entire workspace',
-      description: 'Only receive events from this space',
+      description:
+        'Only receive events from this space. ClickUp applies the most specific location when several are set',
       mode: 'trigger',
       condition: { field: 'selectedTriggerId', value: triggerId },
     },
@@ -78,7 +79,8 @@ export function buildClickUpTriggerSubBlocks(triggerId: string): SubBlockConfig[
       title: 'Folder ID (Optional)',
       type: 'short-input',
       placeholder: 'Leave empty for the entire workspace',
-      description: 'Only receive events from this folder',
+      description:
+        'Only receive events from this folder. ClickUp applies the most specific location when several are set',
       mode: 'trigger',
       condition: { field: 'selectedTriggerId', value: triggerId },
     },
@@ -87,7 +89,8 @@ export function buildClickUpTriggerSubBlocks(triggerId: string): SubBlockConfig[
       title: 'List ID (Optional)',
       type: 'short-input',
       placeholder: 'Leave empty for the entire workspace',
-      description: 'Only receive events from this list',
+      description:
+        'Only receive events from this list. ClickUp applies the most specific location when several are set',
       mode: 'trigger',
       condition: { field: 'selectedTriggerId', value: triggerId },
     },
@@ -96,7 +99,8 @@ export function buildClickUpTriggerSubBlocks(triggerId: string): SubBlockConfig[
       title: 'Task ID (Optional)',
       type: 'short-input',
       placeholder: 'Leave empty for the entire workspace',
-      description: 'Only receive events for this task',
+      description:
+        'Only receive events for this task. ClickUp applies the most specific location when several are set',
       mode: 'trigger',
       condition: { field: 'selectedTriggerId', value: triggerId },
     },