eslint 9.19.0 → 9.20.1

package/README.md

@@ -35,7 +35,6 @@ ESLint is a tool for identifying and reporting on patterns found in ECMAScript/J
 1. [Releases](#releases)
 1. [Security Policy](#security-policy)
 1. [Semantic Versioning Policy](#semantic-versioning-policy)
-1. [Stylistic Rule Updates](#stylistic-rule-updates)
 1. [License](#license)
 1. [Team](#team)
 1. [Sponsors](#sponsors)
@@ -57,6 +56,17 @@ After that, you can run ESLint on any file or directory like this:
 npx eslint yourfile.js
 ```
 
+### pnpm Installation
+
+To use ESLint with pnpm, we recommend setting up a `.npmrc` file with at least the following settings:
+
+```text
+auto-install-peers=true
+node-linker=hoisted
+```
+
+This ensures that pnpm installs dependencies in a way that is more compatible with npm and is less likely to produce errors.
+
 ## Configuration
 
 You can configure rules in your `eslint.config.js` files as in this example:
@@ -191,15 +201,6 @@ ESLint follows [semantic versioning](https://semver.org). However, due to the na
 
 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.
 
-## Stylistic Rule Updates
-
-Stylistic rules are frozen according to [our policy](https://eslint.org/blog/2020/05/changes-to-rules-policies) on how we evaluate new rules and rule changes.
-This means:
-
-* **Bug fixes**: We will still fix bugs in stylistic rules.
-* **New ECMAScript features**: We will also make sure stylistic rules are compatible with new ECMAScript features.
-* **New options**: We will **not** add any new options to stylistic rules unless an option is the only way to fix a bug or support a newly-added ECMAScript feature.
-
 ## License
 
 MIT License
@@ -318,7 +319,7 @@ to get your logo on our READMEs and [website](https://eslint.org/sponsors).
 <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></p><h3>Silver Sponsors</h3>
-<p><a href="https://www.serptriumph.com/"><img src="https://images.opencollective.com/serp-triumph5/fea3074/logo.png" alt="SERP Triumph" height="64"></a> <a href="https://www.jetbrains.com/"><img src="https://images.opencollective.com/jetbrains/fe76f99/logo.png" alt="JetBrains" 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></p><h3>Bronze 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://www.jetbrains.com/"><img src="https://images.opencollective.com/jetbrains/fe76f99/logo.png" alt="JetBrains" 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></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://www.crosswordsolver.org/anagram-solver/"><img src="https://images.opencollective.com/anagram-solver/2666271/logo.png" alt="Anagram Solver" 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://nolebase.ayaka.io"><img src="https://avatars.githubusercontent.com/u/11081491" alt="Neko" 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></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.

package/lib/config/config-loader.js

@@ -499,11 +499,41 @@ class ConfigLoader {
             debug(`Loading config file ${configFilePath}`);
             const fileConfig = await loadConfigFile(configFilePath);
 
-            if (Array.isArray(fileConfig)) {
-                configs.push(...fileConfig);
-            } else {
-                configs.push(fileConfig);
+            /*
+             * It's possible that a config file could be empty or else
+             * have an empty object or array. In this case, we want to
+             * warn the user that they have an empty config.
+             *
+             * An empty CommonJS file exports an empty object while
+             * an empty ESM file exports undefined.
+             */
+
+            let emptyConfig = typeof fileConfig === "undefined";
+
+            debug(`Config file ${configFilePath} is ${emptyConfig ? "empty" : "not empty"}`);
+
+            if (!emptyConfig) {
+                if (Array.isArray(fileConfig)) {
+                    if (fileConfig.length === 0) {
+                        debug(`Config file ${configFilePath} is an empty array`);
+                        emptyConfig = true;
+                    } else {
+                        configs.push(...fileConfig);
+                    }
+                } else {
+                    if (typeof fileConfig === "object" && fileConfig !== null && Object.keys(fileConfig).length === 0) {
+                        debug(`Config file ${configFilePath} is an empty object`);
+                        emptyConfig = true;
+                    } else {
+                        configs.push(fileConfig);
+                    }
+                }
             }
+
+            if (emptyConfig) {
+                globalThis.process?.emitWarning?.(`Running ESLint with an empty config (from ${configFilePath}). Please double-check that this is what you want. If you want to run ESLint with an empty config, export [{}] to remove this warning.`, "ESLintEmptyConfigWarning");
+            }
+
         }
 
         // add in any configured defaults

package/lib/config/config.js

@@ -110,7 +110,12 @@ function languageOptionsToJSON(languageOptions, objectKey = "languageOptions") {
             }
 
             if (typeof value === "function") {
-                throw new TypeError(`Cannot serialize key "${key}" in ${objectKey}: Function values are not supported.`);
+                const error = new TypeError(`Cannot serialize key "${key}" in ${objectKey}: Function values are not supported.`);
+
+                error.messageTemplate = "config-serialize-function";
+                error.messageData = { key, objectKey };
+
+                throw error;
             }
 
         }

package/lib/eslint/eslint.js

@@ -470,7 +470,7 @@ class ESLint {
             defaultConfigs
         };
 
-        this.#configLoader = processedOptions.flags.includes("unstable_config_lookup_from_file")
+        this.#configLoader = linter.hasFlag("unstable_config_lookup_from_file")
             ? new ConfigLoader(configLoaderOptions)
             : new LegacyConfigLoader(configLoaderOptions);
 

package/lib/linter/linter.js

@@ -43,7 +43,7 @@ const { assertIsRuleSeverity } = require("../config/flat-config-schema");
 const { normalizeSeverityToString, normalizeSeverityToNumber } = require("../shared/severity");
 const { deepMergeArrays } = require("../shared/deep-merge-arrays");
 const jslang = require("../languages/js");
-const { activeFlags, inactiveFlags } = require("../shared/flags");
+const { activeFlags, inactiveFlags, getInactivityReasonMessage } = require("../shared/flags");
 const debug = require("debug")("eslint:linter");
 const MAX_AUTOFIX_PASSES = 10;
 const DEFAULT_PARSER_NAME = "espree";
@@ -1326,19 +1326,40 @@ class Linter {
      */
     constructor({ cwd, configType = "flat", flags = [] } = {}) {
 
+        const processedFlags = [];
+
         flags.forEach(flag => {
             if (inactiveFlags.has(flag)) {
-                throw new Error(`The flag '${flag}' is inactive: ${inactiveFlags.get(flag)}`);
+                const inactiveFlagData = inactiveFlags.get(flag);
+                const inactivityReason = getInactivityReasonMessage(inactiveFlagData);
+
+                if (typeof inactiveFlagData.replacedBy === "undefined") {
+                    throw new Error(`The flag '${flag}' is inactive: ${inactivityReason}`);
+                }
+
+                // if there's a replacement, enable it instead of original
+                if (typeof inactiveFlagData.replacedBy === "string") {
+                    processedFlags.push(inactiveFlagData.replacedBy);
+                }
+
+                globalThis.process?.emitWarning?.(
+                    `The flag '${flag}' is inactive: ${inactivityReason}`,
+                    `ESLintInactiveFlag_${flag}`
+                );
+
+                return;
             }
 
             if (!activeFlags.has(flag)) {
                 throw new Error(`Unknown flag '${flag}'.`);
             }
+
+            processedFlags.push(flag);
         });
 
         internalSlotsMap.set(this, {
             cwd: normalizeCwd(cwd),
-            flags,
+            flags: processedFlags,
             lastConfigArray: null,
             lastSourceCode: null,
             lastSuppressedMessages: [],

package/lib/rules/arrow-body-style.js

@@ -143,7 +143,7 @@ module.exports = {
 
                     if (blockBody.length === 0) {
                         messageId = "unexpectedEmptyBlock";
-                    } else if (blockBody.length > 1) {
+                    } else if (blockBody.length > 1 || blockBody[0].type !== "ReturnStatement") {
                         messageId = "unexpectedOtherBlock";
                     } else if (blockBody[0].argument === null) {
                         messageId = "unexpectedSingleBlock";

package/lib/rules/consistent-this.js

@@ -119,8 +119,17 @@ module.exports = {
         function ensureWasAssigned(node) {
             const scope = sourceCode.getScope(node);
 
+            // if this is program scope we also need to check module scope
+            const extraScope = node.type === "Program" && node.sourceType === "module"
+                ? scope.childScopes[0]
+                : null;
+
             aliases.forEach(alias => {
                 checkWasAssigned(alias, scope);
+
+                if (extraScope) {
+                    checkWasAssigned(alias, extraScope);
+                }
             });
         }
 

package/lib/shared/flags.js

@@ -4,6 +4,23 @@
 
 "use strict";
 
+//------------------------------------------------------------------------------
+// Typedefs
+//------------------------------------------------------------------------------
+
+/**
+ * @typedef {Object} InactiveFlagData
+ * @property {string} description Flag description
+ * @property {string | null} [replacedBy] Can be either:
+ *   - An active flag (string) that enables the same feature.
+ *   - `null` if the feature is now enabled by default.
+ *   - Omitted if the feature has been abandoned.
+ */
+
+//-----------------------------------------------------------------------------
+// Exports
+//-----------------------------------------------------------------------------
+
 /**
  * The set of flags that change ESLint behavior with a description.
  * @type {Map<string, string>}
@@ -14,15 +31,36 @@ const activeFlags = new Map([
 ]);
 
 /**
- * The set of flags that used to be active but no longer have an effect.
- * @type {Map<string, string>}
+ * The set of flags that used to be active.
+ * @type {Map<string, InactiveFlagData>}
  */
 const inactiveFlags = new Map([
-    ["test_only_old", "Used only for testing."],
-    ["unstable_ts_config", "This flag is no longer required to enable TypeScript configuration files."]
+    ["test_only_replaced", { description: "Used only for testing flags that have been replaced by other flags.", replacedBy: "test_only" }],
+    ["test_only_enabled_by_default", { description: "Used only for testing flags whose features have been enabled by default.", replacedBy: null }],
+    ["test_only_abandoned", { description: "Used only for testing flags whose features have been abandoned." }],
+    ["unstable_ts_config", { description: "Enable TypeScript configuration files.", replacedBy: null }]
 ]);
 
+/**
+ * Creates a message that describes the reason the flag is inactive.
+ * @param {InactiveFlagData} inactiveFlagData Data for the inactive flag.
+ * @returns {string} Message describing the reason the flag is inactive.
+ */
+function getInactivityReasonMessage({ replacedBy }) {
+    if (typeof replacedBy === "undefined") {
+        return "This feature has been abandoned.";
+    }
+
+    if (typeof replacedBy === "string") {
+        return `This flag has been renamed '${replacedBy}' to reflect its stabilization. Please use '${replacedBy}' instead.`;
+    }
+
+    // null
+    return "This feature is now enabled by default.";
+}
+
 module.exports = {
     activeFlags,
-    inactiveFlags
+    inactiveFlags,
+    getInactivityReasonMessage
 };

package/lib/types/index.d.ts

@@ -26,10 +26,42 @@
  */
 
 import * as ESTree from "estree";
-import { Language } from "@eslint/core";
+import type {
+    RuleVisitor,
+    TextSourceCode,
+    Language,
+    SourceRange,
+    TraversalStep,
+    LanguageOptions as GenericLanguageOptions,
+    RuleDefinition,
+    RuleContext as CoreRuleContext,
+    RuleContextTypeOptions
+} from "@eslint/core";
 import { JSONSchema4 } from "json-schema";
 import { LegacyESLint } from "./use-at-your-own-risk.js";
 
+/*
+ * Need to extend the `RuleContext` interface to include the
+ * deprecated methods that have not yet been removed.
+ * TODO: Remove in v10.0.0.
+ */
+declare module "@eslint/core" {
+    interface RuleContext {
+
+        /** @deprecated Use `sourceCode.getAncestors()` instead */
+        getAncestors(): ESTree.Node[];
+
+        /** @deprecated Use `sourceCode.getDeclaredVariables()` instead */
+        getDeclaredVariables(node: ESTree.Node): Scope.Variable[];
+
+        /** @deprecated Use `sourceCode.getScope()` instead */
+        getScope(): Scope.Scope;
+
+        /** @deprecated Use `sourceCode.markVariableAsUsed()` instead */
+        markVariableAsUsed(name: string): boolean;
+    }
+}
+
 export namespace AST {
     type TokenType =
         | "Boolean"
@@ -149,7 +181,12 @@ export namespace Scope {
 
 // #region SourceCode
 
-export class SourceCode {
+export class SourceCode implements TextSourceCode<{
+    LangOptions: Linter.LanguageOptions;
+    RootNode: AST.Program;
+    SyntaxElementWithLoc: AST.Token | ESTree.Node;
+    ConfigNode: ESTree.Comment;
+}> {
     text: string;
     ast: AST.Program;
     lines: string[];
@@ -163,6 +200,9 @@ export class SourceCode {
 
     static splitLines(text: string): string[];
 
+    getLoc(syntaxElement: AST.Token | ESTree.Node): ESTree.SourceLocation;
+    getRange(syntaxElement: AST.Token | ESTree.Node): SourceRange;
+
     getText(node?: ESTree.Node, beforeCount?: number, afterCount?: number): string;
 
     getLines(): string[];
@@ -238,6 +278,8 @@ export class SourceCode {
     ): boolean;
 
     markVariableAsUsed(name: string, refNode?: ESTree.Node): boolean;
+
+    traverse(): Iterable<TraversalStep>;
 }
 
 export namespace SourceCode {
@@ -507,21 +549,25 @@ export namespace SourceCode {
 // #endregion
 
 export namespace Rule {
-    interface RuleModule {
-        create(context: RuleContext): RuleListener;
-        meta?: RuleMetaData | undefined;
-    }
+
+    type RuleModule = RuleDefinition<{
+        LangOptions: Linter.LanguageOptions,
+        Code: SourceCode,
+        RuleOptions: any[],
+        Visitor: NodeListener,
+        Node: ESTree.Node,
+        MessageIds: string,
+        ExtRuleDocs: {}
+    }>;
 
     type NodeTypes = ESTree.Node["type"];
-    interface NodeListener {
+    interface NodeListener extends RuleVisitor {
         ArrayExpression?: ((node: ESTree.ArrayExpression & NodeParentExtension) => void) | undefined;
         "ArrayExpression:exit"?: ((node: ESTree.ArrayExpression & NodeParentExtension) => void) | undefined;
         ArrayPattern?: ((node: ESTree.ArrayPattern & NodeParentExtension) => void) | undefined;
         "ArrayPattern:exit"?: ((node: ESTree.ArrayPattern & NodeParentExtension) => void) | undefined;
         ArrowFunctionExpression?: ((node: ESTree.ArrowFunctionExpression & NodeParentExtension) => void) | undefined;
-        "ArrowFunctionExpression:exit"?:
-        | ((node: ESTree.ArrowFunctionExpression & NodeParentExtension) => void)
-        | undefined;
+        "ArrowFunctionExpression:exit"?: ((node: ESTree.ArrowFunctionExpression & NodeParentExtension) => void) | undefined;
         AssignmentExpression?: ((node: ESTree.AssignmentExpression & NodeParentExtension) => void) | undefined;
         "AssignmentExpression:exit"?: ((node: ESTree.AssignmentExpression & NodeParentExtension) => void) | undefined;
         AssignmentPattern?: ((node: ESTree.AssignmentPattern & NodeParentExtension) => void) | undefined;
@@ -559,13 +605,9 @@ export namespace Rule {
         ExportAllDeclaration?: ((node: ESTree.ExportAllDeclaration & NodeParentExtension) => void) | undefined;
         "ExportAllDeclaration:exit"?: ((node: ESTree.ExportAllDeclaration & NodeParentExtension) => void) | undefined;
         ExportDefaultDeclaration?: ((node: ESTree.ExportDefaultDeclaration & NodeParentExtension) => void) | undefined;
-        "ExportDefaultDeclaration:exit"?:
-        | ((node: ESTree.ExportDefaultDeclaration & NodeParentExtension) => void)
-        | undefined;
+        "ExportDefaultDeclaration:exit"?: ((node: ESTree.ExportDefaultDeclaration & NodeParentExtension) => void) | undefined;
         ExportNamedDeclaration?: ((node: ESTree.ExportNamedDeclaration & NodeParentExtension) => void) | undefined;
-        "ExportNamedDeclaration:exit"?:
-        | ((node: ESTree.ExportNamedDeclaration & NodeParentExtension) => void)
-        | undefined;
+        "ExportNamedDeclaration:exit"?: ((node: ESTree.ExportNamedDeclaration & NodeParentExtension) => void) | undefined;
         ExportSpecifier?: ((node: ESTree.ExportSpecifier & NodeParentExtension) => void) | undefined;
         "ExportSpecifier:exit"?: ((node: ESTree.ExportSpecifier & NodeParentExtension) => void) | undefined;
         ExpressionStatement?: ((node: ESTree.ExpressionStatement & NodeParentExtension) => void) | undefined;
@@ -587,15 +629,11 @@ export namespace Rule {
         ImportDeclaration?: ((node: ESTree.ImportDeclaration & NodeParentExtension) => void) | undefined;
         "ImportDeclaration:exit"?: ((node: ESTree.ImportDeclaration & NodeParentExtension) => void) | undefined;
         ImportDefaultSpecifier?: ((node: ESTree.ImportDefaultSpecifier & NodeParentExtension) => void) | undefined;
-        "ImportDefaultSpecifier:exit"?:
-        | ((node: ESTree.ImportDefaultSpecifier & NodeParentExtension) => void)
-        | undefined;
+        "ImportDefaultSpecifier:exit"?: ((node: ESTree.ImportDefaultSpecifier & NodeParentExtension) => void) | undefined;
         ImportExpression?: ((node: ESTree.ImportExpression & NodeParentExtension) => void) | undefined;
         "ImportExpression:exit"?: ((node: ESTree.ImportExpression & NodeParentExtension) => void) | undefined;
         ImportNamespaceSpecifier?: ((node: ESTree.ImportNamespaceSpecifier & NodeParentExtension) => void) | undefined;
-        "ImportNamespaceSpecifier:exit"?:
-        | ((node: ESTree.ImportNamespaceSpecifier & NodeParentExtension) => void)
-        | undefined;
+        "ImportNamespaceSpecifier:exit"?: ((node: ESTree.ImportNamespaceSpecifier & NodeParentExtension) => void) | undefined;
         ImportSpecifier?: ((node: ESTree.ImportSpecifier & NodeParentExtension) => void) | undefined;
         "ImportSpecifier:exit"?: ((node: ESTree.ImportSpecifier & NodeParentExtension) => void) | undefined;
         LabeledStatement?: ((node: ESTree.LabeledStatement & NodeParentExtension) => void) | undefined;
@@ -641,9 +679,7 @@ export namespace Rule {
         SwitchStatement?: ((node: ESTree.SwitchStatement & NodeParentExtension) => void) | undefined;
         "SwitchStatement:exit"?: ((node: ESTree.SwitchStatement & NodeParentExtension) => void) | undefined;
         TaggedTemplateExpression?: ((node: ESTree.TaggedTemplateExpression & NodeParentExtension) => void) | undefined;
-        "TaggedTemplateExpression:exit"?:
-        | ((node: ESTree.TaggedTemplateExpression & NodeParentExtension) => void)
-        | undefined;
+        "TaggedTemplateExpression:exit"?: ((node: ESTree.TaggedTemplateExpression & NodeParentExtension) => void) | undefined;
         TemplateElement?: ((node: ESTree.TemplateElement & NodeParentExtension) => void) | undefined;
         "TemplateElement:exit"?: ((node: ESTree.TemplateElement & NodeParentExtension) => void) | undefined;
         TemplateLiteral?: ((node: ESTree.TemplateLiteral & NodeParentExtension) => void) | undefined;
@@ -765,39 +801,11 @@ export namespace Rule {
         hasSuggestions?: boolean | undefined;
     }
 
-    interface RuleContext {
-        id: string;
-        options: any[];
-        settings: { [name: string]: any };
-        parserPath: string | undefined;
-        languageOptions: Linter.LanguageOptions;
-        parserOptions: Linter.ParserOptions;
-        cwd: string;
-        filename: string;
-        physicalFilename: string;
-        sourceCode: SourceCode;
-
-        getAncestors(): ESTree.Node[];
-
-        getDeclaredVariables(node: ESTree.Node): Scope.Variable[];
-
-        /** @deprecated Use property `filename` directly instead */
-        getFilename(): string;
-
-        /** @deprecated Use property `physicalFilename` directly instead */
-        getPhysicalFilename(): string;
-
-        /** @deprecated Use property `cwd` directly instead */
-        getCwd(): string;
-
-        getScope(): Scope.Scope;
-
-        /** @deprecated Use property `sourceCode` directly instead */
-        getSourceCode(): SourceCode;
-
-        markVariableAsUsed(name: string): boolean;
-
-        report(descriptor: ReportDescriptor): void;
+    interface RuleContext extends CoreRuleContext<RuleContextTypeOptions & {
+        LangOptions: Linter.LanguageOptions;
+        Code: SourceCode;
+        Node: ESTree.Node; }> {
+        // report(descriptor: ReportDescriptor): void;
     }
 
     type ReportFixer = (fixer: RuleFixer) => null | Fix | IterableIterator<Fix> | Fix[];
@@ -1325,7 +1333,7 @@ export namespace Linter {
         [name: string]: GlobalConf;
     }
 
-    interface LanguageOptions {
+    interface LanguageOptions extends GenericLanguageOptions {
         /**
          * The version of ECMAScript to support. May be any year (i.e., 2022) or
          * version (i.e., 5). Set to "latest" for the most recent supported version.
@@ -1469,7 +1477,7 @@ export namespace ESLint {
         environments?: Record<string, Environment> | undefined;
         languages?: Record<string, Language> | undefined;
         processors?: Record<string, Linter.Processor> | undefined;
-        rules?: Record<string, Rule.RuleModule> | undefined;
+        rules?: Record<string, RuleDefinition> | undefined;
     }
 
     type FixType = "directive" | "problem" | "suggestion" | "layout";

package/messages/config-serialize-function.js

@@ -0,0 +1,28 @@
+"use strict";
+
+module.exports = function({ key, objectKey }) {
+
+    // special case for parsers
+    const isParser = objectKey === "parser" && (key === "parse" || key === "parseForESLint");
+    const parserMessage = `
+    This typically happens when you're using a custom parser that does not
+provide a "meta" property, which is how ESLint determines the serialized
+representation. Please open an issue with the maintainer of the custom parser
+and share this link:
+
+https://eslint.org/docs/latest/extend/custom-parsers#meta-data-in-custom-parsers
+`.trim();
+
+    return `
+The requested operation requires ESLint to serialize configuration data,
+but the configuration key "${objectKey}.${key}" contains a function value,
+which cannot be serialized.
+
+${
+    isParser ? parserMessage : "Please double-check your configuration for errors."
+}
+
+If you still have problems, please stop by https://eslint.org/chat/help to chat
+with the team.
+`.trimStart();
+};

package/messages/eslintrc-plugins.js

@@ -5,9 +5,13 @@ module.exports = function({ plugins }) {
     const isArrayOfStrings = typeof plugins[0] === "string";
 
     return `
-A config object has a "plugins" key defined as an array${isArrayOfStrings ? " of strings" : ""}.
+A config object has a "plugins" key defined as an array${isArrayOfStrings ? " of strings" : ""}. It looks something like this:
 
-Flat config requires "plugins" to be an object in this form:
+    {
+        "plugins": ${JSON.stringify(plugins)}
+    }
+
+Flat config requires "plugins" to be an object, like this:
 
     {
         plugins: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint",
-  "version": "9.19.0",
+  "version": "9.20.1",
   "author": "Nicholas C. Zakas <nicholas+npm@nczconsulting.com>",
   "description": "An AST-based pattern checker for JavaScript.",
   "type": "commonjs",
@@ -102,9 +102,9 @@
     "@eslint-community/eslint-utils": "^4.2.0",
     "@eslint-community/regexpp": "^4.12.1",
     "@eslint/config-array": "^0.19.0",
-    "@eslint/core": "^0.10.0",
+    "@eslint/core": "^0.11.0",
     "@eslint/eslintrc": "^3.2.0",
-    "@eslint/js": "9.19.0",
+    "@eslint/js": "9.20.0",
     "@eslint/plugin-kit": "^0.2.5",
     "@humanfs/node": "^0.16.6",
     "@humanwhocodes/module-importer": "^1.0.1",
@@ -138,7 +138,7 @@
     "@arethetypeswrong/cli": "^0.17.0",
     "@babel/core": "^7.4.3",
     "@babel/preset-env": "^7.4.3",
-    "@eslint/json": "^0.9.0",
+    "@eslint/json": "^0.10.0",
     "@trunkio/launcher": "^1.3.0",
     "@types/node": "^20.11.5",
     "@typescript-eslint/parser": "^8.4.0",