Fix return to main meeting from breakout room and notifications after joining breakout room (#6037)

* Fix returnFromBreakoutRoom to use BreakoutRooms feature for assigned breakout room

* Refactor CallSubscriber to always clear assigned breakout room notifications and initialize BreakoutRoomsSubscriber unconditionally

* Change files

Author: mgamis-msft

change/@azure-communication-react-645519bf-0029-4580-ba7c-49ae4c988845.json

@@ -0,0 +1,9 @@
+{
+  "type": "patch",
+  "area": "fix",
+  "workstream": "Breakout rooms",
+  "comment": "Fix return to main meeting from breakout room. Fix breakout room notifications after joining breakout room.",
+  "packageName": "@azure/communication-react",
+  "email": "miguelgamis@microsoft.com",
+  "dependentChangeType": "patch"
+}

packages/calling-stateful-client/src/CallSubscriber.ts

@@ -120,15 +120,12 @@ export class CallSubscriber {
       this._call.feature(Features.Spotlight)
     );
 
-    if (this._call.feature(Features.Capabilities) && this._call.feature(Features.Capabilities).capabilities?.joinBreakoutRooms.isPresent) {
-      // Clear assigned breakout room closed notification for this call.
-      this._context.deleteLatestNotification('assignedBreakoutRoomClosed');
-      this._breakoutRoomsSubscriber = new BreakoutRoomsSubscriber(
-        this._callIdRef,
-        this._context,
-        this._call.feature(Features.BreakoutRooms)
-      );
-    }
+    this._context.deleteLatestNotification('assignedBreakoutRoomClosed');
+    this._breakoutRoomsSubscriber = new BreakoutRoomsSubscriber(
+      this._callIdRef,
+      this._context,
+      this._call.feature(Features.BreakoutRooms)
+    );
 
     this._togetherModeSubscriber = new TogetherModeSubscriber(
       this._callIdRef,
@@ -256,9 +253,7 @@ export class CallSubscriber {
     this._capabilitiesSubscriber.unsubscribe();
     this._reactionSubscriber?.unsubscribe();
     this._spotlightSubscriber.unsubscribe();
-    if (this._call.feature(Features.Capabilities) && this._call.feature(Features.Capabilities).capabilities?.joinBreakoutRooms.isPresent && this._breakoutRoomsSubscriber) {
-      this._breakoutRoomsSubscriber.unsubscribe();
-    }
+    this._breakoutRoomsSubscriber?.unsubscribe();
     this._togetherModeSubscriber.unsubscribe();
     this._mediaAccessSubscriber.unsubscribe();
   };

packages/react-composites/src/composites/CallComposite/adapter/AzureCommunicationCallAdapter.ts

@@ -1230,8 +1230,10 @@ export class AzureCommunicationCallAdapter<AgentType extends CallAgent | TeamsCa
   }
 
   public async returnFromBreakoutRoom(): Promise<void> {
-    const callState = this.call?.id ? this.callClient.getState().calls[this.call.id] : undefined;
-    const assignedBreakoutRoom = callState?.breakoutRooms?.assignedBreakoutRoom;
+    // Get assignedBreakoutRoom directly from the SDK's BreakoutRooms feature on the originCall (main meeting).
+    // We can't rely on state because the main meeting's call state may not exist in calls when in a breakout room.
+    const breakoutRoomsFeature = this.originCall?.feature(Features.BreakoutRooms);
+    const assignedBreakoutRoom = breakoutRoomsFeature?.assignedBreakoutRooms;
 
     if (!assignedBreakoutRoom) {
       throw new Error(