eslint 9.13.0 → 9.14.0

package/README.md

@@ -299,8 +299,8 @@ to get your logo on our READMEs and [website](https://eslint.org/sponsors).
 <h3>Platinum Sponsors</h3>
 <p><a href="https://automattic.com"><img src="https://images.opencollective.com/automattic/d0ef3e1/logo.png" alt="Automattic" height="128"></a> <a href="https://www.airbnb.com/"><img src="https://images.opencollective.com/airbnb/d327d66/logo.png" alt="Airbnb" height="128"></a></p><h3>Gold Sponsors</h3>
 <p><a href="https://trunk.io/"><img src="https://images.opencollective.com/trunkio/fb92d60/avatar.png" alt="trunk.io" height="96"></a></p><h3>Silver Sponsors</h3>
-<p><a href="https://www.jetbrains.com/"><img src="https://images.opencollective.com/jetbrains/fe76f99/logo.png" alt="JetBrains" height="64"></a> <a href="https://liftoff.io/"><img src="https://images.opencollective.com/liftoff/5c4fa84/logo.png" alt="Liftoff" height="64"></a> <a href="https://americanexpress.io"><img src="https://avatars.githubusercontent.com/u/3853301?v=4" alt="American Express" height="64"></a> <a href="https://www.workleap.com"><img src="https://avatars.githubusercontent.com/u/53535748?u=d1e55d7661d724bf2281c1bfd33cb8f99fe2465f&v=4" alt="Workleap" height="64"></a></p><h3>Bronze Sponsors</h3>
-<p><a href="https://www.wordhint.net/"><img src="https://images.opencollective.com/wordhint/be86813/avatar.png" alt="WordHint" height="32"></a> <a href="https://www.crosswordsolver.org/anagram-solver/"><img src="https://images.opencollective.com/anagram-solver/2666271/logo.png" alt="Anagram Solver" height="32"></a> <a href="https://icons8.com/"><img src="https://images.opencollective.com/icons8/7fa1641/logo.png" alt="Icons8" height="32"></a> <a href="https://discord.com"><img src="https://images.opencollective.com/discordapp/f9645d9/logo.png" alt="Discord" height="32"></a> <a href="https://www.gitbook.com"><img src="https://avatars.githubusercontent.com/u/7111340?v=4" alt="GitBook" height="32"></a> <a href="https://nx.dev"><img src="https://avatars.githubusercontent.com/u/23692104?v=4" alt="Nx" height="32"></a> <a href="https://herocoders.com"><img src="https://avatars.githubusercontent.com/u/37549774?v=4" alt="HeroCoders" height="32"></a> <a href="https://usenextbase.com"><img src="https://avatars.githubusercontent.com/u/145838380?v=4" alt="Nextbase Starter Kit" height="32"></a></p>
+<p><a href="https://www.serptriumph.com/"><img src="https://images.opencollective.com/serp-triumph5/fea3074/logo.png" alt="SERP Triumph" height="64"></a> <a href="https://www.jetbrains.com/"><img src="https://images.opencollective.com/jetbrains/fe76f99/logo.png" alt="JetBrains" height="64"></a> <a href="https://liftoff.io/"><img src="https://images.opencollective.com/liftoff/5c4fa84/logo.png" alt="Liftoff" height="64"></a> <a href="https://americanexpress.io"><img src="https://avatars.githubusercontent.com/u/3853301?v=4" alt="American Express" height="64"></a> <a href="https://www.workleap.com"><img src="https://avatars.githubusercontent.com/u/53535748?u=d1e55d7661d724bf2281c1bfd33cb8f99fe2465f&v=4" alt="Workleap" height="64"></a></p><h3>Bronze Sponsors</h3>
+<p><a href="https://cybozu.co.jp/"><img src="https://images.opencollective.com/cybozu/933e46d/logo.png" alt="Cybozu" height="32"></a> <a href="https://www.wordhint.net/"><img src="https://images.opencollective.com/wordhint/be86813/avatar.png" alt="WordHint" height="32"></a> <a href="https://www.crosswordsolver.org/anagram-solver/"><img src="https://images.opencollective.com/anagram-solver/2666271/logo.png" alt="Anagram Solver" height="32"></a> <a href="https://icons8.com/"><img src="https://images.opencollective.com/icons8/7fa1641/logo.png" alt="Icons8" height="32"></a> <a href="https://discord.com"><img src="https://images.opencollective.com/discordapp/f9645d9/logo.png" alt="Discord" height="32"></a> <a href="https://www.gitbook.com"><img src="https://avatars.githubusercontent.com/u/7111340?v=4" alt="GitBook" height="32"></a> <a href="https://nx.dev"><img src="https://avatars.githubusercontent.com/u/23692104?v=4" alt="Nx" height="32"></a> <a href="https://herocoders.com"><img src="https://avatars.githubusercontent.com/u/37549774?v=4" alt="HeroCoders" height="32"></a></p>
 <h3>Technology Sponsors</h3>
 Technology sponsors allow us to use their products and services for free as part of a contribution to the open source ecosystem and our work.
 <p><a href="https://netlify.com"><img src="https://raw.githubusercontent.com/eslint/eslint.org/main/src/assets/images/techsponsors/netlify-icon.svg" alt="Netlify" height="32"></a> <a href="https://algolia.com"><img src="https://raw.githubusercontent.com/eslint/eslint.org/main/src/assets/images/techsponsors/algolia-icon.svg" alt="Algolia" height="32"></a> <a href="https://1password.com"><img src="https://raw.githubusercontent.com/eslint/eslint.org/main/src/assets/images/techsponsors/1password-icon.svg" alt="1Password" height="32"></a></p>

package/lib/config/config-loader.js

@@ -14,7 +14,7 @@ const fs = require("node:fs/promises");
 const findUp = require("find-up");
 const { pathToFileURL } = require("node:url");
 const debug = require("debug")("eslint:config-loader");
-const { FlatConfigArray } = require("../config/flat-config-array");
+const { FlatConfigArray } = require("./flat-config-array");
 
 //-----------------------------------------------------------------------------
 // Types
@@ -206,95 +206,6 @@ async function loadConfigFile(filePath, allowTS) {
     return config;
 }
 
-/**
- * Calculates the config array for this run based on inputs.
- * @param {string} configFilePath The absolute path to the config file to use if not overridden.
- * @param {string} basePath The base path to use for relative paths in the config file.
- * @param {ConfigLoaderOptions} options The options to use when loading configuration files.
- * @returns {Promise<FlatConfigArray>} The config array for `eslint`.
- */
-async function calculateConfigArray(configFilePath, basePath, options) {
-
-    const {
-        cwd,
-        baseConfig,
-        ignoreEnabled,
-        ignorePatterns,
-        overrideConfig,
-        defaultConfigs = [],
-        allowTS
-    } = options;
-
-    debug(`Calculating config array from config file ${configFilePath} and base path ${basePath}`);
-
-    const configs = new FlatConfigArray(baseConfig || [], { basePath, shouldIgnore: ignoreEnabled });
-
-    // load config file
-    if (configFilePath) {
-
-        debug(`Loading config file ${configFilePath}`);
-        const fileConfig = await loadConfigFile(configFilePath, allowTS);
-
-        if (Array.isArray(fileConfig)) {
-            configs.push(...fileConfig);
-        } else {
-            configs.push(fileConfig);
-        }
-    }
-
-    // add in any configured defaults
-    configs.push(...defaultConfigs);
-
-    // append command line ignore patterns
-    if (ignorePatterns && ignorePatterns.length > 0) {
-
-        let relativeIgnorePatterns;
-
-        /*
-         * If the config file basePath is different than the cwd, then
-         * the ignore patterns won't work correctly. Here, we adjust the
-         * ignore pattern to include the correct relative path. Patterns
-         * passed as `ignorePatterns` are relative to the cwd, whereas
-         * the config file basePath can be an ancestor of the cwd.
-         */
-        if (basePath === cwd) {
-            relativeIgnorePatterns = ignorePatterns;
-        } else {
-
-            // relative path must only have Unix-style separators
-            const relativeIgnorePath = path.relative(basePath, cwd).replace(/\\/gu, "/");
-
-            relativeIgnorePatterns = ignorePatterns.map(pattern => {
-                const negated = pattern.startsWith("!");
-                const basePattern = negated ? pattern.slice(1) : pattern;
-
-                return (negated ? "!" : "") +
-                    path.posix.join(relativeIgnorePath, basePattern);
-            });
-        }
-
-        /*
-         * Ignore patterns are added to the end of the config array
-         * so they can override default ignores.
-         */
-        configs.push({
-            ignores: relativeIgnorePatterns
-        });
-    }
-
-    if (overrideConfig) {
-        if (Array.isArray(overrideConfig)) {
-            configs.push(...overrideConfig);
-        } else {
-            configs.push(overrideConfig);
-        }
-    }
-
-    await configs.normalize();
-
-    return configs;
-}
-
 //-----------------------------------------------------------------------------
 // Exports
 //-----------------------------------------------------------------------------
@@ -307,13 +218,13 @@ class ConfigLoader {
 
     /**
      * Map of config file paths to the config arrays for those directories.
-     * @type {Map<string, FlatConfigArray>}
+     * @type {Map<string, FlatConfigArray|Promise<FlatConfigArray>>}
      */
     #configArrays = new Map();
 
     /**
      * Map of absolute directory names to the config file paths for those directories.
-     * @type {Map<string, {configFilePath:string,basePath:string}>}
+     * @type {Map<string, {configFilePath:string,basePath:string}|Promise<{configFilePath:string,basePath:string}>>}
      */
     #configFilePaths = new Map();
 
@@ -347,43 +258,22 @@ class ConfigLoader {
             return this.#configFilePaths.get(fromDirectory);
         }
 
-        const configFilenames = this.#options.allowTS
-            ? [...FLAT_CONFIG_FILENAMES, ...TS_FLAT_CONFIG_FILENAMES]
-            : FLAT_CONFIG_FILENAMES;
-
-        // determine where to load config file from
-        let configFilePath;
-        const {
-            cwd,
-            configFile: useConfigFile
-        } = this.#options;
-        let basePath = cwd;
-
-        if (typeof useConfigFile === "string") {
-            debug(`Override config file path is ${useConfigFile}`);
-            configFilePath = path.resolve(cwd, useConfigFile);
-            basePath = cwd;
-        } else if (useConfigFile !== false) {
-            debug("Searching for eslint.config.js");
-            configFilePath = await findUp(
-                configFilenames,
-                { cwd: fromDirectory }
-            );
+        const resultPromise = ConfigLoader.locateConfigFileToUse({
+            useConfigFile: this.#options.configFile,
+            cwd: this.#options.cwd,
+            fromDirectory,
+            allowTS: this.#options.allowTS
+        });
 
-            if (configFilePath) {
-                basePath = path.dirname(configFilePath);
-            }
+        // ensure `ConfigLoader.locateConfigFileToUse` is called only once for `fromDirectory`
+        this.#configFilePaths.set(fromDirectory, resultPromise);
 
-        }
+        // Unwrap the promise. This is primarily for the sync `getCachedConfigArrayForPath` method.
+        const result = await resultPromise;
 
-        // cache the result
-        this.#configFilePaths.set(fromDirectory, { configFilePath, basePath });
-
-        return {
-            configFilePath,
-            basePath
-        };
+        this.#configFilePaths.set(fromDirectory, result);
 
+        return result;
     }
 
     /**
@@ -399,9 +289,14 @@ class ConfigLoader {
             return this.#configArrays.get(configFilePath);
         }
 
-        const configs = await calculateConfigArray(configFilePath, basePath, this.#options);
+        const configsPromise = ConfigLoader.calculateConfigArray(configFilePath, basePath, this.#options);
+
+        // ensure `ConfigLoader.calculateConfigArray` is called only once for `configFilePath`
+        this.#configArrays.set(configFilePath, configsPromise);
+
+        // Unwrap the promise. This is primarily for the sync `getCachedConfigArrayForPath` method.
+        const configs = await configsPromise;
 
-        // cache the config array for this instance
         this.#configArrays.set(configFilePath, configs);
 
         return configs;
@@ -512,9 +407,21 @@ class ConfigLoader {
             throw new Error(`Could not find config file for ${fileOrDirPath}`);
         }
 
-        const { configFilePath } = this.#configFilePaths.get(absoluteDirPath);
+        const configFilePathInfo = this.#configFilePaths.get(absoluteDirPath);
+
+        if (typeof configFilePathInfo.then === "function") {
+            throw new Error(`Config file path for ${fileOrDirPath} has not yet been calculated or an error occurred during the calculation`);
+        }
+
+        const { configFilePath } = configFilePathInfo;
+
+        const configArray = this.#configArrays.get(configFilePath);
 
-        return this.#configArrays.get(configFilePath);
+        if (!configArray || typeof configArray.then === "function") {
+            throw new Error(`Config array for ${fileOrDirPath} has not yet been calculated or an error occurred during the calculation`);
+        }
+
+        return configArray;
     }
 
     /**
@@ -526,6 +433,144 @@ class ConfigLoader {
         return import("jiti");
     }
 
+    /**
+     * Determines which config file to use. This is determined by seeing if an
+     * override config file was specified, and if so, using it; otherwise, as long
+     * as override config file is not explicitly set to `false`, it will search
+     * upwards from `fromDirectory` for a file named `eslint.config.js`.
+     * This method is exposed internally for testing purposes.
+     * @param {Object} [options] the options object
+     * @param {string|false|undefined} options.useConfigFile The path to the config file to use.
+     * @param {string} options.cwd Path to a directory that should be considered as the current working directory.
+     * @param {string} [options.fromDirectory] The directory from which to start searching. Defaults to `cwd`.
+     * @param {boolean} options.allowTS Indicates if TypeScript configuration files are allowed.
+     * @returns {Promise<{configFilePath:string|undefined,basePath:string}>} Location information for
+     *      the config file.
+     */
+    static async locateConfigFileToUse({ useConfigFile, cwd, fromDirectory = cwd, allowTS }) {
+
+        const configFilenames = allowTS
+            ? [...FLAT_CONFIG_FILENAMES, ...TS_FLAT_CONFIG_FILENAMES]
+            : FLAT_CONFIG_FILENAMES;
+
+        // determine where to load config file from
+        let configFilePath;
+        let basePath = cwd;
+
+        if (typeof useConfigFile === "string") {
+            debug(`Override config file path is ${useConfigFile}`);
+            configFilePath = path.resolve(cwd, useConfigFile);
+            basePath = cwd;
+        } else if (useConfigFile !== false) {
+            debug("Searching for eslint.config.js");
+            configFilePath = await findUp(
+                configFilenames,
+                { cwd: fromDirectory }
+            );
+
+            if (configFilePath) {
+                basePath = path.dirname(configFilePath);
+            }
+
+        }
+
+        return {
+            configFilePath,
+            basePath
+        };
+
+    }
+
+    /**
+     * Calculates the config array for this run based on inputs.
+     * This method is exposed internally for testing purposes.
+     * @param {string} configFilePath The absolute path to the config file to use if not overridden.
+     * @param {string} basePath The base path to use for relative paths in the config file.
+     * @param {ConfigLoaderOptions} options The options to use when loading configuration files.
+     * @returns {Promise<FlatConfigArray>} The config array for `eslint`.
+     */
+    static async calculateConfigArray(configFilePath, basePath, options) {
+
+        const {
+            cwd,
+            baseConfig,
+            ignoreEnabled,
+            ignorePatterns,
+            overrideConfig,
+            defaultConfigs = [],
+            allowTS
+        } = options;
+
+        debug(`Calculating config array from config file ${configFilePath} and base path ${basePath}`);
+
+        const configs = new FlatConfigArray(baseConfig || [], { basePath, shouldIgnore: ignoreEnabled });
+
+        // load config file
+        if (configFilePath) {
+
+            debug(`Loading config file ${configFilePath}`);
+            const fileConfig = await loadConfigFile(configFilePath, allowTS);
+
+            if (Array.isArray(fileConfig)) {
+                configs.push(...fileConfig);
+            } else {
+                configs.push(fileConfig);
+            }
+        }
+
+        // add in any configured defaults
+        configs.push(...defaultConfigs);
+
+        // append command line ignore patterns
+        if (ignorePatterns && ignorePatterns.length > 0) {
+
+            let relativeIgnorePatterns;
+
+            /*
+             * If the config file basePath is different than the cwd, then
+             * the ignore patterns won't work correctly. Here, we adjust the
+             * ignore pattern to include the correct relative path. Patterns
+             * passed as `ignorePatterns` are relative to the cwd, whereas
+             * the config file basePath can be an ancestor of the cwd.
+             */
+            if (basePath === cwd) {
+                relativeIgnorePatterns = ignorePatterns;
+            } else {
+
+                // relative path must only have Unix-style separators
+                const relativeIgnorePath = path.relative(basePath, cwd).replace(/\\/gu, "/");
+
+                relativeIgnorePatterns = ignorePatterns.map(pattern => {
+                    const negated = pattern.startsWith("!");
+                    const basePattern = negated ? pattern.slice(1) : pattern;
+
+                    return (negated ? "!" : "") +
+                    path.posix.join(relativeIgnorePath, basePattern);
+                });
+            }
+
+            /*
+             * Ignore patterns are added to the end of the config array
+             * so they can override default ignores.
+             */
+            configs.push({
+                ignores: relativeIgnorePatterns
+            });
+        }
+
+        if (overrideConfig) {
+            if (Array.isArray(overrideConfig)) {
+                configs.push(...overrideConfig);
+            } else {
+                configs.push(overrideConfig);
+            }
+        }
+
+        await configs.normalize();
+
+        return configs;
+    }
+
 }
 
 /**
@@ -542,13 +587,13 @@ class LegacyConfigLoader extends ConfigLoader {
 
     /**
      * The cached config file path for this instance.
-     * @type {{configFilePath:string,basePath:string}|undefined}
+     * @type {Promise<{configFilePath:string,basePath:string}|undefined>}
      */
     #configFilePath;
 
     /**
      * The cached config array for this instance.
-     * @type {FlatConfigArray}
+     * @type {FlatConfigArray|Promise<FlatConfigArray>}
      */
     #configArray;
 
@@ -569,50 +614,16 @@ class LegacyConfigLoader extends ConfigLoader {
      * @returns {Promise<{configFilePath:string|undefined,basePath:string}>} Location information for
      *      the config file.
      */
-    async #locateConfigFileToUse() {
-
-        // check cache first
-        if (this.#configFilePath) {
-            return this.#configFilePath;
-        }
-
-        const configFilenames = this.#options.allowTS
-            ? [...FLAT_CONFIG_FILENAMES, ...TS_FLAT_CONFIG_FILENAMES]
-            : FLAT_CONFIG_FILENAMES;
-
-        // determine where to load config file from
-        let configFilePath;
-        const {
-            cwd,
-            configFile: useConfigFile
-        } = this.#options;
-        let basePath = cwd;
-
-        if (typeof useConfigFile === "string") {
-            debug(`[Legacy]: Override config file path is ${useConfigFile}`);
-            configFilePath = path.resolve(cwd, useConfigFile);
-            basePath = cwd;
-        } else if (useConfigFile !== false) {
-            debug("[Legacy]: Searching for eslint.config.js");
-            configFilePath = await findUp(
-                configFilenames,
-                { cwd }
-            );
-
-            if (configFilePath) {
-                basePath = path.dirname(configFilePath);
-            }
-
+    #locateConfigFileToUse() {
+        if (!this.#configFilePath) {
+            this.#configFilePath = ConfigLoader.locateConfigFileToUse({
+                useConfigFile: this.#options.configFile,
+                cwd: this.#options.cwd,
+                allowTS: this.#options.allowTS
+            });
         }
 
-        // cache the result
-        this.#configFilePath = { configFilePath, basePath };
-
-        return {
-            configFilePath,
-            basePath
-        };
-
+        return this.#configFilePath;
     }
 
     /**
@@ -628,12 +639,13 @@ class LegacyConfigLoader extends ConfigLoader {
             return this.#configArray;
         }
 
-        const configs = await calculateConfigArray(configFilePath, basePath, this.#options);
+        // ensure `ConfigLoader.calculateConfigArray` is called only once
+        this.#configArray = ConfigLoader.calculateConfigArray(configFilePath, basePath, this.#options);
 
-        // cache the config array for this instance
-        this.#configArray = configs;
+        // Unwrap the promise. This is primarily for the sync `getCachedConfigArrayForPath` method.
+        this.#configArray = await this.#configArray;
 
-        return configs;
+        return this.#configArray;
     }
 
 
@@ -696,6 +708,10 @@ class LegacyConfigLoader extends ConfigLoader {
             throw new Error(`Could not find config file for ${dirPath}`);
         }
 
+        if (typeof this.#configArray.then === "function") {
+            throw new Error(`Config array for ${dirPath} has not yet been calculated or an error occurred during the calculation`);
+        }
+
         return this.#configArray;
     }
 }

package/lib/eslint/eslint-helpers.js

@@ -421,20 +421,19 @@ async function globMultiSearch({ searches, configLoader, errorOnUnmatchedPattern
         )
     );
 
-    const filePaths = [];
-
+    /*
+     * The first loop handles errors from the glob searches. Since we can't
+     * use `await` inside `flatMap`, we process errors separately in this loop.
+     * This results in two iterations over `results`, but since the length is
+     * less than or equal to the number of globs and directories passed on the
+     * command line, the performance impact should be minimal.
+     */
     for (let i = 0; i < results.length; i++) {
 
         const result = results[i];
         const currentSearch = normalizedSearches[i];
 
         if (result.status === "fulfilled") {
-
-            // if the search was successful just add the results
-            if (result.value.length > 0) {
-                filePaths.push(...result.value);
-            }
-
             continue;
         }
 
@@ -457,7 +456,8 @@ async function globMultiSearch({ searches, configLoader, errorOnUnmatchedPattern
 
     }
 
-    return filePaths;
+    // second loop for `fulfulled` results
+    return results.flatMap(result => result.value);
 
 }
 

package/lib/eslint/eslint.js

@@ -746,7 +746,7 @@ class ESLint {
         });
         const controller = new AbortController();
         const retryCodes = new Set(["ENFILE", "EMFILE"]);
-        const retrier = new Retrier(error => retryCodes.has(error.code));
+        const retrier = new Retrier(error => retryCodes.has(error.code), { concurrency: 100 });
 
         debug(`${filePaths.length} files found in: ${Date.now() - startTime}ms`);
 
@@ -830,7 +830,7 @@ class ESLint {
                         }
 
                         return result;
-                    }))
+                    }), { signal: controller.signal })
                     .catch(error => {
                         controller.abort(error);
                         throw error;

package/lib/rules/camelcase.js

@@ -238,6 +238,13 @@ module.exports = {
                 return;
             }
 
+            /*
+             * Import attribute keys are always ignored
+             */
+            if (astUtils.isImportAttributeKey(node)) {
+                return;
+            }
+
             report(node);
         }
 
@@ -272,7 +279,7 @@ module.exports = {
                 for (const reference of scope.through) {
                     const id = reference.identifier;
 
-                    if (isGoodName(id.name)) {
+                    if (isGoodName(id.name) || astUtils.isImportAttributeKey(id)) {
                         continue;
                     }
 
@@ -326,7 +333,7 @@ module.exports = {
                 "MethodDefinition > PrivateIdentifier.key",
                 "PropertyDefinition > PrivateIdentifier.key"
             ]](node) {
-                if (properties === "never" || isGoodName(node.name)) {
+                if (properties === "never" || astUtils.isImportAttributeKey(node) || isGoodName(node.name)) {
                     return;
                 }
                 report(node);

package/lib/rules/id-denylist.js

@@ -6,6 +6,12 @@
 
 "use strict";
 
+//------------------------------------------------------------------------------
+// Requirements
+//------------------------------------------------------------------------------
+
+const astUtils = require("./utils/ast-utils");
+
 //------------------------------------------------------------------------------
 // Helpers
 //------------------------------------------------------------------------------
@@ -154,6 +160,12 @@ module.exports = {
          * @returns {boolean} `true` if the node should be checked.
          */
         function shouldCheck(node) {
+
+            // Import attributes are defined by environments, so naming conventions shouldn't apply to them
+            if (astUtils.isImportAttributeKey(node)) {
+                return false;
+            }
+
             const parent = node.parent;
 
             /*

package/lib/rules/id-length.js

@@ -11,7 +11,7 @@
 //------------------------------------------------------------------------------
 
 const { getGraphemeCount } = require("../shared/string-utils");
-const { getModuleExportName } = require("./utils/ast-utils");
+const { getModuleExportName, isImportAttributeKey } = require("./utils/ast-utils");
 
 //------------------------------------------------------------------------------
 // Rule Definition
@@ -115,7 +115,7 @@ module.exports = {
                         isKeyAndValueSame && parent.key === node && properties
                     );
                 }
-                return properties && !parent.computed && parent.key.name === node.name;
+                return properties && !isImportAttributeKey(node) && !parent.computed && parent.key.name === node.name;
             },
             ImportSpecifier(parent, node) {
                 return (

package/lib/rules/id-match.js

@@ -5,6 +5,12 @@
 
 "use strict";
 
+//------------------------------------------------------------------------------
+// Requirements
+//------------------------------------------------------------------------------
+
+const astUtils = require("./utils/ast-utils");
+
 //------------------------------------------------------------------------------
 // Rule Definition
 //------------------------------------------------------------------------------
@@ -180,7 +186,7 @@ module.exports = {
                     parent = node.parent,
                     effectiveParent = (parent.type === "MemberExpression") ? parent.parent : parent;
 
-                if (isReferenceToGlobalVariable(node)) {
+                if (isReferenceToGlobalVariable(node) || astUtils.isImportAttributeKey(node)) {
                     return;
                 }
 

package/lib/rules/utils/ast-utils.js

@@ -1054,7 +1054,7 @@ let needsPrecedingSemicolon;
 {
     const BREAK_OR_CONTINUE = new Set(["BreakStatement", "ContinueStatement"]);
 
-    // Declaration types that must contain a string Literal node at the end.
+    // Declaration types that cannot be continued by a punctuator when ending with a string Literal that is a direct child.
     const DECLARATIONS = new Set(["ExportAllDeclaration", "ExportNamedDeclaration", "ImportDeclaration"]);
 
     const IDENTIFIER_OR_KEYWORD = new Set(["Identifier", "Keyword"]);
@@ -1132,6 +1132,48 @@ let needsPrecedingSemicolon;
     };
 }
 
+/**
+ * Checks if a node is used as an import attribute key, either in a static or dynamic import.
+ * @param {ASTNode} node The node to check.
+ * @returns {boolean} Whether the node is used as an import attribute key.
+ */
+function isImportAttributeKey(node) {
+    const { parent } = node;
+
+    // static import/re-export
+    if (parent.type === "ImportAttribute" && parent.key === node) {
+        return true;
+    }
+
+    // dynamic import
+    if (
+        parent.type === "Property" &&
+        !parent.computed &&
+        (parent.key === node || parent.value === node && parent.shorthand && !parent.method) &&
+        parent.parent.type === "ObjectExpression"
+    ) {
+        const objectExpression = parent.parent;
+        const objectExpressionParent = objectExpression.parent;
+
+        if (
+            objectExpressionParent.type === "ImportExpression" &&
+            objectExpressionParent.options === objectExpression
+        ) {
+            return true;
+        }
+
+        // nested key
+        if (
+            objectExpressionParent.type === "Property" &&
+            objectExpressionParent.value === objectExpression
+        ) {
+            return isImportAttributeKey(objectExpressionParent.key);
+        }
+    }
+
+    return false;
+}
+
 //------------------------------------------------------------------------------
 // Public Interface
 //------------------------------------------------------------------------------
@@ -2288,5 +2330,6 @@ module.exports = {
     isTopLevelExpressionStatement,
     isDirective,
     isStartOfExpressionStatement,
-    needsPrecedingSemicolon
+    needsPrecedingSemicolon,
+    isImportAttributeKey
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint",
-  "version": "9.13.0",
+  "version": "9.14.0",
   "author": "Nicholas C. Zakas <nicholas+npm@nczconsulting.com>",
   "description": "An AST-based pattern checker for JavaScript.",
   "type": "commonjs",
@@ -98,15 +98,15 @@
   "bugs": "https://github.com/eslint/eslint/issues/",
   "dependencies": {
     "@eslint-community/eslint-utils": "^4.2.0",
-    "@eslint-community/regexpp": "^4.11.0",
+    "@eslint-community/regexpp": "^4.12.1",
     "@eslint/config-array": "^0.18.0",
     "@eslint/core": "^0.7.0",
     "@eslint/eslintrc": "^3.1.0",
-    "@eslint/js": "9.13.0",
+    "@eslint/js": "9.14.0",
     "@eslint/plugin-kit": "^0.2.0",
-    "@humanfs/node": "^0.16.5",
+    "@humanfs/node": "^0.16.6",
     "@humanwhocodes/module-importer": "^1.0.1",
-    "@humanwhocodes/retry": "^0.3.1",
+    "@humanwhocodes/retry": "^0.4.0",
     "@types/estree": "^1.0.6",
     "@types/json-schema": "^7.0.15",
     "ajv": "^6.12.4",
@@ -114,9 +114,9 @@
     "cross-spawn": "^7.0.2",
     "debug": "^4.3.2",
     "escape-string-regexp": "^4.0.0",
-    "eslint-scope": "^8.1.0",
-    "eslint-visitor-keys": "^4.1.0",
-    "espree": "^10.2.0",
+    "eslint-scope": "^8.2.0",
+    "eslint-visitor-keys": "^4.2.0",
+    "espree": "^10.3.0",
     "esquery": "^1.5.0",
     "esutils": "^2.0.2",
     "fast-deep-equal": "^3.1.3",
@@ -141,10 +141,10 @@
     "@trunkio/launcher": "^1.3.0",
     "@types/node": "^20.11.5",
     "@typescript-eslint/parser": "^8.4.0",
-    "@wdio/browser-runner": "^9.0.5",
-    "@wdio/cli": "^9.0.5",
-    "@wdio/concise-reporter": "^9.0.4",
-    "@wdio/mocha-framework": "^9.0.5",
+    "@wdio/browser-runner": "^9.2.4",
+    "@wdio/cli": "^9.2.4",
+    "@wdio/concise-reporter": "^9.2.2",
+    "@wdio/mocha-framework": "^9.2.2",
     "babel-loader": "^8.0.5",
     "c8": "^7.12.0",
     "chai": "^4.0.1",