eslint-plugin-barrel-rules 1.4.0 → 1.4.3

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.4.0-blue.svg" alt="Version"/>
+  <img src="https://img.shields.io/badge/version-1.4.3-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>

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.4.0-blue.svg" alt="Version"/>
+  <img src="https://img.shields.io/badge/version-1.4.3-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>

package/dist/index.cjs

@@ -30,7 +30,7 @@ var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: tru
 // src/index.ts
 var index_exports = {};
 __export(index_exports, {
-  default: () => index_default
+  rules: () => rules
 });
 module.exports = __toCommonJS(index_exports);
 
@@ -42,8 +42,8 @@ var import_resolve = __toESM(require("resolve"), 1);
 // src/utils/glob.ts
 var import_fast_glob = __toESM(require("fast-glob"), 1);
 var Glob = class {
-  static resolvePath(path4, baseDir) {
-    const globResult = import_fast_glob.default.sync(path4, {
+  static resolvePath(path5, baseDir) {
+    const globResult = import_fast_glob.default.sync(path5, {
       cwd: baseDir,
       onlyDirectories: true,
       absolute: true
@@ -106,33 +106,59 @@ var Alias = class {
   }
 };
 
-// src/rules/enforce-barrel-pattern.ts
+// src/utils/constants.ts
 var BARREL_ENTRY_POINT_FILE_NAMES = [
   "index.ts",
+  // TypeScript entry
   "index.tsx",
+  // React TypeScript entry
   "index.js",
+  // JavaScript entry
   "index.jsx",
+  // React JavaScript entry
   "index.cjs",
-  "index.mjs",
-  "index.d.ts"
+  // CommonJS entry
+  "index.mjs"
+  // ES Module entry
 ];
 var RESOLVE_EXTENSIONS = [
   ".ts",
+  // TypeScript
   ".js",
+  // JavaScript
   ".tsx",
+  // React TypeScript
   ".jsx",
+  // React JavaScript
   ".json",
+  // JSON file
+  // Type declaration and transformed files
   ".d.js",
+  // JS file with type declarations (rare)
   ".d.ts",
+  // Type declaration file
+  // Various module systems
   ".mjs",
+  // ES Module
   ".cjs",
+  // CommonJS
+  // Module-related TypeScript extensions
   ".mts",
+  // ES Module TypeScript
   ".cts",
+  // CommonJS TypeScript
+  // Type declarations for module-related files
   ".d.mjs",
+  // Type declarations for ES Module
   ".d.cjs",
+  // Type declarations for CommonJS
   ".d.mts",
+  // Type declarations for ES Module TS
   ".d.cts"
+  // Type declarations for CommonJS TS
 ];
+
+// src/rules/enforce-barrel-pattern.ts
 var enforceBarrelPattern = {
   meta: {
     type: "problem",
@@ -265,23 +291,6 @@ var enforceBarrelPattern = {
 var import_types2 = require("@typescript-eslint/types");
 var import_path3 = __toESM(require("path"), 1);
 var import_resolve2 = __toESM(require("resolve"), 1);
-var RESOLVE_EXTENSIONS2 = [
-  ".ts",
-  ".js",
-  ".tsx",
-  ".jsx",
-  ".json",
-  ".d.js",
-  ".d.ts",
-  ".mjs",
-  ".cjs",
-  ".mts",
-  ".cts",
-  ".d.mjs",
-  ".d.cjs",
-  ".d.mts",
-  ".d.cts"
-];
 var isolateBarrelFile = {
   meta: {
     type: "problem",
@@ -316,13 +325,13 @@ var isolateBarrelFile = {
   create(context) {
     const option = context.options[0];
     const baseDir = option.baseDir;
-    const absoluteGlobalAllowPaths = option.globalAllowPaths.flatMap((path4) => {
-      return Glob.resolvePath(path4, baseDir);
+    const absoluteGlobalAllowPaths = option.globalAllowPaths.flatMap((path5) => {
+      return Glob.resolvePath(path5, baseDir);
     });
     const absoluteIsolations = option.isolations.flatMap((isolation) => {
       const isolationPaths = Glob.resolvePath(isolation.path, baseDir);
-      const allowedPaths = isolation.allowedPaths.flatMap((path4) => {
-        return Glob.resolvePath(path4, baseDir);
+      const allowedPaths = isolation.allowedPaths.flatMap((path5) => {
+        return Glob.resolvePath(path5, baseDir);
       });
       return isolationPaths.map((isolationPath) => ({
         isolationPath,
@@ -334,6 +343,9 @@ var isolateBarrelFile = {
       //check only import declaration(ESM)
       ImportDeclaration(node) {
         const rawImportPath = node.source.value;
+        if (rawImportPath.includes("/node_modules/")) {
+          return;
+        }
         const absoluteCurrentFilePath = context.getFilename();
         let absoluteImportPath = null;
         try {
@@ -343,13 +355,16 @@ var isolateBarrelFile = {
           );
           if (aliasResult.type === "success") {
             absoluteImportPath = aliasResult.absolutePath;
+            if (absoluteImportPath.includes("/node_modules/")) {
+              return;
+            }
           } else {
-            if (!rawImportPath.startsWith(".") && !rawImportPath.startsWith("/") || rawImportPath.includes("/node_modules/")) {
+            if (!rawImportPath.startsWith(".") && !rawImportPath.startsWith("/")) {
               return;
             }
             absoluteImportPath = import_resolve2.default.sync(rawImportPath, {
               basedir: import_path3.default.dirname(absoluteCurrentFilePath),
-              extensions: RESOLVE_EXTENSIONS2
+              extensions: RESOLVE_EXTENSIONS
             });
           }
         } catch (e) {
@@ -432,11 +447,259 @@ var noWildcard = {
   }
 };
 
+// src/rules/no-cycle.ts
+var import_types4 = require("@typescript-eslint/types");
+var import_path4 = __toESM(require("path"), 1);
+var import_resolve3 = __toESM(require("resolve"), 1);
+var importGraph = /* @__PURE__ */ new Map();
+var fileToImports = /* @__PURE__ */ new Map();
+var barrelExportsCache = /* @__PURE__ */ new Map();
+function detectCycle(currentFile, visited, recStack, path5) {
+  visited.add(currentFile);
+  recStack.add(currentFile);
+  path5.push(currentFile);
+  const imports = importGraph.get(currentFile) || /* @__PURE__ */ new Set();
+  for (const importedFile of imports) {
+    if (!visited.has(importedFile)) {
+      const cycle = detectCycle(importedFile, visited, recStack, [...path5]);
+      if (cycle) {
+        return cycle;
+      }
+    } else if (recStack.has(importedFile)) {
+      const cycleStart = path5.indexOf(importedFile);
+      return [...path5.slice(cycleStart), importedFile];
+    }
+  }
+  recStack.delete(currentFile);
+  return null;
+}
+function isBarrelFile(filePath) {
+  const fileName = import_path4.default.basename(filePath);
+  return BARREL_ENTRY_POINT_FILE_NAMES.includes(
+    fileName
+  );
+}
+function getExportedModulesFromBarrel(barrelFileDir, sourceCode) {
+  const exportedModules = [];
+  const ast = sourceCode.ast;
+  for (const statement of ast.body) {
+    if (statement.type === "ExportNamedDeclaration" || statement.type === "ExportAllDeclaration") {
+      if (statement.source) {
+        const exportPath = statement.source.value;
+        try {
+          const resolvedPath = import_resolve3.default.sync(exportPath, {
+            basedir: barrelFileDir,
+            extensions: RESOLVE_EXTENSIONS
+          });
+          if (!exportedModules.includes(resolvedPath)) {
+            exportedModules.push(resolvedPath);
+          }
+        } catch (e) {
+        }
+      }
+    }
+    if (statement.type === "ImportDeclaration") {
+      if (statement.source) {
+        const importPath = statement.source.value;
+        if (importPath.startsWith(".") || importPath.startsWith("/")) {
+          try {
+            const resolvedPath = import_resolve3.default.sync(importPath, {
+              basedir: barrelFileDir,
+              extensions: RESOLVE_EXTENSIONS
+            });
+            if (!exportedModules.includes(resolvedPath)) {
+              exportedModules.push(resolvedPath);
+            }
+          } catch (e) {
+          }
+        }
+      }
+    }
+  }
+  return exportedModules;
+}
+var noCycle = {
+  meta: {
+    type: "problem",
+    docs: {
+      description: "Detect circular dependencies and enforce relative imports in barrel files."
+    },
+    schema: [],
+    messages: {
+      CircularDependency: "Circular dependency detected: {{cyclePath}}. This creates a dependency cycle that can cause runtime errors and make code harder to maintain.",
+      BarrelInternalImportDisallowed: "Barrel files (index.ts) must use relative imports (./ or ../) for internal modules. Importing via barrel file or absolute path is not allowed. Use relative path: '{{relativePath}}'",
+      TransformedAliasResolveFailed: "Transformed alias resolve failed. please check the alias config."
+    }
+  },
+  defaultOptions: [],
+  create(context) {
+    const absoluteCurrentFilePath = context.getFilename();
+    if (!importGraph.has(absoluteCurrentFilePath)) {
+      importGraph.set(absoluteCurrentFilePath, /* @__PURE__ */ new Set());
+    }
+    const currentImportsSet = importGraph.get(absoluteCurrentFilePath);
+    currentImportsSet.clear();
+    if (!fileToImports.has(absoluteCurrentFilePath)) {
+      fileToImports.set(absoluteCurrentFilePath, /* @__PURE__ */ new Set());
+    }
+    if (isBarrelFile(absoluteCurrentFilePath)) {
+      const barrelFileDir = import_path4.default.dirname(absoluteCurrentFilePath);
+      const sourceCode = context.getSourceCode();
+      const exportedModules = getExportedModulesFromBarrel(
+        barrelFileDir,
+        sourceCode
+      );
+      barrelExportsCache.set(absoluteCurrentFilePath, exportedModules);
+    }
+    function resolveImportPath(rawImportPath) {
+      try {
+        const aliasResult = Alias.resolvePath(
+          rawImportPath,
+          import_path4.default.dirname(absoluteCurrentFilePath)
+        );
+        if (aliasResult.type === "success") {
+          const resolvedPath = aliasResult.absolutePath;
+          try {
+            const fs = require("fs");
+            const stats = fs.statSync(resolvedPath);
+            if (stats.isDirectory()) {
+              return import_resolve3.default.sync("index", {
+                basedir: resolvedPath,
+                extensions: RESOLVE_EXTENSIONS
+              });
+            }
+          } catch (e) {
+            try {
+              return import_resolve3.default.sync(resolvedPath, {
+                basedir: import_path4.default.dirname(absoluteCurrentFilePath),
+                extensions: RESOLVE_EXTENSIONS
+              });
+            } catch (e2) {
+              return resolvedPath;
+            }
+          }
+          return resolvedPath;
+        } else {
+          if (!rawImportPath.startsWith(".") && !rawImportPath.startsWith("/") || rawImportPath.includes("/node_modules/")) {
+            return null;
+          }
+          return import_resolve3.default.sync(rawImportPath, {
+            basedir: import_path4.default.dirname(absoluteCurrentFilePath),
+            extensions: RESOLVE_EXTENSIONS
+          });
+        }
+      } catch (e) {
+        return null;
+      }
+    }
+    function checker(node) {
+      if (!node.source) {
+        return;
+      }
+      const rawImportPath = node.source.value;
+      const absoluteImportPath = resolveImportPath(rawImportPath);
+      if (!absoluteImportPath) {
+        return;
+      }
+      if (isBarrelFile(absoluteCurrentFilePath)) {
+        const barrelFileDir = import_path4.default.dirname(absoluteCurrentFilePath);
+        const isInternalModule = absoluteImportPath.startsWith(barrelFileDir + import_path4.default.sep) || absoluteImportPath === barrelFileDir;
+        if (isInternalModule) {
+          if (!rawImportPath.startsWith("./") && !rawImportPath.startsWith("../")) {
+            const relativePath = import_path4.default.relative(
+              barrelFileDir,
+              absoluteImportPath
+            );
+            const suggestedRelativePath = relativePath.startsWith(".") ? relativePath : `./${relativePath}`;
+            context.report({
+              node,
+              messageId: "BarrelInternalImportDisallowed",
+              data: {
+                relativePath: suggestedRelativePath.replace(/\\/g, "/")
+              }
+            });
+            return;
+          }
+        }
+      }
+      const isBarrelImportingInternal = isBarrelFile(absoluteCurrentFilePath) && absoluteImportPath.startsWith(
+        import_path4.default.dirname(absoluteCurrentFilePath) + import_path4.default.sep
+      );
+      const currentImports = importGraph.get(absoluteCurrentFilePath);
+      if (currentImports) {
+        currentImports.add(absoluteImportPath);
+        if (isBarrelFile(absoluteImportPath)) {
+          const exportedModules = barrelExportsCache.get(absoluteImportPath) || [];
+          for (const exportedModule of exportedModules) {
+            if (exportedModule !== absoluteCurrentFilePath) {
+              currentImports.add(exportedModule);
+              const exportedModuleImports = importGraph.get(exportedModule);
+              if (exportedModuleImports && exportedModuleImports.has(absoluteCurrentFilePath)) {
+                const cyclePath = `${absoluteCurrentFilePath} \u2192 ${absoluteImportPath} \u2192 ${exportedModule} \u2192 ${absoluteImportPath} \u2192 ${absoluteCurrentFilePath}`;
+                context.report({
+                  node,
+                  messageId: "CircularDependency",
+                  data: {
+                    cyclePath
+                  }
+                });
+                return;
+              }
+            }
+          }
+        }
+      }
+      if (isBarrelImportingInternal) {
+        return;
+      }
+      const importedFileImports = importGraph.get(absoluteImportPath);
+      if (importedFileImports && importedFileImports.has(absoluteCurrentFilePath)) {
+        const cyclePath = `${absoluteCurrentFilePath} \u2192 ${absoluteImportPath} \u2192 ${absoluteCurrentFilePath}`;
+        context.report({
+          node,
+          messageId: "CircularDependency",
+          data: {
+            cyclePath
+          }
+        });
+        return;
+      }
+      const visited = /* @__PURE__ */ new Set();
+      const recStack = /* @__PURE__ */ new Set();
+      const cycle = detectCycle(absoluteCurrentFilePath, visited, recStack, []);
+      if (cycle && cycle.length > 0) {
+        const cyclePath = cycle.join(" \u2192 ");
+        context.report({
+          node,
+          messageId: "CircularDependency",
+          data: {
+            cyclePath
+          }
+        });
+      }
+    }
+    return {
+      ImportDeclaration(node) {
+        return checker(node);
+      },
+      ExportNamedDeclaration(node) {
+        return checker(node);
+      },
+      ExportAllDeclaration(node) {
+        return checker(node);
+      }
+    };
+  }
+};
+
 // src/index.ts
 var rules = {
   "enforce-barrel-pattern": enforceBarrelPattern,
   "isolate-barrel-file": isolateBarrelFile,
-  "no-wildcard": noWildcard
+  "no-wildcard": noWildcard,
+  "no-cycle": noCycle
 };
-var index_default = { rules };
-module.exports = { rules };
+// Annotate the CommonJS export names for ESM import in node:
+0 && (module.exports = {
+  rules
+});

package/dist/index.d.cts

@@ -1,21 +1,20 @@
 import * as _typescript_eslint_utils_ts_eslint from '@typescript-eslint/utils/ts-eslint';
 
-declare const _default: {
-    rules: {
-        "enforce-barrel-pattern": _typescript_eslint_utils_ts_eslint.RuleModule<"DirectImportDisallowed" | "TransformedAliasResolveFailed" | "EmptyEslintConfig", {
-            paths: string[];
-            baseDir: string;
-        }[], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
-        "isolate-barrel-file": _typescript_eslint_utils_ts_eslint.RuleModule<"TransformedAliasResolveFailed" | "IsolatedBarrelImportDisallowed", {
-            isolations: {
-                path: string;
-                allowedPaths: string[];
-            }[];
-            baseDir: string;
-            globalAllowPaths: string[];
-        }[], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
-        "no-wildcard": _typescript_eslint_utils_ts_eslint.RuleModule<"NoWildcardImport" | "NoExportAll", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
-    };
+declare const rules: {
+    "enforce-barrel-pattern": _typescript_eslint_utils_ts_eslint.RuleModule<"DirectImportDisallowed" | "TransformedAliasResolveFailed" | "EmptyEslintConfig", {
+        paths: string[];
+        baseDir: string;
+    }[], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+    "isolate-barrel-file": _typescript_eslint_utils_ts_eslint.RuleModule<"TransformedAliasResolveFailed" | "IsolatedBarrelImportDisallowed", {
+        isolations: {
+            path: string;
+            allowedPaths: string[];
+        }[];
+        baseDir: string;
+        globalAllowPaths: string[];
+    }[], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+    "no-wildcard": _typescript_eslint_utils_ts_eslint.RuleModule<"NoWildcardImport" | "NoExportAll", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+    "no-cycle": _typescript_eslint_utils_ts_eslint.RuleModule<"TransformedAliasResolveFailed" | "CircularDependency" | "BarrelInternalImportDisallowed", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
 };
 
-export { _default as default };
+export { rules };

package/dist/index.d.ts

@@ -1,21 +1,20 @@
 import * as _typescript_eslint_utils_ts_eslint from '@typescript-eslint/utils/ts-eslint';
 
-declare const _default: {
-    rules: {
-        "enforce-barrel-pattern": _typescript_eslint_utils_ts_eslint.RuleModule<"DirectImportDisallowed" | "TransformedAliasResolveFailed" | "EmptyEslintConfig", {
-            paths: string[];
-            baseDir: string;
-        }[], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
-        "isolate-barrel-file": _typescript_eslint_utils_ts_eslint.RuleModule<"TransformedAliasResolveFailed" | "IsolatedBarrelImportDisallowed", {
-            isolations: {
-                path: string;
-                allowedPaths: string[];
-            }[];
-            baseDir: string;
-            globalAllowPaths: string[];
-        }[], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
-        "no-wildcard": _typescript_eslint_utils_ts_eslint.RuleModule<"NoWildcardImport" | "NoExportAll", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
-    };
+declare const rules: {
+    "enforce-barrel-pattern": _typescript_eslint_utils_ts_eslint.RuleModule<"DirectImportDisallowed" | "TransformedAliasResolveFailed" | "EmptyEslintConfig", {
+        paths: string[];
+        baseDir: string;
+    }[], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+    "isolate-barrel-file": _typescript_eslint_utils_ts_eslint.RuleModule<"TransformedAliasResolveFailed" | "IsolatedBarrelImportDisallowed", {
+        isolations: {
+            path: string;
+            allowedPaths: string[];
+        }[];
+        baseDir: string;
+        globalAllowPaths: string[];
+    }[], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+    "no-wildcard": _typescript_eslint_utils_ts_eslint.RuleModule<"NoWildcardImport" | "NoExportAll", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
+    "no-cycle": _typescript_eslint_utils_ts_eslint.RuleModule<"TransformedAliasResolveFailed" | "CircularDependency" | "BarrelInternalImportDisallowed", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener>;
 };
 
-export { _default as default };
+export { rules };

package/dist/index.js

@@ -1,3 +1,10 @@
+var __require = /* @__PURE__ */ ((x) => typeof require !== "undefined" ? require : typeof Proxy !== "undefined" ? new Proxy(x, {
+  get: (a, b) => (typeof require !== "undefined" ? require : a)[b]
+}) : x)(function(x) {
+  if (typeof require !== "undefined") return require.apply(this, arguments);
+  throw Error('Dynamic require of "' + x + '" is not supported');
+});
+
 // src/rules/enforce-barrel-pattern.ts
 import "@typescript-eslint/types";
 import path2 from "path";
@@ -6,8 +13,8 @@ import resolve from "resolve";
 // src/utils/glob.ts
 import FastGlob from "fast-glob";
 var Glob = class {
-  static resolvePath(path4, baseDir) {
-    const globResult = FastGlob.sync(path4, {
+  static resolvePath(path5, baseDir) {
+    const globResult = FastGlob.sync(path5, {
       cwd: baseDir,
       onlyDirectories: true,
       absolute: true
@@ -70,33 +77,59 @@ var Alias = class {
   }
 };
 
-// src/rules/enforce-barrel-pattern.ts
+// src/utils/constants.ts
 var BARREL_ENTRY_POINT_FILE_NAMES = [
   "index.ts",
+  // TypeScript entry
   "index.tsx",
+  // React TypeScript entry
   "index.js",
+  // JavaScript entry
   "index.jsx",
+  // React JavaScript entry
   "index.cjs",
-  "index.mjs",
-  "index.d.ts"
+  // CommonJS entry
+  "index.mjs"
+  // ES Module entry
 ];
 var RESOLVE_EXTENSIONS = [
   ".ts",
+  // TypeScript
   ".js",
+  // JavaScript
   ".tsx",
+  // React TypeScript
   ".jsx",
+  // React JavaScript
   ".json",
+  // JSON file
+  // Type declaration and transformed files
   ".d.js",
+  // JS file with type declarations (rare)
   ".d.ts",
+  // Type declaration file
+  // Various module systems
   ".mjs",
+  // ES Module
   ".cjs",
+  // CommonJS
+  // Module-related TypeScript extensions
   ".mts",
+  // ES Module TypeScript
   ".cts",
+  // CommonJS TypeScript
+  // Type declarations for module-related files
   ".d.mjs",
+  // Type declarations for ES Module
   ".d.cjs",
+  // Type declarations for CommonJS
   ".d.mts",
+  // Type declarations for ES Module TS
   ".d.cts"
+  // Type declarations for CommonJS TS
 ];
+
+// src/rules/enforce-barrel-pattern.ts
 var enforceBarrelPattern = {
   meta: {
     type: "problem",
@@ -229,23 +262,6 @@ var enforceBarrelPattern = {
 import "@typescript-eslint/types";
 import path3 from "path";
 import resolve2 from "resolve";
-var RESOLVE_EXTENSIONS2 = [
-  ".ts",
-  ".js",
-  ".tsx",
-  ".jsx",
-  ".json",
-  ".d.js",
-  ".d.ts",
-  ".mjs",
-  ".cjs",
-  ".mts",
-  ".cts",
-  ".d.mjs",
-  ".d.cjs",
-  ".d.mts",
-  ".d.cts"
-];
 var isolateBarrelFile = {
   meta: {
     type: "problem",
@@ -280,13 +296,13 @@ var isolateBarrelFile = {
   create(context) {
     const option = context.options[0];
     const baseDir = option.baseDir;
-    const absoluteGlobalAllowPaths = option.globalAllowPaths.flatMap((path4) => {
-      return Glob.resolvePath(path4, baseDir);
+    const absoluteGlobalAllowPaths = option.globalAllowPaths.flatMap((path5) => {
+      return Glob.resolvePath(path5, baseDir);
     });
     const absoluteIsolations = option.isolations.flatMap((isolation) => {
       const isolationPaths = Glob.resolvePath(isolation.path, baseDir);
-      const allowedPaths = isolation.allowedPaths.flatMap((path4) => {
-        return Glob.resolvePath(path4, baseDir);
+      const allowedPaths = isolation.allowedPaths.flatMap((path5) => {
+        return Glob.resolvePath(path5, baseDir);
       });
       return isolationPaths.map((isolationPath) => ({
         isolationPath,
@@ -298,6 +314,9 @@ var isolateBarrelFile = {
       //check only import declaration(ESM)
       ImportDeclaration(node) {
         const rawImportPath = node.source.value;
+        if (rawImportPath.includes("/node_modules/")) {
+          return;
+        }
         const absoluteCurrentFilePath = context.getFilename();
         let absoluteImportPath = null;
         try {
@@ -307,13 +326,16 @@ var isolateBarrelFile = {
           );
           if (aliasResult.type === "success") {
             absoluteImportPath = aliasResult.absolutePath;
+            if (absoluteImportPath.includes("/node_modules/")) {
+              return;
+            }
           } else {
-            if (!rawImportPath.startsWith(".") && !rawImportPath.startsWith("/") || rawImportPath.includes("/node_modules/")) {
+            if (!rawImportPath.startsWith(".") && !rawImportPath.startsWith("/")) {
               return;
             }
             absoluteImportPath = resolve2.sync(rawImportPath, {
               basedir: path3.dirname(absoluteCurrentFilePath),
-              extensions: RESOLVE_EXTENSIONS2
+              extensions: RESOLVE_EXTENSIONS
             });
           }
         } catch (e) {
@@ -396,14 +418,258 @@ var noWildcard = {
   }
 };
 
+// src/rules/no-cycle.ts
+import "@typescript-eslint/types";
+import path4 from "path";
+import resolve3 from "resolve";
+var importGraph = /* @__PURE__ */ new Map();
+var fileToImports = /* @__PURE__ */ new Map();
+var barrelExportsCache = /* @__PURE__ */ new Map();
+function detectCycle(currentFile, visited, recStack, path5) {
+  visited.add(currentFile);
+  recStack.add(currentFile);
+  path5.push(currentFile);
+  const imports = importGraph.get(currentFile) || /* @__PURE__ */ new Set();
+  for (const importedFile of imports) {
+    if (!visited.has(importedFile)) {
+      const cycle = detectCycle(importedFile, visited, recStack, [...path5]);
+      if (cycle) {
+        return cycle;
+      }
+    } else if (recStack.has(importedFile)) {
+      const cycleStart = path5.indexOf(importedFile);
+      return [...path5.slice(cycleStart), importedFile];
+    }
+  }
+  recStack.delete(currentFile);
+  return null;
+}
+function isBarrelFile(filePath) {
+  const fileName = path4.basename(filePath);
+  return BARREL_ENTRY_POINT_FILE_NAMES.includes(
+    fileName
+  );
+}
+function getExportedModulesFromBarrel(barrelFileDir, sourceCode) {
+  const exportedModules = [];
+  const ast = sourceCode.ast;
+  for (const statement of ast.body) {
+    if (statement.type === "ExportNamedDeclaration" || statement.type === "ExportAllDeclaration") {
+      if (statement.source) {
+        const exportPath = statement.source.value;
+        try {
+          const resolvedPath = resolve3.sync(exportPath, {
+            basedir: barrelFileDir,
+            extensions: RESOLVE_EXTENSIONS
+          });
+          if (!exportedModules.includes(resolvedPath)) {
+            exportedModules.push(resolvedPath);
+          }
+        } catch (e) {
+        }
+      }
+    }
+    if (statement.type === "ImportDeclaration") {
+      if (statement.source) {
+        const importPath = statement.source.value;
+        if (importPath.startsWith(".") || importPath.startsWith("/")) {
+          try {
+            const resolvedPath = resolve3.sync(importPath, {
+              basedir: barrelFileDir,
+              extensions: RESOLVE_EXTENSIONS
+            });
+            if (!exportedModules.includes(resolvedPath)) {
+              exportedModules.push(resolvedPath);
+            }
+          } catch (e) {
+          }
+        }
+      }
+    }
+  }
+  return exportedModules;
+}
+var noCycle = {
+  meta: {
+    type: "problem",
+    docs: {
+      description: "Detect circular dependencies and enforce relative imports in barrel files."
+    },
+    schema: [],
+    messages: {
+      CircularDependency: "Circular dependency detected: {{cyclePath}}. This creates a dependency cycle that can cause runtime errors and make code harder to maintain.",
+      BarrelInternalImportDisallowed: "Barrel files (index.ts) must use relative imports (./ or ../) for internal modules. Importing via barrel file or absolute path is not allowed. Use relative path: '{{relativePath}}'",
+      TransformedAliasResolveFailed: "Transformed alias resolve failed. please check the alias config."
+    }
+  },
+  defaultOptions: [],
+  create(context) {
+    const absoluteCurrentFilePath = context.getFilename();
+    if (!importGraph.has(absoluteCurrentFilePath)) {
+      importGraph.set(absoluteCurrentFilePath, /* @__PURE__ */ new Set());
+    }
+    const currentImportsSet = importGraph.get(absoluteCurrentFilePath);
+    currentImportsSet.clear();
+    if (!fileToImports.has(absoluteCurrentFilePath)) {
+      fileToImports.set(absoluteCurrentFilePath, /* @__PURE__ */ new Set());
+    }
+    if (isBarrelFile(absoluteCurrentFilePath)) {
+      const barrelFileDir = path4.dirname(absoluteCurrentFilePath);
+      const sourceCode = context.getSourceCode();
+      const exportedModules = getExportedModulesFromBarrel(
+        barrelFileDir,
+        sourceCode
+      );
+      barrelExportsCache.set(absoluteCurrentFilePath, exportedModules);
+    }
+    function resolveImportPath(rawImportPath) {
+      try {
+        const aliasResult = Alias.resolvePath(
+          rawImportPath,
+          path4.dirname(absoluteCurrentFilePath)
+        );
+        if (aliasResult.type === "success") {
+          const resolvedPath = aliasResult.absolutePath;
+          try {
+            const fs = __require("fs");
+            const stats = fs.statSync(resolvedPath);
+            if (stats.isDirectory()) {
+              return resolve3.sync("index", {
+                basedir: resolvedPath,
+                extensions: RESOLVE_EXTENSIONS
+              });
+            }
+          } catch (e) {
+            try {
+              return resolve3.sync(resolvedPath, {
+                basedir: path4.dirname(absoluteCurrentFilePath),
+                extensions: RESOLVE_EXTENSIONS
+              });
+            } catch (e2) {
+              return resolvedPath;
+            }
+          }
+          return resolvedPath;
+        } else {
+          if (!rawImportPath.startsWith(".") && !rawImportPath.startsWith("/") || rawImportPath.includes("/node_modules/")) {
+            return null;
+          }
+          return resolve3.sync(rawImportPath, {
+            basedir: path4.dirname(absoluteCurrentFilePath),
+            extensions: RESOLVE_EXTENSIONS
+          });
+        }
+      } catch (e) {
+        return null;
+      }
+    }
+    function checker(node) {
+      if (!node.source) {
+        return;
+      }
+      const rawImportPath = node.source.value;
+      const absoluteImportPath = resolveImportPath(rawImportPath);
+      if (!absoluteImportPath) {
+        return;
+      }
+      if (isBarrelFile(absoluteCurrentFilePath)) {
+        const barrelFileDir = path4.dirname(absoluteCurrentFilePath);
+        const isInternalModule = absoluteImportPath.startsWith(barrelFileDir + path4.sep) || absoluteImportPath === barrelFileDir;
+        if (isInternalModule) {
+          if (!rawImportPath.startsWith("./") && !rawImportPath.startsWith("../")) {
+            const relativePath = path4.relative(
+              barrelFileDir,
+              absoluteImportPath
+            );
+            const suggestedRelativePath = relativePath.startsWith(".") ? relativePath : `./${relativePath}`;
+            context.report({
+              node,
+              messageId: "BarrelInternalImportDisallowed",
+              data: {
+                relativePath: suggestedRelativePath.replace(/\\/g, "/")
+              }
+            });
+            return;
+          }
+        }
+      }
+      const isBarrelImportingInternal = isBarrelFile(absoluteCurrentFilePath) && absoluteImportPath.startsWith(
+        path4.dirname(absoluteCurrentFilePath) + path4.sep
+      );
+      const currentImports = importGraph.get(absoluteCurrentFilePath);
+      if (currentImports) {
+        currentImports.add(absoluteImportPath);
+        if (isBarrelFile(absoluteImportPath)) {
+          const exportedModules = barrelExportsCache.get(absoluteImportPath) || [];
+          for (const exportedModule of exportedModules) {
+            if (exportedModule !== absoluteCurrentFilePath) {
+              currentImports.add(exportedModule);
+              const exportedModuleImports = importGraph.get(exportedModule);
+              if (exportedModuleImports && exportedModuleImports.has(absoluteCurrentFilePath)) {
+                const cyclePath = `${absoluteCurrentFilePath} \u2192 ${absoluteImportPath} \u2192 ${exportedModule} \u2192 ${absoluteImportPath} \u2192 ${absoluteCurrentFilePath}`;
+                context.report({
+                  node,
+                  messageId: "CircularDependency",
+                  data: {
+                    cyclePath
+                  }
+                });
+                return;
+              }
+            }
+          }
+        }
+      }
+      if (isBarrelImportingInternal) {
+        return;
+      }
+      const importedFileImports = importGraph.get(absoluteImportPath);
+      if (importedFileImports && importedFileImports.has(absoluteCurrentFilePath)) {
+        const cyclePath = `${absoluteCurrentFilePath} \u2192 ${absoluteImportPath} \u2192 ${absoluteCurrentFilePath}`;
+        context.report({
+          node,
+          messageId: "CircularDependency",
+          data: {
+            cyclePath
+          }
+        });
+        return;
+      }
+      const visited = /* @__PURE__ */ new Set();
+      const recStack = /* @__PURE__ */ new Set();
+      const cycle = detectCycle(absoluteCurrentFilePath, visited, recStack, []);
+      if (cycle && cycle.length > 0) {
+        const cyclePath = cycle.join(" \u2192 ");
+        context.report({
+          node,
+          messageId: "CircularDependency",
+          data: {
+            cyclePath
+          }
+        });
+      }
+    }
+    return {
+      ImportDeclaration(node) {
+        return checker(node);
+      },
+      ExportNamedDeclaration(node) {
+        return checker(node);
+      },
+      ExportAllDeclaration(node) {
+        return checker(node);
+      }
+    };
+  }
+};
+
 // src/index.ts
 var rules = {
   "enforce-barrel-pattern": enforceBarrelPattern,
   "isolate-barrel-file": isolateBarrelFile,
-  "no-wildcard": noWildcard
+  "no-wildcard": noWildcard,
+  "no-cycle": noCycle
 };
-var index_default = { rules };
-module.exports = { rules };
 export {
-  index_default as default
+  rules
 };

package/package.json

@@ -25,32 +25,32 @@
     "isolated barrel module",
     "no-wildcard"
   ],
-  "version": "1.4.0",
+  "version": "1.4.3",
   "type": "module",
   "main": "dist/index.cjs",
   "module": "dist/index.js",
   "types": "dist/index.d.ts",
   "dependencies": {
-    "@types/jest": "^30.0.0",
-    "@typescript-eslint/parser": "^8.46.2",
-    "@typescript-eslint/rule-tester": "^8.46.2",
     "@typescript-eslint/types": "^8.46.2",
     "@typescript-eslint/utils": "^8.36.0",
     "fast-glob": "^3.3.3",
-    "jest": "^30.2.0",
     "resolve": "^1.22.10",
-    "ts-jest": "^29.4.5",
     "tsconfig-paths": "^4.2.0"
   },
   "devDependencies": {
+    "jest": "^30.2.0",
     "@types/node": "^24.0.13",
     "@types/resolve": "^1.20.6",
     "tsup": "^8.5.0",
-    "typescript": "~5.8.3"
+    "typescript": "~5.8.3",
+    "@types/jest": "^30.0.0",
+    "@typescript-eslint/parser": "^8.46.2",
+    "@typescript-eslint/rule-tester": "^8.46.2",
+    "ts-jest": "^29.4.5"
   },
   "scripts": {
     "lint": "eslint src/**/*.ts",
-    "build": "pnpm run test && tsup src/index.ts --dts --format cjs,esm",
+    "build": "pnpm run test && tsup",
     "type-check": "tsc --noEmit",
     "test": "jest",
     "test:watch": "jest --watch",