@next/codemod 16.3.0-canary.5 → 16.3.0-canary.51

package/bin/upgrade.js

@@ -500,6 +500,16 @@ function writeOverridesField(packageJson, packageManager, overrides) {
         }
     }
     else if (packageManager === 'pnpm') {
+        // pnpm v11 silently ignores `pnpm.overrides` in package.json. The
+        // canonical location moved to `pnpm-workspace.yaml#overrides`.
+        // See https://pnpm.io/settings and https://github.com/pnpm/pnpm/issues/11536.
+        // When the version cannot be detected, assume the current (v11+) layout
+        // since that's the surface where silently-dropped overrides hurt most.
+        const pnpmMajorVersion = (0, handle_package_1.getPnpmMajorVersion)();
+        if (pnpmMajorVersion === null || pnpmMajorVersion >= 11) {
+            writePnpmWorkspaceOverrides(overrides);
+            return;
+        }
         // pnpm supports pnpm.overrides and pnpm.resolutions
         if (packageJson.resolutions) {
             for (const [key, value] of entries) {
@@ -545,6 +555,28 @@ function writeOverridesField(packageJson, packageManager, overrides) {
         }
     }
 }
+function writePnpmWorkspaceOverrides(overrides) {
+    // Deferred require so `js-yaml` is only loaded when we hit the pnpm v11+
+    // branch (i.e. not for npm/yarn/bun/pnpm-v10 upgrades). The package is CJS,
+    // so a sync `require()` keeps this function synchronous.
+    const yaml = require('js-yaml');
+    const filePath = path_1.default.join(cwd, 'pnpm-workspace.yaml');
+    let doc = {};
+    if (fs_1.default.existsSync(filePath)) {
+        const existing = fs_1.default.readFileSync(filePath, 'utf8');
+        const parsed = yaml.load(existing);
+        if (parsed && typeof parsed === 'object' && !Array.isArray(parsed)) {
+            doc = parsed;
+        }
+    }
+    if (!doc.overrides || typeof doc.overrides !== 'object') {
+        doc.overrides = {};
+    }
+    for (const [key, value] of Object.entries(overrides)) {
+        doc.overrides[key] = value;
+    }
+    fs_1.default.writeFileSync(filePath, yaml.dump(doc));
+}
 function warnDependenciesOutOfRange(appPackageJson, versionMapping) {
     const allDirectDependencies = {
         ...appPackageJson.dependencies,

package/lib/handle-package.js

@@ -3,6 +3,8 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.getPackageManagerVersion = getPackageManagerVersion;
+exports.getPnpmMajorVersion = getPnpmMajorVersion;
 exports.getPkgManager = getPkgManager;
 exports.uninstallPackage = uninstallPackage;
 exports.installPackages = installPackages;
@@ -10,7 +12,49 @@ exports.runInstallation = runInstallation;
 exports.addPackageDependency = addPackageDependency;
 const find_up_1 = __importDefault(require("find-up"));
 const execa_1 = __importDefault(require("execa"));
+const node_child_process_1 = require("node:child_process");
 const node_path_1 = require("node:path");
+/**
+ * Get the full version string for the given package manager.
+ *
+ * First tries to parse from `npm_config_user_agent` (e.g., "pnpm/9.13.2 npm/? ..."),
+ * then falls back to spawning `<packageManager> --version`.
+ *
+ * Returns null if unable to determine the version.
+ *
+ * Mirrors `packages/create-next-app/helpers/get-pkg-manager.ts`.
+ */
+function getPackageManagerVersion(packageManager) {
+    const userAgent = process.env.npm_config_user_agent || '';
+    const userAgentMatch = userAgent.match(new RegExp(`${packageManager}/([\\d.]+[\\w.-]*)`));
+    if (userAgentMatch) {
+        return userAgentMatch[1];
+    }
+    try {
+        const version = (0, node_child_process_1.execSync)(`${packageManager} --version`, {
+            encoding: 'utf8',
+            stdio: ['pipe', 'pipe', 'ignore'],
+        }).trim();
+        if (/^\d+\.\d+\.\d+/.test(version)) {
+            return version;
+        }
+    }
+    catch {
+        // package manager not available or failed to run
+    }
+    return null;
+}
+/**
+ * Get the major version of pnpm being used.
+ * Returns null if unable to determine the version.
+ */
+function getPnpmMajorVersion() {
+    const version = getPackageManagerVersion('pnpm');
+    if (!version)
+        return null;
+    const major = parseInt(version.split('.')[0], 10);
+    return Number.isNaN(major) ? null : major;
+}
 function getPkgManager(baseDir) {
     try {
         const lockFile = find_up_1.default.sync([

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@next/codemod",
-  "version": "16.3.0-canary.5",
+  "version": "16.3.0-canary.51",
   "license": "MIT",
   "repository": {
     "type": "git",
@@ -14,6 +14,7 @@
     "find-up": "4.1.0",
     "globby": "11.0.1",
     "is-git-clean": "1.1.0",
+    "js-yaml": "4.2.0",
     "jscodeshift": "17.0.0",
     "picocolors": "1.0.0",
     "prompts": "2.4.2",
@@ -37,6 +38,7 @@
   "bin": "./bin/next-codemod.js",
   "devDependencies": {
     "@types/find-up": "4.0.0",
+    "@types/js-yaml": "4.0.9",
     "@types/jscodeshift": "0.11.0",
     "@types/prompts": "2.4.2",
     "@types/semver": "7.3.1",

package/transforms/next-experimental-turbo-to-turbopack.js

@@ -4,8 +4,12 @@
  * the new top-level `turbopack` configuration.
  *
  * It moves most properties from experimental.turbo to the top-level turbopack
- * property, with special handling for certain properties like memoryLimit, minify,
+ * property, with special handling for certain properties like minify,
  * treeShaking, and sourceMaps which become experimental.turbopack* properties instead.
+ *
+ * The `memoryLimit` option is no longer supported, so it is removed entirely in
+ * both its old location (`experimental.turbo.memoryLimit`) and the location the
+ * previous version of this codemod produced (`experimental.turbopackMemoryLimit`).
  */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.default = transformer;
@@ -13,11 +17,15 @@ const parser_1 = require("../lib/parser");
 const utils_1 = require("./lib/utils");
 // Properties that need to be moved to experimental.turbopack*
 const RENAMED_EXPERIMENTAL_PROPERTIES = {
-    memoryLimit: 'turbopackMemoryLimit',
     minify: 'turbopackMinify',
     treeShaking: 'turbopackTreeShaking',
     sourceMaps: 'turbopackSourceMaps',
 };
+// `memoryLimit` is no longer supported and is removed entirely. We drop it under
+// both its old `experimental.turbo.*` name and the `experimental.*` name produced
+// by the previous version of this codemod.
+const REMOVED_TURBO_PROPERTY = 'memoryLimit';
+const REMOVED_EXPERIMENTAL_PROPERTY = 'turbopackMemoryLimit';
 function transformer(file, _api, options) {
     const j = (0, parser_1.createParserFromPath)(file.path);
     const root = j(file.source);
@@ -56,6 +64,13 @@ function transformer(file, _api, options) {
         const regularProps = [];
         const specialProps = [];
         turboObj.properties.forEach((prop) => {
+            if (isStaticProperty(prop) &&
+                prop.key &&
+                prop.key.type === 'Identifier' &&
+                prop.key.name === REMOVED_TURBO_PROPERTY) {
+                // Drop: `memoryLimit` is no longer supported.
+                return;
+            }
             if (isStaticProperty(prop) &&
                 prop.key &&
                 prop.key.type === 'Identifier' &&
@@ -71,7 +86,10 @@ function transformer(file, _api, options) {
         const existingProps = experimentalObj.properties.filter((prop) => !(isStaticProperty(prop) &&
             prop.key &&
             prop.key.type === 'Identifier' &&
-            prop.key.name === 'turbo'));
+            // Drop the `turbo` object (moved above) and a pre-existing
+            // `turbopackMemoryLimit` (no longer supported).
+            (prop.key.name === 'turbo' ||
+                prop.key.name === REMOVED_EXPERIMENTAL_PROPERTY)));
         experimentalObj.properties = [...existingProps, ...specialProps];
         // If experimental has no properties, remove it
         if (experimentalObj.properties.length === 0) {
@@ -132,7 +150,13 @@ function transformer(file, _api, options) {
         else {
             return;
         }
-        // For special properties like memoryLimit, minify, etc.
+        // `memoryLimit` is no longer supported: remove the assignment entirely.
+        if (propName === REMOVED_TURBO_PROPERTY) {
+            j(path).remove();
+            hasChanges = true;
+            return;
+        }
+        // For special properties like minify, treeShaking, etc.
         if (propName && RENAMED_EXPERIMENTAL_PROPERTIES[propName]) {
             const newAssignment = j.assignmentExpression('=', j.memberExpression(j.memberExpression(j.identifier(varName), j.identifier('experimental')), j.identifier(RENAMED_EXPERIMENTAL_PROPERTIES[propName])), path.node.right);
             j(path).replaceWith(newAssignment);
@@ -195,6 +219,26 @@ function transformer(file, _api, options) {
             hasChanges = true;
         }
     });
+    // Remove `config.experimental.turbopackMemoryLimit = value` assignments left
+    // behind by the previous version of this codemod (no longer supported).
+    root
+        .find(j.AssignmentExpression, {
+        left: {
+            type: 'MemberExpression',
+            object: {
+                type: 'MemberExpression',
+                property: { type: 'Identifier', name: 'experimental' },
+            },
+            property: {
+                type: 'Identifier',
+                name: REMOVED_EXPERIMENTAL_PROPERTY,
+            },
+        },
+    })
+        .forEach((path) => {
+        j(path).remove();
+        hasChanges = true;
+    });
     // Only return a string if we changed the AST, otherwise return the original source
     return hasChanges ? root.toSource(options) : file.source;
 }