@nx/eslint 17.2.0-beta.6 → 17.2.0-beta.9

package/migrations.json

@@ -32,6 +32,11 @@
       "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"
     }
   },
   "packageJsonUpdates": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nx/eslint",
-  "version": "17.2.0-beta.6",
+  "version": "17.2.0-beta.9",
   "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,10 +35,10 @@
   },
   "dependencies": {
     "tslib": "^2.3.0",
-    "@nx/devkit": "17.2.0-beta.6",
-    "@nx/js": "17.2.0-beta.6",
+    "@nx/devkit": "17.2.0-beta.9",
+    "@nx/js": "17.2.0-beta.9",
     "typescript": "~5.2.2",
-    "@nx/linter": "17.2.0-beta.6"
+    "@nx/linter": "17.2.0-beta.9"
   },
   "peerDependenciesMeta": {
     "eslint": {

package/src/executors/lint/lint.impl.js

@@ -4,6 +4,7 @@ const devkit_1 = require("@nx/devkit");
 const fs_1 = require("fs");
 const path_1 = require("path");
 const eslint_utils_1 = require("./utility/eslint-utils");
+const utils_1 = require("nx/src/tasks-runner/utils");
 async function run(options, context) {
     // this is only used for the hasher
     delete options.hasTypeAwareRules;
@@ -58,8 +59,15 @@ async function run(options, context) {
         }
     }
     let lintResults = [];
+    const normalizedLintFilePatterns = normalizedOptions.lintFilePatterns.map((pattern) => {
+        return (0, utils_1.interpolate)(pattern, {
+            workspaceRoot: '',
+            projectRoot: context.projectsConfigurations.projects[context.projectName].root,
+            projectName: context.projectName,
+        });
+    });
     try {
-        lintResults = await eslint.lintFiles(normalizedOptions.lintFilePatterns);
+        lintResults = await eslint.lintFiles(normalizedLintFilePatterns);
     }
     catch (err) {
         if (err.message.includes('You must therefore provide a value for the "parserOptions.project" property for @typescript-eslint/parser')) {
@@ -81,7 +89,7 @@ Please see https://nx.dev/guides/eslint for full guidance on how to resolve this
         throw err;
     }
     if (lintResults.length === 0 && errorOnUnmatchedPattern) {
-        const ignoredPatterns = (await Promise.all(normalizedOptions.lintFilePatterns.map(async (pattern) => (await eslint.isPathIgnored(pattern)) ? pattern : null)))
+        const ignoredPatterns = (await Promise.all(normalizedLintFilePatterns.map(async (pattern) => (await eslint.isPathIgnored(pattern)) ? pattern : null)))
             .filter((pattern) => !!pattern)
             .map((pattern) => `- '${pattern}'`);
         if (ignoredPatterns.length) {

package/src/executors/lint/schema.json

@@ -17,7 +17,7 @@
     "lintFilePatterns": {
       "type": "array",
       "description": "One or more files/dirs/globs to pass directly to ESLint's `lintFiles()` method.",
-      "default": [],
+      "default": ["{projectRoot}"],
       "items": {
         "type": "string"
       }
@@ -144,6 +144,5 @@
       "default": true
     }
   },
-  "required": ["lintFilePatterns"],
   "examplesFile": "../../../docs/eslint-examples.md"
 }

package/src/generators/convert-to-flat-config/converters/json-converter.js

@@ -5,6 +5,7 @@ const devkit_1 = require("@nx/devkit");
 const ts = require("typescript");
 const ast_utils_1 = require("../../utils/flat-config/ast-utils");
 const eslint_file_1 = require("../../utils/eslint-file");
+const path_utils_1 = require("../../utils/flat-config/path-utils");
 /**
  * Converts an ESLint JSON config to a flat config.
  * Deletes the original file along with .eslintignore if it exists.
@@ -69,7 +70,7 @@ function convertEslintJsonToFlatConfig(tree, root, config, ignorePaths) {
                 override.parser) {
                 isFlatCompatNeeded = true;
             }
-            exportElements.push((0, ast_utils_1.generateFlatOverride)(override, root));
+            exportElements.push((0, ast_utils_1.generateFlatOverride)(override));
         });
     }
     if (config.ignorePatterns) {
@@ -78,7 +79,7 @@ function convertEslintJsonToFlatConfig(tree, root, config, ignorePaths) {
             : [config.ignorePatterns]).filter((pattern) => !['**/*', '!**/*', 'node_modules'].includes(pattern)); // these are useless in a flat config
         if (patterns.length > 0) {
             exportElements.push((0, ast_utils_1.generateAst)({
-                ignores: patterns.map((path) => (0, ast_utils_1.mapFilePath)(path, root)),
+                ignores: patterns.map((path) => (0, path_utils_1.mapFilePath)(path)),
             }));
         }
     }
@@ -88,7 +89,7 @@ function convertEslintJsonToFlatConfig(tree, root, config, ignorePaths) {
                 .read(ignorePath, 'utf-8')
                 .split('\n')
                 .filter((line) => line.length > 0 && line !== 'node_modules')
-                .map((path) => (0, ast_utils_1.mapFilePath)(path, root));
+                .map((path) => (0, path_utils_1.mapFilePath)(path));
             if (patterns.length > 0) {
                 exportElements.push((0, ast_utils_1.generateAst)({ ignores: patterns }));
             }
@@ -97,7 +98,7 @@ function convertEslintJsonToFlatConfig(tree, root, config, ignorePaths) {
     // create the node list and print it to new file
     const nodeList = (0, ast_utils_1.createNodeList)(importsMap, exportElements, isFlatCompatNeeded);
     return {
-        content: (0, ast_utils_1.stringifyNodeList)(nodeList, root),
+        content: (0, ast_utils_1.stringifyNodeList)(nodeList),
         addESLintRC: isFlatCompatNeeded,
         addESLintJS: isESLintJSNeeded,
     };

package/src/generators/init/global-eslint-config.js

@@ -80,18 +80,18 @@ const getGlobalEsLintConfiguration = (unitTestRunner, rootProject) => {
 exports.getGlobalEsLintConfiguration = getGlobalEsLintConfiguration;
 const getGlobalFlatEslintConfiguration = (unitTestRunner, rootProject) => {
     const nodeList = (0, ast_utils_1.createNodeList)(new Map(), [], true);
-    let content = (0, ast_utils_1.stringifyNodeList)(nodeList, '');
+    let content = (0, ast_utils_1.stringifyNodeList)(nodeList);
     content = (0, ast_utils_1.addImportToFlatConfig)(content, 'nxPlugin', '@nx/eslint-plugin');
     content = (0, ast_utils_1.addPluginsToExportsBlock)(content, [
         { name: '@nx', varName: 'nxPlugin', imp: '@nx/eslint-plugin' },
     ]);
     if (!rootProject) {
-        content = (0, ast_utils_1.addBlockToFlatConfigExport)(content, (0, ast_utils_1.generateFlatOverride)(moduleBoundariesOverride, ''));
+        content = (0, ast_utils_1.addBlockToFlatConfigExport)(content, (0, ast_utils_1.generateFlatOverride)(moduleBoundariesOverride));
     }
-    content = (0, ast_utils_1.addBlockToFlatConfigExport)(content, (0, ast_utils_1.generateFlatOverride)(exports.typeScriptOverride, ''));
-    content = (0, ast_utils_1.addBlockToFlatConfigExport)(content, (0, ast_utils_1.generateFlatOverride)(exports.javaScriptOverride, ''));
+    content = (0, ast_utils_1.addBlockToFlatConfigExport)(content, (0, ast_utils_1.generateFlatOverride)(exports.typeScriptOverride));
+    content = (0, ast_utils_1.addBlockToFlatConfigExport)(content, (0, ast_utils_1.generateFlatOverride)(exports.javaScriptOverride));
     if (unitTestRunner === 'jest') {
-        content = (0, ast_utils_1.addBlockToFlatConfigExport)(content, (0, ast_utils_1.generateFlatOverride)(jestOverride, ''));
+        content = (0, ast_utils_1.addBlockToFlatConfigExport)(content, (0, ast_utils_1.generateFlatOverride)(jestOverride));
     }
     return content;
 };

package/src/generators/init/init-migration.js

@@ -23,7 +23,7 @@ function migrateConfigToMonorepoStyle(projects, tree, unitTestRunner) {
     projects.forEach((project) => {
         const lintTarget = findLintTarget(project);
         if (lintTarget) {
-            const eslintFile = lintTarget.options.eslintConfig || (0, eslint_file_1.findEslintFile)(tree, project.root);
+            const eslintFile = lintTarget.options?.eslintConfig || (0, eslint_file_1.findEslintFile)(tree, project.root);
             if (eslintFile) {
                 const projectEslintPath = (0, devkit_1.joinPathFragments)(project.root, eslintFile);
                 migrateEslintFile(projectEslintPath, tree);

package/src/generators/lint-project/lint-project.d.ts

@@ -11,6 +11,5 @@ interface LintProjectOptions {
     unitTestRunner?: string;
     rootProject?: boolean;
 }
-export declare function mapLintPattern(projectRoot: string, extension: string, rootProject?: boolean): string;
 export declare function lintProjectGenerator(tree: Tree, options: LintProjectOptions): Promise<import("@nx/devkit").GeneratorCallback>;
 export {};

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

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.lintProjectGenerator = exports.mapLintPattern = void 0;
+exports.lintProjectGenerator = void 0;
 const devkit_1 = require("@nx/devkit");
 const eslint_file_1 = require("../utils/eslint-file");
 const path_1 = require("path");
@@ -9,15 +9,6 @@ const init_migration_1 = require("../init/init-migration");
 const project_configuration_1 = require("nx/src/generators/utils/project-configuration");
 const flat_config_1 = require("../../utils/flat-config");
 const ast_utils_1 = require("../utils/flat-config/ast-utils");
-function mapLintPattern(projectRoot, extension, rootProject) {
-    if (rootProject && (projectRoot === '.' || projectRoot === '')) {
-        return `${projectRoot}/src/**/*.${extension}`;
-    }
-    else {
-        return `${projectRoot}/**/*.${extension}`;
-    }
-}
-exports.mapLintPattern = mapLintPattern;
 async function lintProjectGenerator(tree, options) {
     const installTask = (0, init_1.lintInitGenerator)(tree, {
         linter: options.linter,
@@ -26,17 +17,24 @@ async function lintProjectGenerator(tree, options) {
         rootProject: options.rootProject,
     });
     const projectConfig = (0, devkit_1.readProjectConfiguration)(tree, options.project);
-    const lintFilePatterns = options.eslintFilePatterns ?? [];
-    if (isBuildableLibraryProject(projectConfig)) {
-        lintFilePatterns.push(`${projectConfig.root}/package.json`);
-    }
     projectConfig.targets['lint'] = {
         executor: '@nx/eslint:lint',
         outputs: ['{options.outputFile}'],
-        options: {
-            lintFilePatterns: lintFilePatterns,
-        },
     };
+    let lintFilePatterns = options.eslintFilePatterns;
+    if (!lintFilePatterns && options.rootProject && projectConfig.root === '.') {
+        lintFilePatterns = ['./src'];
+    }
+    if (lintFilePatterns && lintFilePatterns.length) {
+        if (isBuildableLibraryProject(projectConfig) &&
+            !lintFilePatterns.includes('{projectRoot}')) {
+            lintFilePatterns.push(`{projectRoot}/package.json`);
+        }
+        // only add lintFilePatterns if they are explicitly defined
+        projectConfig.targets['lint'].options = {
+            lintFilePatterns,
+        };
+    }
     // we are adding new project which is not the root project or
     // companion e2e app so we should check if migration to
     // monorepo style is needed
@@ -138,10 +136,10 @@ function createEsLintConfiguration(tree, projectConfig, setParserOptionsProject,
             nodes.push((0, ast_utils_1.generateSpreadElement)('baseConfig'));
         }
         overrides.forEach((override) => {
-            nodes.push((0, ast_utils_1.generateFlatOverride)(override, projectConfig.root));
+            nodes.push((0, ast_utils_1.generateFlatOverride)(override));
         });
         const nodeList = (0, ast_utils_1.createNodeList)(importMap, nodes, isCompatNeeded);
-        const content = (0, ast_utils_1.stringifyNodeList)(nodeList, projectConfig.root);
+        const content = (0, ast_utils_1.stringifyNodeList)(nodeList);
         tree.write((0, path_1.join)(projectConfig.root, 'eslint.config.js'), content);
     }
     else {

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

@@ -4,6 +4,7 @@ exports.getPluginImport = exports.addIgnoresToLintConfig = exports.addPluginsToL
 const devkit_1 = require("@nx/devkit");
 const flat_config_1 = require("../../utils/flat-config");
 const ast_utils_1 = require("./flat-config/ast-utils");
+const path_utils_1 = require("./flat-config/path-utils");
 exports.eslintConfigFileWhitelist = [
     '.eslintrc',
     '.eslintrc.js',
@@ -112,7 +113,7 @@ function addOverrideToLintConfig(tree, root, override, options = {
     const isBase = options.checkBaseConfig && findEslintFile(tree, root).includes('.base');
     if ((0, flat_config_1.useFlatConfig)(tree)) {
         const fileName = (0, devkit_1.joinPathFragments)(root, isBase ? exports.baseEsLintFlatConfigFile : 'eslint.config.js');
-        const flatOverride = (0, ast_utils_1.generateFlatOverride)(override, root);
+        const flatOverride = (0, ast_utils_1.generateFlatOverride)(override);
         let content = tree.read(fileName, 'utf8');
         // we will be using compat here so we need to make sure it's added
         if (overrideNeedsCompat(override)) {
@@ -123,7 +124,7 @@ function addOverrideToLintConfig(tree, root, override, options = {
     else {
         const fileName = (0, devkit_1.joinPathFragments)(root, isBase ? exports.baseEsLintConfigFile : '.eslintrc.json');
         (0, devkit_1.updateJson)(tree, fileName, (json) => {
-            json.overrides ?? [];
+            json.overrides ??= [];
             if (options.insertAtTheEnd) {
                 json.overrides.push(override);
             }
@@ -183,7 +184,7 @@ function replaceOverridesInLintConfig(tree, root, overrides) {
         }
         content = (0, ast_utils_1.removeOverridesFromLintConfig)(content);
         overrides.forEach((override) => {
-            const flatOverride = (0, ast_utils_1.generateFlatOverride)(override, root);
+            const flatOverride = (0, ast_utils_1.generateFlatOverride)(override);
             (0, ast_utils_1.addBlockToFlatConfigExport)(content, flatOverride);
         });
         tree.write(fileName, content);
@@ -247,7 +248,7 @@ function addIgnoresToLintConfig(tree, root, ignorePatterns) {
     if ((0, flat_config_1.useFlatConfig)(tree)) {
         const fileName = (0, devkit_1.joinPathFragments)(root, 'eslint.config.js');
         const block = (0, ast_utils_1.generateAst)({
-            ignores: ignorePatterns.map((path) => (0, ast_utils_1.mapFilePath)(path, root)),
+            ignores: ignorePatterns.map((path) => (0, path_utils_1.mapFilePath)(path)),
         });
         tree.write(fileName, (0, ast_utils_1.addBlockToFlatConfigExport)(tree.read(fileName, 'utf8'), block));
     }

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

@@ -44,7 +44,7 @@ export declare function generatePluginExtendsElement(plugins: string[]): ts.Spre
 /**
  * Stringifies TS nodes to file content string
  */
-export declare function stringifyNodeList(nodes: ts.NodeArray<ts.VariableStatement | ts.Identifier | ts.ExpressionStatement | ts.SourceFile>, root: string): string;
+export declare function stringifyNodeList(nodes: ts.NodeArray<ts.VariableStatement | ts.Identifier | ts.ExpressionStatement | ts.SourceFile>): string;
 /**
  * generates AST require statement
  */
@@ -52,9 +52,8 @@ export declare function generateRequire(variableName: string | ts.ObjectBindingP
 /**
  * Generates AST object or spread element based on JSON override object
  */
-export declare function generateFlatOverride(override: Linter.ConfigOverride<Linter.RulesRecord>, root: string): ts.ObjectLiteralExpression | ts.SpreadElement;
-export declare function mapFilePaths(override: Linter.ConfigOverride<Linter.RulesRecord>, root: string): void;
-export declare function mapFilePath(filePath: string, root: string): string;
+export declare function generateFlatOverride(override: Linter.ConfigOverride<Linter.RulesRecord>): ts.ObjectLiteralExpression | ts.SpreadElement;
+export declare function mapFilePaths(override: Linter.ConfigOverride<Linter.RulesRecord>): void;
 /**
  * Generates an AST from a JSON-type input
  */

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

@@ -1,8 +1,9 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.generateAst = exports.mapFilePath = exports.mapFilePaths = exports.generateFlatOverride = exports.generateRequire = exports.stringifyNodeList = exports.generatePluginExtendsElement = exports.generateSpreadElement = exports.createNodeList = exports.addCompatToFlatConfig = exports.addPluginsToExportsBlock = exports.removeCompatExtends = exports.removePlugin = exports.addBlockToFlatConfigExport = exports.addImportToFlatConfig = exports.replaceOverride = exports.hasOverride = exports.removeOverridesFromLintConfig = void 0;
+exports.generateAst = exports.mapFilePaths = exports.generateFlatOverride = exports.generateRequire = exports.stringifyNodeList = exports.generatePluginExtendsElement = exports.generateSpreadElement = exports.createNodeList = exports.addCompatToFlatConfig = exports.addPluginsToExportsBlock = exports.removeCompatExtends = exports.removePlugin = exports.addBlockToFlatConfigExport = exports.addImportToFlatConfig = exports.replaceOverride = exports.hasOverride = exports.removeOverridesFromLintConfig = void 0;
 const devkit_1 = require("@nx/devkit");
 const ts = require("typescript");
+const path_utils_1 = require("./path-utils");
 /**
  * Remove all overrides from the config file
  */
@@ -63,7 +64,7 @@ function hasOverride(content, lookup) {
                 // strip any spread elements
                 objSource = fullNodeText.replace(/\s*\.\.\.[a-zA-Z0-9_]+,?\n?/, '');
             }
-            const data = JSON.parse(objSource
+            const data = (0, devkit_1.parseJson)(objSource
                 // ensure property names have double quotes so that JSON.parse works
                 .replace(/'/g, '"')
                 .replace(/\s([a-zA-Z0-9_]+)\s*:/g, ' "$1": '));
@@ -77,7 +78,7 @@ function hasOverride(content, lookup) {
 exports.hasOverride = hasOverride;
 const STRIP_SPREAD_ELEMENTS = /\s*\.\.\.[a-zA-Z0-9_]+,?\n?/g;
 function parseTextToJson(text) {
-    return JSON.parse(text
+    return (0, devkit_1.parseJson)(text
         // ensure property names have double quotes so that JSON.parse works
         .replace(/'/g, '"')
         .replace(/\s([a-zA-Z0-9_]+)\s*:/g, ' "$1": '));
@@ -118,7 +119,7 @@ function replaceOverride(content, root, lookup, update) {
                     length: end - start,
                 });
                 const updatedData = update(data);
-                mapFilePaths(updatedData, root);
+                mapFilePaths(updatedData);
                 changes.push({
                     type: devkit_1.ChangeType.Insert,
                     index: start,
@@ -470,9 +471,9 @@ exports.generatePluginExtendsElement = generatePluginExtendsElement;
 /**
  * Stringifies TS nodes to file content string
  */
-function stringifyNodeList(nodes, root) {
+function stringifyNodeList(nodes) {
     const printer = ts.createPrinter({ newLine: ts.NewLineKind.LineFeed });
-    const resultFile = ts.createSourceFile((0, devkit_1.joinPathFragments)(root, ''), '', ts.ScriptTarget.Latest, true, ts.ScriptKind.JS);
+    const resultFile = ts.createSourceFile('', '', ts.ScriptTarget.Latest, true, ts.ScriptKind.JS);
     return (printer
         .printList(ts.ListFormat.MultiLine, nodes, resultFile)
         // add new line before compat initialization
@@ -493,8 +494,8 @@ exports.generateRequire = generateRequire;
 /**
  * Generates AST object or spread element based on JSON override object
  */
-function generateFlatOverride(override, root) {
-    mapFilePaths(override, root);
+function generateFlatOverride(override) {
+    mapFilePaths(override);
     if (!override.env &&
         !override.extends &&
         !override.plugins &&
@@ -515,41 +516,21 @@ function generateFlatOverride(override, root) {
     ]));
 }
 exports.generateFlatOverride = generateFlatOverride;
-function mapFilePaths(override, root) {
+function mapFilePaths(override) {
     if (override.files) {
         override.files = Array.isArray(override.files)
             ? override.files
             : [override.files];
-        override.files = override.files.map((file) => mapFilePath(file, root));
+        override.files = override.files.map((file) => (0, path_utils_1.mapFilePath)(file));
     }
     if (override.excludedFiles) {
         override.excludedFiles = Array.isArray(override.excludedFiles)
             ? override.excludedFiles
             : [override.excludedFiles];
-        override.excludedFiles = override.excludedFiles.map((file) => mapFilePath(file, root));
+        override.excludedFiles = override.excludedFiles.map((file) => (0, path_utils_1.mapFilePath)(file));
     }
 }
 exports.mapFilePaths = mapFilePaths;
-function mapFilePath(filePath, root) {
-    if (filePath.startsWith('!')) {
-        const fileWithoutBang = filePath.slice(1);
-        if (fileWithoutBang.startsWith('*.')) {
-            return `!${(0, devkit_1.joinPathFragments)(root, '**', fileWithoutBang)}`;
-        }
-        else if (!fileWithoutBang.startsWith(root)) {
-            return `!${(0, devkit_1.joinPathFragments)(root, fileWithoutBang)}`;
-        }
-        return filePath;
-    }
-    if (filePath.startsWith('*.')) {
-        return (0, devkit_1.joinPathFragments)(root, '**', filePath);
-    }
-    else if (!filePath.startsWith(root)) {
-        return (0, devkit_1.joinPathFragments)(root, filePath);
-    }
-    return filePath;
-}
-exports.mapFilePath = mapFilePath;
 function addTSObjectProperty(elements, key, value) {
     if (value) {
         elements.push(ts.factory.createPropertyAssignment(key, generateAst(value)));

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

@@ -1,2 +1,3 @@
 import type { Linter } from 'eslint';
-export declare function updateFiles(override: Linter.ConfigOverride<Linter.RulesRecord>, root: string): Linter.ConfigOverride<Linter.RulesRecord>;
+export declare function updateFiles(override: Linter.ConfigOverride<Linter.RulesRecord>): Linter.ConfigOverride<Linter.RulesRecord>;
+export declare function mapFilePath(filePath: string): string;

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

@@ -1,31 +1,28 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.updateFiles = void 0;
+exports.mapFilePath = exports.updateFiles = void 0;
 const devkit_1 = require("@nx/devkit");
-function updateFiles(override, root) {
+function updateFiles(override) {
     if (override.files) {
         override.files = Array.isArray(override.files)
             ? override.files
             : [override.files];
-        override.files = override.files.map((file) => mapFilePath(file, root));
+        override.files = override.files.map((file) => mapFilePath(file));
     }
     return override;
 }
 exports.updateFiles = updateFiles;
-function mapFilePath(filePath, root) {
+function mapFilePath(filePath) {
     if (filePath.startsWith('!')) {
         const fileWithoutBang = filePath.slice(1);
         if (fileWithoutBang.startsWith('*.')) {
-            return `!${(0, devkit_1.joinPathFragments)(root, '**', fileWithoutBang)}`;
-        }
-        else {
-            return `!${(0, devkit_1.joinPathFragments)(root, fileWithoutBang)}`;
+            return `!${(0, devkit_1.joinPathFragments)('**', fileWithoutBang)}`;
         }
+        return filePath;
     }
     if (filePath.startsWith('*.')) {
-        return (0, devkit_1.joinPathFragments)(root, '**', filePath);
-    }
-    else {
-        return (0, devkit_1.joinPathFragments)(root, filePath);
+        return (0, devkit_1.joinPathFragments)('**', filePath);
     }
+    return filePath;
 }
+exports.mapFilePath = mapFilePath;

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

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

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

@@ -0,0 +1,46 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+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);
+}
+exports.default = update;
+function getLintRoot({ root, sourceRoot }) {
+    if (root === '' || root === '.') {
+        return sourceRoot || './src';
+    }
+    return root;
+}