@defcomm/meetingsdk 0.3.1 → 0.3.2

package/dist/index.d.mts

@@ -219,7 +219,7 @@ declare class VideoSDKCore {
     rejectJoinRequest(requestId: string, reason?: string): Promise<void>;
     toggleMic(): void;
     toggleCam(): void;
-    startScreenShare(): Promise<MediaStream>;
+    startScreenShare(): Promise<MediaStream | null>;
     stopScreenShare(): Promise<void>;
     startRecording(): Promise<void>;
     stopRecording(): Promise<void>;
@@ -436,7 +436,7 @@ declare class SFUClient {
         video?: boolean | undefined;
     }): Promise<MediaStream>;
     private publishExistingTracks;
-    shareScreen(): Promise<MediaStream>;
+    shareScreen(): Promise<MediaStream | null>;
     stopScreenShare(): Promise<void>;
     muteAudio(muted: boolean): void;
     muteVideo(muted: boolean): void;

package/dist/index.d.ts

@@ -219,7 +219,7 @@ declare class VideoSDKCore {
     rejectJoinRequest(requestId: string, reason?: string): Promise<void>;
     toggleMic(): void;
     toggleCam(): void;
-    startScreenShare(): Promise<MediaStream>;
+    startScreenShare(): Promise<MediaStream | null>;
     stopScreenShare(): Promise<void>;
     startRecording(): Promise<void>;
     stopRecording(): Promise<void>;
@@ -436,7 +436,7 @@ declare class SFUClient {
         video?: boolean | undefined;
     }): Promise<MediaStream>;
     private publishExistingTracks;
-    shareScreen(): Promise<MediaStream>;
+    shareScreen(): Promise<MediaStream | null>;
     stopScreenShare(): Promise<void>;
     muteAudio(muted: boolean): void;
     muteVideo(muted: boolean): void;

package/dist/index.js

@@ -1362,11 +1362,14 @@ var SFUClient = class {
             sdp: answer.sdp
           }
         });
-        const waiters = this.subscriptionWaiters.splice(0);
-        waiters.forEach((w) => w.resolve());
+        const waiter = this.subscriptionWaiters.shift();
+        waiter?.resolve();
       } catch (error) {
-        const waiters = this.subscriptionWaiters.splice(0);
-        waiters.forEach((w) => w.reject(error));
+        const waiter = this.subscriptionWaiters.shift();
+        if (waiter) {
+          waiter.trackIds.forEach((id) => this.subscribedTracks.delete(id));
+          waiter.reject(error);
+        }
         this.onError(
           new Error(`Failed to process subscriber offer: ${String(error)}`)
         );
@@ -1388,11 +1391,20 @@ var SFUClient = class {
     if (!uniqueIds.length || !this.isSocketOpen()) return;
     uniqueIds.forEach((id) => this.subscribedTracks.add(id));
     return new Promise((resolve, reject) => {
-      this.subscriptionWaiters.push({ resolve, reject });
-      this.send({
+      const sent = this.send({
         type: "subscribe",
         payload: { track_ids: uniqueIds }
       });
+      if (!sent) {
+        uniqueIds.forEach((id) => this.subscribedTracks.delete(id));
+        reject(new Error("Signaling socket is not open"));
+        return;
+      }
+      this.subscriptionWaiters.push({
+        trackIds: uniqueIds,
+        resolve,
+        reject
+      });
     });
   }
   async processSubscriptionQueue() {
@@ -1405,11 +1417,20 @@ var SFUClient = class {
           continue;
         }
         await new Promise((resolve, reject) => {
-          this.subscriptionWaiters.push({ resolve, reject });
-          this.send({
+          const sent = this.send({
             type: "subscribe",
             payload: { track_ids: [trackId] }
           });
+          if (!sent) {
+            this.subscribedTracks.delete(trackId);
+            reject(new Error("Signaling socket is not open"));
+            return;
+          }
+          this.subscriptionWaiters.push({
+            trackIds: [trackId],
+            resolve,
+            reject
+          });
         });
       }
     } finally {
@@ -1467,9 +1488,16 @@ var SFUClient = class {
     if (!this.screenTransceiver || !this.screenSender) {
       throw new Error("Screen transceiver is not initialized");
     }
-    if (this.screenStream) {
-      return this.screenStream;
+    const existing = this.screenStream;
+    const existingTrack = this.screenSender.track;
+    if (existing || existingTrack && existingTrack.readyState === "live") {
+      return existing;
     }
+    try {
+      await this.screenSender.replaceTrack(null);
+    } catch {
+    }
+    this.screenTransceiver.direction = "inactive";
     const screenStream = await navigator.mediaDevices.getDisplayMedia({
       video: true,
       audio: false
@@ -1479,29 +1507,33 @@ var SFUClient = class {
       screenStream.getTracks().forEach((track) => track.stop());
       throw new Error("Screen share returned no video track");
     }
+    const generation = ++this.screenShareGeneration;
     this.screenStream = screenStream;
     try {
       await this.screenSender.replaceTrack(screenTrack);
       this.screenTransceiver.direction = "sendonly";
+      this.send({
+        type: "start_screen_share"
+      });
       const offer = await this.publisher.createOffer();
       await this.publisher.setLocalDescription(offer);
+      const localDescription = this.publisher.localDescription;
+      if (!localDescription?.sdp) {
+        throw new Error("Publisher did not produce a valid screen-share SDP");
+      }
       this.send({
         type: "publish_offer",
         payload: {
           revision: ++this.publisherRevision,
-          sdp: offer.sdp
+          sdp: localDescription.sdp
         }
       });
-      this.send({ type: "start_screen_share" });
       screenTrack.onended = () => {
+        if (generation !== this.screenShareGeneration) {
+          return;
+        }
         void this.stopScreenShare();
       };
-      console.info("[publisher] Screen track attached", {
-        screenTrackId: screenTrack.id,
-        cameraTrackId: this.videoSender?.track?.id,
-        screenMid: this.screenTransceiver.mid,
-        cameraMid: this.cameraTransceiver?.mid
-      });
       return this.screenStream;
     } catch (error) {
       try {
@@ -1509,19 +1541,33 @@ var SFUClient = class {
       } catch {
       }
       this.screenTransceiver.direction = "inactive";
-      screenStream.getTracks().forEach((track) => track.stop());
+      screenTrack.onended = null;
+      screenStream.getTracks().forEach((track) => {
+        try {
+          track.stop();
+        } catch {
+        }
+      });
       this.screenStream = null;
       try {
-        this.send({ type: "stop_screen_share" });
+        this.send({
+          type: "stop_screen_share"
+        });
       } catch {
       }
       throw error;
     }
   }
   async stopScreenShare() {
-    if (!this.screenStream && !this.screenSender?.track) return;
-    const screenTrack = this.screenSender?.track || null;
-    if (screenTrack) screenTrack.onended = null;
+    const stream = this.screenStream;
+    const senderTrack = this.screenSender?.track || null;
+    if (!stream && !senderTrack) {
+      return;
+    }
+    ++this.screenShareGeneration;
+    if (senderTrack) {
+      senderTrack.onended = null;
+    }
     try {
       await this.screenSender?.replaceTrack(null);
     } catch (error) {
@@ -1530,23 +1576,33 @@ var SFUClient = class {
     if (this.screenTransceiver) {
       this.screenTransceiver.direction = "inactive";
     }
-    this.send({ type: "stop_screen_share" });
     try {
-      const offer = await this.publisher?.createOffer();
-      if (offer && this.publisher) {
+      if (this.publisher) {
+        const offer = await this.publisher.createOffer();
         await this.publisher.setLocalDescription(offer);
-        this.send({
-          type: "publish_offer",
-          payload: {
-            revision: ++this.publisherRevision,
-            sdp: offer.sdp
-          }
-        });
+        const localDescription = this.publisher.localDescription;
+        if (localDescription?.sdp) {
+          this.send({
+            type: "publish_offer",
+            payload: {
+              revision: ++this.publisherRevision,
+              sdp: localDescription.sdp
+            }
+          });
+        }
       }
     } catch (error) {
       this.onError(error);
     }
-    this.screenStream?.getTracks().forEach((track) => track.stop());
+    this.send({
+      type: "stop_screen_share"
+    });
+    stream?.getTracks().forEach((track) => {
+      try {
+        track.stop();
+      } catch {
+      }
+    });
     this.screenStream = null;
     if (this.currentParticipantId) {
       this.onScreenShareStopped(this.currentParticipantId);