eslint 9.7.0 → 9.9.0

package/README.md

@@ -295,7 +295,7 @@ The following companies, organizations, and individuals support ESLint's ongoing
 <!--sponsorsstart-->
 <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="#"><img src="https://images.opencollective.com/guest-bf377e88/avatar.png" alt="Eli Schleifer" height="96"></a> <a href="https://engineering.salesforce.com"><img src="https://images.opencollective.com/salesforce/ca8f997/logo.png" alt="Salesforce" height="96"></a></p><h3>Silver Sponsors</h3>
+<p><a href="#"><img src="https://images.opencollective.com/guest-bf377e88/avatar.png" alt="Eli Schleifer" height="96"></a></p><h3>Silver Sponsors</h3>
 <p><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?v=4" alt="American Express" height="64"></a> <a href="https://www.workleap.com"><img src="https://avatars.githubusercontent.com/u/53535748?u=d1e55d7661d724bf2281c1bfd33cb8f99fe2465f&v=4" alt="Workleap" height="64"></a></p><h3>Bronze Sponsors</h3>
 <p><a href="https://www.notion.so"><img src="https://images.opencollective.com/notion/bf3b117/logo.png" alt="notion" 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.ignitionapp.com"><img src="https://avatars.githubusercontent.com/u/5753491?v=4" alt="Ignition" height="32"></a> <a href="https://nx.dev"><img src="https://avatars.githubusercontent.com/u/23692104?v=4" alt="Nx" height="32"></a> <a href="https://herocoders.com"><img src="https://avatars.githubusercontent.com/u/37549774?v=4" alt="HeroCoders" height="32"></a> <a href="https://usenextbase.com"><img src="https://avatars.githubusercontent.com/u/145838380?v=4" alt="Nextbase Starter Kit" height="32"></a></p>
 <!--sponsorsend-->

package/lib/cli.js

@@ -26,7 +26,6 @@ const fs = require("node:fs"),
     { normalizeSeverityToString } = require("./shared/severity");
 const { Legacy: { naming } } = require("@eslint/eslintrc");
 const { ModuleImporter } = require("@humanwhocodes/module-importer");
-const { inactiveFlags, activeFlags } = require("./shared/flags");
 const debug = require("debug")("eslint:cli");
 
 //------------------------------------------------------------------------------
@@ -341,16 +340,17 @@ const cli = {
     /**
      * Calculates the command string for the --inspect-config operation.
      * @param {string} configFile The path to the config file to inspect.
+     * @param {boolean} hasUnstableTSConfigFlag `true` if the `unstable_ts_config` flag is enabled, `false` if it's not.
      * @returns {Promise<string>} The command string to execute.
      */
-    async calculateInspectConfigFlags(configFile) {
+    async calculateInspectConfigFlags(configFile, hasUnstableTSConfigFlag) {
 
         // find the config file
         const {
             configFilePath,
             basePath,
             error
-        } = await locateConfigFileToUse({ cwd: process.cwd(), configFile });
+        } = await locateConfigFileToUse({ cwd: process.cwd(), configFile }, hasUnstableTSConfigFlag);
 
         if (error) {
             throw error;
@@ -455,7 +455,7 @@ const cli = {
             try {
                 const flatOptions = await translateOptions(options, "flat");
                 const spawn = require("cross-spawn");
-                const flags = await cli.calculateInspectConfigFlags(flatOptions.overrideConfigFile);
+                const flags = await cli.calculateInspectConfigFlags(flatOptions.overrideConfigFile, flatOptions.flags ? flatOptions.flags.includes("unstable_ts_config") : false);
 
                 spawn.sync("npx", ["@eslint/config-inspector@latest", ...flags], { encoding: "utf8", stdio: "inherit" });
             } catch (error) {
@@ -488,25 +488,6 @@ const cli = {
 
         const ActiveESLint = usingFlatConfig ? ESLint : LegacyESLint;
         const eslintOptions = await translateOptions(options, usingFlatConfig ? "flat" : "eslintrc");
-
-        if (eslintOptions.flags) {
-            debug("Checking for inactive flags");
-
-            for (const flag of eslintOptions.flags) {
-                if (inactiveFlags.has(flag)) {
-                    log.warn(`InactiveFlag: The '${flag}' flag is no longer active: ${inactiveFlags.get(flag)}`);
-                    continue;
-                }
-
-                if (activeFlags.has(flag)) {
-                    continue;
-                }
-
-                log.error(`InvalidFlag: The '${flag}' flag is invalid.`);
-                return 2;
-            }
-        }
-
         const engine = new ActiveESLint(eslintOptions);
         let results;
 

package/lib/eslint/eslint.js

@@ -63,6 +63,8 @@ const { Retrier } = require("@humanwhocodes/retry");
 /** @typedef {import("../shared/types").RuleConf} RuleConf */
 /** @typedef {import("../shared/types").Rule} Rule */
 /** @typedef {ReturnType<ConfigArray.extractConfig>} ExtractedConfig */
+/** @typedef {import('../cli-engine/cli-engine').CLIEngine} CLIEngine */
+/** @typedef {import('./legacy-eslint').CLIEngineLintReport} CLIEngineLintReport */
 
 /**
  * The options with which to configure the ESLint instance.
@@ -86,7 +88,7 @@ const { Retrier } = require("@humanwhocodes/retry");
  *      when a string.
  * @property {Record<string,Plugin>} [plugins] An array of plugin implementations.
  * @property {boolean} [stats] True enables added statistics on lint results.
- * @property {boolean} warnIgnored Show warnings when the file list includes ignored files
+ * @property {boolean} [warnIgnored] Show warnings when the file list includes ignored files
  * @property {boolean} [passOnNoPatterns=false] When set to true, missing patterns cause
  *      the linting operation to short circuit and not report any failures.
  */
@@ -100,8 +102,18 @@ const FLAT_CONFIG_FILENAMES = [
     "eslint.config.mjs",
     "eslint.config.cjs"
 ];
+const FLAT_CONFIG_FILENAMES_WITH_TS = [
+    ...FLAT_CONFIG_FILENAMES,
+    "eslint.config.ts",
+    "eslint.config.mts",
+    "eslint.config.cts"
+];
 const debug = require("debug")("eslint:eslint");
 const privateMembers = new WeakMap();
+
+/**
+ * @type {Map<string, string>}
+ */
 const importedConfigFileModificationTime = new Map();
 const removedFormatters = new Set([
     "checkstyle",
@@ -262,28 +274,59 @@ function compareResultsByFilePath(a, b) {
  * Searches from the current working directory up until finding the
  * given flat config filename.
  * @param {string} cwd The current working directory to search from.
+ * @param {boolean} hasUnstableTSConfigFlag `true` if the `unstable_ts_config` flag is enabled, `false` if it's not.
  * @returns {Promise<string|undefined>} The filename if found or `undefined` if not.
  */
-function findFlatConfigFile(cwd) {
+function findFlatConfigFile(cwd, hasUnstableTSConfigFlag) {
+    const filenames = hasUnstableTSConfigFlag ? FLAT_CONFIG_FILENAMES_WITH_TS : FLAT_CONFIG_FILENAMES;
+
     return findUp(
-        FLAT_CONFIG_FILENAMES,
+        filenames,
         { cwd }
     );
 }
 
+/**
+ * Check if the file is a TypeScript file.
+ * @param {string} filePath The file path to check.
+ * @returns {boolean} `true` if the file is a TypeScript file, `false` if it's not.
+ */
+function isFileTS(filePath) {
+    const fileExtension = path.extname(filePath);
+
+    return /^\.[mc]?ts$/u.test(fileExtension);
+}
+
+/**
+ * Check if ESLint is running in Bun.
+ * @returns {boolean} `true` if the ESLint is running Bun, `false` if it's not.
+ */
+function isRunningInBun() {
+    return !!globalThis.Bun;
+}
+
+/**
+ * Check if ESLint is running in Deno.
+ * @returns {boolean} `true` if the ESLint is running in Deno, `false` if it's not.
+ */
+function isRunningInDeno() {
+    return !!globalThis.Deno;
+}
+
 /**
  * Load the config array from the given filename.
  * @param {string} filePath The filename to load from.
+ * @param {boolean} hasUnstableTSConfigFlag `true` if the `unstable_ts_config` flag is enabled, `false` if it's not.
  * @returns {Promise<any>} The config loaded from the config file.
  */
-async function loadFlatConfigFile(filePath) {
+async function loadFlatConfigFile(filePath, hasUnstableTSConfigFlag) {
     debug(`Loading config from ${filePath}`);
 
     const fileURL = pathToFileURL(filePath);
 
     debug(`Config file URL is ${fileURL}`);
 
-    const mtime = (await fs.stat(filePath)).mtime.getTime();
+    const mtime = (await fs.stat(filePath)).mtime.getTime().toString();
 
     /*
      * Append a query with the config file's modification time (`mtime`) in order
@@ -314,7 +357,37 @@ async function loadFlatConfigFile(filePath) {
         delete require.cache[filePath];
     }
 
-    const config = (await import(fileURL)).default;
+    const isTS = isFileTS(filePath) && hasUnstableTSConfigFlag;
+
+    const isBun = isRunningInBun();
+
+    const isDeno = isRunningInDeno();
+
+    if (isTS && !isDeno && !isBun) {
+
+        const createJiti = await import("jiti").then(jitiModule => jitiModule.default, () => {
+            throw new Error("The 'jiti' library is required for loading TypeScript configuration files. Make sure to install it.");
+        });
+
+        /*
+         * Disabling `moduleCache` allows us to reload a
+         * config file when the last modified timestamp changes.
+         */
+
+        const jiti = createJiti(__filename, { moduleCache: false });
+
+        if (typeof jiti?.import !== "function") {
+            throw new Error("You are using an outdated version of the 'jiti' library. Please update to the latest version of 'jiti' to ensure compatibility and access to the latest features.");
+        }
+
+        const config = await jiti.import(fileURL.href);
+
+        importedConfigFileModificationTime.set(filePath, mtime);
+
+        return config?.default ?? config;
+    }
+
+    const config = (await import(fileURL.href)).default;
 
     importedConfigFileModificationTime.set(filePath, mtime);
 
@@ -326,11 +399,12 @@ async function loadFlatConfigFile(filePath) {
  * override config file was passed, and if so, using it; otherwise, as long
  * as override config file is not explicitly set to `false`, it will search
  * upwards from the cwd for a file named `eslint.config.js`.
- * @param {import("./eslint").ESLintOptions} options The ESLint instance options.
- * @returns {{configFilePath:string|undefined,basePath:string,error:Error|null}} Location information for
+ * @param {ESLintOptions} options The ESLint instance options.
+ * @param {boolean} hasUnstableTSConfigFlag `true` if the `unstable_ts_config` flag is enabled, `false` if it's not.
+ * @returns {Promise<{configFilePath:string|undefined;basePath:string;error:Error|null}>} Location information for
  *      the config file.
  */
-async function locateConfigFileToUse({ configFile, cwd }) {
+async function locateConfigFileToUse({ configFile, cwd }, hasUnstableTSConfigFlag) {
 
     // determine where to load config file from
     let configFilePath;
@@ -342,7 +416,7 @@ async function locateConfigFileToUse({ configFile, cwd }) {
         configFilePath = path.resolve(cwd, configFile);
     } else if (configFile !== false) {
         debug("Searching for eslint.config.js");
-        configFilePath = await findFlatConfigFile(cwd);
+        configFilePath = await findFlatConfigFile(cwd, hasUnstableTSConfigFlag);
 
         if (configFilePath) {
             basePath = path.resolve(path.dirname(configFilePath));
@@ -364,8 +438,8 @@ async function locateConfigFileToUse({ configFile, cwd }) {
 /**
  * Calculates the config array for this run based on inputs.
  * @param {ESLint} eslint The instance to create the config array for.
- * @param {import("./eslint").ESLintOptions} options The ESLint instance options.
- * @returns {FlatConfigArray} The config array for `eslint``.
+ * @param {ESLintOptions} options The ESLint instance options.
+ * @returns {Promise<typeof FlatConfigArray>} The config array for `eslint``.
  */
 async function calculateConfigArray(eslint, {
     cwd,
@@ -383,7 +457,9 @@ async function calculateConfigArray(eslint, {
         return slots.configs;
     }
 
-    const { configFilePath, basePath, error } = await locateConfigFileToUse({ configFile, cwd });
+    const hasUnstableTSConfigFlag = eslint.hasFlag("unstable_ts_config");
+
+    const { configFilePath, basePath, error } = await locateConfigFileToUse({ configFile, cwd }, hasUnstableTSConfigFlag);
 
     // config file is required to calculate config
     if (error) {
@@ -394,7 +470,7 @@ async function calculateConfigArray(eslint, {
 
     // load config file
     if (configFilePath) {
-        const fileConfig = await loadFlatConfigFile(configFilePath);
+        const fileConfig = await loadFlatConfigFile(configFilePath, hasUnstableTSConfigFlag);
 
         if (Array.isArray(fileConfig)) {
             configs.push(...fileConfig);
@@ -1144,7 +1220,7 @@ class ESLint {
 
             /**
              * The main formatter method.
-             * @param {LintResults[]} results The lint results to format.
+             * @param {LintResult[]} results The lint results to format.
              * @param {ResultsMeta} resultsMeta Warning count and max threshold.
              * @returns {string} The formatted lint results.
              */
@@ -1190,12 +1266,12 @@ class ESLint {
     /**
      * Finds the config file being used by this instance based on the options
      * passed to the constructor.
-     * @returns {string|undefined} The path to the config file being used or
+     * @returns {Promise<string|undefined>} The path to the config file being used or
      *      `undefined` if no config file is being used.
      */
     async findConfigFile() {
         const options = privateMembers.get(this).options;
-        const { configFilePath } = await locateConfigFileToUse(options);
+        const { configFilePath } = await locateConfigFileToUse(options, this.hasFlag("unstable_ts_config"));
 
         return configFilePath;
     }

package/lib/languages/js/source-code/source-code.js

@@ -916,6 +916,25 @@ class SourceCode extends TokenStore {
 
         return ancestorsStartingAtParent.reverse();
     }
+
+    /**
+     * Returns the location of the given node or token.
+     * @param {ASTNode|Token} nodeOrToken The node or token to get the location of.
+     * @returns {SourceLocation} The location of the node or token.
+     */
+    getLoc(nodeOrToken) {
+        return nodeOrToken.loc;
+    }
+
+    /**
+     * Returns the range of the given node or token.
+     * @param {ASTNode|Token} nodeOrToken The node or token to get the range of.
+     * @returns {[number, number]} The range of the node or token.
+     */
+    getRange(nodeOrToken) {
+        return nodeOrToken.range;
+    }
+
     /* eslint-enable class-methods-use-this -- node is owned by SourceCode */
 
     /**

package/lib/linter/apply-disable-directives.js

@@ -61,15 +61,18 @@ function groupByParentDirective(directives) {
  * Creates removal details for a set of directives within the same comment.
  * @param {Directive[]} directives Unused directives to be removed.
  * @param {Token} node The backing Comment token.
+ * @param {SourceCode} sourceCode The source code object for the file being linted.
  * @returns {{ description, fix, unprocessedDirective }[]} Details for later creation of output Problems.
  */
-function createIndividualDirectivesRemoval(directives, node) {
+function createIndividualDirectivesRemoval(directives, node, sourceCode) {
+
+    const range = sourceCode.getRange(node);
 
     /*
      * `node.value` starts right after `//` or `/*`.
      * All calculated offsets will be relative to this index.
      */
-    const commentValueStart = node.range[0] + "//".length;
+    const commentValueStart = range[0] + "//".length;
 
     // Find where the list of rules starts. `\S+` matches with the directive name (e.g. `eslint-disable-line`)
     const listStartOffset = /^\s*\S+\s+/u.exec(node.value)[0].length;
@@ -165,10 +168,11 @@ function createIndividualDirectivesRemoval(directives, node) {
  * Creates a description of deleting an entire unused disable directive.
  * @param {Directive[]} directives Unused directives to be removed.
  * @param {Token} node The backing Comment token.
+ * @param {SourceCode} sourceCode The source code object for the file being linted.
  * @returns {{ description, fix, unprocessedDirective }} Details for later creation of an output problem.
  */
-function createDirectiveRemoval(directives, node) {
-    const { range } = node;
+function createDirectiveRemoval(directives, node, sourceCode) {
+    const range = sourceCode.getRange(node);
     const ruleIds = directives.filter(directive => directive.ruleId).map(directive => `'${directive.ruleId}'`);
 
     return {
@@ -186,9 +190,10 @@ function createDirectiveRemoval(directives, node) {
 /**
  * Parses details from directives to create output Problems.
  * @param {Iterable<Directive>} allDirectives Unused directives to be removed.
+ * @param {SourceCode} sourceCode The source code object for the file being linted.
  * @returns {{ description, fix, unprocessedDirective }[]} Details for later creation of output Problems.
  */
-function processUnusedDirectives(allDirectives) {
+function processUnusedDirectives(allDirectives, sourceCode) {
     const directiveGroups = groupByParentDirective(allDirectives);
 
     return directiveGroups.flatMap(
@@ -201,8 +206,8 @@ function processUnusedDirectives(allDirectives) {
             }
 
             return remainingRuleIds.size
-                ? createIndividualDirectivesRemoval(directives, parentDirective.node)
-                : [createDirectiveRemoval(directives, parentDirective.node)];
+                ? createIndividualDirectivesRemoval(directives, parentDirective.node, sourceCode)
+                : [createDirectiveRemoval(directives, parentDirective.node, sourceCode)];
         }
     );
 }
@@ -309,6 +314,7 @@ function collectUsedEnableDirectives(directives) {
 function applyDirectives(options) {
     const problems = [];
     const usedDisableDirectives = new Set();
+    const { sourceCode } = options;
 
     for (const problem of options.problems) {
         let disableDirectivesForProblem = [];
@@ -370,8 +376,8 @@ function applyDirectives(options) {
         }
     }
 
-    const processed = processUnusedDirectives(unusedDisableDirectivesToReport)
-        .concat(processUnusedDirectives(unusedEnableDirectivesToReport));
+    const processed = processUnusedDirectives(unusedDisableDirectivesToReport, sourceCode)
+        .concat(processUnusedDirectives(unusedEnableDirectivesToReport, sourceCode));
     const columnOffset = options.language.columnStart === 1 ? 0 : 1;
     const lineOffset = options.language.lineStart === 1 ? 0 : 1;
 
@@ -390,11 +396,14 @@ function applyDirectives(options) {
                     ? `Unused eslint-disable directive (no problems were reported from ${description}).`
                     : "Unused eslint-disable directive (no problems were reported).";
             }
+
+            const loc = sourceCode.getLoc(parentDirective.node);
+
             return {
                 ruleId: null,
                 message,
-                line: type === "disable-next-line" ? parentDirective.node.loc.start.line + lineOffset : line,
-                column: type === "disable-next-line" ? parentDirective.node.loc.start.column + columnOffset : column,
+                line: type === "disable-next-line" ? loc.start.line + lineOffset : line,
+                column: type === "disable-next-line" ? loc.start.column + columnOffset : column,
                 severity: options.reportUnusedDisableDirectives === "warn" ? 1 : 2,
                 nodeType: null,
                 ...options.disableFixes ? {} : { fix }
@@ -409,6 +418,7 @@ function applyDirectives(options) {
  * of reported problems, adds the suppression information to the problems.
  * @param {Object} options Information about directives and problems
  * @param {Language} options.language The language being linted.
+ * @param {SourceCode} options.sourceCode The source code object for the file being linted.
  * @param {{
  *      type: ("disable"|"enable"|"disable-line"|"disable-next-line"),
  *      ruleId: (string|null),
@@ -427,7 +437,7 @@ function applyDirectives(options) {
  * @returns {{ruleId: (string|null), line: number, column: number, suppressions?: {kind: string, justification: string}}[]}
  * An object with a list of reported problems, the suppressed of which contain the suppression information.
  */
-module.exports = ({ language, directives, disableFixes, problems, configuredRules, ruleFilter, reportUnusedDisableDirectives = "off" }) => {
+module.exports = ({ language, sourceCode, directives, disableFixes, problems, configuredRules, ruleFilter, reportUnusedDisableDirectives = "off" }) => {
     const blockDirectives = directives
         .filter(directive => directive.type === "disable" || directive.type === "enable")
         .map(directive => Object.assign({}, directive, { unprocessedDirective: directive }))
@@ -477,6 +487,7 @@ module.exports = ({ language, directives, disableFixes, problems, configuredRule
 
     const blockDirectivesResult = applyDirectives({
         language,
+        sourceCode,
         problems,
         directives: blockDirectives,
         disableFixes,
@@ -485,6 +496,7 @@ module.exports = ({ language, directives, disableFixes, problems, configuredRule
     });
     const lineDirectivesResult = applyDirectives({
         language,
+        sourceCode,
         problems: blockDirectivesResult.problems,
         directives: lineDirectives,
         disableFixes,

package/lib/linter/linter.js

@@ -45,7 +45,7 @@ const { RuleValidator } = require("../config/rule-validator");
 const { assertIsRuleSeverity } = require("../config/flat-config-schema");
 const { normalizeSeverityToString } = require("../shared/severity");
 const jslang = require("../languages/js");
-const { activeFlags } = require("../shared/flags");
+const { activeFlags, inactiveFlags } = require("../shared/flags");
 const debug = require("debug")("eslint:linter");
 const MAX_AUTOFIX_PASSES = 10;
 const DEFAULT_PARSER_NAME = "espree";
@@ -323,9 +323,10 @@ function createLintingProblem(options) {
  * @param {ASTNode|token} options.node The Comment node/token.
  * @param {function(string): {create: Function}} ruleMapper A map from rule IDs to defined rules
  * @param {Language} language The language to use to adjust the location information.
+ * @param {SourceCode} sourceCode The SourceCode object to get comments from.
  * @returns {Object} Directives and problems from the comment
  */
-function createDisableDirectives({ type, value, justification, node }, ruleMapper, language) {
+function createDisableDirectives({ type, value, justification, node }, ruleMapper, language, sourceCode) {
     const ruleIds = Object.keys(commentParser.parseListConfig(value));
     const directiveRules = ruleIds.length ? ruleIds : [null];
     const result = {
@@ -336,11 +337,15 @@ function createDisableDirectives({ type, value, justification, node }, ruleMappe
 
     for (const ruleId of directiveRules) {
 
+        const loc = sourceCode.getLoc(node);
+
         // push to directives, if the rule is defined(including null, e.g. /*eslint enable*/)
         if (ruleId === null || !!ruleMapper(ruleId)) {
+
+
             if (type === "disable-next-line") {
                 const { line, column } = updateLocationInformation(
-                    node.loc.end,
+                    loc.end,
                     language
                 );
 
@@ -354,7 +359,7 @@ function createDisableDirectives({ type, value, justification, node }, ruleMappe
                 });
             } else {
                 const { line, column } = updateLocationInformation(
-                    node.loc.start,
+                    loc.start,
                     language
                 );
 
@@ -368,7 +373,7 @@ function createDisableDirectives({ type, value, justification, node }, ruleMappe
                 });
             }
         } else {
-            result.directiveProblems.push(createLintingProblem({ ruleId, loc: node.loc, language }));
+            result.directiveProblems.push(createLintingProblem({ ruleId, loc, language }));
         }
     }
     return result;
@@ -410,25 +415,27 @@ function getDirectiveComments(sourceCode, ruleMapper, warnInlineConfig, config)
             return;
         }
 
+        const loc = sourceCode.getLoc(comment);
+
         if (warnInlineConfig) {
             const kind = comment.type === "Block" ? `/*${directiveText}*/` : `//${directiveText}`;
 
             problems.push(createLintingProblem({
                 ruleId: null,
                 message: `'${kind}' has no effect because you have 'noInlineConfig' setting in ${warnInlineConfig}.`,
-                loc: comment.loc,
+                loc,
                 severity: 1
             }));
             return;
         }
 
-        if (directiveText === "eslint-disable-line" && comment.loc.start.line !== comment.loc.end.line) {
+        if (directiveText === "eslint-disable-line" && loc.start.line !== loc.end.line) {
             const message = `${directiveText} comment should not span multiple lines.`;
 
             problems.push(createLintingProblem({
                 ruleId: null,
                 message,
-                loc: comment.loc
+                loc
             }));
             return;
         }
@@ -446,7 +453,7 @@ function getDirectiveComments(sourceCode, ruleMapper, warnInlineConfig, config)
                     value: directiveValue,
                     justification: justificationPart,
                     node: comment
-                }, ruleMapper, jslang);
+                }, ruleMapper, jslang, sourceCode);
 
                 disableDirectives.push(...directives);
                 problems.push(...directiveProblems);
@@ -467,7 +474,7 @@ function getDirectiveComments(sourceCode, ruleMapper, warnInlineConfig, config)
                     } catch (err) {
                         problems.push(createLintingProblem({
                             ruleId: null,
-                            loc: comment.loc,
+                            loc,
                             message: err.message
                         }));
                         continue;
@@ -494,14 +501,14 @@ function getDirectiveComments(sourceCode, ruleMapper, warnInlineConfig, config)
                         const ruleValue = parseResult.config[name];
 
                         if (!rule) {
-                            problems.push(createLintingProblem({ ruleId: name, loc: comment.loc }));
+                            problems.push(createLintingProblem({ ruleId: name, loc }));
                             return;
                         }
 
                         if (Object.hasOwn(configuredRules, name)) {
                             problems.push(createLintingProblem({
                                 message: `Rule "${name}" is already configured by another configuration comment in the preceding code. This configuration is ignored.`,
-                                loc: comment.loc
+                                loc
                             }));
                             return;
                         }
@@ -563,7 +570,7 @@ function getDirectiveComments(sourceCode, ruleMapper, warnInlineConfig, config)
                             problems.push(createLintingProblem({
                                 ruleId: name,
                                 message: err.message,
-                                loc: comment.loc
+                                loc
                             }));
 
                             // do not apply the config, if found invalid options.
@@ -575,7 +582,7 @@ function getDirectiveComments(sourceCode, ruleMapper, warnInlineConfig, config)
                 } else {
                     const problem = createLintingProblem({
                         ruleId: null,
-                        loc: comment.loc,
+                        loc,
                         message: parseResult.error.message
                     });
 
@@ -623,7 +630,7 @@ function getDirectiveCommentsForFlatConfig(sourceCode, ruleMapper, language) {
         })));
 
         directivesSources.forEach(directive => {
-            const { directives, directiveProblems } = createDisableDirectives(directive, ruleMapper, language);
+            const { directives, directiveProblems } = createDisableDirectives(directive, ruleMapper, language, sourceCode);
 
             disableDirectives.push(...directives);
             problems.push(...directiveProblems);
@@ -1282,9 +1289,20 @@ class Linter {
      * @param {"flat"|"eslintrc"} [config.configType="flat"] the type of config used.
      */
     constructor({ cwd, configType = "flat", flags = [] } = {}) {
+
+        flags.forEach(flag => {
+            if (inactiveFlags.has(flag)) {
+                throw new Error(`The flag '${flag}' is inactive: ${inactiveFlags.get(flag)}`);
+            }
+
+            if (!activeFlags.has(flag)) {
+                throw new Error(`Unknown flag '${flag}'.`);
+            }
+        });
+
         internalSlotsMap.set(this, {
             cwd: normalizeCwd(cwd),
-            flags: flags.filter(flag => activeFlags.has(flag)),
+            flags,
             lastConfigArray: null,
             lastSourceCode: null,
             lastSuppressedMessages: [],
@@ -1462,7 +1480,7 @@ class Linter {
             debug("An error occurred while traversing");
             debug("Filename:", options.filename);
             if (err.currentNode) {
-                const { line } = err.currentNode.loc.start;
+                const { line } = sourceCode.getLoc(err.currentNode).start;
 
                 debug("Line:", line);
                 err.message += `:${line}`;
@@ -1480,6 +1498,7 @@ class Linter {
 
         return applyDisableDirectives({
             language: jslang,
+            sourceCode,
             directives: commentDirectives.disableDirectives,
             disableFixes: options.disableFixes,
             problems: lintingProblems
@@ -1759,10 +1778,14 @@ class Linter {
             if (options.warnInlineConfig) {
                 if (sourceCode.getInlineConfigNodes) {
                     sourceCode.getInlineConfigNodes().forEach(node => {
+
+                        const loc = sourceCode.getLoc(node);
+                        const range = sourceCode.getRange(node);
+
                         inlineConfigProblems.push(createLintingProblem({
                             ruleId: null,
-                            message: `'${sourceCode.text.slice(node.range[0], node.range[1])}' has no effect because you have 'noInlineConfig' setting in ${options.warnInlineConfig}.`,
-                            loc: node.loc,
+                            message: `'${sourceCode.text.slice(range[0], range[1])}' has no effect because you have 'noInlineConfig' setting in ${options.warnInlineConfig}.`,
+                            loc,
                             severity: 1,
                             language: config.language
                         }));
@@ -1942,7 +1965,7 @@ class Linter {
             debug("An error occurred while traversing");
             debug("Filename:", options.filename);
             if (err.currentNode) {
-                const { line } = err.currentNode.loc.start;
+                const { line } = sourceCode.getLoc(err.currentNode).start;
 
                 debug("Line:", line);
                 err.message += `:${line}`;
@@ -1961,6 +1984,7 @@ class Linter {
 
         return applyDisableDirectives({
             language: config.language,
+            sourceCode,
             directives: commentDirectives.disableDirectives,
             disableFixes: options.disableFixes,
             problems: lintingProblems

package/lib/linter/report-translator.js

@@ -10,7 +10,7 @@
 //------------------------------------------------------------------------------
 
 const assert = require("node:assert");
-const ruleFixer = require("./rule-fixer");
+const { RuleFixer } = require("./rule-fixer");
 const { interpolate } = require("./interpolate");
 
 //------------------------------------------------------------------------------
@@ -91,13 +91,10 @@ function assertValidNodeInfo(descriptor) {
  * from the `node` of the original descriptor, or infers the `start` from the `loc` of the original descriptor.
  */
 function normalizeReportLoc(descriptor) {
-    if (descriptor.loc) {
-        if (descriptor.loc.start) {
-            return descriptor.loc;
-        }
-        return { start: descriptor.loc, end: null };
+    if (descriptor.loc.start) {
+        return descriptor.loc;
     }
-    return descriptor.node.loc;
+    return { start: descriptor.loc, end: null };
 }
 
 /**
@@ -190,6 +187,8 @@ function normalizeFixes(descriptor, sourceCode) {
         return null;
     }
 
+    const ruleFixer = new RuleFixer({ sourceCode });
+
     // @type {null | Fix | Fix[] | IterableIterator<Fix>}
     const fix = descriptor.fix(ruleFixer);
 
@@ -335,6 +334,7 @@ module.exports = function createReportTranslator(metadata) {
     return (...args) => {
         const descriptor = normalizeMultiArgReportCall(...args);
         const messages = metadata.messageIds;
+        const { sourceCode } = metadata;
 
         assertValidNodeInfo(descriptor);
 
@@ -367,9 +367,9 @@ module.exports = function createReportTranslator(metadata) {
             node: descriptor.node,
             message: interpolate(computedMessage, descriptor.data),
             messageId: descriptor.messageId,
-            loc: normalizeReportLoc(descriptor),
-            fix: metadata.disableFixes ? null : normalizeFixes(descriptor, metadata.sourceCode),
-            suggestions: metadata.disableFixes ? [] : mapSuggestions(descriptor, metadata.sourceCode, messages),
+            loc: descriptor.loc ? normalizeReportLoc(descriptor) : sourceCode.getLoc(descriptor.node),
+            fix: metadata.disableFixes ? null : normalizeFixes(descriptor, sourceCode),
+            suggestions: metadata.disableFixes ? [] : mapSuggestions(descriptor, sourceCode, messages),
             language: metadata.language
         });
     };

package/lib/linter/rule-fixer.js

@@ -4,6 +4,8 @@
  */
 "use strict";
 
+/* eslint class-methods-use-this: off -- Methods desired on instance */
+
 //------------------------------------------------------------------------------
 // Requirements
 //------------------------------------------------------------------------------
@@ -35,8 +37,22 @@ function insertTextAt(index, text) {
 /**
  * Creates code fixing commands for rules.
  */
+class RuleFixer {
 
-const ruleFixer = Object.freeze({
+    /**
+     * The source code object representing the text to be fixed.
+     * @type {SourceCode}
+     */
+    #sourceCode;
+
+    /**
+     * Creates a new instance.
+     * @param {Object} options The options for the fixer.
+     * @param {SourceCode} options.sourceCode The source code object representing the text to be fixed.
+     */
+    constructor({ sourceCode }) {
+        this.#sourceCode = sourceCode;
+    }
 
     /**
      * Creates a fix command that inserts text after the given node or token.
@@ -46,8 +62,10 @@ const ruleFixer = Object.freeze({
      * @returns {Object} The fix command.
      */
     insertTextAfter(nodeOrToken, text) {
-        return this.insertTextAfterRange(nodeOrToken.range, text);
-    },
+        const range = this.#sourceCode.getRange(nodeOrToken);
+
+        return this.insertTextAfterRange(range, text);
+    }
 
     /**
      * Creates a fix command that inserts text after the specified range in the source text.
@@ -59,7 +77,7 @@ const ruleFixer = Object.freeze({
      */
     insertTextAfterRange(range, text) {
         return insertTextAt(range[1], text);
-    },
+    }
 
     /**
      * Creates a fix command that inserts text before the given node or token.
@@ -69,8 +87,10 @@ const ruleFixer = Object.freeze({
      * @returns {Object} The fix command.
      */
     insertTextBefore(nodeOrToken, text) {
-        return this.insertTextBeforeRange(nodeOrToken.range, text);
-    },
+        const range = this.#sourceCode.getRange(nodeOrToken);
+
+        return this.insertTextBeforeRange(range, text);
+    }
 
     /**
      * Creates a fix command that inserts text before the specified range in the source text.
@@ -82,7 +102,7 @@ const ruleFixer = Object.freeze({
      */
     insertTextBeforeRange(range, text) {
         return insertTextAt(range[0], text);
-    },
+    }
 
     /**
      * Creates a fix command that replaces text at the node or token.
@@ -92,8 +112,10 @@ const ruleFixer = Object.freeze({
      * @returns {Object} The fix command.
      */
     replaceText(nodeOrToken, text) {
-        return this.replaceTextRange(nodeOrToken.range, text);
-    },
+        const range = this.#sourceCode.getRange(nodeOrToken);
+
+        return this.replaceTextRange(range, text);
+    }
 
     /**
      * Creates a fix command that replaces text at the specified range in the source text.
@@ -108,7 +130,7 @@ const ruleFixer = Object.freeze({
             range,
             text
         };
-    },
+    }
 
     /**
      * Creates a fix command that removes the node or token from the source.
@@ -117,8 +139,10 @@ const ruleFixer = Object.freeze({
      * @returns {Object} The fix command.
      */
     remove(nodeOrToken) {
-        return this.removeRange(nodeOrToken.range);
-    },
+        const range = this.#sourceCode.getRange(nodeOrToken);
+
+        return this.removeRange(range);
+    }
 
     /**
      * Creates a fix command that removes the specified range of text from the source.
@@ -133,8 +157,7 @@ const ruleFixer = Object.freeze({
             text: ""
         };
     }
-
-});
+}
 
 
-module.exports = ruleFixer;
+module.exports = { RuleFixer };

package/lib/rules/require-await.js

@@ -42,8 +42,11 @@ module.exports = {
         schema: [],
 
         messages: {
-            missingAwait: "{{name}} has no 'await' expression."
-        }
+            missingAwait: "{{name}} has no 'await' expression.",
+            removeAsync: "Remove 'async'."
+        },
+
+        hasSuggestions: true
     },
 
     create(context) {
@@ -69,6 +72,33 @@ module.exports = {
          */
         function exitFunction(node) {
             if (!node.generator && node.async && !scopeInfo.hasAwait && !astUtils.isEmptyFunction(node)) {
+
+                /*
+                 * If the function belongs to a method definition or
+                 * property, then the function's range may not include the
+                 * `async` keyword and we should look at the parent instead.
+                 */
+                const nodeWithAsyncKeyword =
+                    (node.parent.type === "MethodDefinition" && node.parent.value === node) ||
+                    (node.parent.type === "Property" && node.parent.method && node.parent.value === node)
+                        ? node.parent
+                        : node;
+
+                const asyncToken = sourceCode.getFirstToken(nodeWithAsyncKeyword, token => token.value === "async");
+                const asyncRange = [asyncToken.range[0], sourceCode.getTokenAfter(asyncToken, { includeComments: true }).range[0]];
+
+                /*
+                 * Removing the `async` keyword can cause parsing errors if the current
+                 * statement is relying on automatic semicolon insertion. If ASI is currently
+                 * being used, then we should replace the `async` keyword with a semicolon.
+                 */
+                const nextToken = sourceCode.getTokenAfter(asyncToken);
+                const addSemiColon =
+                    nextToken.type === "Punctuator" &&
+                    (nextToken.value === "[" || nextToken.value === "(") &&
+                    (nodeWithAsyncKeyword.type === "MethodDefinition" || astUtils.isStartOfExpressionStatement(nodeWithAsyncKeyword)) &&
+                    astUtils.needsPrecedingSemicolon(sourceCode, nodeWithAsyncKeyword);
+
                 context.report({
                     node,
                     loc: astUtils.getFunctionHeadLoc(node, sourceCode),
@@ -77,7 +107,11 @@ module.exports = {
                         name: capitalizeFirstLetter(
                             astUtils.getFunctionNameWithKind(node)
                         )
-                    }
+                    },
+                    suggest: [{
+                        messageId: "removeAsync",
+                        fix: fixer => fixer.replaceTextRange(asyncRange, addSemiColon ? ";" : "")
+                    }]
                 });
             }
 

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

@@ -1042,11 +1042,12 @@ function isStartOfExpressionStatement(node) {
 
 /**
  * Determines whether an opening parenthesis `(`, bracket `[` or backtick ``` ` ``` needs to be preceded by a semicolon.
- * This opening parenthesis or bracket should be at the start of an `ExpressionStatement` or at the start of the body of an `ArrowFunctionExpression`.
+ * This opening parenthesis or bracket should be at the start of an `ExpressionStatement`, a `MethodDefinition` or at
+ * the start of the body of an `ArrowFunctionExpression`.
  * @type {(sourceCode: SourceCode, node: ASTNode) => boolean}
  * @param {SourceCode} sourceCode The source code object.
  * @param {ASTNode} node A node at the position where an opening parenthesis or bracket will be inserted.
- * @returns {boolean} Whether a semicolon is required before the opening parenthesis or braket.
+ * @returns {boolean} Whether a semicolon is required before the opening parenthesis or bracket.
  */
 let needsPrecedingSemicolon;
 
@@ -1106,7 +1107,7 @@ let needsPrecedingSemicolon;
 
         if (isClosingBraceToken(prevToken)) {
             return (
-                prevNode.type === "BlockStatement" && prevNode.parent.type === "FunctionExpression" ||
+                prevNode.type === "BlockStatement" && prevNode.parent.type === "FunctionExpression" && prevNode.parent.parent.type !== "MethodDefinition" ||
                 prevNode.type === "ClassBody" && prevNode.parent.type === "ClassExpression" ||
                 prevNode.type === "ObjectExpression"
             );

package/lib/shared/flags.js

@@ -9,7 +9,8 @@
  * @type {Map<string, string>}
  */
 const activeFlags = new Map([
-    ["test_only", "This flag is only used for testing."]
+    ["test_only", "Used only for testing."],
+    ["unstable_ts_config", "Enable TypeScript configuration files."]
 ]);
 
 /**
@@ -17,7 +18,7 @@ const activeFlags = new Map([
  * @type {Map<string, string>}
  */
 const inactiveFlags = new Map([
-    ["test_only_old", "This flag is no longer used for testing."]
+    ["test_only_old", "Used only for testing."]
 ]);
 
 module.exports = {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint",
-  "version": "9.7.0",
+  "version": "9.9.0",
   "author": "Nicholas C. Zakas <nicholas+npm@nczconsulting.com>",
   "description": "An AST-based pattern checker for JavaScript.",
   "bin": {
@@ -68,9 +68,9 @@
   "dependencies": {
     "@eslint-community/eslint-utils": "^4.2.0",
     "@eslint-community/regexpp": "^4.11.0",
-    "@eslint/config-array": "^0.17.0",
+    "@eslint/config-array": "^0.17.1",
     "@eslint/eslintrc": "^3.1.0",
-    "@eslint/js": "9.7.0",
+    "@eslint/js": "9.9.0",
     "@humanwhocodes/module-importer": "^1.0.1",
     "@humanwhocodes/retry": "^0.3.0",
     "@nodelib/fs.walk": "^1.2.8",
@@ -104,13 +104,14 @@
   "devDependencies": {
     "@babel/core": "^7.4.3",
     "@babel/preset-env": "^7.4.3",
-    "@eslint/core": "^0.1.0",
+    "@eslint/core": "^0.3.0",
+    "@eslint/json": "^0.3.0",
     "@types/estree": "^1.0.5",
     "@types/node": "^20.11.5",
-    "@wdio/browser-runner": "^8.38.3",
-    "@wdio/cli": "^8.38.2",
-    "@wdio/concise-reporter": "^8.38.2",
-    "@wdio/mocha-framework": "^8.38.2",
+    "@wdio/browser-runner": "^8.40.1",
+    "@wdio/cli": "^8.40.0",
+    "@wdio/concise-reporter": "^8.39.0",
+    "@wdio/mocha-framework": "^8.40.0",
     "babel-loader": "^8.0.5",
     "c8": "^7.12.0",
     "chai": "^4.0.1",
@@ -131,6 +132,7 @@
     "globals": "^15.0.0",
     "got": "^11.8.3",
     "gray-matter": "^4.0.3",
+    "jiti": "^1.21.6",
     "js-yaml": "^4.1.0",
     "knip": "^5.21.0",
     "lint-staged": "^11.0.0",
@@ -164,6 +166,14 @@
     "webpack-cli": "^4.5.0",
     "yorkie": "^2.0.0"
   },
+  "peerDependencies": {
+    "jiti": "*"
+  },
+  "peerDependenciesMeta": {
+    "jiti": {
+      "optional": true
+    }
+  },
   "keywords": [
     "ast",
     "lint",