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

package/CHANGELOG.md

@@ -1,3 +1,13 @@
+## 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
+
+* Revert "Beta ReactNativeConnect build: 18.0.36" ([609ad7d](https://github.com/aipoweredmarketer/react-native-acoustic-connect-beta/commit/609ad7d3244ec06f27dced5864d40585c5b3c9cf))
 ## 19.1.2 (2026-09-29)
 
 ### Reverts

package/android/build.gradle

@@ -253,22 +253,15 @@ dependencies {
   //noinspection GradleDynamicVersion
   implementation "io.github.go-acoustic:$acousticArtifact:$acousticVersion"
 
-  // No direct eocore/tealeaf compile dependency for the main sources: the
+  // No direct eocore/tealeaf compile dependency anywhere in this module: the
   // bridge only calls Connect's own public API (Connect.updateConfig/
   // getConfigItem*, ConnectMonitoringLevel, ConnectScreenviewType,
   // Connect.logDialogLayout, Connect.canCaptureUserEvents,
-  // Connect.canTakeScreenShot, Connect.getLayoutInfo, Connect.onDestroy, ...).
-  //
-  // android/src/test still needs `com.ibm.eo.util.JsonUtil` directly — it
-  // deliberately drives serialization through EOCore's real JsonUtil rather
-  // than stopping at the bridge boundary (see HybridAcousticConnectRNTest's
-  // "SDK signal serialization" region). That's a fixed historical pin, not
-  // `$acousticVersion`: eocore is no longer guaranteed to be published
-  // standalone for future Connect releases, and JsonUtil's hash-value
-  // behavior isn't expected to change, so there's nothing to gain by
-  // tracking the active Connect version here.
-  //noinspection GradleDynamicVersion
-  testCompileOnly "io.github.go-acoustic:eocore:11.0.36"
+  // Connect.canTakeScreenShot, Connect.getLayoutInfo, Connect.onDestroy, ...),
+  // and android/src/test asserts on toSignalPayload's own output directly
+  // instead of piping it through EOCore's real JsonUtil (see
+  // HybridAcousticConnectRNTest's "SDK signal payload conversion" region) —
+  // CA-159225.
 
   // Firebase Cloud Messaging — required by the push variant. The
   // `connect-push-fcm` AAR calls `com.google.firebase.messaging.*`

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

@@ -1,4 +1,4 @@
-#Tue Sep 29 15:34:17 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.2",
+    "version": "19.1.4",
     "workspaces": [
         "example",
         "Examples/bare-workflow"