@stream-io/video-client 0.4.1 → 0.4.3

package/CHANGELOG.md

@@ -2,6 +2,20 @@
 
 This file was generated using [@jscutlery/semver](https://github.com/jscutlery/semver).
 
+### [0.4.3](https://github.com/GetStream/stream-video-js/compare/@stream-io/video-client-0.4.2...@stream-io/video-client-0.4.3) (2023-11-01)
+
+
+### Bug Fixes
+
+* **client:** optimized device enumeration ([#1111](https://github.com/GetStream/stream-video-js/issues/1111)) ([435bd33](https://github.com/GetStream/stream-video-js/commit/435bd33afbe8b368413690f8f2d67d0b4918dbaa))
+
+### [0.4.2](https://github.com/GetStream/stream-video-js/compare/@stream-io/video-client-0.4.1...@stream-io/video-client-0.4.2) (2023-11-01)
+
+
+### Bug Fixes
+
+* respect server-side settings in the lobby ([#1175](https://github.com/GetStream/stream-video-js/issues/1175)) ([b722a0a](https://github.com/GetStream/stream-video-js/commit/b722a0a4f8fd4e4e56787db3d9a56e45ee195974))
+
 ### [0.4.1](https://github.com/GetStream/stream-video-js/compare/@stream-io/video-client-0.4.0...@stream-io/video-client-0.4.1) (2023-10-30)
 
 

package/dist/index.browser.es.js

@@ -6564,11 +6564,8 @@ class Publisher {
                     ? transceiver.sender.track.stop()
                     : (transceiver.sender.track.enabled = false);
                 // We don't need to notify SFU if unpublishing in response to remote soft mute
-                if (!this.state.localParticipant?.publishedTracks.includes(trackType)) {
-                    return;
-                }
-                else {
-                    return this.notifyTrackMuteStateChanged(undefined, transceiver.sender.track, trackType, true);
+                if (this.state.localParticipant?.publishedTracks.includes(trackType)) {
+                    await this.notifyTrackMuteStateChanged(undefined, transceiver.sender.track, trackType, true);
                 }
             }
         };
@@ -9956,22 +9953,43 @@ const CallTypes = new CallTypesRegistry([
     }),
 ]);
 
-const getDevices = (constraints) => {
+/**
+ * Returns an Observable that emits the list of available devices
+ * that meet the given constraints.
+ *
+ * @param constraints the constraints to use when requesting the devices.
+ * @param kind the kind of devices to enumerate.
+ */
+const getDevices = (constraints, kind) => {
     return new Observable((subscriber) => {
-        navigator.mediaDevices
-            .getUserMedia(constraints)
-            .then((media) => {
-            // in Firefox, devices can be enumerated after userMedia is requested
-            // and permissions granted. Otherwise, device labels are empty
-            navigator.mediaDevices.enumerateDevices().then((devices) => {
-                subscriber.next(devices);
-                // If we stop the tracks before enumerateDevices -> the labels won't show up in Firefox
-                disposeOfMediaStream(media);
-                subscriber.complete();
-            });
+        const enumerate = async () => {
+            let devices = await navigator.mediaDevices.enumerateDevices();
+            // some browsers report empty device labels (Firefox).
+            // in that case, we need to request permissions (via getUserMedia)
+            // to be able to get the device labels
+            const needsGetUserMedia = devices.some((device) => device.kind === kind && device.label === '');
+            if (needsGetUserMedia) {
+                let mediaStream;
+                try {
+                    mediaStream = await navigator.mediaDevices.getUserMedia(constraints);
+                    devices = await navigator.mediaDevices.enumerateDevices();
+                }
+                finally {
+                    if (mediaStream)
+                        disposeOfMediaStream(mediaStream);
+                }
+            }
+            return devices;
+        };
+        enumerate()
+            .then((devices) => {
+            // notify subscribers and complete
+            subscriber.next(devices);
+            subscriber.complete();
         })
             .catch((error) => {
-            getLogger(['devices'])('error', 'Failed to get devices', error);
+            const logger = getLogger(['devices']);
+            logger('error', 'Failed to enumerate devices', error);
             subscriber.error(error);
         });
     });
@@ -9985,7 +10003,7 @@ const checkIfAudioOutputChangeSupported = () => {
     if (typeof document === 'undefined')
         return false;
     const element = document.createElement('audio');
-    return element.sinkId !== undefined;
+    return 'setSinkId' in element;
 };
 /**
  * The default constraints used to request audio devices.
@@ -10036,10 +10054,13 @@ const getDeviceChangeObserver = memoizedObservable(() => {
     }).pipe(debounceTime(500), concatMap(() => from(navigator.mediaDevices.enumerateDevices())), shareReplay(1));
 });
 const getAudioDevicesObserver = memoizedObservable(() => {
-    return merge(getDevices(audioDeviceConstraints), getDeviceChangeObserver()).pipe(shareReplay(1));
+    return merge(getDevices(audioDeviceConstraints, 'audioinput'), getDeviceChangeObserver()).pipe(shareReplay(1));
+});
+const getAudioOutputDevicesObserver = memoizedObservable(() => {
+    return merge(getDevices(audioDeviceConstraints, 'audiooutput'), getDeviceChangeObserver()).pipe(shareReplay(1));
 });
 const getVideoDevicesObserver = memoizedObservable(() => {
-    return merge(getDevices(videoDeviceConstraints), getDeviceChangeObserver()).pipe(shareReplay(1));
+    return merge(getDevices(videoDeviceConstraints, 'videoinput'), getDeviceChangeObserver()).pipe(shareReplay(1));
 });
 /**
  * Prompts the user for a permission to use audio devices (if not already granted) and lists the available 'audioinput' devices, if devices are added/removed the list is updated.
@@ -10057,7 +10078,7 @@ const getVideoDevices = () => {
  * Prompts the user for a permission to use audio devices (if not already granted) and lists the available 'audiooutput' devices, if devices are added/removed the list is updated. Selecting 'audiooutput' device only makes sense if [the browser has support for changing audio output on 'audio' elements](#checkifaudiooutputchangesupported)
  */
 const getAudioOutputDevices = () => {
-    return getAudioDevicesObserver().pipe(map$1((values) => values.filter((d) => d.kind === 'audiooutput')));
+    return getAudioOutputDevicesObserver().pipe(map$1((values) => values.filter((d) => d.kind === 'audiooutput')));
 };
 const getStream = async (constraints) => {
     try {
@@ -10262,9 +10283,8 @@ class InputMediaDeviceManager {
      * Starts stream.
      */
     async enable() {
-        if (this.state.status === 'enabled') {
+        if (this.state.status === 'enabled')
             return;
-        }
         this.enablePromise = this.unmuteStream();
         try {
             await this.enablePromise;
@@ -10281,10 +10301,10 @@ class InputMediaDeviceManager {
      */
     async disable() {
         this.state.prevStatus = this.state.status;
-        if (this.state.status === 'disabled') {
+        if (this.state.status === 'disabled')
             return;
-        }
-        this.disablePromise = this.muteStream(this.state.disableMode === 'stop-tracks');
+        const stopTracks = this.state.disableMode === 'stop-tracks';
+        this.disablePromise = this.muteStream(stopTracks);
         try {
             await this.disablePromise;
             this.state.setStatus('disabled');
@@ -10321,7 +10341,7 @@ class InputMediaDeviceManager {
      *
      * @param constraints the constraints to set.
      */
-    async setDefaultConstraints(constraints) {
+    setDefaultConstraints(constraints) {
         this.state.setDefaultConstraints(constraints);
     }
     /**
@@ -10351,25 +10371,23 @@ class InputMediaDeviceManager {
         return this.state.mediaStream?.getTracks() ?? [];
     }
     async muteStream(stopTracks = true) {
-        if (!this.state.mediaStream) {
+        if (!this.state.mediaStream)
             return;
-        }
         this.logger('debug', `${stopTracks ? 'Stopping' : 'Disabling'} stream`);
         if (this.call.state.callingState === CallingState.JOINED) {
             await this.stopPublishStream(stopTracks);
         }
         this.muteLocalStream(stopTracks);
-        this.getTracks().forEach((track) => {
-            if (track.readyState === 'ended') {
+        const allEnded = this.getTracks().every((t) => t.readyState === 'ended');
+        if (allEnded) {
+            if (this.state.mediaStream &&
                 // @ts-expect-error release() is present in react-native-webrtc
-                // and must be called to dispose the stream
-                if (typeof this.state.mediaStream.release === 'function') {
-                    // @ts-expect-error
-                    this.state.mediaStream.release();
-                }
-                this.state.setMediaStream(undefined);
+                typeof this.state.mediaStream.release === 'function') {
+                // @ts-expect-error called to dispose the stream in RN
+                this.state.mediaStream.release();
             }
-        });
+            this.state.setMediaStream(undefined);
+        }
     }
     muteTracks() {
         this.getTracks().forEach((track) => {
@@ -13942,7 +13960,7 @@ class StreamClient {
             });
         };
         this.getUserAgent = () => {
-            const version = "0.4.1" ;
+            const version = "0.4.3" ;
             return (this.userAgent ||
                 `stream-video-javascript-client-${this.node ? 'node' : 'browser'}-${version}`);
         };