@nx/node 0.0.0-pr-11-e92ae29 → 0.0.0-pr-29464-ce29f31

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nx/node",
-  "version": "0.0.0-pr-11-e92ae29",
+  "version": "0.0.0-pr-29464-ce29f31",
   "private": false,
   "description": "The Node Plugin for Nx contains generators to manage Node applications within an Nx workspace.",
   "repository": {
@@ -32,10 +32,10 @@
   },
   "dependencies": {
     "tslib": "^2.3.0",
-    "@nx/devkit": "0.0.0-pr-11-e92ae29",
-    "@nx/jest": "0.0.0-pr-11-e92ae29",
-    "@nx/js": "0.0.0-pr-11-e92ae29",
-    "@nx/eslint": "0.0.0-pr-11-e92ae29"
+    "@nx/devkit": "0.0.0-pr-29464-ce29f31",
+    "@nx/jest": "0.0.0-pr-29464-ce29f31",
+    "@nx/js": "0.0.0-pr-29464-ce29f31",
+    "@nx/eslint": "0.0.0-pr-29464-ce29f31"
   },
   "publishConfig": {
     "access": "public"

package/src/generators/application/application.d.ts

@@ -1,9 +1,10 @@
 import { GeneratorCallback, Tree } from '@nx/devkit';
 import { Schema } from './schema';
-export interface NormalizedSchema extends Schema {
+export interface NormalizedSchema extends Omit<Schema, 'useTsSolution'> {
     appProjectRoot: string;
     parsedTags: string[];
     outputPath: string;
+    importPath: string;
     isUsingTsSolutionConfig: boolean;
 }
 export declare function addLintingToApplication(tree: Tree, options: NormalizedSchema): Promise<GeneratorCallback>;

package/src/generators/application/application.js

@@ -3,7 +3,6 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.addLintingToApplication = addLintingToApplication;
 exports.applicationGenerator = applicationGenerator;
 exports.applicationGeneratorInternal = applicationGeneratorInternal;
-const get_import_path_1 = require("@nx/js/src/utils/get-import-path");
 const devkit_1 = require("@nx/devkit");
 const project_name_and_root_utils_1 = require("@nx/devkit/src/generators/project-name-and-root-utils");
 const jest_1 = require("@nx/jest");
@@ -79,7 +78,6 @@ function getEsBuildConfig(project, options) {
 }
 function getServeConfig(options) {
     return {
-        continuous: true,
         executor: '@nx/js:node',
         defaultConfiguration: 'development',
         // Run build, which includes dependency on "^build" by default, so the first run
@@ -141,13 +139,11 @@ function addProject(tree, options) {
     project.targets.serve = getServeConfig(options);
     if (options.isUsingTsSolutionConfig) {
         (0, devkit_1.writeJson)(tree, (0, devkit_1.joinPathFragments)(options.appProjectRoot, 'package.json'), {
-            name: (0, get_import_path_1.getImportPath)(tree, options.name),
+            name: options.importPath,
             version: '0.0.1',
             private: true,
             nx: {
-                name: options.name,
-                projectType: 'application',
-                sourceRoot: project.sourceRoot,
+                name: options.name !== options.importPath ? options.name : undefined,
                 targets: project.targets,
                 tags: project.tags?.length ? project.tags : undefined,
             },
@@ -195,16 +191,10 @@ function addAppFiles(tree, options) {
 }
 function addProxy(tree, options) {
     const projectConfig = (0, devkit_1.readProjectConfiguration)(tree, options.frontendProject);
-    if (projectConfig.targets &&
-        ['serve', 'dev'].find((t) => !!projectConfig.targets[t])) {
-        const targetName = ['serve', 'dev'].find((t) => !!projectConfig.targets[t]);
-        projectConfig.targets[targetName].dependsOn = [
-            ...(projectConfig.targets[targetName].dependsOn ?? []),
-            `${options.name}:serve`,
-        ];
+    if (projectConfig.targets && projectConfig.targets.serve) {
         const pathToProxyFile = `${projectConfig.root}/proxy.conf.json`;
-        projectConfig.targets[targetName].options = {
-            ...projectConfig.targets[targetName].options,
+        projectConfig.targets.serve.options = {
+            ...projectConfig.targets.serve.options,
             proxyConfig: pathToProxyFile,
         };
         if (!tree.exists(pathToProxyFile)) {
@@ -381,6 +371,11 @@ async function applicationGeneratorInternal(tree, schema) {
     }
     addAppFiles(tree, options);
     addProject(tree, options);
+    // If we are using the new TS solution
+    // We need to update the workspace file (package.json or pnpm-workspaces.yaml) to include the new project
+    if (options.isUsingTsSolutionConfig) {
+        (0, ts_solution_setup_1.addProjectToTsSolutionWorkspace)(tree, options.appProjectRoot);
+    }
     updateTsConfigOptions(tree, options);
     if (options.linter === eslint_1.Linter.EsLint) {
         const lintTask = await addLintingToApplication(tree, options);
@@ -445,11 +440,6 @@ async function applicationGeneratorInternal(tree, schema) {
             ? ['eslint.config.js', 'eslint.config.cjs', 'eslint.config.mjs']
             : undefined);
     }
-    // If we are using the new TS solution
-    // We need to update the workspace file (package.json or pnpm-workspaces.yaml) to include the new project
-    if (options.isUsingTsSolutionConfig) {
-        (0, ts_solution_setup_1.addProjectToTsSolutionWorkspace)(tree, options.appProjectRoot);
-    }
     (0, sort_fields_1.sortPackageJsonFields)(tree, options.appProjectRoot);
     if (!options.skipFormat) {
         await (0, devkit_1.formatFiles)(tree);
@@ -460,8 +450,8 @@ async function applicationGeneratorInternal(tree, schema) {
     return (0, devkit_1.runTasksInSerial)(...tasks);
 }
 async function normalizeOptions(host, options) {
-    await (0, project_name_and_root_utils_1.ensureProjectName)(host, options, 'application');
-    const { projectName: appProjectName, projectRoot: appProjectRoot } = await (0, project_name_and_root_utils_1.determineProjectNameAndRootOptions)(host, {
+    await (0, project_name_and_root_utils_1.ensureRootProjectName)(options, 'application');
+    const { projectName, projectRoot: appProjectRoot, importPath, } = await (0, project_name_and_root_utils_1.determineProjectNameAndRootOptions)(host, {
         name: options.name,
         projectType: 'application',
         directory: options.directory,
@@ -478,6 +468,7 @@ async function normalizeOptions(host, options) {
         nxJson.useInferencePlugins !== false;
     const isUsingTsSolutionConfig = (0, ts_solution_setup_1.isUsingTsSolutionSetup)(host);
     const swcJest = options.swcJest ?? isUsingTsSolutionConfig;
+    const appProjectName = !isUsingTsSolutionConfig || options.name ? projectName : importPath;
     return {
         addPlugin,
         ...options,
@@ -486,6 +477,7 @@ async function normalizeOptions(host, options) {
             ? (0, devkit_1.names)(options.frontendProject).fileName
             : undefined,
         appProjectRoot,
+        importPath,
         parsedTags,
         linter: options.linter ?? eslint_1.Linter.EsLint,
         unitTestRunner: options.unitTestRunner ?? 'jest',
@@ -493,7 +485,7 @@ async function normalizeOptions(host, options) {
         port: options.port ?? 3000,
         outputPath: isUsingTsSolutionConfig
             ? (0, devkit_1.joinPathFragments)(appProjectRoot, 'dist')
-            : (0, devkit_1.joinPathFragments)('dist', options.rootProject ? options.name : appProjectRoot),
+            : (0, devkit_1.joinPathFragments)('dist', options.rootProject ? appProjectName : appProjectRoot),
         isUsingTsSolutionConfig,
         swcJest,
     };

package/src/generators/e2e-project/e2e-project.js

@@ -12,7 +12,6 @@ const eslint_file_1 = require("@nx/eslint/src/generators/utils/eslint-file");
 const log_show_project_command_1 = require("@nx/devkit/src/utils/log-show-project-command");
 const config_file_1 = require("@nx/jest/src/utils/config/config-file");
 const ts_solution_setup_1 = require("@nx/js/src/utils/typescript/ts-solution-setup");
-const get_import_path_1 = require("@nx/js/src/utils/get-import-path");
 const posix_1 = require("node:path/posix");
 const add_swc_config_1 = require("@nx/js/src/utils/swc/add-swc-config");
 async function e2eProjectGenerator(host, options) {
@@ -29,17 +28,18 @@ async function e2eProjectGeneratorInternal(host, _options) {
     // TODO(@ndcunningham): This is broken.. the outputs are wrong.. and this isn't using the jest generator
     if (isUsingTsSolutionConfig) {
         (0, devkit_1.writeJson)(host, (0, devkit_1.joinPathFragments)(options.e2eProjectRoot, 'package.json'), {
-            name: (0, get_import_path_1.getImportPath)(host, options.e2eProjectName),
+            name: options.importPath,
             version: '0.0.1',
             private: true,
             nx: {
-                name: options.e2eProjectName,
-                projectType: 'application',
+                name: options.e2eProjectName !== options.importPath
+                    ? options.e2eProjectName
+                    : undefined,
                 implicitDependencies: [options.project],
                 targets: {
                     e2e: {
                         executor: '@nx/jest:jest',
-                        outputs: ['{workspaceRoot}/coverage/{e2eProjectRoot}'],
+                        outputs: ['{projectRoot}/test-output/jest/coverage'],
                         options: {
                             jestConfig: `${options.e2eProjectRoot}/jest.config.ts`,
                             passWithNoTests: true,
@@ -101,10 +101,11 @@ async function e2eProjectGeneratorInternal(host, _options) {
     const coverageDirectory = isUsingTsSolutionConfig
         ? 'test-output/jest/coverage'
         : (0, devkit_1.joinPathFragments)(rootOffset, 'coverage', options.e2eProjectName);
+    const projectSimpleName = options.project.split('/').pop();
     if (options.projectType === 'server') {
         (0, devkit_1.generateFiles)(host, path.join(__dirname, 'files/server/common'), options.e2eProjectRoot, {
             ...options,
-            ...(0, devkit_1.names)(options.rootProject ? 'server' : options.project),
+            ...(0, devkit_1.names)(options.rootProject ? 'server' : projectSimpleName),
             tsConfigFile,
             offsetFromRoot: rootOffset,
             jestPreset,
@@ -115,7 +116,7 @@ async function e2eProjectGeneratorInternal(host, _options) {
         if (options.isNest) {
             (0, devkit_1.generateFiles)(host, path.join(__dirname, 'files/server/nest'), options.e2eProjectRoot, {
                 ...options,
-                ...(0, devkit_1.names)(options.rootProject ? 'server' : options.project),
+                ...(0, devkit_1.names)(options.rootProject ? 'server' : projectSimpleName),
                 tsConfigFile,
                 offsetFromRoot: rootOffset,
                 tmpl: '',
@@ -126,7 +127,7 @@ async function e2eProjectGeneratorInternal(host, _options) {
         const mainFile = appProject.targets.build?.options?.outputPath;
         (0, devkit_1.generateFiles)(host, path.join(__dirname, 'files/cli'), options.e2eProjectRoot, {
             ...options,
-            ...(0, devkit_1.names)(options.rootProject ? 'cli' : options.project),
+            ...(0, devkit_1.names)(options.rootProject ? 'cli' : projectSimpleName),
             mainFile,
             tsConfigFile,
             offsetFromRoot: rootOffset,
@@ -201,11 +202,15 @@ async function e2eProjectGeneratorInternal(host, _options) {
     return (0, devkit_1.runTasksInSerial)(...tasks);
 }
 async function normalizeOptions(tree, options) {
-    options.directory = options.directory ?? `${options.project}-e2e`;
-    const { projectName: e2eProjectName, projectRoot: e2eProjectRoot } = await (0, project_name_and_root_utils_1.determineProjectNameAndRootOptions)(tree, {
+    let directory = options.rootProject ? 'e2e' : options.directory;
+    if (!directory) {
+        const projectConfig = (0, devkit_1.readProjectConfiguration)(tree, options.project);
+        directory = `${projectConfig.root}-e2e`;
+    }
+    const { projectName: e2eProjectName, projectRoot: e2eProjectRoot, importPath, } = await (0, project_name_and_root_utils_1.determineProjectNameAndRootOptions)(tree, {
         name: options.name,
-        projectType: 'library',
-        directory: options.rootProject ? 'e2e' : options.directory,
+        projectType: 'application',
+        directory,
     });
     const nxJson = (0, devkit_1.readNxJson)(tree);
     const addPlugin = process.env.NX_ADD_PLUGINS !== 'false' &&
@@ -215,6 +220,7 @@ async function normalizeOptions(tree, options) {
         ...options,
         e2eProjectRoot,
         e2eProjectName,
+        importPath,
         port: options.port ?? 3000,
         rootProject: !!options.rootProject,
     };

package/src/generators/library/library.js

@@ -12,7 +12,6 @@ const versions_1 = require("../../utils/versions");
 const init_1 = require("../init/init");
 const target_defaults_utils_1 = require("@nx/devkit/src/generators/target-defaults-utils");
 const ts_solution_setup_1 = require("@nx/js/src/utils/typescript/ts-solution-setup");
-const get_import_path_1 = require("@nx/js/src/utils/get-import-path");
 const sort_fields_1 = require("@nx/js/src/utils/package-json/sort-fields");
 async function libraryGenerator(tree, schema) {
     return await libraryGeneratorInternal(tree, {
@@ -36,7 +35,7 @@ async function libraryGeneratorInternal(tree, schema) {
         options.publishable ||
         options.buildable) {
         (0, devkit_1.writeJson)(tree, (0, devkit_1.joinPathFragments)(options.projectRoot, 'package.json'), {
-            name: (0, get_import_path_1.getImportPath)(tree, options.name),
+            name: options.importPath,
             version: '0.0.1',
             private: true,
             files: options.publishable ? ['dist', '!**/*.tsbuildinfo'] : undefined,
@@ -75,7 +74,7 @@ async function libraryGeneratorInternal(tree, schema) {
 }
 exports.default = libraryGenerator;
 async function normalizeOptions(tree, options) {
-    await (0, project_name_and_root_utils_1.ensureProjectName)(tree, options, 'library');
+    await (0, project_name_and_root_utils_1.ensureRootProjectName)(options, 'library');
     const { projectName, names: projectNames, projectRoot, importPath, } = await (0, project_name_and_root_utils_1.determineProjectNameAndRootOptions)(tree, {
         name: options.name,
         projectType: 'library',
@@ -96,9 +95,7 @@ async function normalizeOptions(tree, options) {
     return {
         ...options,
         fileName,
-        projectName: isUsingTsSolutionConfig
-            ? (0, get_import_path_1.getImportPath)(tree, projectName)
-            : projectName,
+        projectName: isUsingTsSolutionConfig && !options.name ? importPath : projectName,
         projectRoot,
         parsedTags,
         importPath,