@nx/vitest 23.2.0-beta.5 → 23.2.0-beta.7

package/dist/internal.d.ts

@@ -0,0 +1,6 @@
+/**
+ * The Oxlint plugins a Vitest-tested project needs. Declared here so the linter
+ * does not have to know what a test runner requires; `@nx/js`'s
+ * `addLintingToProject` reads it through `ensurePackage`.
+ */
+export declare const oxlintPlugins: string[];

package/dist/internal.js

@@ -0,0 +1,14 @@
+"use strict";
+// Semi-private surface for first-party Nx packages.
+//
+// External plugins should NOT import from here — this entry is curated for
+// internal consumers and may change without semver protection. Mirrors
+// `@nx/devkit/internal`.
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.oxlintPlugins = void 0;
+/**
+ * The Oxlint plugins a Vitest-tested project needs. Declared here so the linter
+ * does not have to know what a test runner requires; `@nx/js`'s
+ * `addLintingToProject` reads it through `ensurePackage`.
+ */
+exports.oxlintPlugins = ['vitest'];

package/dist/src/generators/configuration/configuration.js

@@ -17,6 +17,17 @@ const versions_1 = require("../../utils/versions");
 const assert_supported_vitest_version_1 = require("../../utils/assert-supported-vitest-version");
 const semver_1 = require("semver");
 let ts;
+// Must match `vitestWorkspaceFiles` in ../../plugins/plugin.ts, or the guards
+// here and inference disagree about what counts as an existing workspace file.
+const WORKSPACE_FILE_EXTENSIONS = [
+    'ts',
+    'mts',
+    'cts',
+    'js',
+    'mjs',
+    'cjs',
+    'json',
+];
 /**
  * Determines whether to use vitest.config.mts instead of vite.config.mts.
  * Returns true for new non-framework projects that don't already have a vite.config.
@@ -188,7 +199,7 @@ getTestBed().initTestEnvironment(
         const projectGlobs = `'**/vite.config.{mjs,js,ts,mts}', '**/vitest.config.{mjs,js,ts,mts}'`;
         const vitestMajorVersion = (0, versions_1.getInstalledVitestMajorVersion)(tree);
         if (vitestMajorVersion === null || vitestMajorVersion >= 4) {
-            const hasWorkspaceFile = ['ts', 'js', 'json'].some((ext) => tree.exists(`vitest.workspace.${ext}`) ||
+            const hasWorkspaceFile = WORKSPACE_FILE_EXTENSIONS.some((ext) => tree.exists(`vitest.workspace.${ext}`) ||
                 tree.exists(`vitest.projects.${ext}`));
             const rootVitestConfig = findRootConfig(tree, 'vitest.config');
             const rootViteConfig = findRootConfig(tree, 'vite.config');
@@ -241,7 +252,9 @@ getTestBed().initTestEnvironment(
                 // root vite.config added later; exclude both so neither is resolved as
                 // an extra project that, carrying no `include`, re-runs every spec via
                 // the default glob.
-                tree.write('vitest.config.ts', `import { defineConfig } from 'vitest/config';
+                // `.mts` keeps it ESM whatever the root package.json `type` is; a
+                // CommonJS-loaded config trips Vite's `configLoader: 'native'` warning.
+                tree.write('vitest.config.mts', `import { defineConfig } from 'vitest/config';
 
 export default defineConfig({
   test: {
@@ -251,13 +264,9 @@ export default defineConfig({
 `);
             }
         }
-        else if (!tree.exists(`vitest.workspace.ts`) &&
-            !tree.exists(`vitest.workspace.js`) &&
-            !tree.exists(`vitest.workspace.json`) &&
-            !tree.exists(`vitest.projects.ts`) &&
-            !tree.exists(`vitest.projects.js`) &&
-            !tree.exists(`vitest.projects.json`)) {
-            tree.write('vitest.workspace.ts', `export default [${projectGlobs}];`);
+        else if (!WORKSPACE_FILE_EXTENSIONS.some((ext) => tree.exists(`vitest.workspace.${ext}`) ||
+            tree.exists(`vitest.projects.${ext}`))) {
+            tree.write('vitest.workspace.mts', `export default [${projectGlobs}];`);
         }
     }
     if (!schema.skipFormat) {

package/dist/src/migrations/update-23-2-0/use-import-meta-dirname.d.ts

@@ -0,0 +1,3 @@
+import { type Tree } from '@nx/devkit';
+export default function useImportMetaDirname(tree: Tree): Promise<void>;
+export declare function rewriteDirname(source: string): string;

package/dist/src/migrations/update-23-2-0/use-import-meta-dirname.js

@@ -0,0 +1,108 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.default = useImportMetaDirname;
+exports.rewriteDirname = rewriteDirname;
+const devkit_1 = require("@nx/devkit");
+// Only ESM-only extensions. A `.ts`/`.js` config can still be loaded as CJS,
+// where `import.meta` is a syntax error.
+const CONFIG_FILE_PATTERN = /(^|\/)(vite|vitest)\.config\.(mts|mjs)$/;
+let ts;
+async function useImportMetaDirname(tree) {
+    let touchedCount = 0;
+    (0, devkit_1.visitNotIgnoredFiles)(tree, '.', (filePath) => {
+        if (!CONFIG_FILE_PATTERN.test(filePath)) {
+            return;
+        }
+        const original = tree.read(filePath, 'utf-8');
+        if (!original?.includes('__dirname')) {
+            return;
+        }
+        const updated = rewriteDirname(original);
+        if (updated !== original) {
+            tree.write(filePath, updated);
+            touchedCount += 1;
+        }
+    });
+    if (touchedCount > 0) {
+        devkit_1.logger.info(`Replaced \`__dirname\` with \`import.meta.dirname\` in ${touchedCount} Vite config file(s).`);
+    }
+    await (0, devkit_1.formatFiles)(tree);
+}
+function rewriteDirname(source) {
+    ts ??= (0, devkit_1.ensurePackage)('typescript', '*');
+    const sourceFile = ts.createSourceFile('tmp.mts', source, ts.ScriptTarget.Latest, 
+    /* setParentNodes */ true, ts.ScriptKind.TS);
+    // A config TypeScript can only error-recover on may not mean what its AST
+    // says, so leave it for a human rather than rewrite it blind.
+    if (sourceFile.parseDiagnostics?.length) {
+        return source;
+    }
+    const references = [];
+    let bailOut = false;
+    const visit = (node) => {
+        if (ts.isIdentifier(node) && node.text === '__dirname') {
+            const kind = classify(node);
+            if (kind === 'bail') {
+                bailOut = true;
+            }
+            else if (kind === 'reference') {
+                references.push(node);
+            }
+        }
+        ts.forEachChild(node, visit);
+    };
+    visit(sourceFile);
+    if (bailOut || references.length === 0) {
+        return source;
+    }
+    let updated = source;
+    for (const node of references.reverse()) {
+        const start = node.getStart(sourceFile);
+        updated =
+            updated.slice(0, start) +
+                'import.meta.dirname' +
+                updated.slice(node.getEnd());
+    }
+    return updated;
+}
+/**
+ * `bail` abandons the whole file: something binds `__dirname` in scope, so the
+ * identifier no longer means the CJS global, or it is a shorthand property
+ * where key and value are one token. `skip` leaves a single identifier alone.
+ *
+ * The name-position test is inverted on purpose - anything sitting in a
+ * parent's `name` or `propertyName` slot is excluded by default, so node kinds
+ * nobody enumerated (class fields, accessors, enum members) cannot be rewritten
+ * into invalid syntax.
+ */
+function classify(node) {
+    const parent = node.parent;
+    if (!parent) {
+        return 'reference';
+    }
+    if (ts.isShorthandPropertyAssignment(parent)) {
+        return 'bail';
+    }
+    if (bindsDirname(parent, node)) {
+        return 'bail';
+    }
+    if (parent.name === node || parent.propertyName === node) {
+        return 'skip';
+    }
+    return 'reference';
+}
+/** Declarations that put a new `__dirname` in scope, shadowing the global. */
+function bindsDirname(parent, node) {
+    if (parent.name !== node) {
+        return false;
+    }
+    return (ts.isVariableDeclaration(parent) ||
+        ts.isParameter(parent) ||
+        ts.isBindingElement(parent) ||
+        ts.isFunctionDeclaration(parent) ||
+        ts.isClassDeclaration(parent) ||
+        ts.isImportClause(parent) ||
+        ts.isImportSpecifier(parent) ||
+        ts.isNamespaceImport(parent) ||
+        ts.isExportSpecifier(parent));
+}

package/dist/src/migrations/update-23-2-0/use-import-meta-dirname.md

@@ -0,0 +1,36 @@
+#### Replace `__dirname` with `import.meta.dirname` in Vite config files
+
+Vite 8 warns when a config uses features that its `configLoader: 'native'` mode cannot support, and `__dirname` is one of them:
+
+```
+(!) Your Vite config uses features that are unsupported by `configLoader: 'native'`, which is planned to become the default in a future major version of Vite:
+  - `__dirname` (packages/utils/vitest.config.mts:4:9). Use `import.meta.dirname` instead
+```
+
+This migration rewrites `__dirname` to `import.meta.dirname` in every `vite.config.mts`, `vite.config.mjs`, `vitest.config.mts`, and `vitest.config.mjs` file in your workspace.
+
+#### Sample Code Changes
+
+##### Before
+
+```ts
+export default defineConfig(() => ({
+  root: __dirname,
+}));
+```
+
+##### After
+
+```ts
+export default defineConfig(() => ({
+  root: import.meta.dirname,
+}));
+```
+
+#### What is not rewritten
+
+`.ts` and `.js` configs are left alone, since those extensions can still be loaded as CommonJS, where `import.meta` is a syntax error. Configs that declare their own `__dirname` (usually `const __dirname = path.dirname(fileURLToPath(import.meta.url))`) are also left alone - that idiom already works under the native config loader.
+
+No migration renames an existing config to `.mts`, because other tooling may reference it by path. So a workspace whose config is `.ts` keeps the companion warning about ESM syntax in a CommonJS-loaded file. Rename it yourself, or set `"type": "module"` in the closest `package.json`, to clear that one. Newly generated configs use `.mts` and are unaffected.
+
+One generated case does put `import.meta.dirname` in a `.ts` config: `@nx/nuxt` falls back to `.ts` on workspaces still using eslintrc, because the legacy `@nuxt/eslint-config` cannot parse `.mts`. Vite's default config loader bundles that file, so it works today, but the file is not loadable under `configLoader: 'native'` - its own `import` statements already make it so.

package/dist/src/utils/generator-utils.js

@@ -9,14 +9,25 @@ const deprecation_1 = require("./deprecation");
 const versions_1 = require("./versions");
 function addOrChangeTestTarget(tree, options, hasPlugin) {
     const nxJson = (0, devkit_1.readNxJson)(tree);
-    hasPlugin = nxJson.plugins?.some((p) => typeof p === 'string'
-        ? p === '@nx/vitest'
-        : p.plugin === '@nx/vitest' || hasPlugin);
+    const target = options.testTarget ?? 'test';
+    // The plugin only infers the target names it is registered for, so a request
+    // for any other name still needs an explicit target.
+    hasPlugin ||=
+        nxJson.plugins?.some((p) => {
+            if (typeof p === 'string') {
+                return p === '@nx/vitest' && target === 'test';
+            }
+            if (p.plugin !== '@nx/vitest') {
+                return false;
+            }
+            const pluginOptions = p.options;
+            return ((pluginOptions?.testTargetName ?? 'test') === target ||
+                pluginOptions?.ciTargetName === target);
+        }) ?? false;
     if (hasPlugin) {
         return;
     }
     const project = (0, devkit_1.readProjectConfiguration)(tree, options.project);
-    const target = options.testTarget ?? 'test';
     const reportsDirectory = (0, devkit_1.joinPathFragments)('coverage', project.root === '.' ? options.project : project.root);
     const testOptions = {
         reportsDirectory,
@@ -94,7 +105,7 @@ function createOrEditViteConfig(tree, options, onlyVitest, extraOptions = {}) {
         }
     }
     if (!onlyVitest && options.includeLib) {
-        plugins.push(`dts({ entryRoot: 'src', tsconfigPath: path.join(__dirname, 'tsconfig.lib.json')${!isTsSolutionSetup ? ', pathsToAliases: false' : ''} })`);
+        plugins.push(`dts({ entryRoot: 'src', tsconfigPath: path.join(import.meta.dirname, 'tsconfig.lib.json')${!isTsSolutionSetup ? ', pathsToAliases: false' : ''} })`);
     }
     const reportsDirectory = isTsSolutionSetup
         ? './test-output/vitest/coverage'
@@ -164,7 +175,7 @@ ${options.inSourceTests
 ${imports.join(';\n')}${imports.length ? ';' : ''}
 
 export default defineConfig(() => ({
-  root: __dirname,
+  root: import.meta.dirname,
   ${printOptions(cacheDir, plugins.length ? `  plugins: [${plugins.join(', ')}],` : '', defineOption, testOption)}
 }));
 `.replace(/\s+(?=(\n|$))/gm, '\n')
@@ -173,7 +184,7 @@ import { defineConfig } from 'vite';
 ${imports.join(';\n')}${imports.length ? ';' : ''}
 
 export default defineConfig(() => ({
-  root: __dirname,
+  root: import.meta.dirname,
   ${printOptions(cacheDir, devServerOption, previewServerOption, `  plugins: [${plugins.join(', ')}],`, workerOption, buildOption, defineOption, testOption)}
 }));
 `.replace(/\s+(?=(\n|$))/gm, '\n');

package/dist/src/utils/ignore-vitest-temp-files.d.ts

@@ -1,4 +1,3 @@
 import { type Tree } from '@nx/devkit';
 export declare function ignoreVitestTempFiles(tree: Tree, projectRoot?: string | undefined): Promise<void>;
 export declare function addVitestTempFilesToGitIgnore(tree: Tree): void;
-export declare function isEslintInstalled(tree: Tree): boolean;

package/dist/src/utils/ignore-vitest-temp-files.js

@@ -2,8 +2,8 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ignoreVitestTempFiles = ignoreVitestTempFiles;
 exports.addVitestTempFilesToGitIgnore = addVitestTempFilesToGitIgnore;
-exports.isEslintInstalled = isEslintInstalled;
 const devkit_1 = require("@nx/devkit");
+const internal_1 = require("@nx/js/internal");
 const versions_1 = require("./versions");
 async function ignoreVitestTempFiles(tree, projectRoot) {
     addVitestTempFilesToGitIgnore(tree);
@@ -20,7 +20,9 @@ function addVitestTempFilesToGitIgnore(tree) {
     tree.write('.gitignore', gitIgnoreContents);
 }
 async function ignoreVitestTempFilesInEslintConfig(tree, projectRoot) {
-    if (!isEslintInstalled(tree)) {
+    // Checked before `ensurePackage` so an Oxlint workspace does not install
+    // `@nx/eslint` only for `isEslintConfigSupported` to send it straight back.
+    if (!(0, internal_1.detectLinters)(tree).includes('eslint')) {
         return;
     }
     (0, devkit_1.ensurePackage)('@nx/eslint', versions_1.nxVersion);
@@ -45,15 +47,3 @@ async function ignoreVitestTempFilesInEslintConfig(tree, projectRoot) {
     const directory = isUsingFlatConfig ? '' : (projectRoot ?? '');
     addIgnoresToLintConfig(tree, directory, ['**/vitest.config.*.timestamp*']);
 }
-function isEslintInstalled(tree) {
-    try {
-        require('eslint');
-        return true;
-    }
-    catch { }
-    // it might not be installed yet, but it might be in the tree pending install
-    const { devDependencies, dependencies } = tree.exists('package.json')
-        ? (0, devkit_1.readJson)(tree, 'package.json')
-        : {};
-    return !!devDependencies?.['eslint'] || !!dependencies?.['eslint'];
-}

package/migrations.json

@@ -28,6 +28,12 @@
       "description": "Rename imports of `createNodesV2` from `@nx/vitest` to the canonical `createNodes` export.",
       "implementation": "./dist/src/migrations/update-23-0-0/migrate-create-nodes-v2-to-create-nodes",
       "documentation": "./dist/src/migrations/update-23-0-0/migrate-create-nodes-v2-to-create-nodes.md"
+    },
+    "update-23-2-0-use-import-meta-dirname": {
+      "version": "23.2.0-beta.6",
+      "description": "Replace `__dirname` with `import.meta.dirname` in `vite.config.mts`/`vitest.config.mts` files so they work with Vite's `configLoader: 'native'`.",
+      "implementation": "./dist/src/migrations/update-23-2-0/use-import-meta-dirname",
+      "documentation": "./dist/src/migrations/update-23-2-0/use-import-meta-dirname.md"
     }
   },
   "packageJsonUpdates": {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@nx/vitest",
   "description": "The Nx Plugin for Vitest to enable fast unit testing with Vitest.",
-  "version": "23.2.0-beta.5",
+  "version": "23.2.0-beta.7",
   "type": "commonjs",
   "files": [
     "dist",
@@ -37,6 +37,9 @@
       "generators": [
         "dist/generators.d.ts"
       ],
+      "internal": [
+        "dist/internal.d.ts"
+      ],
       "executors": [
         "dist/executors.d.ts"
       ]
@@ -57,6 +60,11 @@
       "types": "./dist/generators.d.ts",
       "default": "./dist/generators.js"
     },
+    "./internal": {
+      "@nx/nx-source": "./internal.ts",
+      "types": "./dist/internal.d.ts",
+      "default": "./dist/internal.js"
+    },
     "./executors": {
       "@nx/nx-source": "./executors.ts",
       "types": "./dist/executors.d.ts",
@@ -73,13 +81,13 @@
     "tslib": "^2.3.0",
     "semver": "^7.6.3",
     "@phenomnomnominal/tsquery": "~6.2.0",
-    "@nx/devkit": "23.2.0-beta.5",
-    "@nx/js": "23.2.0-beta.5"
+    "@nx/devkit": "23.2.0-beta.7",
+    "@nx/js": "23.2.0-beta.7"
   },
   "peerDependencies": {
     "vitest": "^3.0.0 || ^4.0.0",
     "vite": "^5.0.0 || ^6.0.0 || ^7.0.0 || ^8.0.0",
-    "@nx/eslint": "23.2.0-beta.5"
+    "@nx/eslint": "23.2.0-beta.7"
   },
   "peerDependenciesMeta": {
     "@nx/eslint": {
@@ -93,6 +101,6 @@
     }
   },
   "devDependencies": {
-    "nx": "23.2.0-beta.5"
+    "nx": "23.2.0-beta.7"
   }
 }