eslint-plugin-barrel-rules 1.3.0 → 1.4.0

package/README.ko.md

@@ -3,7 +3,7 @@
 # **Advanced Barrel Pattern Enforcement for JavaScript/TypeScript Projects**
 
 <div align="center">
-  <img src="https://img.shields.io/badge/version-1.3.0-blue.svg" alt="Version"/>
+  <img src="https://img.shields.io/badge/version-1.4.0-blue.svg" alt="Version"/>
   <img src="https://img.shields.io/badge/License-MIT-yellow.svg" alt="License"/>
   <img src="https://img.shields.io/badge/PRs-welcome-brightgreen.svg" alt="PRs Welcome"/>
 </div>
@@ -28,7 +28,11 @@ JavaScript/TypeScript 프로젝트에서 Barrel Pattern(배럴 패턴)을 강제
 내부 파일을 직접 import하는 것을 차단하여  
 **모듈화, 추상화, 유지보수성, 확장성**을 극대화합니다.
 
-> 💡 Tip:  
+> 💡 Tip:
+>
+> > 이 플러그인은 `node_modules`(외부 패키지)의 import는 제한하거나 검사하지 않습니다.  
+> > 모든 규칙은 오직 프로젝트 내부(로컬 소스 파일) 경로의 import에만 적용됩니다.
+>
 > 코드 품질을 더욱 강화하고 싶다면, 이 플러그인과 함께 [eslint-plugin-import](https://github.com/import-js/eslint-plugin-import)의 `no-cycle` 룰을 사용하는 것을 추천합니다.  
 > 이를 통해 프로젝트 내의 순환 참조(Import Cycle)도 효과적으로 감지하고 방지할 수 있습니다.
 
@@ -222,7 +226,7 @@ export default tseslint.config([
             },
           ],
           baseDir: __dirname,
-          globalAllowPaths: ["src/shares/*", "node_modules/*"],
+          globalAllowPaths: ["src/shares/*"],
         },
       ],
 
@@ -278,6 +282,7 @@ import { SharedUtil } from "@shared/utils"; // allowedImportPaths에 "src/shared
 - **잘못된 경로 설정 검증 기능** (OK)
 - **와일드카드 import/export 제한 규칙** (OK)
 - **지정한 Barrel 경로 격리** (OK)
+- **빈 디렉토리 지원** (예: 'src/shares/\*'가 설정되어 있어도 shares 디렉토리가 비어있어도 됨) (OK)
 
 ---
 

package/README.md

@@ -3,7 +3,7 @@
 # **Advanced Barrel Pattern Enforcement for JavaScript/TypeScript Projects**
 
 <div align="center">
-  <img src="https://img.shields.io/badge/version-1.3.0-blue.svg" alt="Version"/>
+  <img src="https://img.shields.io/badge/version-1.4.0-blue.svg" alt="Version"/>
   <img src="https://img.shields.io/badge/License-MIT-yellow.svg" alt="License"/>
   <img src="https://img.shields.io/badge/PRs-welcome-brightgreen.svg" alt="PRs Welcome"/>
 </div>
@@ -32,6 +32,9 @@ Direct imports from internal files are blocked, maximizing
 **modularity, abstraction, maintainability, and scalability**.
 
 > 💡 Tip:  
+> This plugin does not restrict or track imports from `node_modules` (external packages).  
+> The rules only apply to imports of internal (local source file) paths within your project.
+>
 > For even stronger code quality, we recommend using the `no-cycle` rule from [eslint-plugin-import](https://github.com/import-js/eslint-plugin-import) together with this plugin.  
 > This allows you to detect and prevent circular dependencies (import cycles) in your project.
 
@@ -228,7 +231,7 @@ export default tseslint.config([
             },
           ],
           baseDir: __dirname,
-          globalAllowPaths: ["src/shares/*", "node_modules/*"],
+          globalAllowPaths: ["src/shares/*"],
         },
       ],
 
@@ -288,6 +291,7 @@ import { SharedUtil } from "@shared/utils"; // if "src/shared/*" is in allowedPa
 - **Wrong Path Setup Validator** (OK)
 - **Wildcard Import/Export Protection Rule** (OK)
 - **Isolation Barrel Module** (OK)
+- **Empty Directory Support** (e.g., 'src/shares/\*' can be configured even if the shares directory is empty) (OK)
 
 ---
 

package/dist/index.cjs

@@ -48,11 +48,6 @@ var Glob = class {
       onlyDirectories: true,
       absolute: true
     });
-    if (globResult.length === 0) {
-      throw new Error(
-        `[Glob] In baseDir: ${baseDir}, path: ${path4}, any directory was not found`
-      );
-    }
     return globResult;
   }
 };
@@ -123,13 +118,20 @@ var BARREL_ENTRY_POINT_FILE_NAMES = [
 ];
 var RESOLVE_EXTENSIONS = [
   ".ts",
-  ".tsx",
   ".js",
+  ".tsx",
   ".jsx",
   ".json",
+  ".d.js",
   ".d.ts",
   ".mjs",
-  ".cjs"
+  ".cjs",
+  ".mts",
+  ".cts",
+  ".d.mjs",
+  ".d.cjs",
+  ".d.mts",
+  ".d.cts"
 ];
 var enforceBarrelPattern = {
   meta: {
@@ -183,63 +185,78 @@ var enforceBarrelPattern = {
     const absoluteTargetPaths = option.paths.flatMap((_path) => {
       return Glob.resolvePath(_path, baseDir);
     });
-    return {
-      //check only import declaration(ESM)
-      ImportDeclaration(node) {
-        const rawImportPath = node.source.value;
-        const absoluteCurrentFilePath = context.getFilename();
-        let absoluteImportPath = null;
-        try {
-          const aliasResult = Alias.resolvePath(
-            rawImportPath,
-            import_path2.default.dirname(absoluteCurrentFilePath)
-          );
-          if (aliasResult.type === "success") {
-            absoluteImportPath = aliasResult.absolutePath;
-          } else {
-            absoluteImportPath = import_resolve.default.sync(rawImportPath, {
-              basedir: import_path2.default.dirname(absoluteCurrentFilePath),
-              extensions: RESOLVE_EXTENSIONS
-            });
+    function checker(node) {
+      if (!node.source) {
+        return;
+      }
+      const rawImportPath = node.source.value;
+      const absoluteCurrentFilePath = context.getFilename();
+      let absoluteImportPath = null;
+      try {
+        const aliasResult = Alias.resolvePath(
+          rawImportPath,
+          import_path2.default.dirname(absoluteCurrentFilePath)
+        );
+        if (aliasResult.type === "success") {
+          absoluteImportPath = aliasResult.absolutePath;
+        } else {
+          if (!rawImportPath.startsWith(".") && !rawImportPath.startsWith("/") || rawImportPath.includes("/node_modules/")) {
+            return;
           }
-        } catch (e) {
-          context.report({
-            node,
-            messageId: "TransformedAliasResolveFailed"
+          absoluteImportPath = import_resolve.default.sync(rawImportPath, {
+            basedir: import_path2.default.dirname(absoluteCurrentFilePath),
+            extensions: RESOLVE_EXTENSIONS
           });
-          return;
         }
-        {
-          let matchedLatestTargetPath = null;
-          const invalidDirectedImport = absoluteTargetPaths.some(
-            (absoluteTargetPath) => {
-              const targetPathEntryPoints = BARREL_ENTRY_POINT_FILE_NAMES.map(
-                (entry) => import_path2.default.resolve(absoluteTargetPath, entry)
-              );
-              const closedTargetPath = absoluteTargetPath + "/";
-              const targetPathEntryPointed = targetPathEntryPoints.includes(absoluteImportPath);
-              const importedEnforceBarrelFile = absoluteImportPath.startsWith(closedTargetPath);
-              const currentFileInEnforceBarrel = absoluteCurrentFilePath.startsWith(closedTargetPath);
-              const importedOutsideOfTargetPath = !currentFileInEnforceBarrel && importedEnforceBarrelFile;
-              const invalidImported = !targetPathEntryPointed && importedOutsideOfTargetPath;
-              if (invalidImported) {
-                matchedLatestTargetPath = absoluteTargetPath;
-              }
-              return invalidImported;
+      } catch (e) {
+        context.report({
+          node,
+          messageId: "TransformedAliasResolveFailed"
+        });
+        return;
+      }
+      {
+        let matchedLatestTargetPath = null;
+        const invalidDirectedImport = absoluteTargetPaths.some(
+          (absoluteTargetPath) => {
+            const targetPathEntryPoints = BARREL_ENTRY_POINT_FILE_NAMES.map(
+              (entry) => import_path2.default.resolve(absoluteTargetPath, entry)
+            );
+            const closedTargetPath = absoluteTargetPath + "/";
+            const targetPathEntryPointed = targetPathEntryPoints.includes(absoluteImportPath);
+            const importedEnforceBarrelFile = absoluteImportPath.startsWith(closedTargetPath);
+            const currentFileInEnforceBarrel = absoluteCurrentFilePath.startsWith(closedTargetPath);
+            const importedOutsideOfTargetPath = !currentFileInEnforceBarrel && importedEnforceBarrelFile;
+            const invalidImported = !targetPathEntryPointed && importedOutsideOfTargetPath;
+            if (invalidImported) {
+              matchedLatestTargetPath = absoluteTargetPath;
             }
-          );
-          if (invalidDirectedImport) {
-            context.report({
-              node,
-              messageId: "DirectImportDisallowed",
-              data: {
-                rawImportPath,
-                matchedTargetPath: matchedLatestTargetPath
-              }
-            });
+            return invalidImported;
           }
+        );
+        if (invalidDirectedImport) {
+          context.report({
+            node,
+            messageId: "DirectImportDisallowed",
+            data: {
+              rawImportPath,
+              matchedTargetPath: matchedLatestTargetPath
+            }
+          });
         }
       }
+    }
+    return {
+      ExportNamedDeclaration(node) {
+        return checker(node);
+      },
+      ExportAllDeclaration(node) {
+        return checker(node);
+      },
+      //check only import declaration(ESM)
+      ImportDeclaration(node) {
+        return checker(node);
+      }
     };
   }
 };
@@ -250,12 +267,20 @@ var import_path3 = __toESM(require("path"), 1);
 var import_resolve2 = __toESM(require("resolve"), 1);
 var RESOLVE_EXTENSIONS2 = [
   ".ts",
-  ".tsx",
   ".js",
+  ".tsx",
   ".jsx",
+  ".json",
+  ".d.js",
   ".d.ts",
   ".mjs",
-  ".cjs"
+  ".cjs",
+  ".mts",
+  ".cts",
+  ".d.mjs",
+  ".d.cjs",
+  ".d.mts",
+  ".d.cts"
 ];
 var isolateBarrelFile = {
   meta: {
@@ -319,6 +344,9 @@ var isolateBarrelFile = {
           if (aliasResult.type === "success") {
             absoluteImportPath = aliasResult.absolutePath;
           } else {
+            if (!rawImportPath.startsWith(".") && !rawImportPath.startsWith("/") || rawImportPath.includes("/node_modules/")) {
+              return;
+            }
             absoluteImportPath = import_resolve2.default.sync(rawImportPath, {
               basedir: import_path3.default.dirname(absoluteCurrentFilePath),
               extensions: RESOLVE_EXTENSIONS2

package/dist/index.js

@@ -12,11 +12,6 @@ var Glob = class {
       onlyDirectories: true,
       absolute: true
     });
-    if (globResult.length === 0) {
-      throw new Error(
-        `[Glob] In baseDir: ${baseDir}, path: ${path4}, any directory was not found`
-      );
-    }
     return globResult;
   }
 };
@@ -87,13 +82,20 @@ var BARREL_ENTRY_POINT_FILE_NAMES = [
 ];
 var RESOLVE_EXTENSIONS = [
   ".ts",
-  ".tsx",
   ".js",
+  ".tsx",
   ".jsx",
   ".json",
+  ".d.js",
   ".d.ts",
   ".mjs",
-  ".cjs"
+  ".cjs",
+  ".mts",
+  ".cts",
+  ".d.mjs",
+  ".d.cjs",
+  ".d.mts",
+  ".d.cts"
 ];
 var enforceBarrelPattern = {
   meta: {
@@ -147,63 +149,78 @@ var enforceBarrelPattern = {
     const absoluteTargetPaths = option.paths.flatMap((_path) => {
       return Glob.resolvePath(_path, baseDir);
     });
-    return {
-      //check only import declaration(ESM)
-      ImportDeclaration(node) {
-        const rawImportPath = node.source.value;
-        const absoluteCurrentFilePath = context.getFilename();
-        let absoluteImportPath = null;
-        try {
-          const aliasResult = Alias.resolvePath(
-            rawImportPath,
-            path2.dirname(absoluteCurrentFilePath)
-          );
-          if (aliasResult.type === "success") {
-            absoluteImportPath = aliasResult.absolutePath;
-          } else {
-            absoluteImportPath = resolve.sync(rawImportPath, {
-              basedir: path2.dirname(absoluteCurrentFilePath),
-              extensions: RESOLVE_EXTENSIONS
-            });
+    function checker(node) {
+      if (!node.source) {
+        return;
+      }
+      const rawImportPath = node.source.value;
+      const absoluteCurrentFilePath = context.getFilename();
+      let absoluteImportPath = null;
+      try {
+        const aliasResult = Alias.resolvePath(
+          rawImportPath,
+          path2.dirname(absoluteCurrentFilePath)
+        );
+        if (aliasResult.type === "success") {
+          absoluteImportPath = aliasResult.absolutePath;
+        } else {
+          if (!rawImportPath.startsWith(".") && !rawImportPath.startsWith("/") || rawImportPath.includes("/node_modules/")) {
+            return;
           }
-        } catch (e) {
-          context.report({
-            node,
-            messageId: "TransformedAliasResolveFailed"
+          absoluteImportPath = resolve.sync(rawImportPath, {
+            basedir: path2.dirname(absoluteCurrentFilePath),
+            extensions: RESOLVE_EXTENSIONS
           });
-          return;
         }
-        {
-          let matchedLatestTargetPath = null;
-          const invalidDirectedImport = absoluteTargetPaths.some(
-            (absoluteTargetPath) => {
-              const targetPathEntryPoints = BARREL_ENTRY_POINT_FILE_NAMES.map(
-                (entry) => path2.resolve(absoluteTargetPath, entry)
-              );
-              const closedTargetPath = absoluteTargetPath + "/";
-              const targetPathEntryPointed = targetPathEntryPoints.includes(absoluteImportPath);
-              const importedEnforceBarrelFile = absoluteImportPath.startsWith(closedTargetPath);
-              const currentFileInEnforceBarrel = absoluteCurrentFilePath.startsWith(closedTargetPath);
-              const importedOutsideOfTargetPath = !currentFileInEnforceBarrel && importedEnforceBarrelFile;
-              const invalidImported = !targetPathEntryPointed && importedOutsideOfTargetPath;
-              if (invalidImported) {
-                matchedLatestTargetPath = absoluteTargetPath;
-              }
-              return invalidImported;
+      } catch (e) {
+        context.report({
+          node,
+          messageId: "TransformedAliasResolveFailed"
+        });
+        return;
+      }
+      {
+        let matchedLatestTargetPath = null;
+        const invalidDirectedImport = absoluteTargetPaths.some(
+          (absoluteTargetPath) => {
+            const targetPathEntryPoints = BARREL_ENTRY_POINT_FILE_NAMES.map(
+              (entry) => path2.resolve(absoluteTargetPath, entry)
+            );
+            const closedTargetPath = absoluteTargetPath + "/";
+            const targetPathEntryPointed = targetPathEntryPoints.includes(absoluteImportPath);
+            const importedEnforceBarrelFile = absoluteImportPath.startsWith(closedTargetPath);
+            const currentFileInEnforceBarrel = absoluteCurrentFilePath.startsWith(closedTargetPath);
+            const importedOutsideOfTargetPath = !currentFileInEnforceBarrel && importedEnforceBarrelFile;
+            const invalidImported = !targetPathEntryPointed && importedOutsideOfTargetPath;
+            if (invalidImported) {
+              matchedLatestTargetPath = absoluteTargetPath;
             }
-          );
-          if (invalidDirectedImport) {
-            context.report({
-              node,
-              messageId: "DirectImportDisallowed",
-              data: {
-                rawImportPath,
-                matchedTargetPath: matchedLatestTargetPath
-              }
-            });
+            return invalidImported;
           }
+        );
+        if (invalidDirectedImport) {
+          context.report({
+            node,
+            messageId: "DirectImportDisallowed",
+            data: {
+              rawImportPath,
+              matchedTargetPath: matchedLatestTargetPath
+            }
+          });
         }
       }
+    }
+    return {
+      ExportNamedDeclaration(node) {
+        return checker(node);
+      },
+      ExportAllDeclaration(node) {
+        return checker(node);
+      },
+      //check only import declaration(ESM)
+      ImportDeclaration(node) {
+        return checker(node);
+      }
     };
   }
 };
@@ -214,12 +231,20 @@ import path3 from "path";
 import resolve2 from "resolve";
 var RESOLVE_EXTENSIONS2 = [
   ".ts",
-  ".tsx",
   ".js",
+  ".tsx",
   ".jsx",
+  ".json",
+  ".d.js",
   ".d.ts",
   ".mjs",
-  ".cjs"
+  ".cjs",
+  ".mts",
+  ".cts",
+  ".d.mjs",
+  ".d.cjs",
+  ".d.mts",
+  ".d.cts"
 ];
 var isolateBarrelFile = {
   meta: {
@@ -283,6 +308,9 @@ var isolateBarrelFile = {
           if (aliasResult.type === "success") {
             absoluteImportPath = aliasResult.absolutePath;
           } else {
+            if (!rawImportPath.startsWith(".") && !rawImportPath.startsWith("/") || rawImportPath.includes("/node_modules/")) {
+              return;
+            }
             absoluteImportPath = resolve2.sync(rawImportPath, {
               basedir: path3.dirname(absoluteCurrentFilePath),
               extensions: RESOLVE_EXTENSIONS2

package/package.json

@@ -25,7 +25,7 @@
     "isolated barrel module",
     "no-wildcard"
   ],
-  "version": "1.3.0",
+  "version": "1.4.0",
   "type": "module",
   "main": "dist/index.cjs",
   "module": "dist/index.js",
@@ -49,6 +49,7 @@
     "typescript": "~5.8.3"
   },
   "scripts": {
+    "lint": "eslint src/**/*.ts",
     "build": "pnpm run test && tsup src/index.ts --dts --format cjs,esm",
     "type-check": "tsc --noEmit",
     "test": "jest",