eslint-plugin-barrel-rules 1.3.1 → 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.1-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>
@@ -282,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.1-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>
@@ -291,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;
   }
 };
@@ -190,66 +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 {
-            if (!rawImportPath.startsWith(".") && !rawImportPath.startsWith("/") || rawImportPath.includes("/node_modules/")) {
-              return;
-            }
-            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);
+      }
     };
   }
 };

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;
   }
 };
@@ -154,66 +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 {
-            if (!rawImportPath.startsWith(".") && !rawImportPath.startsWith("/") || rawImportPath.includes("/node_modules/")) {
-              return;
-            }
-            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);
+      }
     };
   }
 };

package/package.json

@@ -25,7 +25,7 @@
     "isolated barrel module",
     "no-wildcard"
   ],
-  "version": "1.3.1",
+  "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",