fix: no need of json parsing let rn bridge do it

Author: santhoshvai

packages/react-native-sdk/android/src/main/java/com/streamvideo/reactnative/audio/AudioDeviceManager.kt

@@ -27,7 +27,6 @@ import com.facebook.react.bridge.Arguments
 import com.facebook.react.bridge.ReactApplicationContext
 import com.facebook.react.bridge.WritableMap
 import com.facebook.react.modules.core.DeviceEventManagerModule
-import com.oney.WebRTCModule.WebRTCModule
 import com.streamvideo.reactnative.audio.utils.AudioDeviceEndpointUtils
 import com.streamvideo.reactnative.audio.utils.AudioFocusUtil
 import com.streamvideo.reactnative.audio.utils.AudioManagerUtil
@@ -524,18 +523,8 @@ class AudioDeviceManager(
 
     fun audioStatusMap(): WritableMap {
         val data = Arguments.createMap()
-        var audioDevicesJson = "["
-        for (s in getCurrentDeviceEndpoints()) {
-            audioDevicesJson += "\"" + s.name + "\","
-        }
-
-        // --- strip the last `,`
-        if (audioDevicesJson.length > 1) {
-            audioDevicesJson = audioDevicesJson.substring(0, audioDevicesJson.length - 1)
-        }
-        audioDevicesJson += "]"
-
-        data.putString("availableAudioDeviceEndpointNamesList", audioDevicesJson)
+        val availableAudioDeviceEndpointNamesList = Arguments.fromList(getCurrentDeviceEndpoints().map { it.name })
+        data.putArray("availableAudioDeviceEndpointNamesList", availableAudioDeviceEndpointNamesList)
         data.putString(
             "selectedAudioDeviceEndpointType",
             endpointTypeDebug(this.selectedAudioDeviceEndpoint?.type)

packages/react-native-sdk/src/StreamInCallManager/index.ts

@@ -11,12 +11,6 @@ export type AudioDeviceStatus = {
   selectedAudioDeviceName: string;
 };
 
-type AudioDeviceStatusUnparsed = {
-  availableAudioDeviceEndpointNamesList: string;
-  selectedAudioDeviceEndpointType: string;
-  selectedAudioDeviceName: string;
-};
-
 export type AudioRole = 'communicator' | 'listener';
 export type DefaultAudioDeviceEndpointType = 'speaker' | 'earpiece';
 
@@ -113,8 +107,8 @@ function addAndroidAudioDeviceStatusChangeListener(
   );
   const subscription = InCallManagerEventEmitter.addListener(
     'onAudioDeviceChanged',
-    (audioDeviceStatus: AudioDeviceStatusUnparsed) => {
-      onChange(parseAudioDeviceStatus(audioDeviceStatus));
+    (audioDeviceStatus: AudioDeviceStatus) => {
+      onChange(audioDeviceStatus);
     },
   );
 
@@ -137,20 +131,6 @@ function chooseAndroidAudioDeviceEndpoint(endpointName: string) {
   StreamInCallManagerNativeModule.chooseAudioDeviceEndpoint(endpointName);
 }
 
-function parseAudioDeviceStatus(
-  audioDeviceStatusUnparsed: AudioDeviceStatusUnparsed,
-) {
-  const audioDeviceStatus: AudioDeviceStatus = {
-    availableAudioDeviceEndpointNamesList: JSON.parse(
-      audioDeviceStatusUnparsed.availableAudioDeviceEndpointNamesList,
-    ),
-    selectedAudioDeviceEndpointType:
-      audioDeviceStatusUnparsed.selectedAudioDeviceEndpointType,
-    selectedAudioDeviceName: audioDeviceStatusUnparsed.selectedAudioDeviceName,
-  };
-  return audioDeviceStatus;
-}
-
 /**
  * Get the current audio device status.
  * @returns The audio device status.
@@ -160,9 +140,8 @@ async function getAndroidAudioDeviceStatus() {
     logger('warn', 'getAndroidAudioDeviceStatus is supported only on Android');
     return;
   }
-  const audioDeviceStatus: AudioDeviceStatus = parseAudioDeviceStatus(
-    await StreamInCallManagerNativeModule.getAudioDeviceStatus(),
-  );
+  const audioDeviceStatus: AudioDeviceStatus =
+    await StreamInCallManagerNativeModule.getAudioDeviceStatus();
   return audioDeviceStatus;
 }