eslint-plugin-import-next 2.3.3 → 2.3.6

package/src/rules/no-duplicates.js

@@ -13,6 +13,7 @@ exports.noDuplicates = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-duplicates.md',
             description: 'Reports duplicate imports',
         },
         fixable: 'code',

package/src/rules/no-dynamic-require.js

@@ -12,7 +12,10 @@ exports.noDynamicRequire = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-dynamic-require.md',
             description: 'Forbid require() calls with expressions',
+            cwe: 'CWE-829',
+            cvss: 5,
         },
         messages: {
             dynamicRequire: (0, eslint_devkit_1.formatLLMMessage)({

package/src/rules/no-empty-named-blocks.js

@@ -12,7 +12,10 @@ exports.noEmptyNamedBlocks = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-empty-named-blocks.md',
             description: 'Forbid empty named import blocks',
+            cwe: 'CWE-561',
+            cvss: 2.5,
         },
         fixable: 'code',
         hasSuggestions: true,
@@ -40,7 +43,7 @@ exports.noEmptyNamedBlocks = (0, eslint_devkit_1.createRule)({
                 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.getSourceCode();
+                const sourceCode = context.sourceCode;
                 const importText = sourceCode.getText(node);
                 // Check for empty braces pattern: {}
                 const hasEmptyBraces = /\{\s*\}/.test(importText);

package/src/rules/no-extraneous-dependencies.js

@@ -15,6 +15,7 @@ exports.noExtraneousDependencies = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-extraneous-dependencies.md',
             description: 'Forbid the use of extraneous packages not listed in package.json',
         },
         hasSuggestions: true,
@@ -134,7 +135,7 @@ exports.noExtraneousDependencies = (0, eslint_devkit_1.createRule)({
     create(context) {
         const [options] = context.options;
         const { devDependencies = true, optionalDependencies = true, peerDependencies = true, bundledDependencies = true, packageJsonPath, packageJson, resolutionStrategy = 'strict', allowPatterns = [], } = options || {};
-        const filename = context.getFilename();
+        const filename = context.filename;
         if (!filename) {
             return {};
         }
@@ -188,6 +189,7 @@ exports.noExtraneousDependencies = (0, eslint_devkit_1.createRule)({
         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('../')) {
@@ -265,6 +267,7 @@ exports.noExtraneousDependencies = (0, eslint_devkit_1.createRule)({
                 : 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/') ||

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

@@ -79,6 +79,7 @@ exports.noFullPackageImport = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-full-package-import.md',
             description: 'Disallow full package imports from known large packages that prevent tree-shaking',
         },
         hasSuggestions: true,

package/src/rules/no-import-module-exports.js

@@ -12,7 +12,10 @@ exports.noImportModuleExports = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-import-module-exports.md',
             description: 'Forbid import statements with CommonJS module.exports',
+            cwe: 'CWE-1078',
+            cvss: 7.5,
         },
         fixable: 'code',
         messages: {

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

@@ -87,6 +87,7 @@ exports.noInternalModules = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-internal-modules.md',
             description: 'Forbid importing the submodules of other modules',
         },
         fixable: 'code',

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

@@ -25,6 +25,7 @@ exports.noLegacyImports = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-legacy-imports.md',
             description: 'Detect imports from deprecated internal paths and suggest alternatives',
         },
         fixable: 'code',

package/src/rules/no-mutable-exports.js

@@ -13,6 +13,7 @@ exports.noMutableExports = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-mutable-exports.md',
             description: 'Forbid the use of mutable exports with `var` or `let`',
         },
         hasSuggestions: false,
@@ -70,7 +71,7 @@ exports.noMutableExports = (0, eslint_devkit_1.createRule)({
     create(context) {
         const [options] = context.options;
         const { allowInFiles = [], ignoreExports = [] } = options || {};
-        const filename = context.getFilename();
+        const filename = context.filename;
         if (!filename) {
             return {};
         }

package/src/rules/no-named-as-default-member.js

@@ -12,7 +12,10 @@ exports.noNamedAsDefaultMember = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-named-as-default-member.md',
             description: 'Forbid use of exported name as property of default export',
+            cwe: 'CWE-1078',
+            cvss: 2.5,
         },
         messages: {
             namedAsDefaultMember: (0, eslint_devkit_1.formatLLMMessage)({

package/src/rules/no-named-as-default.js

@@ -12,7 +12,10 @@ exports.noNamedAsDefault = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-named-as-default.md',
             description: 'Forbid use of exported name as identifier of default export',
+            cwe: 'CWE-1078',
+            cvss: 5,
         },
         messages: {
             namedAsDefault: (0, eslint_devkit_1.formatLLMMessage)({

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

@@ -12,7 +12,10 @@ exports.noNamedDefault = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-named-default.md',
             description: 'Forbid named default exports',
+            cwe: 'CWE-1078',
+            cvss: 2.5,
         },
         messages: {
             namedDefault: (0, eslint_devkit_1.formatLLMMessage)({

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

@@ -13,6 +13,7 @@ exports.noNamedExport = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-named-export.md',
             description: 'Prevents named exports',
         },
         hasSuggestions: false,
@@ -82,7 +83,7 @@ exports.noNamedExport = (0, eslint_devkit_1.createRule)({
         const { allowInFiles = [], allowNames = [], allowPatterns = [],
         // suggestDefault // unused option
          } = options || {};
-        const filename = context.getFilename();
+        const filename = context.filename;
         function shouldAllow(specifier) {
             if (!filename) {
                 return false;

package/src/rules/no-namespace.js

@@ -12,7 +12,10 @@ exports.noNamespace = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-namespace.md',
             description: 'Forbid namespace (a.k.a. "wildcard" *) imports',
+            cwe: 'CWE-1078',
+            cvss: 2.5,
         },
         fixable: 'code',
         messages: {
@@ -45,12 +48,16 @@ exports.noNamespace = (0, eslint_devkit_1.createRule)({
         const ignorePatterns = options.ignore || [];
         function isIgnored(source) {
             return ignorePatterns.some((pattern) => {
-                // Simple glob matching
+                // 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('^' +
-                    pattern
-                        .replace(/\*/g, '.*')
-                        .replace(/\?/g, '.')
-                        .replace(/\//g, '\\/') +
+                    escaped
+                        .replace(/\\\*/g, '.*')
+                        .replace(/\\\?/g, '.') +
                     '$');
                 return regex.test(source);
             });

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

@@ -64,6 +64,7 @@ exports.noNodejsModules = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-nodejs-modules.md',
             description: 'Prevents Node.js builtin imports',
         },
         messages: {
@@ -146,6 +147,7 @@ exports.noNodejsModules = (0, eslint_devkit_1.createRule)({
             }
             return false;
         }
+        // oxlint-disable-next-line consistent-function-scoping
         function getBuiltinName(moduleName) {
             if (moduleName.startsWith('node:')) {
                 return moduleName.slice(5);
@@ -189,7 +191,7 @@ exports.noNodejsModules = (0, eslint_devkit_1.createRule)({
                 data: {
                     moduleName,
                     builtinName,
-                    currentFile: context.getFilename(),
+                    currentFile: context.filename,
                     alternative,
                     suggestion: fixSuggestion,
                 },

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

@@ -33,7 +33,10 @@ exports.noRelativePackages = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-relative-packages.md',
             description: 'Forbid importing packages through relative paths',
+            cwe: 'CWE-1047',
+            cvss: 5,
         },
         fixable: 'code',
         messages: {
@@ -61,7 +64,7 @@ exports.noRelativePackages = (0, eslint_devkit_1.createRule)({
     create(context) {
         const options = context.options[0] || {};
         const { allowSamePackage = true } = options;
-        const filename = context.getFilename();
+        const filename = context.filename;
         const currentDir = path.dirname(filename);
         const currentPackageRoot = findPackageJson(currentDir);
         function checkImport(node, source) {

package/src/rules/no-relative-parent-imports.js

@@ -13,6 +13,7 @@ exports.noRelativeParentImports = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-relative-parent-imports.md',
             description: 'Prevents ../ imports',
         },
         hasSuggestions: false,

package/src/rules/no-restricted-paths.js

@@ -13,6 +13,7 @@ exports.noRestrictedPaths = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-restricted-paths.md',
             description: 'Enforce which files can be imported in a given folder',
         },
         hasSuggestions: false,

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

@@ -15,7 +15,7 @@ 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.getFilename();
+    const filename = context.filename;
     // If the input is from stdin, this test can't fail
     if (filename === '<text>') {
         return false;
@@ -55,6 +55,7 @@ exports.noSelfImport = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-self-import.md',
             description: 'Forbid a module from importing itself',
         },
         messages: {

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

@@ -13,6 +13,7 @@ exports.noUnassignedImport = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-unassigned-import.md',
             description: 'Prevents unassigned imports',
         },
         hasSuggestions: false,

package/src/rules/no-unresolved.js

@@ -19,6 +19,7 @@ exports.noUnresolved = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-unresolved.md',
             description: 'Ensures imports point to resolvable modules',
         },
         hasSuggestions: true,
@@ -117,7 +118,7 @@ exports.noUnresolved = (0, eslint_devkit_1.createRule)({
                 node_module_1.builtinModules.includes(importPath)) {
                 return;
             }
-            const filePath = context.filename || context.getFilename();
+            const filePath = context.filename;
             const resolved = (0, eslint_devkit_3.resolveModule)(importPath, filePath, {
                 extensions: options?.extensions,
             });
@@ -149,7 +150,7 @@ exports.noUnresolved = (0, eslint_devkit_1.createRule)({
                                     }
                                     statement = statement.parent;
                                 }
-                                return fixer.insertTextBefore(statement, '// eslint-disable-next-line eslint-plugin-llm-optimized/no-unresolved\n');
+                                return fixer.insertTextBefore(statement, '// eslint-disable-next-line import-next/no-unresolved\n');
                             },
                         },
                     ],

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

@@ -13,6 +13,7 @@ exports.noUnusedModules = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-unused-modules.md',
             description: 'Forbid modules without exports',
         },
         hasSuggestions: false,

package/src/rules/no-useless-path-segments.js

@@ -14,7 +14,10 @@ exports.noUselessPathSegments = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/no-useless-path-segments.md',
             description: 'Forbid unnecessary path segments in import and require statements',
+            cwe: 'CWE-1078',
+            cvss: 2.5,
         },
         fixable: 'code',
         messages: {

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

@@ -13,6 +13,7 @@ exports.preferDefaultExport = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/prefer-default-export.md',
             description: 'Prefer a default export if module exports a single name',
         },
         hasSuggestions: true,
@@ -77,7 +78,7 @@ exports.preferDefaultExport = (0, eslint_devkit_1.createRule)({
     create(context) {
         const [options] = context.options;
         const { target = 'single', ignoreFiles = [], allowNamedExports = false, } = options || {};
-        const filename = context.getFilename();
+        const filename = context.filename;
         if (!filename) {
             return {};
         }

package/src/rules/prefer-direct-import.js

@@ -47,6 +47,7 @@ exports.preferDirectImport = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/prefer-direct-import.md',
             description: 'Prefer direct imports over barrel imports for better tree-shaking and build performance',
         },
         fixable: 'code',

package/src/rules/prefer-modern-api.js

@@ -126,6 +126,7 @@ exports.preferModernApi = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/prefer-modern-api.md',
             description: 'Suggest modern replacements for deprecated or outdated libraries',
         },
         messages: {

package/src/rules/prefer-node-protocol.js

@@ -58,6 +58,7 @@ exports.preferNodeProtocol = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/prefer-node-protocol.md',
             description: 'Prefer using the node: protocol when importing Node.js builtin modules',
         },
         fixable: 'code',
@@ -160,6 +161,7 @@ exports.preferNodeProtocol = (0, eslint_devkit_1.createRule)({
                 }
             },
         };
+        // oxlint-disable-next-line consistent-function-scoping
         function isBuiltInModule(moduleName, modulesToCheck) {
             // Skip if already using node: protocol
             if (moduleName.startsWith('node:')) {

package/src/rules/prefer-tree-shakeable-imports.js

@@ -39,6 +39,7 @@ exports.preferTreeShakeableImports = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/prefer-tree-shakeable-imports.md',
             description: 'Prefer import patterns that enable effective tree-shaking',
         },
         hasSuggestions: true,

package/src/rules/require-import-approval.js

@@ -44,6 +44,7 @@ exports.requireImportApproval = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'problem',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/require-import-approval.md',
             description: 'Enforce explicit approval for high-risk package imports',
         },
         messages: {

package/src/rules/unambiguous.js

@@ -12,7 +12,10 @@ exports.unambiguous = (0, eslint_devkit_1.createRule)({
     meta: {
         type: 'suggestion',
         docs: {
+            url: 'https://github.com/ofri-peretz/eslint/blob/main/packages/eslint-plugin-import-next/docs/rules/unambiguous.md',
             description: 'Forbid potentially ambiguous parse goal (script vs. module)',
+            cwe: 'CWE-1078',
+            cvss: 2.5,
         },
         messages: {
             ambiguous: (0, eslint_devkit_1.formatLLMMessage)({