@jacques_gordon/expo-mapbox-navigation 2.0.6 → 2.0.7

package/android/src/main/java/expo/modules/mapboxnavigation/ExpoMapboxNavigationView.kt

@@ -1,11 +1,14 @@
 package expo.modules.mapboxnavigation
 
+import android.annotation.SuppressLint
 import android.content.Context
 import android.util.Log
 import com.mapbox.api.directions.v5.models.RouteOptions
 import com.mapbox.geojson.Point
 import com.mapbox.maps.MapView
 import com.mapbox.maps.Style
+import com.mapbox.maps.plugin.animation.camera        // ← correct import for .camera extension
+import com.mapbox.maps.plugin.locationcomponent.location
 import com.mapbox.navigation.base.extensions.applyDefaultNavigationOptions
 import com.mapbox.navigation.base.options.NavigationOptions
 import com.mapbox.navigation.base.route.NavigationRoute
@@ -65,16 +68,22 @@ class ExpoMapboxNavigationView(context: Context, appContext: AppContext) :
         val navigationOptions = NavigationOptions.Builder(context).build()
         mapboxNavigation = MapboxNavigationProvider.create(navigationOptions)
 
-        mapView.getMapboxMap().loadStyle(mapStyle ?: Style.MAPBOX_STREETS) { style ->
+        mapView.mapboxMap.loadStyle(mapStyle ?: Style.MAPBOX_STREETS) { style ->
+            // Route line components
             routeLineApi = MapboxRouteLineApi(MapboxRouteLineApiOptions.Builder().build())
-            routeLineView = MapboxRouteLineView(MapboxRouteLineViewOptions.Builder(context).build())
+            routeLineView = MapboxRouteLineView(
+                MapboxRouteLineViewOptions.Builder(context).build()
+            )
 
-            viewportDataSource = MapboxNavigationViewportDataSource(mapView.getMapboxMap())
+            // NavigationCamera — exactly as in official Mapbox example:
+            // NavigationCamera(mapboxMap, mapView.camera, viewportDataSource)
+            viewportDataSource = MapboxNavigationViewportDataSource(mapView.mapboxMap)
             navigationCamera = NavigationCamera(
-                mapView.getMapboxMap(),
-                mapView.camera,
+                mapView.mapboxMap,
+                mapView.camera,          // ← mapView.camera (Maps plugin extension)
                 viewportDataSource!!
             )
+
             registerObservers()
         }
     }
@@ -82,33 +91,54 @@ class ExpoMapboxNavigationView(context: Context, appContext: AppContext) :
     private fun registerObservers() {
         val nav = mapboxNavigation ?: return
 
+        // Routes observer
         nav.registerRoutesObserver(object : RoutesObserver {
             override fun onRoutesChanged(result: RoutesUpdatedResult) {
                 val routes = result.navigationRoutes
                 if (routes.isNotEmpty()) {
                     routeLineApi?.setNavigationRoutes(routes) { value ->
-                        mapView.getMapboxMap().getStyle()?.let { style ->
+                        mapView.mapboxMap.style?.let { style ->
                             routeLineView?.renderRouteDrawData(style, value)
                         }
                     }
+                    // Update viewport data source with the route
+                    viewportDataSource?.onRouteChanged(routes.first())
+                    viewportDataSource?.evaluate()
+
+                    // Issue #43 fix: safe camera following transition
                     safeCameraFollowing()
-                    emit("onRoutesReady", mapOf(
+
+                    emitEvent("onRoutesReady", mapOf(
                         "routeCount" to routes.size,
-                        "distanceMeters" to (routes.firstOrNull()?.directionsRoute?.distance() ?: 0.0),
-                        "durationSeconds" to (routes.firstOrNull()?.directionsRoute?.duration() ?: 0.0)
+                        "distanceMeters" to (routes.first().directionsRoute.distance() ?: 0.0),
+                        "durationSeconds" to (routes.first().directionsRoute.duration() ?: 0.0)
                     ))
+                } else {
+                    routeLineApi?.clearRouteLine { value ->
+                        mapView.mapboxMap.style?.let { style ->
+                            routeLineView?.renderClearRouteLineValue(style, value)
+                        }
+                    }
+                    viewportDataSource?.clearRouteData()
+                    viewportDataSource?.evaluate()
                 }
             }
         })
 
-        nav.registerRouteProgressObserver(RouteProgressObserver { progress ->
-            emit("onRouteProgressChanged", mapOf(
-                "distanceRemaining" to progress.distanceRemaining,
-                "durationRemaining" to progress.durationRemaining,
-                "distanceTraveled" to progress.distanceTraveled,
-                "fractionTraveled" to progress.fractionTraveled,
+        // Route progress observer
+        nav.registerRouteProgressObserver(RouteProgressObserver { routeProgress ->
+            // Update viewport data source with progress
+            viewportDataSource?.onRouteProgressChanged(routeProgress)
+            viewportDataSource?.evaluate()
+
+            emitEvent("onRouteProgressChanged", mapOf(
+                "distanceRemaining" to routeProgress.distanceRemaining,
+                "durationRemaining" to routeProgress.durationRemaining,
+                "distanceTraveled" to routeProgress.distanceTraveled,
+                "fractionTraveled" to routeProgress.fractionTraveled,
                 "currentStepDistanceRemaining" to
-                    (progress.currentLegProgress?.currentStepProgress?.distanceRemaining ?: 0f)
+                    (routeProgress.currentLegProgress
+                        ?.currentStepProgress?.distanceRemaining ?: 0f)
             ))
         })
     }
@@ -127,7 +157,7 @@ class ExpoMapboxNavigationView(context: Context, appContext: AppContext) :
     }
 
     // ── Issue #31: voice units ────────────────────────────────────────────────
-    private fun resolveUnitType(): String {
+    private fun resolveVoiceUnits(): String {
         return when (voiceUnits?.lowercase()) {
             "metric" -> "metric"
             "imperial" -> "imperial"
@@ -139,6 +169,8 @@ class ExpoMapboxNavigationView(context: Context, appContext: AppContext) :
         }
     }
 
+    // ── Route request ─────────────────────────────────────────────────────────
+    @SuppressLint("MissingPermission")
     private fun fetchRoutes() {
         val nav = mapboxNavigation ?: return
         if (coordinates.size < 2) return
@@ -146,24 +178,25 @@ class ExpoMapboxNavigationView(context: Context, appContext: AppContext) :
         val points = coordinates.map { coord ->
             Point.fromLngLat(coord["longitude"] ?: 0.0, coord["latitude"] ?: 0.0)
         }
-
         val locale = language?.let { Locale.forLanguageTag(it) } ?: Locale.getDefault()
 
         val builder = RouteOptions.builder()
             .applyDefaultNavigationOptions()
             .language(locale.toLanguageTag())
-            .voiceUnits(resolveUnitType())
+            .voiceUnits(resolveVoiceUnits())   // Issue #31 fix
             .coordinatesList(points)
 
         waypointIndices?.let { builder.waypointIndicesList(it) }
         navigationProfile?.let {
             builder.profile(if (it.startsWith("mapbox/")) it else "mapbox/$it")
         }
-        excludeTypes?.takeIf { it.isNotEmpty() }?.let { builder.exclude(it.joinToString(",")) }
+        excludeTypes?.takeIf { it.isNotEmpty() }?.let {
+            builder.exclude(it.joinToString(","))
+        }
         maxHeight?.let { builder.maxHeight(it) }
         maxWidth?.let { builder.maxWidth(it) }
 
-        // Nav SDK v3: NavigationRouterCallback uses @RouterOrigin String (not enum)
+        // Exact Nav SDK v3 callback signature from official Mapbox docs
         nav.requestRoutes(
             builder.build(),
             object : NavigationRouterCallback {
@@ -172,7 +205,7 @@ class ExpoMapboxNavigationView(context: Context, appContext: AppContext) :
                     @RouterOrigin routerOrigin: String
                 ) {
                     if (routes.isEmpty()) {
-                        emit("onRoutesFailed", mapOf("message" to "No routes returned"))
+                        emitEvent("onRoutesFailed", mapOf("message" to "No routes returned"))
                         return
                     }
                     nav.setNavigationRoutes(routes)
@@ -185,7 +218,7 @@ class ExpoMapboxNavigationView(context: Context, appContext: AppContext) :
                 ) {
                     val msg = reasons.firstOrNull()?.message ?: "Unknown error"
                     Log.e(TAG, "Route request failed: $msg")
-                    emit("onRoutesFailed", mapOf("message" to msg))
+                    emitEvent("onRoutesFailed", mapOf("message" to msg))
                 }
 
                 override fun onCanceled(
@@ -198,6 +231,17 @@ class ExpoMapboxNavigationView(context: Context, appContext: AppContext) :
         )
     }
 
+    // ── Expo event emitter (SDK 53 compatible) ────────────────────────────────
+    private fun emitEvent(name: String, data: Map<String, Any>) {
+        try {
+            // Expo modules core SDK 53: use appContext.eventEmitter
+            val emitter = appContext.eventEmitter
+            emitter?.emit(name, data)
+        } catch (e: Exception) {
+            Log.e(TAG, "Failed to emit event $name: ${e.message}")
+        }
+    }
+
     // ── Prop setters ──────────────────────────────────────────────────────────
     fun setCoordinates(coords: List<Map<String, Double>>) {
         coordinates = coords
@@ -216,30 +260,13 @@ class ExpoMapboxNavigationView(context: Context, appContext: AppContext) :
     fun setCustomRasterTileUrl(url: String?) { customRasterTileUrl = url }
     fun setCustomRasterAboveLayerId(layerId: String?) { customRasterAboveLayerId = layerId }
 
-    // ── Helpers ───────────────────────────────────────────────────────────────
-    private fun getAccessToken(): String? {
-        return try {
-            val resId = context.resources.getIdentifier(
-                "mapbox_access_token", "string", context.packageName
-            )
-            if (resId != 0) context.getString(resId) else null
-        } catch (e: Exception) { null }
-    }
-
-    // Correct Expo SDK v53 event emitter API
-    private fun emit(name: String, data: Map<String, Any>) {
-        try {
-            appContext.eventEmitter?.emit(name, data)
-        } catch (e: Exception) {
-            Log.e(TAG, "Failed to emit event $name: ${e.message}")
-        }
-    }
-
+    // ── Lifecycle ─────────────────────────────────────────────────────────────
     override fun onDetachedFromWindow() {
         super.onDetachedFromWindow()
+        routeLineApi?.cancel()
+        routeLineView?.cancel()
         mapboxNavigation?.stopTripSession()
         MapboxNavigationProvider.destroy()
-        routeLineApi?.cancel()
         mapView.onStop()
         mapView.onDestroy()
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jacques_gordon/expo-mapbox-navigation",
-  "version": "2.0.6",
+  "version": "2.0.7",
   "description": "Expo module for Mapbox Navigation SDK with 16KB page size support, NDK27, and Mapbox Maps v11.11.0+",
   "main": "build/index.js",
   "types": "build/index.d.ts",