@stream-io/video-client 1.18.5 → 1.18.7

package/CHANGELOG.md

@@ -2,6 +2,21 @@
 
 This file was generated using [@jscutlery/semver](https://github.com/jscutlery/semver).
 
+## [1.18.7](https://github.com/GetStream/stream-video-js/compare/@stream-io/video-client-1.18.6...@stream-io/video-client-1.18.7) (2025-03-20)
+
+
+### Bug Fixes
+
+* rename `toJSON` to `asJSON` ([#1729](https://github.com/GetStream/stream-video-js/issues/1729)) ([0d7d074](https://github.com/GetStream/stream-video-js/commit/0d7d074dac1032690b5f4af4d6ba5fcdd56dfaa2))
+* update call reject reasons ([#1730](https://github.com/GetStream/stream-video-js/issues/1730)) ([100ed6b](https://github.com/GetStream/stream-video-js/commit/100ed6b9323b66e86123917abf4fc2973a677fca))
+
+## [1.18.6](https://github.com/GetStream/stream-video-js/compare/@stream-io/video-client-1.18.5...@stream-io/video-client-1.18.6) (2025-03-13)
+
+
+### Bug Fixes
+
+* ensure negotiation runs sequentially ([#1722](https://github.com/GetStream/stream-video-js/issues/1722)) ([7e166aa](https://github.com/GetStream/stream-video-js/commit/7e166aaf606c3f751068cf60bd554e6374f701d7))
+
 ## [1.18.5](https://github.com/GetStream/stream-video-js/compare/@stream-io/video-client-1.18.4...@stream-io/video-client-1.18.5) (2025-03-12)
 
 

package/dist/index.browser.es.js

@@ -5247,7 +5247,7 @@ class BasePeerConnection {
                 this.logger('debug', 'null ice candidate');
                 return;
             }
-            const iceCandidate = this.toJSON(candidate);
+            const iceCandidate = this.asJSON(candidate);
             this.sfuClient
                 .iceTrickle({ peerType: this.peerType, iceCandidate })
                 .catch((err) => {
@@ -5259,7 +5259,7 @@ class BasePeerConnection {
         /**
          * Converts the ICE candidate to a JSON string.
          */
-        this.toJSON = (candidate) => {
+        this.asJSON = (candidate) => {
             if (!candidate.usernameFragment) {
                 // react-native-webrtc doesn't include usernameFragment in the candidate
                 const segments = candidate.candidate.split(' ');
@@ -5347,8 +5347,6 @@ class BasePeerConnection {
         this.pc.removeEventListener('icecandidateerror', this.onIceCandidateError);
         this.pc.removeEventListener('signalingstatechange', this.onSignalingChange);
         this.pc.removeEventListener('iceconnectionstatechange', this.onIceConnectionStateChange);
-        // cancel any ongoing ICE restart process
-        withCancellation('onIceConnectionStateChange', () => Promise.resolve());
         this.pc.removeEventListener('icegatheringstatechange', this.onIceGatherChange);
         this.unsubscribeIceTrickle?.();
         this.subscriptions.forEach((unsubscribe) => unsubscribe());
@@ -5378,12 +5376,6 @@ class TransceiverCache {
         this.get = (publishOption) => {
             return this.findTransceiver(publishOption)?.transceiver;
         };
-        /**
-         * Gets the last transceiver for the given track type and publish option id.
-         */
-        this.getWith = (trackType, id) => {
-            return this.findTransceiver({ trackType, id })?.transceiver;
-        };
         /**
          * Checks if the cache has the given publish option.
          */
@@ -5697,7 +5689,7 @@ class Publisher extends BasePeerConnection {
                 const trackToPublish = this.cloneTrack(track);
                 const transceiver = this.transceiverCache.get(publishOption);
                 if (!transceiver) {
-                    this.addTransceiver(trackToPublish, publishOption);
+                    await this.addTransceiver(trackToPublish, publishOption);
                 }
                 else {
                     const previousTrack = transceiver.sender.track;
@@ -5711,7 +5703,7 @@ class Publisher extends BasePeerConnection {
         /**
          * Adds a new transceiver carrying the given track to the peer connection.
          */
-        this.addTransceiver = (track, publishOption) => {
+        this.addTransceiver = async (track, publishOption) => {
             const videoEncodings = computeVideoLayers(track, publishOption);
             const sendEncodings = isSvcCodec(publishOption.codec?.name)
                 ? toSvcEncodings(videoEncodings)
@@ -5723,6 +5715,7 @@ class Publisher extends BasePeerConnection {
             const trackType = publishOption.trackType;
             this.logger('debug', `Added ${TrackType[trackType]} transceiver`);
             this.transceiverCache.add(publishOption, transceiver);
+            await this.negotiate();
         };
         /**
          * Synchronizes the current Publisher state with the provided publish options.
@@ -5742,7 +5735,7 @@ class Publisher extends BasePeerConnection {
                 // take the track from the existing transceiver for the same track type,
                 // clone it and publish it with the new publish options
                 const track = this.cloneTrack(item.transceiver.sender.track);
-                this.addTransceiver(track, publishOption);
+                await this.addTransceiver(track, publishOption);
             }
             // stop publishing with options not required anymore -> [vp9]
             for (const item of this.transceiverCache.items()) {
@@ -5812,7 +5805,9 @@ class Publisher extends BasePeerConnection {
             const enabledLayers = layers.filter((l) => l.active);
             const tag = 'Update publish quality:';
             this.logger('info', `${tag} requested layers by SFU:`, enabledLayers);
-            const sender = this.transceiverCache.getWith(trackType, publishOptionId)?.sender;
+            const transceiverId = this.transceiverCache.find((t) => t.publishOption.id === publishOptionId &&
+                t.publishOption.trackType === trackType);
+            const sender = transceiverId?.transceiver.sender;
             if (!sender) {
                 return this.logger('warn', `${tag} no video sender found.`);
             }
@@ -5820,8 +5815,8 @@ class Publisher extends BasePeerConnection {
             if (params.encodings.length === 0) {
                 return this.logger('warn', `${tag} there are no encodings set.`);
             }
-            const [codecInUse] = params.codecs;
-            const usesSvcCodec = codecInUse && isSvcCodec(codecInUse.mimeType);
+            const codecInUse = transceiverId?.publishOption.codec?.name;
+            const usesSvcCodec = codecInUse && isSvcCodec(codecInUse);
             let changed = false;
             for (const encoder of params.encodings) {
                 const layer = usesSvcCodec
@@ -5879,40 +5874,32 @@ class Publisher extends BasePeerConnection {
             }
             await this.negotiate({ iceRestart: true });
         };
-        this.onNegotiationNeeded = () => {
-            withCancellation('publisher.negotiate', (signal) => this.negotiate().catch((err) => {
-                if (signal.aborted)
-                    return;
-                this.logger('error', `Negotiation failed.`, err);
-                this.onUnrecoverableError?.();
-            }));
-        };
         /**
          * Initiates a new offer/answer exchange with the currently connected SFU.
          *
          * @param options the optional offer options to use.
          */
         this.negotiate = async (options) => {
-            const offer = await this.pc.createOffer(options);
-            const trackInfos = this.getAnnouncedTracks(offer.sdp);
-            if (trackInfos.length === 0) {
-                throw new Error(`Can't negotiate without announcing any tracks`);
-            }
-            try {
-                this.isIceRestarting = options?.iceRestart ?? false;
-                await this.pc.setLocalDescription(offer);
-                const { response } = await this.sfuClient.setPublisher({
-                    sdp: offer.sdp || '',
-                    tracks: trackInfos,
-                });
-                if (response.error)
-                    throw new Error(response.error.message);
-                await this.pc.setRemoteDescription({ type: 'answer', sdp: response.sdp });
-            }
-            finally {
-                this.isIceRestarting = false;
-            }
-            this.addTrickledIceCandidates();
+            return withoutConcurrency('publisher.negotiate', async () => {
+                const offer = await this.pc.createOffer(options);
+                const tracks = this.getAnnouncedTracks(offer.sdp);
+                if (!tracks.length)
+                    throw new Error(`Can't negotiate without any tracks`);
+                try {
+                    this.isIceRestarting = options?.iceRestart ?? false;
+                    await this.pc.setLocalDescription(offer);
+                    const { sdp = '' } = offer;
+                    const { response } = await this.sfuClient.setPublisher({ sdp, tracks });
+                    if (response.error)
+                        throw new Error(response.error.message);
+                    const { sdp: answerSdp } = response;
+                    await this.pc.setRemoteDescription({ type: 'answer', sdp: answerSdp });
+                }
+                finally {
+                    this.isIceRestarting = false;
+                }
+                this.addTrickledIceCandidates();
+            });
         };
         /**
          * Returns a list of tracks that are currently being published.
@@ -5996,7 +5983,6 @@ class Publisher extends BasePeerConnection {
             this.clonedTracks.delete(track);
         };
         this.publishOptions = publishOptions;
-        this.pc.addEventListener('negotiationneeded', this.onNegotiationNeeded);
         this.on('iceRestart', (iceRestart) => {
             if (iceRestart.peerType !== PeerType.PUBLISHER_UNSPECIFIED)
                 return;
@@ -6015,17 +6001,6 @@ class Publisher extends BasePeerConnection {
             return this.syncPublishOptions();
         });
     }
-    /**
-     * Detaches the event handlers from the `RTCPeerConnection`.
-     * This is useful when we want to replace the `RTCPeerConnection`
-     * instance with a new one (in case of migration).
-     */
-    detachEventHandlers() {
-        super.detachEventHandlers();
-        this.pc.removeEventListener('negotiationneeded', this.onNegotiationNeeded);
-        // abort any ongoing negotiation
-        withCancellation('publisher.negotiate', () => Promise.resolve());
-    }
     /**
      * Disposes this Publisher instance.
      */
@@ -7457,7 +7432,7 @@ const aggregate = (stats) => {
     return report;
 };
 
-const version = "1.18.5";
+const version = "1.18.7";
 const [major, minor, patch] = version.split('.');
 let sdkInfo = {
     type: SdkType.PLAIN_JAVASCRIPT,
@@ -10428,7 +10403,7 @@ class Call {
                 }
                 if (callingState === CallingState.RINGING && reject !== false) {
                     if (reject) {
-                        await this.reject(reason);
+                        await this.reject('decline');
                     }
                     else {
                         // if reject was undefined, we still have to cancel the call automatically
@@ -10458,6 +10433,7 @@ class Call {
                 this.initialized = false;
                 this.hasJoinedOnce = false;
                 this.ringingSubject.next(false);
+                this.cancelAutoDrop();
                 this.clientStore.unregisterCall(this);
                 this.camera.dispose();
                 this.microphone.dispose();
@@ -10596,7 +10572,7 @@ class Call {
          *
          * @param reason the reason for rejecting the call.
          */
-        this.reject = async (reason) => {
+        this.reject = async (reason = 'decline') => {
             return this.streamClient.post(`${this.streamClientBasePath}/reject`, { reason: reason });
         };
         /**
@@ -13096,7 +13072,7 @@ class StreamClient {
         this.getUserAgent = () => {
             if (!this.cachedUserAgent) {
                 const { clientAppIdentifier = {} } = this.options;
-                const { sdkName = 'js', sdkVersion = "1.18.5", ...extras } = clientAppIdentifier;
+                const { sdkName = 'js', sdkVersion = "1.18.7", ...extras } = clientAppIdentifier;
                 this.cachedUserAgent = [
                     `stream-video-${sdkName}-v${sdkVersion}`,
                     ...Object.entries(extras).map(([key, value]) => `${key}=${value}`),