eslint-plugin-import-next 2.3.3 → 2.3.6

package/src/rules/dynamic-import-chunkname.js

@@ -12,7 +12,10 @@ exports.dynamicImportChunkname = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/dynamic-import-chunkname.md',
             description: 'Enforce a leading comment with the webpackChunkName for dynamic imports',
+            cwe: 'CWE-1078',
+            cvss: 2.5,
         },
         hasSuggestions: true,
         messages: {
@@ -62,7 +65,7 @@ exports.dynamicImportChunkname = (0, eslint_devkit_1.createRule)({
         const options = context.options[0] || {};
         const { allowEmpty = false, webpackChunknameFormat = '[a-zA-Z0-9-_/.]+' } = options;
         const chunkNamePattern = new RegExp(`^${webpackChunknameFormat}$`);
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
         function checkChunkName(node) {
             // Get leading comments
             const comments = sourceCode.getCommentsBefore(node.source);

package/src/rules/enforce-dependency-direction.js

@@ -48,6 +48,7 @@ exports.enforceDependencyDirection = (0, eslint_devkit_2.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/enforce-dependency-direction.md',
             description: 'Ensures dependencies flow in the correct architectural direction',
         },
         messages: {
@@ -129,7 +130,7 @@ exports.enforceDependencyDirection = (0, eslint_devkit_2.createRule)({
         },
     ],
     create(context, [options = {}]) {
-        const filename = context.getFilename();
+        const filename = context.filename;
         const { layers: providedLayers, layerPatterns: providedLayerPatterns, allowSameLayer = true, } = options || {};
         // Use provided layers or defaults
         const layers = providedLayers || [

package/src/rules/enforce-import-order.js

@@ -30,6 +30,7 @@ exports.enforceImportOrder = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/enforce-import-order.md',
             description: 'Enforces a specific order for import statements',
         },
         fixable: 'code',
@@ -121,7 +122,7 @@ exports.enforceImportOrder = (0, eslint_devkit_1.createRule)({
                     defaultOptions.alphabetize.caseInsensitive,
             },
         };
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode;
         const imports = [];
         function getImportType(node) {
             const source = node.source.value;
@@ -237,7 +238,8 @@ exports.enforceImportOrder = (0, eslint_devkit_1.createRule)({
                     }
                 }
                 const originalImports = [...imports];
-                const sortedImports = [...imports].sort((a, b) => {
+                // oxlint-disable-next-line unicorn/no-array-sort
+                const sortedImports = Array.from(imports).sort((a, b) => {
                     const typeA = getImportType(a);
                     const typeB = getImportType(b);
                     const rankA = getGroupRank(typeA);

package/src/rules/enforce-team-boundaries.js

@@ -7,18 +7,35 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.enforceTeamBoundaries = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
+/**
+ * Translate a user-supplied glob (`foo/**\/*.ts`) into a RegExp.
+ *
+ * Important: regex metacharacters in the input must be escaped BEFORE the
+ * glob tokens are translated. Without escaping, a pattern like `lib\foo` is
+ * interpreted as `\f` (form feed) and `lib.foo` matches `libXfoo` rather
+ * than the literal dot the user wrote. CodeQL flags the un-escaped backslash
+ * specifically (`js/incomplete-sanitization`).
+ *
+ * Algorithm: escape all regex metas (including `\`), then unescape and
+ * translate just the glob tokens `**` and `*` back to their regex form.
+ */
+function globToRegex(pattern) {
+    // Escape regex metacharacters INCLUDING `*`, so the subsequent replaces
+    // can find the now-escaped `\*\*` / `\*` and translate them back to their
+    // glob meanings. Without escaping `*`, the `**` in `src/platform/**`
+    // reaches `new RegExp()` raw and throws "Nothing to repeat".
+    const escaped = pattern.replace(/[.+?^${}()|[\]\\*]/g, '\\$&');
+    return new RegExp(escaped
+        .replace(/\\\*\\\*/g, '.*')
+        .replace(/\\\*/g, '[^/]*'));
+}
 /**
  * Determine which team owns this file based on path patterns
  */
 function getFileTeam(filePath, teams) {
     for (const team of teams) {
         for (const pattern of team.paths) {
-            // Support glob patterns (simplified)
-            const regex = new RegExp(pattern
-                .replace(/\*\*/g, '.*')
-                .replace(/\*/g, '[^/]*')
-                .replace(/\//g, '\\/'));
-            if (regex.test(filePath)) {
+            if (globToRegex(pattern).test(filePath)) {
                 return team;
             }
         }
@@ -31,11 +48,7 @@ function getFileTeam(filePath, teams) {
 function getImportTeam(importPath, teams) {
     for (const team of teams) {
         for (const pattern of team.paths) {
-            const regex = new RegExp(pattern
-                .replace(/\*\*/g, '.*')
-                .replace(/\*/g, '[^/]*')
-                .replace(/\//g, '\\/'));
-            if (regex.test(importPath)) {
+            if (globToRegex(pattern).test(importPath)) {
                 return team;
             }
         }
@@ -77,11 +90,7 @@ function isExternalImport(importPath) {
  */
 function isSharedPath(importPath, sharedPaths) {
     for (const shared of sharedPaths) {
-        const regex = new RegExp(shared
-            .replace(/\*\*/g, '.*')
-            .replace(/\*/g, '[^/]*')
-            .replace(/\//g, '\\/'));
-        if (regex.test(importPath)) {
+        if (globToRegex(shared).test(importPath)) {
             return true;
         }
     }
@@ -92,6 +101,7 @@ exports.enforceTeamBoundaries = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/enforce-team-boundaries.md',
             description: 'Prevent unauthorized cross-team imports in large codebases',
         },
         messages: {
@@ -154,7 +164,7 @@ exports.enforceTeamBoundaries = (0, eslint_devkit_1.createRule)({
     create(context, [options]) {
         const { teams, sharedPaths = [], allowExternalPackages = true } = options;
         // Get the file's team ownership
-        const filename = context.filename || context.getFilename();
+        const filename = context.filename;
         const sourceTeam = getFileTeam(filename, teams);
         // If file doesn't belong to any team, skip checking
         if (!sourceTeam) {

package/src/rules/export.js

@@ -12,7 +12,10 @@ exports.exportRule = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/export.md',
             description: 'Forbid any invalid exports, i.e. re-export of the same name',
+            cwe: 'CWE-694',
+            cvss: 7.5,
         },
         messages: {
             duplicateExport: (0, eslint_devkit_1.formatLLMMessage)({

package/src/rules/exports-last.js

@@ -12,7 +12,10 @@ exports.exportsLast = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/exports-last.md',
             description: 'Ensure all exports appear after other statements',
+            cwe: 'CWE-1078',
+            cvss: 2.5,
         },
         messages: {
             exportNotLast: (0, eslint_devkit_1.formatLLMMessage)({

package/src/rules/extensions.js

@@ -15,6 +15,7 @@ exports.extensions = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/extensions.md',
             description: 'Ensure consistent use of file extensions in imports',
         },
         fixable: 'code',

package/src/rules/first.js

@@ -13,6 +13,7 @@ exports.first = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/first.md',
             description: 'Ensure all imports appear before other statements',
         },
         fixable: 'code',
@@ -48,6 +49,7 @@ exports.first = (0, eslint_devkit_1.createRule)({
                 }
             },
         };
+        // oxlint-disable-next-line consistent-function-scoping
         function isDirective(node) {
             return (node.type === 'ExpressionStatement' &&
                 node.expression.type === 'Literal' &&

package/src/rules/group-exports.js

@@ -12,7 +12,10 @@ exports.groupExports = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/group-exports.md',
             description: 'Prefer named exports to be grouped together in a single export declaration',
+            cwe: 'CWE-1078',
+            cvss: 2.5,
         },
         messages: {
             groupExports: (0, eslint_devkit_1.formatLLMMessage)({

package/src/rules/max-dependencies.js

@@ -13,6 +13,7 @@ exports.maxDependencies = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/max-dependencies.md',
             description: 'Enforce the maximum number of dependencies a module can have',
         },
         hasSuggestions: false,
@@ -87,7 +88,7 @@ exports.maxDependencies = (0, eslint_devkit_1.createRule)({
     create(context) {
         const [options] = context.options;
         const { max = 10, ignoreTypeImports = true, ignoreImports = [], ignoreFiles = [], } = options || {};
-        const filename = context.getFilename();
+        const filename = context.filename;
         if (!filename) {
             return {};
         }

package/src/rules/named.js

@@ -15,6 +15,7 @@ exports.named = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/named.md',
             description: 'Ensure named imports correspond to a named export in the remote file',
         },
         fixable: 'code',

package/src/rules/namespace.js

@@ -15,6 +15,7 @@ exports.namespace = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/namespace.md',
             description: 'Ensure imported namespaces contain dereferenced properties as they are dereferenced',
         },
         messages: {

package/src/rules/newline-after-import.js

@@ -13,6 +13,7 @@ exports.newlineAfterImport = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'layout',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/newline-after-import.md',
             description: 'Enforce a newline after import statements',
         },
         fixable: 'whitespace',

package/src/rules/no-absolute-path.js

@@ -14,7 +14,10 @@ exports.noAbsolutePath = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-absolute-path.md',
             description: 'Forbid import of modules using absolute paths',
+            cwe: 'CWE-426',
+            cvss: 5,
         },
         fixable: 'code',
         messages: {

package/src/rules/no-amd.js

@@ -13,6 +13,7 @@ exports.noAmd = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-amd.md',
             description: 'Prevents AMD require/define calls',
         },
         hasSuggestions: false,
@@ -58,7 +59,7 @@ exports.noAmd = (0, eslint_devkit_1.createRule)({
         const { allow = [],
         // suggestES6 = true // Reserved for future use when hasSuggestions is enabled
          } = options || {};
-        const filename = context.getFilename();
+        const filename = context.filename;
         function shouldAllow() {
             // Check if the call is in an allowed pattern
             return allow.some((pattern) => {

package/src/rules/no-anonymous-default-export.js

@@ -13,6 +13,7 @@ exports.noAnonymousDefaultExport = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-anonymous-default-export.md',
             description: 'Forbid anonymous values as default exports',
         },
         messages: {
@@ -114,6 +115,7 @@ exports.noAnonymousDefaultExport = (0, eslint_devkit_1.createRule)({
                     return true;
             }
         }
+        // oxlint-disable-next-line consistent-function-scoping
         function getSuggestionText(node) {
             const declaration = node.declaration;
             switch (declaration.type) {

package/src/rules/no-barrel-file.js

@@ -74,6 +74,7 @@ exports.noBarrelFile = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-barrel-file.md',
             description: 'Disallow barrel files that harm build performance and tree-shaking efficiency',
         },
         hasSuggestions: true,
@@ -148,7 +149,7 @@ exports.noBarrelFile = (0, eslint_devkit_1.createRule)({
     ],
     create(context, [options = {}]) {
         const { threshold = 3, allowedPaths = [], barrelPatterns = DEFAULT_BARREL_PATTERNS, allowWithLocalExports = false, reexportRatio = 0.8, } = options || {};
-        const filename = context.filename || context.getFilename();
+        const filename = context.filename;
         // Check if file matches barrel patterns
         if (!matchesPattern(filename, barrelPatterns)) {
             return {};

package/src/rules/no-barrel-import.js

@@ -45,6 +45,7 @@ exports.noBarrelImport = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-barrel-import.md',
             description: 'Disallow imports from barrel files to improve build performance and tree-shaking',
         },
         fixable: 'code',

package/src/rules/no-commonjs.js

@@ -13,6 +13,7 @@ exports.noCommonjs = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-commonjs.md',
             description: 'Prevents CommonJS require/module.exports',
         },
         hasSuggestions: true,
@@ -98,7 +99,7 @@ exports.noCommonjs = (0, eslint_devkit_1.createRule)({
     create(context) {
         const [options] = context.options;
         const { allowRequire = false, allowModuleExports = false, allowExports = false, allowInFiles = [], suggestES6 = true, } = options || {};
-        const filename = context.getFilename();
+        const filename = context.filename;
         function shouldAllow() {
             if (!filename) {
                 return false;

package/src/rules/no-cross-domain-imports.js

@@ -130,6 +130,7 @@ exports.noCrossDomainImports = (0, eslint_devkit_2.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-cross-domain-imports.md',
             description: 'Prevents imports across domain/feature boundaries',
         },
         messages: {
@@ -221,7 +222,7 @@ exports.noCrossDomainImports = (0, eslint_devkit_2.createRule)({
         },
     ],
     create(context, [options = {}]) {
-        const filename = context.getFilename();
+        const filename = context.filename;
         /**
          * Check import declarations
          */

package/src/rules/no-cycle.js

@@ -57,7 +57,11 @@ exports.noCycle = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-cycle.md',
             description: 'Detect circular dependencies that cause bundle memory bloat and initialization issues',
+            cwe: 'CWE-407',
+            cvss: 9.5,
+            confidence: 'high',
         },
         messages: {
             // 🎯 Token optimization: 45% reduction (~70→38 tokens per message) for architecture clarity
@@ -104,8 +108,8 @@ exports.noCycle = (0, eslint_devkit_1.createRule)({
                 properties: {
                     maxDepth: {
                         type: 'number',
-                        default: 5,
-                        description: 'Maximum depth to traverse when detecting cycles (performance optimization)',
+                        default: Number.MAX_SAFE_INTEGER,
+                        description: 'Maximum depth to traverse when detecting cycles. Default: unlimited (matches eslint-plugin-import and oxlint). Lower values are a performance escape hatch — but with our nonCyclicFiles cache, traversal cost is amortized, and a low cap silently misses cycles deeper than the limit. Set to a finite number only on huge graphs where the bench latency hurts you.',
                     },
                     ignorePatterns: {
                         type: 'array',
@@ -168,7 +172,12 @@ exports.noCycle = (0, eslint_devkit_1.createRule)({
     },
     defaultOptions: [
         {
-            maxDepth: 10,
+            // Unlimited: matches eslint-plugin-import (Infinity) and oxlint
+            // (u32::MAX). The earlier default of 10 silently missed any cycle
+            // deeper than 10 hops — verified on next.js's webpack-config.ts
+            // (~12 hops). The nonCyclicFiles cache + the depth-limit-truncation
+            // fix in eslint-devkit's dependency-analysis make unlimited safe.
+            maxDepth: Number.MAX_SAFE_INTEGER,
             ignorePatterns: [],
             barrelExports: ['index.ts', 'index.tsx', 'index.js', 'index.jsx'],
             reportAllCycles: true,
@@ -180,7 +189,7 @@ exports.noCycle = (0, eslint_devkit_1.createRule)({
     ],
     create(context) {
         const options = context.options[0] || {};
-        const maxDepth = options.maxDepth ?? 10;
+        const maxDepth = options.maxDepth ?? Number.MAX_SAFE_INTEGER;
         const ignorePatterns = options.ignorePatterns ?? [
             '**/*.test.ts',
             '**/*.test.tsx',
@@ -201,13 +210,15 @@ exports.noCycle = (0, eslint_devkit_1.createRule)({
         const moduleNamingConvention = options.moduleNamingConvention ?? 'semantic';
         const coreModuleSuffix = options.coreModuleSuffix ?? 'core';
         const extendedModuleSuffix = options.extendedModuleSuffix ?? 'extended';
-        const filename = context.getFilename();
-        const workspaceRoot = context.getCwd();
+        const filename = context.filename;
+        const workspaceRoot = context.cwd;
         // Get resolver settings from ESLint settings (compatible with eslint-plugin-import)
-        // eslint-plugin-import uses settings['import/resolver']
+        // eslint-plugin-import uses settings['import/resolver']; allow either the
+        // upstream key or our drop-in `import-next/resolver` form for users
+        // migrating off eslint-plugin-import.
         const settings = context.settings;
         const resolverSettings = settings?.['import/resolver'] ||
-            settings?.['eslint-plugin-llm-optimized/resolver'];
+            settings?.['import-next/resolver'];
         // Skip ignored files early
         if ((0, eslint_devkit_3.shouldIgnoreFile)(filename, ignorePatterns, sharedCache.compiledPatterns)) {
             return {};
@@ -257,9 +268,9 @@ exports.noCycle = (0, eslint_devkit_1.createRule)({
          * Generate DIRECT IMPORT message
          */
         function generateDirectImportMessage(cycle, sourceImport) {
-            const currentFile = (0, eslint_devkit_3.getBasename)(context.getFilename());
+            const currentFile = (0, eslint_devkit_3.getBasename)(context.filename);
             const targetFile = cycle[1];
-            const relativeImport = (0, eslint_devkit_3.getRelativeImportPath)(context.getFilename(), targetFile);
+            const relativeImport = (0, eslint_devkit_3.getRelativeImportPath)(context.filename, targetFile);
             return {
                 cycle: (0, eslint_devkit_3.formatCycleDisplay)(cycle, workspaceRoot),
                 currentFile,
@@ -333,83 +344,93 @@ exports.noCycle = (0, eslint_devkit_1.createRule)({
                     };
             }
         }
-        // Store found cycles to report on specific imports
-        let detectedCycles = [];
         return {
-            Program() {
-                // Clear per-file state
-                detectedCycles = [];
-                // Find ALL circular dependencies starting from current file
-                const cycles = (0, eslint_devkit_4.findAllCircularDependencies)(filename, {
-                    maxDepth,
-                    reportAllCycles,
+            ImportDeclaration(node) {
+                const importSource = node.source.value;
+                // Resolve the import target to an absolute path
+                const resolved = (0, eslint_devkit_4.resolveImportPath)(importSource, {
+                    fromFile: filename,
                     workspaceRoot,
                     barrelExports,
                     cache: sharedCache,
                     resolverSettings,
                 });
-                if (cycles.length > 0) {
-                    for (const cycle of cycles) {
-                        // Extract the minimal cycle (the actual loop)
-                        const minimalCycle = (0, eslint_devkit_4.getMinimalCycle)(cycle);
-                        // Only report if current file is part of the minimal cycle
-                        // (not just a file that imports into the cycle)
-                        if (!minimalCycle.includes(filename)) {
-                            continue;
-                        }
-                        const cycleStart = minimalCycle.indexOf(filename);
-                        if (cycleStart === -1)
-                            continue;
-                        const relevantCycle = [
-                            ...minimalCycle.slice(cycleStart),
-                            ...minimalCycle.slice(0, cycleStart),
-                        ];
-                        const cycleHash = (0, eslint_devkit_4.getCycleHash)(relevantCycle);
-                        // Skip if we've already reported this cycle (shared across all files)
-                        if (sharedCache.reportedCycles.has(cycleHash)) {
-                            continue;
-                        }
-                        sharedCache.reportedCycles.add(cycleHash);
-                        // Select the appropriate fix strategy
-                        const strategy = selectFixStrategy(relevantCycle, fixStrategy);
-                        // Find the target of the first import that causes the cycle
-                        const cycleTarget = relevantCycle[1] || relevantCycle[0]; // The next file in the cycle
-                        detectedCycles.push({
-                            cycleTarget,
-                            strategy,
-                            relevantCycle,
-                            cycleHash,
-                        });
-                    }
+                if (!resolved)
+                    return;
+                // =====================================================================
+                // PER-IMPORT CYCLE DETECTION (Phase 3 optimization)
+                //
+                // Instead of computing all cycles upfront in Program() via full-graph
+                // BFS + Tarjan SCC (which reads ALL ~5K reachable files), we check
+                // each import individually with a targeted DFS.
+                //
+                // Performance advantage:
+                // - Only reads files along the actual DFS path (not all reachable)
+                // - nonCyclicFiles cache provides O(1) rejection after first visit
+                // - getFileImports cache reuses parsed results across files
+                // =====================================================================
+                // Fast path: if the target file is known to be non-cyclic, skip
+                if (sharedCache.nonCyclicFiles.has(resolved)) {
+                    return;
                 }
-            },
-            ImportDeclaration(node) {
-                // Check if this import is part of any detected circular dependency
-                const importSource = node.source.value;
-                const resolved = (0, eslint_devkit_4.resolveImportPath)(importSource, {
-                    fromFile: filename,
+                // Check if following this import leads back to the current file
+                const cycles = (0, eslint_devkit_4.detectCycleFromImport)(filename, resolved, {
+                    maxDepth,
+                    reportAllCycles,
                     workspaceRoot,
                     barrelExports,
                     cache: sharedCache,
                     resolverSettings,
                 });
-                if (!resolved)
+                if (cycles.length === 0)
                     return;
-                for (const cycle of detectedCycles) {
-                    if (resolved === cycle.cycleTarget) {
-                        // This is an import that causes a circular dependency
-                        const sourceImport = {
-                            path: resolved,
-                            source: importSource,
-                        };
-                        // Generate the appropriate message based on strategy
-                        const { messageId, data } = generateMessageData(cycle.relevantCycle, cycle.strategy, sourceImport);
-                        context.report({
-                            node,
-                            messageId,
-                            data,
-                        });
+                for (const cycle of cycles) {
+                    // Extract the minimal cycle (the actual loop)
+                    const minimalCycle = (0, eslint_devkit_4.getMinimalCycle)(cycle);
+                    // Only report if current file is part of the minimal cycle
+                    if (!minimalCycle.includes(filename)) {
+                        continue;
                     }
+                    const cycleStart = minimalCycle.indexOf(filename);
+                    if (cycleStart === -1)
+                        continue;
+                    const relevantCycle = [
+                        ...minimalCycle.slice(cycleStart),
+                        ...minimalCycle.slice(0, cycleStart),
+                    ];
+                    // Dedupe by (file, cycle) instead of cycle alone. Without the
+                    // file in the key, the cycle [A, B] reports only on whichever
+                    // of A or B is linted first; the other gets silently skipped.
+                    // Per-file keying matches oxlint's behavior — every file in a
+                    // cycle gets its own diagnostic. The user can navigate from any
+                    // entry point in the cycle and see the report there.
+                    const cycleHash = `${filename}::${(0, eslint_devkit_4.getCycleHash)(relevantCycle)}`;
+                    // Skip only if THIS file already reported THIS cycle (e.g. the
+                    // same import statement appearing twice in the same file).
+                    // (Earlier we experimented with a `pendingCycleReports` fan-out
+                    // that emits on every cycle member regardless of which one's
+                    // DFS discovered it — closes the with-router.tsx presentational
+                    // gap but caused a 10× slowdown and 23× finding-count explosion
+                    // because each cycle member iterates every other member of every
+                    // cycle it touches. The cycle IS still reported on its other
+                    // end; the one-finding loss isn't worth the perf hit.)
+                    if (sharedCache.reportedCycles.has(cycleHash)) {
+                        continue;
+                    }
+                    sharedCache.reportedCycles.add(cycleHash);
+                    // Select the appropriate fix strategy
+                    const strategy = selectFixStrategy(relevantCycle, fixStrategy);
+                    // Generate the appropriate message based on strategy
+                    const sourceImport = {
+                        path: resolved,
+                        source: importSource,
+                    };
+                    const { messageId, data } = generateMessageData(relevantCycle, strategy, sourceImport);
+                    context.report({
+                        node,
+                        messageId,
+                        data,
+                    });
                 }
             },
         };

package/src/rules/no-default-export.js

@@ -13,6 +13,7 @@ exports.noDefaultExport = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-default-export.md',
             description: 'Prevents default exports',
         },
         fixable: 'code',
@@ -80,7 +81,7 @@ exports.noDefaultExport = (0, eslint_devkit_1.createRule)({
     create(context) {
         const [options] = context.options;
         const { allowInFiles = [], allowPatterns = [], suggestNamed = true, } = options || {};
-        const filename = context.getFilename();
+        const filename = context.filename;
         function shouldAllow() {
             if (!filename) {
                 return false;

package/src/rules/no-deprecated.js

@@ -13,6 +13,7 @@ exports.noDeprecated = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-deprecated.md',
             description: 'Forbid imported names marked with @deprecated documentation tag',
         },
         hasSuggestions: true,
@@ -83,7 +84,7 @@ exports.noDeprecated = (0, eslint_devkit_1.createRule)({
     create(context) {
         const [options] = context.options;
         const { allow = [], deprecationMarkers = ['@deprecated'], checkJSDoc = true, checkDecorators = true, } = options || {};
-        const filename = context.getFilename();
+        const filename = context.filename;
         if (!filename) {
             return {};
         }