eslint 9.15.0 → 9.16.0

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

package/lib/rules/no-use-before-define.js

@@ -249,7 +249,8 @@ module.exports = {
         defaultOptions: [{
             classes: true,
             functions: true,
-            variables: true
+            variables: true,
+            allowNamedExports: false
         }],
 
         messages: {

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

@@ -120,7 +120,7 @@ module.exports = {
     },
     create(context) {
         const sourceCode = context.sourceCode;
-        const enforceForClassMembers = context.options[0]?.enforceForClassMembers ?? true;
+        const [{ enforceForClassMembers }] = context.options;
 
         /**
          * Reports a given node if it violated this rule.

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

@@ -20,7 +20,11 @@ module.exports = {
     meta: {
         type: "suggestion",
 
-        defaultOptions: [{}],
+        defaultOptions: [{
+            ignoreDestructuring: false,
+            ignoreImport: false,
+            ignoreExport: false
+        }],
 
         docs: {
             description: "Disallow renaming import, export, and destructured assignments to the same name",

package/lib/rules/no-void.js

@@ -13,7 +13,9 @@ module.exports = {
     meta: {
         type: "suggestion",
 
-        defaultOptions: [{}],
+        defaultOptions: [{
+            allowAsStatement: false
+        }],
 
         docs: {
             description: "Disallow `void` operators",

package/lib/rules/prefer-const.js

@@ -331,7 +331,10 @@ module.exports = {
     meta: {
         type: "suggestion",
 
-        defaultOptions: [{ destructuring: "any" }],
+        defaultOptions: [{
+            destructuring: "any",
+            ignoreReadBeforeAssign: false
+        }],
 
         docs: {
             description: "Require `const` declarations for variables that are never reassigned after declared",
@@ -357,10 +360,9 @@ module.exports = {
     },
 
     create(context) {
-        const [options] = context.options;
+        const [{ destructuring, ignoreReadBeforeAssign }] = context.options;
+        const shouldMatchAnyDestructuredVariable = destructuring !== "all";
         const sourceCode = context.sourceCode;
-        const shouldMatchAnyDestructuredVariable = options.destructuring !== "all";
-        const ignoreReadBeforeAssign = options.ignoreReadBeforeAssign === true;
         const variables = [];
         let reportCount = 0;
         let checkedId = null;

package/lib/rules/prefer-promise-reject-errors.js

@@ -15,7 +15,9 @@ module.exports = {
     meta: {
         type: "suggestion",
 
-        defaultOptions: [{}],
+        defaultOptions: [{
+            allowEmptyReject: false
+        }],
 
         docs: {
             description: "Require using Error objects as Promise rejection reasons",

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

@@ -112,7 +112,9 @@ module.exports = {
     meta: {
         type: "suggestion",
 
-        defaultOptions: [{}],
+        defaultOptions: [{
+            disallowRedundantWrapping: false
+        }],
 
         docs: {
             description: "Disallow use of the `RegExp` constructor in favor of regular expression literals",

package/lib/rules/require-atomic-updates.js

@@ -170,7 +170,9 @@ module.exports = {
     meta: {
         type: "problem",
 
-        defaultOptions: [{}],
+        defaultOptions: [{
+            allowProperties: false
+        }],
 
         docs: {
             description: "Disallow assignments that can lead to race conditions due to usage of `await` or `yield`",

package/lib/rules/sort-keys.js

@@ -83,6 +83,7 @@ module.exports = {
         defaultOptions: ["asc", {
             allowLineSeparatedGroups: false,
             caseSensitive: true,
+            ignoreComputedKeys: false,
             minKeys: 2,
             natural: false
         }],
@@ -112,6 +113,9 @@ module.exports = {
                     },
                     allowLineSeparatedGroups: {
                         type: "boolean"
+                    },
+                    ignoreComputedKeys: {
+                        type: "boolean"
                     }
                 },
                 additionalProperties: false
@@ -124,7 +128,7 @@ module.exports = {
     },
 
     create(context) {
-        const [order, { caseSensitive, natural, minKeys, allowLineSeparatedGroups }] = context.options;
+        const [order, { caseSensitive, natural, minKeys, allowLineSeparatedGroups, ignoreComputedKeys }] = context.options;
         const insensitive = !caseSensitive;
         const isValidOrder = isValidOrders[
             order + (insensitive ? "I" : "") + (natural ? "N" : "")
@@ -160,6 +164,11 @@ module.exports = {
                     return;
                 }
 
+                if (ignoreComputedKeys && node.computed) {
+                    stack.prevName = null; // reset sort
+                    return;
+                }
+
                 const prevName = stack.prevName;
                 const numKeys = stack.numKeys;
                 const thisName = getPropertyName(node);

package/lib/rules/sort-vars.js

@@ -14,7 +14,9 @@ module.exports = {
     meta: {
         type: "suggestion",
 
-        defaultOptions: [{}],
+        defaultOptions: [{
+            ignoreCase: false
+        }],
 
         docs: {
             description: "Require variables within the same declaration block to be sorted",
@@ -27,8 +29,7 @@ module.exports = {
                 type: "object",
                 properties: {
                     ignoreCase: {
-                        type: "boolean",
-                        default: false
+                        type: "boolean"
                     }
                 },
                 additionalProperties: false

package/lib/rules/use-isnan.js

@@ -84,8 +84,7 @@ module.exports = {
 
     create(context) {
 
-        const enforceForSwitchCase = !context.options[0] || context.options[0].enforceForSwitchCase;
-        const enforceForIndexOf = context.options[0] && context.options[0].enforceForIndexOf;
+        const [{ enforceForIndexOf, enforceForSwitchCase }] = context.options;
         const sourceCode = context.sourceCode;
 
         const fixableOperators = new Set(["==", "===", "!=", "!=="]);

package/lib/rules/valid-typeof.js

@@ -19,7 +19,9 @@ module.exports = {
     meta: {
         type: "problem",
 
-        defaultOptions: [{}],
+        defaultOptions: [{
+            requireStringLiterals: false
+        }],
 
         docs: {
             description: "Enforce comparing `typeof` expressions against valid strings",
@@ -34,8 +36,7 @@ module.exports = {
                 type: "object",
                 properties: {
                     requireStringLiterals: {
-                        type: "boolean",
-                        default: false
+                        type: "boolean"
                     }
                 },
                 additionalProperties: false

package/lib/types/rules/stylistic-issues.d.ts

@@ -1854,6 +1854,10 @@ export interface StylisticIssues extends Linter.RulesRecord {
                  * @default false
                  */
                 allowLineSeparatedGroups: boolean;
+                /**
+                 * @default false
+                 */
+                ignoreComputedKeys: boolean;
             }>,
         ]
     >;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint",
-  "version": "9.15.0",
+  "version": "9.16.0",
   "author": "Nicholas C. Zakas <nicholas+npm@nczconsulting.com>",
   "description": "An AST-based pattern checker for JavaScript.",
   "type": "commonjs",
@@ -102,7 +102,7 @@
     "@eslint/config-array": "^0.19.0",
     "@eslint/core": "^0.9.0",
     "@eslint/eslintrc": "^3.2.0",
-    "@eslint/js": "9.15.0",
+    "@eslint/js": "9.16.0",
     "@eslint/plugin-kit": "^0.2.3",
     "@humanfs/node": "^0.16.6",
     "@humanwhocodes/module-importer": "^1.0.1",
@@ -133,10 +133,10 @@
     "optionator": "^0.9.3"
   },
   "devDependencies": {
-    "@arethetypeswrong/cli": "^0.16.4",
+    "@arethetypeswrong/cli": "^0.17.0",
     "@babel/core": "^7.4.3",
     "@babel/preset-env": "^7.4.3",
-    "@eslint/json": "^0.6.0",
+    "@eslint/json": "^0.8.0",
     "@trunkio/launcher": "^1.3.0",
     "@types/node": "^20.11.5",
     "@typescript-eslint/parser": "^8.4.0",