@morriconeluca/eslint-plugin-faf 0.3.0 → 0.4.0

package/README.md

@@ -317,7 +317,7 @@ Applies **Peer Isolation**:
 
 - Prevents sibling files and folders from importing each other unless an explicit hierarchy is configured:
   - **Inside a Fragment**: Flow is governed by the `roles` array order.
-  - **Between folders**: Flow is governed by `localHorizontalHierarchies` or `globalHorizontalHierarchies`. If undefined, the linter falls back to mapped roles. Otherwise, peer isolation is strictly enforced.
+  - **Between folders**: Flow is governed exclusively by `localHorizontalHierarchies` or `globalHorizontalHierarchies`; a sibling pair without a matching entry is denied in both directions.
   - **Root Nodes**: Flow is governed exclusively by the defined index order in `rootFragments.rootNodes`. Root Nodes are exempt from the Role naming convention, so relationships between them never fall back to the `roles` scale: any relationship without a matching `rootNodes` entry is denied.
 
 ### 7. `faf/category-mutually-exclusive`

package/dist/main.cjs

@@ -39,7 +39,7 @@ module.exports = __toCommonJS(main_exports);
 // package.json
 var package_default = {
   name: "@morriconeluca/eslint-plugin-faf",
-  version: "0.3.0",
+  version: "0.4.0",
   description: "An ESLint plugin for FAF projects.",
   repository: {
     type: "git",
@@ -1220,24 +1220,6 @@ function resolveHorizontalHierarchy(lcaPath, subA, subB, config) {
 }
 
 // src/_rules/no-peer-dependency/no-peer-dependency.rule.ts
-function getDirectoryRole(dirPath, config) {
-  const folderName = import_path13.default.basename(dirPath);
-  if (folderName.startsWith("_")) {
-    return getRoleFromFolderName(folderName, config);
-  }
-  const contents = readDirCached(dirPath);
-  for (const file of contents.files) {
-    if (file.startsWith(folderName + ".")) {
-      const role = getFileRole(file, config);
-      if (role) return role;
-    }
-  }
-  return null;
-}
-function getRoleFromFolderName(name, config) {
-  const catConfig = config.categories?.find((c) => c.name === name);
-  return catConfig ? catConfig.role ?? null : null;
-}
 function getRootNodeIndex(relPath, rfConfig, lcaPath) {
   const relToLca = import_path13.default.relative(lcaPath, relPath).replace(/\\/g, "/");
   for (let i = 0; i < rfConfig.rootNodes.length; i++) {
@@ -1379,23 +1361,10 @@ var rule6 = {
           });
         }
       } else {
-        const roleA = getDirectoryRole(import_path13.default.posix.join(lca, subA), config);
-        const roleB = getDirectoryRole(import_path13.default.posix.join(lca, subB), config);
-        const idxA = roleA ? getRoleHierarchyIndex(roleA, config) : -1;
-        const idxB = roleB ? getRoleHierarchyIndex(roleB, config) : -1;
-        if (idxA !== -1 && idxB !== -1) {
-          if (idxB >= idxA) {
-            context.report({
-              message: `Horizontal hierarchy violation (fallback to roles): sibling directory "${subA}" (mapped to role "${roleA}", level ${idxA}) cannot import from "${subB}" (mapped to role "${roleB}", level ${idxB}) under parent "${lca}".`,
-              node
-            });
-          }
-        } else {
-          context.report({
-            message: `Peer separation violation: sibling directories "${subA}" and "${subB}" cannot import each other because no horizontal hierarchy is defined under parent "${lca}".`,
-            node
-          });
-        }
+        context.report({
+          message: `Peer separation violation: sibling directories "${subA}" and "${subB}" cannot import each other because no horizontal hierarchy is defined under parent "${lca}".`,
+          node
+        });
       }
     }
     return {