expo-modules-autolinking - npm Package Compare versions

Comparing version 2.0.6 to 2.0.7

build/reactNativeConfig/reactNativeConfig.js

@@ -23,3 +23,12 @@ "use strict";
     };
-    const reactNativePath = dependencyRoots['react-native'];
+    // NOTE(@kitten): If this isn't resolved to be the realpath and is a symlink,
+    // the Cocoapods resolution will detect path mismatches and generate nonsensical
+    // relative paths that won't resolve
+    let reactNativePath;
+    try {
+        reactNativePath = await promises_1.default.realpath(dependencyRoots['react-native']);
+    }
+    catch {
+        reactNativePath = dependencyRoots['react-native'];
+    }
     const dependencyConfigs = await Promise.all(Object.entries(dependencyRoots).map(async ([name, packageRoot]) => {
@@ -26,0 +35,0 @@ const config = await resolveDependencyConfigAsync(platform, name, packageRoot, projectConfig);

CHANGELOG.md

@@ -13,2 +13,6 @@ # Changelog
 
+## 2.0.7 — 2025-01-20
+
+_This version does not introduce any user-facing changes._
+
 ## 2.0.6 — 2025-01-19
@@ -19,2 +23,3 @@
 - Added Android `BaseReactPackage` for autolinking. ([#33773](https://github.com/expo/expo/pull/33773) by [@vonovak](https://github.com/vonovak))
+- Resolve `reactNativePath` to its realpath to prevent incorrect relative paths from being generated for isolated dependencies. ([#34203](https://github.com/expo/expo/pull/34203) by [@kitten](https://github.com/kitten))
 
@@ -21,0 +26,0 @@ ## 2.0.5 — 2025-01-10

package.json

 {
   "name": "expo-modules-autolinking",
-  "version": "2.0.6",
+  "version": "2.0.7",
   "description": "Scripts that autolink Expo modules.",
@@ -51,3 +51,3 @@ "main": "build/index.js",
   },
-  "gitHead": "0a0c6d35a691fc59c0d94675d159ac9e3dfd6f26"
+  "gitHead": "eb1c1b83e79921d9fa1b0da1a20a8f24bba2f804"
 }

src/reactNativeConfig/reactNativeConfig.ts

@@ -36,4 +36,13 @@ import fs from 'fs/promises';
   };
-  const reactNativePath = dependencyRoots['react-native'];
 
+  // NOTE(@kitten): If this isn't resolved to be the realpath and is a symlink,
+  // the Cocoapods resolution will detect path mismatches and generate nonsensical
+  // relative paths that won't resolve
+  let reactNativePath: string;
+  try {
+    reactNativePath = await fs.realpath(dependencyRoots['react-native']);
+  } catch {
+    reactNativePath = dependencyRoots['react-native'];
+  }
+
   const dependencyConfigs = await Promise.all(
@@ -40,0 +49,0 @@ Object.entries(dependencyRoots).map(async ([name, packageRoot]) => {

No alert changes

Improved metrics

Total package byte prevSize

388287

0.38%

Lines of code

5339

0.32%

No dependency changes