@nx/eslint-plugin 17.2.6 → 17.2.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nx/eslint-plugin",
-  "version": "17.2.6",
+  "version": "17.2.7",
   "private": false,
   "description": "The eslint-plugin package is an ESLint plugin that contains a collection of recommended ESLint rule configurations which you can extend from in your own ESLint configs, as well as an Nx-specific lint rule called enforce-module-boundaries.",
   "repository": {
@@ -33,8 +33,8 @@
     }
   },
   "dependencies": {
-    "@nx/devkit": "17.2.6",
-    "@nx/js": "17.2.6",
+    "@nx/devkit": "17.2.7",
+    "@nx/js": "17.2.7",
     "@typescript-eslint/type-utils": "^6.9.1",
     "@typescript-eslint/utils": "^6.9.1",
     "chalk": "^4.1.0",
@@ -42,7 +42,7 @@
     "jsonc-eslint-parser": "^2.1.0",
     "semver": "7.5.3",
     "tslib": "^2.3.0",
-    "@nrwl/eslint-plugin-nx": "17.2.6"
+    "@nrwl/eslint-plugin-nx": "17.2.7"
   },
   "publishConfig": {
     "access": "public"

package/src/rules/enforce-module-boundaries.js

@@ -225,7 +225,7 @@ exports.default = utils_2.ESLintUtils.RuleCreator(() => ``)({
             if (sourceProject === targetProject) {
                 if (!allowCircularSelfDependency &&
                     !(0, fileutils_1.isRelativePath)(imp) &&
-                    !(0, runtime_lint_utils_1.isAngularSecondaryEntrypoint)(imp, sourceFilePath, sourceProject.data.root)) {
+                    !(0, runtime_lint_utils_1.belongsToDifferentNgEntryPoint)(imp, sourceFilePath, sourceProject.data.root)) {
                     context.report({
                         node,
                         messageId: 'noSelfCircularDependencies',

package/src/utils/runtime-lint-utils.d.ts

@@ -77,12 +77,9 @@ export declare function groupImports(importsToRemap: {
     importPath: string;
 }[]): string;
 /**
- * Checks if import points to a secondary entry point in Angular project
- * @param targetProjectLocator
- * @param importExpr
- * @returns
+ * Checks if source file belongs to a secondary entry point different than the import one
  */
-export declare function isAngularSecondaryEntrypoint(importExpr: string, filePath: string, projectRoot: string): boolean;
+export declare function belongsToDifferentNgEntryPoint(importExpr: string, filePath: string, projectRoot: string): boolean;
 /**
  * Returns true if the given project contains MFE config with "exposes:" section
  */

package/src/utils/runtime-lint-utils.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.appIsMFERemote = exports.isAngularSecondaryEntrypoint = exports.groupImports = exports.isTerminalRun = exports.hasBuildExecutor = exports.isDirectDependency = exports.hasBannedDependencies = exports.findTransitiveExternalDependencies = exports.hasBannedImport = exports.getSourceFilePath = exports.onlyLoadChildren = exports.findConstraintsFor = exports.findProjectUsingImport = exports.isAbsoluteImportIntoAnotherProject = exports.findProject = exports.getTargetProjectBasedOnRelativeImport = exports.isRelative = exports.matchImportWithWildcard = exports.findDependenciesWithTags = exports.isComboDepConstraint = exports.hasNoneOfTheseTags = exports.stringifyTags = void 0;
+exports.appIsMFERemote = exports.belongsToDifferentNgEntryPoint = exports.groupImports = exports.isTerminalRun = exports.hasBuildExecutor = exports.isDirectDependency = exports.hasBannedDependencies = exports.findTransitiveExternalDependencies = exports.hasBannedImport = exports.getSourceFilePath = exports.onlyLoadChildren = exports.findConstraintsFor = exports.findProjectUsingImport = exports.isAbsoluteImportIntoAnotherProject = exports.findProject = exports.getTargetProjectBasedOnRelativeImport = exports.isRelative = exports.matchImportWithWildcard = exports.findDependenciesWithTags = exports.isComboDepConstraint = exports.hasNoneOfTheseTags = exports.stringifyTags = void 0;
 const tslib_1 = require("tslib");
 const path = tslib_1.__importStar(require("path"));
 const path_1 = require("path");
@@ -311,17 +311,21 @@ function groupImports(importsToRemap) {
 }
 exports.groupImports = groupImports;
 /**
- * Checks if import points to a secondary entry point in Angular project
- * @param targetProjectLocator
- * @param importExpr
- * @returns
+ * Checks if source file belongs to a secondary entry point different than the import one
  */
-function isAngularSecondaryEntrypoint(importExpr, filePath, projectRoot) {
-    const resolvedModule = (0, internal_1.resolveModuleByImport)(importExpr, filePath, (0, path_1.join)(devkit_1.workspaceRoot, (0, js_1.getRootTsConfigFileName)()));
-    return (!!resolvedModule && fileIsSecondaryEntryPoint(resolvedModule, projectRoot));
+function belongsToDifferentNgEntryPoint(importExpr, filePath, projectRoot) {
+    const resolvedImportFile = (0, internal_1.resolveModuleByImport)(importExpr, filePath, // not strictly necessary, but speeds up resolution
+    (0, path_1.join)(devkit_1.workspaceRoot, (0, js_1.getRootTsConfigFileName)()));
+    if (!resolvedImportFile) {
+        return false;
+    }
+    const importEntryPoint = getAngularEntryPoint(resolvedImportFile, projectRoot);
+    const srcEntryPoint = getAngularEntryPoint(filePath, projectRoot);
+    // check if the entry point of import expression is different than the source file's entry point
+    return importEntryPoint !== srcEntryPoint;
 }
-exports.isAngularSecondaryEntrypoint = isAngularSecondaryEntrypoint;
-function fileIsSecondaryEntryPoint(file, projectRoot) {
+exports.belongsToDifferentNgEntryPoint = belongsToDifferentNgEntryPoint;
+function getAngularEntryPoint(file, projectRoot) {
     let parent = (0, devkit_1.joinPathFragments)(file, '../');
     while (parent !== `${projectRoot}/`) {
         // we need to find closest existing ng-package.json
@@ -330,11 +334,11 @@ function fileIsSecondaryEntryPoint(file, projectRoot) {
         if (ngPackageContent) {
             // https://github.com/ng-packagr/ng-packagr/blob/23c718d04eea85e015b4c261310b7bd0c39e5311/src/ng-package.schema.json#L54
             const entryFile = (0, devkit_1.parseJson)(ngPackageContent)?.lib?.entryFile;
-            return entryFile && file === (0, devkit_1.joinPathFragments)(parent, entryFile);
+            return (0, devkit_1.joinPathFragments)(parent, entryFile);
         }
         parent = (0, devkit_1.joinPathFragments)(parent, '../');
     }
-    return false;
+    return undefined;
 }
 /**
  * Returns true if the given project contains MFE config with "exposes:" section