eslint-plugin-import-next 2.3.14 → 2.3.16

package/src/rules/no-cycle.js

@@ -1,396 +1,2 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.noCycle = void 0;
-exports.clearCircularDependencyCache = clearCircularDependencyCache;
-const eslint_devkit_1 = require("@interlace/eslint-devkit");
-const eslint_devkit_2 = require("@interlace/eslint-devkit");
-const eslint_devkit_3 = require("@interlace/eslint-devkit");
-const node_fs_1 = require("node:fs");
-const eslint_devkit_4 = require("@interlace/eslint-devkit");
-const sharedCache = (0, eslint_devkit_4.createFileSystemCache)();
-function clearCircularDependencyCache() {
-    (0, eslint_devkit_4.clearCache)(sharedCache);
-}
-exports.noCycle = (0, eslint_devkit_1.createRule)({
-    name: 'no-cycle',
-    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: {
-            moduleSplit: (0, eslint_devkit_2.formatLLMMessage)({
-                icon: eslint_devkit_2.MessageIcons.ARCHITECTURE,
-                issueName: 'Circular dependency',
-                cwe: 'CWE-407',
-                description: 'Circular dependency detected',
-                severity: 'CRITICAL',
-                fix: 'Split {{moduleToSplit}} into .{{coreFile}} and .{{extendedFile}}',
-                documentationLink: 'https://en.wikipedia.org/wiki/Circular_dependency',
-            }),
-            directImport: (0, eslint_devkit_2.formatLLMMessage)({
-                icon: eslint_devkit_2.MessageIcons.ARCHITECTURE,
-                issueName: 'Circular dependency',
-                cwe: 'CWE-407',
-                description: 'Circular dependency detected',
-                severity: 'MEDIUM',
-                fix: '{{newImport}} (direct imports preferred over barrel exports)',
-                documentationLink: 'https://en.wikipedia.org/wiki/Circular_dependency',
-            }),
-            extractShared: (0, eslint_devkit_2.formatLLMMessage)({
-                icon: eslint_devkit_2.MessageIcons.ARCHITECTURE,
-                issueName: 'Circular dependency',
-                cwe: 'CWE-407',
-                description: 'Circular dependency detected',
-                severity: 'MEDIUM',
-                fix: 'Extract shared types to {{exports}} file',
-                documentationLink: 'https://en.wikipedia.org/wiki/Dependency_inversion_principle',
-            }),
-            dependencyInjection: (0, eslint_devkit_2.formatLLMMessage)({
-                icon: eslint_devkit_2.MessageIcons.ARCHITECTURE,
-                issueName: 'Circular dependency',
-                cwe: 'CWE-407',
-                description: 'Circular dependency detected',
-                severity: 'MEDIUM',
-                fix: 'Use dependency injection pattern to break cycle',
-                documentationLink: 'https://en.wikipedia.org/wiki/Dependency_injection',
-            }),
-        },
-        schema: [
-            {
-                type: 'object',
-                properties: {
-                    maxDepth: {
-                        type: 'number',
-                        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',
-                        items: { type: 'string' },
-                        default: [
-                            '**/*.test.ts',
-                            '**/*.test.tsx',
-                            '**/*.spec.ts',
-                            '**/*.spec.tsx',
-                            '**/*.stories.tsx',
-                            '**/__tests__/**',
-                            '**/__mocks__/**',
-                        ],
-                        description: 'File patterns to ignore (glob patterns)',
-                    },
-                    barrelExports: {
-                        type: 'array',
-                        items: { type: 'string' },
-                        default: ['index.ts', 'index.tsx', 'index.js', 'index.jsx'],
-                        description: 'Files considered barrel exports',
-                    },
-                    reportAllCycles: {
-                        type: 'boolean',
-                        default: true,
-                        description: 'Report all circular dependencies found (not just the first one)',
-                    },
-                    fixStrategy: {
-                        type: 'string',
-                        enum: [
-                            'module-split',
-                            'direct-import',
-                            'extract-shared',
-                            'dependency-injection',
-                            'auto',
-                        ],
-                        default: 'auto',
-                        description: 'Strategy for fixing circular dependencies (auto = smart detection)',
-                    },
-                    moduleNamingConvention: {
-                        type: 'string',
-                        enum: ['semantic', 'numbered'],
-                        default: 'semantic',
-                        description: 'Naming convention for split modules (semantic: .core, .api | numbered: .1, .2)',
-                    },
-                    coreModuleSuffix: {
-                        type: 'string',
-                        default: 'core',
-                        description: 'Suffix for core module when splitting (e.g., "core", "base", "main")',
-                    },
-                    extendedModuleSuffix: {
-                        type: 'string',
-                        default: 'extended',
-                        description: 'Suffix for extended module when splitting (e.g., "extended", "api", "helpers")',
-                    },
-                },
-                additionalProperties: false,
-            },
-        ],
-        fixable: undefined,
-    },
-    defaultOptions: [
-        {
-            maxDepth: Infinity,
-            ignorePatterns: [],
-            barrelExports: ['index.ts', 'index.tsx', 'index.js', 'index.jsx'],
-            reportAllCycles: true,
-            fixStrategy: 'auto',
-            moduleNamingConvention: 'semantic',
-            coreModuleSuffix: 'core',
-            extendedModuleSuffix: 'extended',
-        },
-    ],
-    create(context) {
-        const options = context.options[0] || {};
-        const maxDepth = options.maxDepth ?? Infinity;
-        const ignorePatterns = options.ignorePatterns ?? [
-            '**/*.test.ts',
-            '**/*.test.tsx',
-            '**/*.spec.ts',
-            '**/*.spec.tsx',
-            '**/*.stories.tsx',
-            '**/__tests__/**',
-            '**/__mocks__/**',
-        ];
-        const barrelExports = options.barrelExports ?? [
-            'index.ts',
-            'index.tsx',
-            'index.js',
-            'index.jsx',
-        ];
-        const _reportAllCycles = options.reportAllCycles ?? true;
-        const fixStrategy = options.fixStrategy ?? 'auto';
-        const moduleNamingConvention = options.moduleNamingConvention ?? 'semantic';
-        const coreModuleSuffix = options.coreModuleSuffix ?? 'core';
-        const extendedModuleSuffix = options.extendedModuleSuffix ?? 'extended';
-        const filename = context.filename;
-        const normalizedFilename = (() => {
-            try {
-                return (0, node_fs_1.realpathSync)(filename);
-            }
-            catch {
-                return filename;
-            }
-        })();
-        const workspaceRoot = context.cwd;
-        const settings = context.settings;
-        const resolverSettings = settings?.['import/resolver'] ||
-            settings?.['import-next/resolver'];
-        if ((0, eslint_devkit_3.shouldIgnoreFile)(filename, ignorePatterns, sharedCache.compiledPatterns)) {
-            return {};
-        }
-        sharedCache.pendingCycleReports.clear();
-        const reportedImportNodes = new Set();
-        function selectFixStrategy(cycle, userStrategy) {
-            if (userStrategy !== 'auto') {
-                return userStrategy;
-            }
-            const hasBarrel = cycle.some((file) => (0, eslint_devkit_3.isBarrelExport)(file, barrelExports));
-            const typesOnly = (0, eslint_devkit_4.hasOnlyTypeImports)(cycle, sharedCache);
-            const cycleSize = cycle.length;
-            if (hasBarrel && cycleSize === 2) {
-                return 'direct-import';
-            }
-            if (typesOnly) {
-                return 'extract-shared';
-            }
-            return 'module-split';
-        }
-        function generateModuleSplitMessage(cycle) {
-            const moduleNames = (0, eslint_devkit_3.getModuleNames)(cycle, workspaceRoot);
-            const [module1, module2] = moduleNames;
-            const moduleToSplit = module1;
-            const suffix1 = moduleNamingConvention === 'semantic' ? `.${coreModuleSuffix}` : '.1';
-            const suffix2 = moduleNamingConvention === 'semantic'
-                ? `.${extendedModuleSuffix}`
-                : '.2';
-            return {
-                cycle: (0, eslint_devkit_3.formatCycleDisplay)(cycle, workspaceRoot),
-                moduleToSplit,
-                coreFile: coreModuleSuffix,
-                extendedFile: extendedModuleSuffix,
-                splitCount: '2',
-                fileStructure: `├─ ${moduleToSplit}/${moduleToSplit}${suffix1}.ts (→ ${module2} ✓)\n` +
-                    `└─ ${moduleToSplit}/${moduleToSplit}${suffix2}.ts (→ ${suffix1} + ${module2} ✓)`,
-                result: `${suffix1} → ${module2} → ${suffix2} (no cycle)`,
-            };
-        }
-        function generateDirectImportMessage(cycle, sourceImport) {
-            const currentFile = (0, eslint_devkit_3.getBasename)(context.filename);
-            const targetFile = cycle[1];
-            const relativeImport = (0, eslint_devkit_3.getRelativeImportPath)(context.filename, targetFile);
-            return {
-                cycle: (0, eslint_devkit_3.formatCycleDisplay)(cycle, workspaceRoot),
-                currentFile,
-                oldImport: `import { ... } from '${sourceImport.source}'`,
-                newImport: `import { ... } from '${relativeImport}'`,
-            };
-        }
-        function generateExtractSharedMessage(cycle) {
-            const moduleNames = (0, eslint_devkit_3.getModuleNames)(cycle, workspaceRoot);
-            const [module1, module2] = moduleNames;
-            const exports = [
-                `- export type ${module1}Id, ${module2}Id`,
-                `- export interface ${module1}Summary, ${module2}Summary`,
-            ].join('\n');
-            return {
-                cycle: (0, eslint_devkit_3.formatCycleDisplay)(cycle, workspaceRoot),
-                exports,
-                result: `shared/types ← ${module1}, ${module2} (no cycle)`,
-            };
-        }
-        function generateDependencyInjectionMessage(cycle) {
-            const moduleNames = (0, eslint_devkit_3.getModuleNames)(cycle, workspaceRoot);
-            const [service1, service2] = moduleNames;
-            const steps = [
-                `1. Create interfaces/I${service1}.ts, interfaces/I${service2}.ts`,
-                `2. Both services implement their interface`,
-                `3. Inject via constructor: constructor(private dep?: IDep)`,
-                `4. Wire in container.ts`,
-            ].join('\n');
-            return {
-                cycle: (0, eslint_devkit_3.formatCycleDisplay)(cycle, workspaceRoot),
-                steps,
-            };
-        }
-        function generateMessageData(cycle, strategy, sourceImport) {
-            switch (strategy) {
-                case 'direct-import':
-                    return {
-                        messageId: 'directImport',
-                        data: generateDirectImportMessage(cycle, sourceImport),
-                    };
-                case 'extract-shared':
-                    return {
-                        messageId: 'extractShared',
-                        data: generateExtractSharedMessage(cycle),
-                    };
-                case 'dependency-injection':
-                    return {
-                        messageId: 'dependencyInjection',
-                        data: generateDependencyInjectionMessage(cycle),
-                    };
-                default:
-                    return {
-                        messageId: 'moduleSplit',
-                        data: generateModuleSplitMessage(cycle),
-                    };
-            }
-        }
-        return {
-            'Program'() {
-            },
-            ImportDeclaration(node) {
-                if (node.importKind === 'type')
-                    return;
-                if (reportedImportNodes.has(node))
-                    return;
-                const importSource = node.source.value;
-                const resolved = (0, eslint_devkit_4.resolveImportPath)(importSource, {
-                    fromFile: normalizedFilename,
-                    workspaceRoot,
-                    barrelExports,
-                    cache: sharedCache,
-                    resolverSettings,
-                });
-                if (!resolved)
-                    return;
-                if (sharedCache.nonCyclicFiles.has(resolved))
-                    return;
-                let tgtSCC = sharedCache.sccIndex.get(resolved);
-                if (tgtSCC === undefined) {
-                    (0, eslint_devkit_4.computeSCCsFromFile)(resolved, {
-                        maxDepth: Infinity,
-                        workspaceRoot,
-                        barrelExports,
-                        cache: sharedCache,
-                        resolverSettings,
-                    });
-                    tgtSCC = sharedCache.sccIndex.get(resolved);
-                }
-                let srcSCC = sharedCache.sccIndex.get(normalizedFilename);
-                if (srcSCC === undefined) {
-                    (0, eslint_devkit_4.computeSCCsFromFile)(normalizedFilename, {
-                        maxDepth: Infinity,
-                        workspaceRoot,
-                        barrelExports,
-                        cache: sharedCache,
-                        resolverSettings,
-                    });
-                    srcSCC = sharedCache.sccIndex.get(normalizedFilename);
-                }
-                if (srcSCC === undefined || tgtSCC === undefined || srcSCC !== tgtSCC) {
-                    return;
-                }
-                const cyclePath = (0, eslint_devkit_4.findShortestCyclePath)(normalizedFilename, resolved, {
-                    maxDepth,
-                    workspaceRoot,
-                    barrelExports,
-                    cache: sharedCache,
-                    resolverSettings,
-                });
-                if (!cyclePath)
-                    return;
-                reportedImportNodes.add(node);
-                const cycleStart = cyclePath.indexOf(normalizedFilename);
-                const relevantCycle = cycleStart >= 0
-                    ? [...cyclePath.slice(cycleStart), ...cyclePath.slice(0, cycleStart)]
-                    : cyclePath;
-                const strategy = selectFixStrategy(relevantCycle, fixStrategy);
-                const sourceImport = { path: resolved, source: importSource };
-                const { messageId, data } = generateMessageData(relevantCycle, strategy, sourceImport);
-                context.report({ node, messageId, data });
-            },
-            ExportNamedDeclaration(node) {
-                if (!node.source)
-                    return;
-                if (reportedImportNodes.has(node))
-                    return;
-                const importSource = node.source.value;
-                const resolved = (0, eslint_devkit_4.resolveImportPath)(importSource, {
-                    fromFile: normalizedFilename,
-                    workspaceRoot,
-                    barrelExports,
-                    cache: sharedCache,
-                    resolverSettings,
-                });
-                if (!resolved)
-                    return;
-                if (sharedCache.nonCyclicFiles.has(resolved))
-                    return;
-                let tgtSCC = sharedCache.sccIndex.get(resolved);
-                if (tgtSCC === undefined) {
-                    (0, eslint_devkit_4.computeSCCsFromFile)(resolved, {
-                        maxDepth: Infinity,
-                        workspaceRoot,
-                        barrelExports,
-                        cache: sharedCache,
-                        resolverSettings,
-                    });
-                    tgtSCC = sharedCache.sccIndex.get(resolved);
-                }
-                const srcSCC = sharedCache.sccIndex.get(normalizedFilename);
-                if (srcSCC === undefined || tgtSCC === undefined || srcSCC !== tgtSCC)
-                    return;
-                const cyclePath = (0, eslint_devkit_4.findShortestCyclePath)(normalizedFilename, resolved, {
-                    maxDepth,
-                    workspaceRoot,
-                    barrelExports,
-                    cache: sharedCache,
-                    resolverSettings,
-                });
-                if (!cyclePath)
-                    return;
-                reportedImportNodes.add(node);
-                const cycleStart = cyclePath.indexOf(normalizedFilename);
-                const relevantCycle = cycleStart >= 0
-                    ? [...cyclePath.slice(cycleStart), ...cyclePath.slice(0, cycleStart)]
-                    : cyclePath;
-                const strategy = selectFixStrategy(relevantCycle, fixStrategy);
-                const sourceImport = { path: resolved, source: importSource };
-                const { messageId, data } = generateMessageData(relevantCycle, strategy, sourceImport);
-                context.report({ node, messageId, data });
-            },
-        };
-    },
-});
+"use strict";Object.defineProperty(exports,"__esModule",{value:true});exports.noCycle=void 0;exports.clearCircularDependencyCache=clearCircularDependencyCache;const eslint_devkit_1=require("@interlace/eslint-devkit");const eslint_devkit_2=require("@interlace/eslint-devkit");const eslint_devkit_3=require("@interlace/eslint-devkit");const node_fs_1=require("node:fs");const eslint_devkit_4=require("@interlace/eslint-devkit");const sharedCache=(0,eslint_devkit_4.createFileSystemCache)();function clearCircularDependencyCache(){(0,eslint_devkit_4.clearCache)(sharedCache)}exports.noCycle=(0,eslint_devkit_1.createRule)({name:"no-cycle",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:{moduleSplit:(0,eslint_devkit_2.formatLLMMessage)({icon:eslint_devkit_2.MessageIcons.ARCHITECTURE,issueName:"Circular dependency",cwe:"CWE-407",description:"Circular dependency detected",severity:"CRITICAL",fix:"Split {{moduleToSplit}} into .{{coreFile}} and .{{extendedFile}}",documentationLink:"https://en.wikipedia.org/wiki/Circular_dependency"}),directImport:(0,eslint_devkit_2.formatLLMMessage)({icon:eslint_devkit_2.MessageIcons.ARCHITECTURE,issueName:"Circular dependency",cwe:"CWE-407",description:"Circular dependency detected",severity:"MEDIUM",fix:"{{newImport}} (direct imports preferred over barrel exports)",documentationLink:"https://en.wikipedia.org/wiki/Circular_dependency"}),extractShared:(0,eslint_devkit_2.formatLLMMessage)({icon:eslint_devkit_2.MessageIcons.ARCHITECTURE,issueName:"Circular dependency",cwe:"CWE-407",description:"Circular dependency detected",severity:"MEDIUM",fix:"Extract shared types to {{exports}} file",documentationLink:"https://en.wikipedia.org/wiki/Dependency_inversion_principle"}),dependencyInjection:(0,eslint_devkit_2.formatLLMMessage)({icon:eslint_devkit_2.MessageIcons.ARCHITECTURE,issueName:"Circular dependency",cwe:"CWE-407",description:"Circular dependency detected",severity:"MEDIUM",fix:"Use dependency injection pattern to break cycle",documentationLink:"https://en.wikipedia.org/wiki/Dependency_injection"})},schema:[{type:"object",properties:{maxDepth:{type:"number",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 \u2014 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",items:{type:"string"},default:["**/*.test.ts","**/*.test.tsx","**/*.spec.ts","**/*.spec.tsx","**/*.stories.tsx","**/__tests__/**","**/__mocks__/**"],description:"File patterns to ignore (glob patterns)"},barrelExports:{type:"array",items:{type:"string"},default:["index.ts","index.tsx","index.js","index.jsx"],description:"Files considered barrel exports"},reportAllCycles:{type:"boolean",default:true,description:"Report all circular dependencies found (not just the first one)"},fixStrategy:{type:"string",enum:["module-split","direct-import","extract-shared","dependency-injection","auto"],default:"auto",description:"Strategy for fixing circular dependencies (auto = smart detection)"},moduleNamingConvention:{type:"string",enum:["semantic","numbered"],default:"semantic",description:"Naming convention for split modules (semantic: .core, .api | numbered: .1, .2)"},coreModuleSuffix:{type:"string",default:"core",description:'Suffix for core module when splitting (e.g., "core", "base", "main")'},extendedModuleSuffix:{type:"string",default:"extended",description:'Suffix for extended module when splitting (e.g., "extended", "api", "helpers")'}},additionalProperties:false}],fixable:void 0},defaultOptions:[{maxDepth:Infinity,ignorePatterns:[],barrelExports:["index.ts","index.tsx","index.js","index.jsx"],reportAllCycles:true,fixStrategy:"auto",moduleNamingConvention:"semantic",coreModuleSuffix:"core",extendedModuleSuffix:"extended"}],create(context){const options=context.options[0]||{};const maxDepth=options.maxDepth??Infinity;const ignorePatterns=options.ignorePatterns??["**/*.test.ts","**/*.test.tsx","**/*.spec.ts","**/*.spec.tsx","**/*.stories.tsx","**/__tests__/**","**/__mocks__/**"];const barrelExports=options.barrelExports??["index.ts","index.tsx","index.js","index.jsx"];const _reportAllCycles=options.reportAllCycles??true;const fixStrategy=options.fixStrategy??"auto";const moduleNamingConvention=options.moduleNamingConvention??"semantic";const coreModuleSuffix=options.coreModuleSuffix??"core";const extendedModuleSuffix=options.extendedModuleSuffix??"extended";const filename=context.filename;const normalizedFilename=(()=>{try{return(0,node_fs_1.realpathSync)(filename)}catch{return filename}})();const workspaceRoot=context.cwd;const settings=context.settings;const resolverSettings=settings?.["import/resolver"]||settings?.["import-next/resolver"];if((0,eslint_devkit_3.shouldIgnoreFile)(filename,ignorePatterns,sharedCache.compiledPatterns)){return{}}sharedCache.pendingCycleReports.clear();const reportedImportNodes=new Set;function selectFixStrategy(cycle,userStrategy){if(userStrategy!=="auto"){return userStrategy}const hasBarrel=cycle.some(file=>(0,eslint_devkit_3.isBarrelExport)(file,barrelExports));const typesOnly=(0,eslint_devkit_4.hasOnlyTypeImports)(cycle,sharedCache);const cycleSize=cycle.length;if(hasBarrel&&cycleSize===2){return"direct-import"}if(typesOnly){return"extract-shared"}return"module-split"}function generateModuleSplitMessage(cycle){const moduleNames=(0,eslint_devkit_3.getModuleNames)(cycle,workspaceRoot);const[module1,module2]=moduleNames;const moduleToSplit=module1;const suffix1=moduleNamingConvention==="semantic"?`.${coreModuleSuffix}`:".1";const suffix2=moduleNamingConvention==="semantic"?`.${extendedModuleSuffix}`:".2";return{cycle:(0,eslint_devkit_3.formatCycleDisplay)(cycle,workspaceRoot),moduleToSplit,coreFile:coreModuleSuffix,extendedFile:extendedModuleSuffix,splitCount:"2",fileStructure:`\u251C\u2500 ${moduleToSplit}/${moduleToSplit}${suffix1}.ts (\u2192 ${module2} \u2713)
+\u2514\u2500 ${moduleToSplit}/${moduleToSplit}${suffix2}.ts (\u2192 ${suffix1} + ${module2} \u2713)`,result:`${suffix1} \u2192 ${module2} \u2192 ${suffix2} (no cycle)`}}function generateDirectImportMessage(cycle,sourceImport){const currentFile=(0,eslint_devkit_3.getBasename)(context.filename);const targetFile=cycle[1];const relativeImport=(0,eslint_devkit_3.getRelativeImportPath)(context.filename,targetFile);return{cycle:(0,eslint_devkit_3.formatCycleDisplay)(cycle,workspaceRoot),currentFile,oldImport:`import { ... } from '${sourceImport.source}'`,newImport:`import { ... } from '${relativeImport}'`}}function generateExtractSharedMessage(cycle){const moduleNames=(0,eslint_devkit_3.getModuleNames)(cycle,workspaceRoot);const[module1,module2]=moduleNames;const exports2=[`- export type ${module1}Id, ${module2}Id`,`- export interface ${module1}Summary, ${module2}Summary`].join("\n");return{cycle:(0,eslint_devkit_3.formatCycleDisplay)(cycle,workspaceRoot),exports:exports2,result:`shared/types \u2190 ${module1}, ${module2} (no cycle)`}}function generateDependencyInjectionMessage(cycle){const moduleNames=(0,eslint_devkit_3.getModuleNames)(cycle,workspaceRoot);const[service1,service2]=moduleNames;const steps=[`1. Create interfaces/I${service1}.ts, interfaces/I${service2}.ts`,`2. Both services implement their interface`,`3. Inject via constructor: constructor(private dep?: IDep)`,`4. Wire in container.ts`].join("\n");return{cycle:(0,eslint_devkit_3.formatCycleDisplay)(cycle,workspaceRoot),steps}}function generateMessageData(cycle,strategy,sourceImport){switch(strategy){case"direct-import":return{messageId:"directImport",data:generateDirectImportMessage(cycle,sourceImport)};case"extract-shared":return{messageId:"extractShared",data:generateExtractSharedMessage(cycle)};case"dependency-injection":return{messageId:"dependencyInjection",data:generateDependencyInjectionMessage(cycle)};default:return{messageId:"moduleSplit",data:generateModuleSplitMessage(cycle)}}}return{"Program"(){},ImportDeclaration(node){if(node.importKind==="type")return;if(reportedImportNodes.has(node))return;const importSource=node.source.value;const resolved=(0,eslint_devkit_4.resolveImportPath)(importSource,{fromFile:normalizedFilename,workspaceRoot,barrelExports,cache:sharedCache,resolverSettings});if(!resolved)return;if(sharedCache.nonCyclicFiles.has(resolved))return;let tgtSCC=sharedCache.sccIndex.get(resolved);if(tgtSCC===void 0){(0,eslint_devkit_4.computeSCCsFromFile)(resolved,{maxDepth:Infinity,workspaceRoot,barrelExports,cache:sharedCache,resolverSettings});tgtSCC=sharedCache.sccIndex.get(resolved)}let srcSCC=sharedCache.sccIndex.get(normalizedFilename);if(srcSCC===void 0){(0,eslint_devkit_4.computeSCCsFromFile)(normalizedFilename,{maxDepth:Infinity,workspaceRoot,barrelExports,cache:sharedCache,resolverSettings});srcSCC=sharedCache.sccIndex.get(normalizedFilename)}if(srcSCC===void 0||tgtSCC===void 0||srcSCC!==tgtSCC){return}const cyclePath=(0,eslint_devkit_4.findShortestCyclePath)(normalizedFilename,resolved,{maxDepth,workspaceRoot,barrelExports,cache:sharedCache,resolverSettings});if(!cyclePath)return;reportedImportNodes.add(node);const cycleStart=cyclePath.indexOf(normalizedFilename);const relevantCycle=cycleStart>=0?[...cyclePath.slice(cycleStart),...cyclePath.slice(0,cycleStart)]:cyclePath;const strategy=selectFixStrategy(relevantCycle,fixStrategy);const sourceImport={path:resolved,source:importSource};const{messageId,data}=generateMessageData(relevantCycle,strategy,sourceImport);context.report({node,messageId,data})},ExportNamedDeclaration(node){if(!node.source)return;if(reportedImportNodes.has(node))return;const importSource=node.source.value;const resolved=(0,eslint_devkit_4.resolveImportPath)(importSource,{fromFile:normalizedFilename,workspaceRoot,barrelExports,cache:sharedCache,resolverSettings});if(!resolved)return;if(sharedCache.nonCyclicFiles.has(resolved))return;let tgtSCC=sharedCache.sccIndex.get(resolved);if(tgtSCC===void 0){(0,eslint_devkit_4.computeSCCsFromFile)(resolved,{maxDepth:Infinity,workspaceRoot,barrelExports,cache:sharedCache,resolverSettings});tgtSCC=sharedCache.sccIndex.get(resolved)}const srcSCC=sharedCache.sccIndex.get(normalizedFilename);if(srcSCC===void 0||tgtSCC===void 0||srcSCC!==tgtSCC)return;const cyclePath=(0,eslint_devkit_4.findShortestCyclePath)(normalizedFilename,resolved,{maxDepth,workspaceRoot,barrelExports,cache:sharedCache,resolverSettings});if(!cyclePath)return;reportedImportNodes.add(node);const cycleStart=cyclePath.indexOf(normalizedFilename);const relevantCycle=cycleStart>=0?[...cyclePath.slice(cycleStart),...cyclePath.slice(0,cycleStart)]:cyclePath;const strategy=selectFixStrategy(relevantCycle,fixStrategy);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

@@ -1,213 +1,2 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.noDefaultExport = void 0;
-const eslint_devkit_1 = require("@interlace/eslint-devkit");
-const eslint_devkit_2 = require("@interlace/eslint-devkit");
-exports.noDefaultExport = (0, eslint_devkit_1.createRule)({
-    name: 'no-default-export',
-    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',
-        messages: {
-            defaultExport: (0, eslint_devkit_2.formatLLMMessage)({
-                icon: eslint_devkit_2.MessageIcons.ARCHITECTURE,
-                issueName: 'Default Export Detected',
-                description: 'Default export violates import style guidelines',
-                severity: 'MEDIUM',
-                fix: 'Use named export instead of default export',
-                documentationLink: 'https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-default-export.md',
-            }),
-            preferNamedExport: (0, eslint_devkit_2.formatLLMMessage)({
-                icon: eslint_devkit_2.MessageIcons.ARCHITECTURE,
-                issueName: 'Prefer Named Exports',
-                description: 'Named exports provide better tree-shaking and refactoring',
-                severity: 'LOW',
-                fix: 'Convert default export to named export',
-                documentationLink: 'https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-default-export.md',
-            }),
-            defaultExportRefactor: (0, eslint_devkit_2.formatLLMMessage)({
-                icon: eslint_devkit_2.MessageIcons.ARCHITECTURE,
-                issueName: 'Export Style Violation',
-                description: 'Default export pattern not allowed in this codebase',
-                severity: 'MEDIUM',
-                fix: 'Refactor to use named exports consistently',
-                documentationLink: 'https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-default-export.md',
-            }),
-        },
-        schema: [
-            {
-                type: 'object',
-                properties: {
-                    allowInFiles: {
-                        type: 'array',
-                        items: {
-                            type: 'string',
-                        },
-                        description: 'Allow default exports in specific files.',
-                    },
-                    allowPatterns: {
-                        type: 'array',
-                        items: {
-                            type: 'string',
-                        },
-                        description: 'Allow default exports for specific patterns.',
-                    },
-                    suggestNamed: {
-                        type: 'boolean',
-                        default: true,
-                        description: 'Suggest named export alternative.',
-                    },
-                },
-                additionalProperties: false,
-            },
-        ],
-    },
-    defaultOptions: [
-        {
-            allowInFiles: [],
-            allowPatterns: [],
-            suggestNamed: true,
-        },
-    ],
-    create(context) {
-        const [options] = context.options;
-        const { allowInFiles = [], allowPatterns = [], suggestNamed = true, } = options || {};
-        const filename = context.filename;
-        function shouldAllow() {
-            if (!filename) {
-                return false;
-            }
-            const allowedByFile = allowInFiles.some((pattern) => {
-                if (pattern.includes('*')) {
-                    const regex = new RegExp(pattern.replace(/\*/g, '.*'));
-                    return regex.test(filename);
-                }
-                return filename.includes(pattern);
-            });
-            if (allowedByFile) {
-                return true;
-            }
-            return allowPatterns.some((pattern) => {
-                if (pattern.includes('*')) {
-                    const regex = new RegExp(pattern.replace(/\*/g, '.*'));
-                    return regex.test(filename);
-                }
-                return filename.includes(pattern);
-            });
-        }
-        function generateNamedExportSuggestion(node) {
-            if (!suggestNamed) {
-                return '';
-            }
-            const declaration = node.declaration;
-            let suggestedName = 'exportedValue';
-            if (declaration.type === 'Identifier') {
-                suggestedName = declaration.name;
-            }
-            else if (declaration.type === 'FunctionDeclaration' && declaration.id) {
-                suggestedName = declaration.id.name;
-            }
-            else if (declaration.type === 'ClassDeclaration' && declaration.id) {
-                suggestedName = declaration.id.name;
-            }
-            else if (declaration.type === 'FunctionDeclaration' &&
-                !declaration.id) {
-                suggestedName = 'defaultFunction';
-            }
-            else if (declaration.type === 'ClassDeclaration' && !declaration.id) {
-                suggestedName = 'DefaultClass';
-            }
-            return `// Convert to: export { ${suggestedName} };`;
-        }
-        return {
-            ExportDefaultDeclaration(node) {
-                if (shouldAllow()) {
-                    return;
-                }
-                const suggestion = generateNamedExportSuggestion(node);
-                context.report({
-                    node,
-                    messageId: 'defaultExport',
-                    data: {
-                        currentFile: filename,
-                        suggestion: 'Use named export for better tree-shaking',
-                        example: suggestion,
-                    },
-                    fix(fixer) {
-                        const declaration = node.declaration;
-                        const sourceCode = context.sourceCode;
-                        if (declaration.type === 'Identifier') {
-                            return fixer.replaceText(node, `export { ${declaration.name} };`);
-                        }
-                        else if (declaration.type === 'Literal' ||
-                            declaration.type === 'ObjectExpression' ||
-                            declaration.type === 'ArrayExpression' ||
-                            declaration.type === 'ArrowFunctionExpression' ||
-                            declaration.type === 'CallExpression' ||
-                            declaration.type === 'NewExpression') {
-                            const exprText = sourceCode.getText(declaration);
-                            const namedExport = `const defaultExport = ${exprText};\nexport { defaultExport };`;
-                            return fixer.replaceText(node, namedExport);
-                        }
-                        else {
-                            const nodeText = sourceCode.getText(node);
-                            const namedExport = nodeText.replace(/^export default/, 'export');
-                            return fixer.replaceText(node, namedExport);
-                        }
-                    },
-                });
-            },
-            ExportNamedDeclaration(node) {
-                if (shouldAllow()) {
-                    return;
-                }
-                if (node.specifiers) {
-                    const hasDefaultSpecifier = node.specifiers.some((spec) => (spec.exported.type === 'Identifier' &&
-                        spec.exported.name === 'default') ||
-                        (spec.local &&
-                            spec.local.type === 'Identifier' &&
-                            spec.local.name === 'default'));
-                    if (hasDefaultSpecifier) {
-                        context.report({
-                            node,
-                            messageId: 'preferNamedExport',
-                            data: {
-                                pattern: 'export { default }',
-                                currentFile: filename,
-                                suggestion: 'Use named re-export instead',
-                            },
-                        });
-                    }
-                }
-            },
-            TSExportAssignment(node) {
-                if (shouldAllow()) {
-                    return;
-                }
-                context.report({
-                    node,
-                    messageId: 'defaultExportRefactor',
-                    data: {
-                        pattern: 'export = value',
-                        currentFile: filename,
-                        suggestion: 'Use ES6 named exports',
-                    },
-                });
-            },
-            TSInterfaceDeclaration(node) {
-                if (node.parent && node.parent.type === 'ExportDefaultDeclaration') {
-                    return;
-                }
-            },
-            TSTypeAliasDeclaration(node) {
-                if (node.parent && node.parent.type === eslint_devkit_1.AST_NODE_TYPES.ExportDefaultDeclaration) {
-                    return;
-                }
-            },
-        };
-    },
-});
+"use strict";Object.defineProperty(exports,"__esModule",{value:true});exports.noDefaultExport=void 0;const eslint_devkit_1=require("@interlace/eslint-devkit");const eslint_devkit_2=require("@interlace/eslint-devkit");exports.noDefaultExport=(0,eslint_devkit_1.createRule)({name:"no-default-export",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",messages:{defaultExport:(0,eslint_devkit_2.formatLLMMessage)({icon:eslint_devkit_2.MessageIcons.ARCHITECTURE,issueName:"Default Export Detected",description:"Default export violates import style guidelines",severity:"MEDIUM",fix:"Use named export instead of default export",documentationLink:"https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-default-export.md"}),preferNamedExport:(0,eslint_devkit_2.formatLLMMessage)({icon:eslint_devkit_2.MessageIcons.ARCHITECTURE,issueName:"Prefer Named Exports",description:"Named exports provide better tree-shaking and refactoring",severity:"LOW",fix:"Convert default export to named export",documentationLink:"https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-default-export.md"}),defaultExportRefactor:(0,eslint_devkit_2.formatLLMMessage)({icon:eslint_devkit_2.MessageIcons.ARCHITECTURE,issueName:"Export Style Violation",description:"Default export pattern not allowed in this codebase",severity:"MEDIUM",fix:"Refactor to use named exports consistently",documentationLink:"https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-default-export.md"})},schema:[{type:"object",properties:{allowInFiles:{type:"array",items:{type:"string"},description:"Allow default exports in specific files."},allowPatterns:{type:"array",items:{type:"string"},description:"Allow default exports for specific patterns."},suggestNamed:{type:"boolean",default:true,description:"Suggest named export alternative."}},additionalProperties:false}]},defaultOptions:[{allowInFiles:[],allowPatterns:[],suggestNamed:true}],create(context){const[options]=context.options;const{allowInFiles=[],allowPatterns=[],suggestNamed=true}=options||{};const filename=context.filename;function shouldAllow(){if(!filename){return false}const allowedByFile=allowInFiles.some(pattern=>{if(pattern.includes("*")){const regex=new RegExp(pattern.replace(/\*/g,".*"));return regex.test(filename)}return filename.includes(pattern)});if(allowedByFile){return true}return allowPatterns.some(pattern=>{if(pattern.includes("*")){const regex=new RegExp(pattern.replace(/\*/g,".*"));return regex.test(filename)}return filename.includes(pattern)})}function generateNamedExportSuggestion(node){if(!suggestNamed){return""}const declaration=node.declaration;let suggestedName="exportedValue";if(declaration.type==="Identifier"){suggestedName=declaration.name}else if(declaration.type==="FunctionDeclaration"&&declaration.id){suggestedName=declaration.id.name}else if(declaration.type==="ClassDeclaration"&&declaration.id){suggestedName=declaration.id.name}else if(declaration.type==="FunctionDeclaration"&&!declaration.id){suggestedName="defaultFunction"}else if(declaration.type==="ClassDeclaration"&&!declaration.id){suggestedName="DefaultClass"}return`// Convert to: export { ${suggestedName} };`}return{ExportDefaultDeclaration(node){if(shouldAllow()){return}const suggestion=generateNamedExportSuggestion(node);context.report({node,messageId:"defaultExport",data:{currentFile:filename,suggestion:"Use named export for better tree-shaking",example:suggestion},fix(fixer){const declaration=node.declaration;const sourceCode=context.sourceCode;if(declaration.type==="Identifier"){return fixer.replaceText(node,`export { ${declaration.name} };`)}else if(declaration.type==="Literal"||declaration.type==="ObjectExpression"||declaration.type==="ArrayExpression"||declaration.type==="ArrowFunctionExpression"||declaration.type==="CallExpression"||declaration.type==="NewExpression"){const exprText=sourceCode.getText(declaration);const namedExport=`const defaultExport = ${exprText};
+export { defaultExport };`;return fixer.replaceText(node,namedExport)}else{const nodeText=sourceCode.getText(node);const namedExport=nodeText.replace(/^export default/,"export");return fixer.replaceText(node,namedExport)}}})},ExportNamedDeclaration(node){if(shouldAllow()){return}if(node.specifiers){const hasDefaultSpecifier=node.specifiers.some(spec=>spec.exported.type==="Identifier"&&spec.exported.name==="default"||spec.local&&spec.local.type==="Identifier"&&spec.local.name==="default");if(hasDefaultSpecifier){context.report({node,messageId:"preferNamedExport",data:{pattern:"export { default }",currentFile:filename,suggestion:"Use named re-export instead"}})}}},TSExportAssignment(node){if(shouldAllow()){return}context.report({node,messageId:"defaultExportRefactor",data:{pattern:"export = value",currentFile:filename,suggestion:"Use ES6 named exports"}})},TSInterfaceDeclaration(node){if(node.parent&&node.parent.type==="ExportDefaultDeclaration"){return}},TSTypeAliasDeclaration(node){if(node.parent&&node.parent.type===eslint_devkit_1.AST_NODE_TYPES.ExportDefaultDeclaration){return}}}}});