fix(notion): clamp page_size to 1-100 at the tool layer for list comments/users and block children

Adds a shared clampNotionPageSize helper so the agent-direct path is bounded to Notion's range, not just the block path.

Author: waleedlatif1

apps/sim/tools/notion/list_comments.ts

@@ -1,5 +1,6 @@
 import type { NotionListCommentsParams } from '@/tools/notion/types'
 import { COMMENT_LIST_RESULTS_OUTPUT, PAGINATION_OUTPUT_PROPERTIES } from '@/tools/notion/types'
+import { clampNotionPageSize } from '@/tools/notion/utils'
 import type { ToolConfig } from '@/tools/types'
 
 interface NotionListCommentsResponse {
@@ -57,7 +58,8 @@ export const notionListCommentsTool: ToolConfig<
       const url = new URL('https://api.notion.com/v1/comments')
       url.searchParams.set('block_id', params.blockId.trim())
       if (params.startCursor) url.searchParams.set('start_cursor', params.startCursor.trim())
-      if (params.pageSize != null) url.searchParams.set('page_size', String(params.pageSize))
+      const pageSize = clampNotionPageSize(params.pageSize)
+      if (pageSize != null) url.searchParams.set('page_size', String(pageSize))
       return url.toString()
     },
     method: 'GET',

apps/sim/tools/notion/list_users.ts

@@ -1,5 +1,6 @@
 import type { NotionListUsersParams } from '@/tools/notion/types'
 import { PAGINATION_OUTPUT_PROPERTIES, USER_LIST_RESULTS_OUTPUT } from '@/tools/notion/types'
+import { clampNotionPageSize } from '@/tools/notion/utils'
 import type { ToolConfig } from '@/tools/types'
 
 interface NotionListUsersResponse {
@@ -47,7 +48,8 @@ export const notionListUsersTool: ToolConfig<NotionListUsersParams, NotionListUs
     url: (params: NotionListUsersParams) => {
       const url = new URL('https://api.notion.com/v1/users')
       if (params.startCursor) url.searchParams.set('start_cursor', params.startCursor.trim())
-      if (params.pageSize != null) url.searchParams.set('page_size', String(params.pageSize))
+      const pageSize = clampNotionPageSize(params.pageSize)
+      if (pageSize != null) url.searchParams.set('page_size', String(pageSize))
       return url.toString()
     },
     method: 'GET',

apps/sim/tools/notion/retrieve_block_children.ts

@@ -1,5 +1,6 @@
 import type { NotionRetrieveBlockChildrenParams } from '@/tools/notion/types'
 import { BLOCK_LIST_RESULTS_OUTPUT, PAGINATION_OUTPUT_PROPERTIES } from '@/tools/notion/types'
+import { clampNotionPageSize } from '@/tools/notion/utils'
 import type { ToolConfig } from '@/tools/types'
 
 interface NotionRetrieveBlockChildrenResponse {
@@ -56,7 +57,8 @@ export const notionRetrieveBlockChildrenTool: ToolConfig<
     url: (params: NotionRetrieveBlockChildrenParams) => {
       const url = new URL(`https://api.notion.com/v1/blocks/${params.blockId.trim()}/children`)
       if (params.startCursor) url.searchParams.set('start_cursor', params.startCursor.trim())
-      if (params.pageSize != null) url.searchParams.set('page_size', String(params.pageSize))
+      const pageSize = clampNotionPageSize(params.pageSize)
+      if (pageSize != null) url.searchParams.set('page_size', String(pageSize))
       return url.toString()
     },
     method: 'GET',

apps/sim/tools/notion/utils.ts

@@ -1,3 +1,14 @@
+/**
+ * Clamp a Notion `page_size` value to the API's supported 1-100 range. Returns
+ * undefined when the input isn't a finite number so callers can omit the param.
+ */
+export function clampNotionPageSize(value: unknown): number | undefined {
+  if (value == null || value === '') return undefined
+  const parsed = Number(value)
+  if (!Number.isFinite(parsed)) return undefined
+  return Math.min(Math.max(Math.trunc(parsed), 1), 100)
+}
+
 export function formatPropertyValue(property: any): string {
   switch (property.type) {
     case 'title':