eslint 9.15.0 → 9.17.0

package/README.md

@@ -300,7 +300,7 @@ to get your logo on our READMEs and [website](https://eslint.org/sponsors).
 <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://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://opensource.mercedes-benz.com/"><img src="https://avatars.githubusercontent.com/u/34240465?v=4" alt="Mercedes-Benz Group" 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/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/accessor-pairs.js

@@ -177,10 +177,11 @@ module.exports = {
         }
     },
     create(context) {
-        const [config] = context.options;
-        const checkGetWithoutSet = config.getWithoutSet === true;
-        const checkSetWithoutGet = config.setWithoutGet !== false;
-        const enforceForClassMembers = config.enforceForClassMembers !== false;
+        const [{
+            getWithoutSet: checkGetWithoutSet,
+            setWithoutGet: checkSetWithoutGet,
+            enforceForClassMembers
+        }] = context.options;
         const sourceCode = context.sourceCode;
 
         /**

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/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/getter-return.js

@@ -42,7 +42,9 @@ module.exports = {
     meta: {
         type: "problem",
 
-        defaultOptions: [{}],
+        defaultOptions: [{
+            allowImplicit: false
+        }],
 
         docs: {
             description: "Enforce `return` statements in getters",

package/lib/rules/id-length.js

@@ -25,7 +25,8 @@ module.exports = {
         defaultOptions: [{
             exceptionPatterns: [],
             exceptions: [],
-            min: 2
+            min: 2,
+            properties: "always"
         }],
 
         docs: {

package/lib/rules/id-match.js

@@ -20,7 +20,12 @@ module.exports = {
     meta: {
         type: "suggestion",
 
-        defaultOptions: ["^.+$", {}],
+        defaultOptions: ["^.+$", {
+            classFields: false,
+            ignoreDestructuring: false,
+            onlyDeclarations: false,
+            properties: false
+        }],
 
         docs: {
             description: "Require identifiers to match a specified regular expression",

package/lib/rules/new-cap.js

@@ -118,7 +118,7 @@ module.exports = {
 
     create(context) {
         const [config] = context.options;
-        const skipProperties = config.properties === false;
+        const skipProperties = !config.properties;
 
         const newIsCapExceptions = config.newIsCapExceptions.reduce(invert, {});
         const newIsCapExceptionPattern = config.newIsCapExceptionPattern ? new RegExp(config.newIsCapExceptionPattern, "u") : null;

package/lib/rules/no-bitwise.js

@@ -25,7 +25,10 @@ module.exports = {
     meta: {
         type: "suggestion",
 
-        defaultOptions: [{ allow: [] }],
+        defaultOptions: [{
+            allow: [],
+            int32Hint: false
+        }],
 
         docs: {
             description: "Disallow bitwise operators",

package/lib/rules/no-compare-neg-zero.js

@@ -14,7 +14,7 @@ module.exports = {
         type: "problem",
 
         docs: {
-            description: "Disallow comparing against -0",
+            description: "Disallow comparing against `-0`",
             recommended: true,
             url: "https://eslint.org/docs/latest/rules/no-compare-neg-zero"
         },

package/lib/rules/no-duplicate-imports.js

@@ -232,7 +232,9 @@ module.exports = {
     meta: {
         type: "problem",
 
-        defaultOptions: [{}],
+        defaultOptions: [{
+            includeExports: false
+        }],
 
         docs: {
             description: "Disallow duplicate module imports",

package/lib/rules/no-empty-pattern.js

@@ -15,7 +15,9 @@ module.exports = {
     meta: {
         type: "problem",
 
-        defaultOptions: [{}],
+        defaultOptions: [{
+            allowObjectPatternsAsParameters: false
+        }],
 
         docs: {
             description: "Disallow empty destructuring patterns",

package/lib/rules/no-eval.js

@@ -42,7 +42,9 @@ module.exports = {
     meta: {
         type: "suggestion",
 
-        defaultOptions: [{}],
+        defaultOptions: [{
+            allowIndirect: false
+        }],
 
         docs: {
             description: "Disallow the use of `eval()`",

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

@@ -14,7 +14,9 @@ module.exports = {
     meta: {
         type: "suggestion",
 
-        defaultOptions: [{}],
+        defaultOptions: [{
+            lexicalBindings: false
+        }],
 
         docs: {
             description: "Disallow declarations in the global scope",

package/lib/rules/no-inner-declarations.js

@@ -47,7 +47,7 @@ module.exports = {
     meta: {
         type: "problem",
 
-        defaultOptions: ["functions"],
+        defaultOptions: ["functions", { blockScopedFunctions: "allow" }],
 
         docs: {
             description: "Disallow variable or `function` declarations in nested blocks",
@@ -77,10 +77,10 @@ module.exports = {
 
     create(context) {
         const both = context.options[0] === "both";
+        const { blockScopedFunctions } = context.options[1];
 
         const sourceCode = context.sourceCode;
         const ecmaVersion = context.languageOptions.ecmaVersion;
-        const blockScopedFunctions = context.options[1]?.blockScopedFunctions ?? "allow";
 
         /**
          * Ensure that a given node is at a program or function body's root.

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

@@ -15,7 +15,9 @@ module.exports = {
     meta: {
         type: "suggestion",
 
-        defaultOptions: [{}],
+        defaultOptions: [{
+            ignoreNonDeclaration: false
+        }],
 
         docs: {
             description: "Disallow use of chained assignment expressions",

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

@@ -50,7 +50,9 @@ module.exports = {
     meta: {
         type: "suggestion",
 
-        defaultOptions: [{}],
+        defaultOptions: [{
+            allowForLoopAfterthoughts: false
+        }],
 
         docs: {
             description: "Disallow the unary operators `++` and `--`",

package/lib/rules/no-promise-executor-return.js

@@ -141,7 +141,9 @@ module.exports = {
     meta: {
         type: "problem",
 
-        defaultOptions: [{}],
+        defaultOptions: [{
+            allowVoid: false
+        }],
 
         docs: {
             description: "Disallow returning values from Promise executor functions",

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.js

@@ -31,7 +31,9 @@ module.exports = {
 
         defaultOptions: [{
             allow: [],
-            hoist: "functions"
+            builtinGlobals: false,
+            hoist: "functions",
+            ignoreOnInitialization: false
         }],
 
         docs: {
@@ -45,7 +47,7 @@ module.exports = {
                 type: "object",
                 properties: {
                     builtinGlobals: { type: "boolean" },
-                    hoist: { enum: ["all", "functions", "never"], default: "functions" },
+                    hoist: { enum: ["all", "functions", "never"] },
                     allow: {
                         type: "array",
                         items: {

package/lib/rules/no-undef.js

@@ -28,7 +28,9 @@ module.exports = {
     meta: {
         type: "problem",
 
-        defaultOptions: [{}],
+        defaultOptions: [{
+            typeof: false
+        }],
 
         docs: {
             description: "Disallow the use of undeclared variables unless mentioned in `/*global */` comments",

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

@@ -22,6 +22,7 @@ module.exports = {
             allowFunctionParams: true,
             allowInArrayDestructuring: true,
             allowInObjectDestructuring: true,
+            enforceInClassFields: false,
             enforceInMethodNames: false
         }],
 

package/lib/rules/no-unsafe-negation.js

@@ -51,7 +51,9 @@ module.exports = {
     meta: {
         type: "problem",
 
-        defaultOptions: [{}],
+        defaultOptions: [{
+            enforceForOrderingRelations: false
+        }],
 
         docs: {
             description: "Disallow negating the left operand of relational operators",

package/lib/rules/no-unsafe-optional-chaining.js

@@ -23,7 +23,9 @@ module.exports = {
     meta: {
         type: "problem",
 
-        defaultOptions: [{}],
+        defaultOptions: [{
+            disallowArithmeticOperators: false
+        }],
 
         docs: {
             description: "Disallow use of optional chaining in contexts where the `undefined` value is not allowed",