@nx/eslint 19.2.0-canary.20240530-316dcb9 → 19.2.0-canary.20240601-6f22300

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nx/eslint",
-  "version": "19.2.0-canary.20240530-316dcb9",
+  "version": "19.2.0-canary.20240601-6f22300",
   "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,12 +35,12 @@
     "eslint": "^8.0.0 || ^9.0.0"
   },
   "dependencies": {
-    "@nx/devkit": "19.2.0-canary.20240530-316dcb9",
-    "@nx/js": "19.2.0-canary.20240530-316dcb9",
+    "@nx/devkit": "19.2.0-canary.20240601-6f22300",
+    "@nx/js": "19.2.0-canary.20240601-6f22300",
     "semver": "^7.5.3",
     "tslib": "^2.3.0",
     "typescript": "~5.4.2",
-    "@nx/linter": "19.2.0-canary.20240530-316dcb9"
+    "@nx/linter": "19.2.0-canary.20240601-6f22300"
   },
   "peerDependenciesMeta": {
     "@zkochan/js-yaml": {

package/src/generators/convert-to-inferred/convert-to-inferred.js

@@ -8,8 +8,8 @@ const target_options_map_1 = require("./lib/target-options-map");
 const utils_1 = require("nx/src/tasks-runner/utils");
 async function convertToInferred(tree, options) {
     const projectGraph = await (0, devkit_1.createProjectGraphAsync)();
-    const migratedProjectsModern = await (0, executor_to_plugin_migrator_1.migrateExecutorToPlugin)(tree, projectGraph, '@nx/eslint:lint', '@nx/eslint/plugin', (targetName) => ({ targetName }), postTargetTransformer, plugin_1.createNodes, options.project);
-    const migratedProjectsLegacy = await (0, executor_to_plugin_migrator_1.migrateExecutorToPlugin)(tree, projectGraph, '@nrwl/linter:eslint', '@nx/eslint/plugin', (targetName) => ({ targetName }), postTargetTransformer, plugin_1.createNodes, options.project);
+    const migratedProjectsModern = await (0, executor_to_plugin_migrator_1.migrateExecutorToPluginV1)(tree, projectGraph, '@nx/eslint:lint', '@nx/eslint/plugin', (targetName) => ({ targetName }), postTargetTransformer, plugin_1.createNodes, options.project);
+    const migratedProjectsLegacy = await (0, executor_to_plugin_migrator_1.migrateExecutorToPluginV1)(tree, projectGraph, '@nrwl/linter:eslint', '@nx/eslint/plugin', (targetName) => ({ targetName }), postTargetTransformer, plugin_1.createNodes, options.project);
     const migratedProjects = migratedProjectsModern.size + migratedProjectsLegacy.size;
     if (migratedProjects === 0) {
         throw new Error('Could not find any targets to migrate.');
@@ -21,14 +21,14 @@ async function convertToInferred(tree, options) {
 exports.convertToInferred = convertToInferred;
 function postTargetTransformer(target, tree, projectDetails) {
     if (target.inputs) {
-        target.inputs = target.inputs.filter((input) => typeof input === 'string' &&
+        const inputs = target.inputs.filter((input) => typeof input === 'string' &&
             ![
                 'default',
                 '{workspaceRoot}/.eslintrc.json',
                 '{workspaceRoot}/.eslintignore',
                 '{workspaceRoot}/eslint.config.js',
             ].includes(input));
-        if (target.inputs.length === 0) {
+        if (inputs.length === 0) {
             delete target.inputs;
         }
     }

package/src/generators/init/init.js

@@ -48,7 +48,7 @@ async function initEsLint(tree, options) {
         '_eslint-lint',
     ];
     if (rootEslintFile && options.addPlugin && !hasPlugin) {
-        await (0, add_plugin_1.addPlugin)(tree, graph, '@nx/eslint/plugin', plugin_1.createNodes, {
+        await (0, add_plugin_1.addPluginV1)(tree, graph, '@nx/eslint/plugin', plugin_1.createNodes, {
             targetName: lintTargetNames,
         }, options.updatePackageScripts);
         return () => { };
@@ -58,7 +58,7 @@ async function initEsLint(tree, options) {
     }
     updateProductionFileset(tree);
     if (options.addPlugin) {
-        await (0, add_plugin_1.addPlugin)(tree, graph, '@nx/eslint/plugin', plugin_1.createNodes, {
+        await (0, add_plugin_1.addPluginV1)(tree, graph, '@nx/eslint/plugin', plugin_1.createNodes, {
             targetName: lintTargetNames,
         }, options.updatePackageScripts);
     }

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

@@ -16,6 +16,7 @@ interface LintProjectOptions {
      * @internal
      */
     addExplicitTargets?: boolean;
+    addPackageJsonDependencyChecks?: boolean;
 }
 export declare function lintProjectGenerator(tree: Tree, options: LintProjectOptions): Promise<GeneratorCallback>;
 export declare function lintProjectGeneratorInternal(tree: Tree, options: LintProjectOptions): Promise<GeneratorCallback>;

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

@@ -89,7 +89,7 @@ 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, projectConfig, options.setParserOptionsProject, options.rootProject);
+        createEsLintConfiguration(tree, options, projectConfig, options.setParserOptionsProject, options.rootProject);
     }
     // Buildable libs need source analysis enabled for linting `package.json`.
     if (isBuildableLibraryProject(projectConfig) &&
@@ -108,7 +108,7 @@ async function lintProjectGeneratorInternal(tree, options) {
     return (0, devkit_1.runTasksInSerial)(...tasks);
 }
 exports.lintProjectGeneratorInternal = lintProjectGeneratorInternal;
-function createEsLintConfiguration(tree, projectConfig, setParserOptionsProject, rootProject) {
+function createEsLintConfiguration(tree, options, projectConfig, setParserOptionsProject, rootProject) {
     // 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
@@ -151,7 +151,8 @@ function createEsLintConfiguration(tree, projectConfig, setParserOptionsProject,
             rules: {},
         },
     ];
-    if (isBuildableLibraryProject(projectConfig)) {
+    if (options.addPackageJsonDependencyChecks ||
+        isBuildableLibraryProject(projectConfig)) {
         overrides.push({
             files: ['*.json'],
             parser: 'jsonc-eslint-parser',

package/src/plugins/plugin.js

@@ -32,7 +32,7 @@ exports.createNodes = [
                 nestedEslintRootPatterns.push(`${eslintRootDir}/**/*`);
             }
         }
-        const projectFiles = (0, workspace_context_1.globWithWorkspaceContext)(context.workspaceRoot, [
+        const projectFiles = await (0, workspace_context_1.globWithWorkspaceContext)(context.workspaceRoot, [
             'project.json',
             'package.json',
             '**/project.json',
@@ -48,7 +48,7 @@ exports.createNodes = [
             // anything after is either a nested project or a sibling project, can be excluded
             const nestedProjectRootPatterns = excludePatterns.slice(index + 1);
             // Ignore project roots where the project does not contain any lintable files
-            const lintableFiles = (0, workspace_context_1.globWithWorkspaceContext)(context.workspaceRoot, [(0, node_path_1.join)(childProjectRoot, `**/*.{${options.extensions.join(',')}}`)], 
+            const lintableFiles = await (0, workspace_context_1.globWithWorkspaceContext)(context.workspaceRoot, [(0, node_path_1.join)(childProjectRoot, `**/*.{${options.extensions.join(',')}}`)], 
             // exclude nested eslint roots and nested project roots
             [...nestedEslintRootPatterns, ...nestedProjectRootPatterns]);
             const eslint = new ESLint({
@@ -76,11 +76,17 @@ function getProjectsUsingESLintConfig(configFilePath, childProjectRoots, eslintV
     ].find((f) => (0, node_fs_1.existsSync)((0, node_path_1.join)(context.workspaceRoot, f)));
     // Add a lint target for each child project without an eslint config, with the root level config as an input
     for (const projectRoot of childProjectRoots) {
-        // If there's no src folder, it's not a standalone project, do not add the target at all
-        const isStandaloneWorkspace = projectRoot === '.' &&
-            (0, node_fs_1.existsSync)((0, node_path_1.join)(context.workspaceRoot, projectRoot, 'src')) &&
-            (0, node_fs_1.existsSync)((0, node_path_1.join)(context.workspaceRoot, projectRoot, 'package.json'));
-        if (projectRoot === '.' && !isStandaloneWorkspace) {
+        let standaloneSrcPath;
+        if (projectRoot === '.' &&
+            (0, node_fs_1.existsSync)((0, node_path_1.join)(context.workspaceRoot, projectRoot, 'package.json'))) {
+            if ((0, node_fs_1.existsSync)((0, node_path_1.join)(context.workspaceRoot, projectRoot, 'src'))) {
+                standaloneSrcPath = 'src';
+            }
+            else if ((0, node_fs_1.existsSync)((0, node_path_1.join)(context.workspaceRoot, projectRoot, 'lib'))) {
+                standaloneSrcPath = 'lib';
+            }
+        }
+        if (projectRoot === '.' && !standaloneSrcPath) {
             continue;
         }
         const eslintConfigs = [configFilePath];
@@ -88,16 +94,16 @@ function getProjectsUsingESLintConfig(configFilePath, childProjectRoots, eslintV
             eslintConfigs.unshift(rootEslintConfig);
         }
         projects[projectRoot] = {
-            targets: buildEslintTargets(eslintConfigs, eslintVersion, projectRoot, context.workspaceRoot, options, isStandaloneWorkspace),
+            targets: buildEslintTargets(eslintConfigs, eslintVersion, projectRoot, context.workspaceRoot, options, standaloneSrcPath),
         };
     }
     return projects;
 }
-function buildEslintTargets(eslintConfigs, eslintVersion, projectRoot, workspaceRoot, options, isStandaloneWorkspace = false) {
+function buildEslintTargets(eslintConfigs, eslintVersion, projectRoot, workspaceRoot, options, standaloneSrcPath) {
     const isRootProject = projectRoot === '.';
     const targets = {};
     const targetConfig = {
-        command: `eslint ${isRootProject && isStandaloneWorkspace ? './src' : '.'}`,
+        command: `eslint ${isRootProject && standaloneSrcPath ? `./${standaloneSrcPath}` : '.'}`,
         cache: true,
         options: {
             cwd: projectRoot,