@nx/eslint 21.0.0-beta.1 → 21.0.0-beta.11

package/migrations.json

@@ -1,25 +1,5 @@
 {
   "generators": {
-    "update-17-0-0-rename-to-eslint": {
-      "version": "17.0.0-beta.7",
-      "description": "update-17-0-0-rename-to-eslint",
-      "implementation": "./src/migrations/update-17-0-0-rename-to-eslint/update-17-0-0-rename-to-eslint"
-    },
-    "update-typescript-eslint": {
-      "version": "17.1.0-beta.1",
-      "description": "Updates for @typescript-utils/utils v6.9.1+",
-      "implementation": "./src/migrations/update-17-1-0/update-typescript-eslint"
-    },
-    "simplify-eslint-patterns": {
-      "version": "17.2.0-beta.0",
-      "description": "Simplify eslintFilePatterns",
-      "implementation": "./src/migrations/update-17-2-0/simplify-eslint-patterns"
-    },
-    "move-options-to-target-defaults": {
-      "version": "17.2.9",
-      "description": "Move executor options to target defaults",
-      "implementation": "./src/migrations/update-17-2-9/move-options-to-target-defaults"
-    },
     "update-typescript-eslint-v8.13.0": {
       "version": "20.2.0-beta.5",
       "description": "Update TypeScript ESLint packages to v8.13.0 if they are already on v8",
@@ -32,67 +12,6 @@
     }
   },
   "packageJsonUpdates": {
-    "17.0.0": {
-      "version": "17.0.0-rc.2",
-      "packages": {
-        "eslint-config-prettier": {
-          "version": "^9.0.0"
-        }
-      }
-    },
-    "17.1.0": {
-      "version": "17.1.0-beta.1",
-      "packages": {
-        "@typescript-eslint/parser": {
-          "version": "^6.9.1"
-        },
-        "@typescript-eslint/eslint-plugin": {
-          "version": "^6.9.1"
-        },
-        "@typescript-eslint/utils": {
-          "version": "^6.9.1"
-        }
-      }
-    },
-    "17.2.0": {
-      "version": "17.2.0-beta.2",
-      "packages": {
-        "eslint": {
-          "version": "~8.48.0"
-        }
-      }
-    },
-    "17.3.0": {
-      "version": "17.3.0-beta.0",
-      "packages": {
-        "@typescript-eslint/parser": {
-          "version": "^6.13.2"
-        },
-        "@typescript-eslint/eslint-plugin": {
-          "version": "^6.13.2"
-        },
-        "@typescript-eslint/utils": {
-          "version": "^6.13.2"
-        }
-      }
-    },
-    "18.2.0": {
-      "version": "18.2.0-beta.0",
-      "packages": {
-        "@typescript-eslint/parser": {
-          "version": "^7.3.0"
-        },
-        "@typescript-eslint/eslint-plugin": {
-          "version": "^7.3.0"
-        },
-        "@typescript-eslint/utils": {
-          "version": "^7.3.0"
-        },
-        "eslint": {
-          "version": "~8.57.0"
-        }
-      }
-    },
     "19.5.0": {
       "version": "19.5.0-beta.1",
       "packages": {
@@ -104,6 +23,18 @@
         },
         "@typescript-eslint/utils": {
           "version": "^7.16.0"
+        },
+        "@typescript-eslint/rule-tester": {
+          "version": "^7.16.0",
+          "alwaysAddToPackageJson": false
+        },
+        "@typescript-eslint/scope-manager": {
+          "version": "^7.16.0",
+          "alwaysAddToPackageJson": false
+        },
+        "@typescript-eslint/typescript-estree": {
+          "version": "^7.16.0",
+          "alwaysAddToPackageJson": false
         }
       }
     },
@@ -116,14 +47,26 @@
         "typescript-eslint": {
           "version": "^8.19.0"
         },
-        "@typescript-eslint/parser": {
+        "@typescript-eslint/eslint-plugin": {
           "version": "^8.19.0"
         },
-        "@typescript-eslint/eslint-plugin": {
+        "@typescript-eslint/parser": {
           "version": "^8.19.0"
         },
         "@typescript-eslint/utils": {
           "version": "^8.19.0"
+        },
+        "@typescript-eslint/rule-tester": {
+          "version": "^8.19.0",
+          "alwaysAddToPackageJson": false
+        },
+        "@typescript-eslint/scope-manager": {
+          "version": "^8.19.0",
+          "alwaysAddToPackageJson": false
+        },
+        "@typescript-eslint/typescript-estree": {
+          "version": "^8.19.0",
+          "alwaysAddToPackageJson": false
         }
       }
     },
@@ -136,14 +79,34 @@
         "typescript-eslint": {
           "version": "^8.19.0"
         },
-        "@typescript-eslint/parser": {
+        "@typescript-eslint/eslint-plugin": {
           "version": "^8.19.0"
         },
-        "@typescript-eslint/eslint-plugin": {
+        "@typescript-eslint/parser": {
           "version": "^8.19.0"
         },
         "@typescript-eslint/utils": {
           "version": "^8.19.0"
+        },
+        "@typescript-eslint/rule-tester": {
+          "version": "^8.19.0",
+          "alwaysAddToPackageJson": false
+        },
+        "@typescript-eslint/scope-manager": {
+          "version": "^8.19.0",
+          "alwaysAddToPackageJson": false
+        },
+        "@typescript-eslint/typescript-estree": {
+          "version": "^8.19.0",
+          "alwaysAddToPackageJson": false
+        }
+      }
+    },
+    "20.7.0": {
+      "version": "20.7.0-beta.4",
+      "packages": {
+        "eslint-config-prettier": {
+          "version": "^10.0.0"
         }
       }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nx/eslint",
-  "version": "21.0.0-beta.1",
+  "version": "21.0.0-beta.11",
   "private": false,
   "description": "The ESLint plugin for Nx contains executors, generators and utilities used for linting JavaScript/TypeScript projects within an Nx workspace.",
   "repository": {
@@ -35,8 +35,8 @@
     "eslint": "^8.0.0 || ^9.0.0"
   },
   "dependencies": {
-    "@nx/devkit": "21.0.0-beta.1",
-    "@nx/js": "21.0.0-beta.1",
+    "@nx/devkit": "21.0.0-beta.11",
+    "@nx/js": "21.0.0-beta.11",
     "semver": "^7.5.3",
     "tslib": "^2.3.0",
     "typescript": "~5.7.2"

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

@@ -12,6 +12,7 @@ const flat_config_1 = require("../../utils/flat-config");
 const ast_utils_1 = require("../utils/flat-config/ast-utils");
 const config_file_1 = require("../../utils/config-file");
 const plugin_1 = require("../utils/plugin");
+const versions_1 = require("../../utils/versions");
 const setup_root_eslint_1 = require("./setup-root-eslint");
 const ts_solution_setup_1 = require("@nx/js/src/utils/typescript/ts-solution-setup");
 function lintProjectGenerator(tree, options) {
@@ -96,7 +97,12 @@ async function lintProjectGeneratorInternal(tree, options) {
     // additionally, the companion e2e app would have `rootProject: true`
     // so we need to check for the root path as well
     if (!options.rootProject || projectConfig.root !== '.') {
-        createEsLintConfiguration(tree, options, projectConfig, options.setParserOptionsProject, options.rootProject);
+        const addDependencyChecks = options.addPackageJsonDependencyChecks ||
+            isBuildableLibraryProject(tree, projectConfig);
+        createEsLintConfiguration(tree, options, projectConfig, options.setParserOptionsProject, options.rootProject, addDependencyChecks);
+        if (addDependencyChecks) {
+            tasks.push((0, devkit_1.addDependenciesToPackageJson)(tree, {}, { 'jsonc-eslint-parser': versions_1.jsoncEslintParserVersion }, undefined, true));
+        }
     }
     // Buildable libs need source analysis enabled for linting `package.json`.
     if (isBuildableLibraryProject(tree, projectConfig) &&
@@ -114,7 +120,7 @@ async function lintProjectGeneratorInternal(tree, options) {
     }
     return (0, devkit_1.runTasksInSerial)(...tasks);
 }
-function createEsLintConfiguration(tree, options, projectConfig, setParserOptionsProject, rootProject) {
+function createEsLintConfiguration(tree, options, projectConfig, setParserOptionsProject, rootProject, addDependencyChecks) {
     // we are only extending root for non-standalone projects or their complementary e2e apps
     const extendedRootConfig = rootProject ? undefined : (0, eslint_file_1.findEslintFile)(tree);
     const pathToRootConfig = extendedRootConfig
@@ -132,8 +138,6 @@ function createEsLintConfiguration(tree, options, projectConfig, setParserOption
             options.eslintConfigFormat = (0, eslint_file_1.determineEslintConfigFormat)(tree.read(extendedRootConfig, 'utf-8'));
         }
     }
-    const addDependencyChecks = options.addPackageJsonDependencyChecks ||
-        isBuildableLibraryProject(tree, projectConfig);
     const overrides = (0, flat_config_1.useFlatConfig)(tree)
         ? // For flat configs, we don't need to generate different overrides for each file. Users should add their own overrides as needed.
             []

package/src/generators/utils/eslint-file.js

@@ -357,9 +357,9 @@ function addExtendsToLintConfig(tree, root, plugin, insertAtTheEnd = false) {
         }
         tree.write(fileName, content);
         if (shouldImportEslintCompat) {
-            return (0, devkit_1.addDependenciesToPackageJson)(tree, {}, { '@eslint/compat': versions_1.eslintCompat }, undefined, true);
+            return (0, devkit_1.addDependenciesToPackageJson)(tree, {}, { '@eslint/compat': versions_1.eslintCompat, '@eslint/eslintrc': versions_1.eslintrcVersion }, undefined, true);
         }
-        return () => { };
+        return (0, devkit_1.addDependenciesToPackageJson)(tree, {}, { '@eslint/eslintrc': versions_1.eslintrcVersion }, undefined, true);
     }
     else {
         const plugins = (Array.isArray(plugin) ? plugin : [plugin]).map((p) => typeof p === 'string' ? p : p.name);
@@ -430,13 +430,26 @@ function addIgnoresToLintConfig(tree, root, ignorePatterns) {
                 break;
             }
         }
-        const block = (0, ast_utils_1.generateAst)({
-            ignores: ignorePatterns.map((path) => (0, path_utils_1.mapFilePath)(path)),
-        });
-        tree.write(fileName, (0, ast_utils_1.addBlockToFlatConfigExport)(tree.read(fileName, 'utf8'), block));
+        if (!fileName) {
+            return;
+        }
+        let content = tree.read(fileName, 'utf8');
+        if ((0, ast_utils_1.hasFlatConfigIgnoresBlock)(content)) {
+            content = (0, ast_utils_1.addPatternsToFlatConfigIgnoresBlock)(content, ignorePatterns);
+            tree.write(fileName, content);
+        }
+        else {
+            const block = (0, ast_utils_1.generateAst)({
+                ignores: ignorePatterns.map((path) => (0, path_utils_1.mapFilePath)(path)),
+            });
+            tree.write(fileName, (0, ast_utils_1.addBlockToFlatConfigExport)(content, block));
+        }
     }
     else {
         const fileName = (0, devkit_1.joinPathFragments)(root, '.eslintrc.json');
+        if (!tree.exists(fileName)) {
+            return;
+        }
         (0, devkit_1.updateJson)(tree, fileName, (json) => {
             const ignoreSet = new Set([
                 ...(json.ignorePatterns ?? []),

package/src/generators/utils/flat-config/ast-utils.d.ts

@@ -4,6 +4,8 @@ import * as ts from 'typescript';
  * Remove all overrides from the config file
  */
 export declare function removeOverridesFromLintConfig(content: string): string;
+export declare function addPatternsToFlatConfigIgnoresBlock(content: string, ignorePatterns: string[]): string;
+export declare function hasFlatConfigIgnoresBlock(content: string): boolean;
 export declare function hasOverride(content: string, lookup: (override: Linter.ConfigOverride<Linter.RulesRecord>) => boolean): boolean;
 /**
  * Finds an override matching the lookup function and applies the update function to it

package/src/generators/utils/flat-config/ast-utils.js

@@ -1,6 +1,8 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.removeOverridesFromLintConfig = removeOverridesFromLintConfig;
+exports.addPatternsToFlatConfigIgnoresBlock = addPatternsToFlatConfigIgnoresBlock;
+exports.hasFlatConfigIgnoresBlock = hasFlatConfigIgnoresBlock;
 exports.hasOverride = hasOverride;
 exports.replaceOverride = replaceOverride;
 exports.addImportToFlatConfig = addImportToFlatConfig;
@@ -73,6 +75,55 @@ function findModuleExports(source) {
         }
     });
 }
+function addPatternsToFlatConfigIgnoresBlock(content, ignorePatterns) {
+    const source = ts.createSourceFile('', content, ts.ScriptTarget.Latest, true, ts.ScriptKind.JS);
+    const format = content.includes('export default') ? 'mjs' : 'cjs';
+    const exportsArray = format === 'mjs' ? findExportDefault(source) : findModuleExports(source);
+    if (!exportsArray) {
+        return content;
+    }
+    const changes = [];
+    for (const node of exportsArray) {
+        if (!isFlatConfigIgnoresBlock(node)) {
+            continue;
+        }
+        const start = node.properties.pos + 1; // keep leading line break
+        const data = parseTextToJson(node.getFullText());
+        changes.push({
+            type: devkit_1.ChangeType.Delete,
+            start,
+            length: node.properties.end - start,
+        });
+        data.ignores = Array.from(new Set([...(data.ignores ?? []), ...ignorePatterns]));
+        changes.push({
+            type: devkit_1.ChangeType.Insert,
+            index: start,
+            text: '    ' +
+                JSON.stringify(data, null, 2)
+                    .slice(2, -2) // Remove curly braces and start/end line breaks
+                    .replaceAll(/\n/g, '\n    '), // Maintain indentation
+        });
+        break;
+    }
+    return (0, devkit_1.applyChangesToString)(content, changes);
+}
+function hasFlatConfigIgnoresBlock(content) {
+    const source = ts.createSourceFile('', content, ts.ScriptTarget.Latest, true, ts.ScriptKind.JS);
+    const format = content.includes('export default') ? 'mjs' : 'cjs';
+    const exportsArray = format === 'mjs' ? findExportDefault(source) : findModuleExports(source);
+    if (!exportsArray) {
+        return false;
+    }
+    return exportsArray.some(isFlatConfigIgnoresBlock);
+}
+function isFlatConfigIgnoresBlock(node) {
+    return (ts.isObjectLiteralExpression(node) &&
+        node.properties.length === 1 &&
+        (node.properties[0].name.getText() === 'ignores' ||
+            node.properties[0].name.getText() === '"ignores"') &&
+        ts.isPropertyAssignment(node.properties[0]) &&
+        ts.isArrayLiteralExpression(node.properties[0].initializer));
+}
 function isOverride(node) {
     return ((ts.isObjectLiteralExpression(node) &&
         node.properties.some((p) => p.name.getText() === 'files' || p.name.getText() === '"files"')) ||

package/src/generators/utils/linter.d.ts

@@ -1,3 +1,6 @@
+/**
+ * @deprecated Use LinterType instead. It will be removed in Nx v22.
+ */
 export declare enum Linter {
     EsLint = "eslint",
     None = "none"

package/src/generators/utils/linter.js

@@ -1,8 +1,8 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.Linter = void 0;
-/*
- * @deprecated Use LinterType instead
+/**
+ * @deprecated Use LinterType instead. It will be removed in Nx v22.
  */
 var Linter;
 (function (Linter) {

package/src/plugins/plugin.js

@@ -107,7 +107,7 @@ const internalCreateNodes = async (configFilePath, options, context, projectsCac
         projects,
     };
 };
-const internalCreateNodesV2 = async (configFilePath, options, context, eslintConfigFiles, projectRootsByEslintRoots, lintableFilesPerProjectRoot, projectsCache) => {
+const internalCreateNodesV2 = async (configFilePath, options, context, projectRootsByEslintRoots, lintableFilesPerProjectRoot, projectsCache, hashByRoot) => {
     const configDir = (0, posix_1.dirname)(configFilePath);
     const ESLint = await (0, resolve_eslint_class_1.resolveESLintClass)({
         useFlatConfigOverrideVal: (0, config_file_1.isFlatConfig)(configFilePath),
@@ -115,12 +115,7 @@ const internalCreateNodesV2 = async (configFilePath, options, context, eslintCon
     const eslintVersion = ESLint.version;
     const projects = {};
     await Promise.all(projectRootsByEslintRoots.get(configDir).map(async (projectRoot) => {
-        const parentConfigs = eslintConfigFiles.filter((eslintConfig) => isSubDir(projectRoot, (0, posix_1.dirname)(eslintConfig)));
-        const hash = await (0, calculate_hash_for_create_nodes_1.calculateHashForCreateNodes)(projectRoot, options, {
-            configFiles: eslintConfigFiles,
-            nxJsonConfiguration: context.nxJsonConfiguration,
-            workspaceRoot: context.workspaceRoot,
-        }, [...parentConfigs, (0, posix_1.join)(projectRoot, '.eslintignore')]);
+        const hash = hashByRoot.get(projectRoot);
         if (projectsCache[hash]) {
             // We can reuse the projects in the cache.
             Object.assign(projects, projectsCache[hash]);
@@ -165,8 +160,13 @@ exports.createNodesV2 = [
         const targetsCache = readTargetsCache(cachePath);
         const { eslintConfigFiles, projectRoots, projectRootsByEslintRoots } = splitConfigFiles(configFiles);
         const lintableFilesPerProjectRoot = await collectLintableFilesByProjectRoot(projectRoots, options, context);
+        const hashes = await (0, calculate_hash_for_create_nodes_1.calculateHashesForCreateNodes)(projectRoots, options, context, projectRoots.map((root) => {
+            const parentConfigs = eslintConfigFiles.filter((eslintConfig) => isSubDir(root, (0, posix_1.dirname)(eslintConfig)));
+            return [...parentConfigs, (0, posix_1.join)(root, '.eslintignore')];
+        }));
+        const hashByRoot = new Map(projectRoots.map((r, i) => [r, hashes[i]]));
         try {
-            return await (0, devkit_1.createNodesFromFiles)((configFile, options, context) => internalCreateNodesV2(configFile, options, context, eslintConfigFiles, projectRootsByEslintRoots, lintableFilesPerProjectRoot, targetsCache), eslintConfigFiles, options, context);
+            return await (0, devkit_1.createNodesFromFiles)((configFile, options, context) => internalCreateNodesV2(configFile, options, context, projectRootsByEslintRoots, lintableFilesPerProjectRoot, targetsCache, hashByRoot), eslintConfigFiles, options, context);
         }
         finally {
             writeTargetsToCache(cachePath, targetsCache);

package/src/utils/versions.d.ts

@@ -1,8 +1,9 @@
 export declare const nxVersion: any;
 export declare const eslintVersion = "~8.57.0";
 export declare const eslintrcVersion = "^2.1.1";
-export declare const eslintConfigPrettierVersion = "^9.0.0";
+export declare const eslintConfigPrettierVersion = "^10.0.0";
 export declare const typescriptESLintVersion = "^7.16.0";
+export declare const jsoncEslintParserVersion = "^2.1.0";
 export declare const eslint9__typescriptESLintVersion = "^8.19.0";
 export declare const eslint9__eslintVersion = "^9.8.0";
 export declare const eslintCompat = "^1.1.1";

package/src/utils/versions.js

@@ -1,11 +1,12 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.eslintCompat = exports.eslint9__eslintVersion = exports.eslint9__typescriptESLintVersion = exports.typescriptESLintVersion = exports.eslintConfigPrettierVersion = exports.eslintrcVersion = exports.eslintVersion = exports.nxVersion = void 0;
+exports.eslintCompat = exports.eslint9__eslintVersion = exports.eslint9__typescriptESLintVersion = exports.jsoncEslintParserVersion = exports.typescriptESLintVersion = exports.eslintConfigPrettierVersion = exports.eslintrcVersion = exports.eslintVersion = exports.nxVersion = void 0;
 exports.nxVersion = require('../../package.json').version;
 exports.eslintVersion = '~8.57.0';
 exports.eslintrcVersion = '^2.1.1';
-exports.eslintConfigPrettierVersion = '^9.0.0';
+exports.eslintConfigPrettierVersion = '^10.0.0';
 exports.typescriptESLintVersion = '^7.16.0';
+exports.jsoncEslintParserVersion = '^2.1.0';
 // Updated linting stack for ESLint v9, typescript-eslint v8
 exports.eslint9__typescriptESLintVersion = '^8.19.0';
 exports.eslint9__eslintVersion = '^9.8.0';

package/src/migrations/update-17-0-0-rename-to-eslint/update-17-0-0-rename-to-eslint.d.ts

@@ -1,2 +0,0 @@
-import { Tree } from '@nx/devkit';
-export default function replacePackage(tree: Tree): Promise<void>;

package/src/migrations/update-17-0-0-rename-to-eslint/update-17-0-0-rename-to-eslint.js

@@ -1,47 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.default = replacePackage;
-const devkit_1 = require("@nx/devkit");
-const replace_package_1 = require("@nx/devkit/src/utils/replace-package");
-async function replacePackage(tree) {
-    await (0, replace_package_1.replaceNrwlPackageWithNxPackage)(tree, '@nx/linter', '@nx/eslint');
-    // executor name change from :eslint to :lint
-    updateNxJsonExecutor(tree);
-    updateProjectExecutor(tree);
-    await (0, devkit_1.formatFiles)(tree);
-}
-function updateNxJsonExecutor(tree) {
-    if (!tree.exists('nx.json')) {
-        return;
-    }
-    const nxJson = (0, devkit_1.readNxJson)(tree);
-    let needsUpdate = false;
-    for (const [targetName, targetConfig] of Object.entries(nxJson.targetDefaults ?? {})) {
-        // this will be in a broken state after the package is globally renamed
-        if (targetConfig.executor !== '@nx/eslint:eslint') {
-            continue;
-        }
-        needsUpdate = true;
-        nxJson.targetDefaults[targetName].executor = '@nx/eslint:lint';
-    }
-    if (needsUpdate) {
-        (0, devkit_1.updateNxJson)(tree, nxJson);
-    }
-}
-function updateProjectExecutor(tree) {
-    const projects = (0, devkit_1.getProjects)(tree);
-    for (const [projectName, projectConfiguration] of projects) {
-        let needsUpdate = false;
-        for (const [targetName, targetConfig] of Object.entries(projectConfiguration.targets ?? {})) {
-            // this will be in a broken state after the package is globally renamed
-            if (targetConfig.executor !== '@nx/eslint:eslint') {
-                continue;
-            }
-            needsUpdate = true;
-            projectConfiguration.targets[targetName].executor = '@nx/eslint:lint';
-        }
-        if (needsUpdate) {
-            (0, devkit_1.updateProjectConfiguration)(tree, projectName, projectConfiguration);
-        }
-    }
-}

package/src/migrations/update-17-1-0/update-typescript-eslint.d.ts

@@ -1,2 +0,0 @@
-import { Tree } from '@nx/devkit';
-export default function update(tree: Tree): Promise<void>;

package/src/migrations/update-17-1-0/update-typescript-eslint.js

@@ -1,74 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.default = update;
-const devkit_1 = require("@nx/devkit");
-const versions_1 = require("../../utils/versions");
-const path_1 = require("path");
-const typescript_1 = require("typescript");
-function updateJestConfig(tree) {
-    const jestConfigPath = 'tools/eslint-rules/jest.config.ts';
-    if (tree.exists(jestConfigPath)) {
-        const { removePropertyFromJestConfig } = (0, devkit_1.ensurePackage)('@nx/jest', versions_1.nxVersion);
-        removePropertyFromJestConfig(tree, jestConfigPath, [
-            'moduleNameMapper',
-            '@eslint/eslintrc',
-        ]);
-    }
-}
-function updateTsConfigs(tree) {
-    const tsConfigPath = 'tools/eslint-rules/tsconfig.json';
-    if (tree.exists(tsConfigPath)) {
-        (0, devkit_1.updateJson)(tree, tsConfigPath, (tsConfig) => {
-            tsConfig.compilerOptions ??= {};
-            tsConfig.compilerOptions.moduleResolution = 'node16';
-            tsConfig.compilerOptions.module = 'node16';
-            return tsConfig;
-        });
-    }
-    const tsConfigSpec = 'tools/eslint-rules/tsconfig.spec.json';
-    if (tree.exists(tsConfigSpec)) {
-        (0, devkit_1.updateJson)(tree, tsConfigSpec, (tsConfigSpec) => {
-            delete tsConfigSpec.compilerOptions?.module;
-            delete tsConfigSpec.compilerOptions?.moduleResolution;
-            return tsConfigSpec;
-        });
-    }
-}
-function updateRecommended(tree) {
-    (0, devkit_1.visitNotIgnoredFiles)(tree, 'tools/eslint-rules', (path) => {
-        if ((0, path_1.extname)(path) !== '.ts') {
-            return;
-        }
-        const contents = tree.read(path, 'utf-8');
-        const sourceFile = (0, typescript_1.createSourceFile)(path, contents, typescript_1.ScriptTarget.ESNext, true);
-        const changes = [];
-        const visit = (node) => {
-            if ((0, typescript_1.isPropertyAssignment)(node) &&
-                (0, typescript_1.isIdentifier)(node.name) &&
-                node.name.text === 'recommended' &&
-                (0, typescript_1.isStringLiteral)(node.initializer)) {
-                changes.push({
-                    type: devkit_1.ChangeType.Delete,
-                    start: node.initializer.getStart(sourceFile),
-                    length: node.initializer.getWidth(sourceFile),
-                });
-                changes.push({
-                    type: devkit_1.ChangeType.Insert,
-                    index: node.initializer.getStart(sourceFile),
-                    text: "'recommended'",
-                });
-            }
-            else {
-                (0, typescript_1.forEachChild)(node, visit);
-            }
-        };
-        (0, typescript_1.forEachChild)(sourceFile, visit);
-        tree.write(path, (0, devkit_1.applyChangesToString)(contents, changes));
-    });
-}
-async function update(tree) {
-    updateJestConfig(tree);
-    updateTsConfigs(tree);
-    updateRecommended(tree);
-    await (0, devkit_1.formatFiles)(tree);
-}

package/src/migrations/update-17-2-0/simplify-eslint-patterns.d.ts

@@ -1,2 +0,0 @@
-import { Tree } from '@nx/devkit';
-export default function update(tree: Tree): Promise<void>;

package/src/migrations/update-17-2-0/simplify-eslint-patterns.js

@@ -1,46 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.default = update;
-const devkit_1 = require("@nx/devkit");
-async function update(tree) {
-    const projects = (0, devkit_1.getProjects)(tree);
-    for (const [projectName, projectConfiguration] of projects) {
-        let needsUpdate = false;
-        for (const [targetName, targetConfig] of Object.entries(projectConfiguration.targets ?? {})) {
-            if (targetConfig.executor !== '@nx/eslint:lint') {
-                continue;
-            }
-            needsUpdate = true;
-            if (projectConfiguration.targets[targetName].options?.lintFilePatterns) {
-                const rootPattern = getLintRoot(projectConfiguration);
-                const nonRootPatterns = projectConfiguration.targets[targetName].options.lintFilePatterns.filter((p) => !p.startsWith(rootPattern) && !p.startsWith('{projectRoot}'));
-                if (nonRootPatterns.length === 0 &&
-                    rootPattern === projectConfiguration.root) {
-                    // delete the lintFilePatterns option if it's the only option and matches the root of the project
-                    delete projectConfiguration.targets[targetName].options
-                        .lintFilePatterns;
-                    if (Object.keys(projectConfiguration.targets[targetName].options)
-                        .length === 0) {
-                        delete projectConfiguration.targets[targetName].options;
-                    }
-                }
-                else {
-                    projectConfiguration.targets[targetName].options.lintFilePatterns = [
-                        rootPattern,
-                        ...nonRootPatterns,
-                    ];
-                }
-            }
-        }
-        if (needsUpdate) {
-            (0, devkit_1.updateProjectConfiguration)(tree, projectName, projectConfiguration);
-        }
-    }
-    await (0, devkit_1.formatFiles)(tree);
-}
-function getLintRoot({ root, sourceRoot }) {
-    if (root === '' || root === '.') {
-        return sourceRoot || './src';
-    }
-    return root;
-}

package/src/migrations/update-17-2-9/move-options-to-target-defaults.d.ts

@@ -1,2 +0,0 @@
-import { Tree } from '@nx/devkit';
-export default function update(tree: Tree): Promise<void>;

package/src/migrations/update-17-2-9/move-options-to-target-defaults.js

@@ -1,107 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.default = update;
-const devkit_1 = require("@nx/devkit");
-const executor_options_utils_1 = require("@nx/devkit/src/generators/executor-options-utils");
-const eslint_file_1 = require("../../generators/utils/eslint-file");
-const project_configuration_utils_1 = require("nx/src/project-graph/utils/project-configuration-utils");
-async function update(tree) {
-    const nxJson = (0, devkit_1.readNxJson)(tree);
-    // Don't override anything if there are already target defaults for eslint
-    if (nxJson.targetDefaults?.['@nx/eslint:lint']) {
-        return;
-    }
-    nxJson.targetDefaults ??= {};
-    const graph = await (0, devkit_1.createProjectGraphAsync)();
-    const lintTargets = new Set();
-    (0, executor_options_utils_1.forEachExecutorOptionsInGraph)(graph, '@nx/eslint:lint', (value, proj, targetName) => {
-        lintTargets.add(targetName);
-    });
-    // Workspace does not use eslint?
-    if (lintTargets.size === 0) {
-        return;
-    }
-    const lintDefaults = (nxJson.targetDefaults['@nx/eslint:lint'] = {});
-    // All eslint targets have the same name
-    if (lintTargets.size === 1) {
-        const targetName = Array.from(lintTargets)[0];
-        if (nxJson.targetDefaults[targetName]) {
-            Object.assign(lintDefaults, nxJson.targetDefaults[targetName]);
-        }
-    }
-    lintDefaults.cache ??= true;
-    if (!lintDefaults.inputs) {
-        const eslintConfig = (0, eslint_file_1.findEslintFile)(tree);
-        lintDefaults.inputs = [
-            'default',
-            ...(eslintConfig ? [`{workspaceRoot}/${eslintConfig}`] : []),
-            '{workspaceRoot}/tools/eslint-rules/**/*',
-        ];
-    }
-    // Cleanup old target defaults
-    const projects = graph.nodes;
-    const projectMap = (0, devkit_1.getProjects)(tree);
-    for (const [targetDefaultKey, targetDefault] of Object.entries(nxJson.targetDefaults)) {
-        if (!isTargetDefaultUsed(targetDefault, nxJson.targetDefaults, projects, projectMap)) {
-            delete nxJson.targetDefaults[targetDefaultKey];
-        }
-    }
-    let addOutputs = false;
-    (0, executor_options_utils_1.forEachExecutorOptions)(tree, '@nx/eslint:lint', (options, proj, targetName, configuration) => {
-        const projConfig = projectMap.get(proj);
-        // modify lint file patterns
-        if (options.lintFilePatterns) {
-            const projectRootRegex = new RegExp(`^${projConfig.root}/`);
-            projConfig.targets[targetName].options.lintFilePatterns =
-                options.lintFilePatterns
-                    .filter((filePattern) => filePattern !== '{projectRoot}' &&
-                    filePattern !== projConfig.root)
-                    .map((filePattern) => filePattern.replace(projectRootRegex, '{projectRoot}/'));
-            // remove lintFilePatterns if empty
-            if (projConfig.targets[targetName].options.lintFilePatterns.length === 0) {
-                delete projConfig.targets[targetName].options.lintFilePatterns;
-            }
-        }
-        // remove inputs if they are not bringing any new inputs
-        if (projConfig.targets[targetName].inputs &&
-            projConfig.targets[targetName].inputs.every((i) => lintDefaults.inputs.includes(i))) {
-            delete projConfig.targets[targetName].inputs;
-        }
-        // remove obsolete eslint config definition, unless it's a custom one
-        const projectEslintConfig = (0, eslint_file_1.findEslintFile)(tree, projConfig.root);
-        if (options.eslintConfig === `${projConfig.root}/${projectEslintConfig}` ||
-            options.eslintConfig === `{projectRoot}/${projectEslintConfig}`) {
-            delete projConfig.targets[targetName].options.eslintConfig;
-        }
-        // remove options if empty
-        if (Object.keys(projConfig.targets[targetName]?.options ?? {}).length === 0) {
-            delete projConfig.targets[targetName].options;
-        }
-        // track output
-        if (options.outputFile) {
-            addOutputs = true;
-        }
-        if (projConfig.targets[targetName].outputs?.length === 1 &&
-            projConfig.targets[targetName].outputs[0] === '{options.outputFile}') {
-            delete projConfig.targets[targetName].outputs;
-        }
-        (0, devkit_1.updateProjectConfiguration)(tree, proj, projConfig);
-    });
-    if (addOutputs) {
-        lintDefaults.outputs = ['{options.outputFile}'];
-    }
-    (0, devkit_1.updateNxJson)(tree, nxJson);
-    await (0, devkit_1.formatFiles)(tree);
-}
-function isTargetDefaultUsed(targetDefault, targetDefaults, projects, projectMap) {
-    for (const p of Object.values(projects)) {
-        for (const targetName in p.data?.targets ?? {}) {
-            if ((0, project_configuration_utils_1.readTargetDefaultsForTarget)(targetName, targetDefaults, 
-            // It might seem like we should use the graph here too but we don't want to pass an executor which was processed in the graph
-            projectMap.get(p.name)?.targets?.[targetName]?.executor) === targetDefault) {
-                return true;
-            }
-        }
-    }
-    return false;
-}