fix(rippling): add missing supergroup fields and fix validation issues

- Add 5 missing supergroup fields (allow_non_employees, can_override_role_states, priority, is_invisible, ignore_prov_group_matching) to types, list, and get tools
- Fix ok fallback from true to false in supergroup inclusion/exclusion member update tools
- Fix truthy check to null check for description param in create_custom_object_field

Author: waleedlatif1

apps/sim/tools/rippling/create_custom_object_field.ts

@@ -62,7 +62,7 @@ export const ripplingCreateCustomObjectFieldTool: ToolConfig<RipplingCreateCusto
       }),
       body: (params) => {
         const body: Record<string, unknown> = { name: params.name, data_type: params.dataType }
-        if (params.description) body.description = params.description
+        if (params.description != null) body.description = params.description
         if (params.isUnique != null) body.is_unique = params.isUnique
         if (params.enableHistory != null) body.enable_history = params.enableHistory
         return body

apps/sim/tools/rippling/get_supergroup.ts

@@ -44,6 +44,11 @@ export const ripplingGetSupergroupTool: ToolConfig<RipplingGetSupergroupParams>
         mutually_exclusive_key: (data.mutually_exclusive_key as string) ?? null,
         cumulatively_exhaustive_default: (data.cumulatively_exhaustive_default as boolean) ?? null,
         include_terminated: (data.include_terminated as boolean) ?? null,
+        allow_non_employees: (data.allow_non_employees as boolean) ?? null,
+        can_override_role_states: (data.can_override_role_states as boolean) ?? null,
+        priority: (data.priority as number) ?? null,
+        is_invisible: (data.is_invisible as boolean) ?? null,
+        ignore_prov_group_matching: (data.ignore_prov_group_matching as boolean) ?? null,
       },
     }
   },
@@ -74,5 +79,26 @@ export const ripplingGetSupergroupTool: ToolConfig<RipplingGetSupergroupParams>
       description: 'Whether the group includes terminated roles',
       optional: true,
     },
+    allow_non_employees: {
+      type: 'boolean',
+      description: 'Whether the group allows non-employees',
+      optional: true,
+    },
+    can_override_role_states: {
+      type: 'boolean',
+      description: 'Whether the group can override role states',
+      optional: true,
+    },
+    priority: { type: 'number', description: 'Group priority', optional: true },
+    is_invisible: {
+      type: 'boolean',
+      description: 'Whether the group is invisible',
+      optional: true,
+    },
+    ignore_prov_group_matching: {
+      type: 'boolean',
+      description: 'Whether to ignore provisioning group matching',
+      optional: true,
+    },
   },
 }

apps/sim/tools/rippling/list_supergroups.ts

@@ -64,6 +64,11 @@ export const ripplingListSupergroupsTool: ToolConfig<RipplingListSupergroupsPara
           cumulatively_exhaustive_default:
             (item.cumulatively_exhaustive_default as boolean) ?? null,
           include_terminated: (item.include_terminated as boolean) ?? null,
+          allow_non_employees: (item.allow_non_employees as boolean) ?? null,
+          can_override_role_states: (item.can_override_role_states as boolean) ?? null,
+          priority: (item.priority as number) ?? null,
+          is_invisible: (item.is_invisible as boolean) ?? null,
+          ignore_prov_group_matching: (item.ignore_prov_group_matching as boolean) ?? null,
         })),
         totalCount: results.length,
         nextLink: (data.next_link as string) ?? null,

apps/sim/tools/rippling/types.ts

@@ -59,6 +59,7 @@ export const COMPANY_OUTPUT_PROPERTIES = {
   primary_email: { type: 'string', description: 'Primary email' },
   parent_legal_entity_id: { type: 'string', description: 'Parent legal entity ID' },
   legal_entities_id: { type: 'json', description: 'Array of legal entity IDs' },
+  physical_address: { type: 'json', description: 'Physical address of the holding entity' },
 } as const satisfies Record<string, OutputProperty>
 
 /** Department output properties */
@@ -170,6 +171,32 @@ export const SUPERGROUP_OUTPUT_PROPERTIES = {
   app_owner_id: { type: 'string', description: 'App owner ID' },
   group_type: { type: 'string', description: 'Group type' },
   name: { type: 'string', description: 'Name' },
+  sub_group_type: { type: 'string', description: 'Sub group type' },
+  read_only: { type: 'boolean', description: 'Whether the group is read only' },
+  parent: { type: 'string', description: 'Parent group ID' },
+  mutually_exclusive_key: { type: 'string', description: 'Mutually exclusive key' },
+  cumulatively_exhaustive_default: {
+    type: 'boolean',
+    description: 'Whether the group is the cumulatively exhaustive default',
+  },
+  include_terminated: {
+    type: 'boolean',
+    description: 'Whether the group includes terminated roles',
+  },
+  allow_non_employees: {
+    type: 'boolean',
+    description: 'Whether the group allows non-employees',
+  },
+  can_override_role_states: {
+    type: 'boolean',
+    description: 'Whether the group can override role states',
+  },
+  priority: { type: 'number', description: 'Group priority' },
+  is_invisible: { type: 'boolean', description: 'Whether the group is invisible' },
+  ignore_prov_group_matching: {
+    type: 'boolean',
+    description: 'Whether to ignore provisioning group matching',
+  },
 } as const satisfies Record<string, OutputProperty>
 
 /** Group member output properties */
@@ -192,6 +219,9 @@ export const CUSTOM_OBJECT_OUTPUT_PROPERTIES = {
   api_name: { type: 'string', description: 'API name' },
   plural_label: { type: 'string', description: 'Plural label' },
   category_id: { type: 'string', description: 'Category ID' },
+  native_category_id: { type: 'string', description: 'Native category ID' },
+  managed_package_install_id: { type: 'string', description: 'Package install ID' },
+  owner_id: { type: 'string', description: 'Owner ID' },
   enable_history: { type: 'boolean', description: 'Whether history is enabled' },
 } as const satisfies Record<string, OutputProperty>
 
@@ -209,6 +239,7 @@ export const CUSTOM_OBJECT_FIELD_OUTPUT_PROPERTIES = {
   is_immutable: { type: 'boolean', description: 'Whether the field is immutable' },
   is_standard: { type: 'boolean', description: 'Whether the field is standard' },
   enable_history: { type: 'boolean', description: 'Whether history is enabled' },
+  managed_package_install_id: { type: 'string', description: 'Package install ID' },
 } as const satisfies Record<string, OutputProperty>
 
 /** Custom object record output properties */
@@ -228,6 +259,8 @@ export const CUSTOM_APP_OUTPUT_PROPERTIES = {
   name: { type: 'string', description: 'App name' },
   api_name: { type: 'string', description: 'API name' },
   description: { type: 'string', description: 'Description' },
+  icon: { type: 'string', description: 'Icon URL' },
+  pages: { type: 'json', description: 'Array of page summaries' },
 } as const satisfies Record<string, OutputProperty>
 
 /** Custom page output properties */

apps/sim/tools/rippling/update_supergroup_exclusion_members.ts

@@ -44,7 +44,7 @@ export const ripplingUpdateSupergroupExclusionMembersTool: ToolConfig<RipplingUp
         throw new Error(`Rippling API error (${response.status}): ${errorText}`)
       }
       const data = await response.json()
-      return { success: true, output: { ok: data.ok ?? true } }
+      return { success: true, output: { ok: data.ok ?? false } }
     },
     outputs: {
       ok: { type: 'boolean', description: 'Whether the operation succeeded' },

apps/sim/tools/rippling/update_supergroup_inclusion_members.ts

@@ -44,7 +44,7 @@ export const ripplingUpdateSupergroupInclusionMembersTool: ToolConfig<RipplingUp
         throw new Error(`Rippling API error (${response.status}): ${errorText}`)
       }
       const data = await response.json()
-      return { success: true, output: { ok: data.ok ?? true } }
+      return { success: true, output: { ok: data.ok ?? false } }
     },
     outputs: {
       ok: { type: 'boolean', description: 'Whether the operation succeeded' },