eslint 9.17.0 → 9.19.0

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-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-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"
         },
 

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

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

package/lib/rules/no-useless-computed-key.js

@@ -100,6 +100,7 @@ module.exports = {
         docs: {
             description: "Disallow unnecessary computed property keys in objects and classes",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/no-useless-computed-key"
         },
 

package/lib/rules/no-useless-concat.js

@@ -72,6 +72,7 @@ module.exports = {
         docs: {
             description: "Disallow unnecessary concatenation of literals or template literals",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/no-useless-concat"
         },
 

package/lib/rules/no-void.js

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

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

@@ -27,6 +27,7 @@ module.exports = {
         docs: {
             description: "Disallow specified warning terms in comments",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/no-warning-comments"
         },
 

package/lib/rules/object-shorthand.js

@@ -30,6 +30,7 @@ module.exports = {
         docs: {
             description: "Require or disallow method and property shorthand syntax for object literals",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/object-shorthand"
         },
 

package/lib/rules/one-var.js

@@ -36,6 +36,7 @@ module.exports = {
         docs: {
             description: "Enforce variables to be declared either together or separately in functions",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/one-var"
         },
 

package/lib/rules/operator-assignment.js

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

package/lib/rules/prefer-arrow-callback.js

@@ -155,6 +155,7 @@ module.exports = {
         docs: {
             description: "Require using arrow functions for callbacks",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/prefer-arrow-callback"
         },
 

package/lib/rules/prefer-destructuring.js

@@ -28,6 +28,7 @@ module.exports = {
         docs: {
             description: "Require destructuring from arrays and/or objects",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/prefer-destructuring"
         },
 

package/lib/rules/prefer-exponentiation-operator.js

@@ -93,6 +93,7 @@ module.exports = {
         docs: {
             description: "Disallow the use of `Math.pow` in favor of the `**` operator",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/prefer-exponentiation-operator"
         },
 

package/lib/rules/prefer-numeric-literals.js

@@ -47,6 +47,7 @@ module.exports = {
         docs: {
             description: "Disallow `parseInt()` and `Number.parseInt()` in favor of binary, octal, and hexadecimal literals",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/prefer-numeric-literals"
         },
 

package/lib/rules/prefer-object-spread.js

@@ -248,6 +248,7 @@ module.exports = {
             description:
                 "Disallow using `Object.assign` with an object literal as the first argument and prefer the use of object spread instead",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/prefer-object-spread"
         },
 

package/lib/rules/prefer-spread.js

@@ -51,6 +51,7 @@ module.exports = {
         docs: {
             description: "Require spread operators instead of `.apply()`",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/prefer-spread"
         },
 

package/lib/rules/prefer-template.js

@@ -130,6 +130,7 @@ module.exports = {
         docs: {
             description: "Require template literals instead of string concatenation",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/prefer-template"
         },
 

package/lib/rules/sort-imports.js

@@ -1,5 +1,5 @@
 /**
- * @fileoverview Rule to require sorting of import declarations
+ * @fileoverview Rule to enforce sorted `import` declarations within modules
  * @author Christian Schuller
  */
 
@@ -23,8 +23,9 @@ module.exports = {
         }],
 
         docs: {
-            description: "Enforce sorted import declarations within modules",
+            description: "Enforce sorted `import` declarations within modules",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/sort-imports"
         },
 

package/lib/rules/sort-keys.js

@@ -91,6 +91,7 @@ module.exports = {
         docs: {
             description: "Require object keys to be sorted",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/sort-keys"
         },
 

package/lib/rules/sort-vars.js

@@ -21,6 +21,7 @@ module.exports = {
         docs: {
             description: "Require variables within the same declaration block to be sorted",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/sort-vars"
         },
 

package/lib/rules/vars-on-top.js

@@ -17,6 +17,7 @@ module.exports = {
         docs: {
             description: "Require `var` declarations be placed at the top of their containing scope",
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/vars-on-top"
         },
 

package/lib/rules/yoda.js

@@ -119,6 +119,7 @@ module.exports = {
         docs: {
             description: 'Require or disallow "Yoda" conditions',
             recommended: false,
+            frozen: true,
             url: "https://eslint.org/docs/latest/rules/yoda"
         },
 

package/lib/shared/flags.js

@@ -10,8 +10,7 @@
  */
 const activeFlags = new Map([
     ["test_only", "Used only for testing."],
-    ["unstable_config_lookup_from_file", "Look up `eslint.config.js` from the file being linted."],
-    ["unstable_ts_config", "Enable TypeScript configuration files."]
+    ["unstable_config_lookup_from_file", "Look up `eslint.config.js` from the file being linted."]
 ]);
 
 /**
@@ -19,7 +18,8 @@ const activeFlags = new Map([
  * @type {Map<string, string>}
  */
 const inactiveFlags = new Map([
-    ["test_only_old", "Used only for testing."]
+    ["test_only_old", "Used only for testing."],
+    ["unstable_ts_config", "This flag is no longer required to enable TypeScript configuration files."]
 ]);
 
 module.exports = {

package/lib/shared/option-utils.js

@@ -0,0 +1,56 @@
+/**
+ * @fileoverview Utilities to operate on option objects.
+ * @author Josh Goldberg
+ */
+
+"use strict";
+
+/**
+ * Determines whether any of input's properties are different
+ * from values that already exist in original.
+ * @template T
+ * @param {Partial<T>} input New value.
+ * @param {T} original Original value.
+ * @returns {boolean} Whether input includes an explicit difference.
+ */
+function containsDifferentProperty(input, original) {
+    if (input === original) {
+        return false;
+    }
+
+    if (
+        typeof input !== typeof original ||
+        Array.isArray(input) !== Array.isArray(original)
+    ) {
+        return true;
+    }
+
+    if (Array.isArray(input)) {
+        return (
+            input.length !== original.length ||
+            input.some((value, i) =>
+                containsDifferentProperty(value, original[i]))
+        );
+    }
+
+    if (typeof input === "object") {
+        if (input === null || original === null) {
+            return true;
+        }
+
+        const inputKeys = Object.keys(input);
+        const originalKeys = Object.keys(original);
+
+        return inputKeys.length !== originalKeys.length || inputKeys.some(
+            inputKey =>
+                !Object.hasOwn(original, inputKey) ||
+                containsDifferentProperty(input[inputKey], original[inputKey])
+        );
+    }
+
+    return true;
+}
+
+module.exports = {
+    containsDifferentProperty
+};

package/lib/types/index.d.ts

@@ -1371,6 +1371,12 @@ export namespace Linter {
          * tracked and reported.
          */
         reportUnusedDisableDirectives?: Severity | StringSeverity | boolean;
+
+        /**
+         * A severity value indicating if and how unused inline configs should be
+         * tracked and reported.
+         */
+        reportUnusedInlineConfigs?: Severity | StringSeverity;
     }
 
     interface Stats {
@@ -1484,7 +1490,7 @@ export namespace ESLint {
         allowInlineConfig?: boolean | undefined;
         baseConfig?: Linter.Config | Linter.Config[] | null | undefined;
         overrideConfig?: Linter.Config | Linter.Config[] | null | undefined;
-        overrideConfigFile?: string | boolean | undefined;
+        overrideConfigFile?: string | true | null | undefined;
         plugins?: Record<string, Plugin> | null | undefined;
         ruleFilter?: ((arg: { ruleId: string; severity: Exclude<Linter.Severity, 0> }) => boolean) | undefined;
         stats?: boolean | undefined;