react-native-pointr 9.9.2 → 9.9.3

package/CHANGELOG.md

@@ -4,6 +4,12 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [9.9.3] - 2026-07-22
+
+### Changed
+- Mobile SDK 9.9.3 integration.
+
+
 ## [9.9.2] - 2026-07-09
 
 ### Changed

package/android/build.gradle

@@ -93,7 +93,7 @@ dependencies {
   //noinspection GradleDynamicVersion
   implementation "com.facebook.react:react-android:0.82.1"
   implementation "org.jetbrains.kotlin:kotlin-stdlib:$kotlin_version"
-  implementation("com.pointrlabs:pointr:9.9.2")
+  implementation("com.pointrlabs:pointr:9.9.3")
   implementation ("org.jetbrains.kotlin:kotlin-reflect:$kotlin_version")
   implementation 'com.google.android.material:material:1.12.0'
   implementation 'androidx.constraintlayout:constraintlayout:2.2.1'

package/android/src/main/java/com/pointr/PTRMapWidgetManager.kt

@@ -245,6 +245,18 @@ class PTRMapWidgetManager(private val reactContext: ReactApplicationContext) :
         Choreographer.getInstance().postFrameCallback(frameCallback)
         Log.v(name, "Adding itself as listener to map widget")
         ptrMapWidgetFragment?.addListener(this)
+        // The SDK adds the fragment via an ASYNCHRONOUS commit. Force it to complete
+        // now, while the container view is still attached. Otherwise, if the widget is
+        // closed before that add lands (e.g. closed mid-load), the queued transaction
+        // runs afterwards against the detached container and crashes with
+        // "No view found for id 0x…". Running it here removes that pending-add window.
+        try {
+            (reactContext.currentActivity as? FragmentActivity)
+                ?.supportFragmentManager
+                ?.executePendingTransactions()
+        } catch (e: Exception) {
+            Log.e(name, "executePendingTransactions after fragment create failed: ${e.message}")
+        }
         return ptrMapWidgetFragment!!
     }
 
@@ -263,6 +275,51 @@ class PTRMapWidgetManager(private val reactContext: ReactApplicationContext) :
         return frameLayout
     }
 
+    /**
+     * Called by React Native when the <PTRMapWidget /> view is unmounted.
+     *
+     * The map widget is a Fragment attached to the host Activity's FragmentManager,
+     * so removing the React view alone does NOT tear it down — its dialogs/popups
+     * (welcome, onboarding, init-error, info page, …) live at the window level and
+     * would otherwise remain visible after the widget is closed. Removing the
+     * Fragment runs its onDestroyView(), which dismisses every dialog it owns, and
+     * cancelling the frame callback stops the per-frame command loop from leaking.
+     */
+    override fun onDropViewInstance(view: FrameLayout) {
+        super.onDropViewInstance(view)
+
+        // Stop the per-frame loop that keeps re-measuring / re-triggering commands.
+        Choreographer.getInstance().removeFrameCallback(frameCallback)
+
+        val fragment = ptrMapWidgetFragment
+        ptrMapWidgetFragment = null
+        // Reset so a subsequent open builds a fresh widget and re-runs its command.
+        commandDidExecute = false
+        commandTriggered = false
+
+        if (fragment != null) {
+            fragment.removeListener(this)
+            val fragmentManager =
+                (reactContext.currentActivity as? FragmentActivity)?.supportFragmentManager
+            if (fragmentManager != null && !fragmentManager.isDestroyed && fragment.isAdded) {
+                // Remove the fragment SYNCHRONOUSLY (commitNow) while the container view
+                // still exists. An async commit()/commitAllowingStateLoss() would run a
+                // beat later — after React Native has already detached this FrameLayout —
+                // and the FragmentManager would then try to (re)create the fragment's view
+                // into a container that is gone, crashing with "No view found for id".
+                // Wrapped defensively so a lifecycle race can degrade to a no-op, never a
+                // crash, on the app's main thread.
+                try {
+                    fragmentManager.beginTransaction()
+                        .remove(fragment)
+                        .commitNowAllowingStateLoss()
+                } catch (e: Exception) {
+                    Log.e(name, "Failed to remove map widget fragment on view drop: ${e.message}")
+                }
+            }
+        }
+    }
+
     /**
      * Handle "create" command (called from JS) and call createFragment method
      */

package/android/src/main/java/com/pointr/PointrModule.kt

@@ -155,6 +155,7 @@ class PointrModule(reactContext: ReactApplicationContext) :
 
     @ReactMethod
     fun stop() {
+        removePointrListeners()
         Pointr.getPointr()?.stop()
     }
 

package/ios/PTRMapWidgetContainerView.swift

@@ -339,6 +339,20 @@ import PointrKit
         }
     }
 
+    // React Native is unmounting this view when it is moved to a nil window.
+    // Tear the map widget down here so its popups/sheets cannot surface after the
+    // widget is closed: removing ourselves as a listener breaks the widget→container
+    // retain cycle, letting the view controller (and its whole view hierarchy,
+    // including any popup) deallocate instead of lingering.
+    override func willMove(toWindow newWindow: UIWindow?) {
+        super.willMove(toWindow: newWindow)
+        guard newWindow == nil, let mapWidget = mapWidget else { return }
+        mapWidget.removeListener(self)
+        mapWidget.view.removeFromSuperview()
+        wayfindingEventsAdapter = nil
+        self.mapWidget = nil
+    }
+
     private func applySDKConfigIfNeeded() {
         guard let jsonString = sdkConfig,
               let data = jsonString.data(using: .utf8),

package/ios/PTRNativeLibrary.swift

@@ -224,10 +224,21 @@ class PTRNativeLibrary: RCTEventEmitter, PTREventManagerDelegate {
         } else {
             let dispatchGroup = DispatchGroup()
             dispatchGroup.enter()
+            // Pointr.shared.start(with:completion:) does not fire its completion
+            // exactly once — it's invoked on every subsequent state change too
+            // (e.g. the transition triggered by a later stop() call also matches
+            // `state.rawValue <= 0`). Guard so dispatchGroup.leave() only ever
+            // runs once; otherwise a second leave() on an already-balanced group
+            // is a fatal "BUG IN CLIENT OF LIBDISPATCH" crash.
+            var hasLeftDispatchGroup = false
+            let dispatchGroupLock = NSLock()
             Pointr.shared.start(with: PTRNativeLibrary.params) { state in
-                if (state == .running || state.rawValue <= 0) {
-                  dispatchGroup.leave()
-                }
+                guard state == .running || state.rawValue <= 0 else { return }
+                dispatchGroupLock.lock()
+                defer { dispatchGroupLock.unlock() }
+                guard !hasLeftDispatchGroup else { return }
+                hasLeftDispatchGroup = true
+                dispatchGroup.leave()
             }
             dispatchGroup.wait()
             if Pointr.shared.state == .running {
@@ -239,9 +250,20 @@ class PTRNativeLibrary: RCTEventEmitter, PTREventManagerDelegate {
     }
     
     @objc func stop() {
-        Pointr.shared.positioningManager?.removeListener(self)
-        Pointr.shared.geofenceManager?.removeListener(self)
-        Pointr.shared.stop()
+        guard Pointr.shared.state == .running else {
+            // Nothing to tear down — avoids double-stop crashes if called twice
+            // in a row or before the SDK finished starting.
+            return
+        }
+        // This module reports `requiresMainQueueSetup() -> false`, so React Native
+        // invokes this method on a background queue. Pointr.shared.stop() tears down
+        // location/Bluetooth/map resources that must be released on the main thread,
+        // so hop over explicitly to avoid a crash.
+        DispatchQueue.main.async {
+            Pointr.shared.positioningManager?.removeListener(self)
+            Pointr.shared.geofenceManager?.removeListener(self)
+            Pointr.shared.stop()
+        }
     }
     
     @objc func requestPermissions() {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-native-pointr",
-  "version": "9.9.2",
+  "version": "9.9.3",
   "description": "Pointr React-Native Module",
   "main": "src/index",
   "files": [

package/react-native-pointr.podspec

@@ -16,7 +16,7 @@ Pod::Spec.new do |s|
 
   s.source_files = "ios/**/*.{h,m,mm,swift}"
 
-  s.dependency 'PointrKit', '9.9.2'
+  s.dependency 'PointrKit', '9.9.3'
 
   # Use install_modules_dependencies helper to install the dependencies if React Native version >=0.71.0.
   # See https://github.com/facebook/react-native/blob/febf6b7f33fdb4904669f99d795eba4c0f95d7bf/scripts/cocoapods/new_architecture.rb#L79.