eslint-plugin-import-next 2.3.9 → 2.3.11

package/src/rules/no-deprecated.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.noDeprecated = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
@@ -88,7 +83,6 @@ exports.noDeprecated = (0, eslint_devkit_1.createRule)({
         if (!filename) {
             return {};
         }
-        // Track deprecated items found in the current file
         const deprecatedItems = new Map();
         function isAllowed(name) {
             return allow.includes(name);
@@ -97,24 +91,19 @@ exports.noDeprecated = (0, eslint_devkit_1.createRule)({
             const lines = comment.split('\n');
             for (const line of lines) {
                 const trimmed = line.trim();
-                // Check for @deprecated tag
                 for (const marker of deprecationMarkers) {
                     if (trimmed.includes(marker)) {
                         const afterMarker = trimmed
                             .substring(trimmed.indexOf(marker) + marker.length)
                             .trim();
-                        // Try to extract reason and replacement
                         let reason;
                         let replacement;
-                        // Look for common patterns like "Use X instead" or "Replaced by Y"
                         const insteadMatch = afterMarker.match(/(?:use|replaced by|see)\s+([^\s]+(?:\s+[^\s]+)*?)(?:\s|$)/i);
                         if (insteadMatch) {
-                            // Remove common trailing words
                             replacement = insteadMatch[1]
                                 .trim()
                                 .replace(/\s+instead$|\s+or$|\s+as$|\s+with$/i, '');
                         }
-                        // The rest is usually the reason
                         const reasonText = afterMarker
                             .replace(/use\s+[^.]*[.]?/i, '')
                             .replace(/replaced by\s+[^.]*[.]?/i, '')
@@ -133,14 +122,12 @@ exports.noDeprecated = (0, eslint_devkit_1.createRule)({
                 return;
             const sourceCode = context.sourceCode;
             const comments = sourceCode.getCommentsBefore(node);
-            // Also check leading comments attached to the node
             const allComments = [...comments];
             if ('leadingComments' in node && Array.isArray(node.leadingComments)) {
                 allComments.push(...node.leadingComments);
             }
             for (const comment of allComments) {
                 if (comment.type === 'Block' && comment.value.includes('*')) {
-                    // JSDoc comment
                     const deprecationInfo = parseJSDocDeprecated(comment.value);
                     if (deprecationInfo.deprecated) {
                         deprecatedItems.set(name, {
@@ -156,10 +143,8 @@ exports.noDeprecated = (0, eslint_devkit_1.createRule)({
         function checkDecoratorsForDeprecation(node, name) {
             if (!checkDecorators)
                 return;
-            // Check for @deprecated decorator in TypeScript
             if ('decorators' in node &&
                 Array.isArray(node.decorators)) {
-                // The Array.isArray guard above proves decorators is an array.
                 const decorators = node.decorators;
                 for (const decorator of decorators) {
                     const dec = decorator;
@@ -214,7 +199,6 @@ exports.noDeprecated = (0, eslint_devkit_1.createRule)({
             });
         }
         return {
-            // Check for deprecated declarations in the current file
             VariableDeclaration(node) {
                 node.declarations.forEach((decl) => {
                     if (decl.id.type === 'Identifier') {
@@ -237,13 +221,11 @@ exports.noDeprecated = (0, eslint_devkit_1.createRule)({
             },
             MethodDefinition(node) {
                 if (node.key.type === 'Identifier') {
-                    // For methods, check comments on both the method node and the key
                     checkComments(node, node.key.name);
                     checkComments(node.key, node.key.name);
                     checkDecoratorsForDeprecation(node, node.key.name);
                 }
             },
-            // Check for deprecated exports in the current file
             ExportNamedDeclaration(node) {
                 if (node.declaration) {
                     if (node.declaration.type === 'VariableDeclaration') {
@@ -265,7 +247,6 @@ exports.noDeprecated = (0, eslint_devkit_1.createRule)({
                         checkDecoratorsForDeprecation(node.declaration, node.declaration.id.name);
                     }
                 }
-                // Check exported specifiers
                 if (node.specifiers) {
                     node.specifiers.forEach((specifier) => {
                         if (specifier.exported.type === 'Identifier') {
@@ -278,16 +259,9 @@ exports.noDeprecated = (0, eslint_devkit_1.createRule)({
                 checkComments(node, 'default');
                 checkDecoratorsForDeprecation(node, 'default');
             },
-            // Check for usage of deprecated imported items
-            // eslint-disable-next-line @typescript-eslint/no-unused-vars
             ImportDeclaration(node) {
-                // This would need cross-file analysis to check if imported items are deprecated
-                // For now, we'll focus on deprecated items within the same file
             },
-            // Check for usage of deprecated items
             MemberExpression(node) {
-                // No has() pre-check: reportDeprecatedUsage bails out when the
-                // object name has no recorded deprecation info.
                 if (node.object.type === 'Identifier' &&
                     node.property.type === 'Identifier') {
                     reportDeprecatedUsage(node, node.object.name, `${node.object.name}.${node.property.name}`);
@@ -306,9 +280,7 @@ exports.noDeprecated = (0, eslint_devkit_1.createRule)({
                 }
             },
             Identifier(node) {
-                // Check if this identifier references a deprecated item
                 if (deprecatedItems.has(node.name)) {
-                    // Only report if it's not part of a declaration
                     const parent = node.parent;
                     if (parent &&
                         ![
@@ -319,22 +291,18 @@ exports.noDeprecated = (0, eslint_devkit_1.createRule)({
                             'ExportSpecifier',
                             'MethodDefinition',
                         ].includes(parent.type)) {
-                        // Don't report if this identifier is the callee of a CallExpression (handled separately)
                         if (parent.type === 'CallExpression' &&
                             parent.callee === node) {
                             return;
                         }
-                        // Don't report if this identifier is the callee of a NewExpression (handled separately)
                         if (parent.type === 'NewExpression' &&
                             parent.callee === node) {
                             return;
                         }
-                        // Don't report if this identifier is the object of a MemberExpression (handled separately)
                         if (parent.type === 'MemberExpression' &&
                             parent.object === node) {
                             return;
                         }
-                        // Don't report if this identifier is the property of a MemberExpression (handled separately)
                         if (parent.type === 'MemberExpression' &&
                             parent.property === node) {
                             return;

package/src/rules/no-duplicates.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.noDuplicates = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
@@ -51,22 +46,19 @@ exports.noDuplicates = (0, eslint_devkit_1.createRule)({
                                 node: duplicate,
                                 messageId: 'noDuplicates',
                                 fix(fixer) {
-                                    // Simple fix: if both are named imports, merge them
                                     const firstHasDefault = first.specifiers.some((s) => s.type === 'ImportDefaultSpecifier');
                                     const duplicateHasDefault = duplicate.specifiers.some((s) => s.type === 'ImportDefaultSpecifier');
                                     const firstHasNamespace = first.specifiers.some((s) => s.type === 'ImportNamespaceSpecifier');
                                     const duplicateHasNamespace = duplicate.specifiers.some((s) => s.type === 'ImportNamespaceSpecifier');
                                     if (firstHasNamespace || duplicateHasNamespace) {
-                                        return null; // Too complex to merge namespace imports automatically
+                                        return null;
                                     }
                                     if (firstHasDefault && duplicateHasDefault) {
-                                        return null; // Cannot have two default imports
+                                        return null;
                                     }
-                                    // If duplicate is empty (side-effect), just remove it
                                     if (duplicate.specifiers.length === 0) {
                                         return fixer.remove(duplicate);
                                     }
-                                    // If first is empty (side-effect), we can't easily merge into it without more logic
                                     if (first.specifiers.length === 0) {
                                         return null;
                                     }

package/src/rules/no-dynamic-require.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.noDynamicRequire = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
@@ -58,7 +53,6 @@ exports.noDynamicRequire = (0, eslint_devkit_1.createRule)({
         }
         return {
             CallExpression(node) {
-                // Check require() calls
                 if (node.callee.type === eslint_devkit_1.AST_NODE_TYPES.Identifier &&
                     node.callee.name === 'require' &&
                     node.arguments.length >= 1) {

package/src/rules/no-empty-named-blocks.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.noEmptyNamedBlocks = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
@@ -37,24 +32,18 @@ exports.noEmptyNamedBlocks = (0, eslint_devkit_1.createRule)({
     create(context) {
         return {
             ImportDeclaration(node) {
-                // Check if this import has empty named imports: import {} from 'foo'
-                // or import Foo, {} from 'foo'
                 const hasDefaultImport = node.specifiers.some((spec) => spec.type === 'ImportDefaultSpecifier');
                 const hasNamespaceImport = node.specifiers.some((spec) => spec.type === 'ImportNamespaceSpecifier');
                 const namedImports = node.specifiers.filter((spec) => spec.type === 'ImportSpecifier');
-                // Get the source code
                 const sourceCode = context.sourceCode;
                 const importText = sourceCode.getText(node);
-                // Check for empty braces pattern: {}
                 const hasEmptyBraces = /\{\s*\}/.test(importText);
                 if (hasEmptyBraces && namedImports.length === 0) {
-                    // If there's only empty braces (no default, no namespace)
                     if (!hasDefaultImport && !hasNamespaceImport) {
                         context.report({
                             node,
                             messageId: 'emptyNamedBlock',
                             fix(fixer) {
-                                // Remove the entire import statement
                                 return fixer.remove(node);
                             },
                             suggest: [
@@ -68,13 +57,10 @@ exports.noEmptyNamedBlocks = (0, eslint_devkit_1.createRule)({
                         });
                     }
                     else if (hasDefaultImport) {
-                        // Has default import with empty braces: import Foo, {} from 'bar'
-                        // Fix to: import Foo from 'bar'
                         context.report({
                             node,
                             messageId: 'emptyNamedBlock',
                             fix(fixer) {
-                                // Remove the ", {}" part
                                 const fixedText = importText.replace(/,\s*\{\s*\}/, '');
                                 return fixer.replaceText(node, fixedText);
                             },
@@ -92,11 +78,9 @@ exports.noEmptyNamedBlocks = (0, eslint_devkit_1.createRule)({
                 }
             },
             ExportNamedDeclaration(node) {
-                // Check for: export {} from 'foo' or export {}
                 if (node.specifiers.length === 0 &&
                     !node.declaration &&
-                    node.source // Only for re-exports, local `export {}` is technically valid but useless
-                ) {
+                    node.source) {
                     context.report({
                         node,
                         messageId: 'emptyNamedBlock',

package/src/rules/no-extraneous-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.noExtraneousDependencies = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
@@ -139,67 +134,49 @@ exports.noExtraneousDependencies = (0, eslint_devkit_1.createRule)({
         if (!filename) {
             return {};
         }
-        // Use provided package.json if available (for testing)
         let resolvedPackageJson = null;
         if (packageJson) {
-            // Use directly provided package.json (for testing)
             resolvedPackageJson = packageJson;
         }
         else if (packageJsonPath) {
-            // Use specified package.json file
             resolvedPackageJson = (0, eslint_devkit_3.readJsonFileSync)(packageJsonPath);
             if (!resolvedPackageJson) {
-                // Invalid package.json - skip
                 return {};
             }
         }
         else {
-            // Find package.json automatically
             const packageJsonFile = (0, eslint_devkit_3.findFileUpward)('package.json', (0, eslint_devkit_4.getDirname)(filename));
             if (!packageJsonFile) {
-                // No package.json found - skip this rule
                 return {};
             }
-            // Read and parse package.json
             resolvedPackageJson = (0, eslint_devkit_3.readJsonFileSync)(packageJsonFile);
             if (!resolvedPackageJson) {
-                // Invalid package.json - skip
                 return {};
             }
         }
-        // Build allowed packages map
         const allowedPackages = new Set();
-        // Add dependencies
         if (resolvedPackageJson?.dependencies) {
             Object.keys(resolvedPackageJson.dependencies).forEach((pkg) => allowedPackages.add(pkg));
         }
-        // Add dev dependencies if allowed
         if (devDependencies && resolvedPackageJson?.devDependencies) {
             Object.keys(resolvedPackageJson.devDependencies).forEach((pkg) => allowedPackages.add(pkg));
         }
-        // Add optional dependencies if allowed
         if (optionalDependencies && resolvedPackageJson?.optionalDependencies) {
             Object.keys(resolvedPackageJson.optionalDependencies).forEach((pkg) => allowedPackages.add(pkg));
         }
-        // Add peer dependencies if allowed
         if (peerDependencies && resolvedPackageJson?.peerDependencies) {
             Object.keys(resolvedPackageJson.peerDependencies).forEach((pkg) => allowedPackages.add(pkg));
         }
-        // Add bundled dependencies if allowed
         if (bundledDependencies && resolvedPackageJson?.bundledDependencies) {
             resolvedPackageJson.bundledDependencies.forEach((pkg) => allowedPackages.add(pkg));
         }
-        // oxlint-disable-next-line consistent-function-scoping
         function isExternalPackage(importPath) {
-            // Skip relative imports
             if (importPath.startsWith('./') || importPath.startsWith('../')) {
                 return null;
             }
-            // Skip absolute paths
             if (importPath.startsWith('/')) {
                 return null;
             }
-            // Skip Node.js built-ins
             const nodeBuiltins = [
                 'assert',
                 'buffer',
@@ -261,15 +238,12 @@ exports.noExtraneousDependencies = (0, eslint_devkit_1.createRule)({
             if (nodeBuiltins.includes(importPath)) {
                 return null;
             }
-            // Extract package name from scoped packages or regular packages
             const packageName = importPath.startsWith('@')
                 ? importPath.split('/').slice(0, 2).join('/')
                 : importPath.split('/')[0];
             return packageName;
         }
-        // oxlint-disable-next-line consistent-function-scoping
         function isWorkspacePackage(packageName) {
-            // For testing purposes, consider packages with @workspace/ or @company/ as workspace packages
             return (packageName.startsWith('@workspace/') ||
                 packageName.startsWith('@company/'));
         }
@@ -278,7 +252,6 @@ exports.noExtraneousDependencies = (0, eslint_devkit_1.createRule)({
             if (!packageName) {
                 return;
             }
-            // Check allow patterns first
             const isAllowedByPattern = allowPatterns.some((pattern) => {
                 try {
                     const regex = new RegExp(pattern);
@@ -291,15 +264,12 @@ exports.noExtraneousDependencies = (0, eslint_devkit_1.createRule)({
             if (isAllowedByPattern) {
                 return;
             }
-            // Apply resolution strategy
             const shouldCheckDependency = () => {
                 switch (resolutionStrategy) {
                     case 'workspace':
-                        // Allow packages that are in dependencies OR are workspace packages
                         return (!allowedPackages.has(packageName) &&
                             !isWorkspacePackage(packageName));
                     case 'monorepo':
-                        // Allow cross-package imports in monorepos
                         return (!allowedPackages.has(packageName) &&
                             !isWorkspacePackage(packageName));
                     case 'strict':
@@ -308,11 +278,9 @@ exports.noExtraneousDependencies = (0, eslint_devkit_1.createRule)({
                 }
             };
             if (shouldCheckDependency()) {
-                // Check if it's in devDependencies but we're in production code
                 const isInDevDeps = resolvedPackageJson?.devDependencies &&
                     resolvedPackageJson.devDependencies[packageName];
                 if (isInDevDeps && !devDependencies) {
-                    // Dev dependency used in production code
                     context.report({
                         node,
                         messageId: 'devDependencyInProduction',
@@ -325,7 +293,6 @@ exports.noExtraneousDependencies = (0, eslint_devkit_1.createRule)({
                             {
                                 messageId: 'moveToDependencies',
                                 fix(fixer) {
-                                    // Find the parent statement to insert before
                                     let parentStatement = node;
                                     while (parentStatement &&
                                         parentStatement.type !== 'ImportDeclaration' &&
@@ -343,7 +310,6 @@ exports.noExtraneousDependencies = (0, eslint_devkit_1.createRule)({
                     });
                 }
                 else {
-                    // Completely missing dependency
                     context.report({
                         node,
                         messageId: 'missingDependency',
@@ -358,7 +324,6 @@ exports.noExtraneousDependencies = (0, eslint_devkit_1.createRule)({
                             {
                                 messageId: 'addToDependencies',
                                 fix(fixer) {
-                                    // Find the parent statement to insert before
                                     let parentStatement = node;
                                     while (parentStatement &&
                                         parentStatement.type !== 'ImportDeclaration' &&
@@ -375,7 +340,6 @@ exports.noExtraneousDependencies = (0, eslint_devkit_1.createRule)({
                             {
                                 messageId: 'addToDevDependencies',
                                 fix(fixer) {
-                                    // Find the parent statement to insert before
                                     let parentStatement = node;
                                     while (parentStatement &&
                                         parentStatement.type !== 'ImportDeclaration' &&
@@ -401,7 +365,6 @@ exports.noExtraneousDependencies = (0, eslint_devkit_1.createRule)({
                 }
             },
             CallExpression(node) {
-                // Check require() calls
                 if (node.callee.type === 'Identifier' &&
                     node.callee.name === 'require' &&
                     node.arguments.length === 1) {
@@ -412,7 +375,6 @@ exports.noExtraneousDependencies = (0, eslint_devkit_1.createRule)({
                 }
             },
             ImportExpression(node) {
-                // Check dynamic imports
                 if (node.source.type === 'Literal' &&
                     typeof node.source.value === 'string') {
                     checkImport(node.source.value, node.source);

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

@@ -1,13 +1,7 @@
 "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.noFullPackageImport = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
-// Known large packages that should use subpath/named imports
 const DEFAULT_BLOCKED_PACKAGES = [
     {
         name: 'lodash',
@@ -55,22 +49,12 @@ const DEFAULT_BLOCKED_PACKAGES = [
         example: "import { S3Client } from '@aws-sdk/client-s3';",
     },
 ];
-/**
- * Check if import is a full package import (not a subpath)
- */
 function isFullPackageImport(importPath, packageName) {
-    // Exact match means full package import
     return importPath === packageName;
 }
-/**
- * Check if import uses namespace pattern
- */
 function hasNamespaceImport(node) {
     return node.specifiers.some((spec) => spec.type === eslint_devkit_1.AST_NODE_TYPES.ImportNamespaceSpecifier);
 }
-/**
- * Check if import uses default pattern
- */
 function hasDefaultImport(node) {
     return node.specifiers.some((spec) => spec.type === eslint_devkit_1.AST_NODE_TYPES.ImportDefaultSpecifier);
 }
@@ -128,13 +112,10 @@ exports.noFullPackageImport = (0, eslint_devkit_1.createRule)({
                 const importPath = node.source.value;
                 if (typeof importPath !== 'string')
                     return;
-                // Check against blocked packages
                 for (const blocked of blockedPackages) {
                     if (!isFullPackageImport(importPath, blocked.name)) {
                         continue;
                     }
-                    // Only flag namespace or default imports from full package path
-                    // Named imports might be tree-shakeable depending on the package
                     if (hasNamespaceImport(node) || hasDefaultImport(node)) {
                         context.report({
                             node,

package/src/rules/no-import-module-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.noImportModuleExports = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
@@ -52,7 +47,6 @@ exports.noImportModuleExports = (0, eslint_devkit_1.createRule)({
                 importNodes.push(node);
             },
             MemberExpression(node) {
-                // Check for module.exports
                 if (node.object.type === eslint_devkit_1.AST_NODE_TYPES.Identifier &&
                     node.object.name === 'module' &&
                     node.property.type === eslint_devkit_1.AST_NODE_TYPES.Identifier &&
@@ -61,7 +55,6 @@ exports.noImportModuleExports = (0, eslint_devkit_1.createRule)({
                 }
             },
             AssignmentExpression(node) {
-                // Check for exports.xxx = ...
                 if (node.left.type === eslint_devkit_1.AST_NODE_TYPES.MemberExpression &&
                     node.left.object.type === eslint_devkit_1.AST_NODE_TYPES.Identifier &&
                     node.left.object.name === 'exports') {
@@ -69,7 +62,6 @@ exports.noImportModuleExports = (0, eslint_devkit_1.createRule)({
                 }
             },
             'Program:exit'() {
-                // If we have both imports and module.exports, report on module.exports
                 if (hasImport && moduleExportsNodes.length > 0) {
                     moduleExportsNodes.forEach((node) => {
                         context.report({

package/src/rules/no-internal-modules.js

@@ -1,19 +1,9 @@
 "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.noInternalModules = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
 const eslint_devkit_2 = require("@interlace/eslint-devkit");
-/**
- * Simple glob pattern matching
- * Supports * (any chars) and ** (any path segments)
- */
 function matchGlob(pattern, path) {
-    // Escape special regex chars except *
     const regexPattern = pattern
         .replace(/[.+^${}()|[\]\\]/g, '\\$&')
         .replace(/\*\*/g, '{{DOUBLE_STAR}}')
@@ -24,35 +14,18 @@ function matchGlob(pattern, path) {
 function matchesAnyPattern(path, patterns) {
     return patterns.some((pattern) => matchGlob(pattern, path));
 }
-/**
- * Calculate import depth
- * - 'lodash' => 0
- * - 'lodash/get' => 1
- * - '@company/ui' => 0 (scoped package root)
- * - '@company/ui/components' => 1
- * - '@company/ui/components/Button' => 2
- * - './utils' => 1
- * - './utils/helpers' => 2
- */
 function getImportDepth(importPath) {
-    // Handle relative imports
     if (importPath.startsWith('./') || importPath.startsWith('../')) {
         const parts = importPath.split('/').filter((p) => p !== '.' && p !== '..');
         return parts.length;
     }
-    // Handle scoped packages (@org/package)
     if (importPath.startsWith('@')) {
         const parts = importPath.split('/');
-        // @org/package = 0 depth, @org/package/sub = 1 depth
         return Math.max(0, parts.length - 2);
     }
-    // Handle regular packages
     const parts = importPath.split('/');
     return parts.length - 1;
 }
-/**
- * Get the root/base import path
- */
 function getRootImport(importPath) {
     if (importPath.startsWith('./') || importPath.startsWith('../')) {
         return '.';
@@ -63,9 +36,6 @@ function getRootImport(importPath) {
     }
     return importPath.split('/')[0];
 }
-/**
- * Get import path trimmed to specific depth
- */
 function getImportAtDepth(importPath, maxDepth) {
     if (importPath.startsWith('./') || importPath.startsWith('../')) {
         if (maxDepth === 0)
@@ -76,7 +46,6 @@ function getImportAtDepth(importPath, maxDepth) {
     }
     if (importPath.startsWith('@')) {
         const parts = importPath.split('/');
-        // @org/package is depth 0
         return parts.slice(0, 2 + maxDepth).join('/');
     }
     const parts = importPath.split('/');
@@ -170,18 +139,15 @@ exports.noInternalModules = (0, eslint_devkit_1.createRule)({
         const [options] = context.options;
         const { maxDepth = 1, allow = [], forbid = [], ignorePaths = [], strategy = 'error', } = options || {};
         function checkImport(importPath, node) {
-            // Skip if path should be ignored
             if (matchesAnyPattern(importPath, ignorePaths)) {
                 return;
             }
-            // Skip if path is explicitly allowed
             if (matchesAnyPattern(importPath, allow)) {
                 return;
             }
             const depth = getImportDepth(importPath);
             const isForbidden = forbid.length > 0 && matchesAnyPattern(importPath, forbid);
             const exceedsDepth = depth > maxDepth;
-            // No violation
             if (!isForbidden && !exceedsDepth) {
                 return;
             }
@@ -204,8 +170,6 @@ exports.noInternalModules = (0, eslint_devkit_1.createRule)({
                     messageId: 'internalModuleImport',
                     data: reportData,
                     fix(fixer) {
-                        // Find the string literal node to replace
-                        // Autofix always goes to the root package for safety
                         const sourceNode = node.type === 'Literal'
                             ? node
                             : node.source;
@@ -229,7 +193,6 @@ exports.noInternalModules = (0, eslint_devkit_1.createRule)({
                                 return fixer.replaceText(sourceNode, `'${rootImport}'`);
                             },
                         },
-                        // Add barrel export suggestion if different from root
                         ...(barrelPath !== rootImport && barrelPath !== suggestedPath
                             ? [
                                 {
@@ -248,7 +211,6 @@ exports.noInternalModules = (0, eslint_devkit_1.createRule)({
                 });
             }
             else {
-                // 'error' or 'warn' strategy - just report
                 context.report({
                     node,
                     messageId: 'internalModuleImport',
@@ -274,7 +236,6 @@ exports.noInternalModules = (0, eslint_devkit_1.createRule)({
                 }
             },
             CallExpression(node) {
-                // Check require() calls
                 if (node.callee.type === 'Identifier' &&
                     node.callee.name === 'require' &&
                     node.arguments.length === 1) {
@@ -284,7 +245,6 @@ exports.noInternalModules = (0, eslint_devkit_1.createRule)({
                     }
                 }
             },
-            // Handle dynamic imports: import('path')
             ImportExpression(node) {
                 if (node.source.type === 'Literal' &&
                     typeof node.source.value === 'string') {