eslint 10.3.0 → 10.4.1

package/lib/rules/no-loop-func.js

@@ -40,12 +40,11 @@ function isIIFE(node) {
 module.exports = {
 	meta: {
 		type: "suggestion",
-		dialects: ["typescript", "javascript"],
-		language: "javascript",
 
 		docs: {
 			description:
 				"Disallow function declarations that contain unsafe references inside loop statements",
+			dialects: ["JavaScript", "TypeScript"],
 			recommended: false,
 			url: "https://eslint.org/docs/latest/rules/no-loop-func",
 		},

package/lib/rules/no-loss-of-precision.js

@@ -219,11 +219,10 @@ function losesPrecision(node) {
 module.exports = {
 	meta: {
 		type: "problem",
-		dialects: ["typescript", "javascript"],
-		language: "javascript",
 
 		docs: {
 			description: "Disallow literal numbers that lose precision",
+			dialects: ["JavaScript", "TypeScript"],
 			recommended: true,
 			url: "https://eslint.org/docs/latest/rules/no-loss-of-precision",
 		},

package/lib/rules/no-magic-numbers.js

@@ -91,11 +91,10 @@ function isAncestorTSIndexedAccessType(node) {
 module.exports = {
 	meta: {
 		type: "suggestion",
-		dialects: ["typescript", "javascript"],
-		language: "javascript",
 
 		docs: {
 			description: "Disallow magic numbers",
+			dialects: ["JavaScript", "TypeScript"],
 			recommended: false,
 			frozen: true,
 			url: "https://eslint.org/docs/latest/rules/no-magic-numbers",
@@ -107,11 +106,9 @@ module.exports = {
 				properties: {
 					detectObjects: {
 						type: "boolean",
-						default: false,
 					},
 					enforceConst: {
 						type: "boolean",
-						default: false,
 					},
 					ignore: {
 						type: "array",
@@ -128,37 +125,45 @@ module.exports = {
 					},
 					ignoreArrayIndexes: {
 						type: "boolean",
-						default: false,
 					},
 					ignoreDefaultValues: {
 						type: "boolean",
-						default: false,
 					},
 					ignoreClassFieldInitialValues: {
 						type: "boolean",
-						default: false,
 					},
 					ignoreEnums: {
 						type: "boolean",
-						default: false,
 					},
 					ignoreNumericLiteralTypes: {
 						type: "boolean",
-						default: false,
 					},
 					ignoreReadonlyClassProperties: {
 						type: "boolean",
-						default: false,
 					},
 					ignoreTypeIndexes: {
 						type: "boolean",
-						default: false,
 					},
 				},
 				additionalProperties: false,
 			},
 		],
 
+		defaultOptions: [
+			{
+				detectObjects: false,
+				enforceConst: false,
+				ignore: [],
+				ignoreArrayIndexes: false,
+				ignoreDefaultValues: false,
+				ignoreClassFieldInitialValues: false,
+				ignoreEnums: false,
+				ignoreNumericLiteralTypes: false,
+				ignoreReadonlyClassProperties: false,
+				ignoreTypeIndexes: false,
+			},
+		],
+
 		messages: {
 			useConst: "Number constants declarations must use 'const'.",
 			noMagic: "No magic number: {{raw}}.",
@@ -166,19 +171,19 @@ module.exports = {
 	},
 
 	create(context) {
-		const config = context.options[0] || {},
-			detectObjects = !!config.detectObjects,
-			enforceConst = !!config.enforceConst,
-			ignore = new Set((config.ignore || []).map(normalizeIgnoreValue)),
-			ignoreArrayIndexes = !!config.ignoreArrayIndexes,
-			ignoreDefaultValues = !!config.ignoreDefaultValues,
-			ignoreClassFieldInitialValues =
-				!!config.ignoreClassFieldInitialValues,
-			ignoreEnums = !!config.ignoreEnums,
-			ignoreNumericLiteralTypes = !!config.ignoreNumericLiteralTypes,
-			ignoreReadonlyClassProperties =
-				!!config.ignoreReadonlyClassProperties,
-			ignoreTypeIndexes = !!config.ignoreTypeIndexes;
+		const {
+			detectObjects,
+			enforceConst,
+			ignore: rawIgnore,
+			ignoreArrayIndexes,
+			ignoreDefaultValues,
+			ignoreClassFieldInitialValues,
+			ignoreEnums,
+			ignoreNumericLiteralTypes,
+			ignoreReadonlyClassProperties,
+			ignoreTypeIndexes,
+		} = context.options[0];
+		const ignore = new Set(rawIgnore.map(normalizeIgnoreValue));
 
 		const okTypes = detectObjects
 			? []

package/lib/rules/no-restricted-exports.js

@@ -92,6 +92,8 @@ module.exports = {
 			},
 		],
 
+		defaultOptions: [{}],
+
 		messages: {
 			restrictedNamed:
 				"'{{name}}' is restricted from being used as an exported name.",
@@ -100,14 +102,12 @@ module.exports = {
 	},
 
 	create(context) {
-		const restrictedNames = new Set(
-			context.options[0] && context.options[0].restrictedNamedExports,
-		);
-		const restrictedNamePattern =
-			context.options[0] &&
-			context.options[0].restrictedNamedExportsPattern;
-		const restrictDefaultExports =
-			context.options[0] && context.options[0].restrictDefaultExports;
+		const {
+			restrictedNamedExports,
+			restrictedNamedExportsPattern: restrictedNamePattern,
+			restrictDefaultExports,
+		} = context.options[0];
+		const restrictedNames = new Set(restrictedNamedExports);
 		const sourceCode = context.sourceCode;
 
 		/**

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

@@ -53,12 +53,11 @@ const arrayOfGlobals = {
 /** @type {import('../types').Rule.RuleModule} */
 module.exports = {
 	meta: {
-		dialects: ["javascript", "typescript"],
-		language: "javascript",
 		type: "suggestion",
 
 		docs: {
 			description: "Disallow specified global variables",
+			dialects: ["JavaScript", "TypeScript"],
 			recommended: false,
 			url: "https://eslint.org/docs/latest/rules/no-restricted-globals",
 		},

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

@@ -171,11 +171,10 @@ const arrayOfStringsOrObjectPatterns = {
 module.exports = {
 	meta: {
 		type: "suggestion",
-		dialects: ["typescript", "javascript"],
-		language: "javascript",
 
 		docs: {
 			description: "Disallow specified modules when loaded by `import`",
+			dialects: ["JavaScript", "TypeScript"],
 			recommended: false,
 			url: "https://eslint.org/docs/latest/rules/no-restricted-imports",
 		},

package/lib/rules/no-restricted-properties.js

@@ -70,6 +70,8 @@ module.exports = {
 			uniqueItems: true,
 		},
 
+		defaultOptions: [],
+
 		messages: {
 			restrictedObjectProperty:
 				// eslint-disable-next-line eslint-plugin/report-message-format -- Custom message might not end in a period

package/lib/rules/no-restricted-syntax.js

@@ -41,6 +41,8 @@ module.exports = {
 			minItems: 0,
 		},
 
+		defaultOptions: [],
+
 		messages: {
 			// eslint-disable-next-line eslint-plugin/report-message-format -- Custom message might not end in a period
 			restrictedSyntax: "{{message}}",

package/lib/rules/no-shadow.js

@@ -55,8 +55,6 @@ const ALLOWED_FUNCTION_VARIABLE_DEF_TYPES = new Set([
 module.exports = {
 	meta: {
 		type: "suggestion",
-		dialects: ["typescript", "javascript"],
-		language: "javascript",
 
 		defaultOptions: [
 			{
@@ -72,6 +70,7 @@ module.exports = {
 		docs: {
 			description:
 				"Disallow variable declarations from shadowing variables declared in the outer scope",
+			dialects: ["JavaScript", "TypeScript"],
 			recommended: false,
 			url: "https://eslint.org/docs/latest/rules/no-shadow",
 		},

package/lib/rules/no-unassigned-vars.js

@@ -12,12 +12,11 @@
 module.exports = {
 	meta: {
 		type: "problem",
-		dialects: ["typescript", "javascript"],
-		language: "javascript",
 
 		docs: {
 			description:
 				"Disallow `let` or `var` variables that are read but never assigned",
+			dialects: ["JavaScript", "TypeScript"],
 			recommended: true,
 			url: "https://eslint.org/docs/latest/rules/no-unassigned-vars",
 		},

package/lib/rules/no-unused-expressions.js

@@ -29,12 +29,11 @@ function alwaysFalse() {
 /** @type {import('../types').Rule.RuleModule} */
 module.exports = {
 	meta: {
-		dialects: ["javascript", "typescript"],
-		language: "javascript",
 		type: "suggestion",
 
 		docs: {
 			description: "Disallow unused expressions",
+			dialects: ["JavaScript", "TypeScript"],
 			recommended: false,
 			url: "https://eslint.org/docs/latest/rules/no-unused-expressions",
 		},

package/lib/rules/no-unused-vars.js

@@ -42,6 +42,20 @@ const astUtils = require("./utils/ast-utils");
  * @property {string} additional Any additional info to be appended at the end.
  */
 
+//------------------------------------------------------------------------------
+// Helpers
+//------------------------------------------------------------------------------
+
+const DEFAULT_OPTIONS = {
+	vars: "all",
+	args: "after-used",
+	ignoreRestSiblings: false,
+	caughtErrors: "all",
+	ignoreClassWithStaticInitBlock: false,
+	ignoreUsingDeclarations: false,
+	reportUsedIgnorePattern: false,
+};
+
 //------------------------------------------------------------------------------
 // Rule Definition
 //------------------------------------------------------------------------------
@@ -108,6 +122,8 @@ module.exports = {
 			},
 		],
 
+		defaultOptions: [DEFAULT_OPTIONS],
+
 		messages: {
 			unusedVar:
 				"'{{varName}}' is {{action}} but never used{{additional}}.",
@@ -123,65 +139,46 @@ module.exports = {
 		const REST_PROPERTY_TYPE =
 			/^(?:RestElement|(?:Experimental)?RestProperty)$/u;
 
-		const config = {
-			vars: "all",
-			args: "after-used",
-			ignoreRestSiblings: false,
-			caughtErrors: "all",
-			ignoreClassWithStaticInitBlock: false,
-			ignoreUsingDeclarations: false,
-			reportUsedIgnorePattern: false,
-		};
+		let config;
 
 		const firstOption = context.options[0];
 
-		if (firstOption) {
-			if (typeof firstOption === "string") {
-				config.vars = firstOption;
-			} else {
-				config.vars = firstOption.vars || config.vars;
-				config.args = firstOption.args || config.args;
-				config.ignoreRestSiblings =
-					firstOption.ignoreRestSiblings || config.ignoreRestSiblings;
-				config.caughtErrors =
-					firstOption.caughtErrors || config.caughtErrors;
-				config.ignoreClassWithStaticInitBlock =
-					firstOption.ignoreClassWithStaticInitBlock ||
-					config.ignoreClassWithStaticInitBlock;
-				config.ignoreUsingDeclarations =
-					firstOption.ignoreUsingDeclarations ||
-					config.ignoreUsingDeclarations;
-				config.reportUsedIgnorePattern =
-					firstOption.reportUsedIgnorePattern ||
-					config.reportUsedIgnorePattern;
-
-				if (firstOption.varsIgnorePattern) {
-					config.varsIgnorePattern = new RegExp(
-						firstOption.varsIgnorePattern,
-						"u",
-					);
-				}
+		if (typeof firstOption === "string") {
+			config = {
+				...DEFAULT_OPTIONS,
+				vars: firstOption,
+			};
+		} else {
+			config = {
+				...firstOption,
+			};
 
-				if (firstOption.argsIgnorePattern) {
-					config.argsIgnorePattern = new RegExp(
-						firstOption.argsIgnorePattern,
-						"u",
-					);
-				}
+			if (firstOption.varsIgnorePattern) {
+				config.varsIgnorePattern = new RegExp(
+					firstOption.varsIgnorePattern,
+					"u",
+				);
+			}
 
-				if (firstOption.caughtErrorsIgnorePattern) {
-					config.caughtErrorsIgnorePattern = new RegExp(
-						firstOption.caughtErrorsIgnorePattern,
-						"u",
-					);
-				}
+			if (firstOption.argsIgnorePattern) {
+				config.argsIgnorePattern = new RegExp(
+					firstOption.argsIgnorePattern,
+					"u",
+				);
+			}
 
-				if (firstOption.destructuredArrayIgnorePattern) {
-					config.destructuredArrayIgnorePattern = new RegExp(
-						firstOption.destructuredArrayIgnorePattern,
-						"u",
-					);
-				}
+			if (firstOption.caughtErrorsIgnorePattern) {
+				config.caughtErrorsIgnorePattern = new RegExp(
+					firstOption.caughtErrorsIgnorePattern,
+					"u",
+				);
+			}
+
+			if (firstOption.destructuredArrayIgnorePattern) {
+				config.destructuredArrayIgnorePattern = new RegExp(
+					firstOption.destructuredArrayIgnorePattern,
+					"u",
+				);
 			}
 		}
 

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

@@ -276,13 +276,12 @@ function isClassRefInClassDecorator(variable, reference) {
 /** @type {import('../types').Rule.RuleModule} */
 module.exports = {
 	meta: {
-		dialects: ["javascript", "typescript"],
-		language: "javascript",
 		type: "problem",
 
 		docs: {
 			description:
 				"Disallow the use of variables before they are defined",
+			dialects: ["JavaScript", "TypeScript"],
 			recommended: false,
 			url: "https://eslint.org/docs/latest/rules/no-use-before-define",
 		},

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

@@ -164,12 +164,11 @@ function isRedundantSuperCall(body, ctorParams) {
 /** @type {import('../types').Rule.RuleModule} */
 module.exports = {
 	meta: {
-		dialects: ["javascript", "typescript"],
-		language: "javascript",
 		type: "suggestion",
 
 		docs: {
 			description: "Disallow unnecessary constructors",
+			dialects: ["JavaScript", "TypeScript"],
 			recommended: false,
 			url: "https://eslint.org/docs/latest/rules/no-useless-constructor",
 		},

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",
@@ -324,6 +323,24 @@ module.exports = {
 								return;
 							}
 
+							const functionToken = sourceCode.getFirstToken(
+								node,
+								node.async ? 1 : 0,
+							);
+							const leftParenToken = sourceCode.getTokenAfter(
+								functionToken,
+								astUtils.isOpeningParenToken,
+							);
+
+							if (node.async) {
+								if (
+									functionToken.loc.end.line <
+									leftParenToken.loc.start.line
+								) {
+									return;
+								}
+							}
+
 							// Remove `.bind(this)` if exists.
 							if (callbackInfo.isLexicalThis) {
 								const memberNode = node.parent;
@@ -376,14 +393,6 @@ module.exports = {
 							}
 
 							// Convert the function expression to an arrow function.
-							const functionToken = sourceCode.getFirstToken(
-								node,
-								node.async ? 1 : 0,
-							);
-							const leftParenToken = sourceCode.getTokenAfter(
-								functionToken,
-								astUtils.isOpeningParenToken,
-							);
 							const tokenBeforeBody = sourceCode.getTokenBefore(
 								node.body,
 							);

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/lib/types/index.d.ts

@@ -723,6 +723,22 @@ export namespace Rule {
 		| "class-field-initializer"
 		| "class-static-block";
 
+	interface CodePathSegmentTraversalController {
+		skip(): void;
+		break(): void;
+	}
+
+	type CodePathSegmentTraversalCallback = (
+		this: CodePath,
+		segment: CodePathSegment,
+		controller: CodePathSegmentTraversalController,
+	) => void;
+
+	interface CodePathTraversalOptions {
+		first?: CodePathSegment | undefined;
+		last?: CodePathSegment | undefined;
+	}
+
 	interface CodePath {
 		id: string;
 		origin: CodePathOrigin;
@@ -732,12 +748,19 @@ export namespace Rule {
 		thrownSegments: CodePathSegment[];
 		upper: CodePath | null;
 		childCodePaths: CodePath[];
+		traverseSegments(callback: CodePathSegmentTraversalCallback): void;
+		traverseSegments(
+			options: CodePathTraversalOptions,
+			callback: CodePathSegmentTraversalCallback,
+		): void;
 	}
 
 	interface CodePathSegment {
 		id: string;
 		nextSegments: CodePathSegment[];
 		prevSegments: CodePathSegment[];
+		allNextSegments: CodePathSegment[];
+		allPrevSegments: CodePathSegment[];
 		reachable: boolean;
 	}
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint",
-  "version": "10.3.0",
+  "version": "10.4.1",
   "author": "Nicholas C. Zakas <nicholas+npm@nczconsulting.com>",
   "description": "An AST-based pattern checker for JavaScript.",
   "type": "commonjs",
@@ -122,9 +122,9 @@
     "@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",
+    "@eslint/plugin-kit": "^0.7.2",
     "@humanfs/node": "^0.16.6",
     "@humanwhocodes/module-importer": "^1.0.1",
     "@humanwhocodes/retry": "^0.4.2",
@@ -183,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",
@@ -201,7 +201,6 @@
     "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",
@@ -229,5 +228,10 @@
   "license": "MIT",
   "engines": {
     "node": "^20.19.0 || ^22.13.0 || >=24"
+  },
+  "overrides": {
+    "@arethetypeswrong/core": {
+      "fflate": "0.8.2"
+    }
   }
 }