@stream-io/video-client 1.18.7 → 1.18.8

package/src/gen/video/sfu/signal_rpc/signal.client.ts

@@ -1,31 +1,35 @@
-/* eslint-disable */
 // @generated by protobuf-ts 2.9.4 with parameter long_type_string,client_generic,server_none,eslint_disable,optimize_code_size
 // @generated from protobuf file "video/sfu/signal_rpc/signal.proto" (package "stream.video.sfu.signal", syntax proto3)
 // tslint:disable
-import type { RpcTransport } from '@protobuf-ts/runtime-rpc';
-import type { ServiceInfo } from '@protobuf-ts/runtime-rpc';
+import type {
+  RpcOptions,
+  RpcTransport,
+  ServiceInfo,
+  UnaryCall,
+} from '@protobuf-ts/runtime-rpc';
+import { stackIntercept } from '@protobuf-ts/runtime-rpc';
+import type {
+  ICERestartRequest,
+  ICERestartResponse,
+  ICETrickleResponse,
+  SendAnswerRequest,
+  SendAnswerResponse,
+  SendStatsRequest,
+  SendStatsResponse,
+  SetPublisherRequest,
+  SetPublisherResponse,
+  StartNoiseCancellationRequest,
+  StartNoiseCancellationResponse,
+  StopNoiseCancellationRequest,
+  StopNoiseCancellationResponse,
+  UpdateMuteStatesRequest,
+  UpdateMuteStatesResponse,
+  UpdateSubscriptionsRequest,
+  UpdateSubscriptionsResponse,
+} from './signal';
 import { SignalServer } from './signal';
-import type { StopNoiseCancellationResponse } from './signal';
-import type { StopNoiseCancellationRequest } from './signal';
-import type { StartNoiseCancellationResponse } from './signal';
-import type { StartNoiseCancellationRequest } from './signal';
-import type { SendStatsResponse } from './signal';
-import type { SendStatsRequest } from './signal';
-import type { ICERestartResponse } from './signal';
-import type { ICERestartRequest } from './signal';
-import type { UpdateMuteStatesResponse } from './signal';
-import type { UpdateMuteStatesRequest } from './signal';
-import type { UpdateSubscriptionsResponse } from './signal';
-import type { UpdateSubscriptionsRequest } from './signal';
-import type { ICETrickleResponse } from './signal';
 import type { ICETrickle } from '../models/models';
-import type { SendAnswerResponse } from './signal';
-import type { SendAnswerRequest } from './signal';
-import { stackIntercept } from '@protobuf-ts/runtime-rpc';
-import type { SetPublisherResponse } from './signal';
-import type { SetPublisherRequest } from './signal';
-import type { UnaryCall } from '@protobuf-ts/runtime-rpc';
-import type { RpcOptions } from '@protobuf-ts/runtime-rpc';
+
 /**
  * @generated from protobuf service stream.video.sfu.signal.SignalServer
  */

package/src/gen/video/sfu/signal_rpc/signal.ts

@@ -1,19 +1,21 @@
-/* eslint-disable */
 // @generated by protobuf-ts 2.9.4 with parameter long_type_string,client_generic,server_none,eslint_disable,optimize_code_size
 // @generated from protobuf file "video/sfu/signal_rpc/signal.proto" (package "stream.video.sfu.signal", syntax proto3)
 // tslint:disable
-import { ICETrickle } from '../models/models';
+import {
+  AndroidState,
+  AppleState,
+  Error,
+  ICETrickle,
+  InputDevices,
+  PeerType,
+  TrackInfo,
+  TrackType,
+  VideoDimension,
+  WebsocketReconnectStrategy,
+} from '../models/models';
 import { ServiceType } from '@protobuf-ts/runtime-rpc';
 import { MessageType } from '@protobuf-ts/runtime';
-import { TrackInfo } from '../models/models';
-import { VideoDimension } from '../models/models';
-import { TrackType } from '../models/models';
-import { PeerType } from '../models/models';
-import { AppleState } from '../models/models';
-import { AndroidState } from '../models/models';
-import { InputDevices } from '../models/models';
-import { WebsocketReconnectStrategy } from '../models/models';
-import { Error } from '../models/models';
+
 /**
  * @generated from protobuf message stream.video.sfu.signal.StartNoiseCancellationRequest
  */

package/src/helpers/RNSpeechDetector.ts

@@ -33,7 +33,7 @@ export class RNSpeechDetector {
         e.streams[0].getTracks().forEach((track) => {
           // In RN, the remote track is automatically added to the audio output device
           // so we need to mute it to avoid hearing the audio back
-          // @ts-ignore _setVolume is a private method in react-native-webrtc
+          // @ts-expect-error _setVolume is a private method in react-native-webrtc
           track._setVolume(0);
         });
       });
@@ -79,10 +79,9 @@ export class RNSpeechDetector {
     const initialBaselineNoiseLevel = 0.13;
     let baselineNoiseLevel = initialBaselineNoiseLevel;
     let speechDetected = false;
-    let intervalId: NodeJS.Timeout | undefined;
     let speechTimer: NodeJS.Timeout | undefined;
     let silenceTimer: NodeJS.Timeout | undefined;
-    let audioLevelHistory = []; // Store recent audio levels for smoother detection
+    const audioLevelHistory: number[] = []; // Store recent audio levels for smoother detection
     const historyLength = 10;
     const silenceThreshold = 1.1;
     const resetThreshold = 0.9;
@@ -157,7 +156,7 @@ export class RNSpeechDetector {
     };
 
     // Call checkAudioLevel periodically (every 100ms)
-    intervalId = setInterval(checkAudioLevel, 100);
+    const intervalId = setInterval(checkAudioLevel, 100);
 
     return () => {
       clearInterval(intervalId);

package/src/helpers/__tests__/DynascaleManager.test.ts

@@ -42,7 +42,7 @@ describe('DynascaleManager', () => {
         },
       );
 
-      // @ts-ignore
+      // @ts-expect-error incomplete data
       call.state.updateOrAddParticipant('session-id', {
         userId: 'user-id',
         sessionId: 'session-id',
@@ -90,9 +90,9 @@ describe('DynascaleManager', () => {
 
     beforeEach(() => {
       videoElement = document.createElement('video');
-      // @ts-ignore
+      // @ts-expect-error private property
       videoElement.clientWidth = 100;
-      // @ts-ignore
+      // @ts-expect-error private property
       videoElement.clientHeight = 100;
     });
 
@@ -100,17 +100,16 @@ describe('DynascaleManager', () => {
       vi.useFakeTimers();
       const audioElement = document.createElement('audio');
       const play = vi.spyOn(audioElement, 'play').mockResolvedValue();
-      // @ts-expect-error setSinkId is not defined in types
       audioElement.setSinkId = vi.fn();
 
-      // @ts-ignore
+      // @ts-expect-error incomplete data
       call.state.updateOrAddParticipant('session-id', {
         userId: 'user-id',
         sessionId: 'session-id',
         publishedTracks: [],
       });
 
-      // @ts-ignore
+      // @ts-expect-error incomplete data
       call.state.updateOrAddParticipant('session-id-local', {
         userId: 'user-id-local',
         sessionId: 'session-id-local',
@@ -134,15 +133,10 @@ describe('DynascaleManager', () => {
 
       expect(play).toHaveBeenCalled();
       expect(audioElement.srcObject).toBe(mediaStream);
-
       expect(audioElement.volume).toBe(1);
-
-      // @ts-expect-error setSinkId is not defined in types
       expect(audioElement.setSinkId).not.toHaveBeenCalled();
 
       call.speaker.select('different-device-id');
-
-      // @ts-expect-error setSinkId is not defined in types
       expect(audioElement.setSinkId).toHaveBeenCalledWith(
         'different-device-id',
       );
@@ -164,7 +158,7 @@ describe('DynascaleManager', () => {
       const audioElement = document.createElement('audio');
       const play = vi.spyOn(audioElement, 'play').mockResolvedValue();
 
-      // @ts-ignore
+      // @ts-expect-error incomplete data
       call.state.updateOrAddParticipant('session-id', {
         userId: 'user-id',
         sessionId: 'session-id',
@@ -199,7 +193,7 @@ describe('DynascaleManager', () => {
         'updateParticipantTracks',
       );
 
-      // @ts-ignore
+      // @ts-expect-error incomplete data
       call.state.updateOrAddParticipant('session-id', {
         userId: 'user-id',
         sessionId: 'session-id',
@@ -257,7 +251,7 @@ describe('DynascaleManager', () => {
       );
       const play = vi.spyOn(videoElement, 'play').mockResolvedValue();
 
-      // @ts-ignore
+      // @ts-expect-error incomplete data
       call.state.updateOrAddParticipant('session-id', {
         userId: 'user-id',
         sessionId: 'session-id',
@@ -297,7 +291,7 @@ describe('DynascaleManager', () => {
     });
 
     it('video: should update subscription when element becomes visible', () => {
-      // @ts-ignore
+      // @ts-expect-error incomplete data
       call.state.updateOrAddParticipant('session-id', {
         userId: 'user-id',
         sessionId: 'session-id',
@@ -374,7 +368,7 @@ describe('DynascaleManager', () => {
     });
 
     it('video: should update subscription when element resizes', () => {
-      // @ts-ignore
+      // @ts-expect-error incomplete data
       call.state.updateOrAddParticipant('session-id', {
         userId: 'user-id',
         sessionId: 'session-id',
@@ -385,12 +379,16 @@ describe('DynascaleManager', () => {
         },
       });
 
-      let updateSubscription = vi.spyOn(call.state, 'updateParticipantTracks');
+      const updateSubscription = vi.spyOn(
+        call.state,
+        'updateParticipantTracks',
+      );
 
-      let resizeObserverCallback: ResizeObserverCallback;
+      let resizeObserverCallback: ResizeObserverCallback | undefined =
+        undefined;
       window.ResizeObserver = class ResizeObserver {
         observe = vi.fn().mockImplementation(() => {
-          // @ts-ignore simulate initial trigger
+          // @ts-expect-error simulate initial trigger
           resizeObserverCallback();
         });
         unobserve = vi.fn();
@@ -416,12 +414,12 @@ describe('DynascaleManager', () => {
         },
       });
 
-      // @ts-ignore simulate resize
+      // @ts-expect-error simulate resize
       videoElement.clientHeight = 101;
-      // @ts-ignore simulate resize
+      // @ts-expect-error simulate resize
       videoElement.clientWidth = 101;
 
-      // @ts-ignore simulate resize
+      // @ts-expect-error simulate resize
       resizeObserverCallback();
 
       expect(updateSubscription).toHaveBeenCalledWith('videoTrack', {
@@ -436,7 +434,7 @@ describe('DynascaleManager', () => {
     });
 
     it('video: should unsubscribe when element dimensions are zero', () => {
-      // @ts-ignore
+      // @ts-expect-error incomplete data
       call.state.updateOrAddParticipant('session-id', {
         userId: 'user-id',
         sessionId: 'session-id',
@@ -447,11 +445,14 @@ describe('DynascaleManager', () => {
         },
       });
 
-      let updateSubscription = vi.spyOn(call.state, 'updateParticipantTracks');
+      const updateSubscription = vi.spyOn(
+        call.state,
+        'updateParticipantTracks',
+      );
 
-      // @ts-ignore simulate resize
+      // @ts-expect-error simulate resize
       videoElement.clientHeight = 0;
-      // @ts-ignore simulate resize
+      // @ts-expect-error simulate resize
       videoElement.clientWidth = 0;
 
       const cleanup = dynascaleManager.bindVideoElement(

package/src/helpers/__tests__/clientUtils.test.ts

@@ -72,7 +72,6 @@ describe('clientUtils', () => {
 
     it('should use default x-stream-client header', () => {
       const sdk = getSdkInfo();
-      // @ts-expect-error
       setSdkInfo(undefined);
       const client = createCoordinatorClient('apiKey', {});
       expect(client).toBeDefined();

package/src/helpers/client-details.ts

@@ -151,7 +151,7 @@ export const getClientDetails = async (): Promise<ClientDetails> => {
         'platform',
         'platformVersion',
       ]);
-    } catch (e) {
+    } catch {
       // Ignore the error
     }
   }

package/src/helpers/promise.ts

@@ -50,8 +50,8 @@ export type PromiseWithResolvers<T> = {
   promise: Promise<T>;
   resolve: (value: T | PromiseLike<T>) => void;
   reject: (reason: any) => void;
-  isResolved: boolean;
-  isRejected: boolean;
+  isResolved: () => boolean;
+  isRejected: () => boolean;
 };
 
 /**
@@ -85,7 +85,7 @@ export const promiseWithResolvers = <T = void>(): PromiseWithResolvers<T> => {
     promise,
     resolve: resolver,
     reject: rejecter,
-    isResolved,
-    isRejected,
+    isResolved: () => isResolved,
+    isRejected: () => isRejected,
   };
 };

package/src/rtc/Dispatcher.ts

@@ -20,7 +20,7 @@ export type DispatchableMessage<K extends SfuEventKinds> = {
   };
 };
 
-const sfuEventKinds: { [key in SfuEventKinds]: undefined } = {
+const sfuEventKinds: Record<SfuEventKinds, undefined> = {
   subscriberOffer: undefined,
   publisherAnswer: undefined,
   connectionQualityChanged: undefined,

package/src/rtc/Publisher.ts

@@ -250,8 +250,8 @@ export class Publisher extends BasePeerConnection {
         ? // for SVC, we only have one layer (q) and often rid is omitted
           enabledLayers[0]
         : // for non-SVC, we need to find the layer by rid (simulcast)
-          enabledLayers.find((l) => l.name === encoder.rid) ??
-          (params.encodings.length === 1 ? enabledLayers[0] : undefined);
+          (enabledLayers.find((l) => l.name === encoder.rid) ??
+          (params.encodings.length === 1 ? enabledLayers[0] : undefined));
 
       // flip 'active' flag only when necessary
       const shouldActivate = !!layer?.active;

package/src/rtc/__tests__/Publisher.test.ts

@@ -51,7 +51,7 @@ describe('Publisher', () => {
     // @ts-expect-error readonly field
     sfuClient.iceTrickleBuffer = new IceTrickleBuffer();
 
-    // @ts-ignore
+    // @ts-expect-error private field
     sfuClient['sessionId'] = sessionId;
 
     state = new CallState();
@@ -84,7 +84,7 @@ describe('Publisher', () => {
   describe('Publishing', () => {
     it('should throw when publishing ended tracks', async () => {
       const track = new MediaStreamTrack();
-      // @ts-ignore readonly field
+      // @ts-expect-error readonly field
       track.readyState = 'ended';
       await expect(publisher.publish(track, TrackType.VIDEO)).rejects.toThrow();
     });
@@ -128,7 +128,7 @@ describe('Publisher', () => {
       vi.spyOn(track, 'clone').mockReturnValue(clone);
 
       const transceiver = new RTCRtpTransceiver();
-      // @ts-ignore test setup
+      // @ts-expect-error test setup
       transceiver.sender.track = track;
       publisher['transceiverCache'].add(
         publisher['publishOptions'][0],
@@ -218,9 +218,9 @@ describe('Publisher', () => {
     });
 
     it(`should drop consequent ICE restart requests`, async () => {
-      // @ts-ignore
+      // @ts-expect-error private method
       publisher['pc'].signalingState = 'have-local-offer';
-      // @ts-ignore
+      // @ts-expect-error private method
       vi.spyOn(publisher, 'negotiate').mockResolvedValue();
 
       await publisher.restartIce();
@@ -229,7 +229,7 @@ describe('Publisher', () => {
 
     it(`should perform ICE restart when connection state changes to 'failed'`, () => {
       vi.spyOn(publisher, 'restartIce').mockResolvedValue();
-      // @ts-ignore
+      // @ts-expect-error private api
       publisher['pc'].iceConnectionState = 'failed';
       publisher['onIceConnectionStateChange']();
       expect(publisher.restartIce).toHaveBeenCalled();
@@ -237,7 +237,7 @@ describe('Publisher', () => {
 
     it(`should perform ICE restart when connection state changes to 'disconnected'`, () => {
       vi.spyOn(publisher, 'restartIce').mockResolvedValue();
-      // @ts-ignore
+      // @ts-expect-error private api
       publisher['pc'].iceConnectionState = 'disconnected';
       publisher['onIceConnectionStateChange']();
       expect(publisher.restartIce).toHaveBeenCalled();
@@ -564,7 +564,7 @@ describe('Publisher', () => {
 
       const track = new MediaStreamTrack();
       const transceiver = new RTCRtpTransceiver();
-      // @ts-ignore test setup
+      // @ts-expect-error test setup
       transceiver.sender.track = track;
 
       publisher['transceiverCache'].add(publishOptions[0], transceiver);

package/src/rtc/__tests__/Subscriber.test.ts

@@ -20,7 +20,7 @@ vi.mock('../../StreamSfuClient', () => {
 describe('Subscriber', () => {
   let sfuClient: StreamSfuClient;
   let subscriber: Subscriber;
-  let state = new CallState();
+  const state = new CallState();
   let dispatcher: Dispatcher;
 
   beforeEach(() => {
@@ -61,7 +61,7 @@ describe('Subscriber', () => {
   describe('Subscriber ICE restart', () => {
     it(`should drop consequent ICE restart requests`, async () => {
       sfuClient.iceRestart = vi.fn();
-      // @ts-ignore
+      // @ts-expect-error - private field
       subscriber['pc'].signalingState = 'have-remote-offer';
 
       await subscriber.restartIce();
@@ -70,7 +70,7 @@ describe('Subscriber', () => {
 
     it('should skip ICE restart when connection is still new', async () => {
       sfuClient.iceRestart = vi.fn();
-      // @ts-ignore
+      // @ts-expect-error - private field
       subscriber['pc'].connectionState = 'new';
 
       await subscriber.restartIce();
@@ -79,7 +79,7 @@ describe('Subscriber', () => {
 
     it('should ask the SFU for ICE restart', async () => {
       sfuClient.iceRestart = vi.fn();
-      // @ts-ignore
+      // @ts-expect-error - private field
       subscriber['pc'].connectionState = 'connected';
 
       await subscriber.restartIce();
@@ -90,7 +90,7 @@ describe('Subscriber', () => {
 
     it(`should perform ICE restart when connection state changes to 'failed'`, () => {
       vi.spyOn(subscriber, 'restartIce').mockResolvedValue();
-      // @ts-ignore
+      // @ts-expect-error - private field
       subscriber['pc'].iceConnectionState = 'failed';
       subscriber['onIceConnectionStateChange']();
       expect(subscriber.restartIce).toHaveBeenCalled();
@@ -98,7 +98,7 @@ describe('Subscriber', () => {
 
     it(`should perform ICE restart when connection state changes to 'disconnected'`, () => {
       vi.spyOn(subscriber, 'restartIce').mockResolvedValue();
-      // @ts-ignore
+      // @ts-expect-error - private field
       subscriber['pc'].iceConnectionState = 'disconnected';
       subscriber['onIceConnectionStateChange']();
       expect(subscriber.restartIce).toHaveBeenCalled();
@@ -109,7 +109,7 @@ describe('Subscriber', () => {
     it('should add unknown tracks to the to the call state', () => {
       const mediaStream = new MediaStream();
       const mediaStreamTrack = new MediaStreamTrack();
-      // @ts-ignore - mock
+      // @ts-expect-error - mock
       mediaStream.id = '123:TRACK_TYPE_VIDEO';
 
       const registerOrphanedTrackSpy = vi.spyOn(state, 'registerOrphanedTrack');
@@ -131,7 +131,7 @@ describe('Subscriber', () => {
     it('should assign known tracks to the participant', () => {
       const mediaStream = new MediaStream();
       const mediaStreamTrack = new MediaStreamTrack();
-      // @ts-ignore - mock
+      // @ts-expect-error - mock
       mediaStream.id = '123:TRACK_TYPE_VIDEO';
 
       const registerOrphanedTrackSpy = vi.spyOn(state, 'registerOrphanedTrack');
@@ -156,7 +156,7 @@ describe('Subscriber', () => {
     it('should replace participant stream when a new one arrives', () => {
       const mediaStream = new MediaStream();
       const mediaStreamTrack = new MediaStreamTrack();
-      // @ts-ignore - mock
+      // @ts-expect-error - mock
       mediaStream.id = '123:TRACK_TYPE_VIDEO';
 
       const updateParticipantSpy = vi.spyOn(state, 'updateParticipant');

package/src/rtc/__tests__/mocks/webrtc.mocks.ts

@@ -51,7 +51,7 @@ vi.stubGlobal('MediaStreamTrack', MediaStreamTrackMock);
 
 const RTCRtpTransceiverMock = vi.fn((): Partial<RTCRtpTransceiver> => {
   return {
-    // @ts-ignore
+    // @ts-expect-error - incomplete mock
     sender: {
       track: null,
       replaceTrack: vi.fn(),
@@ -84,7 +84,7 @@ vi.stubGlobal('RTCRtpReceiver', RTCRtpReceiverMock);
 const RTCRtpSenderMock = vi.fn((): Partial<typeof RTCRtpSender> => {
   return {
     getCapabilities: vi.fn(),
-    // @ts-ignore
+    // @ts-expect-error - incomplete mock
     track: vi.fn(),
   };
 });

package/src/rtc/helpers/__tests__/sdp.test.ts

@@ -29,7 +29,7 @@ a=rtcp-fb:96 nack pli
 describe('sdp', () => {
   it('should extract mid from transceiver', () => {
     const transceiver = new RTCRtpTransceiver();
-    // @ts-ignore - mid is a readonly property
+    // @ts-expect-error - mid is a readonly property
     transceiver.mid = '10';
     expect(extractMid(transceiver, -1, '')).toBe('10');
   });
@@ -40,7 +40,7 @@ describe('sdp', () => {
 
   it('should extract mid from SDP', () => {
     const track = new MediaStreamTrack();
-    // @ts-ignore - id is a readonly property
+    // @ts-expect-error - id is a readonly property
     track.id = '8d240fd6-26a1-40f6-a769-4d7d24cfd286';
     const transceiver = new RTCRtpTransceiver();
     vi.spyOn(transceiver.sender, 'track', 'get').mockReturnValue(track);
@@ -50,7 +50,7 @@ describe('sdp', () => {
 
   it('should fallback to transceiverInitIndex when mid can not be found in SDP', () => {
     const track = new MediaStreamTrack();
-    // @ts-ignore - id is a readonly property
+    // @ts-expect-error - id is a readonly property
     track.id = 'not-known';
     const transceiver = new RTCRtpTransceiver();
     vi.spyOn(transceiver.sender, 'track', 'get').mockReturnValue(track);

package/src/stats/CallStateStatsReporter.ts

@@ -98,9 +98,9 @@ export const createStatsReporter = ({
     for (const track of tracks) {
       const report = await pc.getStats(track);
       const stats = transform(report, {
-        // @ts-ignore
-        trackKind: track.kind,
+        trackKind: track.kind as 'audio' | 'video',
         kind,
+        publisher: undefined,
       });
       statsForStream.push(stats);
     }
@@ -298,7 +298,6 @@ const transform = (
         jitter: rtcStreamStats.jitter,
         kind: rtcStreamStats.kind,
         mediaSourceId: rtcStreamStats.mediaSourceId,
-        // @ts-ignore: available in Chrome only, TS doesn't recognize this
         qualityLimitationReason: rtcStreamStats.qualityLimitationReason,
         rid: rtcStreamStats.rid,
         ssrc: rtcStreamStats.ssrc,