eslint 10.2.1 → 10.4.0

package/lib/rules/no-var.js

@@ -229,11 +229,10 @@ function hasReferenceBeforeDeclaration(variable) {
 module.exports = {
 	meta: {
 		type: "suggestion",
-		dialects: ["typescript", "javascript"],
-		language: "javascript",
 
 		docs: {
 			description: "Require `let` or `const` instead of `var`",
+			dialects: ["JavaScript", "TypeScript"],
 			recommended: false,
 			url: "https://eslint.org/docs/latest/rules/no-var",
 		},

package/lib/rules/object-shorthand.js

@@ -186,6 +186,8 @@ module.exports = {
 			],
 		},
 
+		defaultOptions: ["always"],
+
 		messages: {
 			expectedAllPropertiesShorthanded:
 				"Expected shorthand for all properties.",
@@ -201,7 +203,7 @@ module.exports = {
 	},
 
 	create(context) {
-		const APPLY = context.options[0] || OPTIONS.always;
+		const APPLY = context.options[0];
 		const APPLY_TO_METHODS =
 			APPLY === OPTIONS.methods || APPLY === OPTIONS.always;
 		const APPLY_TO_PROPS =

package/lib/rules/one-var.js

@@ -89,6 +89,8 @@ module.exports = {
 			},
 		],
 
+		defaultOptions: ["always"],
+
 		messages: {
 			combineUninitialized:
 				"Combine this with the previous '{{type}}' statement with uninitialized variables.",
@@ -109,7 +111,7 @@ module.exports = {
 		const MODE_ALWAYS = "always";
 		const MODE_NEVER = "never";
 		const MODE_CONSECUTIVE = "consecutive";
-		const mode = context.options[0] || MODE_ALWAYS;
+		const mode = context.options[0];
 
 		const options = {};
 

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

@@ -158,8 +158,6 @@ function hasDuplicateParams(paramsList) {
 module.exports = {
 	meta: {
 		type: "suggestion",
-		dialects: ["javascript", "typescript"],
-		language: "javascript",
 
 		defaultOptions: [
 			{ allowNamedFunctions: false, allowUnboundThis: true },
@@ -167,6 +165,7 @@ module.exports = {
 
 		docs: {
 			description: "Require using arrow functions for callbacks",
+			dialects: ["JavaScript", "TypeScript"],
 			recommended: false,
 			frozen: true,
 			url: "https://eslint.org/docs/latest/rules/prefer-arrow-callback",

package/lib/rules/require-await.js

@@ -110,12 +110,14 @@ module.exports = {
 				 */
 				const nextToken = sourceCode.getTokenAfter(asyncToken);
 				const addSemiColon =
-					nextToken.type === "Punctuator" &&
-					(nextToken.value === "[" || nextToken.value === "(") &&
-					(nodeWithAsyncKeyword.type === "MethodDefinition" ||
+					((astUtils.isOpeningParenToken(nextToken) &&
 						astUtils.isStartOfExpressionStatement(
 							nodeWithAsyncKeyword,
-						)) &&
+						)) ||
+						(nodeWithAsyncKeyword.type === "MethodDefinition" &&
+							astUtils.canContinueExpressionInClassBody(
+								nextToken,
+							))) &&
 					astUtils.needsPrecedingSemicolon(
 						sourceCode,
 						nodeWithAsyncKeyword,

package/lib/rules/utils/ast-utils.js

@@ -1258,12 +1258,31 @@ function isStartOfExpressionStatement(node) {
 }
 
 /**
- * Determines whether an opening parenthesis `(`, bracket `[` or backtick ``` ` ``` needs to be preceded by a semicolon.
- * This opening parenthesis or bracket should be at the start of an `ExpressionStatement`, a `MethodDefinition` or at
- * the start of the body of an `ArrowFunctionExpression`.
+ * Checks whether a token can cause continuation of a preceding expression
+ * (for example, of a class field initializer expression) in a class body.
+ * This function checks specifically for tokens that can appear at the start
+ * of a class member: `[` (computed key), `*` (generator method), `in` or `instanceof` (valid keys)
+ * Without a preceding semicolon, these tokens would be parsed as index access or operators.
+ * @param {Token} token The token to check.
+ * @returns {boolean} Whether the token can cause continuation of a preceding expression.
+ */
+function canContinueExpressionInClassBody(token) {
+	return (
+		(token.type === "Punctuator" &&
+			(token.value === "[" || token.value === "*")) ||
+		// Different parsers may return these tokens as either "Identifier" or "Keyword"
+		((token.type === "Identifier" || token.type === "Keyword") &&
+			(token.value === "in" || token.value === "instanceof"))
+	);
+}
+
+/**
+ * Determines whether an opening parenthesis `(`, bracket `[`, asterisk `*`, or backtick ``` ` ``` needs to be preceded by a semicolon.
+ * This opening parenthesis or bracket should be at the start of an `ExpressionStatement`, a `MethodDefinition`, a `PropertyDefinition`,
+ * or at the start of the body of an `ArrowFunctionExpression`.
  * @type {(sourceCode: SourceCode, node: ASTNode) => boolean}
  * @param {SourceCode} sourceCode The source code object.
- * @param {ASTNode} node A node at the position where an opening parenthesis or bracket will be inserted.
+ * @param {ASTNode} node A node at the position where an opening parenthesis, bracket, or asterisk will be inserted.
  * @returns {boolean} Whether a semicolon is required before the opening parenthesis or bracket.
  */
 let needsPrecedingSemicolon;
@@ -1354,6 +1373,18 @@ let needsPrecedingSemicolon;
 
 		const prevNode = sourceCode.getNodeByRangeIndex(prevToken.range[0]);
 
+		// Uninitialized class fields don't need a semicolon
+		if (
+			// Key
+			(prevNode.parent.type === "PropertyDefinition" &&
+				prevNode.parent.key === prevNode) ||
+			// Closing bracket of a computed key
+			(prevNode.type === "PropertyDefinition" &&
+				isClosingBracketToken(prevToken))
+		) {
+			return false;
+		}
+
 		if (
 			prevNode.type === "TSDeclareFunction" ||
 			prevNode.parent.type === "TSImportEqualsDeclaration" ||
@@ -2842,6 +2873,7 @@ module.exports = {
 	isTopLevelExpressionStatement,
 	isDirective,
 	isStartOfExpressionStatement,
+	canContinueExpressionInClassBody,
 	needsPrecedingSemicolon,
 	isImportAttributeKey,
 	getOpeningParenOfParams,

package/lib/types/config-api.d.ts

@@ -7,6 +7,7 @@ import {
 	type Config,
 	defineConfig,
 	globalIgnores,
+	includeIgnoreFile,
 } from "@eslint/config-helpers";
 
-export { type Config, defineConfig, globalIgnores };
+export { type Config, defineConfig, globalIgnores, includeIgnoreFile };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint",
-  "version": "10.2.1",
+  "version": "10.4.0",
   "author": "Nicholas C. Zakas <nicholas+npm@nczconsulting.com>",
   "description": "An AST-based pattern checker for JavaScript.",
   "type": "commonjs",
@@ -72,6 +72,8 @@
     "test": "node Makefile.js test",
     "test:browser": "node Makefile.js cypress",
     "test:cli": "mocha",
+    "test:ecosystem": "node tools/test-ecosystem/index.mjs",
+    "test:ecosystem:update": "node tools/test-ecosystem/update.mjs",
     "test:emfile": "node tools/check-emfile-handling.js",
     "test:fuzz": "node Makefile.js fuzz",
     "test:performance": "node Makefile.js perf",
@@ -120,7 +122,7 @@
     "@eslint-community/eslint-utils": "^4.8.0",
     "@eslint-community/regexpp": "^4.12.2",
     "@eslint/config-array": "^0.23.5",
-    "@eslint/config-helpers": "^0.5.5",
+    "@eslint/config-helpers": "^0.6.0",
     "@eslint/core": "^1.2.1",
     "@eslint/plugin-kit": "^0.7.1",
     "@humanfs/node": "^0.16.6",
@@ -157,7 +159,7 @@
     "@eslint/json": "^1.2.0",
     "@types/esquery": "^1.5.4",
     "@types/node": "^22.13.14",
-    "@typescript-eslint/parser": "^8.56.0",
+    "@typescript-eslint/parser": "^8.58.2",
     "babel-loader": "^8.0.5",
     "c8": "^11.0.0",
     "chai": "^4.0.1",
@@ -181,7 +183,7 @@
     "got": "^11.8.3",
     "gray-matter": "^4.0.3",
     "jiti": "^2.6.1",
-    "knip": "^5.60.2",
+    "knip": "^6.13.1",
     "lint-staged": "^11.0.0",
     "markdown-it": "^12.2.0",
     "markdown-it-container": "^3.0.0",
@@ -196,15 +198,14 @@
     "mocha": "^11.7.1",
     "node-polyfill-webpack-plugin": "^1.0.3",
     "npm-license": "^0.3.3",
-    "prettier": "3.8.2",
+    "prettier": "3.8.3",
     "progress": "^2.0.3",
     "proxyquire": "^2.0.1",
-    "recast": "^0.23.0",
     "regenerator-runtime": "^0.14.0",
     "semver": "^7.5.3",
     "shelljs": "^0.10.0",
     "sinon": "^11.0.0",
-    "typescript": "^5.9.3",
+    "typescript": "^6.0.3",
     "webpack": "^5.23.0",
     "webpack-cli": "^4.5.0",
     "yorkie": "^2.0.0"