react-native-acoustic-connect-beta 19.1.3 → 19.1.4

package/CHANGELOG.md

@@ -1,3 +1,8 @@
+## 19.1.4 (2026-09-30)
+
+### Reverts
+
+* Revert "Beta ReactNativeConnect build: 18.0.36" ([609ad7d](https://github.com/aipoweredmarketer/react-native-acoustic-connect-beta/commit/609ad7d3244ec06f27dced5864d40585c5b3c9cf))
 ## 19.1.3 (2026-09-30)
 
 ### Reverts

package/android/src/main/assets/ConnectBasicConfig.properties

@@ -1,4 +1,4 @@
-#Wed Sep 30 09:13:37 GMT 2026
+#Wed Sep 30 13:22:22 GMT 2026
 UseWhiteList=true
 PrintScreen=3
 UseRandomSample=false

package/android/src/main/java/com/acousticconnectrn/HybridAcousticConnectRN.kt

@@ -819,32 +819,23 @@ class HybridAcousticConnectRN : HybridAcousticConnectRNSpec(),
     /**
      * Log click events on react native control.
      *
+     * The view lookup and the SDK call both run on the main looper — see
+     * [withResolvedView].
+     *
      * @param target    Target id of the control.
      * @param controlId Accessibility ID(virtual id).
-     * @return True if the operation was successful, false otherwise.
+     * @return True if the event was handed to the main looper, false without a
+     *   React context. It cannot report whether the view resolved.
      */
     override fun logClickEvent(target: Double, controlId: String): Boolean {
-        val viewTag = target.toInt()
-        return try {
-            val ctx = applicationContext ?: return false
-            val uiManager = UIManagerHelper.getUIManagerForReactTag(ctx, viewTag)
-                ?: return false
-            val view = uiManager.resolveView(viewTag) ?: return false
-
-            Handler(Looper.getMainLooper()).post {
-                val activity = getCurrentActivity() ?: return@post
-                if (view is EditText) {
-                    addFocusAndRegister(view, null, activity)
-                } else if (TextUtils.isEmpty(controlId)) {
-                    logEvent(view, "click")
-                } else {
-                    logEvent(view, "click", controlId)
-                }
+        return withResolvedView(target.toInt(), "logClickEvent") { view, activity ->
+            if (view is EditText) {
+                addFocusAndRegister(view, null, activity)
+            } else if (TextUtils.isEmpty(controlId)) {
+                logEvent(view, "click")
+            } else {
+                logEvent(view, "click", controlId)
             }
-            true
-        } catch (e: Exception) {
-            Log.v(TAG, "logClickEvent error: ${e.message}", e)
-            false
         }
     }
 
@@ -852,28 +843,16 @@ class HybridAcousticConnectRN : HybridAcousticConnectRNSpec(),
      * Log click events on react native control.
      *
      * @param target    Target id of the control.
-     * @return True if the operation was successful, false otherwise.
+     * @return True if the event was handed to the main looper, false without a
+     *   React context. It cannot report whether the view resolved.
      */
     fun logClickEvent(target: Double): Boolean {
-        val viewTag = target.toInt()
-        return try {
-            val ctx = applicationContext ?: return false
-            val uiManager = UIManagerHelper.getUIManagerForReactTag(ctx, viewTag)
-                ?: return false
-            val view = uiManager.resolveView(viewTag) ?: return false
-
-            Handler(Looper.getMainLooper()).post {
-                val activity = getCurrentActivity() ?: return@post
-                if (view is EditText) {
-                    addFocusAndRegister(view, null, activity)
-                } else {
-                    logEvent(view, "click")
-                }
+        return withResolvedView(target.toInt(), "logClickEvent") { view, activity ->
+            if (view is EditText) {
+                addFocusAndRegister(view, null, activity)
+            } else {
+                logEvent(view, "click")
             }
-            true
-        } catch (e: Exception) {
-            Log.v(TAG, "logClickEvent error: ${e.message}")
-            false
         }
     }
 
@@ -883,30 +862,59 @@ class HybridAcousticConnectRN : HybridAcousticConnectRNSpec(),
      * @param target    A valid native View Id for lookup.
      * @param controlId Accessibility ID(virtual id).
      * @param text      The input string.
-     * @return True if the operation was successful, false otherwise.
+     * @return True if the event was handed to the main looper, false without a
+     *   React context. It cannot report whether the view resolved.
      */
     override fun logTextChangeEvent(target: Double, controlId: String, text: Variant_NullType_String?): Boolean {
-        val viewTag = target.toInt()
-        return try {
-            val ctx = applicationContext ?: return false
-            val uiManager = UIManagerHelper.getUIManagerForReactTag(ctx, viewTag)
-                ?: return false
-            val view = uiManager.resolveView(viewTag) ?: return false
-
-            Handler(Looper.getMainLooper()).post {
-                val activity = getCurrentActivity() ?: return@post
-                if (view is EditText && view.onFocusChangeListener == null) {
-                    logEvent(view, TLF_ON_FOCUS_CHANGE_IN, controlId)
-                    addFocusAndRegister(view, controlId, activity)
-                }
+        return withResolvedView(target.toInt(), "logTextChangeEvent") { view, activity ->
+            if (view is EditText && view.onFocusChangeListener == null) {
+                logEvent(view, TLF_ON_FOCUS_CHANGE_IN, controlId)
+                addFocusAndRegister(view, controlId, activity)
             }
-            true
-        } catch (e: Exception) {
-            Log.v(TAG, "logTextChangeEvent error: ${e.message}")
-            false
         }
     }
 
+    /**
+     * Resolves the native view behind a React tag and hands it to [block], with
+     * the lookup and the block both running on the main looper.
+     *
+     * Nitro calls arrive on the JS thread, and Fabric's `resolveView` must run
+     * on the UI thread. The bridge used to resolve first and post second, so
+     * every tap logged `AssertionException: Expected to run on UI thread!` in
+     * debug builds. That assertion is soft — the view still came back — but
+     * the lookup was reading the mounting layer's view map while the UI thread
+     * mutates it.
+     *
+     * The whole block is guarded, not just the lookup. `getView` throws when
+     * the view was unmounted between the tap and this post running, and an
+     * exception escaping a posted block is uncaught on the main thread — a
+     * crash, where the old off-thread lookup only returned false.
+     *
+     * @param viewTag React tag of the touched view.
+     * @param api     Name of the bridge method, for the log line.
+     * @param block   Work to do with the resolved view and current activity;
+     *   skipped when either is unavailable.
+     * @return False without a React context, true once the work is posted.
+     */
+    private fun withResolvedView(
+        viewTag: Int,
+        api: String,
+        block: (View, Activity) -> Unit
+    ): Boolean {
+        val ctx = applicationContext ?: return false
+        runOnMain {
+            try {
+                val view = UIManagerHelper.getUIManagerForReactTag(ctx, viewTag)
+                    ?.resolveView(viewTag) ?: return@runOnMain
+                val activity = getCurrentActivity() ?: return@runOnMain
+                block(view, activity)
+            } catch (e: Exception) {
+                Log.v(TAG, "$api error: ${e.message}", e)
+            }
+        }
+        return true
+    }
+
     /**
      * Whether the bridge may touch capture at all.
      *

package/package.json

@@ -228,7 +228,7 @@
     "source": "src/index",
     "summary": "react-native ios android tealeaf connect cxa wxca er enhanced-replay",
     "types": "./lib/typescript/src/index.d.ts",
-    "version": "19.1.3",
+    "version": "19.1.4",
     "workspaces": [
         "example",
         "Examples/bare-workflow"