improvement(build-tools): address ESLint rule violations in build-tools

build-tools/packages/build-cli/src/BasePackageCommand.ts

@@ -151,6 +151,7 @@ export abstract class PackageCommand<
 		}
 
 		try {
+			// eslint-disable-next-line import-x/no-named-as-default-member -- async.mapLimit is the idiomatic usage
 			await async.mapLimit(packages, this.flags.concurrency, async (pkg: PackageWithKind) => {
 				started += 1;
 				updateStatus();

build-tools/packages/build-cli/src/codeCoverage/getCoverageMetrics.ts

@@ -56,7 +56,7 @@ const extractCoverageMetrics = (
 
 /**
  * Method that returns the coverage report for the build from the artifact.
- * @param baselineZip - zipped coverage files for the build
+ * @param artifactZip - zipped coverage files for the build
  * @param logger - The logger to log messages.
  * @returns an map of coverage metrics for build containing packageName, lineCoverage and branchCoverage
  */

build-tools/packages/build-cli/src/commands/bump/deps.ts

@@ -5,6 +5,7 @@
 
 import { Flags } from "@oclif/core";
 import chalk from "picocolors";
+// eslint-disable-next-line import-x/no-named-as-default -- prompts default export is the intended API
 import prompts from "prompts";
 import stripAnsi from "strip-ansi";
 

build-tools/packages/build-cli/src/commands/check/policy.ts

@@ -6,7 +6,7 @@
 import { strict as assert } from "node:assert";
 import * as fs from "node:fs";
 import { EOL as newline } from "node:os";
-import * as path from "node:path";
+import path from "node:path";
 import process from "node:process";
 import { Flags } from "@oclif/core";
 
@@ -313,7 +313,7 @@ export class CheckPolicy extends BaseCommand<typeof CheckPolicy> {
 	): Promise<void> {
 		const { exclusions, gitRoot, pathRegex } = commandContext;
 
-		const filePath = path.join(gitRoot, inputPath).trim().replace(/\\/g, "/");
+		const filePath = path.join(gitRoot, inputPath).trim().replaceAll("\\", "/");
 		assert(path.isAbsolute(filePath) === true);
 
 		if (!pathRegex.test(inputPath) || !fs.existsSync(filePath)) {

build-tools/packages/build-cli/src/commands/generate/assertTags.ts

@@ -295,8 +295,8 @@ The format of the configuration is specified by the "AssertTaggingPackageConfig"
 						if (comments.length > 0) {
 							let originalErrorText = comments[0]
 								.getText()
-								.replace(/\/\*/g, "")
-								.replace(/\*\//g, "")
+								.replaceAll("/*", "")
+								.replaceAll("*/", "")
 								.trim();
 
 							// Replace leading+trailing double quotes and backticks.

build-tools/packages/build-cli/src/commands/generate/changeset.ts

@@ -11,6 +11,7 @@ import { Flags, ux } from "@oclif/core";
 import { PackageName } from "@rushstack/node-core-library";
 import { humanId } from "human-id";
 import chalk from "picocolors";
+// eslint-disable-next-line import-x/no-named-as-default -- prompts default export is the intended API
 import prompts from "prompts";
 
 import { releaseGroupFlag } from "../../flags.js";

build-tools/packages/build-cli/src/commands/generate/typetests.ts

@@ -432,8 +432,8 @@ function getNodeTypeData(node: Node, log: Logger, exportedName: string): TypeDat
 			tags: getTags(docs),
 		};
 
-		const escapedTypeName = exportedName.replace(/\./g, "_");
-		const trimmedKind = node.getKindName().replace(/Declaration/g, "");
+		const escapedTypeName = exportedName.replaceAll(".", "_");
+		const trimmedKind = node.getKindName().replaceAll("Declaration", "");
 
 		if (
 			// Covers instance type of the class (including generics of it)

build-tools/packages/build-cli/src/commands/release/fromTag.ts

@@ -127,6 +127,7 @@ export default class FromTagCommand extends ReleaseReportBaseCommand<typeof From
 		const tag = input.startsWith(tagRefPrefix) ? input.slice(tagRefPrefix.length) : input;
 		const [rg, ver] = tag.split("_v");
 
+		// eslint-disable-next-line import-x/no-named-as-default-member -- semver.parse is the idiomatic usage
 		const version = semver.parse(ver);
 		if (version === null) {
 			throw new Error(`Invalid version parsed from tag: ${ver}`);

build-tools/packages/build-cli/src/commands/release/report-unreleased.ts

@@ -70,6 +70,7 @@ export class UnreleasedReportCommand extends BaseCommand<typeof UnreleasedReport
  * @param fullReleaseReport - The format of the "full" release report.
  * @param version - The version string for the reports.
  * @param outDir - The output directory for the reports.
+ * @param branchName - The branch name for the reports.
  * @param log - The logger object for logging messages.
  */
 async function generateReleaseReport(

build-tools/packages/build-cli/src/commands/release/report.ts

@@ -119,7 +119,7 @@ export abstract class ReleaseReportBaseCommand<
 	 *
 	 * @param context - The {@link Context}.
 	 * @param mode - The {@link ReleaseSelectionMode} to use to determine the release to report on.
-	 * @param releaseGroup - If provided, the release data collected will be limited to only the pakages in this release
+	 * @param releaseGroupOrPackage - If provided, the release data collected will be limited to only the packages in this release
 	 * group and its direct Fluid dependencies.
 	 * @param includeDependencies - If true, the release data will include the Fluid dependencies of the release group.
 	 */
@@ -205,7 +205,7 @@ export abstract class ReleaseReportBaseCommand<
 	/**
 	 * Collects the releases of a given release group or package.
 	 *
-	 * @param context - The {@link Context}.
+	 * @param repo - The {@link Repository}.
 	 * @param releaseGroupOrPackage - The release group or package to collect release data for.
 	 * @param repoVersion - The version of the release group or package in the repo.
 	 * @param latestReleaseChooseMode - Controls which release is considered the latest.
@@ -734,5 +734,6 @@ async function writeReport(
 	const reportOutput = toReportKind(report, kind);
 
 	await writeJson(reportPath, reportOutput, { spaces: 2 });
+	// eslint-disable-next-line import-x/no-named-as-default-member -- sortJson.overwrite is the correct usage
 	sortJson.overwrite(reportPath);
 }

build-tools/packages/build-infrastructure/api-report/build-infrastructure.api.md

@@ -259,6 +259,4 @@ export interface WorkspaceDefinition {
 // @public
 export type WorkspaceName = Opaque<string, "WorkspaceName">;
 
-// (No @packageDocumentation comment for this package)
-
 ```

build-tools/packages/build-infrastructure/src/buildProject.ts

@@ -136,7 +136,7 @@ export class BuildProject<P extends IPackage> implements IBuildProject<P> {
 	 */
 	public relativeToRepo(p: string): string {
 		// Replace \ in result with / in case OS is Windows.
-		return path.relative(this.root, p).replace(/\\/g, "/");
+		return path.relative(this.root, p).replaceAll("\\", "/");
 	}
 
 	/**

build-tools/packages/build-infrastructure/src/index.ts

@@ -11,7 +11,7 @@
  * interdependencies between packages across a BuildProject. It then provides APIs to select, filter, and work with
  * those package groups.
  *
- * @module default entrypoint
+ * @packageDocumentation
  */
 
 export {

build-tools/packages/build-infrastructure/src/test/git.test.ts

@@ -6,7 +6,7 @@
 import { strict as assert } from "node:assert";
 import { unlink } from "node:fs/promises";
 import * as os from "node:os";
-import * as path from "node:path";
+import path from "node:path";
 
 import { expect } from "chai";
 import { readJson, writeJson } from "fs-extra/esm";

build-tools/packages/build-infrastructure/src/test/packageJsonUtils.test.ts

@@ -4,7 +4,7 @@
  */
 
 import { strict as assert } from "node:assert";
-import * as path from "node:path";
+import path from "node:path";
 
 import { describe, it } from "mocha";
 

build-tools/packages/version-tools/src/commands/version.ts

@@ -37,7 +37,6 @@ interface VersionInfo {
 /**
  * The root `version` command.
  */
-// eslint-disable-next-line import-x/no-default-export
 export default class VersionCommand extends Command {
 	static readonly description =
 		"Convert version strings between regular semver and the Fluid internal version scheme.";

build-tools/packages/version-tools/src/commands/version/latest.ts

@@ -15,7 +15,6 @@ import { getLatestReleaseFromList } from "../../schemes";
  * standard tools (e.g. `npm show versions`) don't fully work in that scenario because the Fluid internal version scheme
  * overloads the semver prerelease field.
  */
-// eslint-disable-next-line import-x/no-default-export
 export default class LatestCommand extends Command {
 	static readonly description =
 		"Find the latest version from a list of version strings, accounting for the Fluid internal version scheme.";

build-tools/packages/version-tools/src/internalVersionScheme.ts

@@ -344,7 +344,7 @@ export function bumpInternalVersion(
 	version: semver.SemVer | string,
 	bumpType: VersionBumpTypeExtended,
 ): semver.SemVer {
-	validateVersionScheme(version, true, undefined);
+	validateVersionScheme(version, true);
 	const [pubVer, intVer, prereleaseId] = fromInternalScheme(version, true, true);
 
 	const newIntVer =
@@ -376,7 +376,7 @@ export function getVersionRange(
 	version: semver.SemVer | string,
 	maxAutomaticBump: "minor" | "patch" | "~" | "^",
 ): string {
-	validateVersionScheme(version, true, undefined);
+	validateVersionScheme(version, true);
 
 	const lowVersion = version;
 	let highVersion: semver.SemVer;