fix(microsoft-ad): prevent empty values leaking into PATCH requests

- Use operation-aware checks for accountEnabled and visibility in block
  params to prevent create defaults bleeding into update operations
- Change tool body guards from `!== undefined` to truthy checks so
  empty-string inputs from unfilled subBlocks are omitted from PATCH

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

Author: waleedlatif1

apps/sim/blocks/blocks/microsoft_ad.ts

@@ -333,8 +333,12 @@ export const MicrosoftAdBlock: BlockConfig<MicrosoftAdResponse> = {
         if (params.top) result.top = Number(params.top)
         if (params.filter) result.filter = params.filter
         if (params.search) result.search = params.search
-        const accountEnabledVal = params.accountEnabled || params.accountEnabledCreate
-        if (accountEnabledVal) result.accountEnabled = accountEnabledVal === 'true'
+        if (params.operation === 'update_user') {
+          if (params.accountEnabled) result.accountEnabled = params.accountEnabled === 'true'
+        } else if (params.operation === 'create_user') {
+          if (params.accountEnabledCreate)
+            result.accountEnabled = params.accountEnabledCreate === 'true'
+        }
         if (params.mailEnabled !== undefined) result.mailEnabled = params.mailEnabled === 'true'
         if (params.securityEnabled !== undefined)
           result.securityEnabled = params.securityEnabled === 'true'
@@ -343,8 +347,11 @@ export const MicrosoftAdBlock: BlockConfig<MicrosoftAdResponse> = {
         if (params.groupMailNickname) result.mailNickname = params.groupMailNickname
         if (params.groupDescription) result.description = params.groupDescription
         if (params.groupTypes !== undefined) result.groupTypes = params.groupTypes
-        const visibilityVal = params.visibility || params.visibilityCreate
-        if (visibilityVal) result.visibility = visibilityVal
+        if (params.operation === 'update_group') {
+          if (params.visibility) result.visibility = params.visibility
+        } else if (params.operation === 'create_group') {
+          if (params.visibilityCreate) result.visibility = params.visibilityCreate
+        }
         return result
       },
     },

apps/sim/tools/microsoft_ad/update_group.ts

@@ -68,10 +68,10 @@ export const updateGroupTool: ToolConfig<
     }),
     body: (params) => {
       const body: Record<string, unknown> = {}
-      if (params.displayName !== undefined) body.displayName = params.displayName
-      if (params.description !== undefined) body.description = params.description
-      if (params.mailNickname !== undefined) body.mailNickname = params.mailNickname
-      if (params.visibility !== undefined) body.visibility = params.visibility
+      if (params.displayName) body.displayName = params.displayName
+      if (params.description) body.description = params.description
+      if (params.mailNickname) body.mailNickname = params.mailNickname
+      if (params.visibility) body.visibility = params.visibility
       return body
     },
   },

apps/sim/tools/microsoft_ad/update_user.ts

@@ -92,13 +92,13 @@ export const updateUserTool: ToolConfig<
     }),
     body: (params) => {
       const body: Record<string, unknown> = {}
-      if (params.displayName !== undefined) body.displayName = params.displayName
-      if (params.givenName !== undefined) body.givenName = params.givenName
-      if (params.surname !== undefined) body.surname = params.surname
-      if (params.jobTitle !== undefined) body.jobTitle = params.jobTitle
-      if (params.department !== undefined) body.department = params.department
-      if (params.officeLocation !== undefined) body.officeLocation = params.officeLocation
-      if (params.mobilePhone !== undefined) body.mobilePhone = params.mobilePhone
+      if (params.displayName) body.displayName = params.displayName
+      if (params.givenName) body.givenName = params.givenName
+      if (params.surname) body.surname = params.surname
+      if (params.jobTitle) body.jobTitle = params.jobTitle
+      if (params.department) body.department = params.department
+      if (params.officeLocation) body.officeLocation = params.officeLocation
+      if (params.mobilePhone) body.mobilePhone = params.mobilePhone
       if (params.accountEnabled !== undefined) body.accountEnabled = params.accountEnabled
       return body
     },