eslint 10.3.0 → 10.4.0

package/README.md

@@ -315,8 +315,8 @@ Tanuj Kanti
 </a>
 </td><td align="center" valign="top" width="11%">
 <a href="https://github.com/lumirlumir">
-<img src="https://github.com/lumirlumir.png?s=75" width="75" height="75" alt="루밀LuMir's Avatar"><br />
-루밀LuMir
+<img src="https://github.com/lumirlumir.png?s=75" width="75" height="75" alt="lumir's Avatar"><br />
+lumir
 </a>
 </td><td align="center" valign="top" width="11%">
 <a href="https://github.com/Pixel998">
@@ -360,7 +360,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></p><h3>Gold Sponsors</h3>
 <p><a href="https://qlty.sh/"><img src="https://images.opencollective.com/qltysh/33d157d/logo.png" alt="Qlty Software" height="96"></a></p><h3>Silver Sponsors</h3>
 <p><a href="https://vite.dev/"><img src="https://images.opencollective.com/vite/d472863/logo.png" alt="Vite" height="64"></a> <a href="https://liftoff.io/"><img src="https://images.opencollective.com/liftoff/2d6c3b6/logo.png" alt="Liftoff" height="64"></a> <a href="https://stackblitz.com"><img src="https://avatars.githubusercontent.com/u/28635252" alt="StackBlitz" 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://opensource.sap.com"><img src="https://avatars.githubusercontent.com/u/2531208" alt="SAP" height="32"></a> <a href="https://www.crawljobs.com/"><img src="https://images.opencollective.com/crawljobs-poland/fa43a17/logo.png" alt="CrawlJobs" height="32"></a> <a href="https://depot.dev"><img src="https://images.opencollective.com/depot/39125a1/logo.png" alt="Depot" 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" alt="GitBook" height="32"></a> <a href="https://herocoders.com"><img src="https://avatars.githubusercontent.com/u/37549774" alt="HeroCoders" height="32"></a> <a href="https://citadel.co.jp"><img src="https://avatars.githubusercontent.com/u/75781367" alt="Citadel AI" height="32"></a> <a href="https://www.lambdatest.com"><img src="https://avatars.githubusercontent.com/u/171592363" alt="TestMu AI Open Source Office (Formerly LambdaTest)" 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://opensource.sap.com"><img src="https://avatars.githubusercontent.com/u/2531208" alt="SAP" height="32"></a> <a href="https://www.crawljobs.com/"><img src="https://images.opencollective.com/crawljobs-poland/fa43a17/logo.png" alt="CrawlJobs" height="32"></a> <a href="https://syntax.fm"><img src="https://github.com/syntaxfm.png" alt="Syntax" height="32"></a> <a href="https://depot.dev"><img src="https://images.opencollective.com/depot/39125a1/logo.png" alt="Depot" 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" alt="GitBook" height="32"></a> <a href="https://herocoders.com"><img src="https://avatars.githubusercontent.com/u/37549774" alt="HeroCoders" height="32"></a> <a href="https://citadel.co.jp"><img src="https://avatars.githubusercontent.com/u/75781367" alt="Citadel AI" height="32"></a> <a href="https://www.lambdatest.com"><img src="https://avatars.githubusercontent.com/u/171592363" alt="TestMu AI Open Source Office (Formerly LambdaTest)" 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/config-api.js

@@ -4,9 +4,15 @@
  */
 
 "use strict";
-const { defineConfig, globalIgnores } = require("@eslint/config-helpers");
+
+const {
+	defineConfig,
+	globalIgnores,
+	includeIgnoreFile,
+} = require("@eslint/config-helpers");
 
 module.exports = {
 	defineConfig,
 	globalIgnores,
+	includeIgnoreFile,
 };

package/lib/eslint/eslint.js

@@ -107,6 +107,34 @@ function createRulesMeta(rules) {
 	}, {});
 }
 
+/**
+ * Gets the replacement rule names from a deprecated rule's metadata.
+ * @param {RulesMeta} meta The rule metadata.
+ * @returns {string[]} Replacement rule names.
+ */
+function getDeprecatedRuleReplacements(meta) {
+	if (typeof meta.deprecated !== "object") {
+		return meta.replacedBy || [];
+	}
+
+	const { replacedBy } = meta.deprecated;
+
+	if (!Array.isArray(replacedBy)) {
+		return [];
+	}
+
+	return replacedBy.map(replacement => {
+		if (typeof replacement !== "object" || replacement === null) {
+			return "";
+		}
+
+		const pluginName = replacement.plugin?.name;
+		const ruleName = replacement.rule?.name;
+
+		return `${typeof pluginName === "string" ? `${getShorthandName(pluginName, "eslint-plugin")}/` : ""}${typeof ruleName === "string" ? ruleName : ""}`;
+	});
+}
+
 /** @type {WeakMap<CalculatedConfig, DeprecatedRuleInfo[]>} */
 const usedDeprecatedRulesCache = new WeakMap();
 
@@ -144,12 +172,7 @@ function getOrFindUsedDeprecatedRules(eslint, maybeFilePath) {
 
 					retv.push({
 						ruleId,
-						replacedBy: usesNewFormat
-							? (meta.deprecated.replacedBy?.map(
-									replacement =>
-										`${replacement.plugin?.name !== void 0 ? `${getShorthandName(replacement.plugin.name, "eslint-plugin")}/` : ""}${replacement.rule?.name ?? ""}`,
-								) ?? [])
-							: meta.replacedBy || [],
+						replacedBy: getDeprecatedRuleReplacements(meta),
 						info: usesNewFormat ? meta.deprecated : void 0,
 					});
 				}

package/lib/linter/code-path-analysis/debug-helpers.js

@@ -45,6 +45,15 @@ function nodeToString(node, label) {
 	}
 }
 
+/**
+ * Escape text for use in a DOT label.
+ * @param {string} value The value to escape.
+ * @returns {string} The escaped value.
+ */
+function escapeDotLabelText(value) {
+	return value.replace(/\\/gu, String.raw`\\`).replace(/"/gu, String.raw`\"`);
+}
+
 //------------------------------------------------------------------------------
 // Public Interface
 //------------------------------------------------------------------------------
@@ -147,7 +156,9 @@ module.exports = {
 					}
 
 					if (segment.internal.nodes.length > 0) {
-						text += segment.internal.nodes.join("\\n");
+						text += segment.internal.nodes
+							.map(escapeDotLabelText)
+							.join("\\n");
 					} else {
 						text += "????";
 					}

package/lib/rules/capitalized-comments.js

@@ -131,6 +131,8 @@ module.exports = {
 			},
 		],
 
+		defaultOptions: ["always"],
+
 		messages: {
 			unexpectedLowercaseComment:
 				"Comments should not begin with a lowercase character.",
@@ -140,7 +142,7 @@ module.exports = {
 	},
 
 	create(context) {
-		const capitalize = context.options[0] || "always",
+		const capitalize = context.options[0],
 			normalizedOptions = getAllNormalizedOptions(context.options[1]),
 			sourceCode = context.sourceCode;
 

package/lib/rules/class-methods-use-this.js

@@ -18,8 +18,6 @@ const astUtils = require("./utils/ast-utils");
 /** @type {import('../types').Rule.RuleModule} */
 module.exports = {
 	meta: {
-		dialects: ["javascript", "typescript"],
-		language: "javascript",
 		type: "suggestion",
 
 		defaultOptions: [
@@ -32,6 +30,7 @@ module.exports = {
 
 		docs: {
 			description: "Enforce that class methods utilize `this`",
+			dialects: ["JavaScript", "TypeScript"],
 			recommended: false,
 			url: "https://eslint.org/docs/latest/rules/class-methods-use-this",
 		},

package/lib/rules/default-param-last.js

@@ -21,12 +21,11 @@ function isRequiredParameter(node) {
 /** @type {import('../types').Rule.RuleModule} */
 module.exports = {
 	meta: {
-		dialects: ["javascript", "typescript"],
-		language: "javascript",
 		type: "suggestion",
 
 		docs: {
 			description: "Enforce default parameters to be last",
+			dialects: ["JavaScript", "TypeScript"],
 			recommended: false,
 			frozen: true,
 			url: "https://eslint.org/docs/latest/rules/default-param-last",

package/lib/rules/eqeqeq.js

@@ -59,6 +59,8 @@ module.exports = {
 			],
 		},
 
+		defaultOptions: ["always"],
+
 		fixable: "code",
 
 		messages: {
@@ -70,7 +72,7 @@ module.exports = {
 	},
 
 	create(context) {
-		const config = context.options[0] || "always";
+		const config = context.options[0];
 		const options = context.options[1] || {};
 		const sourceCode = context.sourceCode;
 

package/lib/rules/for-direction.js

@@ -117,6 +117,50 @@ module.exports = {
 			}
 			return 0;
 		}
+		/**
+		 * Collects all expressions that modify the counter.
+		 * @param {ASTNode} node The expression node to check.
+		 * @param {string} counter The name of the counter variable.
+		 * @returns {ASTNode[]} An array of modifying expressions.
+		 */
+		function getModifyingExpressions(node, counter) {
+			if (node.type === "SequenceExpression") {
+				return node.expressions.flatMap(expr =>
+					getModifyingExpressions(expr, counter),
+				);
+			}
+			if (
+				node.type === "UpdateExpression" &&
+				node.argument.type === "Identifier" &&
+				node.argument.name === counter
+			) {
+				return [node];
+			}
+			if (
+				node.type === "AssignmentExpression" &&
+				node.left.type === "Identifier" &&
+				node.left.name === counter
+			) {
+				return [node];
+			}
+			return [];
+		}
+
+		/**
+		 * Determines the direction of a single update expression for the counter.
+		 * @param {ASTNode} expr An expression node to check (UpdateExpression or AssignmentExpression).
+		 * @param {string} counter The variable name of the counter.
+		 * @returns {number} 1 if incrementing, -1 if decrementing, 0 if unknown or not modifying the counter.
+		 */
+		function getDirectionFromExpression(expr, counter) {
+			if (expr.type === "UpdateExpression") {
+				return getUpdateDirection(expr, counter);
+			}
+			if (expr.type === "AssignmentExpression") {
+				return getAssignmentDirection(expr, counter);
+			}
+			return 0;
+		}
 
 		return {
 			ForStatement(node) {
@@ -146,17 +190,17 @@ module.exports = {
 							return;
 						}
 
-						if (update.type === "UpdateExpression") {
-							if (
-								getUpdateDirection(update, counter) ===
-								wrongDirection
-							) {
-								report(node);
-							}
-						} else if (
-							update.type === "AssignmentExpression" &&
-							getAssignmentDirection(update, counter) ===
-								wrongDirection
+						const mutatingExpressions = getModifyingExpressions(
+							update,
+							counter,
+						);
+
+						if (
+							mutatingExpressions.length === 1 &&
+							getDirectionFromExpression(
+								mutatingExpressions[0],
+								counter,
+							) === wrongDirection
 						) {
 							report(node);
 						}

package/lib/rules/func-name-matching.js

@@ -105,6 +105,8 @@ module.exports = {
 			],
 		},
 
+		defaultOptions: ["always"],
+
 		messages: {
 			matchProperty:
 				"Function name `{{funcName}}` should match property name `{{name}}`.",

package/lib/rules/func-style.js

@@ -11,8 +11,6 @@
 /** @type {import('../types').Rule.RuleModule} */
 module.exports = {
 	meta: {
-		dialects: ["javascript", "typescript"],
-		language: "javascript",
 		type: "suggestion",
 
 		defaultOptions: [
@@ -27,6 +25,7 @@ module.exports = {
 		docs: {
 			description:
 				"Enforce the consistent use of either `function` declarations or expressions assigned to variables",
+			dialects: ["JavaScript", "TypeScript"],
 			recommended: false,
 			frozen: true,
 			url: "https://eslint.org/docs/latest/rules/func-style",

package/lib/rules/init-declarations.js

@@ -50,12 +50,11 @@ function isInitialized(node) {
 module.exports = {
 	meta: {
 		type: "suggestion",
-		dialects: ["typescript", "javascript"],
-		language: "javascript",
 
 		docs: {
 			description:
 				"Require or disallow initialization in variable declarations",
+			dialects: ["JavaScript", "TypeScript"],
 			recommended: false,
 			frozen: true,
 			url: "https://eslint.org/docs/latest/rules/init-declarations",
@@ -94,6 +93,9 @@ module.exports = {
 				},
 			],
 		},
+
+		defaultOptions: ["always"],
+
 		messages: {
 			initialized:
 				"Variable '{{idName}}' should be initialized on declaration.",
@@ -103,10 +105,7 @@ module.exports = {
 	},
 
 	create(context) {
-		const MODE_ALWAYS = "always",
-			MODE_NEVER = "never";
-
-		const mode = context.options[0] || MODE_ALWAYS;
+		const mode = context.options[0];
 		const params = context.options[1] || {};
 
 		// Track whether we're inside a declared namespace
@@ -145,10 +144,10 @@ module.exports = {
 							params.ignoreForLoopInit && isForLoop(node.parent);
 					let messageId = "";
 
-					if (mode === MODE_ALWAYS && !initialized) {
+					if (mode === "always" && !initialized) {
 						messageId = "initialized";
 					} else if (
-						mode === MODE_NEVER &&
+						mode === "never" &&
 						!CONSTANT_BINDINGS.has(kind) &&
 						initialized &&
 						!isIgnoredForLoop

package/lib/rules/logical-assignment-operators.js

@@ -259,6 +259,9 @@ module.exports = {
 				},
 			],
 		},
+
+		defaultOptions: ["always"],
+
 		fixable: "code",
 		hasSuggestions: true,
 		messages: {
@@ -281,7 +284,7 @@ module.exports = {
 	},
 
 	create(context) {
-		const mode = context.options[0] === "never" ? "never" : "always";
+		const mode = context.options[0];
 		const checkIf =
 			mode === "always" &&
 			context.options.length > 1 &&

package/lib/rules/max-classes-per-file.js

@@ -47,16 +47,18 @@ module.exports = {
 			},
 		],
 
+		defaultOptions: [1],
+
 		messages: {
 			maximumExceeded:
 				"File has too many classes ({{ classCount }}). Maximum allowed is {{ max }}.",
 		},
 	},
 	create(context) {
-		const [option = {}] = context.options;
+		const option = context.options[0];
 		const [ignoreExpressions, max] =
 			typeof option === "number"
-				? [false, option || 1]
+				? [false, option]
 				: [option.ignoreExpressions, option.max || 1];
 
 		let classCount = 0;

package/lib/rules/max-depth.js

@@ -44,6 +44,9 @@ module.exports = {
 				],
 			},
 		],
+
+		defaultOptions: [4],
+
 		messages: {
 			tooDeeply:
 				"Blocks are nested too deeply ({{depth}}). Maximum allowed is {{maxDepth}}.",

package/lib/rules/max-lines-per-function.js

@@ -78,6 +78,9 @@ module.exports = {
 		},
 
 		schema: [OPTIONS_OR_INTEGER_SCHEMA],
+
+		defaultOptions: [50],
+
 		messages: {
 			exceed: "{{name}} has too many lines ({{lineCount}}). Maximum allowed is {{maxLines}}.",
 		},

package/lib/rules/max-lines.js

@@ -65,6 +65,9 @@ module.exports = {
 				],
 			},
 		],
+
+		defaultOptions: [300],
+
 		messages: {
 			exceed: "File has too many lines ({{actual}}). Maximum allowed is {{max}}.",
 		},

package/lib/rules/max-nested-callbacks.js

@@ -44,6 +44,9 @@ module.exports = {
 				],
 			},
 		],
+
+		defaultOptions: [10],
+
 		messages: {
 			exceed: "Too many nested callbacks ({{num}}). Maximum allowed is {{max}}.",
 		},

package/lib/rules/max-params.js

@@ -20,12 +20,11 @@ const { upperCaseFirst } = require("../shared/string-utils");
 module.exports = {
 	meta: {
 		type: "suggestion",
-		dialects: ["typescript", "javascript"],
-		language: "javascript",
 
 		docs: {
 			description:
 				"Enforce a maximum number of parameters in function definitions",
+			dialects: ["JavaScript", "TypeScript"],
 			recommended: false,
 			url: "https://eslint.org/docs/latest/rules/max-params",
 		},
@@ -65,6 +64,9 @@ module.exports = {
 				],
 			},
 		],
+
+		defaultOptions: [3],
+
 		messages: {
 			exceed: "{{name}} has too many parameters ({{count}}). Maximum allowed is {{max}}.",
 		},

package/lib/rules/max-statements.js

@@ -61,6 +61,9 @@ module.exports = {
 				additionalProperties: false,
 			},
 		],
+
+		defaultOptions: [10],
+
 		messages: {
 			exceed: "{{name}} has too many statements ({{count}}). Maximum allowed is {{max}}.",
 		},

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

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

package/lib/rules/no-dupe-class-members.js

@@ -15,11 +15,10 @@ const astUtils = require("./utils/ast-utils");
 module.exports = {
 	meta: {
 		type: "problem",
-		dialects: ["javascript", "typescript"],
-		language: "javascript",
 
 		docs: {
 			description: "Disallow duplicate class members",
+			dialects: ["JavaScript", "TypeScript"],
 			recommended: true,
 			url: "https://eslint.org/docs/latest/rules/no-dupe-class-members",
 		},

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

@@ -294,8 +294,6 @@ function handleImportsExports(
 /** @type {import('../types').Rule.RuleModule} */
 module.exports = {
 	meta: {
-		dialects: ["javascript", "typescript"],
-		language: "javascript",
 		type: "problem",
 
 		defaultOptions: [
@@ -307,6 +305,7 @@ module.exports = {
 
 		docs: {
 			description: "Disallow duplicate module imports",
+			dialects: ["JavaScript", "TypeScript"],
 			recommended: false,
 			url: "https://eslint.org/docs/latest/rules/no-duplicate-imports",
 		},

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

@@ -103,8 +103,6 @@ function isParameterPropertiesConstructor(node) {
 /** @type {import('../types').Rule.RuleModule} */
 module.exports = {
 	meta: {
-		dialects: ["javascript", "typescript"],
-		language: "javascript",
 		hasSuggestions: true,
 		type: "suggestion",
 
@@ -112,6 +110,7 @@ module.exports = {
 
 		docs: {
 			description: "Disallow empty functions",
+			dialects: ["JavaScript", "TypeScript"],
 			recommended: false,
 			url: "https://eslint.org/docs/latest/rules/no-empty-function",
 		},

package/lib/rules/no-invalid-this.js

@@ -36,8 +36,6 @@ function isCodePathWithLexicalThis(codePath, node) {
 /** @type {import('../types').Rule.RuleModule} */
 module.exports = {
 	meta: {
-		dialects: ["javascript", "typescript"],
-		language: "javascript",
 		type: "suggestion",
 
 		defaultOptions: [{ capIsConstructor: true }],
@@ -45,6 +43,7 @@ module.exports = {
 		docs: {
 			description:
 				"Disallow use of `this` in contexts where the value of `this` is `undefined`",
+			dialects: ["JavaScript", "TypeScript"],
 			recommended: false,
 			url: "https://eslint.org/docs/latest/rules/no-invalid-this",
 		},

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

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.3.0",
+  "version": "10.4.0",
   "author": "Nicholas C. Zakas <nicholas+npm@nczconsulting.com>",
   "description": "An AST-based pattern checker for JavaScript.",
   "type": "commonjs",
@@ -122,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",
@@ -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",