fix(outlook): scope online-meeting comments to what Graph documents

onlineMeetingProvider is optional and defaults to unknown; the docs state that
setting isOnlineMeeting alone initializes onlineMeeting. They do not document
Graph substituting the calendar's defaultOnlineMeetingProvider, so the comments
now claim only that, plus the real reason not to pin teamsForBusiness (mailboxes
that disallow it via allowedOnlineMeetingProviders).

Author: waleedlatif1

apps/sim/tools/outlook/calendar-tools.test.ts

@@ -135,8 +135,8 @@ describe('outlook calendar tools', () => {
     expect(built.start).toEqual({ dateTime: '2025-06-03T18:00:00', timeZone: 'UTC' })
     expect(built.attendees).toHaveLength(2)
     expect(built.isOnlineMeeting).toBe(true)
-    // We intentionally do NOT set onlineMeetingProvider so Graph uses the mailbox
-    // default (Teams on work/school accounts) instead of hardcoding a work-only value.
+    // onlineMeetingProvider is optional and pinning it would break mailboxes that
+    // disallow that provider; isOnlineMeeting alone initializes the online meeting.
     expect('onlineMeetingProvider' in built).toBe(false)
   })
 

apps/sim/tools/outlook/calendar_create_event.ts

@@ -155,10 +155,11 @@ export const outlookCalendarCreateEventTool: ToolConfig<
       }
 
       if (toBool(params.isOnlineMeeting)) {
-        // Let Graph use the mailbox's default online-meeting provider (Teams on
-        // work/school accounts) rather than hardcoding teamsForBusiness. Personal
-        // accounts have no supported provider (Skype consumer was retired in 2025),
-        // so joinUrl stays null there regardless.
+        // Setting isOnlineMeeting alone is enough: Graph initializes `onlineMeeting` from
+        // it, and `onlineMeetingProvider` is optional. We deliberately do not pin
+        // teamsForBusiness, since that value is invalid on mailboxes that don't allow it
+        // (see calendar.allowedOnlineMeetingProviders).
+        // https://learn.microsoft.com/en-us/graph/api/resources/event
         event.isOnlineMeeting = true
       }
 

apps/sim/tools/outlook/calendar_update_event.ts

@@ -169,9 +169,9 @@ export const outlookCalendarUpdateEventTool: ToolConfig<
       }
 
       if (params.isOnlineMeeting !== undefined) {
-        // Let Graph use the mailbox's default online-meeting provider (Teams on
-        // work/school accounts) rather than hardcoding teamsForBusiness. Personal
-        // accounts have no supported provider (Skype consumer was retired in 2025).
+        // See calendar_create_event: isOnlineMeeting alone initializes `onlineMeeting`, and
+        // pinning a provider would break mailboxes that disallow it. Note Graph ignores
+        // further changes once a meeting has been made online.
         event.isOnlineMeeting = toBool(params.isOnlineMeeting)
       }