eslint 9.21.0 → 9.23.0

package/lib/shared/runtime-info.js

@@ -24,129 +24,138 @@ const packageJson = require("../../package.json");
  * @returns {string} A string that contains execution environment information.
  */
 function environment() {
-    const cache = new Map();
-
-    /**
-     * Checks if a path is a child of a directory.
-     * @param {string} parentPath The parent path to check.
-     * @param {string} childPath The path to check.
-     * @returns {boolean} Whether or not the given path is a child of a directory.
-     */
-    function isChildOfDirectory(parentPath, childPath) {
-        return !path.relative(parentPath, childPath).startsWith("..");
-    }
-
-    /**
-     * Synchronously executes a shell command and formats the result.
-     * @param {string} cmd The command to execute.
-     * @param {Array} args The arguments to be executed with the command.
-     * @throws {Error} As may be collected by `cross-spawn.sync`.
-     * @returns {string} The version returned by the command.
-     */
-    function execCommand(cmd, args) {
-        const key = [cmd, ...args].join(" ");
-
-        if (cache.has(key)) {
-            return cache.get(key);
-        }
-
-        const process = spawn.sync(cmd, args, { encoding: "utf8" });
-
-        if (process.error) {
-            throw process.error;
-        }
-
-        const result = process.stdout.trim();
-
-        cache.set(key, result);
-        return result;
-    }
-
-    /**
-     * Normalizes a version number.
-     * @param {string} versionStr The string to normalize.
-     * @returns {string} The normalized version number.
-     */
-    function normalizeVersionStr(versionStr) {
-        return versionStr.startsWith("v") ? versionStr : `v${versionStr}`;
-    }
-
-    /**
-     * Gets bin version.
-     * @param {string} bin The bin to check.
-     * @throws {Error} As may be collected by `cross-spawn.sync`.
-     * @returns {string} The normalized version returned by the command.
-     */
-    function getBinVersion(bin) {
-        const binArgs = ["--version"];
-
-        try {
-            return normalizeVersionStr(execCommand(bin, binArgs));
-        } catch (e) {
-            log.error(`Error finding ${bin} version running the command \`${bin} ${binArgs.join(" ")}\``);
-            throw e;
-        }
-    }
-
-    /**
-     * Gets installed npm package version.
-     * @param {string} pkg The package to check.
-     * @param {boolean} global Whether to check globally or not.
-     * @throws {Error} As may be collected by `cross-spawn.sync`.
-     * @returns {string} The normalized version returned by the command.
-     */
-    function getNpmPackageVersion(pkg, { global = false } = {}) {
-        const npmBinArgs = ["bin", "-g"];
-        const npmLsArgs = ["ls", "--depth=0", "--json", pkg];
-
-        if (global) {
-            npmLsArgs.push("-g");
-        }
-
-        try {
-            const parsedStdout = JSON.parse(execCommand("npm", npmLsArgs));
-
-            /*
-             * Checking globally returns an empty JSON object, while local checks
-             * include the name and version of the local project.
-             */
-            if (Object.keys(parsedStdout).length === 0 || !(parsedStdout.dependencies && parsedStdout.dependencies.eslint)) {
-                return "Not found";
-            }
-
-            const [, processBinPath] = process.argv;
-            let npmBinPath;
-
-            try {
-                npmBinPath = execCommand("npm", npmBinArgs);
-            } catch (e) {
-                log.error(`Error finding npm binary path when running command \`npm ${npmBinArgs.join(" ")}\``);
-                throw e;
-            }
-
-            const isGlobal = isChildOfDirectory(npmBinPath, processBinPath);
-            let pkgVersion = parsedStdout.dependencies.eslint.version;
-
-            if ((global && isGlobal) || (!global && !isGlobal)) {
-                pkgVersion += " (Currently used)";
-            }
-
-            return normalizeVersionStr(pkgVersion);
-        } catch (e) {
-            log.error(`Error finding ${pkg} version running the command \`npm ${npmLsArgs.join(" ")}\``);
-            throw e;
-        }
-    }
-
-    return [
-        "Environment Info:",
-        "",
-        `Node version: ${getBinVersion("node")}`,
-        `npm version: ${getBinVersion("npm")}`,
-        `Local ESLint version: ${getNpmPackageVersion("eslint", { global: false })}`,
-        `Global ESLint version: ${getNpmPackageVersion("eslint", { global: true })}`,
-        `Operating System: ${os.platform()} ${os.release()}`
-    ].join("\n");
+	const cache = new Map();
+
+	/**
+	 * Checks if a path is a child of a directory.
+	 * @param {string} parentPath The parent path to check.
+	 * @param {string} childPath The path to check.
+	 * @returns {boolean} Whether or not the given path is a child of a directory.
+	 */
+	function isChildOfDirectory(parentPath, childPath) {
+		return !path.relative(parentPath, childPath).startsWith("..");
+	}
+
+	/**
+	 * Synchronously executes a shell command and formats the result.
+	 * @param {string} cmd The command to execute.
+	 * @param {Array} args The arguments to be executed with the command.
+	 * @throws {Error} As may be collected by `cross-spawn.sync`.
+	 * @returns {string} The version returned by the command.
+	 */
+	function execCommand(cmd, args) {
+		const key = [cmd, ...args].join(" ");
+
+		if (cache.has(key)) {
+			return cache.get(key);
+		}
+
+		const process = spawn.sync(cmd, args, { encoding: "utf8" });
+
+		if (process.error) {
+			throw process.error;
+		}
+
+		const result = process.stdout.trim();
+
+		cache.set(key, result);
+		return result;
+	}
+
+	/**
+	 * Normalizes a version number.
+	 * @param {string} versionStr The string to normalize.
+	 * @returns {string} The normalized version number.
+	 */
+	function normalizeVersionStr(versionStr) {
+		return versionStr.startsWith("v") ? versionStr : `v${versionStr}`;
+	}
+
+	/**
+	 * Gets bin version.
+	 * @param {string} bin The bin to check.
+	 * @throws {Error} As may be collected by `cross-spawn.sync`.
+	 * @returns {string} The normalized version returned by the command.
+	 */
+	function getBinVersion(bin) {
+		const binArgs = ["--version"];
+
+		try {
+			return normalizeVersionStr(execCommand(bin, binArgs));
+		} catch (e) {
+			log.error(
+				`Error finding ${bin} version running the command \`${bin} ${binArgs.join(" ")}\``,
+			);
+			throw e;
+		}
+	}
+
+	/**
+	 * Gets installed npm package version.
+	 * @param {string} pkg The package to check.
+	 * @param {boolean} global Whether to check globally or not.
+	 * @throws {Error} As may be collected by `cross-spawn.sync`.
+	 * @returns {string} The normalized version returned by the command.
+	 */
+	function getNpmPackageVersion(pkg, { global = false } = {}) {
+		const npmBinArgs = ["bin", "-g"];
+		const npmLsArgs = ["ls", "--depth=0", "--json", pkg];
+
+		if (global) {
+			npmLsArgs.push("-g");
+		}
+
+		try {
+			const parsedStdout = JSON.parse(execCommand("npm", npmLsArgs));
+
+			/*
+			 * Checking globally returns an empty JSON object, while local checks
+			 * include the name and version of the local project.
+			 */
+			if (
+				Object.keys(parsedStdout).length === 0 ||
+				!(parsedStdout.dependencies && parsedStdout.dependencies.eslint)
+			) {
+				return "Not found";
+			}
+
+			const [, processBinPath] = process.argv;
+			let npmBinPath;
+
+			try {
+				npmBinPath = execCommand("npm", npmBinArgs);
+			} catch (e) {
+				log.error(
+					`Error finding npm binary path when running command \`npm ${npmBinArgs.join(" ")}\``,
+				);
+				throw e;
+			}
+
+			const isGlobal = isChildOfDirectory(npmBinPath, processBinPath);
+			let pkgVersion = parsedStdout.dependencies.eslint.version;
+
+			if ((global && isGlobal) || (!global && !isGlobal)) {
+				pkgVersion += " (Currently used)";
+			}
+
+			return normalizeVersionStr(pkgVersion);
+		} catch (e) {
+			log.error(
+				`Error finding ${pkg} version running the command \`npm ${npmLsArgs.join(" ")}\``,
+			);
+			throw e;
+		}
+	}
+
+	return [
+		"Environment Info:",
+		"",
+		`Node version: ${getBinVersion("node")}`,
+		`npm version: ${getBinVersion("npm")}`,
+		`Local ESLint version: ${getNpmPackageVersion("eslint", { global: false })}`,
+		`Global ESLint version: ${getNpmPackageVersion("eslint", { global: true })}`,
+		`Operating System: ${os.platform()} ${os.release()}`,
+	].join("\n");
 }
 
 /**
@@ -154,7 +163,7 @@ function environment() {
  * @returns {string} The version from the currently executing ESLint's package.json.
  */
 function version() {
-    return `v${packageJson.version}`;
+	return `v${packageJson.version}`;
 }
 
 //------------------------------------------------------------------------------
@@ -162,7 +171,7 @@ function version() {
 //------------------------------------------------------------------------------
 
 module.exports = {
-    __esModule: true, // Indicate intent for imports, remove ambiguity for Knip (see: https://github.com/eslint/eslint/pull/18005#discussion_r1484422616)
-    environment,
-    version
+	__esModule: true, // Indicate intent for imports, remove ambiguity for Knip (see: https://github.com/eslint/eslint/pull/18005#discussion_r1484422616)
+	environment,
+	version,
 };

package/lib/shared/serialization.js

@@ -12,14 +12,14 @@
  * @private
  */
 function isSerializablePrimitiveOrPlainObject(val) {
-    return (
-        val === null ||
-        typeof val === "string" ||
-         typeof val === "boolean" ||
-         typeof val === "number" ||
-         (typeof val === "object" && val.constructor === Object) ||
-         Array.isArray(val)
-    );
+	return (
+		val === null ||
+		typeof val === "string" ||
+		typeof val === "boolean" ||
+		typeof val === "number" ||
+		(typeof val === "object" && val.constructor === Object) ||
+		Array.isArray(val)
+	);
 }
 
 /**
@@ -30,26 +30,26 @@ function isSerializablePrimitiveOrPlainObject(val) {
  * @returns {boolean} true if the value is serializable
  */
 function isSerializable(val) {
-    if (!isSerializablePrimitiveOrPlainObject(val)) {
-        return false;
-    }
-    if (typeof val === "object") {
-        for (const property in val) {
-            if (Object.hasOwn(val, property)) {
-                if (!isSerializablePrimitiveOrPlainObject(val[property])) {
-                    return false;
-                }
-                if (typeof val[property] === "object") {
-                    if (!isSerializable(val[property])) {
-                        return false;
-                    }
-                }
-            }
-        }
-    }
-    return true;
+	if (!isSerializablePrimitiveOrPlainObject(val)) {
+		return false;
+	}
+	if (typeof val === "object") {
+		for (const property in val) {
+			if (Object.hasOwn(val, property)) {
+				if (!isSerializablePrimitiveOrPlainObject(val[property])) {
+					return false;
+				}
+				if (typeof val[property] === "object") {
+					if (!isSerializable(val[property])) {
+						return false;
+					}
+				}
+			}
+		}
+	}
+	return true;
 }
 
 module.exports = {
-    isSerializable
+	isSerializable,
 };

package/lib/shared/severity.js

@@ -12,16 +12,16 @@
  * @returns {string} severity string
  */
 function normalizeSeverityToString(severity) {
-    if ([2, "2", "error"].includes(severity)) {
-        return "error";
-    }
-    if ([1, "1", "warn"].includes(severity)) {
-        return "warn";
-    }
-    if ([0, "0", "off"].includes(severity)) {
-        return "off";
-    }
-    throw new Error(`Invalid severity value: ${severity}`);
+	if ([2, "2", "error"].includes(severity)) {
+		return "error";
+	}
+	if ([1, "1", "warn"].includes(severity)) {
+		return "warn";
+	}
+	if ([0, "0", "off"].includes(severity)) {
+		return "off";
+	}
+	throw new Error(`Invalid severity value: ${severity}`);
 }
 
 /**
@@ -31,19 +31,19 @@ function normalizeSeverityToString(severity) {
  * @returns {number} severity number
  */
 function normalizeSeverityToNumber(severity) {
-    if ([2, "2", "error"].includes(severity)) {
-        return 2;
-    }
-    if ([1, "1", "warn"].includes(severity)) {
-        return 1;
-    }
-    if ([0, "0", "off"].includes(severity)) {
-        return 0;
-    }
-    throw new Error(`Invalid severity value: ${severity}`);
+	if ([2, "2", "error"].includes(severity)) {
+		return 2;
+	}
+	if ([1, "1", "warn"].includes(severity)) {
+		return 1;
+	}
+	if ([0, "0", "off"].includes(severity)) {
+		return 0;
+	}
+	throw new Error(`Invalid severity value: ${severity}`);
 }
 
 module.exports = {
-    normalizeSeverityToString,
-    normalizeSeverityToNumber
+	normalizeSeverityToString,
+	normalizeSeverityToNumber,
 };

package/lib/shared/stats.js

@@ -10,7 +10,7 @@
  * @returns {[number, number]} t variable for tracking time
  */
 function startTime() {
-    return process.hrtime();
+	return process.hrtime();
 }
 
 /**
@@ -19,12 +19,12 @@ function startTime() {
  * @returns {number} The measured time in milliseconds
  */
 function endTime(t) {
-    const time = process.hrtime(t);
+	const time = process.hrtime(t);
 
-    return time[0] * 1e3 + time[1] / 1e6;
+	return time[0] * 1e3 + time[1] / 1e6;
 }
 
 module.exports = {
-    startTime,
-    endTime
+	startTime,
+	endTime,
 };

package/lib/shared/string-utils.js

@@ -25,10 +25,10 @@ let segmenter;
  * @returns {string} The converted string
  */
 function upperCaseFirst(string) {
-    if (string.length <= 1) {
-        return string.toUpperCase();
-    }
-    return string[0].toUpperCase() + string.slice(1);
+	if (string.length <= 1) {
+		return string.toUpperCase();
+	}
+	return string[0].toUpperCase() + string.slice(1);
 }
 
 /**
@@ -37,22 +37,22 @@ function upperCaseFirst(string) {
  * @returns {number} The number of graphemes in `value`.
  */
 function getGraphemeCount(value) {
-    if (ASCII_REGEX.test(value)) {
-        return value.length;
-    }
+	if (ASCII_REGEX.test(value)) {
+		return value.length;
+	}
 
-    segmenter ??= new Intl.Segmenter("en-US"); // en-US locale should be supported everywhere
-    let graphemeCount = 0;
+	segmenter ??= new Intl.Segmenter("en-US"); // en-US locale should be supported everywhere
+	let graphemeCount = 0;
 
-    // eslint-disable-next-line no-unused-vars -- for-of needs a variable
-    for (const unused of segmenter.segment(value)) {
-        graphemeCount++;
-    }
+	// eslint-disable-next-line no-unused-vars -- for-of needs a variable
+	for (const unused of segmenter.segment(value)) {
+		graphemeCount++;
+	}
 
-    return graphemeCount;
+	return graphemeCount;
 }
 
 module.exports = {
-    upperCaseFirst,
-    getGraphemeCount
+	upperCaseFirst,
+	getGraphemeCount,
 };

package/lib/shared/text-table.js

@@ -32,36 +32,37 @@
 
 "use strict";
 
-module.exports = function(rows_, opts) {
-    const hsep = "  ";
-    const align = opts.align;
-    const stringLength = opts.stringLength;
+module.exports = function (rows_, opts) {
+	const hsep = "  ";
+	const align = opts.align;
+	const stringLength = opts.stringLength;
 
-    const sizes = rows_.reduce((acc, row) => {
-        row.forEach((c, ix) => {
-            const n = stringLength(c);
+	const sizes = rows_.reduce((acc, row) => {
+		row.forEach((c, ix) => {
+			const n = stringLength(c);
 
-            if (!acc[ix] || n > acc[ix]) {
-                acc[ix] = n;
-            }
-        });
-        return acc;
-    }, []);
+			if (!acc[ix] || n > acc[ix]) {
+				acc[ix] = n;
+			}
+		});
+		return acc;
+	}, []);
 
-    return rows_
-        .map(row =>
-            row
-                .map((c, ix) => {
-                    const n = sizes[ix] - stringLength(c) || 0;
-                    const s = Array(Math.max(n + 1, 1)).join(" ");
+	return rows_
+		.map(row =>
+			row
+				.map((c, ix) => {
+					const n = sizes[ix] - stringLength(c) || 0;
+					const s = Array(Math.max(n + 1, 1)).join(" ");
 
-                    if (align[ix] === "r") {
-                        return s + c;
-                    }
+					if (align[ix] === "r") {
+						return s + c;
+					}
 
-                    return c + s;
-                })
-                .join(hsep)
-                .trimEnd())
-        .join("\n");
+					return c + s;
+				})
+				.join(hsep)
+				.trimEnd(),
+		)
+		.join("\n");
 };