@nx/jest 22.2.0-beta.0 → 22.2.0-beta.2

package/migrations.json

@@ -29,6 +29,11 @@
       },
       "description": "Replace removed matcher aliases in Jest v30 with their corresponding matcher",
       "implementation": "./src/migrations/update-21-3-0/replace-removed-matcher-aliases"
+    },
+    "convert-jest-config-to-cjs": {
+      "version": "22.2.0-beta.2",
+      "description": "Convert jest.config.ts files from ESM to CJS syntax (export default -> module.exports, import -> require) for projects using CommonJS resolution to ensure correct loading under Node.js type-stripping.",
+      "implementation": "./src/migrations/update-22-2-0/convert-jest-config-to-cjs"
     }
   },
   "packageJsonUpdates": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nx/jest",
-  "version": "22.2.0-beta.0",
+  "version": "22.2.0-beta.2",
   "private": false,
   "description": "The Nx Plugin for Jest contains executors and generators allowing your workspace to use the powerful Jest testing capabilities.",
   "repository": {
@@ -37,8 +37,8 @@
   "dependencies": {
     "@jest/reporters": "^30.0.2",
     "@jest/test-result": "^30.0.2",
-    "@nx/devkit": "22.2.0-beta.0",
-    "@nx/js": "22.2.0-beta.0",
+    "@nx/devkit": "22.2.0-beta.2",
+    "@nx/js": "22.2.0-beta.2",
     "@phenomnomnominal/tsquery": "~5.0.1",
     "identity-obj-proxy": "3.0.0",
     "jest-config": "^30.0.2",
@@ -52,7 +52,7 @@
     "yargs-parser": "21.1.1"
   },
   "devDependencies": {
-    "nx": "22.2.0-beta.0"
+    "nx": "22.2.0-beta.2"
   },
   "publishConfig": {
     "access": "public"

package/src/migrations/update-22-2-0/convert-jest-config-to-cjs.d.ts

@@ -0,0 +1,23 @@
+import { Tree } from '@nx/devkit';
+/**
+ * Migration to convert jest.config.ts files from ESM to CJS syntax for projects
+ * using CommonJS resolution. This is needed because Node.js type-stripping
+ * in newer versions (22+, 24+) can cause issues with ESM syntax in .ts files
+ * when the project is configured for CommonJS.
+ *
+ * This migration only runs if @nx/jest/plugin is registered in nx.json.
+ *
+ * Conversions:
+ * - `export default { ... }` -> `module.exports = { ... }`
+ * - `import { x } from 'y'` -> `const { x } = require('y')`
+ * - `import x from 'y'` -> `const x = require('y').default ?? require('y')`
+ *
+ * ESM-only features that cannot be converted (will warn user):
+ * - `import.meta`
+ * - top-level `await`
+ *
+ * Projects with `type: module` in package.json will be warned as they are
+ * incompatible with @nx/jest/plugin which forces CommonJS resolution.
+ */
+export default function convertJestConfigToCjs(tree: Tree): Promise<() => void>;
+//# sourceMappingURL=convert-jest-config-to-cjs.d.ts.map

package/src/migrations/update-22-2-0/convert-jest-config-to-cjs.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"convert-jest-config-to-cjs.d.ts","sourceRoot":"","sources":["../../../../../../packages/jest/src/migrations/update-22-2-0/convert-jest-config-to-cjs.ts"],"names":[],"mappings":"AAAA,OAAO,EAOL,IAAI,EACL,MAAM,YAAY,CAAC;AAIpB;;;;;;;;;;;;;;;;;;;GAmBG;AACH,wBAA8B,sBAAsB,CAAC,IAAI,EAAE,IAAI,uBA+F9D"}

package/src/migrations/update-22-2-0/convert-jest-config-to-cjs.js

@@ -0,0 +1,204 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.default = convertJestConfigToCjs;
+const devkit_1 = require("@nx/devkit");
+const find_plugin_for_config_file_1 = require("@nx/devkit/src/utils/find-plugin-for-config-file");
+const path_1 = require("path");
+/**
+ * Migration to convert jest.config.ts files from ESM to CJS syntax for projects
+ * using CommonJS resolution. This is needed because Node.js type-stripping
+ * in newer versions (22+, 24+) can cause issues with ESM syntax in .ts files
+ * when the project is configured for CommonJS.
+ *
+ * This migration only runs if @nx/jest/plugin is registered in nx.json.
+ *
+ * Conversions:
+ * - `export default { ... }` -> `module.exports = { ... }`
+ * - `import { x } from 'y'` -> `const { x } = require('y')`
+ * - `import x from 'y'` -> `const x = require('y').default ?? require('y')`
+ *
+ * ESM-only features that cannot be converted (will warn user):
+ * - `import.meta`
+ * - top-level `await`
+ *
+ * Projects with `type: module` in package.json will be warned as they are
+ * incompatible with @nx/jest/plugin which forces CommonJS resolution.
+ */
+async function convertJestConfigToCjs(tree) {
+    // If @nx/jest/plugin not used, then there will not be any problems with graph construction, which
+    // is what we're trying to address.
+    if (!isJestPluginRegistered(tree))
+        return;
+    const { tsquery } = require('@phenomnomnominal/tsquery');
+    const jestConfigPaths = await (0, devkit_1.globAsync)(tree, ['**/jest.config.ts']);
+    const projectsWithEsmOnlyFeatures = [];
+    const projectsWithTypeModule = [];
+    const modifiedFiles = [];
+    for (const configPath of jestConfigPaths) {
+        // Skip config files that are excluded from the plugin via include/exclude patterns
+        const pluginRegistration = await (0, find_plugin_for_config_file_1.findPluginForConfigFile)(tree, '@nx/jest/plugin', configPath);
+        if (!pluginRegistration)
+            continue;
+        const projectRoot = (0, path_1.dirname)(configPath);
+        const packageJsonPath = (0, devkit_1.joinPathFragments)(projectRoot, 'package.json');
+        const rootPackageJsonPath = 'package.json';
+        // Check project-level package.json first, then root
+        let projectPackageJson = null;
+        let rootPackageJson = null;
+        if (tree.exists(packageJsonPath)) {
+            projectPackageJson = (0, devkit_1.readJson)(tree, packageJsonPath);
+        }
+        if (tree.exists(rootPackageJsonPath)) {
+            rootPackageJson = (0, devkit_1.readJson)(tree, rootPackageJsonPath);
+        }
+        const effectiveType = projectPackageJson?.type ?? rootPackageJson?.type ?? 'commonjs'; // CJS is default if missing
+        // If type is "module", warn user - this is incompatible with @nx/jest/plugin
+        // Should not be possible, but it's possible that there's a way to get this working that we're unaware of
+        if (effectiveType === 'module') {
+            projectsWithTypeModule.push(configPath);
+            continue;
+        }
+        let content = tree.read(configPath, 'utf-8');
+        // Check for ESM-only features that can't be converted
+        const hasImportMeta = tsquery.query(content, 'MetaProperty').length > 0 ||
+            content.includes('import.meta');
+        const hasTopLevelAwait = checkForTopLevelAwait(content, tsquery);
+        if (hasImportMeta || hasTopLevelAwait) {
+            projectsWithEsmOnlyFeatures.push(configPath);
+            continue;
+        }
+        content = convertImportsToRequire(content, tsquery);
+        content = convertExportDefaultToModuleExports(content, tsquery);
+        tree.write(configPath, content);
+        modifiedFiles.push(configPath);
+    }
+    if (modifiedFiles.length > 0) {
+        await (0, devkit_1.formatFiles)(tree);
+    }
+    const hasWarnings = projectsWithEsmOnlyFeatures.length > 0 || projectsWithTypeModule.length > 0;
+    if (hasWarnings) {
+        return () => {
+            if (projectsWithTypeModule.length > 0) {
+                devkit_1.logger.warn(`The following projects have "type": "module" in their package.json which is incompatible ` +
+                    `with @nx/jest/plugin. Consider removing "type": "module" ` +
+                    `or using a different Jest configuration approach:\n` +
+                    projectsWithTypeModule.map((p) => `  - ${p}`).join('\n'));
+            }
+            if (projectsWithEsmOnlyFeatures.length > 0) {
+                devkit_1.logger.warn(`The following jest.config.ts files use ESM-only features (import.meta or top-level await) ` +
+                    `and could not be automatically converted to CommonJS. Please update them manually:\n` +
+                    projectsWithEsmOnlyFeatures.map((p) => `  - ${p}`).join('\n'));
+            }
+        };
+    }
+}
+function checkForTopLevelAwait(content, tsquery) {
+    const ts = require('typescript');
+    // Check for await expressions that are not inside a function
+    const ast = tsquery.ast(content);
+    const awaitExpressions = tsquery.query(ast, 'AwaitExpression');
+    for (const awaitExpr of awaitExpressions) {
+        let parent = awaitExpr.parent;
+        let isInsideFunction = false;
+        while (parent) {
+            if (ts.isFunctionLike(parent)) {
+                isInsideFunction = true;
+                break;
+            }
+            parent = parent.parent;
+        }
+        if (!isInsideFunction) {
+            return true;
+        }
+    }
+    return false;
+}
+function convertImportsToRequire(content, tsquery) {
+    const ts = require('typescript');
+    const ast = tsquery.ast(content);
+    const importDeclarations = tsquery.query(ast, 'ImportDeclaration');
+    if (importDeclarations.length === 0) {
+        return content;
+    }
+    // Sort imports by position (descending) to replace from end to start
+    // This preserves positions of earlier nodes
+    const sortedImports = [...importDeclarations].sort((a, b) => b.getStart() - a.getStart());
+    for (const importDecl of sortedImports) {
+        const moduleSpecifier = importDecl.moduleSpecifier
+            .getText()
+            .replace(/['"]/g, '');
+        const importClause = importDecl.importClause;
+        if (!importClause) {
+            // Side-effect import: import 'module'
+            const requireStatement = `require('${moduleSpecifier}')`;
+            content = replaceNode(content, importDecl, requireStatement);
+            continue;
+        }
+        const parts = [];
+        // Default import: import x from 'module'
+        if (importClause.name) {
+            const defaultName = importClause.name.getText();
+            parts.push(`const ${defaultName} = require('${moduleSpecifier}').default ?? require('${moduleSpecifier}')`);
+        }
+        // Named imports: import { a, b } from 'module'
+        if (importClause.namedBindings) {
+            if (ts.isNamedImports(importClause.namedBindings)) {
+                const namedImports = importClause.namedBindings.elements
+                    .map((element) => {
+                    const name = element.name.getText();
+                    const propertyName = element.propertyName?.getText();
+                    if (propertyName) {
+                        return `${propertyName}: ${name}`;
+                    }
+                    return name;
+                })
+                    .join(', ');
+                parts.push(`const { ${namedImports} } = require('${moduleSpecifier}')`);
+            }
+            else if (ts.isNamespaceImport(importClause.namedBindings)) {
+                // Namespace import: import * as x from 'module'
+                const namespaceName = importClause.namedBindings.name.getText();
+                parts.push(`const ${namespaceName} = require('${moduleSpecifier}')`);
+            }
+        }
+        const requireStatement = parts.join(';\n');
+        content = replaceNode(content, importDecl, requireStatement);
+    }
+    return content;
+}
+function convertExportDefaultToModuleExports(content, tsquery) {
+    // Handle: export default { ... }
+    const exportAssignments = tsquery.query(content, 'ExportAssignment');
+    if (exportAssignments.length > 0) {
+        for (const exportAssignment of exportAssignments) {
+            const expression = exportAssignment.expression;
+            if (expression) {
+                const exportedValue = expression.getText();
+                const replacement = `module.exports = ${exportedValue}`;
+                content = replaceNode(content, exportAssignment, replacement);
+            }
+        }
+    }
+    return content;
+}
+function replaceNode(content, node, replacement) {
+    const start = node.getStart();
+    const end = node.getEnd();
+    // Remove trailing semicolon if present to avoid double semicolons
+    let endPos = end;
+    if (content[end] === ';') {
+        endPos = end + 1;
+    }
+    return content.slice(0, start) + replacement + ';' + content.slice(endPos);
+}
+function isJestPluginRegistered(tree) {
+    if (!tree.exists('nx.json')) {
+        return false;
+    }
+    const nxJson = (0, devkit_1.readJson)(tree, 'nx.json');
+    const plugins = nxJson.plugins ?? [];
+    return plugins.some((plugin) => {
+        const pluginName = typeof plugin === 'string' ? plugin : plugin.plugin;
+        return pluginName === '@nx/jest/plugin' || pluginName === '@nx/jest';
+    });
+}

package/src/plugins/plugin.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"plugin.d.ts","sourceRoot":"","sources":["../../../../../packages/jest/src/plugins/plugin.ts"],"names":[],"mappings":"AAAA,OAAO,EAGL,aAAa,EASd,MAAM,YAAY,CAAC;AA6BpB,MAAM,WAAW,iBAAiB;IAChC,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,YAAY,CAAC,EAAE,MAAM,CAAC;IAEtB;;OAEG;IACH,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB;;;;OAIG;IACH,kBAAkB,CAAC,EAAE,OAAO,CAAC;CAC9B;AAmBD,eAAO,MAAM,WAAW,EAAE,aAAa,CAAC,iBAAiB,CAiFxD,CAAC;AAEF,eAAO,MAAM,aAAa,kCAAc,CAAC"}
+{"version":3,"file":"plugin.d.ts","sourceRoot":"","sources":["../../../../../packages/jest/src/plugins/plugin.ts"],"names":[],"mappings":"AAAA,OAAO,EAGL,aAAa,EASd,MAAM,YAAY,CAAC;AA4BpB,MAAM,WAAW,iBAAiB;IAChC,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,YAAY,CAAC,EAAE,MAAM,CAAC;IAEtB;;OAEG;IACH,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB;;;;OAIG;IACH,kBAAkB,CAAC,EAAE,OAAO,CAAC;CAC9B;AAmBD,eAAO,MAAM,WAAW,EAAE,aAAa,CAAC,iBAAiB,CAiFxD,CAAC;AAEF,eAAO,MAAM,aAAa,kCAAc,CAAC"}

package/src/plugins/plugin.js

@@ -15,7 +15,6 @@ const globs_1 = require("nx/src/utils/globs");
 const installation_directory_1 = require("nx/src/utils/installation-directory");
 const plugins_1 = require("nx/src/utils/plugins");
 const workspace_context_1 = require("nx/src/utils/workspace-context");
-const semver_1 = require("semver");
 const versions_1 = require("../utils/versions");
 const pmc = (0, devkit_1.getPackageManagerCommand)();
 function readTargetsCache(cachePath) {
@@ -122,19 +121,9 @@ async function buildJestTargets(configFilePath, projectRoot, options, context, p
         module: 'commonjs',
         customConditions: null,
     });
-    // Jest 30 + Node.js 24 can't parse TS configs with imports.
-    // This flag does not exist in Node 20/22.
-    // https://github.com/jestjs/jest/issues/15682
-    const nodeVersion = (0, semver_1.major)(process.version);
     const env = {
         TS_NODE_COMPILER_OPTIONS: tsNodeCompilerOptions,
     };
-    if (nodeVersion >= 24) {
-        const currentOptions = process.env.NODE_OPTIONS || '';
-        if (!currentOptions.includes('--no-experimental-strip-types')) {
-            env.NODE_OPTIONS = (currentOptions + ' --no-experimental-strip-types').trim();
-        }
-    }
     const target = (targets[options.targetName] = {
         command: 'jest',
         options: {