eslint-plugin-import-next 2.3.9 → 2.3.11

package/src/rules/no-legacy-imports.js

@@ -1,22 +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.noLegacyImports = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
-/**
- * Check if import matches a legacy pattern
- */
 function matchLegacyPattern(importPath, pattern) {
     try {
         const regex = new RegExp(pattern);
         return regex.test(importPath);
     }
     catch {
-        // Fallback to exact match
         return importPath === pattern || importPath.startsWith(pattern);
     }
 }
@@ -96,11 +87,10 @@ exports.noLegacyImports = (0, eslint_devkit_1.createRule)({
                             deadline: deadlineText,
                         },
                         fix: (fixer) => {
-                            // Simple replacement - replace the import path
                             return fixer.replaceText(sourceNode, `'${mapping.replacement}'`);
                         },
                     });
-                    break; // Only report first matching pattern
+                    break;
                 }
             }
         }

package/src/rules/no-mutable-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.noMutableExports = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
@@ -120,7 +115,6 @@ exports.noMutableExports = (0, eslint_devkit_1.createRule)({
                     return;
                 }
                 const { kind, declarations } = node.declaration;
-                // Only check var and let declarations
                 if (kind !== 'var' && kind !== 'let') {
                     return;
                 }
@@ -128,25 +122,17 @@ exports.noMutableExports = (0, eslint_devkit_1.createRule)({
                     checkPattern(decl.id, kind);
                 });
             },
-            // Also check for re-exports that might be mutable
-            // This is more complex as it requires following the import chain
             VariableDeclaration(node) {
-                // Check if this variable declaration is exported
                 const parent = node.parent;
                 if (parent && parent.type === 'ExportNamedDeclaration') {
-                    // This is already handled above
                     return;
                 }
-                // Check if this variable is exported elsewhere in the file
-                // This is a simplified check - a full implementation would need to track all exports
                 const sourceCode = context.sourceCode;
                 const fileText = sourceCode.getText();
                 if (node.kind === 'var' || node.kind === 'let') {
                     node.declarations.forEach((decl) => {
                         if (decl.id.type === 'Identifier') {
                             const varName = decl.id.name;
-                            // Check if this variable is exported later in the file
-                            // This is a heuristic - real implementation would need AST analysis
                             const exportPattern = new RegExp(`export\\s*{\\s*${varName}\\s*}`, 'g');
                             if (exportPattern.test(fileText)) {
                                 reportMutableExport(decl.id, varName, node.kind);

package/src/rules/no-named-as-default-member.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.noNamedAsDefaultMember = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
@@ -32,9 +27,7 @@ exports.noNamedAsDefaultMember = (0, eslint_devkit_1.createRule)({
     },
     defaultOptions: [],
     create(context) {
-        // Track default imports and their sources
         const defaultImports = new Map();
-        // Track named imports per source
         const namedImportsBySource = new Map();
         return {
             ImportDeclaration(node) {
@@ -55,21 +48,17 @@ exports.noNamedAsDefaultMember = (0, eslint_devkit_1.createRule)({
                 });
             },
             MemberExpression(node) {
-                // Check if accessing property of default import
                 if (node.object.type !== eslint_devkit_1.AST_NODE_TYPES.Identifier) {
                     return;
                 }
-                // Skip computed properties like obj[prop]
                 if (node.computed)
                     return;
                 const defaultImportInfo = defaultImports.get(node.object.name);
                 if (!defaultImportInfo)
                     return;
-                // Get the property name being accessed
                 if (node.property.type !== eslint_devkit_1.AST_NODE_TYPES.Identifier)
                     return;
                 const propertyName = node.property.name;
-                // Check if this property is a known named export from the same source
                 const namedExports = namedImportsBySource.get(defaultImportInfo.source);
                 if (namedExports && namedExports.has(propertyName)) {
                     context.report({

package/src/rules/no-named-as-default.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.noNamedAsDefault = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
@@ -32,14 +27,9 @@ exports.noNamedAsDefault = (0, eslint_devkit_1.createRule)({
     },
     defaultOptions: [],
     create(context) {
-        // This rule requires resolving the imported module to see its exports
-        // For a basic implementation without full resolution, we track what we can
-        // In practice, this would need TypeScript language service or resolver
-        // Track exports from the current file for same-file detection
         const exportedNames = new Set();
         let defaultExportName = null;
         return {
-            // Collect exports to detect potential issues
             ExportNamedDeclaration(node) {
                 if (node.declaration) {
                     if ((node.declaration.type === 'FunctionDeclaration' ||
@@ -62,7 +52,6 @@ exports.noNamedAsDefault = (0, eslint_devkit_1.createRule)({
                 });
             },
             ExportDefaultDeclaration(node) {
-                // Track if default export is a named identifier
                 if (node.declaration.type === 'Identifier') {
                     defaultExportName = node.declaration.name;
                 }
@@ -73,20 +62,14 @@ exports.noNamedAsDefault = (0, eslint_devkit_1.createRule)({
                 }
             },
             'Program:exit'() {
-                // If the default export shares a name with a named export, report
                 if (defaultExportName && exportedNames.has(defaultExportName)) {
-                    // This is a heuristic - ideally we'd track the import site
-                    // For now, we report on the pattern where importing default as the name
-                    // would shadow the named export
                 }
             },
             ImportDeclaration(node) {
-                // Check if default import name matches a named import from same module
                 const defaultSpec = node.specifiers.find((s) => s.type === 'ImportDefaultSpecifier');
                 if (!defaultSpec)
                     return;
                 const defaultName = defaultSpec.local.name;
-                // Check if there's also a named import with the same name being imported
                 const namedSpecs = node.specifiers.filter((s) => s.type === 'ImportSpecifier');
                 for (const namedSpec of namedSpecs) {
                     const importedName = namedSpec.imported.type === 'Identifier'

package/src/rules/no-named-default.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.noNamedDefault = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");

package/src/rules/no-named-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.noNamedExport = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
@@ -80,26 +75,20 @@ exports.noNamedExport = (0, eslint_devkit_1.createRule)({
     ],
     create(context) {
         const [options] = context.options;
-        const { allowInFiles = [], allowNames = [], allowPatterns = [],
-        // suggestDefault // unused option
-         } = options || {};
+        const { allowInFiles = [], allowNames = [], allowPatterns = [], } = options || {};
         const filename = context.filename;
         function shouldAllow(specifier) {
             if (!filename) {
                 return false;
             }
-            // Check if file is in allowed list
             const allowedByFile = allowInFiles.some((pattern) => {
                 if (pattern.includes('*')) {
                     const regexStr = '^' + pattern
                         .replace(/[.+?^${}()|[\]\\]/g, (match) => {
-                        // The char class above never matches '*', so no special-casing needed.
                         return '\\' + match;
                     })
                         .replace(/\*\*/g, '.*')
                         .replace(/\*/g, '[^/]*') + '$';
-                    // regexStr escapes every regex metacharacter before translating
-                    // glob tokens, so new RegExp() can never throw here.
                     const regex = new RegExp(regexStr);
                     return regex.test(filename);
                 }
@@ -108,18 +97,14 @@ exports.noNamedExport = (0, eslint_devkit_1.createRule)({
             if (allowedByFile) {
                 return true;
             }
-            // Check if filename matches allowed patterns
             const allowedByPattern = allowPatterns.some((pattern) => {
                 if (pattern.includes('*')) {
                     const regexStr = '^' + pattern
                         .replace(/[.+?^${}()|[\]\\]/g, (match) => {
-                        // The char class above never matches '*', so no special-casing needed.
                         return '\\' + match;
                     })
                         .replace(/\*\*/g, '.*')
                         .replace(/\*/g, '[^/]*') + '$';
-                    // regexStr escapes every regex metacharacter before translating
-                    // glob tokens, so new RegExp() can never throw here.
                     const regex = new RegExp(regexStr);
                     return regex.test(filename);
                 }
@@ -128,7 +113,6 @@ exports.noNamedExport = (0, eslint_devkit_1.createRule)({
             if (allowedByPattern) {
                 return true;
             }
-            // Check if specific export name is allowed
             if (specifier &&
                 specifier.exported.type === 'Identifier' &&
                 allowNames.includes(specifier.exported.name)) {
@@ -138,17 +122,14 @@ exports.noNamedExport = (0, eslint_devkit_1.createRule)({
         }
         return {
             ExportNamedDeclaration(node) {
-                // Skip type exports
                 if (node.exportKind === 'type' ||
                     (node.declaration &&
                         (node.declaration.type === 'TSInterfaceDeclaration' ||
                             node.declaration.type === 'TSTypeAliasDeclaration'))) {
                     return;
                 }
-                // Check each specifier individually
                 if (node.specifiers && node.specifiers.length > 0) {
                     for (const specifier of node.specifiers) {
-                        // Allow re-exporting default as named export
                         if (specifier.exported.type === 'Identifier' &&
                             specifier.exported.name !== 'default' &&
                             !shouldAllow(specifier)) {
@@ -161,10 +142,8 @@ exports.noNamedExport = (0, eslint_devkit_1.createRule)({
                         }
                     }
                 }
-                // Check for export declarations (export const foo = 1;)
                 else if (node.declaration) {
                     if (node.declaration.type === 'VariableDeclaration') {
-                        // Report on each variable declarator individually
                         for (const declarator of node.declaration.declarations) {
                             const varName = declarator.id.type === 'Identifier'
                                 ? declarator.id.name

package/src/rules/no-namespace.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.noNamespace = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
@@ -47,11 +42,6 @@ exports.noNamespace = (0, eslint_devkit_1.createRule)({
         const ignorePatterns = options.ignore || [];
         function isIgnored(source) {
             return ignorePatterns.some((pattern) => {
-                // Escape ALL regex metacharacters — including `*`, `?`, and `\` — so
-                // the subsequent replaces can find the now-escaped `\*` and `\?` and
-                // translate them back to glob meanings. Without escaping `*` and `?`,
-                // the inner replaces don't match anything and raw `*` / `?` reach
-                // `new RegExp()` (CodeQL: `js/incomplete-sanitization`).
                 const escaped = pattern.replace(/[.+^${}()|[\]\\*?]/g, '\\$&');
                 const regex = new RegExp('^' +
                     escaped

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

@@ -1,14 +1,8 @@
 "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.noNodejsModules = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
 const eslint_devkit_2 = require("@interlace/eslint-devkit");
-// Node.js builtin modules that should be flagged
 const NODEJS_BUILTINS = new Set([
     'assert',
     'buffer',
@@ -42,7 +36,6 @@ const NODEJS_BUILTINS = new Set([
     'module',
     'process',
 ]);
-// Alternative suggestions for common builtins
 const BUILTIN_ALTERNATIVES = {
     fs: 'Use platform-specific file APIs or isomorphic libraries',
     path: 'Use URL constructor or path utilities from your bundler',
@@ -131,23 +124,18 @@ exports.noNodejsModules = (0, eslint_devkit_1.createRule)({
     create(context) {
         const [options] = context.options;
         const { allow = [], additionalBuiltins = [], suggestAlternatives = true, } = options || {};
-        // Create set of all builtins to check
         const allBuiltins = new Set([...NODEJS_BUILTINS, ...additionalBuiltins]);
-        // Create set of allowed builtins
         const allowedBuiltins = new Set(allow);
         function isNodejsBuiltin(moduleName) {
-            // Check exact match
             if (allBuiltins.has(moduleName)) {
                 return !allowedBuiltins.has(moduleName);
             }
-            // Check node: protocol prefix (Node.js 14.18.0+)
             if (moduleName.startsWith('node:')) {
                 const builtinName = moduleName.slice(5);
                 return (allBuiltins.has(builtinName) && !allowedBuiltins.has(builtinName));
             }
             return false;
         }
-        // oxlint-disable-next-line consistent-function-scoping
         function getBuiltinName(moduleName) {
             if (moduleName.startsWith('node:')) {
                 return moduleName.slice(5);
@@ -205,7 +193,6 @@ exports.noNodejsModules = (0, eslint_devkit_1.createRule)({
                 }
             },
             CallExpression(node) {
-                // Check require() calls
                 if (node.callee.type === 'Identifier' &&
                     node.callee.name === 'require' &&
                     node.arguments.length === 1) {
@@ -218,7 +205,6 @@ exports.noNodejsModules = (0, eslint_devkit_1.createRule)({
                     }
                 }
             },
-            // Check dynamic imports via ImportExpression
             ImportExpression(node) {
                 const source = node.source;
                 if (source.type === 'Literal' && typeof source.value === 'string') {
@@ -229,7 +215,6 @@ exports.noNodejsModules = (0, eslint_devkit_1.createRule)({
                 }
             },
             TSImportEqualsDeclaration(node) {
-                // Handle TypeScript import = require() syntax
                 if (node.moduleReference.type === 'TSExternalModuleReference' &&
                     node.moduleReference.expression.type === 'Literal' &&
                     typeof node.moduleReference.expression.value === 'string') {

package/src/rules/no-relative-packages.js

@@ -1,16 +1,10 @@
 "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.noRelativePackages = void 0;
 const tslib_1 = require("tslib");
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
 const path = tslib_1.__importStar(require("node:path"));
 const fs = tslib_1.__importStar(require("node:fs"));
-// Cache for package.json lookups
 const packageJsonCache = new Map();
 function findPackageJson(startDir) {
     if (packageJsonCache.has(startDir)) {
@@ -71,22 +65,17 @@ exports.noRelativePackages = (0, eslint_devkit_1.createRule)({
             const importPath = source.value;
             if (typeof importPath !== 'string')
                 return;
-            // Only check relative imports
             if (!importPath.startsWith('./') && !importPath.startsWith('../')) {
                 return;
             }
-            // Resolve the import path
             const resolvedPath = path.resolve(currentDir, importPath);
             const importPackageRoot = findPackageJson(path.dirname(resolvedPath));
-            // Skip if we're in the same package
             if (allowSamePackage && currentPackageRoot === importPackageRoot) {
                 return;
             }
-            // Skip if either path doesn't have a package.json
             if (!currentPackageRoot || !importPackageRoot) {
                 return;
             }
-            // Different packages - report error
             if (currentPackageRoot !== importPackageRoot) {
                 try {
                     const pkgJsonPath = path.join(importPackageRoot, 'package.json');
@@ -100,9 +89,6 @@ exports.noRelativePackages = (0, eslint_devkit_1.createRule)({
                             packageName,
                         },
                         fix(fixer) {
-                            // Calculate the subpath within the package
-                            // subPath is never '' or '.': the package root is found by walking
-                            // UP from the resolved path's parent, so it is a strict ancestor.
                             const subPath = path.relative(importPackageRoot, resolvedPath);
                             const newImport = `${packageName}/${subPath.replace(/\.[^/.]+$/, '')}`;
                             return fixer.replaceText(source, `'${newImport}'`);
@@ -110,7 +96,6 @@ exports.noRelativePackages = (0, eslint_devkit_1.createRule)({
                     });
                 }
                 catch {
-                    // If we can't read package.json, just report without fix
                     context.report({
                         node: source,
                         messageId: 'relativePackage',

package/src/rules/no-relative-parent-imports.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.noRelativeParentImports = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
@@ -72,7 +67,6 @@ exports.noRelativeParentImports = (0, eslint_devkit_1.createRule)({
         const [options] = context.options;
         const { commonjs = true, amd = false } = options || {};
         function checkImport(importPath, node) {
-            // Check if the import path starts with .. (parent directory)
             if (importPath.startsWith('..')) {
                 context.report({
                     node,
@@ -88,7 +82,6 @@ exports.noRelativeParentImports = (0, eslint_devkit_1.createRule)({
                 }
             },
             CallExpression(node) {
-                // Check require() calls if commonjs is enabled
                 if (commonjs &&
                     node.callee.type === 'Identifier' &&
                     node.callee.name === 'require' &&
@@ -98,12 +91,10 @@ exports.noRelativeParentImports = (0, eslint_devkit_1.createRule)({
                         checkImport(arg.value, arg);
                     }
                 }
-                // Check AMD define/require calls if amd is enabled
                 if (amd &&
                     node.callee.type === 'Identifier' &&
                     (node.callee.name === 'define' || node.callee.name === 'require') &&
                     node.arguments.length >= 1) {
-                    // AMD dependencies are usually in the first argument (array of strings)
                     const depsArg = node.arguments[0];
                     if (depsArg.type === 'ArrayExpression') {
                         depsArg.elements.forEach((element) => {
@@ -114,10 +105,8 @@ exports.noRelativeParentImports = (0, eslint_devkit_1.createRule)({
                         });
                     }
                 }
-                // Note: Dynamic imports (import()) are handled by ImportExpression visitor
             },
             TSImportEqualsDeclaration(node) {
-                // Handle TypeScript import = require() syntax
                 if (node.moduleReference.type === 'TSExternalModuleReference' &&
                     node.moduleReference.expression.type === 'Literal' &&
                     typeof node.moduleReference.expression.value === 'string') {

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

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

@@ -1,26 +1,15 @@
 "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.noSelfImport = void 0;
 const tslib_1 = require("tslib");
-/**
- * ESLint Rule: no-self-import
- * Forbid a module from importing itself (eslint-plugin-import inspired)
- */
 const path = tslib_1.__importStar(require("node:path"));
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
 const eslint_devkit_2 = require("@interlace/eslint-devkit");
 function isImportingSelf(context, node, importPath) {
     const filename = context.filename;
-    // If the input is from stdin, this test can't fail
     if (filename === '<text>') {
         return false;
     }
-    // Skip test files if allowed
     const [options] = context.options;
     const { allowInTests = false } = options || {};
     if (allowInTests &&
@@ -30,22 +19,17 @@ function isImportingSelf(context, node, importPath) {
             filename.includes('__tests__'))) {
         return false;
     }
-    // Resolve the import path
     let resolvedPath;
     if (importPath.startsWith('./') || importPath.startsWith('../')) {
-        // Relative import - resolve relative to current file
         const currentDir = path.dirname(filename);
         resolvedPath = path.resolve(currentDir, importPath);
     }
     else if (importPath.startsWith('/')) {
-        // Absolute import
         resolvedPath = importPath;
     }
     else {
-        // Module import (like 'lodash') - not a self-import
         return false;
     }
-    // Compare resolved paths (normalize by removing extensions)
     const normalizedCurrent = filename.replace(/\.[^/.]+$/, '');
     const normalizedImport = resolvedPath.replace(/\.[^/.]+$/, '');
     return normalizedCurrent === normalizedImport;
@@ -84,7 +68,6 @@ exports.noSelfImport = (0, eslint_devkit_1.createRule)({
     },
     defaultOptions: [{ allowInTests: false }],
     create(context) {
-        // Visit all module imports and requires
         function visitModule(source, node) {
             const importPath = source.value;
             if (typeof importPath === 'string' &&
@@ -97,14 +80,12 @@ exports.noSelfImport = (0, eslint_devkit_1.createRule)({
         }
         return {
             ImportDeclaration(node) {
-                // Allow type imports in TypeScript
                 if (node.importKind === 'type') {
                     return;
                 }
                 visitModule(node.source, node);
             },
             CallExpression(node) {
-                // Handle require() calls
                 if (node.callee.type === 'Identifier' &&
                     node.callee.name === 'require' &&
                     node.arguments.length === 1) {
@@ -115,19 +96,16 @@ exports.noSelfImport = (0, eslint_devkit_1.createRule)({
                 }
             },
             ImportExpression(node) {
-                // Handle dynamic imports - only check if source is a literal
                 if (node.source.type === 'Literal') {
                     visitModule(node.source, node);
                 }
             },
             ExportNamedDeclaration(node) {
-                // Handle re-exports
                 if (node.source) {
                     visitModule(node.source, node);
                 }
             },
             ExportAllDeclaration(node) {
-                // Handle re-exports
                 visitModule(node.source, node);
             },
         };

package/src/rules/no-unassigned-import.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.noUnassignedImport = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
@@ -79,17 +74,14 @@ exports.noUnassignedImport = (0, eslint_devkit_1.createRule)({
                 if (shouldAllow(importPath)) {
                     return;
                 }
-                // Check if import has any specifiers
                 const hasSpecifiers = node.specifiers && node.specifiers.length > 0;
                 if (!hasSpecifiers) {
-                    // This is a bare import like: import 'module'
                     context.report({
                         node: node.source,
                         messageId: 'unassignedImport',
                     });
                 }
             },
-            // Also check require() calls
             CallExpression(node) {
                 if (node.callee.type === eslint_devkit_1.AST_NODE_TYPES.Identifier &&
                     node.callee.name === 'require' &&
@@ -98,18 +90,12 @@ exports.noUnassignedImport = (0, eslint_devkit_1.createRule)({
                     if (arg.type === 'Literal' && typeof arg.value === 'string') {
                         if (!shouldAllow(arg.value)) {
                             const parent = node.parent;
-                            // Only report if the parent is an ExpressionStatement
-                            // This strictly catches `require('foo');` which is a side-effect import.
-                            // Any other parent (VariableDecl, Assignment, Call, Conditional, etc.) means it's used.
                             if (parent && parent.type === eslint_devkit_1.AST_NODE_TYPES.ExpressionStatement) {
                                 context.report({
                                     node: arg,
                                     messageId: 'unassignedImport',
                                 });
                             }
-                            // Also check SequenceExpression: (a, require('b')) -> b is returned, a is unassigned.
-                            // But strictly `no-unassigned-import` might only care about top level.
-                            // For safety and compatibility, we report if it's in a sequence and NOT the last item.
                             else if (parent &&
                                 parent.type === eslint_devkit_1.AST_NODE_TYPES.SequenceExpression) {
                                 const expressions = parent.expressions;