eslint-plugin-import-next 2.3.9 → 2.3.11

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

@@ -1,29 +1,17 @@
 "use strict";
-/**
- * Copyright (c) 2025 Ofri Peretz
- * Licensed under the MIT License. Use of this source code is governed by the
- * MIT license that can be found in the LICENSE file.
- */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.enforceDependencyDirection = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
 const eslint_devkit_2 = require("@interlace/eslint-devkit");
 const eslint_devkit_3 = require("@interlace/eslint-devkit");
-/**
- * Check if two arrays are equal
- */
 function arraysEqual(a, b) {
     if (a.length !== b.length)
         return false;
     return a.every((val, index) => val === b[index]);
 }
-/**
- * Extract layer from file path
- */
 function extractLayer(filePath, layerPatterns) {
     const normalizedPath = (0, eslint_devkit_3.normalizePath)(filePath);
     for (const pattern of layerPatterns) {
-        // Check for layer in path segments (handles both /layer/ and layer/ at start)
         if (normalizedPath.includes(`/${pattern}/`) ||
             normalizedPath.includes(`\\${pattern}\\`) ||
             normalizedPath.startsWith(`${pattern}/`) ||
@@ -33,12 +21,9 @@ function extractLayer(filePath, layerPatterns) {
     }
     return null;
 }
-/**
- * Get layer order index
- */
 function getLayerOrder(layer, layers) {
     if (!layer) {
-        return -1; // Unknown layer
+        return -1;
     }
     const index = layers.indexOf(layer);
     return index >= 0 ? index : -1;
@@ -132,14 +117,12 @@ exports.enforceDependencyDirection = (0, eslint_devkit_2.createRule)({
     create(context, [options = {}]) {
         const filename = context.filename;
         const { layers: providedLayers, layerPatterns: providedLayerPatterns, allowSameLayer = true, } = options || {};
-        // Use provided layers or defaults
         const layers = providedLayers || [
             'domain',
             'application',
             'infrastructure',
             'presentation',
         ];
-        // Default layer patterns
         const defaultLayerPatterns = [
             'domain',
             'application',
@@ -148,26 +131,19 @@ exports.enforceDependencyDirection = (0, eslint_devkit_2.createRule)({
             'ui',
             'api',
         ];
-        // Use provided layerPatterns, or if layers were provided but layerPatterns weren't (i.e., layerPatterns is still default), use layers
         let layerPatterns;
         if (providedLayerPatterns &&
             !arraysEqual(providedLayerPatterns, defaultLayerPatterns)) {
-            // layerPatterns was explicitly provided and is different from default
             layerPatterns = providedLayerPatterns;
         }
         else if (providedLayers) {
-            // layers were provided but layerPatterns weren't, so use layers
             layerPatterns = providedLayers;
         }
         else {
-            // neither were provided, use defaults
             layerPatterns = defaultLayerPatterns;
         }
         const sourceLayer = extractLayer(filename, layerPatterns);
         const sourceOrder = getLayerOrder(sourceLayer, layers);
-        /**
-         * Check import declarations
-         */
         function checkImport(node) {
             let importPath = '';
             if (node.type === 'ImportDeclaration') {
@@ -178,19 +154,13 @@ exports.enforceDependencyDirection = (0, eslint_devkit_2.createRule)({
                     importPath = node.source.value;
                 }
                 else {
-                    return; // Dynamic import
+                    return;
                 }
             }
-            // Only check relative imports
             if (!importPath.startsWith('.')) {
                 return;
             }
-            // For relative imports, check the import path string directly for layer patterns
-            // This works for test cases where paths like '../infrastructure/repository' contain the layer
-            // The early return above guarantees importPath starts with '.' here.
             let targetLayer = null;
-            // Check if import path contains layer patterns
-            // e.g., '../infrastructure/repository' -> extract 'infrastructure'
             const segments = importPath.split(/[/\\]/);
             for (const segment of segments) {
                 if (layerPatterns.includes(segment.toLowerCase())) {
@@ -198,7 +168,6 @@ exports.enforceDependencyDirection = (0, eslint_devkit_2.createRule)({
                     break;
                 }
             }
-            // Fallback: try regex matching
             if (!targetLayer) {
                 for (const pattern of layerPatterns) {
                     const regex = new RegExp(`[/\\\\]${pattern}[/\\\\]`, 'i');
@@ -209,23 +178,17 @@ exports.enforceDependencyDirection = (0, eslint_devkit_2.createRule)({
                 }
             }
             const targetOrder = getLayerOrder(targetLayer, layers);
-            // Skip if layers are unknown
             if (sourceOrder < 0 || targetOrder < 0) {
                 return;
             }
-            // Check if same layer (allowed by default)
             if (allowSameLayer && sourceLayer === targetLayer) {
                 return;
             }
-            // Check dependency direction: lower layers should not depend on higher layers
-            // Domain (0) should not depend on Application (1), etc.
             if (targetOrder > sourceOrder) {
                 context.report({
                     node,
                     messageId: 'dependencyDirectionViolation',
                     data: {
-                        // Both are guaranteed non-null here: sourceOrder/targetOrder >= 0
-                        // implies the layers were resolved (see the guard above).
                         sourceLayer: sourceLayer,
                         targetLayer: targetLayer,
                     },

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

@@ -1,9 +1,4 @@
 "use strict";
-/**
- * Copyright (c) 2025 Ofri Peretz
- * Licensed under the MIT License. Use of this source code is governed by the
- * MIT license that can be found in the LICENSE file.
- */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.enforceImportOrder = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
@@ -126,13 +121,9 @@ exports.enforceImportOrder = (0, eslint_devkit_1.createRule)({
         const imports = [];
         function getImportType(node) {
             const source = node.source.value;
-            // console.log(`DEBUG: getImportType for ${source}`);
-            // Side-effect imports (no specifiers)
-            // import 'foo.css';
             if (node.specifiers.length === 0) {
                 return 'side-effect';
             }
-            // Built-in modules (node:)
             if (source.startsWith('node:') ||
                 [
                     'fs',
@@ -145,10 +136,8 @@ exports.enforceImportOrder = (0, eslint_devkit_1.createRule)({
                     'stream',
                     'child_process',
                 ].includes(source)) {
-                // console.log('DEBUG: identified as builtin');
                 return 'builtin';
             }
-            // Relative imports
             if (source.startsWith('.')) {
                 if (source.startsWith('../')) {
                     return 'parent';
@@ -158,7 +147,6 @@ exports.enforceImportOrder = (0, eslint_devkit_1.createRule)({
                 }
                 return 'sibling';
             }
-            // Internal imports (based on patterns)
             if (options.internalPatterns && options.internalPatterns.length > 0) {
                 for (const pattern of options.internalPatterns) {
                     if (new RegExp(pattern).test(source)) {
@@ -166,7 +154,6 @@ exports.enforceImportOrder = (0, eslint_devkit_1.createRule)({
                     }
                 }
             }
-            // External imports (default)
             return 'external';
         }
         function getGroupRank(importType) {
@@ -177,18 +164,13 @@ exports.enforceImportOrder = (0, eslint_devkit_1.createRule)({
             let start = node.range[0];
             let end = node.range[1];
             const commentsBefore = sourceCode.getCommentsBefore(node);
-            // getCommentsBefore only returns comments strictly between the previous
-            // non-comment token and this node, so they always belong to this import —
-            // include them all in the range.
             if (commentsBefore.length > 0) {
                 start = commentsBefore[0].range[0];
             }
-            // Include semicolon if present
             const tokenAfter = sourceCode.getTokenAfter(node);
             if (tokenAfter && tokenAfter.value === ';') {
                 end = tokenAfter.range[1];
             }
-            // Include trailing comment on the same line
             const commentsAfter = sourceCode.getCommentsAfter(node);
             const sameLineComment = commentsAfter.find((c) => c.loc.start.line === node.loc.end.line);
             if (sameLineComment) {
@@ -208,8 +190,6 @@ exports.enforceImportOrder = (0, eslint_devkit_1.createRule)({
                 if (imports.length === 0) {
                     return;
                 }
-                // 1. Check for contiguousness
-                // Verify no non-whitespace code between imports
                 for (let i = 0; i < imports.length - 1; i++) {
                     const current = imports[i];
                     const next = imports[i + 1];
@@ -217,9 +197,6 @@ exports.enforceImportOrder = (0, eslint_devkit_1.createRule)({
                         includeComments: true,
                     });
                     if (tokensBetween.length > 0) {
-                        // Found something between imports that isn't a comment?
-                        // getTokensBetween includes comments if includeComments: true
-                        // We want to know if there is CODE.
                         const codeTokens = sourceCode.getTokensBetween(current, next, {
                             includeComments: false,
                         });
@@ -228,12 +205,11 @@ exports.enforceImportOrder = (0, eslint_devkit_1.createRule)({
                                 node: next,
                                 messageId: 'importsNotContiguous',
                             });
-                            return; // Don't try to sort if interspersed with code
+                            return;
                         }
                     }
                 }
                 const originalImports = [...imports];
-                // oxlint-disable-next-line unicorn/no-array-sort
                 const sortedImports = Array.from(imports).sort((a, b) => {
                     const typeA = getImportType(a);
                     const typeB = getImportType(b);
@@ -254,7 +230,6 @@ exports.enforceImportOrder = (0, eslint_devkit_1.createRule)({
                     }
                     return 0;
                 });
-                // Check if order is correct
                 let isSorted = true;
                 for (let i = 0; i < originalImports.length; i++) {
                     if (originalImports[i] !== sortedImports[i]) {
@@ -262,7 +237,6 @@ exports.enforceImportOrder = (0, eslint_devkit_1.createRule)({
                         break;
                     }
                 }
-                // Check for newlines if sorted
                 if (isSorted && options.newlinesBetween !== 'ignore') {
                     for (let i = 0; i < imports.length - 1; i++) {
                         const current = imports[i];
@@ -272,16 +246,13 @@ exports.enforceImportOrder = (0, eslint_devkit_1.createRule)({
                         const rankCurrent = getGroupRank(typeCurrent);
                         const rankNext = getGroupRank(typeNext);
                         if (rankCurrent !== rankNext) {
-                            // Check lines between
                             const linesBetween = next.loc.start.line - current.loc.end.line;
-                            // console.log(`DEBUG: linesBetween ${linesBetween} | options.newlinesBetween ${options.newlinesBetween}`);
                             if (options.newlinesBetween === 'always' && linesBetween < 2) {
-                                // console.log('DEBUG: Triggering fix due to missing newline');
-                                isSorted = false; // Trigger fix
+                                isSorted = false;
                                 break;
                             }
                             if (options.newlinesBetween === 'never' && linesBetween > 1) {
-                                isSorted = false; // Trigger fix
+                                isSorted = false;
                                 break;
                             }
                         }
@@ -317,7 +288,6 @@ exports.enforceImportOrder = (0, eslint_devkit_1.createRule)({
                                 }
                                 newCode += getImportText(node) + '\n';
                             }
-                            // Remove last newline
                             newCode = newCode.trimEnd();
                             return fixer.replaceTextRange([rangeStart, rangeEnd], newCode);
                         },

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

@@ -1,37 +1,13 @@
 "use strict";
-/**
- * Copyright (c) 2025 Ofri Peretz
- * Licensed under the MIT License. Use of this source code is governed by the
- * MIT license that can be found in the LICENSE file.
- */
 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) {
@@ -42,9 +18,6 @@ function getFileTeam(filePath, teams) {
     }
     return null;
 }
-/**
- * Determine which team owns the imported module
- */
 function getImportTeam(importPath, teams) {
     for (const team of teams) {
         for (const pattern of team.paths) {
@@ -52,7 +25,6 @@ function getImportTeam(importPath, teams) {
                 return team;
             }
         }
-        // Check public packages
         if (team.publicPackages) {
             for (const pkg of team.publicPackages) {
                 if (importPath.startsWith(pkg)) {
@@ -63,31 +35,21 @@ function getImportTeam(importPath, teams) {
     }
     return null;
 }
-/**
- * Check if import path is external (node_modules)
- */
 function isExternalImport(importPath) {
-    // Relative imports start with . or ..
     if (importPath.startsWith('.') || importPath.startsWith('/')) {
         return false;
     }
-    // Alias imports (configurable patterns)
     if (importPath.startsWith('@/') || importPath.startsWith('~/')) {
         return false;
     }
-    // Internal project paths (src/, lib/, packages/, apps/)
     if (importPath.startsWith('src/') ||
         importPath.startsWith('lib/') ||
         importPath.startsWith('packages/') ||
         importPath.startsWith('apps/')) {
         return false;
     }
-    // Otherwise, likely external package
     return true;
 }
-/**
- * Check if path is in shared paths
- */
 function isSharedPath(importPath, sharedPaths) {
     for (const shared of sharedPaths) {
         if (globToRegex(shared).test(importPath)) {
@@ -163,15 +125,11 @@ 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;
         const sourceTeam = getFileTeam(filename, teams);
-        // If file doesn't belong to any team, skip checking
         if (!sourceTeam) {
             return {};
         }
-        // Narrowed binding: the early return above guarantees sourceTeam is non-null,
-        // and closures cannot rely on that narrowing without re-checking.
         const owningTeam = sourceTeam;
         function checkImport(node) {
             let importPath = null;
@@ -192,30 +150,23 @@ exports.enforceTeamBoundaries = (0, eslint_devkit_1.createRule)({
             }
             if (!importPath)
                 return;
-            // Allow external packages
             if (allowExternalPackages && isExternalImport(importPath)) {
                 return;
             }
-            // Allow shared paths
             if (isSharedPath(importPath, sharedPaths)) {
                 return;
             }
-            // Get target team
             const targetTeam = getImportTeam(importPath, teams);
-            // If import doesn't belong to any team, allow it (might be shared code)
             if (!targetTeam) {
                 return;
             }
-            // Same team - always allowed
             if (targetTeam.team === owningTeam.team) {
                 return;
             }
-            // Check if target team is in allowed dependencies
             const allowedDeps = owningTeam.allowedDependencies || [];
             if (allowedDeps.includes(targetTeam.team)) {
                 return;
             }
-            // Unauthorized cross-team import!
             context.report({
                 node,
                 messageId: 'unauthorizedTeamImport',

package/src/rules/export.js

@@ -1,9 +1,4 @@
 "use strict";
-/**
- * Copyright (c) 2025 Ofri Peretz
- * Licensed under the MIT License. Use of this source code is governed by the
- * MIT license that can be found in the LICENSE file.
- */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.exportRule = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
@@ -71,7 +66,6 @@ exports.exportRule = (0, eslint_devkit_1.createRule)({
                 checkDefaultExport(node);
             },
             ExportNamedDeclaration(node) {
-                // Export with declaration: export const foo = 1;
                 if (node.declaration) {
                     if (node.declaration.type === eslint_devkit_1.AST_NODE_TYPES.VariableDeclaration) {
                         node.declaration.declarations.forEach((decl) => {
@@ -94,7 +88,6 @@ exports.exportRule = (0, eslint_devkit_1.createRule)({
                         }
                     }
                 }
-                // Export specifiers: export { foo, bar };
                 node.specifiers.forEach((spec) => {
                     const exportedName = spec.exported.type === eslint_devkit_1.AST_NODE_TYPES.Identifier
                         ? spec.exported.name
@@ -108,12 +101,10 @@ exports.exportRule = (0, eslint_devkit_1.createRule)({
                 });
             },
             ExportAllDeclaration(node) {
-                // export * as name from '...'
                 if (node.exported) {
                     const exportedName = node.exported.name;
                     checkAndAddExport(exportedName, node);
                 }
-                // Regular export * doesn't create named conflicts we can statically detect
             },
         };
     },

package/src/rules/exports-last.js

@@ -1,9 +1,4 @@
 "use strict";
-/**
- * Copyright (c) 2025 Ofri Peretz
- * Licensed under the MIT License. Use of this source code is governed by the
- * MIT license that can be found in the LICENSE file.
- */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.exportsLast = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
@@ -36,7 +31,6 @@ exports.exportsLast = (0, eslint_devkit_1.createRule)({
             'Program:exit'(node) {
                 const body = node.body;
                 let lastNonExportIndex = -1;
-                // Find all export statements and non-export statements
                 const exportIndices = [];
                 const nonExportIndices = [];
                 body.forEach((statement, index) => {
@@ -44,11 +38,8 @@ exports.exportsLast = (0, eslint_devkit_1.createRule)({
                         statement.type === eslint_devkit_1.AST_NODE_TYPES.ExportNamedDeclaration ||
                         statement.type === eslint_devkit_1.AST_NODE_TYPES.ExportAllDeclaration;
                     if (isExport) {
-                        // Skip export declarations that are also declarations (export const x = 1)
-                        // These often need to appear where they're declared
                         if (statement.type === eslint_devkit_1.AST_NODE_TYPES.ExportNamedDeclaration &&
                             statement.declaration) {
-                            // This is an inline export, treat as non-export for ordering
                             nonExportIndices.push(index);
                         }
                         else {
@@ -62,7 +53,6 @@ exports.exportsLast = (0, eslint_devkit_1.createRule)({
                 if (nonExportIndices.length > 0) {
                     lastNonExportIndex = Math.max(...nonExportIndices);
                 }
-                // Check if any export appears before the last non-export
                 for (const exportIndex of exportIndices) {
                     if (exportIndex < lastNonExportIndex) {
                         context.report({

package/src/rules/extensions.js

@@ -1,9 +1,4 @@
 "use strict";
-/**
- * Copyright (c) 2025 Ofri Peretz
- * Licensed under the MIT License. Use of this source code is governed by the
- * MIT license that can be found in the LICENSE file.
- */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.extensions = void 0;
 const tslib_1 = require("tslib");
@@ -84,8 +79,8 @@ exports.extensions = (0, eslint_devkit_1.createRule)({
             ImportDeclaration(node) {
                 const source = node.source.value;
                 if (!source.startsWith('.'))
-                    return; // Only check relative imports
-                const ext = node_path_1.default.extname(source).slice(1); // remove dot
+                    return;
+                const ext = node_path_1.default.extname(source).slice(1);
                 const expected = pattern[ext] || defaultBehavior;
                 if (ext && expected === 'never') {
                     context.report({
@@ -97,14 +92,10 @@ exports.extensions = (0, eslint_devkit_1.createRule)({
                     });
                 }
                 else if (!ext) {
-                    // Hard to know what the extension *should* be without checking file system
-                    // But if default is 'always', we might flag it.
-                    // For now, let's assume if it's missing and we expect 'always', it's a problem.
                     if (defaultBehavior === 'always') {
                         context.report({
                             node: node.source,
                             messageId: 'missingExtension',
-                            // Can't fix without knowing extension
                         });
                     }
                 }

package/src/rules/first.js

@@ -1,9 +1,4 @@
 "use strict";
-/**
- * Copyright (c) 2025 Ofri Peretz
- * Licensed under the MIT License. Use of this source code is governed by the
- * MIT license that can be found in the LICENSE file.
- */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.first = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
@@ -48,7 +43,6 @@ 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

@@ -1,9 +1,4 @@
 "use strict";
-/**
- * Copyright (c) 2025 Ofri Peretz
- * Licensed under the MIT License. Use of this source code is governed by the
- * MIT license that can be found in the LICENSE file.
- */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.groupExports = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
@@ -32,15 +27,12 @@ exports.groupExports = (0, eslint_devkit_1.createRule)({
     },
     defaultOptions: [],
     create(context) {
-        // Track separate export statements (not counting inline exports)
         const separateExports = [];
         const reExportSources = new Map();
         return {
             ExportNamedDeclaration(node) {
-                // Only track export specifier lists, not inline declarations
                 if (!node.declaration && node.specifiers.length > 0) {
                     if (node.source) {
-                        // Re-export from another module
                         const source = node.source.value;
                         if (!reExportSources.has(source)) {
                             reExportSources.set(source, []);
@@ -48,15 +40,12 @@ exports.groupExports = (0, eslint_devkit_1.createRule)({
                         reExportSources.get(source)?.push(node);
                     }
                     else {
-                        // Local export
                         separateExports.push(node);
                     }
                 }
             },
             'Program:exit'() {
-                // Report if there are multiple separate local exports that could be combined
                 if (separateExports.length > 1) {
-                    // Report all but the first one
                     for (let i = 1; i < separateExports.length; i++) {
                         context.report({
                             node: separateExports[i],
@@ -64,10 +53,8 @@ exports.groupExports = (0, eslint_devkit_1.createRule)({
                         });
                     }
                 }
-                // Report if there are multiple re-exports from the same source
                 for (const [, exports] of reExportSources) {
                     if (exports.length > 1) {
-                        // Report all but the first one
                         for (let i = 1; i < exports.length; i++) {
                             context.report({
                                 node: exports[i],

package/src/rules/max-dependencies.js

@@ -1,9 +1,4 @@
 "use strict";
-/**
- * Copyright (c) 2025 Ofri Peretz
- * Licensed under the MIT License. Use of this source code is governed by the
- * MIT license that can be found in the LICENSE file.
- */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.maxDependencies = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
@@ -92,7 +87,6 @@ exports.maxDependencies = (0, eslint_devkit_1.createRule)({
         if (!filename) {
             return {};
         }
-        // Check if file should be ignored
         const shouldIgnoreFile = ignoreFiles.some((pattern) => {
             if (pattern.includes('*')) {
                 const regex = new RegExp(pattern.replace(/\*/g, '.*'));
@@ -105,19 +99,15 @@ exports.maxDependencies = (0, eslint_devkit_1.createRule)({
         }
         const dependencies = new Set();
         function shouldCountImport(importPath, node) {
-            // Skip ignored imports
             if (ignoreImports.includes(importPath)) {
                 return false;
             }
-            // Skip type-only imports if configured
             if (ignoreTypeImports && node.importKind === 'type') {
                 return false;
             }
-            // Skip relative imports (internal modules)
             if (importPath.startsWith('./') || importPath.startsWith('../')) {
                 return false;
             }
-            // Skip Node.js built-ins
             const nodeBuiltins = [
                 'assert',
                 'buffer',
@@ -154,15 +144,12 @@ exports.maxDependencies = (0, eslint_devkit_1.createRule)({
             return true;
         }
         function shouldCountRequire(importPath) {
-            // Skip ignored imports
             if (ignoreImports.includes(importPath)) {
                 return false;
             }
-            // Skip relative imports
             if (importPath.startsWith('./') || importPath.startsWith('../')) {
                 return false;
             }
-            // Skip Node.js built-ins
             const nodeBuiltins = [
                 'assert',
                 'buffer',
@@ -199,7 +186,6 @@ exports.maxDependencies = (0, eslint_devkit_1.createRule)({
             return true;
         }
         function addDependency(importPath) {
-            // Extract package name from scoped packages or regular packages
             const packageName = importPath.startsWith('@')
                 ? importPath.split('/').slice(0, 2).join('/')
                 : importPath.split('/')[0];
@@ -214,7 +200,6 @@ exports.maxDependencies = (0, eslint_devkit_1.createRule)({
                 }
             },
             CallExpression(node) {
-                // Check require() calls
                 if (node.callee.type === 'Identifier' &&
                     node.callee.name === 'require' &&
                     node.arguments.length === 1) {
@@ -226,7 +211,6 @@ exports.maxDependencies = (0, eslint_devkit_1.createRule)({
                     }
                 }
             },
-            // Handle dynamic imports (import() expressions)
             ImportExpression(node) {
                 const source = node.source;
                 if (source.type === 'Literal' && typeof source.value === 'string') {