@vercel/build-utils 8.3.3 → 8.3.5

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @vercel/build-utils
 
+## 8.3.5
+
+### Patch Changes
+
+- Revert "warn on mismatched corepack and detected package managers" ([#11879](https://github.com/vercel/vercel/pull/11879))
+
+## 8.3.4
+
+### Patch Changes
+
+- Revert "Fix corepack `packageManager` detection on monorepos" ([#11865](https://github.com/vercel/vercel/pull/11865))
+
 ## 8.3.3
 
 ### Patch Changes

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

@@ -26,12 +26,6 @@ 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;
 }
 export interface TraverseUpDirectoriesProps {
     /**
@@ -103,12 +97,11 @@ export declare function getEnvForPackageManager({ cliType, lockfileVersion, pack
  * Helper to get the binary paths that link to the used package manager.
  * Note: Make sure it doesn't contain any `console.log` calls.
  */
-export declare function getPathOverrideForPackageManager({ cliType, lockfileVersion, corepackPackageManager, corepackEnabled, }: {
+export declare function getPathOverrideForPackageManager({ cliType, lockfileVersion, corepackEnabled, nodeVersion, }: {
     cliType: CliType;
     lockfileVersion: number | undefined;
-    corepackPackageManager: string | undefined;
+    corepackEnabled: boolean;
     nodeVersion: NodeVersion | undefined;
-    corepackEnabled?: boolean;
 }): {
     /**
      * Which lockfile was detected.
@@ -124,15 +117,6 @@ export declare function getPathOverrideForPackageManager({ cliType, lockfileVers
      */
     path: string | undefined;
 };
-export declare function detectPackageManager(cliType: CliType, lockfileVersion: number | undefined): {
-    path: string;
-    detectedLockfile: string;
-    detectedPackageManager: string;
-} | {
-    path: undefined;
-    detectedLockfile: string;
-    detectedPackageManager: string;
-} | undefined;
 /**
  * Helper to get the binary paths that link to the used package manager.
  * Note: Make sure it doesn't contain any `console.log` calls.

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

@@ -28,7 +28,6 @@ var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__ge
 var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
 var run_user_scripts_exports = {};
 __export(run_user_scripts_exports, {
-  detectPackageManager: () => detectPackageManager,
   execCommand: () => execCommand,
   getEnvForPackageManager: () => getEnvForPackageManager,
   getNodeBinPath: () => getNodeBinPath,
@@ -65,11 +64,6 @@ var import_node_version = require("./node-version");
 var import_read_config_file = require("./read-config-file");
 var import_clone_env = require("../clone-env");
 const runNpmInstallSema = new import_async_sema.default(1);
-const NO_OVERRIDE = {
-  detectedLockfile: void 0,
-  detectedPackageManager: void 0,
-  path: void 0
-};
 function spawnAsync(command, args, opts = {}) {
   return new Promise((resolve, reject) => {
     const stderrLogs = [];
@@ -219,10 +213,7 @@ 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 {
-    paths: [yarnLockPath, npmLockPath, pnpmLockPath, bunLockPath],
-    packageJsonPackageManager
-  } = await walkParentDirsMulti({
+  const [yarnLockPath, npmLockPath, pnpmLockPath, bunLockPath] = await walkParentDirsMulti({
     base,
     start: destPath,
     filenames: [
@@ -261,21 +252,19 @@ async function scanParentDirs(destPath, readPackageJson = false, base = "/") {
     lockfilePath = bunLockPath;
     lockfileVersion = 0;
   } else {
-    cliType = detectPackageManagerNameWithoutLockfile(
-      packageJsonPackageManager
-    );
+    cliType = packageJson ? detectPackageManagerNameWithoutLockfile(packageJson) : "npm";
   }
   const packageJsonPath = pkgJsonPath || void 0;
   return {
     cliType,
     packageJson,
-    packageJsonPackageManager,
     lockfilePath,
     lockfileVersion,
     packageJsonPath
   };
 }
-function detectPackageManagerNameWithoutLockfile(packageJsonPackageManager) {
+function detectPackageManagerNameWithoutLockfile(packageJson) {
+  const packageJsonPackageManager = packageJson.packageManager;
   if (usingCorepack(process.env, packageJsonPackageManager)) {
     const corepackPackageManager = validateVersionSpecifier(
       packageJsonPackageManager
@@ -320,26 +309,17 @@ 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 {
-        paths: fullPaths.map((f, i) => existResults[i] ? f : void 0),
-        packageJsonPackageManager: packageManager
-      };
+      return fullPaths.map((f, i) => existResults[i] ? f : void 0);
     }
   }
-  return { paths: [], packageJsonPackageManager: packageManager };
+  return [];
 }
 function isSet(v) {
   return v?.constructor?.name === "Set";
@@ -352,12 +332,7 @@ async function runNpmInstall(destPath, args = [], spawnOpts, meta, nodeVersion)
   (0, import_assert.default)(import_path.default.isAbsolute(destPath));
   try {
     await runNpmInstallSema.acquire();
-    const {
-      cliType,
-      packageJsonPath,
-      lockfileVersion,
-      packageJsonPackageManager
-    } = await scanParentDirs(destPath);
+    const { cliType, packageJsonPath, packageJson, lockfileVersion } = await scanParentDirs(destPath, true);
     if (!packageJsonPath) {
       (0, import_debug.default)(
         `Skipping dependency installation because no package.json was found for ${destPath}`
@@ -386,7 +361,7 @@ async function runNpmInstall(destPath, args = [], spawnOpts, meta, nodeVersion)
     opts.env = getEnvForPackageManager({
       cliType,
       lockfileVersion,
-      packageJsonPackageManager,
+      packageJsonPackageManager: packageJson?.packageManager,
       nodeVersion,
       env
     });
@@ -450,9 +425,8 @@ function getEnvForPackageManager({
   } = getPathOverrideForPackageManager({
     cliType,
     lockfileVersion,
-    corepackPackageManager: packageJsonPackageManager,
-    nodeVersion,
-    corepackEnabled
+    corepackEnabled,
+    nodeVersion
   });
   if (corepackEnabled) {
     (0, import_debug.default)(
@@ -490,8 +464,10 @@ function getEnvForPackageManager({
   }
   return newEnv;
 }
-function detectPnpmVersion(lockfileVersion) {
+function detectPnpmVersion(lockfileVersion, corepackEnabled) {
   switch (true) {
+    case corepackEnabled:
+      return "corepack_enabled";
     case lockfileVersion === void 0:
       return "not found";
     case lockfileVersion === 5.3:
@@ -506,155 +482,99 @@ function detectPnpmVersion(lockfileVersion) {
       return "not found";
   }
 }
-function validLockfileForPackageManager(cliType, lockfileVersion, packageManagerVersion) {
-  const packageManagerMajorVersion = packageManagerVersion.major;
-  switch (cliType) {
-    case "npm":
-    case "bun":
-    case "yarn":
-      return true;
-    case "pnpm":
-      switch (packageManagerMajorVersion) {
-        case 9:
-          if ("9.0.0" === packageManagerVersion.version && lockfileVersion === 6) {
-            return false;
-          }
-          return [6, 7, 9].includes(lockfileVersion);
-        case 8:
-          return [6, 6.1].includes(lockfileVersion);
-        case 7:
-          return [5.3, 5.4].includes(lockfileVersion);
-        case 6:
-          return [5.3, 5.4].includes(lockfileVersion);
-        default:
-          return true;
-      }
-  }
+function shouldUseNpm7(lockfileVersion, nodeVersion) {
+  if (lockfileVersion === void 0)
+    return false;
+  return lockfileVersion >= 2 && (nodeVersion?.major || 0) < 16;
 }
 function getPathOverrideForPackageManager({
   cliType,
   lockfileVersion,
-  corepackPackageManager,
-  corepackEnabled = true
+  corepackEnabled,
+  nodeVersion
 }) {
-  const detectedPackageManger = detectPackageManager(cliType, lockfileVersion);
-  if (!corepackPackageManager) {
-    return detectedPackageManger ?? NO_OVERRIDE;
-  }
-  if (lockfileVersion === void 0 || !corepackEnabled) {
-    return NO_OVERRIDE;
-  }
-  if (validateCorepackPackageManager(
-    cliType,
-    lockfileVersion,
-    corepackPackageManager
-  )) {
-    return NO_OVERRIDE;
-  }
-  console.warn(
-    `WARN [package-manager-warning-1] Detected lockfile "${lockfileVersion}" which is not compatible with the intended corepack package manager "${corepackPackageManager}". Update your lockfile or change to a compatible corepack version.`
-  );
-  return NO_OVERRIDE;
-}
-function validateCorepackPackageManager(cliType, lockfileVersion, corepackPackageManager) {
-  const validatedCorepackPackageManager = validateVersionSpecifier(
-    corepackPackageManager
-  );
-  if (!validatedCorepackPackageManager) {
-    console.warn(
-      `WARN [package-manager-warning-2] Intended corepack defined package manager "${corepackPackageManager}" is not a valid semver value.`
-    );
-    return false;
-  }
-  if (cliType !== validatedCorepackPackageManager.packageName) {
-    console.warn(
-      `WARN [package-manager-warning-3] Detected package manager "${cliType}" does not match intended corepack defined package manager "${validatedCorepackPackageManager.packageName}". Change your lockfile or "package.json#packageManager" value to match.`
-    );
-    return false;
-  }
-  const corepackPackageManagerVersion = (0, import_semver.coerce)(
-    validatedCorepackPackageManager.packageVersionRange
-  );
-  if (corepackPackageManagerVersion === null) {
-    return true;
-  } else {
-    return validLockfileForPackageManager(
-      cliType,
-      lockfileVersion,
-      corepackPackageManagerVersion
-    );
-  }
-}
-function validateVersionSpecifier(version) {
-  if (!version) {
-    return void 0;
-  }
-  const [before, after, ...extra] = version.split("@");
-  if (extra.length) {
-    return void 0;
-  }
-  if (!before) {
-    return void 0;
-  }
-  if (!after) {
-    return void 0;
-  }
-  if (!(0, import_semver.validRange)(after)) {
-    return void 0;
-  }
-  return {
-    packageName: before,
-    packageVersionRange: after
+  const no_override = {
+    detectedLockfile: void 0,
+    detectedPackageManager: void 0,
+    path: void 0
   };
-}
-function detectPackageManager(cliType, lockfileVersion) {
   switch (cliType) {
     case "npm":
-      return void 0;
+      switch (true) {
+        case corepackEnabled:
+          return no_override;
+        case shouldUseNpm7(lockfileVersion, nodeVersion):
+          return {
+            path: "/node16/bin-npm7",
+            detectedLockfile: "package-lock.json",
+            detectedPackageManager: "npm 7+"
+          };
+        default:
+          return no_override;
+      }
     case "pnpm":
-      switch (detectPnpmVersion(lockfileVersion)) {
+      switch (detectPnpmVersion(lockfileVersion, corepackEnabled)) {
+        case "corepack_enabled":
+          return no_override;
         case "pnpm 7":
           return {
             path: "/pnpm7/node_modules/.bin",
             detectedLockfile: "pnpm-lock.yaml",
-            detectedPackageManager: "pnpm@7.x"
+            detectedPackageManager: "pnpm 7"
           };
         case "pnpm 8":
           return {
             path: "/pnpm8/node_modules/.bin",
             detectedLockfile: "pnpm-lock.yaml",
-            detectedPackageManager: "pnpm@8.x"
+            detectedPackageManager: "pnpm 8"
           };
         case "pnpm 9":
           return {
             path: "/pnpm9/node_modules/.bin",
             detectedLockfile: "pnpm-lock.yaml",
-            detectedPackageManager: "pnpm@9.x"
+            detectedPackageManager: "pnpm 9"
           };
         case "pnpm 6":
-          return {
-            // undefined because pnpm@6 is the current default in the build container
-            path: void 0,
-            detectedLockfile: "pnpm-lock.yaml",
-            detectedPackageManager: "pnpm 6"
-          };
         default:
-          return void 0;
+          return no_override;
       }
     case "bun":
-      return {
-        path: "/bun1",
-        detectedLockfile: "bun.lockb",
-        detectedPackageManager: "bun@1.x"
-      };
+      switch (true) {
+        case corepackEnabled:
+          return no_override;
+        default:
+          return {
+            path: "/bun1",
+            detectedLockfile: "bun.lockb",
+            detectedPackageManager: "Bun"
+          };
+      }
     case "yarn":
-      return {
-        path: void 0,
-        detectedLockfile: "yarn.lock",
-        detectedPackageManager: "yarn"
-      };
+      return no_override;
   }
 }
+function validateVersionSpecifier(version) {
+  if (!version) {
+    return void 0;
+  }
+  const [before, after, ...extra] = version.split("@");
+  if (extra.length) {
+    return void 0;
+  }
+  if (!before) {
+    return void 0;
+  }
+  if (!after) {
+    return void 0;
+  }
+  if (!(0, import_semver.validRange)(after)) {
+    return void 0;
+  }
+  return {
+    packageName: before,
+    packageVersionRange: after
+  };
+}
 function getPathForPackageManager({
   cliType,
   lockfileVersion,
@@ -662,15 +582,12 @@ function getPathForPackageManager({
   env
 }) {
   const corepackEnabled = env.ENABLE_EXPERIMENTAL_COREPACK === "1";
-  let overrides = getPathOverrideForPackageManager({
+  const overrides = getPathOverrideForPackageManager({
     cliType,
     lockfileVersion,
-    corepackPackageManager: void 0,
+    corepackEnabled,
     nodeVersion
   });
-  if (corepackEnabled) {
-    overrides = NO_OVERRIDE;
-  }
   const alreadyInPath = (newPath) => {
     const oldPath = env.PATH ?? "";
     return oldPath.split(import_path.default.delimiter).includes(newPath);
@@ -696,11 +613,14 @@ async function runCustomInstallCommand({
   spawnOpts
 }) {
   console.log(`Running "install" command: \`${installCommand}\`...`);
-  const { cliType, lockfileVersion, packageJsonPackageManager } = await scanParentDirs(destPath);
+  const { cliType, lockfileVersion, packageJson } = await scanParentDirs(
+    destPath,
+    true
+  );
   const env = getEnvForPackageManager({
     cliType,
     lockfileVersion,
-    packageJsonPackageManager,
+    packageJsonPackageManager: packageJson?.packageManager,
     nodeVersion,
     env: spawnOpts?.env || {}
   });
@@ -713,7 +633,10 @@ async function runCustomInstallCommand({
 }
 async function runPackageJsonScript(destPath, scriptNames, spawnOpts) {
   (0, import_assert.default)(import_path.default.isAbsolute(destPath));
-  const { packageJson, cliType, lockfileVersion, packageJsonPackageManager } = await scanParentDirs(destPath, true);
+  const { packageJson, cliType, lockfileVersion } = await scanParentDirs(
+    destPath,
+    true
+  );
   const scriptName = getScriptName(
     packageJson,
     typeof scriptNames === "string" ? [scriptNames] : scriptNames
@@ -728,7 +651,7 @@ async function runPackageJsonScript(destPath, scriptNames, spawnOpts) {
     env: getEnvForPackageManager({
       cliType,
       lockfileVersion,
-      packageJsonPackageManager,
+      packageJsonPackageManager: packageJson?.packageManager,
       nodeVersion: void 0,
       env: (0, import_clone_env.cloneEnv)(process.env, spawnOpts?.env)
     })
@@ -785,7 +708,6 @@ const installDependencies = (0, import_util.deprecate)(
 );
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {
-  detectPackageManager,
   execCommand,
   getEnvForPackageManager,
   getNodeBinPath,

package/dist/index.js

@@ -13072,7 +13072,7 @@ var require_readShebang = __commonJS({
 // ../../node_modules/.pnpm/semver@5.7.2/node_modules/semver/semver.js
 var require_semver = __commonJS({
   "../../node_modules/.pnpm/semver@5.7.2/node_modules/semver/semver.js"(exports2, module2) {
-    exports2 = module2.exports = SemVer2;
+    exports2 = module2.exports = SemVer;
     var debug2;
     if (typeof process === "object" && process.env && process.env.NODE_DEBUG && /\bsemver\b/i.test(process.env.NODE_DEBUG)) {
       debug2 = function() {
@@ -13205,7 +13205,7 @@ var require_semver = __commonJS({
           includePrerelease: false
         };
       }
-      if (version instanceof SemVer2) {
+      if (version instanceof SemVer) {
         return version;
       }
       if (typeof version !== "string") {
@@ -13219,7 +13219,7 @@ var require_semver = __commonJS({
         return null;
       }
       try {
-        return new SemVer2(version, options);
+        return new SemVer(version, options);
       } catch (er) {
         return null;
       }
@@ -13234,15 +13234,15 @@ var require_semver = __commonJS({
       var s = parse2(version.trim().replace(/^[=v]+/, ""), options);
       return s ? s.version : null;
     }
-    exports2.SemVer = SemVer2;
-    function SemVer2(version, options) {
+    exports2.SemVer = SemVer;
+    function SemVer(version, options) {
       if (!options || typeof options !== "object") {
         options = {
           loose: !!options,
           includePrerelease: false
         };
       }
-      if (version instanceof SemVer2) {
+      if (version instanceof SemVer) {
         if (version.loose === options.loose) {
           return version;
         } else {
@@ -13254,8 +13254,8 @@ var require_semver = __commonJS({
       if (version.length > MAX_LENGTH) {
         throw new TypeError("version is longer than " + MAX_LENGTH + " characters");
       }
-      if (!(this instanceof SemVer2)) {
-        return new SemVer2(version, options);
+      if (!(this instanceof SemVer)) {
+        return new SemVer(version, options);
       }
       debug2("SemVer", version, options);
       this.options = options;
@@ -13293,32 +13293,32 @@ var require_semver = __commonJS({
       this.build = m[5] ? m[5].split(".") : [];
       this.format();
     }
-    SemVer2.prototype.format = function() {
+    SemVer.prototype.format = function() {
       this.version = this.major + "." + this.minor + "." + this.patch;
       if (this.prerelease.length) {
         this.version += "-" + this.prerelease.join(".");
       }
       return this.version;
     };
-    SemVer2.prototype.toString = function() {
+    SemVer.prototype.toString = function() {
       return this.version;
     };
-    SemVer2.prototype.compare = function(other) {
+    SemVer.prototype.compare = function(other) {
       debug2("SemVer.compare", this.version, this.options, other);
-      if (!(other instanceof SemVer2)) {
-        other = new SemVer2(other, this.options);
+      if (!(other instanceof SemVer)) {
+        other = new SemVer(other, this.options);
       }
       return this.compareMain(other) || this.comparePre(other);
     };
-    SemVer2.prototype.compareMain = function(other) {
-      if (!(other instanceof SemVer2)) {
-        other = new SemVer2(other, this.options);
+    SemVer.prototype.compareMain = function(other) {
+      if (!(other instanceof SemVer)) {
+        other = new SemVer(other, this.options);
       }
       return compareIdentifiers(this.major, other.major) || compareIdentifiers(this.minor, other.minor) || compareIdentifiers(this.patch, other.patch);
     };
-    SemVer2.prototype.comparePre = function(other) {
-      if (!(other instanceof SemVer2)) {
-        other = new SemVer2(other, this.options);
+    SemVer.prototype.comparePre = function(other) {
+      if (!(other instanceof SemVer)) {
+        other = new SemVer(other, this.options);
       }
       if (this.prerelease.length && !other.prerelease.length) {
         return -1;
@@ -13345,7 +13345,7 @@ var require_semver = __commonJS({
         }
       } while (++i2);
     };
-    SemVer2.prototype.inc = function(release, identifier) {
+    SemVer.prototype.inc = function(release, identifier) {
       switch (release) {
         case "premajor":
           this.prerelease.length = 0;
@@ -13431,7 +13431,7 @@ var require_semver = __commonJS({
         loose = void 0;
       }
       try {
-        return new SemVer2(version, loose).inc(release, identifier).version;
+        return new SemVer(version, loose).inc(release, identifier).version;
       } catch (er) {
         return null;
       }
@@ -13475,19 +13475,19 @@ var require_semver = __commonJS({
     }
     exports2.major = major;
     function major(a, loose) {
-      return new SemVer2(a, loose).major;
+      return new SemVer(a, loose).major;
     }
     exports2.minor = minor;
     function minor(a, loose) {
-      return new SemVer2(a, loose).minor;
+      return new SemVer(a, loose).minor;
     }
     exports2.patch = patch;
     function patch(a, loose) {
-      return new SemVer2(a, loose).patch;
+      return new SemVer(a, loose).patch;
     }
     exports2.compare = compare;
     function compare(a, b, loose) {
-      return new SemVer2(a, loose).compare(new SemVer2(b, loose));
+      return new SemVer(a, loose).compare(new SemVer(b, loose));
     }
     exports2.compareLoose = compareLoose;
     function compareLoose(a, b) {
@@ -13610,7 +13610,7 @@ var require_semver = __commonJS({
       if (!m[2]) {
         this.semver = ANY;
       } else {
-        this.semver = new SemVer2(m[2], this.options.loose);
+        this.semver = new SemVer(m[2], this.options.loose);
       }
     };
     Comparator.prototype.toString = function() {
@@ -13622,7 +13622,7 @@ var require_semver = __commonJS({
         return true;
       }
       if (typeof version === "string") {
-        version = new SemVer2(version, this.options);
+        version = new SemVer(version, this.options);
       }
       return cmp(version, this.operator, this.semver, this.options);
     };
@@ -13919,7 +13919,7 @@ var require_semver = __commonJS({
         return false;
       }
       if (typeof version === "string") {
-        version = new SemVer2(version, this.options);
+        version = new SemVer(version, this.options);
       }
       for (var i2 = 0; i2 < this.set.length; i2++) {
         if (testSet(this.set[i2], version, this.options)) {
@@ -13973,7 +13973,7 @@ var require_semver = __commonJS({
         if (rangeObj.test(v)) {
           if (!max || maxSV.compare(v) === -1) {
             max = v;
-            maxSV = new SemVer2(max, options);
+            maxSV = new SemVer(max, options);
           }
         }
       });
@@ -13992,7 +13992,7 @@ var require_semver = __commonJS({
         if (rangeObj.test(v)) {
           if (!min || minSV.compare(v) === 1) {
             min = v;
-            minSV = new SemVer2(min, options);
+            minSV = new SemVer(min, options);
           }
         }
       });
@@ -14001,11 +14001,11 @@ var require_semver = __commonJS({
     exports2.minVersion = minVersion;
     function minVersion(range, loose) {
       range = new Range(range, loose);
-      var minver = new SemVer2("0.0.0");
+      var minver = new SemVer("0.0.0");
       if (range.test(minver)) {
         return minver;
       }
-      minver = new SemVer2("0.0.0-0");
+      minver = new SemVer("0.0.0-0");
       if (range.test(minver)) {
         return minver;
       }
@@ -14013,7 +14013,7 @@ var require_semver = __commonJS({
       for (var i2 = 0; i2 < range.set.length; ++i2) {
         var comparators = range.set[i2];
         comparators.forEach(function(comparator) {
-          var compver = new SemVer2(comparator.semver.version);
+          var compver = new SemVer(comparator.semver.version);
           switch (comparator.operator) {
             case ">":
               if (compver.prerelease.length === 0) {
@@ -14059,7 +14059,7 @@ var require_semver = __commonJS({
     }
     exports2.outside = outside;
     function outside(version, range, hilo, options) {
-      version = new SemVer2(version, options);
+      version = new SemVer(version, options);
       range = new Range(range, options);
       var gtfn, ltefn, ltfn, comp, ecomp;
       switch (hilo) {
@@ -14123,7 +14123,7 @@ var require_semver = __commonJS({
     }
     exports2.coerce = coerce2;
     function coerce2(version) {
-      if (version instanceof SemVer2) {
+      if (version instanceof SemVer) {
         return version;
       }
       if (typeof version !== "string") {
@@ -14304,7 +14304,7 @@ var require_cross_spawn = __commonJS({
 // ../../node_modules/.pnpm/semver@6.3.1/node_modules/semver/semver.js
 var require_semver2 = __commonJS({
   "../../node_modules/.pnpm/semver@6.3.1/node_modules/semver/semver.js"(exports2, module2) {
-    exports2 = module2.exports = SemVer2;
+    exports2 = module2.exports = SemVer;
     var debug2;
     if (typeof process === "object" && process.env && process.env.NODE_DEBUG && /\bsemver\b/i.test(process.env.NODE_DEBUG)) {
       debug2 = function() {
@@ -14446,7 +14446,7 @@ var require_semver2 = __commonJS({
           includePrerelease: false
         };
       }
-      if (version instanceof SemVer2) {
+      if (version instanceof SemVer) {
         return version;
       }
       if (typeof version !== "string") {
@@ -14460,7 +14460,7 @@ var require_semver2 = __commonJS({
         return null;
       }
       try {
-        return new SemVer2(version, options);
+        return new SemVer(version, options);
       } catch (er) {
         return null;
       }
@@ -14475,15 +14475,15 @@ var require_semver2 = __commonJS({
       var s = parse2(version.trim().replace(/^[=v]+/, ""), options);
       return s ? s.version : null;
     }
-    exports2.SemVer = SemVer2;
-    function SemVer2(version, options) {
+    exports2.SemVer = SemVer;
+    function SemVer(version, options) {
       if (!options || typeof options !== "object") {
         options = {
           loose: !!options,
           includePrerelease: false
         };
       }
-      if (version instanceof SemVer2) {
+      if (version instanceof SemVer) {
         if (version.loose === options.loose) {
           return version;
         } else {
@@ -14495,8 +14495,8 @@ var require_semver2 = __commonJS({
       if (version.length > MAX_LENGTH) {
         throw new TypeError("version is longer than " + MAX_LENGTH + " characters");
       }
-      if (!(this instanceof SemVer2)) {
-        return new SemVer2(version, options);
+      if (!(this instanceof SemVer)) {
+        return new SemVer(version, options);
       }
       debug2("SemVer", version, options);
       this.options = options;
@@ -14534,32 +14534,32 @@ var require_semver2 = __commonJS({
       this.build = m[5] ? m[5].split(".") : [];
       this.format();
     }
-    SemVer2.prototype.format = function() {
+    SemVer.prototype.format = function() {
       this.version = this.major + "." + this.minor + "." + this.patch;
       if (this.prerelease.length) {
         this.version += "-" + this.prerelease.join(".");
       }
       return this.version;
     };
-    SemVer2.prototype.toString = function() {
+    SemVer.prototype.toString = function() {
       return this.version;
     };
-    SemVer2.prototype.compare = function(other) {
+    SemVer.prototype.compare = function(other) {
       debug2("SemVer.compare", this.version, this.options, other);
-      if (!(other instanceof SemVer2)) {
-        other = new SemVer2(other, this.options);
+      if (!(other instanceof SemVer)) {
+        other = new SemVer(other, this.options);
       }
       return this.compareMain(other) || this.comparePre(other);
     };
-    SemVer2.prototype.compareMain = function(other) {
-      if (!(other instanceof SemVer2)) {
-        other = new SemVer2(other, this.options);
+    SemVer.prototype.compareMain = function(other) {
+      if (!(other instanceof SemVer)) {
+        other = new SemVer(other, this.options);
       }
       return compareIdentifiers(this.major, other.major) || compareIdentifiers(this.minor, other.minor) || compareIdentifiers(this.patch, other.patch);
     };
-    SemVer2.prototype.comparePre = function(other) {
-      if (!(other instanceof SemVer2)) {
-        other = new SemVer2(other, this.options);
+    SemVer.prototype.comparePre = function(other) {
+      if (!(other instanceof SemVer)) {
+        other = new SemVer(other, this.options);
       }
       if (this.prerelease.length && !other.prerelease.length) {
         return -1;
@@ -14586,9 +14586,9 @@ var require_semver2 = __commonJS({
         }
       } while (++i2);
     };
-    SemVer2.prototype.compareBuild = function(other) {
-      if (!(other instanceof SemVer2)) {
-        other = new SemVer2(other, this.options);
+    SemVer.prototype.compareBuild = function(other) {
+      if (!(other instanceof SemVer)) {
+        other = new SemVer(other, this.options);
       }
       var i2 = 0;
       do {
@@ -14608,7 +14608,7 @@ var require_semver2 = __commonJS({
         }
       } while (++i2);
     };
-    SemVer2.prototype.inc = function(release, identifier) {
+    SemVer.prototype.inc = function(release, identifier) {
       switch (release) {
         case "premajor":
           this.prerelease.length = 0;
@@ -14694,7 +14694,7 @@ var require_semver2 = __commonJS({
         loose = void 0;
       }
       try {
-        return new SemVer2(version, loose).inc(release, identifier).version;
+        return new SemVer(version, loose).inc(release, identifier).version;
       } catch (er) {
         return null;
       }
@@ -14738,19 +14738,19 @@ var require_semver2 = __commonJS({
     }
     exports2.major = major;
     function major(a, loose) {
-      return new SemVer2(a, loose).major;
+      return new SemVer(a, loose).major;
     }
     exports2.minor = minor;
     function minor(a, loose) {
-      return new SemVer2(a, loose).minor;
+      return new SemVer(a, loose).minor;
     }
     exports2.patch = patch;
     function patch(a, loose) {
-      return new SemVer2(a, loose).patch;
+      return new SemVer(a, loose).patch;
     }
     exports2.compare = compare;
     function compare(a, b, loose) {
-      return new SemVer2(a, loose).compare(new SemVer2(b, loose));
+      return new SemVer(a, loose).compare(new SemVer(b, loose));
     }
     exports2.compareLoose = compareLoose;
     function compareLoose(a, b) {
@@ -14758,8 +14758,8 @@ var require_semver2 = __commonJS({
     }
     exports2.compareBuild = compareBuild;
     function compareBuild(a, b, loose) {
-      var versionA = new SemVer2(a, loose);
-      var versionB = new SemVer2(b, loose);
+      var versionA = new SemVer(a, loose);
+      var versionB = new SemVer(b, loose);
       return versionA.compare(versionB) || versionA.compareBuild(versionB);
     }
     exports2.rcompare = rcompare;
@@ -14879,7 +14879,7 @@ var require_semver2 = __commonJS({
       if (!m[2]) {
         this.semver = ANY;
       } else {
-        this.semver = new SemVer2(m[2], this.options.loose);
+        this.semver = new SemVer(m[2], this.options.loose);
       }
     };
     Comparator.prototype.toString = function() {
@@ -14892,7 +14892,7 @@ var require_semver2 = __commonJS({
       }
       if (typeof version === "string") {
         try {
-          version = new SemVer2(version, this.options);
+          version = new SemVer(version, this.options);
         } catch (er) {
           return false;
         }
@@ -15213,7 +15213,7 @@ var require_semver2 = __commonJS({
       }
       if (typeof version === "string") {
         try {
-          version = new SemVer2(version, this.options);
+          version = new SemVer(version, this.options);
         } catch (er) {
           return false;
         }
@@ -15270,7 +15270,7 @@ var require_semver2 = __commonJS({
         if (rangeObj.test(v)) {
           if (!max || maxSV.compare(v) === -1) {
             max = v;
-            maxSV = new SemVer2(max, options);
+            maxSV = new SemVer(max, options);
           }
         }
       });
@@ -15289,7 +15289,7 @@ var require_semver2 = __commonJS({
         if (rangeObj.test(v)) {
           if (!min || minSV.compare(v) === 1) {
             min = v;
-            minSV = new SemVer2(min, options);
+            minSV = new SemVer(min, options);
           }
         }
       });
@@ -15298,11 +15298,11 @@ var require_semver2 = __commonJS({
     exports2.minVersion = minVersion;
     function minVersion(range, loose) {
       range = new Range(range, loose);
-      var minver = new SemVer2("0.0.0");
+      var minver = new SemVer("0.0.0");
       if (range.test(minver)) {
         return minver;
       }
-      minver = new SemVer2("0.0.0-0");
+      minver = new SemVer("0.0.0-0");
       if (range.test(minver)) {
         return minver;
       }
@@ -15310,7 +15310,7 @@ var require_semver2 = __commonJS({
       for (var i2 = 0; i2 < range.set.length; ++i2) {
         var comparators = range.set[i2];
         comparators.forEach(function(comparator) {
-          var compver = new SemVer2(comparator.semver.version);
+          var compver = new SemVer(comparator.semver.version);
           switch (comparator.operator) {
             case ">":
               if (compver.prerelease.length === 0) {
@@ -15356,7 +15356,7 @@ var require_semver2 = __commonJS({
     }
     exports2.outside = outside;
     function outside(version, range, hilo, options) {
-      version = new SemVer2(version, options);
+      version = new SemVer(version, options);
       range = new Range(range, options);
       var gtfn, ltefn, ltfn, comp, ecomp;
       switch (hilo) {
@@ -15420,7 +15420,7 @@ var require_semver2 = __commonJS({
     }
     exports2.coerce = coerce2;
     function coerce2(version, options) {
-      if (version instanceof SemVer2) {
+      if (version instanceof SemVer) {
         return version;
       }
       if (typeof version === "number") {
@@ -21786,11 +21786,6 @@ function cloneEnv(...envs) {
 
 // src/fs/run-user-scripts.ts
 var runNpmInstallSema = new import_async_sema4.default(1);
-var NO_OVERRIDE = {
-  detectedLockfile: void 0,
-  detectedPackageManager: void 0,
-  path: void 0
-};
 function spawnAsync(command, args, opts = {}) {
   return new Promise((resolve, reject) => {
     const stderrLogs = [];
@@ -21940,10 +21935,7 @@ async function scanParentDirs(destPath, readPackageJson = false, base = "/") {
     filename: "package.json"
   });
   const packageJson = readPackageJson && pkgJsonPath ? JSON.parse(await import_fs_extra7.default.readFile(pkgJsonPath, "utf8")) : void 0;
-  const {
-    paths: [yarnLockPath, npmLockPath, pnpmLockPath, bunLockPath],
-    packageJsonPackageManager
-  } = await walkParentDirsMulti({
+  const [yarnLockPath, npmLockPath, pnpmLockPath, bunLockPath] = await walkParentDirsMulti({
     base,
     start: destPath,
     filenames: [
@@ -21982,21 +21974,19 @@ async function scanParentDirs(destPath, readPackageJson = false, base = "/") {
     lockfilePath = bunLockPath;
     lockfileVersion = 0;
   } else {
-    cliType = detectPackageManagerNameWithoutLockfile(
-      packageJsonPackageManager
-    );
+    cliType = packageJson ? detectPackageManagerNameWithoutLockfile(packageJson) : "npm";
   }
   const packageJsonPath = pkgJsonPath || void 0;
   return {
     cliType,
     packageJson,
-    packageJsonPackageManager,
     lockfilePath,
     lockfileVersion,
     packageJsonPath
   };
 }
-function detectPackageManagerNameWithoutLockfile(packageJsonPackageManager) {
+function detectPackageManagerNameWithoutLockfile(packageJson) {
+  const packageJsonPackageManager = packageJson.packageManager;
   if (usingCorepack(process.env, packageJsonPackageManager)) {
     const corepackPackageManager = validateVersionSpecifier(
       packageJsonPackageManager
@@ -22041,26 +22031,17 @@ async function walkParentDirsMulti({
   start,
   filenames
 }) {
-  let packageManager;
   for (const dir of traverseUpDirectories({ start, base })) {
     const fullPaths = filenames.map((f) => import_path5.default.join(dir, f));
     const existResults = await Promise.all(
       fullPaths.map((f) => import_fs_extra7.default.pathExists(f))
     );
     const foundOneOrMore = existResults.some((b) => b);
-    const packageJsonPath = import_path5.default.join(dir, "package.json");
-    const packageJson = await import_fs_extra7.default.readJSON(packageJsonPath).catch(() => null);
-    if (packageJson?.packageManager) {
-      packageManager = packageJson.packageManager;
-    }
     if (foundOneOrMore) {
-      return {
-        paths: fullPaths.map((f, i) => existResults[i] ? f : void 0),
-        packageJsonPackageManager: packageManager
-      };
+      return fullPaths.map((f, i) => existResults[i] ? f : void 0);
     }
   }
-  return { paths: [], packageJsonPackageManager: packageManager };
+  return [];
 }
 function isSet(v) {
   return v?.constructor?.name === "Set";
@@ -22073,12 +22054,7 @@ async function runNpmInstall(destPath, args = [], spawnOpts, meta, nodeVersion)
   (0, import_assert6.default)(import_path5.default.isAbsolute(destPath));
   try {
     await runNpmInstallSema.acquire();
-    const {
-      cliType,
-      packageJsonPath,
-      lockfileVersion,
-      packageJsonPackageManager
-    } = await scanParentDirs(destPath);
+    const { cliType, packageJsonPath, packageJson, lockfileVersion } = await scanParentDirs(destPath, true);
     if (!packageJsonPath) {
       debug(
         `Skipping dependency installation because no package.json was found for ${destPath}`
@@ -22107,7 +22083,7 @@ async function runNpmInstall(destPath, args = [], spawnOpts, meta, nodeVersion)
     opts.env = getEnvForPackageManager({
       cliType,
       lockfileVersion,
-      packageJsonPackageManager,
+      packageJsonPackageManager: packageJson?.packageManager,
       nodeVersion,
       env
     });
@@ -22171,9 +22147,8 @@ function getEnvForPackageManager({
   } = getPathOverrideForPackageManager({
     cliType,
     lockfileVersion,
-    corepackPackageManager: packageJsonPackageManager,
-    nodeVersion,
-    corepackEnabled
+    corepackEnabled,
+    nodeVersion
   });
   if (corepackEnabled) {
     debug(
@@ -22211,8 +22186,10 @@ function getEnvForPackageManager({
   }
   return newEnv;
 }
-function detectPnpmVersion(lockfileVersion) {
+function detectPnpmVersion(lockfileVersion, corepackEnabled) {
   switch (true) {
+    case corepackEnabled:
+      return "corepack_enabled";
     case lockfileVersion === void 0:
       return "not found";
     case lockfileVersion === 5.3:
@@ -22227,154 +22204,98 @@ function detectPnpmVersion(lockfileVersion) {
       return "not found";
   }
 }
-function validLockfileForPackageManager(cliType, lockfileVersion, packageManagerVersion) {
-  const packageManagerMajorVersion = packageManagerVersion.major;
-  switch (cliType) {
-    case "npm":
-    case "bun":
-    case "yarn":
-      return true;
-    case "pnpm":
-      switch (packageManagerMajorVersion) {
-        case 9:
-          if ("9.0.0" === packageManagerVersion.version && lockfileVersion === 6) {
-            return false;
-          }
-          return [6, 7, 9].includes(lockfileVersion);
-        case 8:
-          return [6, 6.1].includes(lockfileVersion);
-        case 7:
-          return [5.3, 5.4].includes(lockfileVersion);
-        case 6:
-          return [5.3, 5.4].includes(lockfileVersion);
-        default:
-          return true;
-      }
-  }
+function shouldUseNpm7(lockfileVersion, nodeVersion) {
+  if (lockfileVersion === void 0)
+    return false;
+  return lockfileVersion >= 2 && (nodeVersion?.major || 0) < 16;
 }
 function getPathOverrideForPackageManager({
   cliType,
   lockfileVersion,
-  corepackPackageManager,
-  corepackEnabled = true
+  corepackEnabled,
+  nodeVersion
 }) {
-  const detectedPackageManger = detectPackageManager(cliType, lockfileVersion);
-  if (!corepackPackageManager) {
-    return detectedPackageManger ?? NO_OVERRIDE;
-  }
-  if (lockfileVersion === void 0 || !corepackEnabled) {
-    return NO_OVERRIDE;
-  }
-  if (validateCorepackPackageManager(
-    cliType,
-    lockfileVersion,
-    corepackPackageManager
-  )) {
-    return NO_OVERRIDE;
-  }
-  console.warn(
-    `WARN [package-manager-warning-1] Detected lockfile "${lockfileVersion}" which is not compatible with the intended corepack package manager "${corepackPackageManager}". Update your lockfile or change to a compatible corepack version.`
-  );
-  return NO_OVERRIDE;
-}
-function validateCorepackPackageManager(cliType, lockfileVersion, corepackPackageManager) {
-  const validatedCorepackPackageManager = validateVersionSpecifier(
-    corepackPackageManager
-  );
-  if (!validatedCorepackPackageManager) {
-    console.warn(
-      `WARN [package-manager-warning-2] Intended corepack defined package manager "${corepackPackageManager}" is not a valid semver value.`
-    );
-    return false;
-  }
-  if (cliType !== validatedCorepackPackageManager.packageName) {
-    console.warn(
-      `WARN [package-manager-warning-3] Detected package manager "${cliType}" does not match intended corepack defined package manager "${validatedCorepackPackageManager.packageName}". Change your lockfile or "package.json#packageManager" value to match.`
-    );
-    return false;
-  }
-  const corepackPackageManagerVersion = (0, import_semver2.coerce)(
-    validatedCorepackPackageManager.packageVersionRange
-  );
-  if (corepackPackageManagerVersion === null) {
-    return true;
-  } else {
-    return validLockfileForPackageManager(
-      cliType,
-      lockfileVersion,
-      corepackPackageManagerVersion
-    );
-  }
-}
-function validateVersionSpecifier(version) {
-  if (!version) {
-    return void 0;
-  }
-  const [before, after, ...extra] = version.split("@");
-  if (extra.length) {
-    return void 0;
-  }
-  if (!before) {
-    return void 0;
-  }
-  if (!after) {
-    return void 0;
-  }
-  if (!(0, import_semver2.validRange)(after)) {
-    return void 0;
-  }
-  return {
-    packageName: before,
-    packageVersionRange: after
+  const no_override = {
+    detectedLockfile: void 0,
+    detectedPackageManager: void 0,
+    path: void 0
   };
-}
-function detectPackageManager(cliType, lockfileVersion) {
   switch (cliType) {
     case "npm":
-      return void 0;
+      switch (true) {
+        case corepackEnabled:
+          return no_override;
+        case shouldUseNpm7(lockfileVersion, nodeVersion):
+          return {
+            path: "/node16/bin-npm7",
+            detectedLockfile: "package-lock.json",
+            detectedPackageManager: "npm 7+"
+          };
+        default:
+          return no_override;
+      }
     case "pnpm":
-      switch (detectPnpmVersion(lockfileVersion)) {
+      switch (detectPnpmVersion(lockfileVersion, corepackEnabled)) {
+        case "corepack_enabled":
+          return no_override;
         case "pnpm 7":
           return {
             path: "/pnpm7/node_modules/.bin",
             detectedLockfile: "pnpm-lock.yaml",
-            detectedPackageManager: "pnpm@7.x"
+            detectedPackageManager: "pnpm 7"
           };
         case "pnpm 8":
           return {
             path: "/pnpm8/node_modules/.bin",
             detectedLockfile: "pnpm-lock.yaml",
-            detectedPackageManager: "pnpm@8.x"
+            detectedPackageManager: "pnpm 8"
           };
         case "pnpm 9":
           return {
             path: "/pnpm9/node_modules/.bin",
             detectedLockfile: "pnpm-lock.yaml",
-            detectedPackageManager: "pnpm@9.x"
+            detectedPackageManager: "pnpm 9"
           };
         case "pnpm 6":
-          return {
-            // undefined because pnpm@6 is the current default in the build container
-            path: void 0,
-            detectedLockfile: "pnpm-lock.yaml",
-            detectedPackageManager: "pnpm 6"
-          };
         default:
-          return void 0;
+          return no_override;
       }
     case "bun":
-      return {
-        path: "/bun1",
-        detectedLockfile: "bun.lockb",
-        detectedPackageManager: "bun@1.x"
-      };
+      switch (true) {
+        case corepackEnabled:
+          return no_override;
+        default:
+          return {
+            path: "/bun1",
+            detectedLockfile: "bun.lockb",
+            detectedPackageManager: "Bun"
+          };
+      }
     case "yarn":
-      return {
-        path: void 0,
-        detectedLockfile: "yarn.lock",
-        detectedPackageManager: "yarn"
-      };
+      return no_override;
+  }
+}
+function validateVersionSpecifier(version) {
+  if (!version) {
+    return void 0;
+  }
+  const [before, after, ...extra] = version.split("@");
+  if (extra.length) {
+    return void 0;
+  }
+  if (!before) {
+    return void 0;
   }
+  if (!after) {
+    return void 0;
+  }
+  if (!(0, import_semver2.validRange)(after)) {
+    return void 0;
+  }
+  return {
+    packageName: before,
+    packageVersionRange: after
+  };
 }
 function getPathForPackageManager({
   cliType,
@@ -22383,15 +22304,12 @@ function getPathForPackageManager({
   env
 }) {
   const corepackEnabled = env.ENABLE_EXPERIMENTAL_COREPACK === "1";
-  let overrides = getPathOverrideForPackageManager({
+  const overrides = getPathOverrideForPackageManager({
     cliType,
     lockfileVersion,
-    corepackPackageManager: void 0,
+    corepackEnabled,
     nodeVersion
   });
-  if (corepackEnabled) {
-    overrides = NO_OVERRIDE;
-  }
   const alreadyInPath = (newPath) => {
     const oldPath = env.PATH ?? "";
     return oldPath.split(import_path5.default.delimiter).includes(newPath);
@@ -22417,11 +22335,14 @@ async function runCustomInstallCommand({
   spawnOpts
 }) {
   console.log(`Running "install" command: \`${installCommand}\`...`);
-  const { cliType, lockfileVersion, packageJsonPackageManager } = await scanParentDirs(destPath);
+  const { cliType, lockfileVersion, packageJson } = await scanParentDirs(
+    destPath,
+    true
+  );
   const env = getEnvForPackageManager({
     cliType,
     lockfileVersion,
-    packageJsonPackageManager,
+    packageJsonPackageManager: packageJson?.packageManager,
     nodeVersion,
     env: spawnOpts?.env || {}
   });
@@ -22434,7 +22355,10 @@ async function runCustomInstallCommand({
 }
 async function runPackageJsonScript(destPath, scriptNames, spawnOpts) {
   (0, import_assert6.default)(import_path5.default.isAbsolute(destPath));
-  const { packageJson, cliType, lockfileVersion, packageJsonPackageManager } = await scanParentDirs(destPath, true);
+  const { packageJson, cliType, lockfileVersion } = await scanParentDirs(
+    destPath,
+    true
+  );
   const scriptName = getScriptName(
     packageJson,
     typeof scriptNames === "string" ? [scriptNames] : scriptNames
@@ -22449,7 +22373,7 @@ async function runPackageJsonScript(destPath, scriptNames, spawnOpts) {
     env: getEnvForPackageManager({
       cliType,
       lockfileVersion,
-      packageJsonPackageManager,
+      packageJsonPackageManager: packageJson?.packageManager,
       nodeVersion: void 0,
       env: cloneEnv(process.env, spawnOpts?.env)
     })

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vercel/build-utils",
-  "version": "8.3.3",
+  "version": "8.3.5",
   "license": "Apache-2.0",
   "main": "./dist/index.js",
   "types": "./dist/index.d.js",