eslint 9.15.0 → 9.17.0

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

@@ -1,5 +1,5 @@
 /**
- * @fileoverview Prefers object spread property over Object.assign
+ * @fileoverview Rule to disallow using `Object.assign` with an object literal as the first argument and prefer the use of object spread instead
  * @author Sharmila Jesupaul
  */
 
@@ -246,7 +246,7 @@ module.exports = {
 
         docs: {
             description:
-                "Disallow using Object.assign with an object literal as the first argument and prefer the use of object spread instead",
+                "Disallow using `Object.assign` with an object literal as the first argument and prefer the use of object spread instead",
             recommended: false,
             url: "https://eslint.org/docs/latest/rules/prefer-object-spread"
         },

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/require-unicode-regexp.js

@@ -1,5 +1,5 @@
 /**
- * @fileoverview Rule to enforce the use of `u` flag on RegExp.
+ * @fileoverview Rule to enforce the use of `u` or `v` flag on regular expressions.
  * @author Toru Nagashima
  */
 
@@ -48,7 +48,7 @@ module.exports = {
         type: "suggestion",
 
         docs: {
-            description: "Enforce the use of `u` or `v` flag on RegExp",
+            description: "Enforce the use of `u` or `v` flag on regular expressions",
             recommended: false,
             url: "https://eslint.org/docs/latest/rules/require-unicode-regexp"
         },

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/shared/flags.js

@@ -10,7 +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_config_lookup_from_file", "Look up `eslint.config.js` from the file being linted."],
     ["unstable_ts_config", "Enable TypeScript configuration files."]
 ]);
 

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.17.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.17.0",
     "@eslint/plugin-kit": "^0.2.3",
     "@humanfs/node": "^0.16.6",
     "@humanwhocodes/module-importer": "^1.0.1",
@@ -111,7 +111,7 @@
     "@types/json-schema": "^7.0.15",
     "ajv": "^6.12.4",
     "chalk": "^4.0.0",
-    "cross-spawn": "^7.0.5",
+    "cross-spawn": "^7.0.6",
     "debug": "^4.3.2",
     "escape-string-regexp": "^4.0.0",
     "eslint-scope": "^8.2.0",
@@ -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",
@@ -154,7 +154,7 @@
     "eslint": "file:.",
     "eslint-config-eslint": "file:packages/eslint-config-eslint",
     "eslint-plugin-eslint-plugin": "^6.0.0",
-    "eslint-plugin-expect-type": "^0.4.0",
+    "eslint-plugin-expect-type": "^0.6.0",
     "eslint-plugin-yml": "^1.14.0",
     "eslint-release": "^3.3.0",
     "eslint-rule-composer": "^0.3.0",