fix: avoid JS function binding on single-param functions when calling native module

Author: jamsch

README.md

@@ -37,7 +37,7 @@ expo-speech-recognition implements the iOS [`SFSpeechRecognizer`](https://develo
   - [getSpeechRecognitionServices()](#getspeechrecognitionservices-string-android-only)
   - [getDefaultRecognitionService()](#getdefaultrecognitionservice--packagename-string--android-only)
   - [getAssistantService()](#getassistantservice--packagename-string--android-only)
-  - [supportsOnDeviceRecognition()](#supportsondevicerecognition-boolean-android-only)
+  - [supportsOnDeviceRecognition()](#supportsondevicerecognition-boolean)
   - [supportsRecording()](#supportsrecording-boolean-android-only)
   - [androidTriggerOfflineModelDownload()](#androidtriggerofflinemodeldownload-locale-string--promise-status-opened_dialog--download_success--download_canceled-message-string-)
   - [setCategoryIOS()](#setcategoryios-void-ios-only)

example/App.tsx

@@ -18,13 +18,13 @@ import {
   type ExpoSpeechRecognitionOptions,
   type AndroidIntentOptions,
   useSpeechRecognitionEvent,
-  AudioEncodingAndroidValue,
+  type AudioEncodingAndroidValue,
   TaskHintIOS,
   AVAudioSessionCategory,
   type AVAudioSessionCategoryValue,
   AVAudioSessionCategoryOptions,
   type AVAudioSessionCategoryOptionsValue,
-  SetCategoryOptions,
+  type SetCategoryOptions,
   AVAudioSessionMode,
   type AVAudioSessionModeValue,
   ExpoWebSpeechRecognition,
@@ -131,10 +131,6 @@ export default function App() {
     });
   };
 
-  const stopListening = () => {
-    ExpoSpeechRecognitionModule.stop();
-  };
-
   return (
     <SafeAreaView style={styles.container}>
       <StatusBar style="dark" translucent={false} />
@@ -191,12 +187,12 @@ export default function App() {
             <BigButton
               title="Stop"
               disabled={status !== "recognizing"}
-              onPress={stopListening}
+              onPress={ExpoSpeechRecognitionModule.stop}
             />
             <BigButton
               title="Abort"
               disabled={status !== "recognizing"}
-              onPress={() => ExpoSpeechRecognitionModule.abort()}
+              onPress={ExpoSpeechRecognitionModule.abort}
             />
           </View>
         )}
@@ -991,7 +987,7 @@ function WebSpeechAPIDemo() {
       reconizer.removeEventListener("error", handleError);
       reconizer.removeEventListener("end", handleEnd);
     };
-  }, [listening]);
+  }, [listening, reconizer]);
 
   const startListeningWeb = () => {
     setListening(true);

package.json

@@ -1,6 +1,6 @@
 {
   "name": "expo-speech-recognition",
-  "version": "0.2.16",
+  "version": "0.2.17",
   "description": "Speech Recognition for React Native Expo projects",
   "main": "build/index.js",
   "types": "build/index.d.ts",

src/ExpoSpeechRecognitionModule.ts

@@ -8,9 +8,31 @@ import type { ExpoSpeechRecognitionModuleType } from "./ExpoSpeechRecognitionMod
 
 // It loads the native module object from the JSI or falls back to
 // the bridge module (from NativeModulesProxy) if the remote debugger is on.
-export const ExpoSpeechRecognitionModule =
+const ExpoSpeechRecognitionNativeModule =
   requireNativeModule<ExpoSpeechRecognitionModuleType>("ExpoSpeechRecognition");
 
+export const ExpoSpeechRecognitionModule = {
+  ...ExpoSpeechRecognitionNativeModule,
+  // Avoid any function bindings when calling the native module
+  stop: () => ExpoSpeechRecognitionNativeModule.stop(),
+  abort: () => ExpoSpeechRecognitionNativeModule.abort(),
+  requestPermissionsAsync: () =>
+    ExpoSpeechRecognitionNativeModule.requestPermissionsAsync(),
+  getPermissionsAsync: () =>
+    ExpoSpeechRecognitionNativeModule.getPermissionsAsync(),
+  getStateAsync: () => ExpoSpeechRecognitionNativeModule.getStateAsync(),
+  getAssistantService: () =>
+    ExpoSpeechRecognitionNativeModule.getAssistantService(),
+  getDefaultRecognitionService: () =>
+    ExpoSpeechRecognitionNativeModule.getDefaultRecognitionService(),
+  getSpeechRecognitionServices: () =>
+    ExpoSpeechRecognitionNativeModule.getSpeechRecognitionServices(),
+  supportsOnDeviceRecognition: () =>
+    ExpoSpeechRecognitionNativeModule.supportsOnDeviceRecognition(),
+  supportsRecording: () =>
+    ExpoSpeechRecognitionNativeModule.supportsRecording(),
+};
+
 export const ExpoSpeechRecognitionModuleEmitter = new EventEmitter(
-  ExpoSpeechRecognitionModule ?? NativeModulesProxy.ExpoSpeechRecognition,
+  ExpoSpeechRecognitionNativeModule ?? NativeModulesProxy.ExpoSpeechRecognition,
 );