react-native-acoustic-connect-beta 19.0.16 → 19.0.17

package/CHANGELOG.md

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

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

@@ -1,4 +1,4 @@
-#Mon Aug 31 14:13:24 GMT 2026
+#Wed Sep 02 13:28:45 GMT 2026
 UseWhiteList=true
 PrintScreen=3
 UseRandomSample=false

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

@@ -877,7 +877,27 @@ class HybridAcousticConnectRN : HybridAcousticConnectRNSpec(),
      * @param logicalPageName The logical page name to be set.
      * @return True if the operation was successful, false otherwise.
      */
+    /**
+     * Whether the bridge may touch capture at all.
+     *
+     * `disable()` reaches `Tealeaf.disable()`, which unregisters the activity
+     * lifecycle callbacks and clears the enabled flag — enough on iOS, where
+     * `<Connect>` only advances the current screen name. On Android the same
+     * component also drives capture explicitly on every navigation
+     * (`logScreenViewPageName` + `logScreenLayout`), and those paths used to run
+     * regardless of the flag: `setCurrentScreenName` calls through to
+     * `Tealeaf.resumeTealeaf`, which resumes logging, and `logScreenLayout`
+     * captured a full layout. The net effect was that `disable()` silenced
+     * capture on iOS but not on Android, for an app doing nothing unusual.
+     *
+     * Checked here rather than in JS so every caller is covered, including apps
+     * that call the bridge methods directly.
+     */
+    private fun isCaptureAllowed(): Boolean = Connect.isEnabled()
+
     override fun setCurrentScreenName(logicalPageName: String): Boolean {
+        // resumeConnect resumes logging, so this must not run while disabled.
+        if (!isCaptureAllowed()) return false
         val result = resumeConnect(getCurrentActivity(), logicalPageName, false)
         return result
     }
@@ -890,8 +910,14 @@ class HybridAcousticConnectRN : HybridAcousticConnectRNSpec(),
      * @return True if the operation was successful, false otherwise.
      */
     override fun logScreenViewContextLoad(logicalPageName: Variant_NullType_String?, referrer: Variant_NullType_String?): Boolean {
-        val result = logScreenview(getCurrentActivity()!!, logicalPageName?.asSecondOrNull().toString(), ScreenviewType.LOAD, referrer?.asSecondOrNull());
-        return result
+        // Emits a screenview message; nothing should reach the queue while disabled.
+        if (!isCaptureAllowed()) return false
+        // No activity means nothing to log a screenview against. This used to be
+        // a force-unwrap, which turned a backgrounded or not-yet-attached activity
+        // into a crash; the sample now calls this method directly, so the path is
+        // easy to reach from JS.
+        val activity = getCurrentActivity() ?: return false
+        return logScreenview(activity, logicalPageName?.asSecondOrNull().toString(), ScreenviewType.LOAD, referrer?.asSecondOrNull())
     }
 
     /**
@@ -902,8 +928,14 @@ class HybridAcousticConnectRN : HybridAcousticConnectRNSpec(),
      * @return True if the operation was successful, false otherwise.
      */
     override fun logScreenViewContextUnload(logicalPageName: Variant_NullType_String?, referrer: Variant_NullType_String?): Boolean {
-        val result = logScreenview(getCurrentActivity()!!, logicalPageName?.asSecondOrNull().toString(), ScreenviewType.UNLOAD, referrer?.asSecondOrNull());
-        return result
+        // Emits a screenview message; nothing should reach the queue while disabled.
+        if (!isCaptureAllowed()) return false
+        // No activity means nothing to log a screenview against. This used to be
+        // a force-unwrap, which turned a backgrounded or not-yet-attached activity
+        // into a crash; the sample now calls this method directly, so the path is
+        // easy to reach from JS.
+        val activity = getCurrentActivity() ?: return false
+        return logScreenview(activity, logicalPageName?.asSecondOrNull().toString(), ScreenviewType.UNLOAD, referrer?.asSecondOrNull())
     }
 
     /**
@@ -917,12 +949,28 @@ class HybridAcousticConnectRN : HybridAcousticConnectRNSpec(),
      * @return True if the operation was successful, false otherwise.
      */
     override fun logScreenLayout(name: String, delay: Double): Boolean {
+        // The wrapper calls this on every Android navigation.
+        if (!isCaptureAllowed()) return false
+        // Advisory on purpose: a false here means the screen name did not advance,
+        // which is not a reason to skip the screenview and layout that follow — they
+        // are what the caller asked for. Treating it as fatal would suppress capture
+        // in cases that capture fine today. The guard inside setCurrentScreenName is
+        // redundant against the check above and deliberately left in, so the method
+        // stays safe for callers that reach it directly.
         setCurrentScreenName(name)
+        // The 3-arg overload takes a nullable Activity, so this one is safe to pass
+        // through — unlike the 4-arg overload the context methods use.
         logScreenview(getCurrentActivity(), name, ScreenviewType.LOAD)
         var result = false
-        if (LayoutUtil.canCaptureUserEvents(null, name)) {
+        // The layout capture below needs a real activity: it used to go through
+        // Objects.requireNonNull, which threw when the activity had gone away
+        // between the check above and here. Skip the capture instead and report
+        // false — this runs on every Android navigation, so a throw here is a
+        // crash on an ordinary screen change.
+        val activity = getCurrentActivity()
+        if (activity != null && LayoutUtil.canCaptureUserEvents(null, name)) {
             result = logScreenLayout(
-                Objects.requireNonNull<Activity?>(getCurrentActivity()),
+                activity,
                 name,
                 resolveCaptureLayoutDelayMs(name, delay),
                 true

package/package.json

@@ -230,7 +230,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.0.16",
+    "version": "19.0.17",
     "workspaces": [
         "example",
         "Examples/bare-workflow"