eslint-plugin-stratified-design 0.5.2 → 0.6.1-beta

package/lib/helpers/lowerLevelImports/1 layer.js

@@ -65,6 +65,20 @@ const createModulePath = (cwd, fileDir, aliases) => {
   };
 };
 
+/**
+ * @param {*} options
+ * @param {string} fileDor
+ * @param {string} modulePath
+ */
+const isFileIndexOfModule = (options, fileDir, filePath) => (modulePath) => {
+  if (!options.isIndexHighest || parsePath(filePath).name !== "index") {
+    return false;
+  }
+  const segments = toSegments(toRelative(fileDir, modulePath));
+  if (segments.length === 2 && segments[0] === ".") return true;
+  return false;
+};
+
 /**
  * Report error about using `options.useLevelNumber`
  * @param {import('eslint').Rule.RuleContext} context
@@ -222,6 +236,7 @@ module.exports = {
   createRootDir,
   parseFileSource,
   createModulePath,
+  isFileIndexOfModule,
   reportHasProperLevelNumber,
   reportInSubDirOfFileDir,
   reportHasProperLevel,

package/lib/rules/lower-level-imports.js

@@ -21,6 +21,7 @@ const {
   parseFileSource,
   createRootDir,
 } = require("../helpers/lowerLevelImports/");
+const { isFileIndexOfModule } = require("../helpers/lowerLevelImports/1 layer");
 
 //------------------------------------------------------------------------------
 // Rule Definition
@@ -89,6 +90,9 @@ module.exports = {
             useLevelNumber: {
               type: "boolean",
             },
+            isIndexHighest: {
+              type: "boolean",
+            },
           },
           additionalProperties: false,
         },
@@ -151,17 +155,31 @@ module.exports = {
       filePath
     );
 
-    return {
-      ImportDeclaration(node) {
-        if (isExcludedFile) return;
+    const isFileIndex = isFileIndexOfModule(options, fileDir, filePath);
+
+    const report = (node) => {
+      if (isExcludedFile) return;
+
+      const modulePath = createModulePath(node.source.value);
 
-        const modulePath = createModulePath(node.source.value);
+      if (isFileIndex(modulePath)) return;
 
-        if (reportHasProperLevelNumber(node, modulePath) === FINISHED) return;
+      if (reportHasProperLevelNumber(node, modulePath) === FINISHED) return;
 
-        if (reportInSubDirOfFileDir(node, modulePath) === FINISHED) return;
+      if (reportInSubDirOfFileDir(node, modulePath) === FINISHED) return;
 
-        reportHasProperLevel(node, modulePath);
+      reportHasProperLevel(node, modulePath);
+    };
+
+    return {
+      ImportDeclaration(node) {
+        report(node);
+      },
+      ExportNamedDeclaration(node) {
+        if (node.source) report(node);
+      },
+      ExportAllDeclaration(node) {
+        if (node.source) report(node);
       },
     };
   },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-stratified-design",
-  "version": "0.5.2",
+  "version": "0.6.1-beta",
   "description": "ESlint rules for stratified design",
   "keywords": [
     "eslint",

package/tests/lib/rules/lower-level-imports.js

@@ -37,6 +37,21 @@ ruleTester.run("lower-level-imports", rule, {
       filename: "./src/otherLayerA.js",
       options: [{ structure, root: "./src" }],
     },
+    {
+      code: "export { func } from './otherLayerB/otherSubLayer'",
+      filename: "./src/otherLayerA.js",
+      options: [{ structure, root: "./src" }],
+    },
+    {
+      code: "export * from './otherLayerB/otherSubLayer'",
+      filename: "./src/otherLayerA.js",
+      options: [{ structure, root: "./src" }],
+    },
+    {
+      code: "const func = () => null; export { func }; ",
+      filename: "./src/otherLayerA.js",
+      options: [{ structure, root: "./src" }],
+    },
     {
       code: "import { func } from '../layer1/subLayer2'",
       filename: "./src/layer1/subLayer1.js",
@@ -199,6 +214,18 @@ ruleTester.run("lower-level-imports", rule, {
         },
       ],
     },
+    {
+      code: "import { func } from '@/other/file.js'",
+      filename: "./src/other/index.js",
+      options: [
+        {
+          structure,
+          root: "./src",
+          aliases: { "@/": "./src/" },
+          isIndexHighest: true,
+        },
+      ],
+    },
   ],
   invalid: [
     {
@@ -212,6 +239,28 @@ ruleTester.run("lower-level-imports", rule, {
         },
       ],
     },
+    {
+      code: "export { func } from './otherLayerB'",
+      filename: "./src/otherLayerA.js",
+      options: [{ structure, root: "./src" }],
+      errors: [
+        {
+          messageId: "not-registered:file",
+          data: { module: "./otherLayerB", file: "./otherLayerA" },
+        },
+      ],
+    },
+    {
+      code: "export * from './otherLayerB'",
+      filename: "./src/otherLayerA.js",
+      options: [{ structure, root: "./src" }],
+      errors: [
+        {
+          messageId: "not-registered:file",
+          data: { module: "./otherLayerB", file: "./otherLayerA" },
+        },
+      ],
+    },
     {
       code: "import { func } from 'node-module'",
       filename: "./src/layer3/subLayer1.js",
@@ -350,5 +399,22 @@ ruleTester.run("lower-level-imports", rule, {
         },
       ],
     },
+    {
+      code: "import { func } from '@/other/file.js'",
+      filename: "./src/other/index.js",
+      options: [
+        {
+          structure,
+          root: "./src",
+          aliases: { "@/": "./src/" },
+        },
+      ],
+      errors: [
+        {
+          messageId: "not-registered:file",
+          data: { module: "./other/file.js", file: "./other/index" },
+        },
+      ],
+    },
   ],
 });