@stream-io/video-client 1.44.3 → 1.44.4

package/dist/src/devices/BrowserPermission.d.ts

@@ -1,6 +1,8 @@
+import { Tracer } from '../stats';
 interface BrowserPermissionConfig {
     constraints: DisplayMediaStreamOptions;
     queryName: PermissionName;
+    tracer: Tracer | undefined;
 }
 export type BrowserPermissionState = PermissionState | 'prompting';
 export declare class BrowserPermission {

package/dist/src/devices/CameraManagerState.d.ts

@@ -1,4 +1,5 @@
 import { DeviceManagerState } from './DeviceManagerState';
+import { Tracer } from '../stats';
 export type CameraDirection = 'front' | 'back' | undefined;
 export declare class CameraManagerState extends DeviceManagerState {
     private directionSubject;
@@ -8,7 +9,7 @@ export declare class CameraManagerState extends DeviceManagerState {
      * back - means the camera facing the environment
      */
     direction$: import("rxjs").Observable<CameraDirection>;
-    constructor();
+    constructor(tracer: Tracer | undefined);
     /**
      * The preferred camera direction
      * front - means the camera facing the user

package/dist/src/devices/MicrophoneManagerState.d.ts

@@ -1,12 +1,13 @@
 import { TrackDisableMode } from './DeviceManagerState';
 import { AudioDeviceManagerState } from './AudioDeviceManagerState';
+import { Tracer } from '../stats';
 export declare class MicrophoneManagerState extends AudioDeviceManagerState<MediaTrackConstraints> {
     private speakingWhileMutedSubject;
     /**
      * An Observable that emits `true` if the user's microphone is muted, but they're speaking.
      */
     speakingWhileMuted$: import("rxjs").Observable<boolean>;
-    constructor(disableMode: TrackDisableMode);
+    constructor(disableMode: TrackDisableMode, tracer: Tracer | undefined);
     /**
      * `true` if the user's microphone is muted but they're speaking.
      */

package/dist/src/devices/devices.d.ts

@@ -9,12 +9,12 @@ export declare const checkIfAudioOutputChangeSupported: () => boolean;
  * Keeps track of the browser permission to use microphone. This permission also
  * affects an ability to enumerate audio devices.
  */
-export declare const getAudioBrowserPermission: (...args: never[]) => BrowserPermission;
+export declare const getAudioBrowserPermission: (...args: (Tracer | undefined)[]) => BrowserPermission;
 /**
  * Keeps track of the browser permission to use camera. This permission also
  * affects an ability to enumerate video devices.
  */
-export declare const getVideoBrowserPermission: (...args: never[]) => BrowserPermission;
+export declare const getVideoBrowserPermission: (...args: (Tracer | undefined)[]) => BrowserPermission;
 /**
  * Prompts the user for a permission to use audio devices (if not already granted
  * and was not prompted before) and lists the available 'audioinput' devices,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stream-io/video-client",
-  "version": "1.44.3",
+  "version": "1.44.4",
   "main": "dist/index.cjs.js",
   "module": "dist/index.es.js",
   "browser": "dist/index.browser.es.js",
@@ -46,7 +46,7 @@
     "@openapitools/openapi-generator-cli": "^2.25.0",
     "@rollup/plugin-replace": "^6.0.2",
     "@rollup/plugin-typescript": "^12.1.4",
-    "@stream-io/audio-filters-web": "^0.7.2",
+    "@stream-io/audio-filters-web": "^0.7.3",
     "@stream-io/node-sdk": "^0.7.28",
     "@total-typescript/shoehorn": "^0.1.2",
     "@types/sdp-transform": "^2.15.0",

package/src/devices/BrowserPermission.ts

@@ -3,10 +3,12 @@ import { isReactNative } from '../helpers/platforms';
 import { disposeOfMediaStream } from './utils';
 import { withoutConcurrency } from '../helpers/concurrency';
 import { videoLoggerSystem } from '../logger';
+import { Tracer } from '../stats';
 
 interface BrowserPermissionConfig {
   constraints: DisplayMediaStreamOptions;
   queryName: PermissionName;
+  tracer: Tracer | undefined;
 }
 
 export type BrowserPermissionState = PermissionState | 'prompting';
@@ -162,6 +164,9 @@ export class BrowserPermission {
 
   private setState(state: BrowserPermissionState) {
     if (this.state !== state) {
+      const { tracer, queryName } = this.permission;
+      const traceKey = `navigator.mediaDevices.${queryName}.permission`;
+      tracer?.trace(traceKey, { previous: this.state, state });
       this.state = state;
       this.listeners.forEach((listener) => listener(state));
     }

package/src/devices/CameraManager.ts

@@ -25,7 +25,12 @@ export class CameraManager extends DeviceManager<CameraManagerState> {
     call: Call,
     devicePersistence: Required<DevicePersistenceOptions>,
   ) {
-    super(call, new CameraManagerState(), TrackType.VIDEO, devicePersistence);
+    super(
+      call,
+      new CameraManagerState(call.tracer),
+      TrackType.VIDEO,
+      devicePersistence,
+    );
   }
 
   private isDirectionSupportedByDevice() {

package/src/devices/CameraManagerState.ts

@@ -3,6 +3,7 @@ import { DeviceManagerState } from './DeviceManagerState';
 import { isReactNative } from '../helpers/platforms';
 import { getVideoBrowserPermission } from './devices';
 import { RxUtils } from '../store';
+import { Tracer } from '../stats';
 
 export type CameraDirection = 'front' | 'back' | undefined;
 
@@ -18,8 +19,8 @@ export class CameraManagerState extends DeviceManagerState {
     .asObservable()
     .pipe(distinctUntilChanged());
 
-  constructor() {
-    super('stop-tracks', getVideoBrowserPermission());
+  constructor(tracer: Tracer | undefined) {
+    super('stop-tracks', getVideoBrowserPermission(tracer));
   }
 
   /**

package/src/devices/MicrophoneManager.ts

@@ -51,7 +51,7 @@ export class MicrophoneManager extends AudioDeviceManager<MicrophoneManagerState
   ) {
     super(
       call,
-      new MicrophoneManagerState(disableMode),
+      new MicrophoneManagerState(disableMode, call.tracer),
       TrackType.AUDIO,
       devicePersistence,
     );
@@ -169,7 +169,6 @@ export class MicrophoneManager extends AudioDeviceManager<MicrophoneManagerState
                 deviceId,
                 label,
               };
-              console.log(event);
               this.call.tracer.trace('mic.capture_report', event);
               this.call.streamClient.dispatchEvent(event);
             },

package/src/devices/MicrophoneManagerState.ts

@@ -4,6 +4,7 @@ import { TrackDisableMode } from './DeviceManagerState';
 import { AudioDeviceManagerState } from './AudioDeviceManagerState';
 import { getAudioBrowserPermission, resolveDeviceId } from './devices';
 import { AudioBitrateProfile } from '../gen/video/sfu/models/models';
+import { Tracer } from '../stats';
 
 export class MicrophoneManagerState extends AudioDeviceManagerState<MediaTrackConstraints> {
   private speakingWhileMutedSubject = new BehaviorSubject<boolean>(false);
@@ -15,10 +16,10 @@ export class MicrophoneManagerState extends AudioDeviceManagerState<MediaTrackCo
     .asObservable()
     .pipe(distinctUntilChanged());
 
-  constructor(disableMode: TrackDisableMode) {
+  constructor(disableMode: TrackDisableMode, tracer: Tracer | undefined) {
     super(
       disableMode,
-      getAudioBrowserPermission(),
+      getAudioBrowserPermission(tracer),
       AudioBitrateProfile.VOICE_STANDARD_UNSPECIFIED,
     );
   }

package/src/devices/devices.ts

@@ -93,10 +93,11 @@ const videoDeviceConstraints = {
  * affects an ability to enumerate audio devices.
  */
 export const getAudioBrowserPermission = lazy(
-  () =>
+  (tracer: Tracer | undefined) =>
     new BrowserPermission({
       constraints: audioDeviceConstraints,
       queryName: 'microphone' as PermissionName,
+      tracer,
     }),
 );
 
@@ -105,10 +106,11 @@ export const getAudioBrowserPermission = lazy(
  * affects an ability to enumerate video devices.
  */
 export const getVideoBrowserPermission = lazy(
-  () =>
+  (tracer: Tracer | undefined) =>
     new BrowserPermission({
       constraints: videoDeviceConstraints,
       queryName: 'camera' as PermissionName,
+      tracer,
     }),
 );
 
@@ -132,11 +134,11 @@ const getDeviceChangeObserver = lazy((tracer: Tracer | undefined) => {
 export const getAudioDevices = lazy((tracer?: Tracer) => {
   return merge(
     getDeviceChangeObserver(tracer),
-    getAudioBrowserPermission().asObservable(),
+    getAudioBrowserPermission(tracer).asObservable(),
   ).pipe(
     startWith([]),
     concatMap(() =>
-      getDevices(getAudioBrowserPermission(), 'audioinput', tracer),
+      getDevices(getAudioBrowserPermission(tracer), 'audioinput', tracer),
     ),
     shareReplay(1),
   );
@@ -151,11 +153,11 @@ export const getAudioDevices = lazy((tracer?: Tracer) => {
 export const getVideoDevices = lazy((tracer?: Tracer) => {
   return merge(
     getDeviceChangeObserver(tracer),
-    getVideoBrowserPermission().asObservable(),
+    getVideoBrowserPermission(tracer).asObservable(),
   ).pipe(
     startWith([]),
     concatMap(() =>
-      getDevices(getVideoBrowserPermission(), 'videoinput', tracer),
+      getDevices(getVideoBrowserPermission(tracer), 'videoinput', tracer),
     ),
     shareReplay(1),
   );
@@ -170,11 +172,11 @@ export const getVideoDevices = lazy((tracer?: Tracer) => {
 export const getAudioOutputDevices = lazy((tracer?: Tracer) => {
   return merge(
     getDeviceChangeObserver(tracer),
-    getAudioBrowserPermission().asObservable(),
+    getAudioBrowserPermission(tracer).asObservable(),
   ).pipe(
     startWith([]),
     concatMap(() =>
-      getDevices(getAudioBrowserPermission(), 'audiooutput', tracer),
+      getDevices(getAudioBrowserPermission(tracer), 'audiooutput', tracer),
     ),
     shareReplay(1),
   );
@@ -259,28 +261,24 @@ export const getAudioStream = async (
   };
 
   try {
-    await getAudioBrowserPermission().prompt({
+    await getAudioBrowserPermission(tracer).prompt({
       throwOnNotAllowed: true,
       forcePrompt: true,
     });
     return await getStream(constraints, tracer);
   } catch (error) {
+    const logger = videoLoggerSystem.getLogger('devices');
     if (isNotFoundOrOverconstrainedError(error) && trackConstraints?.deviceId) {
       // eslint-disable-next-line @typescript-eslint/no-unused-vars
       const { deviceId, ...relaxedConstraints } = trackConstraints;
-      videoLoggerSystem
-        .getLogger('devices')
-        .warn(
-          'Failed to get audio stream, will try again with relaxed constraints',
-          { error, constraints, relaxedConstraints },
-        );
+      logger.warn(
+        'Failed to get audio stream, will try again with relaxed constraints',
+        { error, constraints, relaxedConstraints },
+      );
       return getAudioStream(relaxedConstraints, tracer);
     }
 
-    videoLoggerSystem.getLogger('devices').error('Failed to get audio stream', {
-      error,
-      constraints,
-    });
+    logger.error('Failed to get audio stream', { error, constraints });
     throw error;
   }
 };
@@ -304,28 +302,24 @@ export const getVideoStream = async (
     },
   };
   try {
-    await getVideoBrowserPermission().prompt({
+    await getVideoBrowserPermission(tracer).prompt({
       throwOnNotAllowed: true,
       forcePrompt: true,
     });
     return await getStream(constraints, tracer);
   } catch (error) {
+    const logger = videoLoggerSystem.getLogger('devices');
     if (isNotFoundOrOverconstrainedError(error) && trackConstraints?.deviceId) {
       // eslint-disable-next-line @typescript-eslint/no-unused-vars
       const { deviceId, ...relaxedConstraints } = trackConstraints;
-      videoLoggerSystem
-        .getLogger('devices')
-        .warn(
-          'Failed to get video stream, will try again with relaxed constraints',
-          { error, constraints, relaxedConstraints },
-        );
-      return getVideoStream(relaxedConstraints);
+      logger.warn(
+        'Failed to get video stream, will try again with relaxed constraints',
+        { error, constraints, relaxedConstraints },
+      );
+      return getVideoStream(relaxedConstraints, tracer);
     }
 
-    videoLoggerSystem.getLogger('devices').error('Failed to get video stream', {
-      error,
-      constraints,
-    });
+    logger.error('Failed to get video stream', { error, constraints });
     throw error;
   }
 };