fix(okta): prevent silent data loss in update operations

- update_group: always include description in PUT body (defaults to '')
  since PUT replaces the full profile object
- update_user: use !== undefined checks so empty strings can clear fields
  via Okta's POST partial update
- block: allow empty strings through passthrough loop and use !== undefined
  for groupDescription mapping

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: waleedlatif1

apps/sim/blocks/blocks/okta.ts

@@ -289,9 +289,10 @@ export const OktaBlock: BlockConfig<OktaResponse> = {
 
         // Map group-specific UI fields to tool param names
         if (params.groupName) result.name = params.groupName
-        if (params.groupDescription) result.description = params.groupDescription
+        if (params.groupDescription !== undefined) result.description = params.groupDescription
 
         // Pass through all other non-empty params
+        // Allow empty strings so users can clear fields (e.g. update_user partial updates)
         const skipKeys = new Set([
           'operation',
           'apiKey',
@@ -301,7 +302,7 @@ export const OktaBlock: BlockConfig<OktaResponse> = {
           'groupDescription',
         ])
         for (const [key, value] of Object.entries(params)) {
-          if (!skipKeys.has(key) && value !== undefined && value !== null && value !== '') {
+          if (!skipKeys.has(key) && value !== undefined && value !== null) {
             result[key] = value
           }
         }

apps/sim/tools/okta/update_group.ts

@@ -61,11 +61,12 @@ export const oktaUpdateGroupTool: ToolConfig<OktaUpdateGroupParams, OktaUpdateGr
       Accept: 'application/json',
       'Content-Type': 'application/json',
     }),
-    body: (params) => {
-      const profile: Record<string, string> = { name: params.name }
-      if (params.description) profile.description = params.description
-      return { profile }
-    },
+    body: (params) => ({
+      profile: {
+        name: params.name,
+        description: params.description ?? '',
+      },
+    }),
   },
 
   transformResponse: async (response: Response) => {

apps/sim/tools/okta/update_user.ts

@@ -93,13 +93,13 @@ export const oktaUpdateUserTool: ToolConfig<OktaUpdateUserParams, OktaUpdateUser
     body: (params) => {
       const profile: Record<string, string> = {}
 
-      if (params.firstName) profile.firstName = params.firstName
-      if (params.lastName) profile.lastName = params.lastName
-      if (params.email) profile.email = params.email
-      if (params.login) profile.login = params.login
-      if (params.mobilePhone) profile.mobilePhone = params.mobilePhone
-      if (params.title) profile.title = params.title
-      if (params.department) profile.department = params.department
+      if (params.firstName !== undefined) profile.firstName = params.firstName
+      if (params.lastName !== undefined) profile.lastName = params.lastName
+      if (params.email !== undefined) profile.email = params.email
+      if (params.login !== undefined) profile.login = params.login
+      if (params.mobilePhone !== undefined) profile.mobilePhone = params.mobilePhone
+      if (params.title !== undefined) profile.title = params.title
+      if (params.department !== undefined) profile.department = params.department
 
       return { profile }
     },