eslint 9.25.1 → 9.27.0

package/README.md

@@ -20,9 +20,9 @@
 
 ESLint is a tool for identifying and reporting on patterns found in ECMAScript/JavaScript code. In many ways, it is similar to JSLint and JSHint with a few exceptions:
 
--   ESLint uses [Espree](https://github.com/eslint/js/tree/main/packages/espree) for JavaScript parsing.
--   ESLint uses an AST to evaluate patterns in code.
--   ESLint is completely pluggable, every single rule is a plugin and you can add more at runtime.
+- ESLint uses [Espree](https://github.com/eslint/js/tree/main/packages/espree) for JavaScript parsing.
+- ESLint uses an AST to evaluate patterns in code.
+- ESLint is completely pluggable, every single rule is a plugin and you can add more at runtime.
 
 ## Table of Contents
 
@@ -87,9 +87,9 @@ export default defineConfig([
 
 The names `"prefer-const"` and `"no-constant-binary-expression"` are the names of [rules](https://eslint.org/docs/rules) in ESLint. The first value is the error level of the rule and can be one of these values:
 
--   `"off"` or `0` - turn the rule off
--   `"warn"` or `1` - turn the rule on as a warning (doesn't affect exit code)
--   `"error"` or `2` - turn the rule on as an error (exit code will be 1)
+- `"off"` or `0` - turn the rule off
+- `"warn"` or `1` - turn the rule on as a warning (doesn't affect exit code)
+- `"error"` or `2` - turn the rule on as an error (exit code will be 1)
 
 The three error levels allow you fine-grained control over how ESLint applies rules (for more configuration options and details, see the [configuration docs](https://eslint.org/docs/latest/use/configure)).
 
@@ -109,10 +109,10 @@ ESLint adheres to the [OpenJS Foundation Code of Conduct](https://eslint.org/con
 
 Before filing an issue, please be sure to read the guidelines for what you're reporting:
 
--   [Bug Report](https://eslint.org/docs/latest/contribute/report-bugs)
--   [Propose a New Rule](https://eslint.org/docs/latest/contribute/propose-new-rule)
--   [Proposing a Rule Change](https://eslint.org/docs/latest/contribute/propose-rule-change)
--   [Request a Change](https://eslint.org/docs/latest/contribute/request-change)
+- [Bug Report](https://eslint.org/docs/latest/contribute/report-bugs)
+- [Propose a New Rule](https://eslint.org/docs/latest/contribute/propose-new-rule)
+- [Proposing a Rule Change](https://eslint.org/docs/latest/contribute/propose-rule-change)
+- [Request a Change](https://eslint.org/docs/latest/contribute/request-change)
 
 ## Frequently Asked Questions
 
@@ -174,32 +174,32 @@ ESLint takes security seriously. We work hard to ensure that ESLint is safe for
 
 ESLint follows [semantic versioning](https://semver.org). However, due to the nature of ESLint as a code quality tool, it's not always clear when a minor or major version bump occurs. To help clarify this for everyone, we've defined the following semantic versioning policy for ESLint:
 
--   Patch release (intended to not break your lint build)
-    -   A bug fix in a rule that results in ESLint reporting fewer linting errors.
-    -   A bug fix to the CLI or core (including formatters).
-    -   Improvements to documentation.
-    -   Non-user-facing changes such as refactoring code, adding, deleting, or modifying tests, and increasing test coverage.
-    -   Re-releasing after a failed release (i.e., publishing a release that doesn't work for anyone).
--   Minor release (might break your lint build)
-    -   A bug fix in a rule that results in ESLint reporting more linting errors.
-    -   A new rule is created.
-    -   A new option to an existing rule that does not result in ESLint reporting more linting errors by default.
-    -   A new addition to an existing rule to support a newly-added language feature (within the last 12 months) that will result in ESLint reporting more linting errors by default.
-    -   An existing rule is deprecated.
-    -   A new CLI capability is created.
-    -   New capabilities to the public API are added (new classes, new methods, new arguments to existing methods, etc.).
-    -   A new formatter is created.
-    -   `eslint:recommended` is updated and will result in strictly fewer linting errors (e.g., rule removals).
--   Major release (likely to break your lint build)
-    -   `eslint:recommended` is updated and may result in new linting errors (e.g., rule additions, most rule option updates).
-    -   A new option to an existing rule that results in ESLint reporting more linting errors by default.
-    -   An existing formatter is removed.
-    -   Part of the public API is removed or changed in an incompatible way. The public API includes:
-        -   Rule schemas
-        -   Configuration schema
-        -   Command-line options
-        -   Node.js API
-        -   Rule, formatter, parser, plugin APIs
+- Patch release (intended to not break your lint build)
+    - A bug fix in a rule that results in ESLint reporting fewer linting errors.
+    - A bug fix to the CLI or core (including formatters).
+    - Improvements to documentation.
+    - Non-user-facing changes such as refactoring code, adding, deleting, or modifying tests, and increasing test coverage.
+    - Re-releasing after a failed release (i.e., publishing a release that doesn't work for anyone).
+- Minor release (might break your lint build)
+    - A bug fix in a rule that results in ESLint reporting more linting errors.
+    - A new rule is created.
+    - A new option to an existing rule that does not result in ESLint reporting more linting errors by default.
+    - A new addition to an existing rule to support a newly-added language feature (within the last 12 months) that will result in ESLint reporting more linting errors by default.
+    - An existing rule is deprecated.
+    - A new CLI capability is created.
+    - New capabilities to the public API are added (new classes, new methods, new arguments to existing methods, etc.).
+    - A new formatter is created.
+    - `eslint:recommended` is updated and will result in strictly fewer linting errors (e.g., rule removals).
+- Major release (likely to break your lint build)
+    - `eslint:recommended` is updated and may result in new linting errors (e.g., rule additions, most rule option updates).
+    - A new option to an existing rule that results in ESLint reporting more linting errors by default.
+    - An existing formatter is removed.
+    - Part of the public API is removed or changed in an incompatible way. The public API includes:
+        - Rule schemas
+        - Configuration schema
+        - Command-line options
+        - Node.js API
+        - Rule, formatter, parser, plugin APIs
 
 According to our policy, any minor update may report more linting errors than the previous release (ex: from a bug fix). As such, we recommend using the tilde (`~`) in `package.json` e.g. `"eslint": "~3.1.0"` to guarantee the results of your builds.
 
@@ -286,6 +286,11 @@ Josh Goldberg ✨
 <img src="https://github.com/Tanujkanti4441.png?s=75" width="75" height="75" alt="Tanuj Kanti's Avatar"><br />
 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
+</a>
 </td></tr></tbody></table>
 
 ### Website Team
@@ -299,8 +304,8 @@ Amaresh  S M
 </a>
 </td><td align="center" valign="top" width="11%">
 <a href="https://github.com/harish-sethuraman">
-<img src="https://github.com/harish-sethuraman.png?s=75" width="75" height="75" alt="Strek's Avatar"><br />
-Strek
+<img src="https://github.com/harish-sethuraman.png?s=75" width="75" height="75" alt="Harish's Avatar"><br />
+Harish
 </a>
 </td><td align="center" valign="top" width="11%">
 <a href="https://github.com/kecrily">
@@ -320,7 +325,7 @@ The following companies, organizations, and individuals support ESLint's ongoing
 to get your logo on our READMEs and [website](https://eslint.org/sponsors).
 
 <h3>Diamond Sponsors</h3>
-<p><a href="https://www.ag-grid.com/"><img src="https://images.opencollective.com/ag-grid/2c8d545/logo.png" alt="AG Grid" height="128"></a></p><h3>Platinum Sponsors</h3>
+<p><a href="https://www.ag-grid.com/"><img src="https://images.opencollective.com/ag-grid/bec0580/logo.png" alt="AG Grid" height="128"></a></p><h3>Platinum Sponsors</h3>
 <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://trunk.io/"><img src="https://images.opencollective.com/trunkio/fb92d60/avatar.png" alt="trunk.io" 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/5c4fa84/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>

package/bin/eslint.js

@@ -155,6 +155,21 @@ ${getErrorMessage(error)}`;
 		return;
 	}
 
+	// start the MCP server if `--mcp` is present
+	if (process.argv.includes("--mcp")) {
+		console.warn(
+			"You can also run this command directly using 'npx @eslint/mcp@latest'.",
+		);
+
+		const spawn = require("cross-spawn");
+
+		spawn.sync("npx", ["@eslint/mcp@latest"], {
+			encoding: "utf8",
+			stdio: "inherit",
+		});
+		return;
+	}
+
 	// Otherwise, call the CLI.
 	const cli = require("../lib/cli");
 	const exitCode = await cli.execute(

package/conf/rule-type-list.json

@@ -69,7 +69,8 @@
       "removed": "space-in-brackets",
       "replacedBy": [
         { "rule": { "name": "object-curly-spacing" } },
-        { "rule": { "name": "array-bracket-spacing" } }
+        { "rule": { "name": "array-bracket-spacing" } },
+        { "rule": { "name": "computed-property-spacing" } }
       ]
     },
     {

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

@@ -58,15 +58,15 @@ const validFixTypes = new Set(["directive", "problem", "suggestion", "layout"]);
 //------------------------------------------------------------------------------
 
 // For VSCode IntelliSense
-/** @typedef {import("../shared/types").ConfigData} ConfigData */
-/** @typedef {import("../shared/types").DeprecatedRuleInfo} DeprecatedRuleInfo */
-/** @typedef {import("../shared/types").LintMessage} LintMessage */
-/** @typedef {import("../shared/types").SuppressedLintMessage} SuppressedLintMessage */
-/** @typedef {import("../shared/types").ParserOptions} ParserOptions */
-/** @typedef {import("../shared/types").Plugin} Plugin */
-/** @typedef {import("../shared/types").RuleConf} RuleConf */
-/** @typedef {import("../types").Rule.RuleModule} Rule */
+/** @typedef {import("../types").ESLint.ConfigData} ConfigData */
+/** @typedef {import("../types").ESLint.DeprecatedRuleUse} DeprecatedRuleInfo */
 /** @typedef {import("../types").ESLint.FormatterFunction} FormatterFunction */
+/** @typedef {import("../types").Linter.LintMessage} LintMessage */
+/** @typedef {import("../types").Linter.ParserOptions} ParserOptions */
+/** @typedef {import("../types").ESLint.Plugin} Plugin */
+/** @typedef {import("../types").Rule.RuleModule} Rule */
+/** @typedef {import("../types").Linter.RuleEntry} RuleConf */
+/** @typedef {import("../types").Linter.SuppressedLintMessage} SuppressedLintMessage */
 /** @typedef {ReturnType<CascadingConfigArrayFactory.getConfigArrayForFile>} ConfigArray */
 /** @typedef {ReturnType<ConfigArray.extractConfig>} ExtractedConfig */
 

package/lib/cli.js

@@ -40,12 +40,13 @@ const debug = require("debug")("eslint:cli");
 // Types
 //------------------------------------------------------------------------------
 
-/** @typedef {import("./eslint/eslint").ESLintOptions} ESLintOptions */
-/** @typedef {import("./eslint/eslint").LintMessage} LintMessage */
-/** @typedef {import("./eslint/eslint").LintResult} LintResult */
+/** @import { ESLintOptions } from "./eslint/eslint.js" */
+
 /** @typedef {import("./options").ParsedCLIOptions} ParsedCLIOptions */
-/** @typedef {import("./shared/types").Plugin} Plugin */
-/** @typedef {import("./shared/types").ResultsMeta} ResultsMeta */
+/** @typedef {import("./types").Linter.LintMessage} LintMessage */
+/** @typedef {import("./types").ESLint.LintResult} LintResult */
+/** @typedef {import("./types").ESLint.Plugin} Plugin */
+/** @typedef {import("./types").ESLint.ResultsMeta} ResultsMeta */
 
 //------------------------------------------------------------------------------
 // Helpers

package/lib/config/config-loader.js

@@ -20,19 +20,17 @@ const { FlatConfigArray } = require("./flat-config-array");
 // Types
 //-----------------------------------------------------------------------------
 
-/**
- * @import { ConfigData, ConfigData as FlatConfigObject } from "../shared/types.js";
- */
+/** @typedef {import("../types").Linter.Config} Config */
 
 /**
  * @typedef {Object} ConfigLoaderOptions
  * @property {string|false|undefined} configFile The path to the config file to use.
  * @property {string} cwd The current working directory.
  * @property {boolean} ignoreEnabled Indicates if ignore patterns should be honored.
- * @property {FlatConfigArray} [baseConfig] The base config to use.
- * @property {Array<FlatConfigObject>} [defaultConfigs] The default configs to use.
+ * @property {Config|Array<Config>} [baseConfig] The base config to use.
+ * @property {Array<Config>} [defaultConfigs] The default configs to use.
  * @property {Array<string>} [ignorePatterns] The ignore patterns to use.
- * @property {FlatConfigObject|Array<FlatConfigObject>} [overrideConfig] The override config to use.
+ * @property {Config|Array<Config>} [overrideConfig] The override config to use.
  * @property {boolean} [hasUnstableNativeNodeJsTSConfigFlag] The flag to indicate whether the `unstable_native_nodejs_ts_config` flag is enabled.
  */
 
@@ -394,8 +392,7 @@ class ConfigLoader {
 	 * This is the same logic used by the ESLint CLI executable to determine
 	 * configuration for each file it processes.
 	 * @param {string} filePath The path of the file or directory to retrieve config for.
-	 * @returns {Promise<ConfigData|undefined>} A configuration object for the file
-	 *      or `undefined` if there is no configuration data for the file.
+	 * @returns {Promise<FlatConfigArray>} A configuration object for the file.
 	 * @throws {Error} If no configuration for `filePath` exists.
 	 */
 	async loadConfigArrayForFile(filePath) {
@@ -415,8 +412,7 @@ class ConfigLoader {
 	 * This is the same logic used by the ESLint CLI executable to determine
 	 * configuration for each file it processes.
 	 * @param {string} dirPath The path of the directory to retrieve config for.
-	 * @returns {Promise<ConfigData|undefined>} A configuration object for the directory
-	 *      or `undefined` if there is no configuration data for the directory.
+	 * @returns {Promise<FlatConfigArray>} A configuration object for the directory.
 	 */
 	async loadConfigArrayForDirectory(dirPath) {
 		assertValidFilePath(dirPath);
@@ -440,8 +436,7 @@ class ConfigLoader {
 	 * intended to be used in locations where we know the config file has already
 	 * been loaded and we just need to get the configuration for a file.
 	 * @param {string} filePath The path of the file to retrieve a config object for.
-	 * @returns {ConfigData|undefined} A configuration object for the file
-	 *     or `undefined` if there is no configuration data for the file.
+	 * @returns {FlatConfigArray} A configuration object for the file.
 	 * @throws {Error} If `filePath` is not a non-empty string.
 	 * @throws {Error} If `filePath` is not an absolute path.
 	 * @throws {Error} If the config file was not already loaded.
@@ -460,8 +455,7 @@ class ConfigLoader {
 	 * intended to be used in locations where we know the config file has already
 	 * been loaded and we just need to get the configuration for a file.
 	 * @param {string} fileOrDirPath The path of the directory to retrieve a config object for.
-	 * @returns {ConfigData|undefined} A configuration object for the directory
-	 *     or `undefined` if there is no configuration data for the directory.
+	 * @returns {FlatConfigArray} A configuration object for the directory.
 	 * @throws {Error} If `dirPath` is not a non-empty string.
 	 * @throws {Error} If `dirPath` is not an absolute path.
 	 * @throws {Error} If the config file was not already loaded.
@@ -789,8 +783,7 @@ class LegacyConfigLoader extends ConfigLoader {
 	 * This is the same logic used by the ESLint CLI executable to determine
 	 * configuration for each file it processes.
 	 * @param {string} dirPath The path of the directory to retrieve config for.
-	 * @returns {Promise<ConfigData|undefined>} A configuration object for the file
-	 *      or `undefined` if there is no configuration data for the file.
+	 * @returns {Promise<FlatConfigArray>} A configuration object for the file.
 	 */
 	async loadConfigArrayForDirectory(dirPath) {
 		assertValidFilePath(dirPath);
@@ -812,8 +805,7 @@ class LegacyConfigLoader extends ConfigLoader {
 	 * intended to be used in locations where we know the config file has already
 	 * been loaded and we just need to get the configuration for a file.
 	 * @param {string} dirPath The path of the directory to retrieve a config object for.
-	 * @returns {ConfigData|undefined} A configuration object for the file
-	 *     or `undefined` if there is no configuration data for the file.
+	 * @returns {FlatConfigArray} A configuration object for the file.
 	 * @throws {Error} If `dirPath` is not a non-empty string.
 	 * @throws {Error} If `dirPath` is not an absolute path.
 	 * @throws {Error} If the config file was not already loaded.