fix(thrive): surface malformed JSON errors, drop redundant limit param

- parseThriveArray/parseThriveJsonObject now throw a descriptive error on
  malformed JSON instead of silently sending an empty/omitted value
- additionalFields parse errors are surfaced to the caller
- remove the redundant 'limit' query param (perPage already covers paging and
  the API prioritises perPage over limit) from the five list tools and block

Author: waleedlatif1

apps/docs/content/docs/en/integrations/thrive.mdx

@@ -330,7 +330,6 @@ List audiences and structures in Thrive with pagination.
 | `updatedSince` | string | No | Return only audiences updated on or after this date/time \(ISO 8601\) |
 | `page` | number | No | Page number for pagination \(default 1\) |
 | `perPage` | number | No | Number of results per page \(1-1000, default 100\) |
-| `limit` | number | No | Number of results to page through \(1-100, default 100\) |
 
 #### Output
 
@@ -487,7 +486,6 @@ List the members of a Thrive audience with pagination.
 | `audienceId` | string | Yes | The audience id or audience reference |
 | `page` | number | No | Page number for pagination \(default 1\) |
 | `perPage` | number | No | Number of results per page \(1-1000, default 100\) |
-| `limit` | number | No | Number of results to page through \(1-100, default 100\) |
 
 #### Output
 
@@ -700,7 +698,6 @@ List compliance assignments in Thrive, optionally filtered by audience.
 | `updatedSince` | string | No | Return only items updated on or after this date/time \(ISO 8601\) |
 | `page` | number | No | Page number for pagination \(default 1\) |
 | `perPage` | number | No | Number of results per page \(1-100, default 100\) |
-| `limit` | number | No | Number of items to retrieve per page \(1-100, default 100\) |
 
 #### Output
 
@@ -860,7 +857,6 @@ List enrolments for a compliance assignment in Thrive.
 | `status` | string | No | Filter by enrolment status: archived, complete, open, overdue, scheduled, or unassigned |
 | `page` | number | No | Page number for pagination \(default 1\) |
 | `perPage` | number | No | Number of results per page \(1-100, default 100\) |
-| `limit` | number | No | Number of items to retrieve per page \(1-100, default 100\) |
 
 #### Output
 
@@ -940,7 +936,6 @@ List learning completion records in Thrive, optionally filtered by user or conte
 | `completedDateRangeEnd` | string | No | Filter by completedDate \(completedDate <= this date/date-time\) |
 | `page` | number | No | Page number for pagination \(default 1\) |
 | `perPage` | number | No | Number of results per page \(1-1000, default 1000\) |
-| `limit` | number | No | Number of items to retrieve per page \(1-1000, default 1000\) |
 
 #### Output
 

apps/sim/blocks/blocks/thrive.ts

@@ -17,14 +17,6 @@ const PAGINATED_OPS = [
   'list_tags',
 ]
 
-const LIMIT_OPS = [
-  'list_audiences',
-  'list_audience_members',
-  'list_assignments',
-  'list_enrolments',
-  'list_completions',
-]
-
 const UPDATED_SINCE_OPS = [
   'search_users',
   'list_audiences',
@@ -768,14 +760,6 @@ export const ThriveBlock: BlockConfig = {
       condition: { field: 'operation', value: PAGINATED_OPS },
       mode: 'advanced',
     },
-    {
-      id: 'limit',
-      title: 'Limit',
-      type: 'short-input',
-      placeholder: 'Items per page (alternative to Per Page)',
-      condition: { field: 'operation', value: LIMIT_OPS },
-      mode: 'advanced',
-    },
   ],
 
   tools: {
@@ -832,7 +816,7 @@ export const ThriveBlock: BlockConfig = {
       tool: (params) => `thrive_${params.operation}`,
       params: (params) => {
         const result: Record<string, unknown> = {}
-        const numberFields = ['page', 'perPage', 'limit', 'completionPeriod', 'recurrence']
+        const numberFields = ['page', 'perPage', 'completionPeriod', 'recurrence']
         for (const field of numberFields) {
           const value = params[field]
           if (value !== undefined && value !== '') result[field] = Number(value)
@@ -928,7 +912,6 @@ export const ThriveBlock: BlockConfig = {
     updatedSince: { type: 'string', description: 'Return items updated on/after this date/time' },
     page: { type: 'number', description: 'Page number for pagination' },
     perPage: { type: 'number', description: 'Number of results per page' },
-    limit: { type: 'number', description: 'Items per page (alternative to perPage)' },
   },
 
   outputs: {

apps/sim/tools/thrive/add_audience_managers.ts

@@ -56,7 +56,7 @@ export const addAudienceManagersTool: ToolConfig<
       `${getThriveBaseUrl(params.host, 'v1')}/audiences/${encodeURIComponent(params.audienceId)}/managers`,
     method: 'POST',
     headers: (params) => getThriveHeaders(params.tenantId, params.apiKey),
-    body: (params) => JSON.stringify(parseThriveArray(params.managers)),
+    body: (params) => JSON.stringify(parseThriveArray(params.managers, 'managers')),
   },
 
   transformResponse: async (response: Response): Promise<ThriveAddUsersResponse> => {

apps/sim/tools/thrive/add_audience_members.ts

@@ -54,7 +54,7 @@ export const addAudienceMembersTool: ToolConfig<ThriveAudienceUsersParams, Thriv
         `${getThriveBaseUrl(params.host, 'v1')}/audiences/${encodeURIComponent(params.audienceId)}/members`,
       method: 'POST',
       headers: (params) => getThriveHeaders(params.tenantId, params.apiKey),
-      body: (params) => JSON.stringify(parseThriveArray<string>(params.users)),
+      body: (params) => JSON.stringify(parseThriveArray<string>(params.users, 'users')),
     },
 
     transformResponse: async (response: Response): Promise<ThriveAddUsersResponse> => {

apps/sim/tools/thrive/add_user_tags.ts

@@ -51,7 +51,7 @@ export const addUserTagsTool: ToolConfig<ThriveAddUserTagsParams, ThriveMessageR
       `${getThriveBaseUrl(params.host, 'v1')}/users/${encodeURIComponent(params.userId)}/tags`,
     method: 'POST',
     headers: (params) => getThriveHeaders(params.tenantId, params.apiKey),
-    body: (params) => ({ tags: parseThriveArray<string>(params.tags) }),
+    body: (params) => ({ tags: parseThriveArray<string>(params.tags, 'tags') }),
   },
 
   transformResponse: async (response: Response): Promise<ThriveMessageResponse> => {

apps/sim/tools/thrive/create_assignment.ts

@@ -84,7 +84,10 @@ export const createAssignmentTool: ToolConfig<
         contentId: params.contentId,
       }
       if (params.alternativeContentIds) {
-        body.alternativeContentIds = parseThriveArray<string>(params.alternativeContentIds)
+        body.alternativeContentIds = parseThriveArray<string>(
+          params.alternativeContentIds,
+          'alternativeContentIds'
+        )
       }
       if (params.hideAlternativeContent !== undefined) {
         body.hideAlternativeContent = params.hideAlternativeContent

apps/sim/tools/thrive/create_user.ts

@@ -1,6 +1,11 @@
 import type { ThriveCreateUserParams, ThriveUserResponse } from '@/tools/thrive/types'
 import { THRIVE_USER_LIFECYCLE_OUTPUT_PROPERTIES } from '@/tools/thrive/types'
-import { getThriveBaseUrl, getThriveHeaders, parseThriveResponse } from '@/tools/thrive/utils'
+import {
+  getThriveBaseUrl,
+  getThriveHeaders,
+  parseThriveJsonObject,
+  parseThriveResponse,
+} from '@/tools/thrive/utils'
 import type { ToolConfig } from '@/tools/types'
 
 export const createUserTool: ToolConfig<ThriveCreateUserParams, ThriveUserResponse> = {
@@ -143,11 +148,7 @@ export const createUserTool: ToolConfig<ThriveCreateUserParams, ThriveUserRespon
       if (params.sso !== undefined) body.sso = params.sso
       if (params.domain) body.domain = params.domain
       if (params.additionalFields) {
-        try {
-          body.additionalFields = JSON.parse(params.additionalFields)
-        } catch {
-          // Ignore malformed custom-field JSON; the API validates required fields.
-        }
+        body.additionalFields = parseThriveJsonObject(params.additionalFields, 'additionalFields')
       }
       return body
     },

apps/sim/tools/thrive/list_assignments.ts

@@ -63,12 +63,6 @@ export const listAssignmentsTool: ToolConfig<
       visibility: 'user-or-llm',
       description: 'Number of results per page (1-100, default 100)',
     },
-    limit: {
-      type: 'number',
-      required: false,
-      visibility: 'user-or-llm',
-      description: 'Number of items to retrieve per page (1-100, default 100)',
-    },
   },
 
   request: {
@@ -78,7 +72,6 @@ export const listAssignmentsTool: ToolConfig<
       appendThriveQuery(url, 'updatedSince', params.updatedSince)
       appendThriveQuery(url, 'page', params.page)
       appendThriveQuery(url, 'perPage', params.perPage)
-      appendThriveQuery(url, 'limit', params.limit)
       return url.toString()
     },
     method: 'GET',

apps/sim/tools/thrive/list_audience_members.ts

@@ -60,12 +60,6 @@ export const listAudienceMembersTool: ToolConfig<
       visibility: 'user-or-llm',
       description: 'Number of results per page (1-1000, default 100)',
     },
-    limit: {
-      type: 'number',
-      required: false,
-      visibility: 'user-or-llm',
-      description: 'Number of results to page through (1-100, default 100)',
-    },
   },
 
   request: {
@@ -75,7 +69,6 @@ export const listAudienceMembersTool: ToolConfig<
       )
       appendThriveQuery(url, 'page', params.page)
       appendThriveQuery(url, 'perPage', params.perPage)
-      appendThriveQuery(url, 'limit', params.limit)
       return url.toString()
     },
     method: 'GET',

apps/sim/tools/thrive/list_audiences.ts

@@ -61,12 +61,6 @@ export const listAudiencesTool: ToolConfig<ThriveListAudiencesParams, ThriveList
         visibility: 'user-or-llm',
         description: 'Number of results per page (1-1000, default 100)',
       },
-      limit: {
-        type: 'number',
-        required: false,
-        visibility: 'user-or-llm',
-        description: 'Number of results to page through (1-100, default 100)',
-      },
     },
 
     request: {
@@ -76,7 +70,6 @@ export const listAudiencesTool: ToolConfig<ThriveListAudiencesParams, ThriveList
         appendThriveQuery(url, 'updatedSince', params.updatedSince)
         appendThriveQuery(url, 'page', params.page)
         appendThriveQuery(url, 'perPage', params.perPage)
-        appendThriveQuery(url, 'limit', params.limit)
         return url.toString()
       },
       method: 'GET',