@vercel/build-utils 8.4.9 → 8.4.11

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # @vercel/build-utils
 
+## 8.4.11
+
+### Patch Changes
+
+- Improve parsing of `turbo.json` ([#12266](https://github.com/vercel/vercel/pull/12266))
+
+- Fix corepack `packageManager` detection on monorepos ([#12258](https://github.com/vercel/vercel/pull/12258))
+
+## 8.4.10
+
+### Patch Changes
+
+- Parse `turbo.json` as json5 ([#12259](https://github.com/vercel/vercel/pull/12259))
+
 ## 8.4.9
 
 ### Patch Changes

package/dist/fs/run-user-scripts.d.ts

@@ -26,6 +26,12 @@ export interface ScanParentDirsResult {
      * or `undefined` if not found.
      */
     lockfileVersion?: number;
+    /**
+     * The contents of the `packageManager` field from `package.json` if found.
+     * The value may come from a different `package.json` file than the one
+     * specified by `packageJsonPath`, in the case of a monorepo.
+     */
+    packageJsonPackageManager?: string;
     /**
      * Whether Turborepo supports the `COREPACK_HOME` environment variable.
      * `undefined` if not a Turborepo project.

package/dist/fs/run-user-scripts.js

@@ -66,6 +66,7 @@ var import_errors = require("../errors");
 var import_node_version = require("./node-version");
 var import_read_config_file = require("./read-config-file");
 var import_clone_env = require("../clone-env");
+var import_json5 = __toESM(require("json5"));
 const runNpmInstallSema = new import_async_sema.default(1);
 const NO_OVERRIDE = {
   detectedLockfile: void 0,
@@ -237,7 +238,10 @@ async function scanParentDirs(destPath, readPackageJson = false, base = "/") {
     filename: "package.json"
   });
   const packageJson = readPackageJson && pkgJsonPath ? JSON.parse(await import_fs_extra.default.readFile(pkgJsonPath, "utf8")) : void 0;
-  const [yarnLockPath, npmLockPath, pnpmLockPath, bunLockPath] = await walkParentDirsMulti({
+  const {
+    paths: [yarnLockPath, npmLockPath, pnpmLockPath, bunLockPath],
+    packageJsonPackageManager
+  } = await walkParentDirsMulti({
     base,
     start: destPath,
     filenames: [
@@ -285,15 +289,16 @@ async function scanParentDirs(destPath, readPackageJson = false, base = "/") {
     lockfilePath = bunLockPath;
     lockfileVersion = 0;
   } else {
-    cliType = packageJson && rootProjectInfo ? detectPackageManagerNameWithoutLockfile(
-      packageJson,
+    cliType = detectPackageManagerNameWithoutLockfile(
+      packageJsonPackageManager,
       turboSupportsCorepackHome
-    ) : "npm";
+    );
   }
   const packageJsonPath = pkgJsonPath || void 0;
   return {
     cliType,
     packageJson,
+    packageJsonPackageManager,
     lockfilePath,
     lockfileVersion,
     packageJsonPath,
@@ -306,8 +311,16 @@ async function checkTurboSupportsCorepack(turboVersionRange, rootDir) {
   }
   const turboJsonPath = import_path.default.join(rootDir, "turbo.json");
   const turboJsonExists = await import_fs_extra.default.pathExists(turboJsonPath);
-  const turboJson = turboJsonExists ? JSON.parse(await import_fs_extra.default.readFile(turboJsonPath, "utf8")) : void 0;
-  return turboJson?.globalPassThroughEnv?.includes("COREPACK_HOME") || false;
+  let turboJson = null;
+  if (turboJsonExists) {
+    try {
+      turboJson = import_json5.default.parse(await import_fs_extra.default.readFile(turboJsonPath, "utf8"));
+    } catch (err) {
+      console.warn(`WARNING: Failed to parse turbo.json`);
+    }
+  }
+  const turboJsonIncludesCorepackHome = turboJson !== null && typeof turboJson === "object" && "globalPassThroughEnv" in turboJson && Array.isArray(turboJson.globalPassThroughEnv) && turboJson.globalPassThroughEnv.includes("COREPACK_HOME");
+  return turboJsonIncludesCorepackHome;
 }
 function turboVersionSpecifierSupportsCorepack(turboVersionSpecifier) {
   if (!(0, import_semver.validRange)(turboVersionSpecifier)) {
@@ -320,8 +333,7 @@ function turboVersionSpecifierSupportsCorepack(turboVersionSpecifier) {
   }
   return (0, import_semver.gte)(minTurboBeingUsed, versionSupportingCorepack);
 }
-function detectPackageManagerNameWithoutLockfile(packageJson, turboSupportsCorepackHome) {
-  const packageJsonPackageManager = packageJson.packageManager;
+function detectPackageManagerNameWithoutLockfile(packageJsonPackageManager, turboSupportsCorepackHome) {
   if (usingCorepack(
     process.env,
     packageJsonPackageManager,
@@ -378,17 +390,26 @@ async function walkParentDirsMulti({
   start,
   filenames
 }) {
+  let packageManager;
   for (const dir of traverseUpDirectories({ start, base })) {
     const fullPaths = filenames.map((f) => import_path.default.join(dir, f));
     const existResults = await Promise.all(
       fullPaths.map((f) => import_fs_extra.default.pathExists(f))
     );
     const foundOneOrMore = existResults.some((b) => b);
+    const packageJsonPath = import_path.default.join(dir, "package.json");
+    const packageJson = await import_fs_extra.default.readJSON(packageJsonPath).catch(() => null);
+    if (packageJson?.packageManager) {
+      packageManager = packageJson.packageManager;
+    }
     if (foundOneOrMore) {
-      return fullPaths.map((f, i) => existResults[i] ? f : void 0);
+      return {
+        paths: fullPaths.map((f, i) => existResults[i] ? f : void 0),
+        packageJsonPackageManager: packageManager
+      };
     }
   }
-  return [];
+  return { paths: [], packageJsonPackageManager: packageManager };
 }
 function isSet(v) {
   return v?.constructor?.name === "Set";
@@ -406,6 +427,7 @@ async function runNpmInstall(destPath, args = [], spawnOpts, meta, nodeVersion)
       packageJsonPath,
       packageJson,
       lockfileVersion,
+      packageJsonPackageManager,
       turboSupportsCorepackHome
     } = await scanParentDirs(destPath, true);
     if (!packageJsonPath) {
@@ -436,7 +458,7 @@ async function runNpmInstall(destPath, args = [], spawnOpts, meta, nodeVersion)
     opts.env = getEnvForPackageManager({
       cliType,
       lockfileVersion,
-      packageJsonPackageManager: packageJson?.packageManager,
+      packageJsonPackageManager,
       nodeVersion,
       env,
       packageJsonEngines: packageJson?.engines,
@@ -781,11 +803,17 @@ async function runCustomInstallCommand({
   spawnOpts
 }) {
   console.log(`Running "install" command: \`${installCommand}\`...`);
-  const { cliType, lockfileVersion, packageJson, turboSupportsCorepackHome } = await scanParentDirs(destPath, true);
+  const {
+    cliType,
+    lockfileVersion,
+    packageJson,
+    packageJsonPackageManager,
+    turboSupportsCorepackHome
+  } = await scanParentDirs(destPath, true);
   const env = getEnvForPackageManager({
     cliType,
     lockfileVersion,
-    packageJsonPackageManager: packageJson?.packageManager,
+    packageJsonPackageManager,
     nodeVersion,
     env: spawnOpts?.env || {},
     packageJsonEngines: packageJson?.engines,
@@ -800,7 +828,13 @@ async function runCustomInstallCommand({
 }
 async function runPackageJsonScript(destPath, scriptNames, spawnOpts) {
   (0, import_assert.default)(import_path.default.isAbsolute(destPath));
-  const { packageJson, cliType, lockfileVersion, turboSupportsCorepackHome } = await scanParentDirs(destPath, true);
+  const {
+    packageJson,
+    cliType,
+    lockfileVersion,
+    packageJsonPackageManager,
+    turboSupportsCorepackHome
+  } = await scanParentDirs(destPath, true);
   const scriptName = getScriptName(
     packageJson,
     typeof scriptNames === "string" ? [scriptNames] : scriptNames
@@ -815,7 +849,7 @@ async function runPackageJsonScript(destPath, scriptNames, spawnOpts) {
     env: getEnvForPackageManager({
       cliType,
       lockfileVersion,
-      packageJsonPackageManager: packageJson?.packageManager,
+      packageJsonPackageManager,
       nodeVersion: void 0,
       env: (0, import_clone_env.cloneEnv)(process.env, spawnOpts?.env),
       packageJsonEngines: packageJson?.engines,