@afosecure/meetingsdk 1.3.0 → 1.3.2

package/dist/index.d.mts

@@ -130,7 +130,7 @@ declare class VideoSDKCore {
     private localStream;
     private screenStream;
     private isScreenSharing;
-    private screenSenders;
+    private peerTransceivers;
     private pingInterval;
     private pendingIceCandidates;
     private pendingOffers;
@@ -160,14 +160,27 @@ declare class VideoSDKCore {
     private handleJoinApproved;
     private handle;
     private createPeer;
+    /**
+     * Capture the screen transceiver's MID after SDP negotiation completes.
+     * The MID is assigned during negotiation and is stable for the life of the connection.
+     */
+    private captureScreenMid;
     private createOffer;
     private shouldInitiate;
     private handleOffer;
     private closePeer;
+    /**
+     * Start screen sharing using replaceTrack on the pre-established screen transceiver.
+     * No need to add/remove tracks, no renegotiation needed (transceiver already in SDP).
+     * Just swap the track and update direction if needed.
+     */
     startScreenShare(): Promise<MediaStream>;
-    stopScreenShare(): void;
+    /**
+     * Stop screen sharing: clear the screen transceiver track and flip direction back to recvonly.
+     */
+    stopScreenShare(): Promise<void>;
     sendChatMessage(payload: ChatInput): void;
-    disconnect(): void;
+    disconnect(): Promise<void>;
     private flushIce;
     private send;
     approveJoinRequest(requestId: string): void;

package/dist/index.d.ts

@@ -130,7 +130,7 @@ declare class VideoSDKCore {
     private localStream;
     private screenStream;
     private isScreenSharing;
-    private screenSenders;
+    private peerTransceivers;
     private pingInterval;
     private pendingIceCandidates;
     private pendingOffers;
@@ -160,14 +160,27 @@ declare class VideoSDKCore {
     private handleJoinApproved;
     private handle;
     private createPeer;
+    /**
+     * Capture the screen transceiver's MID after SDP negotiation completes.
+     * The MID is assigned during negotiation and is stable for the life of the connection.
+     */
+    private captureScreenMid;
     private createOffer;
     private shouldInitiate;
     private handleOffer;
     private closePeer;
+    /**
+     * Start screen sharing using replaceTrack on the pre-established screen transceiver.
+     * No need to add/remove tracks, no renegotiation needed (transceiver already in SDP).
+     * Just swap the track and update direction if needed.
+     */
     startScreenShare(): Promise<MediaStream>;
-    stopScreenShare(): void;
+    /**
+     * Stop screen sharing: clear the screen transceiver track and flip direction back to recvonly.
+     */
+    stopScreenShare(): Promise<void>;
     sendChatMessage(payload: ChatInput): void;
-    disconnect(): void;
+    disconnect(): Promise<void>;
     private flushIce;
     private send;
     approveJoinRequest(requestId: string): void;

package/dist/index.js

@@ -210,7 +210,8 @@ var VideoSDKCore = class {
     this.localStream = null;
     this.screenStream = null;
     this.isScreenSharing = false;
-    this.screenSenders = {};
+    // Transceiver-based tracking: maps peerId -> { cameraTransceiver, screenTransceiver, screenMid }
+    this.peerTransceivers = {};
     this.pingInterval = null;
     this.pendingIceCandidates = {};
     this.pendingOffers = {};
@@ -239,7 +240,7 @@ var VideoSDKCore = class {
     this.joinRejecter = void 0;
     console.error("[MeetingSDK Error]", err);
   }
-  // ---------------- STREAM ----------------
+  // STREAM
   async initLocal(video, name) {
     this.participantName = name;
     try {
@@ -276,7 +277,7 @@ var VideoSDKCore = class {
       throw err;
     }
   }
-  // ---------------- CONNECT ----------------
+  // CONNECT
   async connect(roomId, name) {
     this.room.id = roomId;
     this.reset();
@@ -422,10 +423,11 @@ var VideoSDKCore = class {
       this.pingInterval = null;
     }
   }
-  // ---------------- RESET ----------------
+  // RESET
   reset() {
     Object.values(this.peers).forEach((pc) => pc.close());
     this.peers = {};
+    this.peerTransceivers = {};
     this.initiators.clear();
     this.pendingIceCandidates = {};
     this.state.resetRemoteState();
@@ -480,6 +482,7 @@ var VideoSDKCore = class {
             type: "answer",
             sdp: msg.payload
           });
+          this.captureScreenMid(msg.sender);
           await this.flushIce(msg.sender, pc);
         } catch (err) {
           console.error("[Signaling] Failed to apply answer:", err);
@@ -516,7 +519,6 @@ var VideoSDKCore = class {
         if (msg.presenterId) {
           this.state.setPresenterId(msg.presenterId);
           this.events.onScreenShareStarted?.(msg.presenterId, null);
-          this.state.setPresenterId(msg.presenterId);
         }
         for (const p of msg.participants || []) {
           if (!p?.id || p.id === this.myId) continue;
@@ -599,12 +601,10 @@ var VideoSDKCore = class {
         });
         break;
       }
-      // ============ NEW: HANDLE JOIN_APPROVED WITH RECONNECT ============
       case "JOIN_APPROVED": {
         await this.handleJoinApproved(msg);
         break;
       }
-      // ============ END: JOIN_APPROVED ============
       case "JOIN_REJECTED": {
         const decision = "rejected";
         console.log("JOIN_REJECTED - user not allowed to join");
@@ -657,8 +657,7 @@ var VideoSDKCore = class {
         if (!this.state.presenterId) {
           this.state.setPresenterId(peerId2);
         }
-        const screenStream = this.state.getParticipant(peerId2)?.media?.screenStream;
-        this.events.onScreenShareStarted?.(peerId2, screenStream || null);
+        this.events.onScreenShareStarted?.(peerId2, null);
         break;
       }
       case "SCREEN_SHARE_STOP": {
@@ -685,8 +684,7 @@ var VideoSDKCore = class {
       }
     }
   }
-  // ---------------- PEER ----------------
-  createPeer(id) {
+  async createPeer(id) {
     if (!this.localStream) throw new Error("No local stream");
     if (!this.iceServers || this.iceServers.length === 0) {
       throw new Error(
@@ -694,7 +692,9 @@ var VideoSDKCore = class {
       );
     }
     console.log(
-      "Adding tracks",
+      "Creating peer connection for",
+      id,
+      "with tracks:",
       this.localStream.getTracks().map((t) => ({
         kind: t.kind,
         enabled: t.enabled,
@@ -704,20 +704,52 @@ var VideoSDKCore = class {
     const pc = new RTCPeerConnection({
       iceServers: this.iceServers
     });
+    const audioTransceiver = pc.addTransceiver("audio", {
+      direction: "sendrecv"
+    });
+    const audioTrack = this.localStream.getAudioTracks()[0];
+    if (audioTrack) {
+      await audioTransceiver.sender.replaceTrack(audioTrack);
+    }
+    const cameraTransceiver = pc.addTransceiver("video", {
+      direction: "sendrecv"
+    });
+    const videoTrack = this.localStream.getVideoTracks()[0];
+    if (videoTrack) {
+      await cameraTransceiver.sender.replaceTrack(videoTrack);
+    }
+    const screenTransceiver = pc.addTransceiver("video", {
+      direction: this.isScreenSharing ? "sendrecv" : "recvonly"
+    });
+    if (this.isScreenSharing && this.screenStream) {
+      const screenTrack = this.screenStream.getVideoTracks()[0];
+      if (screenTrack) {
+        await screenTransceiver.sender.replaceTrack(screenTrack);
+      }
+    }
+    this.peerTransceivers[id] = {
+      cameraTransceiver,
+      screenTransceiver,
+      screenMid: null
+      // will be populated after negotiation
+    };
     pc.ontrack = (event) => {
+      const transceiver = event.transceiver;
+      const isScreenTrack = transceiver === this.peerTransceivers[id]?.screenTransceiver;
+      console.log(
+        `[ontrack] ${id}: kind=${event.track.kind}, mid=${transceiver.mid}, isScreen=${isScreenTrack}`
+      );
       const incomingStream = event.streams?.[0] || new MediaStream([event.track]);
-      const participant = this.state.getParticipant(id);
-      const isScreenStream = participant?.media?.isScreenSharing && incomingStream.id === participant?.media?.remoteScreenStreamId;
       if (event.track.muted) {
         event.track.onunmute = () => {
-          console.log(`${event.track.kind} track unmuted for ${id}`);
+          console.log(`[ontrack] ${event.track.kind} track unmuted for ${id}`);
         };
       }
-      if (isScreenStream) {
-        const videoTrack = event.track.kind === "video" ? event.track : incomingStream.getVideoTracks()[0] || participant?.media?.screenTrack;
+      if (isScreenTrack) {
+        const videoTrack2 = event.track.kind === "video" ? event.track : incomingStream.getVideoTracks()[0];
         this.state.updateParticipantMedia(id, {
           screenStream: incomingStream,
-          screenTrack: videoTrack,
+          screenTrack: videoTrack2,
           isScreenSharing: true
         });
         if (!this.state.presenterId) {
@@ -743,29 +775,41 @@ var VideoSDKCore = class {
       });
     };
     pc.oniceconnectionstatechange = () => {
-      console.log(`ICE Connection State: ${pc.iceConnectionState}`);
+      console.log(`[ICE Connection] ${id}: ${pc.iceConnectionState}`);
     };
     pc.onconnectionstatechange = () => {
+      console.log(`[Connection] ${id}: ${pc.connectionState}`);
       if (pc.connectionState === "failed") {
         try {
           pc.restartIce();
-        } catch {
+        } catch (e) {
+          console.warn("Failed to restart ICE:", e);
         }
       }
     };
-    this.localStream.getTracks().forEach((track) => {
-      pc.addTrack(track, this.localStream);
-    });
-    if (this.isScreenSharing && this.screenStream) {
-      this.screenSenders[id] = [];
-      this.screenStream.getTracks().forEach((track) => {
-        const sender = pc.addTrack(track, this.screenStream);
-        this.screenSenders[id].push(sender);
-      });
-    }
     return pc;
   }
-  // ---------------- OFFER ----------------
+  /**
+   * Capture the screen transceiver's MID after SDP negotiation completes.
+   * The MID is assigned during negotiation and is stable for the life of the connection.
+   */
+  captureScreenMid(peerId) {
+    const pc = this.peers[peerId];
+    if (!pc) return;
+    const transceivers = pc.getTransceivers();
+    const screenTransceiver = this.peerTransceivers[peerId]?.screenTransceiver;
+    if (!screenTransceiver) return;
+    const negotiatedTransceiver = transceivers.find(
+      (t) => t === screenTransceiver
+    );
+    if (negotiatedTransceiver?.mid) {
+      this.peerTransceivers[peerId].screenMid = negotiatedTransceiver.mid;
+      console.log(
+        `[Negotiation] Captured screenMid for ${peerId}: ${negotiatedTransceiver.mid}`
+      );
+    }
+  }
+  // OFFER
   async createOffer(id, isRenegotiation = false) {
     if (!isRenegotiation && !this.shouldInitiate(id)) {
       console.debug(
@@ -783,12 +827,13 @@ var VideoSDKCore = class {
       this.initiators.add(id);
     }
     if (!this.peers[id]) {
-      this.peers[id] = this.createPeer(id);
+      this.peers[id] = await this.createPeer(id);
     }
     const pc = this.peers[id];
     try {
       const offer = await pc.createOffer();
       await pc.setLocalDescription(offer);
+      this.captureScreenMid(id);
       this.send({
         type: "OFFER",
         payload: offer.sdp,
@@ -798,12 +843,18 @@ var VideoSDKCore = class {
       console.debug(`[Offer] Sent to ${id}`);
     } catch (err) {
       console.error(`[Offer] Failed for ${id}:`, err);
+      this.emitError(
+        "OFFER_CREATION_FAILED",
+        `Failed to create offer for ${id}`,
+        err,
+        true
+      );
     }
   }
   shouldInitiate(peerId) {
     return this.myId < peerId;
   }
-  // ---------------- ANSWER ----------------
+  // ANSWER
   async handleOffer(sdp, id) {
     if (!this.iceServers || this.iceServers.length === 0) {
       console.warn("[Offer] Waiting for iceServers, queuing offer from", id);
@@ -811,7 +862,7 @@ var VideoSDKCore = class {
       return;
     }
     if (!this.peers[id]) {
-      this.peers[id] = this.createPeer(id);
+      this.peers[id] = await this.createPeer(id);
     }
     const pc = this.peers[id];
     try {
@@ -827,8 +878,9 @@ var VideoSDKCore = class {
           );
           pc.close();
           delete this.peers[id];
+          delete this.peerTransceivers[id];
           this.initiators.delete(id);
-          this.peers[id] = this.createPeer(id);
+          this.peers[id] = await this.createPeer(id);
         }
       }
       if (this.peers[id].signalingState !== "stable" && this.peers[id].signalingState !== "have-local-offer") {
@@ -841,6 +893,7 @@ var VideoSDKCore = class {
         type: "offer",
         sdp
       });
+      this.captureScreenMid(id);
       const pending = this.pendingIceCandidates[id] || [];
       for (const candidate of pending) {
         try {
@@ -870,18 +923,26 @@ var VideoSDKCore = class {
       );
     }
   }
-  // ---------------- CLEANUP ----------------
+  // CLEANUP
   closePeer(id) {
     const pc = this.peers[id];
     if (!pc) return;
     pc.ontrack = null;
     pc.onicecandidate = null;
     pc.onconnectionstatechange = null;
+    pc.oniceconnectionstatechange = null;
     pc.close();
     delete this.peers[id];
+    delete this.peerTransceivers[id];
     this.initiators.delete(id);
     this.state.removeParticipant(id);
   }
+  // SCREEN SHARE (TRANSCEIVER-BASED)
+  /**
+   * Start screen sharing using replaceTrack on the pre-established screen transceiver.
+   * No need to add/remove tracks, no renegotiation needed (transceiver already in SDP).
+   * Just swap the track and update direction if needed.
+   */
   async startScreenShare() {
     try {
       if (this.state.presenterId && this.state.presenterId !== this.myId) {
@@ -892,34 +953,55 @@ var VideoSDKCore = class {
       }
       this.screenStream = await navigator.mediaDevices.getDisplayMedia({
         video: true
-        // audio: true,
       });
+      const screenTrack = this.screenStream.getVideoTracks()[0];
+      if (!screenTrack) {
+        throw new Error("No video track in screen stream");
+      }
       this.isScreenSharing = true;
       this.state.updateLocalParticipant({
         media: {
           isScreenSharing: true,
           screenStream: this.screenStream,
-          screenTrack: this.screenStream.getVideoTracks()[0]
+          screenTrack
         }
       });
       this.state.setPresenterId(this.myId);
-      this.screenStream.getVideoTracks()[0].onended = () => {
+      screenTrack.onended = () => {
+        console.log("[Screen Share] User stopped via browser button");
         this.stopScreenShare();
       };
-      Object.entries(this.peers).forEach(([peerId, pc]) => {
-        this.screenSenders[peerId] = [];
-        this.screenStream.getTracks().forEach((track) => {
-          const sender = pc.addTrack(track, this.screenStream);
-          this.screenSenders[peerId].push(sender);
-        });
-        this.createOffer(peerId, true);
-      });
+      for (const [peerId, pc] of Object.entries(this.peers)) {
+        const txInfo = this.peerTransceivers[peerId];
+        if (!txInfo) {
+          console.warn(
+            `[Screen Share] No transceiver info for ${peerId}, skipping`
+          );
+          continue;
+        }
+        try {
+          await txInfo.screenTransceiver.sender.replaceTrack(screenTrack);
+          if (txInfo.screenTransceiver.currentDirection === "recvonly") {
+            txInfo.screenTransceiver.direction = "sendrecv";
+            console.log(
+              `[Screen Share] Flipped ${peerId} screen transceiver to sendrecv`
+            );
+            await this.createOffer(peerId, true);
+          }
+        } catch (err) {
+          console.error(
+            `[Screen Share] Failed to update transceiver for ${peerId}:`,
+            err
+          );
+        }
+      }
       this.send({
         type: "SCREEN_SHARE_START",
         sender: this.myId,
         room_id: this.room.id,
         stream_id: this.screenStream.id.replace(/[{}]/g, "")
       });
+      console.log("[Screen Share] Started successfully");
       return this.screenStream;
     } catch (err) {
       this.emitError(
@@ -933,21 +1015,32 @@ var VideoSDKCore = class {
       throw err;
     }
   }
-  stopScreenShare() {
+  /**
+   * Stop screen sharing: clear the screen transceiver track and flip direction back to recvonly.
+   */
+  async stopScreenShare() {
     if (!this.screenStream) return;
+    console.log("[Screen Share] Stopping...");
     this.screenStream.getTracks().forEach((t) => t.stop());
-    Object.entries(this.peers).forEach(([peerId, pc]) => {
-      const senders = this.screenSenders[peerId] || [];
-      senders.forEach((sender) => {
-        try {
-          pc.removeTrack(sender);
-        } catch (err) {
-          console.warn(err);
+    for (const [peerId, pc] of Object.entries(this.peers)) {
+      const txInfo = this.peerTransceivers[peerId];
+      if (!txInfo) continue;
+      try {
+        await txInfo.screenTransceiver.sender.replaceTrack(null);
+        if (txInfo.screenTransceiver.currentDirection === "sendrecv") {
+          txInfo.screenTransceiver.direction = "recvonly";
+          console.log(
+            `[Screen Share] Flipped ${peerId} screen transceiver to recvonly`
+          );
+          await this.createOffer(peerId, true);
         }
-      });
-      delete this.screenSenders[peerId];
-      this.createOffer(peerId, true);
-    });
+      } catch (err) {
+        console.error(
+          `[Screen Share] Failed to clear transceiver for ${peerId}:`,
+          err
+        );
+      }
+    }
     this.screenStream = null;
     this.isScreenSharing = false;
     this.state.updateLocalParticipant({
@@ -965,7 +1058,9 @@ var VideoSDKCore = class {
       sender: this.myId,
       room_id: this.room.id
     });
+    console.log("[Screen Share] Stopped");
   }
+  // CHAT
   sendChatMessage(payload) {
     if (!this.ws || this.ws.readyState !== WebSocket.OPEN) {
       console.warn("WS not connected");
@@ -998,11 +1093,13 @@ var VideoSDKCore = class {
       client_ts: Date.now()
     });
   }
-  disconnect() {
+  // DISCONNECT
+  async disconnect() {
     this.intentionalDisconnect = true;
-    this.stopScreenShare();
+    await this.stopScreenShare();
     Object.values(this.peers).forEach((pc) => pc.close());
     this.peers = {};
+    this.peerTransceivers = {};
     this.initiators.clear();
     this.stopHeartbeat();
     if (this.ws?.readyState === WebSocket.OPEN) {