@stream-io/video-client 1.8.0 → 1.8.2

package/dist/src/helpers/RNSpeechDetector.d.ts

@@ -3,6 +3,7 @@ export declare class RNSpeechDetector {
     private pc1;
     private pc2;
     private intervalId;
+    private audioStream;
     /**
      * Starts the speech detection.
      */
@@ -15,4 +16,5 @@ export declare class RNSpeechDetector {
      * Public method that detects the audio levels and returns the status.
      */
     onSpeakingDetectedStateChange(onSoundDetectedStateChanged: SoundStateChangeHandler): () => void;
+    private cleanupAudioStream;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stream-io/video-client",
-  "version": "1.8.0",
+  "version": "1.8.2",
   "packageManager": "yarn@3.2.4",
   "main": "dist/index.cjs.js",
   "module": "dist/index.es.js",

package/src/devices/InputMediaDeviceManager.ts

@@ -279,7 +279,14 @@ export abstract class InputMediaDeviceManager<
 
   private stopTracks() {
     this.getTracks().forEach((track) => {
-      if (track.readyState === 'live') track.stop();
+      if (track.readyState === 'live') {
+        track.stop();
+        // @ts-expect-error release() is present in react-native-webrtc and must be called to dispose the track
+        if (typeof track.release === 'function') {
+          // @ts-expect-error
+          track.release();
+        }
+      }
     });
   }
 

package/src/devices/devices.ts

@@ -277,6 +277,11 @@ export const disposeOfMediaStream = (stream: MediaStream) => {
   stream.getTracks().forEach((track) => {
     track.stop();
     stream.removeTrack(track);
+    // @ts-expect-error release() is present in react-native-webrtc and must be called to dispose the track
+    if (typeof track.release === 'function') {
+      // @ts-expect-error
+      track.release();
+    }
   });
   // @ts-expect-error release() is present in react-native-webrtc and must be called to dispose the stream
   if (typeof stream.release === 'function') {

package/src/events/call.ts

@@ -76,9 +76,8 @@ export const watchCallEnded = (call: Call) => {
   return function onCallEnded() {
     const { callingState } = call.state;
     if (
-      callingState === CallingState.RINGING ||
-      callingState === CallingState.JOINED ||
-      callingState === CallingState.JOINING
+      callingState !== CallingState.IDLE &&
+      callingState !== CallingState.LEFT
     ) {
       call.leave({ reason: 'call.ended event received' }).catch((err) => {
         call.logger('error', 'Failed to leave call after call.ended ', err);

package/src/helpers/RNSpeechDetector.ts

@@ -8,15 +8,18 @@ export class RNSpeechDetector {
   private pc1 = new RTCPeerConnection({});
   private pc2 = new RTCPeerConnection({});
   private intervalId: NodeJS.Timeout | undefined;
+  private audioStream: MediaStream | undefined;
 
   /**
    * Starts the speech detection.
    */
   public async start() {
     try {
+      this.cleanupAudioStream();
       const audioStream = await navigator.mediaDevices.getUserMedia({
         audio: true,
       });
+      this.audioStream = audioStream;
 
       this.pc1.addEventListener('icecandidate', async (e) => {
         await this.pc2.addIceCandidate(
@@ -55,6 +58,7 @@ export class RNSpeechDetector {
   public stop() {
     this.pc1.close();
     this.pc2.close();
+    this.cleanupAudioStream();
     if (this.intervalId) {
       clearInterval(this.intervalId);
     }
@@ -97,4 +101,25 @@ export class RNSpeechDetector {
       clearInterval(this.intervalId);
     };
   }
+
+  private cleanupAudioStream() {
+    if (!this.audioStream) {
+      return;
+    }
+    this.audioStream.getTracks().forEach((track) => {
+      track.stop();
+      // @ts-expect-error release() is present in react-native-webrtc and must be called to dispose the track
+      if (typeof track.release === 'function') {
+        // @ts-expect-error
+        track.release();
+      }
+    });
+    if (
+      // @ts-expect-error release() is present in react-native-webrtc
+      typeof this.audioStream.release === 'function'
+    ) {
+      // @ts-expect-error called to dispose the stream in RN
+      this.audioStream.release();
+    }
+  }
 }

package/src/rtc/Publisher.ts

@@ -316,6 +316,11 @@ export class Publisher {
       if (previousTrack && previousTrack !== track) {
         previousTrack.stop();
         previousTrack.removeEventListener('ended', handleTrackEnded);
+        // @ts-expect-error release() is present in react-native-webrtc and must be called to dispose the track
+        if (typeof previousTrack.release === 'function') {
+          // @ts-expect-error
+          track.release();
+        }
         track.addEventListener('ended', handleTrackEnded);
       }
       if (!track.enabled) {
@@ -337,16 +342,18 @@ export class Publisher {
     const transceiver = this.pc
       .getTransceivers()
       .find((t) => t === this.transceiverRegistry[trackType] && t.sender.track);
-    if (
-      transceiver &&
-      transceiver.sender.track &&
-      (stopTrack
-        ? transceiver.sender.track.readyState === 'live'
-        : transceiver.sender.track.enabled)
-    ) {
-      stopTrack
-        ? transceiver.sender.track.stop()
-        : (transceiver.sender.track.enabled = false);
+    const track = transceiver?.sender.track;
+    if (track && (stopTrack ? track.readyState === 'live' : track.enabled)) {
+      if (stopTrack) {
+        track.stop();
+        // @ts-expect-error release() is present in react-native-webrtc and must be called to dispose the track
+        if (typeof track.release === 'function') {
+          // @ts-expect-error
+          track.release();
+        }
+      } else {
+        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)) {
         await this.notifyTrackMuteStateChanged(undefined, trackType, true);
@@ -399,7 +406,13 @@ export class Publisher {
   private stopPublishing = () => {
     this.logger('debug', 'Stopping publishing all tracks');
     this.pc.getSenders().forEach((s) => {
-      s.track?.stop();
+      const track = s.track;
+      track?.stop();
+      // @ts-expect-error release() is present in react-native-webrtc and must be called to dispose the track
+      if (typeof track?.release === 'function') {
+        // @ts-expect-error
+        track.release();
+      }
       if (this.pc.signalingState !== 'closed') {
         this.pc.removeTrack(s);
       }