@glassly/crust 0.1.0-dev.90 → 0.1.0-dev.97

package/android/src/main/AndroidManifest.xml

@@ -26,8 +26,7 @@
       android:permission="${applicationId}.permission.CRUST_NOTIFICATION_BRIDGE"
       android:process=":notif" />
 
-    <!-- The Mapbox Navigation SDK delivers turn-by-turn steps inline via
-         RouteProgress (no Messenger-IPC service), so the Google-era
-         NavInfoReceiverService is gone. Nothing to register here for nav. -->
+    <!-- Nothing to register for nav: the Mapbox Navigation SDK delivers
+         turn-by-turn steps inline via RouteProgress, with no IPC service. -->
   </application>
 </manifest>

package/android/src/main/java/com/glassly/crust/navigation/NavigationManager.kt

@@ -51,35 +51,33 @@ import com.mapbox.navigation.core.trip.session.RouteProgressObserver
  * Headless: never mounts a MapView. The mobile app does not render a
  * map natively (the miniapp WebView owns the visible map).
  *
- * ## Contract (UNCHANGED from the Google implementation)
+ * ## Contract
  *
- * The public surface — `start` / `stop` / `ensureTermsAccepted` /
- * `simulateDeviation` / `setWrongSidewalkOffset` /
- * `setSkipCrossings`, plus the `StartOptions` / `Callbacks` /
- * `ManeuverPayload` / `LocationPayload` / `RoutePoint` / `RouteStep` types —
- * is byte-identical to the Google version. CrustModule and everything above
- * the bridge (the `@glassly/miniapp` SDK, the PivotEngine) are unaffected by
- * this migration. See issues/mapbox-navigation-migration.md.
+ * The public surface is `start` / `stop` / `ensureTermsAccepted` /
+ * `simulateDeviation` / `setWrongSidewalkOffset` / `setSkipCrossings`, plus
+ * the `StartOptions` / `Callbacks` / `ManeuverPayload` / `LocationPayload` /
+ * `RoutePoint` / `RouteStep` types. Keep it in parity with
+ * `NavigationManager.swift` — CrustModule and everything above the bridge
+ * (the `@glassly/miniapp` SDK, the PivotEngine) code against this shape.
  *
- * ## How Mapbox differs from Google (mapped onto the same contract)
+ * ## Mapbox behaviors this wraps
  *
  * - **Off-route + reroute is automatic.** Mapbox detects off-route and
  *   re-requests a route by default. We surface `OffRouteObserver` →
  *   `onOffRoute`, `RerouteState` transitions → `onRerouting`, and
- *   `RoutesObserver` (new route) → `onRoute`. No hand-rolled 30m
- *   perpendicular detector for the live trip.
+ *   `RoutesObserver` (new route) → `onRoute`. Do not hand-roll a
+ *   perpendicular-distance detector for the live trip.
  * - **Steps arrive inline** on each `RouteProgress` / off the
- *   `NavigationRoute` directlegs — no Messenger-IPC NavInfo service
- *   (NavInfoReceiverService is deleted). Road names + maneuver come from
- *   the Directions `LegStep` / `BannerInstructions`.
+ *   `NavigationRoute` directlegs. Road names + maneuver come from the
+ *   Directions `LegStep` / `BannerInstructions`.
  * - **No T&C dialog.** Mapbox has no runtime terms screen, so
  *   `ensureTermsAccepted` resolves immediately `true` — keeping the
  *   callback shape the miniapp's `requestPermission()` expects.
- * - **Enhanced (map-matched) location** replaces RoadSnappedLocationProvider
- *   via `LocationObserver.onNewLocationMatcherResult`.
+ * - **Enhanced (map-matched) location** arrives via
+ *   `LocationObserver.onNewLocationMatcherResult`.
  * - **Simulation** uses `MapboxReplayer` + `ReplayRouteMapper`; the three
- *   dev walkers (deviate / wrong-sidewalk / skip-crossings) are
- *   re-implemented by pushing synthetic `ReplayEventUpdateLocation`s.
+ *   dev walkers (deviate / wrong-sidewalk / skip-crossings) push synthetic
+ *   `ReplayEventUpdateLocation`s.
  */
 @OptIn(ExperimentalPreviewMapboxNavigationAPI::class)
 object NavigationManager {
@@ -693,8 +691,7 @@ object NavigationManager {
     routeProgressObserver = progressObs
     nav.registerRouteProgressObserver(progressObs)
 
-    // LocationObserver — enhanced (map-matched) location replaces Google's
-    // RoadSnappedLocationProvider.
+    // LocationObserver — prefer the enhanced (map-matched) location.
     val locObs = object : LocationObserver {
       override fun onNewRawLocation(rawLocation: Location) { /* prefer matched */ }
       override fun onNewLocationMatcherResult(result: LocationMatcherResult) {
@@ -1012,7 +1009,6 @@ object NavigationManager {
 
   // =====================================================================
   // Maneuver mapping — Mapbox `type` + `modifier` → our ManeuverKind union.
-  // (Replaces NavInfoReceiverService.mapManeuver's Google-enum table.)
   // =====================================================================
 
   /**
@@ -1061,10 +1057,10 @@ object NavigationManager {
   /**
    * Dev-only: shove the simulated puck OFF the active route so Mapbox's
    * built-in off-route detection + automatic rerouting kick in — exactly
-   * like a real GPS divergence. We no longer hand-roll the off-route math
-   * or call requestRoutes ourselves; we just teleport the replay puck
-   * `DEVIATE_OFFSET_M` perpendicular to the route and let the SDK do the
-   * rest (OffRouteObserver → onOffRoute, RerouteState → onRerouting,
+   * like a real GPS divergence. Do not hand-roll the off-route math or call
+   * requestRoutes here; just teleport the replay puck `DEVIATE_OFFSET_M`
+   * perpendicular to the route and let the SDK do the rest
+   * (OffRouteObserver → onOffRoute, RerouteState → onRerouting,
    * RoutesObserver(REROUTE) → onRoute).
    *
    * Requires simulate mode (a replayer to inject into). On a real-GPS trip

package/ios/navigation/NavigationManager.swift

@@ -7,19 +7,17 @@ import MapboxNavigationCore
 
 /// NavigationManager (iOS)
 ///
-/// Singleton wrapper around the **Mapbox Navigation SDK v3 for iOS** (migrated
-/// from the Google Navigation SDK). Mirrors the Android
-/// `NavigationManager.kt`: owns the `MapboxNavigation` lifecycle, subscribes to
-/// the Combine publishers (route progress, location, arrival, rerouting), and
-/// fans out the SAME coarse callbacks the Google version exposed to
-/// `CrustModule`.
+/// Singleton wrapper around the **Mapbox Navigation SDK v3 for iOS**. Mirrors
+/// the Android `NavigationManager.kt`: owns the `MapboxNavigation` lifecycle,
+/// subscribes to the Combine publishers (route progress, location, arrival,
+/// rerouting), and fans out coarse callbacks to `CrustModule`.
 ///
-/// ## Contract (UNCHANGED from the Google implementation)
+/// ## Contract
 ///
-/// The public surface — `start` / `stop` / `requestPermission` /
+/// The public surface is `start` / `stop` / `requestPermission` /
 /// `simulateDeviation`, plus the `onEvent` / `onLocation` / `onRoute` payload
-/// shapes — is identical to the Google version, so `CrustModule.swift` and
-/// everything above it (the `@glassly/miniapp` SDK) does not change.
+/// shapes. Keep it in parity with Android — `CrustModule.swift` and everything
+/// above it (the `@glassly/miniapp` SDK) code against this shape.
 ///
 /// ## iOS-vs-Android API differences (why this isn't a line-for-line port)
 ///
@@ -33,9 +31,8 @@ import MapboxNavigationCore
 /// - Mapbox owns off-route detection + auto-reroute on iOS too, same as
 ///   Android — we observe, we don't re-derive.
 ///
-/// VERIFY-IN-XCODE markers flag the few exact field/case/method names that the
-/// (auth-gated) v3 docs didn't let me confirm. Everything else is from the
-/// official v3 examples.
+/// VERIFY-IN-XCODE markers flag exact field/case/method names that the
+/// auth-gated v3 docs could not confirm; check them against the SDK headers.
 // @MainActor: the Mapbox Navigation v3 API surface (MapboxNavigation,
 // NavigationController, SessionController, the routeProgress/locationMatching
 // publishers) is main-actor-isolated. All our Mapbox access already happens on
@@ -128,9 +125,9 @@ final class NavigationManager: NSObject {
 
   // MARK: - Permissions
 
-  /// Mapbox needs no Terms & Conditions dialog (that was Google-specific).
-  /// Resolve immediately with `true` so the JS permission gate passes —
-  /// mirrors Android `ensureTermsAccepted`, which is a no-op on Mapbox.
+  /// Mapbox needs no Terms & Conditions dialog. Resolve immediately with
+  /// `true` so the JS permission gate passes — mirrors Android
+  /// `ensureTermsAccepted`, which is a no-op on Mapbox.
   /// (Location authorization itself is handled by the standard iOS prompts
   /// driven from the NSLocation* Info.plist usage strings + CoreLocation.)
   func requestPermission(completion: @escaping (Bool) -> Void) {
@@ -421,8 +418,8 @@ final class NavigationManager: NSObject {
     }
   }
 
-  /// SDK-agnostic mode string → Mapbox Directions profile. Google's
-  /// `two_wheeler` has no Mapbox equivalent → driving (migration doc #3).
+  /// SDK-agnostic mode string → Mapbox Directions profile. `two_wheeler` has
+  /// no Mapbox equivalent, so it falls back to driving.
   private func profileFor(_ mode: String) -> ProfileIdentifier {
     switch mode.lowercased() {
     case "walking": return .walking

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@glassly/crust",
-  "version": "0.1.0-dev.90",
+  "version": "0.1.0-dev.97",
   "description": "Glassly Native Module",
   "main": "build/index.js",
   "types": "build/index.d.ts",
@@ -35,7 +35,7 @@
   "license": "MIT",
   "homepage": "https://github.com/fossephate/crust#readme",
   "dependencies": {
-    "@glassly/jspolyfill": "0.1.0-dev.90"
+    "@glassly/jspolyfill": "0.1.0-dev.97"
   },
   "devDependencies": {
     "@expo/config-plugins": ">=8.0.0",