@jacques_gordon/expo-mapbox-navigation 2.0.5 → 2.0.6

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

@@ -5,20 +5,17 @@ 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.MapboxMap
 import com.mapbox.maps.Style
-import com.mapbox.maps.plugin.animation.camera
 import com.mapbox.navigation.base.extensions.applyDefaultNavigationOptions
-import com.mapbox.navigation.base.formatter.UnitType
 import com.mapbox.navigation.base.options.NavigationOptions
 import com.mapbox.navigation.base.route.NavigationRoute
 import com.mapbox.navigation.base.route.NavigationRouterCallback
 import com.mapbox.navigation.base.route.RouterFailure
+import com.mapbox.navigation.base.route.RouterOrigin
 import com.mapbox.navigation.core.MapboxNavigation
 import com.mapbox.navigation.core.MapboxNavigationProvider
 import com.mapbox.navigation.core.directions.session.RoutesObserver
 import com.mapbox.navigation.core.directions.session.RoutesUpdatedResult
-import com.mapbox.navigation.core.lifecycle.MapboxNavigationApp
 import com.mapbox.navigation.core.trip.session.RouteProgressObserver
 import com.mapbox.navigation.ui.maps.camera.NavigationCamera
 import com.mapbox.navigation.ui.maps.camera.data.MapboxNavigationViewportDataSource
@@ -36,17 +33,14 @@ private const val TAG = "ExpoMapboxNavigation"
 class ExpoMapboxNavigationView(context: Context, appContext: AppContext) :
     ExpoView(context, appContext) {
 
-    // ── UI ───────────────────────────────────────────────────────────────────
     private val mapView: MapView = MapView(context)
     private var navigationCamera: NavigationCamera? = null
     private var viewportDataSource: MapboxNavigationViewportDataSource? = null
     private var routeLineApi: MapboxRouteLineApi? = null
     private var routeLineView: MapboxRouteLineView? = null
-
-    // ── Navigation ───────────────────────────────────────────────────────────
     private var mapboxNavigation: MapboxNavigation? = null
 
-    // ── Props ────────────────────────────────────────────────────────────────
+    // Props
     private var coordinates: List<Map<String, Double>> = emptyList()
     private var waypointIndices: List<Int>? = null
     private var language: String? = null
@@ -67,51 +61,27 @@ class ExpoMapboxNavigationView(context: Context, appContext: AppContext) :
         setupNavigation()
     }
 
-    // ─────────────────────────────────────────────────────────────────────────
-    // Setup
-    // ─────────────────────────────────────────────────────────────────────────
-
     private fun setupNavigation() {
-        val accessToken = getAccessToken() ?: run {
-            Log.e(TAG, "Mapbox access token not found.")
-            return
-        }
-
-        // Create MapboxNavigation instance (Nav SDK v3 API)
-        val navigationOptions = NavigationOptions.Builder(context)
-            .build()
-
+        val navigationOptions = NavigationOptions.Builder(context).build()
         mapboxNavigation = MapboxNavigationProvider.create(navigationOptions)
 
-        // Setup map style + route line + camera once map is loaded
-        mapView.getMapboxMap().loadStyle(
-            mapStyle ?: Style.MAPBOX_STREETS
-        ) { style ->
-            setupRouteLineComponents(style)
-            setupCamera()
+        mapView.getMapboxMap().loadStyle(mapStyle ?: Style.MAPBOX_STREETS) { style ->
+            routeLineApi = MapboxRouteLineApi(MapboxRouteLineApiOptions.Builder().build())
+            routeLineView = MapboxRouteLineView(MapboxRouteLineViewOptions.Builder(context).build())
+
+            viewportDataSource = MapboxNavigationViewportDataSource(mapView.getMapboxMap())
+            navigationCamera = NavigationCamera(
+                mapView.getMapboxMap(),
+                mapView.camera,
+                viewportDataSource!!
+            )
             registerObservers()
         }
     }
 
-    private fun setupRouteLineComponents(style: Style) {
-        routeLineApi = MapboxRouteLineApi(MapboxRouteLineApiOptions.Builder().build())
-        routeLineView = MapboxRouteLineView(MapboxRouteLineViewOptions.Builder(context).build())
-    }
-
-    private fun setupCamera() {
-        val mapboxMap = mapView.getMapboxMap()
-        viewportDataSource = MapboxNavigationViewportDataSource(mapboxMap)
-        navigationCamera = NavigationCamera(
-            mapboxMap,
-            mapView.camera,
-            viewportDataSource!!
-        )
-    }
-
     private fun registerObservers() {
         val nav = mapboxNavigation ?: return
 
-        // Routes observer — updates route line on map
         nav.registerRoutesObserver(object : RoutesObserver {
             override fun onRoutesChanged(result: RoutesUpdatedResult) {
                 val routes = result.navigationRoutes
@@ -121,10 +91,8 @@ class ExpoMapboxNavigationView(context: Context, appContext: AppContext) :
                             routeLineView?.renderRouteDrawData(style, value)
                         }
                     }
-                    // Issue #43 fix: safe camera transition to following
                     safeCameraFollowing()
-
-                    sendEvent("onRoutesReady", mapOf(
+                    emit("onRoutesReady", mapOf(
                         "routeCount" to routes.size,
                         "distanceMeters" to (routes.firstOrNull()?.directionsRoute?.distance() ?: 0.0),
                         "durationSeconds" to (routes.firstOrNull()?.directionsRoute?.duration() ?: 0.0)
@@ -133,21 +101,19 @@ class ExpoMapboxNavigationView(context: Context, appContext: AppContext) :
             }
         })
 
-        // Route progress observer
         nav.registerRouteProgressObserver(RouteProgressObserver { progress ->
-            sendEvent("onRouteProgressChanged", mapOf(
+            emit("onRouteProgressChanged", mapOf(
                 "distanceRemaining" to progress.distanceRemaining,
                 "durationRemaining" to progress.durationRemaining,
                 "distanceTraveled" to progress.distanceTraveled,
                 "fractionTraveled" to progress.fractionTraveled,
-                "currentStepDistanceRemaining" to (progress.currentLegProgress?.currentStepProgress?.distanceRemaining ?: 0f)
+                "currentStepDistanceRemaining" to
+                    (progress.currentLegProgress?.currentStepProgress?.distanceRemaining ?: 0f)
             ))
         })
     }
 
-    // ─────────────────────────────────────────────────────────────────────────
-    // Issue #43 fix: safe camera transition
-    // ─────────────────────────────────────────────────────────────────────────
+    // ── Issue #43: safe camera transition ────────────────────────────────────
     private fun safeCameraFollowing() {
         try {
             navigationCamera?.requestNavigationCameraToFollowing(
@@ -160,9 +126,7 @@ class ExpoMapboxNavigationView(context: Context, appContext: AppContext) :
         }
     }
 
-    // ─────────────────────────────────────────────────────────────────────────
-    // Issue #31 fix: voice units
-    // ─────────────────────────────────────────────────────────────────────────
+    // ── Issue #31: voice units ────────────────────────────────────────────────
     private fun resolveUnitType(): String {
         return when (voiceUnits?.lowercase()) {
             "metric" -> "metric"
@@ -175,9 +139,6 @@ class ExpoMapboxNavigationView(context: Context, appContext: AppContext) :
         }
     }
 
-    // ─────────────────────────────────────────────────────────────────────────
-    // Route request
-    // ─────────────────────────────────────────────────────────────────────────
     private fun fetchRoutes() {
         val nav = mapboxNavigation ?: return
         if (coordinates.size < 2) return
@@ -187,37 +148,31 @@ class ExpoMapboxNavigationView(context: Context, appContext: AppContext) :
         }
 
         val locale = language?.let { Locale.forLanguageTag(it) } ?: Locale.getDefault()
-        val unitType = resolveUnitType()
 
         val builder = RouteOptions.builder()
             .applyDefaultNavigationOptions()
             .language(locale.toLanguageTag())
-            .voiceUnits(unitType)  // Issue #31 fix
+            .voiceUnits(resolveUnitType())
             .coordinatesList(points)
 
         waypointIndices?.let { builder.waypointIndicesList(it) }
-
         navigationProfile?.let {
-            val profile = if (it.startsWith("mapbox/")) it else "mapbox/$it"
-            builder.profile(profile)
-        }
-
-        excludeTypes?.takeIf { it.isNotEmpty() }?.let {
-            builder.exclude(it.joinToString(","))
+            builder.profile(if (it.startsWith("mapbox/")) it else "mapbox/$it")
         }
-
+        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)
         nav.requestRoutes(
             builder.build(),
             object : NavigationRouterCallback {
                 override fun onRoutesReady(
                     routes: List<NavigationRoute>,
-                    routerOrigin: com.mapbox.navigation.base.route.RouterOrigin
+                    @RouterOrigin routerOrigin: String
                 ) {
                     if (routes.isEmpty()) {
-                        sendEvent("onRoutesFailed", mapOf("message" to "No routes returned"))
+                        emit("onRoutesFailed", mapOf("message" to "No routes returned"))
                         return
                     }
                     nav.setNavigationRoutes(routes)
@@ -230,12 +185,12 @@ class ExpoMapboxNavigationView(context: Context, appContext: AppContext) :
                 ) {
                     val msg = reasons.firstOrNull()?.message ?: "Unknown error"
                     Log.e(TAG, "Route request failed: $msg")
-                    sendEvent("onRoutesFailed", mapOf("message" to msg))
+                    emit("onRoutesFailed", mapOf("message" to msg))
                 }
 
                 override fun onCanceled(
                     routeOptions: RouteOptions,
-                    routerOrigin: com.mapbox.navigation.base.route.RouterOrigin
+                    @RouterOrigin routerOrigin: String
                 ) {
                     Log.d(TAG, "Route request cancelled")
                 }
@@ -243,15 +198,11 @@ class ExpoMapboxNavigationView(context: Context, appContext: AppContext) :
         )
     }
 
-    // ─────────────────────────────────────────────────────────────────────────
-    // Prop setters
-    // ─────────────────────────────────────────────────────────────────────────
-
+    // ── Prop setters ──────────────────────────────────────────────────────────
     fun setCoordinates(coords: List<Map<String, Double>>) {
         coordinates = coords
         if (coords.size >= 2) fetchRoutes()
     }
-
     fun setWaypointIndices(indices: List<Int>?) { waypointIndices = indices }
     fun setLanguage(lang: String?) { language = lang }
     fun setVoiceUnits(units: String?) { voiceUnits = units }
@@ -265,10 +216,7 @@ class ExpoMapboxNavigationView(context: Context, appContext: AppContext) :
     fun setCustomRasterTileUrl(url: String?) { customRasterTileUrl = url }
     fun setCustomRasterAboveLayerId(layerId: String?) { customRasterAboveLayerId = layerId }
 
-    // ─────────────────────────────────────────────────────────────────────────
-    // Helpers
-    // ─────────────────────────────────────────────────────────────────────────
-
+    // ── Helpers ───────────────────────────────────────────────────────────────
     private fun getAccessToken(): String? {
         return try {
             val resId = context.resources.getIdentifier(
@@ -278,8 +226,13 @@ class ExpoMapboxNavigationView(context: Context, appContext: AppContext) :
         } catch (e: Exception) { null }
     }
 
-    private fun sendEvent(name: String, data: Map<String, Any>) {
-        appContext.eventEmitter?.emit(name, data)
+    // 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}")
+        }
     }
 
     override fun onDetachedFromWindow() {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jacques_gordon/expo-mapbox-navigation",
-  "version": "2.0.5",
+  "version": "2.0.6",
   "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",