@nx/vite 23.0.0-beta.2 → 23.0.0-beta.20

package/{src → dist/src}/generators/convert-to-inferred/convert-to-inferred.js

@@ -2,21 +2,18 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.convertToInferred = convertToInferred;
 const devkit_1 = require("@nx/devkit");
-const executor_to_plugin_migrator_1 = require("@nx/devkit/src/generators/plugin-migrations/executor-to-plugin-migrator");
+const internal_1 = require("@nx/devkit/internal");
 const plugin_1 = require("../../plugins/plugin");
 const build_post_target_transformer_1 = require("./lib/build-post-target-transformer");
 const serve_post_target_transformer_1 = require("./lib/serve-post-target-transformer");
 const preview_post_target_transformer_1 = require("./lib/preview-post-target-transformer");
-const test_post_target_transformer_1 = require("./lib/test-post-target-transformer");
-const aggregate_log_util_1 = require("@nx/devkit/src/generators/plugin-migrations/aggregate-log-util");
 async function convertToInferred(tree, options) {
     const projectGraph = await (0, devkit_1.createProjectGraphAsync)();
-    const migrationLogs = new aggregate_log_util_1.AggregatedLog();
-    const migratedProjects = await (0, executor_to_plugin_migrator_1.migrateProjectExecutorsToPlugin)(tree, projectGraph, '@nx/vite/plugin', plugin_1.createNodesV2, {
+    const migrationLogs = new internal_1.AggregatedLog();
+    const migratedProjects = await (0, internal_1.migrateProjectExecutorsToPlugin)(tree, projectGraph, '@nx/vite/plugin', plugin_1.createNodesV2, {
         buildTargetName: 'build',
         serveTargetName: 'serve',
         previewTargetName: 'preview',
-        testTargetName: 'test',
         serveStaticTargetName: 'serve-static',
     }, [
         {
@@ -34,14 +31,9 @@ async function convertToInferred(tree, options) {
             postTargetTransformer: (0, preview_post_target_transformer_1.previewPostTargetTransformer)(migrationLogs),
             targetPluginOptionMapper: (target) => ({ previewTargetName: target }),
         },
-        {
-            executors: ['@nx/vite:test'],
-            postTargetTransformer: test_post_target_transformer_1.testPostTargetTransformer,
-            targetPluginOptionMapper: (target) => ({ testTargetName: target }),
-        },
     ], options.project);
     if (migratedProjects.size === 0) {
-        throw new executor_to_plugin_migrator_1.NoTargetsToMigrateError();
+        throw new internal_1.NoTargetsToMigrateError();
     }
     if (!options.skipFormat) {
         await (0, devkit_1.formatFiles)(tree);

package/{src → dist/src}/generators/convert-to-inferred/lib/build-post-target-transformer.d.ts

@@ -4,4 +4,3 @@ export declare function buildPostTargetTransformer(target: TargetConfiguration,
     root: string;
 }, inferredTargetConfiguration: TargetConfiguration): TargetConfiguration<any>;
 export declare function moveBuildLibsFromSourceToViteConfig(tree: Tree, configPath: string): void;
-//# sourceMappingURL=build-post-target-transformer.d.ts.map

package/{src → dist/src}/generators/convert-to-inferred/lib/build-post-target-transformer.js

@@ -2,11 +2,11 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.buildPostTargetTransformer = buildPostTargetTransformer;
 exports.moveBuildLibsFromSourceToViteConfig = moveBuildLibsFromSourceToViteConfig;
+const internal_1 = require("@nx/devkit/internal");
 const devkit_1 = require("@nx/devkit");
 const tsquery_1 = require("@phenomnomnominal/tsquery");
 const posix_1 = require("path/posix");
 const utils_1 = require("./utils");
-const plugin_migration_utils_1 = require("@nx/devkit/src/generators/plugin-migrations/plugin-migration-utils");
 function buildPostTargetTransformer(target, tree, projectDetails, inferredTargetConfiguration) {
     let viteConfigPath = (0, utils_1.getViteConfigPath)(tree, projectDetails.root);
     const configValues = {
@@ -53,7 +53,7 @@ function buildPostTargetTransformer(target, tree, projectDetails, inferredTarget
         removePropertiesFromTargetOptions(tree, target.options, viteConfigPath, projectDetails.root, configValues['default'], true);
     }
     if (target.outputs) {
-        (0, plugin_migration_utils_1.processTargetOutputs)(target, [{ newName: 'outDir', oldName: 'outputPath' }], inferredTargetConfiguration, {
+        (0, internal_1.processTargetOutputs)(target, [{ newName: 'outDir', oldName: 'outputPath' }], inferredTargetConfiguration, {
             projectName: projectDetails.projectName,
             projectRoot: projectDetails.root,
         });

package/{src → dist/src}/generators/convert-to-inferred/lib/preview-post-target-transformer.d.ts

@@ -1,7 +1,6 @@
 import { type TargetConfiguration, type Tree } from '@nx/devkit';
-import { AggregatedLog } from '@nx/devkit/src/generators/plugin-migrations/aggregate-log-util';
+import { AggregatedLog } from '@nx/devkit/internal';
 export declare function previewPostTargetTransformer(migrationLogs: AggregatedLog): (target: TargetConfiguration, tree: Tree, projectDetails: {
     projectName: string;
     root: string;
 }, inferredTargetConfiguration: TargetConfiguration) => TargetConfiguration<any>;
-//# sourceMappingURL=preview-post-target-transformer.d.ts.map

package/{src → dist/src}/generators/convert-to-inferred/lib/serve-post-target-transformer.d.ts

@@ -1,7 +1,6 @@
 import { type TargetConfiguration, type Tree } from '@nx/devkit';
-import { AggregatedLog } from '@nx/devkit/src/generators/plugin-migrations/aggregate-log-util';
+import { AggregatedLog } from '@nx/devkit/internal';
 export declare function servePostTargetTransformer(migrationLogs: AggregatedLog): (target: TargetConfiguration, tree: Tree, projectDetails: {
     projectName: string;
     root: string;
 }, inferredTargetConfiguration: TargetConfiguration) => TargetConfiguration<any>;
-//# sourceMappingURL=serve-post-target-transformer.d.ts.map

package/{src → dist/src}/generators/convert-to-inferred/lib/utils.d.ts

@@ -2,4 +2,3 @@ import { type Tree } from '@nx/devkit';
 export declare function toProjectRelativePath(path: string, projectRoot: string): string;
 export declare function getViteConfigPath(tree: Tree, root: string): string;
 export declare function addConfigValuesToViteConfig(tree: Tree, configFile: string, configValues: Record<string, Record<string, unknown>>): void;
-//# sourceMappingURL=utils.d.ts.map

package/{src → dist/src}/generators/init/init.d.ts

@@ -4,4 +4,3 @@ export declare function updateNxJsonSettings(tree: Tree): void;
 export declare function initGenerator(tree: Tree, schema: InitGeneratorSchema): Promise<GeneratorCallback>;
 export declare function initGeneratorInternal(tree: Tree, schema: InitGeneratorSchema): Promise<GeneratorCallback>;
 export default initGenerator;
-//# sourceMappingURL=init.d.ts.map

package/{src → dist/src}/generators/init/init.js

@@ -3,8 +3,8 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.updateNxJsonSettings = updateNxJsonSettings;
 exports.initGenerator = initGenerator;
 exports.initGeneratorInternal = initGeneratorInternal;
+const internal_1 = require("@nx/devkit/internal");
 const devkit_1 = require("@nx/devkit");
-const add_plugin_1 = require("@nx/devkit/src/utils/add-plugin");
 const setup_paths_plugin_1 = require("../setup-paths-plugin/setup-paths-plugin");
 const plugin_1 = require("../../plugins/plugin");
 const utils_1 = require("./lib/utils");
@@ -27,9 +27,8 @@ async function initGeneratorInternal(tree, schema) {
         nxJson.useInferencePlugins !== false;
     schema.addPlugin ??= addPluginDefault;
     if (schema.addPlugin) {
-        await (0, add_plugin_1.addPlugin)(tree, await (0, devkit_1.createProjectGraphAsync)(), '@nx/vite/plugin', plugin_1.createNodesV2, {
+        await (0, internal_1.addPlugin)(tree, await (0, devkit_1.createProjectGraphAsync)(), '@nx/vite/plugin', plugin_1.createNodesV2, {
             buildTargetName: ['build', 'vite:build', 'vite-build'],
-            testTargetName: ['test', 'vite:test', 'vite-test'],
             serveTargetName: ['serve', 'vite:serve', 'vite-serve'],
             devTargetName: ['dev', 'vite:dev', 'vite-dev'],
             previewTargetName: ['preview', 'vite:preview', 'vite-preview'],

package/{src → dist/src}/generators/init/lib/utils.d.ts

@@ -2,4 +2,3 @@ import { Tree } from '@nx/devkit';
 import { InitGeneratorSchema } from '../schema';
 export declare function checkDependenciesInstalled(host: Tree, schema: InitGeneratorSchema): Promise<import("@nx/devkit").GeneratorCallback>;
 export declare function moveToDevDependencies(tree: Tree): () => void;
-//# sourceMappingURL=utils.d.ts.map

package/{src → dist/src}/generators/init/lib/utils.js

@@ -3,9 +3,9 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.checkDependenciesInstalled = checkDependenciesInstalled;
 exports.moveToDevDependencies = moveToDevDependencies;
 const devkit_1 = require("@nx/devkit");
-const versions_1 = require("@nx/js/src/utils/versions");
+const internal_1 = require("@nx/js/internal");
 const semver_1 = require("semver");
-const versions_2 = require("../../../utils/versions");
+const versions_1 = require("../../../utils/versions");
 const version_utils_1 = require("../../../utils/version-utils");
 function hasIncompatibleInstalledEsbuild(host) {
     const installedEsbuildVersion = (0, devkit_1.getDependencyVersionFromPackageJson)(host, 'esbuild');
@@ -13,7 +13,7 @@ function hasIncompatibleInstalledEsbuild(host) {
         return false;
     }
     try {
-        return !(0, semver_1.intersects)(installedEsbuildVersion, versions_1.esbuildVersion, {
+        return !(0, semver_1.intersects)(installedEsbuildVersion, internal_1.esbuildVersion, {
             includePrerelease: true,
         });
     }
@@ -22,7 +22,6 @@ function hasIncompatibleInstalledEsbuild(host) {
     }
 }
 async function checkDependenciesInstalled(host, schema) {
-    const { vitest } = await (0, version_utils_1.getVitestDependenciesVersionsToInstall)(host);
     // Determine which vite version to install:
     // 1. Explicit flags take priority (useViteV5/V6/V7)
     // 2. If vite is already installed, keep the matching major version
@@ -36,30 +35,28 @@ async function checkDependenciesInstalled(host, schema) {
             title: 'Installed esbuild is incompatible with Vite 8. Using Vite 7.',
             bodyLines: [
                 `Found esbuild version "${installedEsbuildVersion}" in the workspace root package.json.`,
-                `Update esbuild to a range compatible with ${versions_1.esbuildVersion} if you want newly generated Vite projects to use Vite 8 by default.`,
+                `Update esbuild to a range compatible with ${internal_1.esbuildVersion} if you want newly generated Vite projects to use Vite 8 by default.`,
             ],
         });
     }
     const viteVersionToInstall = schema.useViteV5
-        ? versions_2.viteV5Version
+        ? versions_1.viteV5Version
         : schema.useViteV6
-            ? versions_2.viteV6Version
+            ? versions_1.viteV6Version
             : schema.useViteV7 ||
                 installedMajor === 7 ||
                 useViteV7ForEsbuildCompatibility
-                ? versions_2.viteV7Version
+                ? versions_1.viteV7Version
                 : installedMajor === 6
-                    ? versions_2.viteV6Version
+                    ? versions_1.viteV6Version
                     : installedMajor === 5
-                        ? versions_2.viteV5Version
-                        : versions_2.viteVersion;
+                        ? versions_1.viteV5Version
+                        : versions_1.viteVersion;
     return (0, devkit_1.addDependenciesToPackageJson)(host, {}, {
-        '@nx/vite': versions_2.nxVersion,
-        '@nx/web': versions_2.nxVersion,
+        '@nx/vite': versions_1.nxVersion,
+        '@nx/web': versions_1.nxVersion,
         vite: viteVersionToInstall,
-        vitest: vitest,
-        '@vitest/ui': vitest,
-        jiti: versions_2.jitiVersion,
+        jiti: versions_1.jitiVersion,
     }, undefined, schema.keepExistingVersions);
 }
 function moveToDevDependencies(tree) {

package/dist/src/generators/setup-paths-plugin/lib/utils.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/src/generators/setup-paths-plugin/lib/utils.js

@@ -0,0 +1,2 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });

package/{src → dist/src}/generators/setup-paths-plugin/setup-paths-plugin.d.ts

@@ -3,4 +3,3 @@ export declare function setupPathsPlugin(tree: Tree, schema: {
     skipFormat?: boolean;
 }): Promise<void>;
 export default setupPathsPlugin;
-//# sourceMappingURL=setup-paths-plugin.d.ts.map

package/{src → dist/src}/migrations/update-20-0-4/add-vite-temp-files-to-git-ignore.d.ts

@@ -1,3 +1,2 @@
 import { Tree } from '@nx/devkit';
 export default function addViteTempFilesToGitIgnore(tree: Tree): void;
-//# sourceMappingURL=add-vite-temp-files-to-git-ignore.d.ts.map

package/dist/src/migrations/update-20-0-4/add-vite-temp-files-to-git-ignore.md

@@ -0,0 +1,12 @@
+#### Add Vite Temp Files to Git Ignore
+
+Add gitignore entry for temporary vite config files.
+
+#### Sample Code Changes
+
+Adds the following entries to the `.gitignore` file.
+
+```text title=".gitignore"
+vite.config.*.timestamp*
+vitest.config.*.timestamp*
+```

package/{src → dist/src}/migrations/update-20-5-0/eslint-ignore-vite-temp-files.d.ts

@@ -1,3 +1,2 @@
 import { type Tree } from '@nx/devkit';
 export default function (tree: Tree): Promise<void>;
-//# sourceMappingURL=eslint-ignore-vite-temp-files.d.ts.map

package/dist/src/migrations/update-20-5-0/eslint-ignore-vite-temp-files.js

@@ -0,0 +1,44 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.default = default_1;
+const devkit_1 = require("@nx/devkit");
+const ignore_vite_temp_files_1 = require("../../utils/ignore-vite-temp-files");
+const versions_1 = require("../../utils/versions");
+async function default_1(tree) {
+    if (!(0, ignore_vite_temp_files_1.isEslintInstalled)(tree)) {
+        return;
+    }
+    (0, devkit_1.ensurePackage)('@nx/eslint', versions_1.nxVersion);
+    // CommonJS `require` instead of dynamic ESM `import` — `ensurePackage`
+    // exposes the temp install via `Module._initPaths`, which ESM ignores.
+    const { addIgnoresToLintConfig, isEslintConfigSupported, useFlatConfig, } = require('@nx/eslint/internal');
+    if (!isEslintConfigSupported(tree)) {
+        return;
+    }
+    const isUsingFlatConfig = useFlatConfig(tree);
+    if (isUsingFlatConfig) {
+        // using flat config, so we update the root eslint config
+        addIgnoresToLintConfig(tree, '', [
+            '**/vite.config.*.timestamp*',
+            '**/vitest.config.*.timestamp*',
+        ]);
+    }
+    else {
+        // not using flat config, so we update each project's eslint config
+        const projects = (0, devkit_1.getProjects)(tree);
+        for (const [, { root: projectRoot }] of projects) {
+            const viteConfigFiles = await (0, devkit_1.globAsync)(tree, [
+                `${projectRoot}/**/{vite,vitest}.config.{js,ts,mjs,mts,cjs,cts}`,
+            ]);
+            if (!viteConfigFiles.length) {
+                // the project doesn't use vite or vitest, so we skip it
+                continue;
+            }
+            addIgnoresToLintConfig(tree, projectRoot, [
+                '**/vite.config.*.timestamp*',
+                '**/vitest.config.*.timestamp*',
+            ]);
+        }
+    }
+    await (0, devkit_1.formatFiles)(tree);
+}

package/dist/src/migrations/update-20-5-0/eslint-ignore-vite-temp-files.md

@@ -0,0 +1,45 @@
+#### Sample Code Changes
+
+Add `vite.config.*.timestamp*` and `vitest.config.*.timestamp*` to the root `eslint.config.mjs` file (using **ESLint Flat Config**).
+
+##### Before
+
+```js title="eslint.config.mjs"
+export default [
+  {
+    ignores: ['dist'],
+  },
+];
+```
+
+##### After
+
+```js title="eslint.config.mjs" {3}
+export default [
+  {
+    ignores: ['dist', 'vite.config.*.timestamp*', 'vitest.config.*.timestamp*'],
+  },
+];
+```
+
+Add `vite.config.*.timestamp*` and `vitest.config.*.timestamp*` to the project's `.eslintrc.json` file (using **eslintrc** format config).
+
+##### Before
+
+```json title="apps/app1/eslintrc.json"
+{
+  "ignorePatterns": ["!**/*"]
+}
+```
+
+##### After
+
+```json title="apps/app1/eslintrc.json" {4-5}
+{
+  "ignorePatterns": [
+    "!**/*",
+    "vite.config.*.timestamp*",
+    "vitest.config.*.timestamp*"
+  ]
+}
+```

package/{src → dist/src}/migrations/update-20-5-0/install-jiti.d.ts

@@ -1,3 +1,2 @@
 import { type Tree } from '@nx/devkit';
 export default function installJiti(tree: Tree): Promise<import("@nx/devkit").GeneratorCallback>;
-//# sourceMappingURL=install-jiti.d.ts.map

package/dist/src/migrations/update-20-5-0/install-jiti.md

@@ -0,0 +1,6 @@
+#### Installs the `jiti` package
+
+This migration ensures that the [`jiti`](https://github.com/unjs/jiti) package is installed.  
+This is a requirement for Vite to parse `postcss` configuration files that use TypeScript.
+
+Learn more: [https://vite.dev/guide/migration#postcss-load-config](https://vite.dev/guide/migration#postcss-load-config)

package/{src → dist/src}/migrations/update-20-5-0/update-resolve-conditions.d.ts

@@ -1,3 +1,2 @@
 import { type Tree } from '@nx/devkit';
 export default function (tree: Tree): Promise<void>;
-//# sourceMappingURL=update-resolve-conditions.d.ts.map

package/dist/src/migrations/update-20-5-0/update-resolve-conditions.md

@@ -0,0 +1,42 @@
+#### Update `resolve.conditions` to include defaults
+
+In previous Vite versions, the `resolve.conditions` option had defaults that were added internally (i.e. `['module', 'browser', 'development|production']`).  
+This default was removed in Vite 6, so this migration adds it to your existing configuration to ensure that the behavior remains intact.
+
+Learn more: [https://vite.dev/guide/migration#default-value-for-resolve-conditions](https://vite.dev/guide/migration#default-value-for-resolve-conditions)
+
+:::note[Remix]
+Remix does not currently support Vite 6 and therefore any `vite.config` file for Remix will not be migrated.
+:::
+
+#### Sample Code Changes
+
+##### Before
+
+```typescript title="vite.config.ts"
+import { defineConfig } from 'vite';
+
+export default defineConfig({
+  resolve: {
+    conditions: ['require'],
+  },
+  build: {
+    outDir: 'dist',
+  },
+});
+```
+
+##### After
+
+```typescript title="vite.config.ts" {4-6}
+import { defineConfig } from 'vite';
+
+export default defineConfig({
+  resolve: {
+    conditions: ['require', 'module', 'browser', 'development|production'],
+  },
+  build: {
+    outDir: 'dist',
+  },
+});
+```

package/{src → dist/src}/migrations/update-22-2-0/migrate-vitest-to-vitest-package.d.ts

@@ -9,4 +9,3 @@ import { type GeneratorCallback, type Tree } from '@nx/devkit';
  * 4. Migrates targetDefaults from @nx/vite:test to @nx/vitest:test
  */
 export default function migrateVitestToVitestPackage(tree: Tree): Promise<GeneratorCallback>;
-//# sourceMappingURL=migrate-vitest-to-vitest-package.d.ts.map

package/{src → dist/src}/migrations/update-22-2-0/migrate-vitest-to-vitest-package.js

@@ -1,8 +1,8 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.default = migrateVitestToVitestPackage;
+const internal_1 = require("@nx/devkit/internal");
 const devkit_1 = require("@nx/devkit");
-const executor_options_utils_1 = require("@nx/devkit/src/generators/executor-options-utils");
 const versions_1 = require("../../utils/versions");
 /**
  * Migrates Vitest usage from @nx/vite to @nx/vitest package.
@@ -32,7 +32,7 @@ function installVitestPackageIfNeeded(tree) {
 }
 function migrateExecutorUsages(tree) {
     const projectsToUpdate = new Set();
-    (0, executor_options_utils_1.forEachExecutorOptions)(tree, '@nx/vite:test', (_options, projectName, _targetName, _configuration) => {
+    (0, internal_1.forEachExecutorOptions)(tree, '@nx/vite:test', (_options, projectName, _targetName, _configuration) => {
         projectsToUpdate.add(projectName);
     });
     for (const projectName of projectsToUpdate) {
@@ -119,19 +119,35 @@ function migrateTargetDefaults(tree) {
         return;
     }
     let hasChanges = false;
-    for (const [targetOrExecutor, targetConfig] of Object.entries(nxJson.targetDefaults)) {
-        // Pattern A: Executor-keyed (e.g., "@nx/vite:test": { ... })
-        if (targetOrExecutor === '@nx/vite:test') {
-            // Move config to new executor key
-            nxJson.targetDefaults['@nx/vitest:test'] ??= {};
-            Object.assign(nxJson.targetDefaults['@nx/vitest:test'], targetConfig);
-            delete nxJson.targetDefaults['@nx/vite:test'];
-            hasChanges = true;
+    if (Array.isArray(nxJson.targetDefaults)) {
+        for (const entry of nxJson.targetDefaults) {
+            // Pattern A: an executor entry for @nx/vite:test
+            if (entry.executor === '@nx/vite:test' && entry.target === undefined) {
+                entry.executor = '@nx/vitest:test';
+                hasChanges = true;
+            }
+            // Pattern B: a target entry whose executor field references @nx/vite:test
+            else if (entry.executor === '@nx/vite:test') {
+                entry.executor = '@nx/vitest:test';
+                hasChanges = true;
+            }
         }
-        // Pattern B: Target-name-keyed (e.g., "test": { "executor": "@nx/vite:test", ... })
-        else if (targetConfig.executor === '@nx/vite:test') {
-            targetConfig.executor = '@nx/vitest:test';
-            hasChanges = true;
+    }
+    else {
+        for (const [targetOrExecutor, targetConfig] of Object.entries(nxJson.targetDefaults)) {
+            // Pattern A: Executor-keyed (e.g., "@nx/vite:test": { ... })
+            if (targetOrExecutor === '@nx/vite:test') {
+                // Move config to new executor key
+                nxJson.targetDefaults['@nx/vitest:test'] ??= {};
+                Object.assign(nxJson.targetDefaults['@nx/vitest:test'], targetConfig);
+                delete nxJson.targetDefaults['@nx/vite:test'];
+                hasChanges = true;
+            }
+            // Pattern B: Target-name-keyed (e.g., "test": { "executor": "@nx/vite:test", ... })
+            else if (targetConfig.executor === '@nx/vite:test') {
+                targetConfig.executor = '@nx/vitest:test';
+                hasChanges = true;
+            }
         }
     }
     if (hasChanges) {