eslint 10.0.0-alpha.1 → 10.0.0-beta.0

package/README.md

@@ -343,7 +343,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> <a href="https://www.airbnb.com/"><img src="https://images.opencollective.com/airbnb/d327d66/logo.png" alt="Airbnb" 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> <a href="https://shopify.engineering/"><img src="https://avatars.githubusercontent.com/u/8085" alt="Shopify" height="96"></a></p><h3>Silver Sponsors</h3>
 <p><a href="https://vite.dev/"><img src="https://images.opencollective.com/vite/e6d15e1/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://americanexpress.io"><img src="https://avatars.githubusercontent.com/u/3853301" alt="American Express" 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://syntax.fm"><img src="https://github.com/syntaxfm.png" alt="Syntax" height="32"></a> <a href="https://www.n-ix.com/"><img src="https://images.opencollective.com/n-ix-ltd/575a7a5/logo.png" alt="N-iX Ltd" 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://nx.dev"><img src="https://avatars.githubusercontent.com/u/23692104" alt="Nx" height="32"></a> <a href="https://opensource.mercedes-benz.com/"><img src="https://avatars.githubusercontent.com/u/34240465" alt="Mercedes-Benz Group" height="32"></a> <a href="https://herocoders.com"><img src="https://avatars.githubusercontent.com/u/37549774" alt="HeroCoders" height="32"></a> <a href="https://www.lambdatest.com"><img src="https://avatars.githubusercontent.com/u/171592363" alt="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://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://www.n-ix.com/"><img src="https://images.opencollective.com/n-ix-ltd/575a7a5/logo.png" alt="N-iX Ltd" 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://nx.dev"><img src="https://avatars.githubusercontent.com/u/23692104" alt="Nx" height="32"></a> <a href="https://herocoders.com"><img src="https://avatars.githubusercontent.com/u/37549774" alt="HeroCoders" height="32"></a> <a href="https://www.lambdatest.com"><img src="https://avatars.githubusercontent.com/u/171592363" alt="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/api.js

@@ -19,8 +19,8 @@ const { SourceCode } = require("./languages/js/source-code");
 //-----------------------------------------------------------------------------
 
 /**
- * Loads the correct ESLint constructor given the options.
- * @returns {Promise<ESLint>} The ESLint constructor
+ * Loads the correct `ESLint` constructor.
+ * @returns {Promise<ESLint>} The ESLint constructor.
  */
 async function loadESLint() {
 	return ESLint;

package/lib/cli-engine/formatters/stylish.js

@@ -4,14 +4,30 @@
  */
 "use strict";
 
-const chalk = require("chalk"),
-	util = require("node:util"),
+const util = require("node:util"),
 	table = require("../../shared/text-table");
 
 //------------------------------------------------------------------------------
 // Helpers
 //------------------------------------------------------------------------------
 
+/**
+ * Returns a styling function based on the color option.
+ * @param {boolean|undefined} color Indicates whether to use colors.
+ * @returns {Function} A function that styles text.
+ */
+function getStyleText(color) {
+	if (typeof color === "undefined") {
+		return (format, text) =>
+			util.styleText(format, text, { validateStream: true });
+	}
+	if (color) {
+		return (format, text) =>
+			util.styleText(format, text, { validateStream: false });
+	}
+	return (_, text) => text;
+}
+
 /**
  * Given a word and a count, append an s if count is not one.
  * @param {string} word A word in its singular form.
@@ -26,7 +42,9 @@ function pluralize(word, count) {
 // Public Interface
 //------------------------------------------------------------------------------
 
-module.exports = function (results) {
+module.exports = function (results, data) {
+	const styleText = getStyleText(data?.color);
+
 	let output = "\n",
 		errorCount = 0,
 		warningCount = 0,
@@ -46,17 +64,17 @@ module.exports = function (results) {
 		fixableErrorCount += result.fixableErrorCount;
 		fixableWarningCount += result.fixableWarningCount;
 
-		output += `${chalk.underline(result.filePath)}\n`;
+		output += `${styleText("underline", result.filePath)}\n`;
 
 		output += `${table(
 			messages.map(message => {
 				let messageType;
 
 				if (message.fatal || message.severity === 2) {
-					messageType = chalk.red("error");
+					messageType = styleText("red", "error");
 					summaryColor = "red";
 				} else {
-					messageType = chalk.yellow("warning");
+					messageType = styleText("yellow", "warning");
 				}
 
 				return [
@@ -65,7 +83,7 @@ module.exports = function (results) {
 					String(message.column || 0),
 					messageType,
 					message.message.replace(/([^ ])\.$/u, "$1"),
-					chalk.dim(message.ruleId || ""),
+					message.ruleId ? styleText("dim", message.ruleId) : "",
 				];
 			}),
 			{
@@ -78,7 +96,7 @@ module.exports = function (results) {
 			.split("\n")
 			.map(el =>
 				el.replace(/(\d+)\s+(\d+)/u, (m, p1, p2) =>
-					chalk.dim(`${p1}:${p2}`),
+					styleText("dim", `${p1}:${p2}`),
 				),
 			)
 			.join("\n")}\n\n`;
@@ -86,37 +104,50 @@ module.exports = function (results) {
 
 	const total = errorCount + warningCount;
 
+	/*
+	 * We can't use a single `styleText` call like `styleText([summaryColor, "bold"], text)` here.
+	 * This is a bug in `util.styleText` in Node.js versions earlier than v22.15.0 (https://github.com/nodejs/node/issues/56717).
+	 * As a workaround, we use nested `styleText` calls.
+	 */
 	if (total > 0) {
-		output += chalk[summaryColor].bold(
-			[
-				"\u2716 ",
-				total,
-				pluralize(" problem", total),
-				" (",
-				errorCount,
-				pluralize(" error", errorCount),
-				", ",
-				warningCount,
-				pluralize(" warning", warningCount),
-				")\n",
-			].join(""),
-		);
-
-		if (fixableErrorCount > 0 || fixableWarningCount > 0) {
-			output += chalk[summaryColor].bold(
+		output += `${styleText(
+			summaryColor,
+			styleText(
+				"bold",
 				[
-					"  ",
-					fixableErrorCount,
-					pluralize(" error", fixableErrorCount),
-					" and ",
-					fixableWarningCount,
-					pluralize(" warning", fixableWarningCount),
-					" potentially fixable with the `--fix` option.\n",
+					"\u2716 ",
+					total,
+					pluralize(" problem", total),
+					" (",
+					errorCount,
+					pluralize(" error", errorCount),
+					", ",
+					warningCount,
+					pluralize(" warning", warningCount),
+					")",
 				].join(""),
-			);
+			),
+		)}\n`;
+
+		if (fixableErrorCount > 0 || fixableWarningCount > 0) {
+			output += `${styleText(
+				summaryColor,
+				styleText(
+					"bold",
+					[
+						"  ",
+						fixableErrorCount,
+						pluralize(" error", fixableErrorCount),
+						" and ",
+						fixableWarningCount,
+						pluralize(" warning", fixableWarningCount),
+						" potentially fixable with the `--fix` option.",
+					].join(""),
+				),
+			)}\n`;
 		}
 	}
 
 	// Resets output color, for prevent change on top level
-	return total > 0 ? chalk.reset(output) : "";
+	return total > 0 ? styleText("reset", output) : "";
 };

package/lib/cli.js

@@ -454,14 +454,24 @@ const cli = {
 		const tooManyWarnings =
 			options.maxWarnings >= 0 &&
 			resultCounts.warningCount > options.maxWarnings;
-		const resultsMeta = tooManyWarnings
-			? {
-					maxWarningsExceeded: {
-						maxWarnings: options.maxWarnings,
-						foundWarnings: resultCounts.warningCount,
-					},
-				}
-			: {};
+		const resultsMeta = /** @type {ResultsMeta} */ ({});
+
+		/*
+		 * `--color` was set, `options.color` is `true`.
+		 * `--no-color` was set, `options.color` is `false`.
+		 * Neither option was provided, `options.color` is omitted, so `undefined`.
+		 */
+		if (options.color !== void 0) {
+			debug(`Color setting for output: ${options.color}`);
+			resultsMeta.color = options.color;
+		}
+
+		if (tooManyWarnings) {
+			resultsMeta.maxWarningsExceeded = {
+				maxWarnings: options.maxWarnings,
+				foundWarnings: resultCounts.warningCount,
+			};
+		}
 
 		if (
 			await printResults(

package/lib/eslint/eslint.js

@@ -1207,6 +1207,7 @@ class ESLint {
 		const { cwd } = privateMembers.get(this).options;
 
 		let formatterPath;
+		let isBuiltInFormatter = false;
 
 		// if there's a slash, then it's a file (TODO: this check seems dubious for scoped npm packages)
 		if (!namespace && normalizedFormatName.includes("/")) {
@@ -1228,6 +1229,7 @@ class ESLint {
 					"formatters",
 					`${normalizedFormatName}.js`,
 				);
+				isBuiltInFormatter = true;
 			}
 		}
 
@@ -1237,7 +1239,7 @@ class ESLint {
 			formatter = (await import(pathToFileURL(formatterPath))).default;
 		} catch (ex) {
 			// check for formatters that have been removed
-			if (removedFormatters.has(name)) {
+			if (isBuiltInFormatter && removedFormatters.has(name)) {
 				ex.message = `The ${name} formatter is no longer part of core ESLint. Install it manually with \`npm install -D eslint-formatter-${name}\``;
 			} else {
 				ex.message = `There was a problem loading formatter: ${formatterPath}\nError: ${ex.message}`;
@@ -1344,7 +1346,7 @@ class ESLint {
 
 /**
  * Returns whether flat config should be used.
- * @returns {Promise<boolean>} Whether flat config should be used.
+ * @returns {Promise<true>} Whether flat config should be used.
  */
 async function shouldUseFlatConfig() {
 	return true;

package/lib/languages/js/index.js

@@ -55,6 +55,7 @@ function analyzeScope(ast, languageOptions, visitorKeys) {
 		sourceType: languageOptions.sourceType || "script",
 		childVisitorKeys: visitorKeys || evk.KEYS,
 		fallback: evk.getKeys,
+		jsx: ecmaFeatures.jsx,
 	});
 }
 

package/lib/linter/linter.js

@@ -473,6 +473,7 @@ function analyzeScope(ast, languageOptions, visitorKeys) {
 		sourceType: languageOptions.sourceType || "script",
 		childVisitorKeys: visitorKeys || evk.KEYS,
 		fallback: Traverser.getKeys,
+		jsx: ecmaFeatures.jsx,
 	});
 }
 

package/lib/rule-tester/rule-tester.js

@@ -387,9 +387,10 @@ function normalizeTestCase(item) {
  * Asserts that the `errors` property of an invalid test case is valid.
  * @param {number | string[]} errors The `errors` property of the invalid test case.
  * @param {string} ruleName The name of the rule being tested.
+ * @param {Object} [assertionOptions] The assertion options for the test case.
  * @returns {void}
  */
-function assertErrorsProperty(errors, ruleName) {
+function assertErrorsProperty(errors, ruleName, assertionOptions = {}) {
 	const isNumber = typeof errors === "number";
 	const isArray = Array.isArray(errors);
 
@@ -407,12 +408,75 @@ function assertErrorsProperty(errors, ruleName) {
 		}
 	}
 
+	const { requireMessage = false, requireLocation = false } =
+		assertionOptions;
+
 	if (isArray) {
 		assert.ok(
 			errors.length !== 0,
 			"Invalid cases must have at least one error",
 		);
+
+		for (const [number, error] of errors.entries()) {
+			if (typeof error === "string" || error instanceof RegExp) {
+				// Just an error message.
+				assert.ok(
+					requireMessage !== "messageId" && !requireLocation,
+					`errors[${number}] should be an object when 'assertionOptions.requireMessage' is 'messageId' or 'assertionOptions.requireLocation' is true.`,
+				);
+			} else if (typeof error === "object" && error !== null) {
+				/*
+				 * Error object.
+				 * This may have a message, messageId, data, line, and/or column.
+				 */
+
+				for (const propertyName of Object.keys(error)) {
+					assert.ok(
+						errorObjectParameters.has(propertyName),
+						`Invalid error property name '${propertyName}'. Expected one of ${friendlyErrorObjectParameterList}.`,
+					);
+				}
+
+				if (requireMessage === "message") {
+					assert.ok(
+						!hasOwnProperty(error, "messageId") &&
+							hasOwnProperty(error, "message"),
+						`errors[${number}] should specify 'message' (and not 'messageId') when 'assertionOptions.requireMessage' is 'message'.`,
+					);
+				} else if (requireMessage === "messageId") {
+					assert.ok(
+						!hasOwnProperty(error, "message") &&
+							hasOwnProperty(error, "messageId"),
+						`errors[${number}] should specify 'messageId' (and not 'message') when 'assertionOptions.requireMessage' is 'messageId'.`,
+					);
+				}
+
+				if (hasOwnProperty(error, "message")) {
+					assert.ok(
+						!hasOwnProperty(error, "messageId"),
+						`errors[${number}] should not specify both 'message' and 'messageId'.`,
+					);
+					assert.ok(
+						!hasOwnProperty(error, "data"),
+						`errors[${number}] should not specify both 'data' and 'message'.`,
+					);
+				} else {
+					assert.ok(
+						hasOwnProperty(error, "messageId"),
+						`errors[${number}] must specify either 'messageId' or 'message'.`,
+					);
+				}
+			} else {
+				assert.fail(
+					`errors[${number}] must be a string, RegExp, or an object.`,
+				);
+			}
+		}
 	} else {
+		assert.ok(
+			!requireMessage && !requireLocation,
+			"Invalid cases must have 'errors' value as an array",
+		);
 		assert.ok(
 			errors > 0,
 			"Invalid cases must have 'error' value greater than 0",
@@ -564,13 +628,19 @@ function assertValidTestCase(item, seenTestCases) {
  * @param {Object} item The invalid test case object to check.
  * @param {Set<string>} seenTestCases Set of serialized test cases to check for duplicates.
  * @param {string} ruleName The name of the rule being tested.
+ * @param {Object} [assertionOptions] The assertion options for the test case.
  * @returns {void}
  * @throws {AssertionError} If the test case is not valid.
  */
-function assertInvalidTestCase(item, seenTestCases, ruleName) {
+function assertInvalidTestCase(
+	item,
+	seenTestCases,
+	ruleName,
+	assertionOptions = {},
+) {
 	assertTestCommonProperties(item);
 
-	assertErrorsProperty(item.errors, ruleName);
+	assertErrorsProperty(item.errors, ruleName, assertionOptions);
 
 	// 'output' is optional, but if it exists it must be a string or null
 	if (hasOwnProperty(item, "output")) {
@@ -763,6 +833,10 @@ class RuleTester {
 	 * @param {string} ruleName The name of the rule to run.
 	 * @param {RuleDefinition} rule The rule to test.
 	 * @param {{
+	 *   assertionOptions?: {
+	 *     requireMessage?: boolean | "message" | "messageId",
+	 *     requireLocation?: boolean
+	 *   },
 	 *   valid: (ValidTestCase | string)[],
 	 *   invalid: InvalidTestCase[]
 	 * }} test The collection of tests to run.
@@ -1143,6 +1217,9 @@ class RuleTester {
 		 * @private
 		 */
 		function testInvalidTemplate(item) {
+			const { requireMessage = false, requireLocation = false } =
+				test.assertionOptions ?? {};
+
 			const ruleHasMetaMessages =
 				hasOwnProperty(rule, "meta") &&
 				hasOwnProperty(rule.meta, "messages");
@@ -1152,6 +1229,11 @@ class RuleTester {
 						.join(", ")}]`
 				: null;
 
+			assert.ok(
+				ruleHasMetaMessages || requireMessage !== "messageId",
+				`Assertion options can not use 'requireMessage: "messageId"' if rule under test doesn't define 'meta.messages'.`,
+			);
+
 			const result = runRuleForItem(item);
 			const messages = result.messages;
 
@@ -1225,22 +1307,7 @@ class RuleTester {
 						 * This may have a message, messageId, data, line, and/or column.
 						 */
 
-						Object.keys(error).forEach(propertyName => {
-							assert.ok(
-								errorObjectParameters.has(propertyName),
-								`Invalid error property name '${propertyName}'. Expected one of ${friendlyErrorObjectParameterList}.`,
-							);
-						});
-
 						if (hasOwnProperty(error, "message")) {
-							assert.ok(
-								!hasOwnProperty(error, "messageId"),
-								"Error should not specify both 'message' and a 'messageId'.",
-							);
-							assert.ok(
-								!hasOwnProperty(error, "data"),
-								"Error should not specify both 'data' and 'message'.",
-							);
 							assertMessageMatches(
 								message.message,
 								error.message,
@@ -1298,33 +1365,34 @@ class RuleTester {
 									`Hydrated message "${rehydratedMessage}" does not match "${message.message}"`,
 								);
 							}
-						} else {
-							assert.fail(
-								"Test error must specify either a 'messageId' or 'message'.",
-							);
 						}
 
+						const locationProperties = [
+							"line",
+							"column",
+							"endLine",
+							"endColumn",
+						];
 						const actualLocation = {};
 						const expectedLocation = {};
 
-						if (hasOwnProperty(error, "line")) {
-							actualLocation.line = message.line;
-							expectedLocation.line = error.line;
-						}
-
-						if (hasOwnProperty(error, "column")) {
-							actualLocation.column = message.column;
-							expectedLocation.column = error.column;
-						}
-
-						if (hasOwnProperty(error, "endLine")) {
-							actualLocation.endLine = message.endLine;
-							expectedLocation.endLine = error.endLine;
+						for (const key of locationProperties) {
+							if (hasOwnProperty(error, key)) {
+								actualLocation[key] = message[key];
+								expectedLocation[key] = error[key];
+							}
 						}
 
-						if (hasOwnProperty(error, "endColumn")) {
-							actualLocation.endColumn = message.endColumn;
-							expectedLocation.endColumn = error.endColumn;
+						if (requireLocation) {
+							const missingKeys = locationProperties.filter(
+								key =>
+									!hasOwnProperty(error, key) &&
+									hasOwnProperty(message, key),
+							);
+							assert.ok(
+								missingKeys.length === 0,
+								`Error is missing expected location properties: ${missingKeys.join(", ")}`,
+							);
 						}
 
 						if (Object.keys(expectedLocation).length > 0) {
@@ -1550,11 +1618,6 @@ class RuleTester {
 								}
 							}
 						}
-					} else {
-						// Message was an unexpected type
-						assert.fail(
-							`Error should be a string, object, or RegExp, but found (${util.inspect(message)})`,
-						);
 					}
 				}
 			}
@@ -1631,6 +1694,7 @@ class RuleTester {
 										item,
 										seenTestCases,
 										ruleName,
+										test.assertionOptions,
 									);
 									testInvalidTemplate(item);
 								} finally {

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

@@ -10,6 +10,30 @@
 
 const astUtils = require("./utils/ast-utils");
 
+//------------------------------------------------------------------------------
+// Helpers
+//------------------------------------------------------------------------------
+
+/**
+ * Format import names for error messages.
+ * @param {string[]} importNames The import names to format.
+ * @returns {string} The formatted import names.
+ */
+function formatImportNames(importNames) {
+	return new Intl.ListFormat("en-US").format(
+		importNames.map(name => `'${name}'`),
+	);
+}
+
+/**
+ * Returns "is" or "are" based on the number of import names.
+ * @param {string[]} importNames The import names to check.
+ * @returns {string} "is" if one import name, otherwise "are".
+ */
+function isOrAre(importNames) {
+	return importNames.length === 1 ? "is" : "are";
+}
+
 //------------------------------------------------------------------------------
 // Rule Definition
 //------------------------------------------------------------------------------
@@ -175,22 +199,22 @@ module.exports = {
 				"'{{importName}}' import from '{{importSource}}' is restricted from being used by a pattern. {{customMessage}}",
 
 			patternAndEverything:
-				"* import is invalid because '{{importNames}}' from '{{importSource}}' is restricted from being used by a pattern.",
+				"* import is invalid because {{importNames}} from '{{importSource}}' {{isOrAre}} restricted from being used by a pattern.",
 
 			patternAndEverythingWithRegexImportName:
 				"* import is invalid because import name matching '{{importNames}}' pattern from '{{importSource}}' is restricted from being used.",
 			patternAndEverythingWithCustomMessage:
 				// eslint-disable-next-line eslint-plugin/report-message-format -- Custom message might not end in a period
-				"* import is invalid because '{{importNames}}' from '{{importSource}}' is restricted from being used by a pattern. {{customMessage}}",
+				"* import is invalid because {{importNames}} from '{{importSource}}' {{isOrAre}} restricted from being used by a pattern. {{customMessage}}",
 			patternAndEverythingWithRegexImportNameAndCustomMessage:
 				// eslint-disable-next-line eslint-plugin/report-message-format -- Custom message might not end in a period
 				"* import is invalid because import name matching '{{importNames}}' pattern from '{{importSource}}' is restricted from being used. {{customMessage}}",
 
 			everything:
-				"* import is invalid because '{{importNames}}' from '{{importSource}}' is restricted.",
+				"* import is invalid because {{importNames}} from '{{importSource}}' {{isOrAre}} restricted.",
 			everythingWithCustomMessage:
 				// eslint-disable-next-line eslint-plugin/report-message-format -- Custom message might not end in a period
-				"* import is invalid because '{{importNames}}' from '{{importSource}}' is restricted. {{customMessage}}",
+				"* import is invalid because {{importNames}} from '{{importSource}}' {{isOrAre}} restricted. {{customMessage}}",
 
 			importName:
 				"'{{importName}}' import from '{{importSource}}' is restricted.",
@@ -199,16 +223,16 @@ module.exports = {
 				"'{{importName}}' import from '{{importSource}}' is restricted. {{customMessage}}",
 
 			allowedImportName:
-				"'{{importName}}' import from '{{importSource}}' is restricted because only '{{allowedImportNames}}' import(s) is/are allowed.",
+				"'{{importName}}' import from '{{importSource}}' is restricted because only {{allowedImportNames}} {{isOrAre}} allowed.",
 			allowedImportNameWithCustomMessage:
 				// eslint-disable-next-line eslint-plugin/report-message-format -- Custom message might not end in a period
-				"'{{importName}}' import from '{{importSource}}' is restricted because only '{{allowedImportNames}}' import(s) is/are allowed. {{customMessage}}",
+				"'{{importName}}' import from '{{importSource}}' is restricted because only {{allowedImportNames}} {{isOrAre}} allowed. {{customMessage}}",
 
 			everythingWithAllowImportNames:
-				"* import is invalid because only '{{allowedImportNames}}' from '{{importSource}}' is/are allowed.",
+				"* import is invalid because only {{allowedImportNames}} from '{{importSource}}' {{isOrAre}} allowed.",
 			everythingWithAllowImportNamesAndCustomMessage:
 				// eslint-disable-next-line eslint-plugin/report-message-format -- Custom message might not end in a period
-				"* import is invalid because only '{{allowedImportNames}}' from '{{importSource}}' is/are allowed. {{customMessage}}",
+				"* import is invalid because only {{allowedImportNames}} from '{{importSource}}' {{isOrAre}} allowed. {{customMessage}}",
 
 			allowedImportNamePattern:
 				"'{{importName}}' import from '{{importSource}}' is restricted because only imports that match the pattern '{{allowedImportNamePattern}}' are allowed from '{{importSource}}'.",
@@ -452,7 +476,10 @@ module.exports = {
 									loc: specifier.loc,
 									data: {
 										importSource,
-										importNames: restrictedImportNames,
+										importNames: formatImportNames(
+											restrictedImportNames,
+										),
+										isOrAre: isOrAre(restrictedImportNames),
 										customMessage,
 									},
 								});
@@ -465,7 +492,11 @@ module.exports = {
 									loc: specifier.loc,
 									data: {
 										importSource,
-										allowedImportNames,
+										allowedImportNames:
+											formatImportNames(
+												allowedImportNames,
+											),
+										isOrAre: isOrAre(allowedImportNames),
 										customMessage,
 									},
 								});
@@ -525,7 +556,11 @@ module.exports = {
 										importSource,
 										customMessage,
 										importName,
-										allowedImportNames,
+										allowedImportNames:
+											formatImportNames(
+												allowedImportNames,
+											),
+										isOrAre: isOrAre(allowedImportNames),
 									},
 								});
 							});
@@ -612,7 +647,10 @@ module.exports = {
 							loc: specifier.loc,
 							data: {
 								importSource,
-								importNames: restrictedImportNames,
+								importNames: formatImportNames(
+									restrictedImportNames,
+								),
+								isOrAre: isOrAre(restrictedImportNames),
 								customMessage,
 							},
 						});
@@ -625,7 +663,9 @@ module.exports = {
 							loc: specifier.loc,
 							data: {
 								importSource,
-								allowedImportNames,
+								allowedImportNames:
+									formatImportNames(allowedImportNames),
+								isOrAre: isOrAre(allowedImportNames),
 								customMessage,
 							},
 						});
@@ -713,7 +753,9 @@ module.exports = {
 								importSource,
 								customMessage,
 								importName,
-								allowedImportNames,
+								allowedImportNames:
+									formatImportNames(allowedImportNames),
+								isOrAre: isOrAre(allowedImportNames),
 							},
 						});
 					});

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

@@ -336,9 +336,12 @@ module.exports = {
 				}
 
 				if (
-					targetAssignment.variable.references.some(
-						ref => ref.identifier.type !== "Identifier",
-					)
+					targetAssignment.variable.references.some(ref => {
+						const type = ref.identifier.type;
+						return (
+							type !== "Identifier" && type !== "JSXIdentifier"
+						);
+					})
 				) {
 					/**
 					 * Skip checking for a variable that has at least one non-identifier reference.
@@ -529,7 +532,7 @@ module.exports = {
 			TryStatement(node) {
 				scopeStack.tryStatementBlocks.push(node.block);
 			},
-			Identifier(node) {
+			"Identifier, JSXIdentifier"(node) {
 				for (const segment of scopeStack.currentSegments) {
 					const segmentInfo = scopeStack.segments[segment.id];
 
@@ -539,7 +542,7 @@ module.exports = {
 					segmentInfo.last = node;
 				}
 			},
-			":matches(VariableDeclarator[init!=null], AssignmentExpression, UpdateExpression):exit"(
+			"VariableDeclarator[init!=null], AssignmentExpression, UpdateExpression:exit"(
 				node,
 			) {
 				if (scopeStack.currentSegments.size === 0) {

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

@@ -704,60 +704,63 @@ function isKeywordToken(token) {
 }
 
 /**
- * Check to see if its a ES6 export declaration.
- * @param {ASTNode} astNode An AST node.
- * @returns {boolean} whether the given node represents an export declaration.
+ * Checks whether the given node represents an ES6 export declaration.
+ * @param {ASTNode} node A node to check.
+ * @returns {boolean} `true` if the node is an export declaration.
  * @private
  */
-function looksLikeExport(astNode) {
+function isExportDeclaration(node) {
 	return (
-		astNode.type === "ExportDefaultDeclaration" ||
-		astNode.type === "ExportNamedDeclaration" ||
-		astNode.type === "ExportAllDeclaration" ||
-		astNode.type === "ExportSpecifier"
+		node.type === "ExportDefaultDeclaration" ||
+		node.type === "ExportNamedDeclaration" ||
+		node.type === "ExportAllDeclaration"
 	);
 }
 
 /**
- * Retrieves the JSDoc comment for a given node.
- * @param {ASTNode} node The AST node to get the comment for.
+ * Checks for the presence of a JSDoc comment for the given node and returns it.
+ * @param {ASTNode} node The node to get the comment for.
  * @param {SourceCode} sourceCode A SourceCode instance to get comments.
  * @returns {Token|null} The Block comment token containing the JSDoc comment for the given node or null if not found.
  * @private
  */
-function getJSDocComment(node, sourceCode) {
-	/**
-	 * Checks for the presence of a JSDoc comment for the given node and returns it.
-	 * @param {ASTNode} astNode The AST node to get the comment for.
-	 * @returns {Token|null} The Block comment token containing the JSDoc comment for the given node or null if not found.
-	 * @private
-	 */
-	function findJSDocComment(astNode) {
-		const tokenBefore = sourceCode.getTokenBefore(astNode, {
-			includeComments: true,
-		});
+function findJSDocComment(node, sourceCode) {
+	const tokenBefore = sourceCode.getTokenBefore(node, {
+		includeComments: true,
+	});
 
-		if (
-			tokenBefore &&
-			isCommentToken(tokenBefore) &&
-			tokenBefore.type === "Block" &&
-			tokenBefore.value.charAt(0) === "*" &&
-			astNode.loc.start.line - tokenBefore.loc.end.line <= 1
-		) {
-			return tokenBefore;
-		}
-
-		return null;
+	if (
+		tokenBefore &&
+		tokenBefore.type === "Block" &&
+		tokenBefore.value.charAt(0) === "*" &&
+		node.loc.start.line - tokenBefore.loc.end.line <= 1
+	) {
+		return tokenBefore;
 	}
+
+	return null;
+}
+
+/**
+ * Retrieves the JSDoc comment for a given node.
+ * @param {ASTNode} node The node to get the comment for.
+ * @param {SourceCode} sourceCode A SourceCode instance to get comments.
+ * @returns {Token|null} The Block comment token containing the JSDoc comment for the given node or null if not found.
+ * @private
+ */
+function getJSDocComment(node, sourceCode) {
 	let parent = node.parent;
 
 	switch (node.type) {
 		case "ClassDeclaration":
 		case "FunctionDeclaration":
-			return findJSDocComment(looksLikeExport(parent) ? parent : node);
+			return findJSDocComment(
+				isExportDeclaration(parent) ? parent : node,
+				sourceCode,
+			);
 
 		case "ClassExpression":
-			return findJSDocComment(parent.parent);
+			return findJSDocComment(parent.parent, sourceCode);
 
 		case "ArrowFunctionExpression":
 		case "FunctionExpression":
@@ -783,11 +786,11 @@ function getJSDocComment(node, sourceCode) {
 					parent.type !== "FunctionDeclaration" &&
 					parent.type !== "Program"
 				) {
-					return findJSDocComment(parent);
+					return findJSDocComment(parent, sourceCode);
 				}
 			}
 
-			return findJSDocComment(node);
+			return findJSDocComment(node, sourceCode);
 
 		// falls through
 		default:

package/lib/types/index.d.ts

@@ -115,7 +115,7 @@ export namespace AST {
 		end: ESTree.Position;
 	}
 
-	type Range = [number, number];
+	type Range = SourceRange;
 
 	interface Program extends ESTree.Program {
 		comments: ESTree.Comment[];
@@ -125,6 +125,11 @@ export namespace AST {
 	}
 }
 
+interface JSXIdentifier extends ESTree.BaseNode {
+	type: "JSXIdentifier";
+	name: string;
+}
+
 export namespace Scope {
 	interface ScopeManager {
 		scopes: Scope[];
@@ -177,7 +182,7 @@ export namespace Scope {
 	}
 
 	interface Reference {
-		identifier: ESTree.Identifier;
+		identifier: ESTree.Identifier | JSXIdentifier;
 		from: Scope;
 		resolved: Variable | null;
 		writeExpr: ESTree.Node | null;
@@ -1256,9 +1261,8 @@ export namespace ESLint {
 		foundWarnings: number;
 	}
 
-	interface LintResultData {
+	interface LintResultData extends ResultsMeta {
 		cwd: string;
-		maxWarningsExceeded?: MaxWarningsExceeded | undefined;
 		rulesMeta: {
 			[ruleId: string]: Rule.RuleMetaData;
 		};
@@ -1290,6 +1294,14 @@ export namespace ESLint {
 	 * Metadata about results for formatters.
 	 */
 	interface ResultsMeta {
+		/**
+		 * Whether or not to use color in the formatter output.
+		 * - If `--color` was set, this property is `true`.
+		 * - If `--no-color` was set, it is `false`.
+		 * - If neither option was provided, the property is omitted.
+		 */
+		color?: boolean | undefined;
+
 		/**
 		 * Present if the maxWarnings threshold was exceeded.
 		 */
@@ -1301,9 +1313,9 @@ export namespace ESLint {
 		/**
 		 * Used to call the underlying formatter.
 		 * @param results An array of lint results to format.
-		 * @param resultsMeta An object with an optional `maxWarningsExceeded` property that will be
+		 * @param resultsMeta An object with optional `color` and `maxWarningsExceeded` properties that will be
 		 * passed to the underlying formatter function along with other properties set by ESLint.
-		 * This argument can be omitted if `maxWarningsExceeded` is not needed.
+		 * This argument can be omitted if `color` and `maxWarningsExceeded` are not needed.
 		 * @return The formatter output.
 		 */
 		format(
@@ -1332,9 +1344,10 @@ export namespace ESLint {
 
 // #endregion
 
-export function loadESLint(options?: {
-	useFlatConfig?: boolean | undefined;
-}): Promise<typeof ESLint>;
+/**
+ * Loads the correct `ESLint` constructor.
+ */
+export function loadESLint(): Promise<typeof ESLint>;
 
 // #region RuleTester
 
@@ -1342,6 +1355,9 @@ export class RuleTester {
 	static describe: ((...args: any) => any) | null;
 	static it: ((...args: any) => any) | null;
 	static itOnly: ((...args: any) => any) | null;
+	static setDefaultConfig(config: Linter.Config): void;
+	static getDefaultConfig(): Linter.Config;
+	static resetDefaultConfig(): void;
 
 	constructor(config?: Linter.Config);
 
@@ -1351,6 +1367,26 @@ export class RuleTester {
 		tests: {
 			valid: Array<string | RuleTester.ValidTestCase>;
 			invalid: RuleTester.InvalidTestCase[];
+			/**
+			 * Additional assertions for the "error" matchers of invalid test cases to enforce consistency.
+			 */
+			assertionOptions?: {
+				/**
+				 * If true, each `errors` block must check the expected error
+				 * message, either via a string in the `errors` array, or via
+				 * `message`/`messageId` in an errors object.
+				 * `"message"`/`"messageId"` can be used to further limit the
+				 * message assertions to the respective versions.
+				 */
+				requireMessage?: boolean | "message" | "messageId";
+				/**
+				 * If true, each `errors` block must be an array of objects,
+				 * that each check all location properties `line`, `column`,
+				 * `endLine`, `endColumn`, the later may be omitted, if the
+				 * error does not contain them.
+				 */
+				requireLocation?: boolean;
+			};
 		},
 	): void;
 
@@ -1360,14 +1396,22 @@ export class RuleTester {
 }
 
 export namespace RuleTester {
-	interface ValidTestCase {
+	interface ValidTestCase
+		extends Omit<
+			Linter.Config,
+			| "name"
+			| "basePath"
+			| "files"
+			| "ignores"
+			| "linterOptions"
+			| "plugins"
+			| "rules"
+		> {
 		name?: string;
 		code: string;
-		options?: any;
+		options?: any[];
 		filename?: string | undefined;
 		only?: boolean;
-		languageOptions?: Linter.LanguageOptions | undefined;
-		settings?: { [name: string]: any } | undefined;
 		before?: () => void;
 		after?: () => void;
 	}
@@ -1380,7 +1424,7 @@ export namespace RuleTester {
 	}
 
 	interface InvalidTestCase extends ValidTestCase {
-		errors: number | Array<TestCaseError | string>;
+		errors: number | Array<TestCaseError | string | RegExp>;
 		output?: string | null | undefined;
 	}
 
@@ -1392,7 +1436,7 @@ export namespace RuleTester {
 		column?: number | undefined;
 		endLine?: number | undefined;
 		endColumn?: number | undefined;
-		suggestions?: SuggestionOutput[] | undefined;
+		suggestions?: SuggestionOutput[] | number | undefined;
 	}
 }
 

package/lib/types/use-at-your-own-risk.d.ts

@@ -31,4 +31,4 @@ import { Rule } from "./index.js";
 export const builtinRules: Map<string, Rule.RuleModule>;
 
 /** @deprecated */
-export function shouldUseFlatConfig(): Promise<boolean>;
+export function shouldUseFlatConfig(): Promise<true>;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint",
-  "version": "10.0.0-alpha.1",
+  "version": "10.0.0-beta.0",
   "author": "Nicholas C. Zakas <nicholas+npm@nczconsulting.com>",
   "description": "An AST-based pattern checker for JavaScript.",
   "type": "commonjs",
@@ -107,7 +107,7 @@
   "bugs": "https://github.com/eslint/eslint/issues/",
   "dependencies": {
     "@eslint-community/eslint-utils": "^4.8.0",
-    "@eslint-community/regexpp": "^4.12.1",
+    "@eslint-community/regexpp": "^4.12.2",
     "@eslint/config-array": "^0.23.0",
     "@eslint/config-helpers": "^0.5.0",
     "@eslint/core": "^1.0.0",
@@ -117,7 +117,6 @@
     "@humanwhocodes/retry": "^0.4.2",
     "@types/estree": "^1.0.6",
     "ajv": "^6.12.4",
-    "chalk": "^4.0.0",
     "cross-spawn": "^7.0.6",
     "debug": "^4.3.2",
     "escape-string-regexp": "^4.0.0",
@@ -144,7 +143,7 @@
     "@babel/preset-env": "^7.4.3",
     "@cypress/webpack-preprocessor": "^6.0.2",
     "@eslint/json": "^0.14.0",
-    "@eslint/eslintrc": "^3.3.1",
+    "@eslint/eslintrc": "^3.3.3",
     "@trunkio/launcher": "^1.3.4",
     "@types/esquery": "^1.5.4",
     "@types/node": "^22.13.14",