@stream-io/video-client 0.0.30 → 0.0.31

package/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 This file was generated using [@jscutlery/semver](https://github.com/jscutlery/semver).
 
+### [0.0.31](https://github.com/GetStream/stream-video-js/compare/client0.0.30...client0.0.31) (2023-07-03)
+
+
+### Bug Fixes
+
+* safeguard against potential race conditions during join-flow ([#741](https://github.com/GetStream/stream-video-js/issues/741)) ([54f1ef6](https://github.com/GetStream/stream-video-js/commit/54f1ef636d3c46b29c538a8c2c7bc5031fde43c9))
+
 ### [0.0.30](https://github.com/GetStream/stream-video-js/compare/client0.0.29...client0.0.30) (2023-07-03)
 
 

package/dist/index.browser.es.js

@@ -8162,38 +8162,45 @@ const watchCallMemberUpdated = (state) => {
 /**
  * An event responder which handles the `participantJoined` event.
  */
-const watchParticipantJoined = (dispatcher, state) => {
-    return dispatcher.on('participantJoined', (e) => {
+const watchParticipantJoined = (state) => {
+    return function onParticipantJoined(e) {
         if (e.eventPayload.oneofKind !== 'participantJoined')
             return;
         const { participant } = e.eventPayload.participantJoined;
         if (!participant)
             return;
-        state.setParticipants((participants) => [
-            ...participants,
-            Object.assign(Object.assign({}, participant), { viewportVisibilityState: VisibilityState.UNKNOWN }),
-        ]);
-    });
+        // `state.updateOrAddParticipant` acts as a safeguard against
+        // potential duplicate events from the SFU.
+        //
+        // Although the SFU should not send duplicate events, we have seen
+        // some race conditions in the past during the `join-flow` where
+        // the SFU would send participant info as part of the `join`
+        // response and then follow up with a `participantJoined` event for
+        // already announced participants.
+        state.updateOrAddParticipant(participant.sessionId, Object.assign(participant, {
+            viewportVisibilityState: VisibilityState.UNKNOWN,
+        }));
+    };
 };
 /**
  * An event responder which handles the `participantLeft` event.
  */
-const watchParticipantLeft = (dispatcher, state) => {
-    return dispatcher.on('participantLeft', (e) => {
+const watchParticipantLeft = (state) => {
+    return function onParticipantLeft(e) {
         if (e.eventPayload.oneofKind !== 'participantLeft')
             return;
         const { participant } = e.eventPayload.participantLeft;
         if (!participant)
             return;
         state.setParticipants((participants) => participants.filter((p) => p.sessionId !== participant.sessionId));
-    });
+    };
 };
 /**
  * An event responder which handles the `trackPublished` event.
  * The SFU will send this event when a participant publishes a track.
  */
-const watchTrackPublished = (dispatcher, state) => {
-    return dispatcher.on('trackPublished', (e) => {
+const watchTrackPublished = (state) => {
+    return function onTrackPublished(e) {
         if (e.eventPayload.oneofKind !== 'trackPublished')
             return;
         const { trackPublished: { type, sessionId, participant }, } = e.eventPayload;
@@ -8202,34 +8209,34 @@ const watchTrackPublished = (dispatcher, state) => {
         // events, and instead, it would only provide the participant's information
         // once they start publishing a track.
         if (participant) {
-            state.updateOrAddParticipant(participant.sessionId, participant);
+            state.updateOrAddParticipant(sessionId, participant);
         }
         else {
             state.updateParticipant(sessionId, (p) => ({
                 publishedTracks: [...p.publishedTracks, type].filter(unique),
             }));
         }
-    });
+    };
 };
 /**
  * An event responder which handles the `trackUnpublished` event.
  * The SFU will send this event when a participant unpublishes a track.
  */
-const watchTrackUnpublished = (dispatcher, state) => {
-    return dispatcher.on('trackUnpublished', (e) => {
+const watchTrackUnpublished = (state) => {
+    return function onTrackUnpublished(e) {
         if (e.eventPayload.oneofKind !== 'trackUnpublished')
             return;
         const { trackUnpublished: { type, sessionId, participant }, } = e.eventPayload;
         // An optimization for large calls. See `watchTrackPublished`.
         if (participant) {
-            state.updateOrAddParticipant(participant.sessionId, participant);
+            state.updateOrAddParticipant(sessionId, participant);
         }
         else {
             state.updateParticipant(sessionId, (p) => ({
                 publishedTracks: p.publishedTracks.filter((t) => t !== type),
             }));
         }
-    });
+    };
 };
 const unique = (v, i, arr) => arr.indexOf(v) === i;
 
@@ -8477,10 +8484,10 @@ const registerEventHandlers = (call, state, dispatcher) => {
         watchChangePublishQuality(dispatcher, call),
         watchConnectionQualityChanged(dispatcher, state),
         watchParticipantCountChanged(dispatcher, state),
-        watchParticipantJoined(dispatcher, state),
-        watchParticipantLeft(dispatcher, state),
-        watchTrackPublished(dispatcher, state),
-        watchTrackUnpublished(dispatcher, state),
+        call.on('participantJoined', watchParticipantJoined(state)),
+        call.on('participantLeft', watchParticipantLeft(state)),
+        call.on('trackPublished', watchTrackPublished(state)),
+        call.on('trackUnpublished', watchTrackUnpublished(state)),
         watchAudioLevelChanged(dispatcher, state),
         watchDominantSpeakerChanged(dispatcher, state),
         call.on('callGrantsUpdated', watchCallGrantsUpdated(state)),
@@ -11871,7 +11878,7 @@ class StreamClient {
     }
     getUserAgent() {
         return (this.userAgent ||
-            `stream-video-javascript-client-${this.node ? 'node' : 'browser'}-${"0.0.29"}`);
+            `stream-video-javascript-client-${this.node ? 'node' : 'browser'}-${"0.0.30"}`);
     }
     setUserAgent(userAgent) {
         this.userAgent = userAgent;