@jacques_gordon/expo-mapbox-navigation 2.2.18 → 2.2.19

package/app.plugin.js

@@ -318,19 +318,34 @@ def _expo_mapbox_nav_add_spm(installer)
 end
 `;
 
-      // Find the last post_install block and add our call inside it,
-      // or add a new post_install block if none exists.
-      if (podfile.includes('post_install do |installer|')) {
-        // Add our helper def before the first post_install
-        // and our call inside the existing post_install
+      // Find the last post_integrate block and add our call inside it,
+      // or add a new post_integrate block if none exists.
+      //
+      // WHY post_integrate INSTEAD OF post_install:
+      // React Native/Expo's own CocoaPods post_install logic runs a step
+      // that re-derives and regenerates each pod target's SPM package
+      // product dependencies (visible in the pod install log as
+      // "[SPM] Cleaning old SPM dependencies from Pods project" / "[SPM]
+      // Adding SPM dependencies to Pods project"). Since our hook was
+      // previously injected at the START of the existing post_install
+      // block, it ran BEFORE that RN/Expo step — which then silently
+      // dropped our manually-added MapboxNavigationNative product
+      // dependency (while, by chance, preserving MapboxNavigationCore and
+      // MapboxNavigationUIKit, since something in RN/Expo's own derivation
+      // logic happens to recognize those two but not a product that's
+      // needed only transitively). CocoaPods' post_integrate hook is
+      // documented to run strictly after all post_install hooks AND after
+      // the Pods project has already been saved to disk, specifically so
+      // that later hooks can alter it after the fact. Injecting here
+      // guarantees our changes are applied last and are not overwritten.
+      if (podfile.includes('post_integrate do |installer|')) {
         podfile = spmHook + podfile.replace(
-          'post_install do |installer|',
-          'post_install do |installer|\n  _expo_mapbox_nav_add_spm(installer)'
+          'post_integrate do |installer|',
+          'post_integrate do |installer|\n  _expo_mapbox_nav_add_spm(installer)'
         );
       } else {
-        // No post_install block — add both the helper and a new block
         podfile = podfile + spmHook + `
-post_install do |installer|
+post_integrate do |installer|
   _expo_mapbox_nav_add_spm(installer)
 end
 `;

package/package.json

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

package/plugin/src/index.js

@@ -318,19 +318,34 @@ def _expo_mapbox_nav_add_spm(installer)
 end
 `;
 
-      // Find the last post_install block and add our call inside it,
-      // or add a new post_install block if none exists.
-      if (podfile.includes('post_install do |installer|')) {
-        // Add our helper def before the first post_install
-        // and our call inside the existing post_install
+      // Find the last post_integrate block and add our call inside it,
+      // or add a new post_integrate block if none exists.
+      //
+      // WHY post_integrate INSTEAD OF post_install:
+      // React Native/Expo's own CocoaPods post_install logic runs a step
+      // that re-derives and regenerates each pod target's SPM package
+      // product dependencies (visible in the pod install log as
+      // "[SPM] Cleaning old SPM dependencies from Pods project" / "[SPM]
+      // Adding SPM dependencies to Pods project"). Since our hook was
+      // previously injected at the START of the existing post_install
+      // block, it ran BEFORE that RN/Expo step — which then silently
+      // dropped our manually-added MapboxNavigationNative product
+      // dependency (while, by chance, preserving MapboxNavigationCore and
+      // MapboxNavigationUIKit, since something in RN/Expo's own derivation
+      // logic happens to recognize those two but not a product that's
+      // needed only transitively). CocoaPods' post_integrate hook is
+      // documented to run strictly after all post_install hooks AND after
+      // the Pods project has already been saved to disk, specifically so
+      // that later hooks can alter it after the fact. Injecting here
+      // guarantees our changes are applied last and are not overwritten.
+      if (podfile.includes('post_integrate do |installer|')) {
         podfile = spmHook + podfile.replace(
-          'post_install do |installer|',
-          'post_install do |installer|\n  _expo_mapbox_nav_add_spm(installer)'
+          'post_integrate do |installer|',
+          'post_integrate do |installer|\n  _expo_mapbox_nav_add_spm(installer)'
         );
       } else {
-        // No post_install block — add both the helper and a new block
         podfile = podfile + spmHook + `
-post_install do |installer|
+post_integrate do |installer|
   _expo_mapbox_nav_add_spm(installer)
 end
 `;