eslint 9.16.0 → 9.18.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.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://syntax.fm"><img src="https://github.com/syntaxfm.png" alt="Syntax" 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>
+<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" alt="American Express" 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.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" alt="GitBook" height="32"></a> <a href="https://nx.dev"><img src="https://avatars.githubusercontent.com/u/23692104" alt="Nx" height="32"></a> <a href="https://opensource.mercedes-benz.com/"><img src="https://avatars.githubusercontent.com/u/34240465" alt="Mercedes-Benz Group" height="32"></a> <a href="https://herocoders.com"><img src="https://avatars.githubusercontent.com/u/37549774" 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/cli.js

@@ -341,16 +341,15 @@ const cli = {
     /**
      * Calculates the command string for the --inspect-config operation.
      * @param {string} configFile The path to the config file to inspect.
-     * @param {boolean} hasUnstableTSConfigFlag `true` if the `unstable_ts_config` flag is enabled, `false` if it's not.
      * @returns {Promise<string>} The command string to execute.
      */
-    async calculateInspectConfigFlags(configFile, hasUnstableTSConfigFlag) {
+    async calculateInspectConfigFlags(configFile) {
 
         // find the config file
         const {
             configFilePath,
             basePath
-        } = await locateConfigFileToUse({ cwd: process.cwd(), configFile }, hasUnstableTSConfigFlag);
+        } = await locateConfigFileToUse({ cwd: process.cwd(), configFile });
 
         return ["--config", configFilePath, "--basePath", basePath];
     },
@@ -451,7 +450,7 @@ const cli = {
             try {
                 const flatOptions = await translateOptions(options, "flat");
                 const spawn = require("cross-spawn");
-                const flags = await cli.calculateInspectConfigFlags(flatOptions.overrideConfigFile, flatOptions.flags ? flatOptions.flags.includes("unstable_ts_config") : false);
+                const flags = await cli.calculateInspectConfigFlags(flatOptions.overrideConfigFile);
 
                 spawn.sync("npx", ["@eslint/config-inspector@latest", ...flags], { encoding: "utf8", stdio: "inherit" });
             } catch (error) {

package/lib/config/config-loader.js

@@ -31,7 +31,6 @@ const { FlatConfigArray } = require("./flat-config-array");
  * @property {Array<FlatConfigObject>} [defaultConfigs] The default configs to use.
  * @property {Array<string>} [ignorePatterns] The ignore patterns to use.
  * @property {FlatConfigObject|Array<FlatConfigObject>} overrideConfig The override config to use.
- * @property {boolean} allowTS Indicates if TypeScript configuration files are allowed.
  */
 
 //------------------------------------------------------------------------------
@@ -41,10 +40,7 @@ const { FlatConfigArray } = require("./flat-config-array");
 const FLAT_CONFIG_FILENAMES = [
     "eslint.config.js",
     "eslint.config.mjs",
-    "eslint.config.cjs"
-];
-
-const TS_FLAT_CONFIG_FILENAMES = [
+    "eslint.config.cjs",
     "eslint.config.ts",
     "eslint.config.mts",
     "eslint.config.cts"
@@ -119,10 +115,9 @@ function isRunningInDeno() {
 /**
  * Load the config array from the given filename.
  * @param {string} filePath The filename to load from.
- * @param {boolean} allowTS Indicates if TypeScript configuration files are allowed.
  * @returns {Promise<any>} The config loaded from the config file.
  */
-async function loadConfigFile(filePath, allowTS) {
+async function loadConfigFile(filePath) {
 
     debug(`Loading config from ${filePath}`);
 
@@ -171,7 +166,7 @@ async function loadConfigFile(filePath, allowTS) {
      *
      * When Node.js supports native TypeScript imports, we can remove this check.
      */
-    if (allowTS && isTS && !isDeno && !isBun) {
+    if (isTS && !isDeno && !isBun) {
 
         // eslint-disable-next-line no-use-before-define -- `ConfigLoader.loadJiti` can be overwritten for testing
         const { createJiti } = await ConfigLoader.loadJiti().catch(() => {
@@ -261,8 +256,7 @@ class ConfigLoader {
         const resultPromise = ConfigLoader.locateConfigFileToUse({
             useConfigFile: this.#options.configFile,
             cwd: this.#options.cwd,
-            fromDirectory,
-            allowTS: this.#options.allowTS
+            fromDirectory
         });
 
         // ensure `ConfigLoader.locateConfigFileToUse` is called only once for `fromDirectory`
@@ -443,15 +437,10 @@ class ConfigLoader {
      * @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;
+    static async locateConfigFileToUse({ useConfigFile, cwd, fromDirectory = cwd }) {
 
         // determine where to load config file from
         let configFilePath;
@@ -464,7 +453,7 @@ class ConfigLoader {
         } else if (useConfigFile !== false) {
             debug("Searching for eslint.config.js");
             configFilePath = await findUp(
-                configFilenames,
+                FLAT_CONFIG_FILENAMES,
                 { cwd: fromDirectory }
             );
 
@@ -497,8 +486,7 @@ class ConfigLoader {
             ignoreEnabled,
             ignorePatterns,
             overrideConfig,
-            defaultConfigs = [],
-            allowTS
+            defaultConfigs = []
         } = options;
 
         debug(`Calculating config array from config file ${configFilePath} and base path ${basePath}`);
@@ -509,7 +497,7 @@ class ConfigLoader {
         if (configFilePath) {
 
             debug(`Loading config file ${configFilePath}`);
-            const fileConfig = await loadConfigFile(configFilePath, allowTS);
+            const fileConfig = await loadConfigFile(configFilePath);
 
             if (Array.isArray(fileConfig)) {
                 configs.push(...fileConfig);
@@ -618,8 +606,7 @@ class LegacyConfigLoader extends ConfigLoader {
         if (!this.#configFilePath) {
             this.#configFilePath = ConfigLoader.locateConfigFileToUse({
                 useConfigFile: this.#options.configFile,
-                cwd: this.#options.cwd,
-                allowTS: this.#options.allowTS
+                cwd: this.#options.cwd
             });
         }
 

package/lib/eslint/eslint.js

@@ -266,15 +266,13 @@ function compareResultsByFilePath(a, b) {
  * This function is used primarily by the `--inspect-config` option. For now,
  * we will maintain the existing behavior, which is to search up from the cwd.
  * @param {ESLintOptions} options The ESLint instance options.
- * @param {boolean} allowTS `true` if the `unstable_ts_config` flag is enabled, `false` if it's not.
  * @returns {Promise<{configFilePath:string|undefined;basePath:string}>} Location information for
  *      the config file.
  */
-async function locateConfigFileToUse({ configFile, cwd }, allowTS) {
+async function locateConfigFileToUse({ configFile, cwd }) {
 
     const configLoader = new ConfigLoader({
         cwd,
-        allowTS,
         configFile
     });
 
@@ -469,8 +467,7 @@ class ESLint {
             configFile: processedOptions.configFile,
             ignoreEnabled: processedOptions.ignore,
             ignorePatterns: processedOptions.ignorePatterns,
-            defaultConfigs,
-            allowTS: processedOptions.flags.includes("unstable_ts_config")
+            defaultConfigs
         };
 
         this.#configLoader = processedOptions.flags.includes("unstable_config_lookup_from_file")

package/lib/linter/source-code-fixer.js

@@ -107,7 +107,7 @@ SourceCodeFixer.applyFixes = function(sourceText, messages, shouldFix) {
     }
 
     messages.forEach(problem => {
-        if (Object.hasOwn(problem, "fix")) {
+        if (Object.hasOwn(problem, "fix") && problem.fix) {
             fixes.push(problem);
         } else {
             remainingMessages.push(problem);

package/lib/rules/arrow-body-style.js

@@ -24,6 +24,7 @@ module.exports = {
         docs: {
             description: "Require braces around arrow function bodies",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/arrow-body-style"
         },
 

package/lib/rules/camelcase.js

@@ -31,6 +31,7 @@ module.exports = {
         docs: {
             description: "Enforce camelcase naming convention",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/camelcase"
         },
 

package/lib/rules/capitalized-comments.js

@@ -107,6 +107,7 @@ module.exports = {
         docs: {
             description: "Enforce or disallow capitalization of the first letter of a comment",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/capitalized-comments"
         },
 

package/lib/rules/consistent-this.js

@@ -16,6 +16,7 @@ module.exports = {
         docs: {
             description: "Enforce consistent naming when capturing the current execution context",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/consistent-this"
         },
 

package/lib/rules/curly.js

@@ -22,6 +22,7 @@ module.exports = {
         docs: {
             description: "Enforce consistent brace style for all control statements",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/curly"
         },
 

package/lib/rules/default-case-last.js

@@ -1,5 +1,5 @@
 /**
- * @fileoverview Rule to enforce default clauses in switch statements to be last
+ * @fileoverview Rule to enforce `default` clauses in `switch` statements to be last
  * @author Milos Djermanovic
  */
 
@@ -15,7 +15,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: "Enforce default clauses in switch statements to be last",
+            description: "Enforce `default` clauses in `switch` statements to be last",
             recommended: false,
             url: "https://eslint.org/docs/latest/rules/default-case-last"
         },

package/lib/rules/default-param-last.js

@@ -13,6 +13,7 @@ module.exports = {
         docs: {
             description: "Enforce default parameters to be last",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/default-param-last"
         },
 

package/lib/rules/dot-notation.js

@@ -33,6 +33,7 @@ module.exports = {
         docs: {
             description: "Enforce dot notation whenever possible",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/dot-notation"
         },
 

package/lib/rules/for-direction.js

@@ -1,5 +1,5 @@
 /**
- * @fileoverview enforce "for" loop update clause moving the counter in the right direction.(for-direction)
+ * @fileoverview enforce `for` loop update clause moving the counter in the right direction.(for-direction)
  * @author Aladdin-ADD<hh_2013@foxmail.com>
  */
 
@@ -21,7 +21,7 @@ module.exports = {
         type: "problem",
 
         docs: {
-            description: "Enforce \"for\" loop update clause moving the counter in the right direction",
+            description: "Enforce `for` loop update clause moving the counter in the right direction",
             recommended: true,
             url: "https://eslint.org/docs/latest/rules/for-direction"
         },

package/lib/rules/func-name-matching.js

@@ -76,6 +76,7 @@ module.exports = {
         docs: {
             description: "Require function names to match the name of the variable or property to which they are assigned",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/func-name-matching"
         },
 

package/lib/rules/func-style.js

@@ -21,6 +21,7 @@ module.exports = {
         docs: {
             description: "Enforce the consistent use of either `function` declarations or expressions assigned to variables",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/func-style"
         },
 

package/lib/rules/id-denylist.js

@@ -109,6 +109,7 @@ module.exports = {
         docs: {
             description: "Disallow specified identifiers",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/id-denylist"
         },
 

package/lib/rules/id-length.js

@@ -32,6 +32,7 @@ module.exports = {
         docs: {
             description: "Enforce minimum and maximum identifier lengths",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/id-length"
         },
 

package/lib/rules/id-match.js

@@ -30,6 +30,7 @@ module.exports = {
         docs: {
             description: "Require identifiers to match a specified regular expression",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/id-match"
         },
 

package/lib/rules/init-declarations.js

@@ -50,6 +50,7 @@ module.exports = {
         docs: {
             description: "Require or disallow initialization in variable declarations",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/init-declarations"
         },
 

package/lib/rules/logical-assignment-operators.js

@@ -186,6 +186,7 @@ module.exports = {
         docs: {
             description: "Require or disallow logical assignment operator shorthand",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/logical-assignment-operators"
         },
 

package/lib/rules/no-console.js

@@ -49,6 +49,7 @@ module.exports = {
 
         messages: {
             unexpected: "Unexpected console statement.",
+            limited: "Unexpected console statement. Only these console methods are allowed: {{ allowed }}.",
             removeConsole: "Remove the console.{{ propertyName }}()."
         }
     },
@@ -169,7 +170,8 @@ module.exports = {
             context.report({
                 node,
                 loc: node.loc,
-                messageId: "unexpected",
+                messageId: allowed.length ? "limited" : "unexpected",
+                data: { allowed: allowed.join(", ") },
                 suggest: canProvideSuggestions(node)
                     ? [{
                         messageId: "removeConsole",

package/lib/rules/no-continue.js

@@ -17,6 +17,7 @@ module.exports = {
         docs: {
             description: "Disallow `continue` statements",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/no-continue"
         },
 

package/lib/rules/no-div-regex.js

@@ -17,6 +17,7 @@ module.exports = {
         docs: {
             description: "Disallow equal signs explicitly at the beginning of regular expressions",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/no-div-regex"
         },
 

package/lib/rules/no-else-return.js

@@ -26,6 +26,7 @@ module.exports = {
         docs: {
             description: "Disallow `else` blocks after `return` statements in `if` statements",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/no-else-return"
         },
 

package/lib/rules/no-extra-boolean-cast.js

@@ -28,6 +28,7 @@ module.exports = {
         docs: {
             description: "Disallow unnecessary boolean casts",
             recommended: true,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/no-extra-boolean-cast"
         },
 

package/lib/rules/no-extra-label.js

@@ -23,6 +23,7 @@ module.exports = {
         docs: {
             description: "Disallow unnecessary labels",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/no-extra-label"
         },
 

package/lib/rules/no-implicit-coercion.js

@@ -179,6 +179,7 @@ module.exports = {
         docs: {
             description: "Disallow shorthand type conversions",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/no-implicit-coercion"
         },
 

package/lib/rules/no-inline-comments.js

@@ -20,6 +20,7 @@ module.exports = {
         docs: {
             description: "Disallow inline comments after code",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/no-inline-comments"
         },
 

package/lib/rules/no-label-var.js

@@ -23,6 +23,7 @@ module.exports = {
         docs: {
             description: "Disallow labels that share a name with a variable",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/no-label-var"
         },
 

package/lib/rules/no-labels.js

@@ -27,6 +27,7 @@ module.exports = {
         docs: {
             description: "Disallow labeled statements",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/no-labels"
         },
 

package/lib/rules/no-lonely-if.js

@@ -22,6 +22,7 @@ module.exports = {
         docs: {
             description: "Disallow `if` statements as the only statement in `else` blocks",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/no-lonely-if"
         },
 

package/lib/rules/no-magic-numbers.js

@@ -34,6 +34,7 @@ module.exports = {
         docs: {
             description: "Disallow magic numbers",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/no-magic-numbers"
         },
 

package/lib/rules/no-multi-str.js

@@ -23,6 +23,7 @@ module.exports = {
         docs: {
             description: "Disallow multiline strings",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/no-multi-str"
         },
 

package/lib/rules/no-negated-condition.js

@@ -16,6 +16,7 @@ module.exports = {
         docs: {
             description: "Disallow negated conditions",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/no-negated-condition"
         },
 

package/lib/rules/no-nested-ternary.js

@@ -17,6 +17,7 @@ module.exports = {
         docs: {
             description: "Disallow nested ternary expressions",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/no-nested-ternary"
         },
 

package/lib/rules/no-param-reassign.js

@@ -1,5 +1,5 @@
 /**
- * @fileoverview Disallow reassignment of function parameters.
+ * @fileoverview Disallow reassigning function parameters.
  * @author Nat Burns
  */
 "use strict";
@@ -16,7 +16,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: "Disallow reassigning `function` parameters",
+            description: "Disallow reassigning function parameters",
             recommended: false,
             url: "https://eslint.org/docs/latest/rules/no-param-reassign"
         },

package/lib/rules/no-plusplus.js

@@ -57,6 +57,7 @@ module.exports = {
         docs: {
             description: "Disallow the unary operators `++` and `--`",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/no-plusplus"
         },
 

package/lib/rules/no-script-url.js

@@ -1,5 +1,5 @@
 /**
- * @fileoverview Rule to flag when using javascript: urls
+ * @fileoverview Rule to disallow `javascript:` URLs
  * @author Ilya Volodin
  */
 /* eslint no-script-url: 0 -- Code is checking to report such URLs */
@@ -18,7 +18,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: "Disallow `javascript:` urls",
+            description: "Disallow `javascript:` URLs",
             recommended: false,
             url: "https://eslint.org/docs/latest/rules/no-script-url"
         },
@@ -33,7 +33,7 @@ module.exports = {
     create(context) {
 
         /**
-         * Check whether a node's static value starts with "javascript:" or not.
+         * Check whether a node's static value starts with `javascript:` or not.
          * And report an error for unexpected script URL.
          * @param {ASTNode} node node to check
          * @returns {void}

package/lib/rules/no-shadow-restricted-names.js

@@ -45,17 +45,27 @@ module.exports = {
         const RESTRICTED = new Set(["undefined", "NaN", "Infinity", "arguments", "eval"]);
         const sourceCode = context.sourceCode;
 
+        // Track reported nodes to avoid duplicate reports. For example, on class declarations.
+        const reportedNodes = new Set();
+
         return {
-            "VariableDeclaration, :function, CatchClause"(node) {
+            "VariableDeclaration, :function, CatchClause, ImportDeclaration, ClassDeclaration, ClassExpression"(node) {
                 for (const variable of sourceCode.getDeclaredVariables(node)) {
                     if (variable.defs.length > 0 && RESTRICTED.has(variable.name) && !safelyShadowsUndefined(variable)) {
-                        context.report({
-                            node: variable.defs[0].name,
-                            messageId: "shadowingRestrictedName",
-                            data: {
-                                name: variable.name
+                        for (const def of variable.defs) {
+                            const nodeToReport = def.name;
+
+                            if (!reportedNodes.has(nodeToReport)) {
+                                reportedNodes.add(nodeToReport);
+                                context.report({
+                                    node: nodeToReport,
+                                    messageId: "shadowingRestrictedName",
+                                    data: {
+                                        name: variable.name
+                                    }
+                                });
                             }
-                        });
+                        }
                     }
                 }
             }

package/lib/rules/no-ternary.js

@@ -17,6 +17,7 @@ module.exports = {
         docs: {
             description: "Disallow ternary operators",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/no-ternary"
         },
 

package/lib/rules/no-undef-init.js

@@ -19,6 +19,7 @@ module.exports = {
         docs: {
             description: "Disallow initializing variables to `undefined`",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/no-undef-init"
         },
 

package/lib/rules/no-undefined.js

@@ -16,6 +16,7 @@ module.exports = {
         docs: {
             description: "Disallow the use of `undefined` as an identifier",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/no-undefined"
         },
 

package/lib/rules/no-underscore-dangle.js

@@ -29,6 +29,7 @@ module.exports = {
         docs: {
             description: "Disallow dangling underscores in identifiers",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/no-underscore-dangle"
         },