@nx/angular 20.2.1 → 20.2.2

package/migrations.json

@@ -1368,6 +1368,35 @@
           "alwaysAddToPackageJson": false
         }
       }
+    },
+    "20.2.2-angular-eslint": {
+      "version": "20.2.2-beta.0",
+      "requires": {
+        "eslint": "^8.57.0 || ^9.0.0",
+        "@angular/core": ">= 19.0.0 < 20.0.0"
+      },
+      "packages": {
+        "angular-eslint": {
+          "version": "^19.0.2",
+          "alwaysAddToPackageJson": false
+        },
+        "@angular-eslint/eslint-plugin": {
+          "version": "^19.0.2",
+          "alwaysAddToPackageJson": false
+        },
+        "@angular-eslint/eslint-plugin-template": {
+          "version": "^19.0.2",
+          "alwaysAddToPackageJson": false
+        },
+        "@angular-eslint/template-parser": {
+          "version": "^19.0.2",
+          "alwaysAddToPackageJson": false
+        },
+        "@angular-eslint/utils": {
+          "version": "^19.0.2",
+          "alwaysAddToPackageJson": false
+        }
+      }
     }
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nx/angular",
-  "version": "20.2.1",
+  "version": "20.2.2",
   "private": false,
   "description": "The Nx Plugin for Angular contains executors, generators, and utilities for managing Angular applications and libraries within an Nx workspace. It provides: \n\n- Integration with libraries such as Storybook, Jest, ESLint, Tailwind CSS, Playwright and Cypress. \n\n- Generators to help scaffold code quickly (like: Micro Frontends, Libraries, both internal to your codebase and publishable to npm) \n\n- Single Component Application Modules (SCAMs) \n\n- NgRx helpers. \n\n- Utilities for automatic workspace refactoring.",
   "repository": {
@@ -71,13 +71,13 @@
     "semver": "^7.5.3",
     "tslib": "^2.3.0",
     "webpack-merge": "^5.8.0",
-    "@nx/devkit": "20.2.1",
-    "@nx/js": "20.2.1",
-    "@nx/eslint": "20.2.1",
-    "@nx/webpack": "20.2.1",
-    "@nx/module-federation": "20.2.1",
-    "@nx/web": "20.2.1",
-    "@nx/workspace": "20.2.1",
+    "@nx/devkit": "20.2.2",
+    "@nx/js": "20.2.2",
+    "@nx/eslint": "20.2.2",
+    "@nx/webpack": "20.2.2",
+    "@nx/module-federation": "20.2.2",
+    "@nx/web": "20.2.2",
+    "@nx/workspace": "20.2.2",
     "piscina": "^4.4.0"
   },
   "peerDependencies": {

package/src/executors/application/utils/normalize-options.js

@@ -24,10 +24,11 @@ function normalizeOptions(options) {
             }
         }
     }
-    return {
-        ...options,
-        appShell: angularMajorVersion < 19 ? options.appShell ?? false : undefined,
-        prerender: angularMajorVersion < 19 ? options.prerender ?? false : undefined,
-        security,
-    };
+    let appShell = options.appShell;
+    let prerender = options.prerender;
+    if (angularMajorVersion < 19) {
+        appShell ??= false;
+        prerender ??= false;
+    }
+    return { ...options, appShell, prerender, security };
 }

package/src/generators/component/lib/normalize-options.js

@@ -11,6 +11,8 @@ async function normalizeOptions(tree, options) {
         name: options.name,
         path: options.path,
         suffix: options.type ?? 'component',
+        allowedFileExtensions: ['ts'],
+        fileExtension: 'ts',
     });
     const { className } = (0, devkit_1.names)(name);
     const { className: suffixClassName } = (0, devkit_1.names)(options.type);

package/src/generators/component/schema.json

@@ -9,7 +9,7 @@
   "properties": {
     "path": {
       "type": "string",
-      "description": "The file path to the component without the file extension and suffix. Relative to the current working directory.",
+      "description": "The file path to the component. Relative to the current working directory.",
       "$default": {
         "$source": "argv",
         "index": 0

package/src/generators/directive/lib/normalize-options.js

@@ -10,6 +10,8 @@ async function normalizeOptions(tree, options) {
         name: options.name,
         path: options.path,
         suffix: 'directive',
+        allowedFileExtensions: ['ts'],
+        fileExtension: 'ts',
     });
     const { className } = (0, devkit_1.names)(name);
     const { className: suffixClassName } = (0, devkit_1.names)('directive');

package/src/generators/directive/schema.json

@@ -9,6 +9,10 @@
   "examples": [
     {
       "description": "Generate a directive with the exported symbol matching the file name. It results in the directive `FooDirective` at `mylib/src/lib/foo.directive.ts`",
+      "command": "nx g @nx/angular:directive mylib/src/lib/foo.directive.ts"
+    },
+    {
+      "description": "Generate a directive without providing the file extension. It results in the directive `FooDirective` at `mylib/src/lib/foo.directive.ts`",
       "command": "nx g @nx/angular:directive mylib/src/lib/foo"
     },
     {
@@ -19,7 +23,7 @@
   "properties": {
     "path": {
       "type": "string",
-      "description": "The file path to the directive without the file extension and suffix. Relative to the current working directory.",
+      "description": "The file path to the directive. Relative to the current working directory.",
       "$default": {
         "$source": "argv",
         "index": 0

package/src/generators/pipe/lib/normalize-options.js

@@ -9,6 +9,8 @@ async function normalizeOptions(tree, options) {
         name: options.name,
         path: options.path,
         suffix: 'pipe',
+        allowedFileExtensions: ['ts'],
+        fileExtension: 'ts',
     });
     const { className } = (0, devkit_1.names)(name);
     const { className: suffixClassName } = (0, devkit_1.names)('pipe');

package/src/generators/pipe/schema.json

@@ -9,6 +9,10 @@
   "examples": [
     {
       "description": "Generate a pipe with the exported symbol matching the file name. It results in the pipe `FooPipe` at `mylib/src/lib/foo.pipe.ts`",
+      "command": "nx g @nx/angular:pipe mylib/src/lib/foo.pipe.ts"
+    },
+    {
+      "description": "Generate a pipe without providing the file extension. It results in the pipe `FooPipe` at `mylib/src/lib/foo.pipe.ts`",
       "command": "nx g @nx/angular:pipe mylib/src/lib/foo"
     },
     {
@@ -19,7 +23,7 @@
   "properties": {
     "path": {
       "type": "string",
-      "description": "The file path to the pipe without the file extension and suffix. Relative to the current working directory.",
+      "description": "The file path to the pipe. Relative to the current working directory.",
       "$default": {
         "$source": "argv",
         "index": 0

package/src/generators/scam/lib/normalize-options.js

@@ -10,6 +10,8 @@ async function normalizeOptions(tree, options) {
         name: options.name,
         path: options.path,
         suffix: options.type ?? 'component',
+        allowedFileExtensions: ['ts'],
+        fileExtension: 'ts',
     });
     const { className } = (0, devkit_1.names)(name);
     const { className: suffixClassName } = (0, devkit_1.names)(options.type);

package/src/generators/scam/schema.json

@@ -7,6 +7,10 @@
   "examples": [
     {
       "description": "Generate a component with the exported symbol matching the file name. It results in the component `FooComponent` at `mylib/src/lib/foo.component.ts`",
+      "command": "nx g @nx/angular:scam mylib/src/lib/foo.component.ts"
+    },
+    {
+      "description": "Generate a component without providing the file extension. It results in the component `FooComponent` at `mylib/src/lib/foo.component.ts`",
       "command": "nx g @nx/angular:scam mylib/src/lib/foo"
     },
     {
@@ -19,7 +23,7 @@
   "properties": {
     "path": {
       "type": "string",
-      "description": "The file path to the SCAM without the file extension and suffix. Relative to the current working directory.",
+      "description": "The file path to the SCAM. Relative to the current working directory.",
       "$default": {
         "$source": "argv",
         "index": 0

package/src/generators/scam-directive/lib/normalize-options.js

@@ -9,6 +9,8 @@ async function normalizeOptions(tree, options) {
         name: options.name,
         path: options.path,
         suffix: 'directive',
+        allowedFileExtensions: ['ts'],
+        fileExtension: 'ts',
     });
     const { className } = (0, devkit_1.names)(name);
     const { className: suffixClassName } = (0, devkit_1.names)('directive');

package/src/generators/scam-directive/schema.json

@@ -7,6 +7,10 @@
   "examples": [
     {
       "description": "Generate a directive with the exported symbol matching the file name. It results in the directive `FooDirective` at `mylib/src/lib/foo.directive.ts`",
+      "command": "nx g @nx/angular:scam-directive mylib/src/lib/foo.directive.ts"
+    },
+    {
+      "description": "Generate a directive without providing the file extension. It results in the directive `FooDirective` at `mylib/src/lib/foo.directive.ts`",
       "command": "nx g @nx/angular:scam-directive mylib/src/lib/foo"
     },
     {
@@ -19,7 +23,7 @@
   "properties": {
     "path": {
       "type": "string",
-      "description": "The file path to the SCAM directive without the file extension and suffix. Relative to the current working directory.",
+      "description": "The file path to the SCAM directive. Relative to the current working directory.",
       "$default": {
         "$source": "argv",
         "index": 0

package/src/generators/scam-pipe/lib/normalize-options.js

@@ -9,6 +9,8 @@ async function normalizeOptions(tree, options) {
         name: options.name,
         path: options.path,
         suffix: 'pipe',
+        allowedFileExtensions: ['ts'],
+        fileExtension: 'ts',
     });
     const { className } = (0, devkit_1.names)(name);
     const { className: suffixClassName } = (0, devkit_1.names)('pipe');

package/src/generators/scam-pipe/schema.json

@@ -7,6 +7,10 @@
   "examples": [
     {
       "description": "Generate a pipe with the exported symbol matching the file name. It results in the pipe `FooPipe` at `mylib/src/lib/foo.pipe.ts`",
+      "command": "nx g @nx/angular:scam-pipe mylib/src/lib/foo.pipe.ts"
+    },
+    {
+      "description": "Generate a pipe without providing the file extension. It results in the pipe `FooPipe` at `mylib/src/lib/foo.pipe.ts`",
       "command": "nx g @nx/angular:scam-pipe mylib/src/lib/foo"
     },
     {
@@ -19,7 +23,7 @@
   "properties": {
     "path": {
       "type": "string",
-      "description": "The file path to the SCAM pipe without the file extension and suffix. Relative to the current working directory.",
+      "description": "The file path to the SCAM pipe. Relative to the current working directory.",
       "$default": {
         "$source": "argv",
         "index": 0

package/src/migrations/update-20-2-0/remove-angular-eslint-rules.d.ts

@@ -1,2 +1,3 @@
 import { type Tree } from '@nx/devkit';
+export declare const rulesToRemove: string[];
 export default function (tree: Tree): Promise<void>;

package/src/migrations/update-20-2-0/remove-angular-eslint-rules.js

@@ -1,33 +1,62 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.rulesToRemove = void 0;
 exports.default = default_1;
 const devkit_1 = require("@nx/devkit");
 const eslint_file_1 = require("@nx/eslint/src/generators/utils/eslint-file");
 const projects_1 = require("../utils/projects");
+exports.rulesToRemove = [
+    '@angular-eslint/no-host-metadata-property',
+    '@angular-eslint/sort-ngmodule-metadata-arrays',
+    '@angular-eslint/prefer-standalone-component',
+];
 async function default_1(tree) {
     const projects = await (0, projects_1.getProjectsFilteredByDependencies)(tree, [
         'npm:@angular/core',
     ]);
+    let hasRootProject = false;
     for (const { project: { root }, } of projects) {
         if (!(0, eslint_file_1.isEslintConfigSupported)(tree, root)) {
             // ESLint config is not supported, skip
             continue;
         }
-        removeRule(tree, root, '@angular-eslint/no-host-metadata-property');
-        removeRule(tree, root, '@angular-eslint/sort-ngmodule-metadata-arrays');
-        removeRule(tree, root, '@angular-eslint/prefer-standalone-component');
+        if (root === '.') {
+            hasRootProject = true;
+        }
+        removeRules(tree, root);
+    }
+    /**
+     * We need to handle both a root config file (e.g. eslint.config.js) and a
+     * potential base config file (e.g. eslint.base.config.js). We can't use
+     * `findEslintFile` because it would return only one or the other depending
+     * on whether a root is provided and the existence of the files. So, we
+     * handle each of them separately.
+     */
+    // check root config, provide a root so it doesn't try to lookup a base config
+    if (!hasRootProject) {
+        // if there is no root project the root eslint config has not been processed
+        if ((0, eslint_file_1.isEslintConfigSupported)(tree, '')) {
+            removeRules(tree, '');
+        }
+    }
+    // handle root base config, not providing a root will prioritize a base config
+    const baseEslintConfig = (0, eslint_file_1.findEslintFile)(tree);
+    if (baseEslintConfig && baseEslintConfig.includes('.base.')) {
+        removeRules(tree, baseEslintConfig);
     }
     await (0, devkit_1.formatFiles)(tree);
 }
-function removeRule(tree, root, rule) {
-    const lookup = (o) => !!o.rules?.[rule];
-    if (!(0, eslint_file_1.lintConfigHasOverride)(tree, root, lookup, true)) {
-        // it's not using the rule, skip
-        return;
+function removeRules(tree, root) {
+    for (const rule of exports.rulesToRemove) {
+        const lookup = (o) => !!o.rules?.[rule];
+        if (!(0, eslint_file_1.lintConfigHasOverride)(tree, root, lookup)) {
+            // it's not using the rule, skip
+            continue;
+        }
+        // there is an override containing the rule, remove the rule
+        (0, eslint_file_1.updateOverrideInLintConfig)(tree, root, lookup, (o) => {
+            delete o.rules[rule];
+            return o;
+        });
     }
-    // there is an override containing the rule, remove the rule
-    (0, eslint_file_1.updateOverrideInLintConfig)(tree, root, lookup, (o) => {
-        delete o.rules[rule];
-        return o;
-    });
 }

package/src/utils/versions.d.ts

@@ -14,7 +14,7 @@ export declare const typesExpressVersion = "4.17.14";
 export declare const browserSyncVersion = "^3.0.0";
 export declare const moduleFederationNodeVersion = "~2.6.11";
 export declare const moduleFederationEnhancedVersion = "0.7.6";
-export declare const angularEslintVersion = "^19.0.0";
+export declare const angularEslintVersion = "^19.0.2";
 export declare const typescriptEslintVersion = "^7.16.0";
 export declare const tailwindVersion = "^3.0.2";
 export declare const postcssVersion = "^8.4.5";

package/src/utils/versions.js

@@ -17,7 +17,7 @@ exports.typesExpressVersion = '4.17.14';
 exports.browserSyncVersion = '^3.0.0';
 exports.moduleFederationNodeVersion = '~2.6.11';
 exports.moduleFederationEnhancedVersion = '0.7.6';
-exports.angularEslintVersion = '^19.0.0';
+exports.angularEslintVersion = '^19.0.2';
 exports.typescriptEslintVersion = '^7.16.0';
 exports.tailwindVersion = '^3.0.2';
 exports.postcssVersion = '^8.4.5';