chore(biome): useNumberNamespace and useConst

biome.jsonc

@@ -38,6 +38,9 @@
 				"useArrowFunction": "error"
 			},
 			"style": {
+				"noNonNullAssertion": "off",
+				"useNumberNamespace": "error",
+				"useConst": "error",
 				"useImportType": "error",
 				"useExportType": "error"
 			}

packages/playground/cases/file/missing-files/index.test.ts

@@ -4,7 +4,7 @@ test("missing files should be able to recover if being added back", async ({
 	page,
 	fileAction
 }) => {
-	let overlay = page.frameLocator("#webpack-dev-server-client-overlay");
+	const overlay = page.frameLocator("#webpack-dev-server-client-overlay");
 	await expect(
 		overlay.getByText("Can't resolve './missing-file-1'")
 	).toBeVisible();

packages/playground/fixtures/pathInfo.ts

@@ -40,7 +40,7 @@ async function calcPathInfo(
 
 export const pathInfoFixtures: Fixtures<PathInfoFixtures> = {
 	pathInfo: async ({}, use, { file, workerIndex }) => {
-		let pathInfo: PathInfo = await calcPathInfo(file, String(workerIndex));
+		const pathInfo: PathInfo = await calcPathInfo(file, String(workerIndex));
 		await use(pathInfo);
 		await fs.remove(pathInfo.tempProjectDir);
 	}

packages/rspack-cli/src/cli.ts

@@ -46,8 +46,8 @@ export class RspackCLI {
 		process.env.RSPACK_CONFIG_VALIDATE = "loose";
 		process.env.WATCHPACK_WATCHER_LIMIT =
 			process.env.WATCHPACK_WATCHER_LIMIT || "20";
-		let nodeEnv = process?.env?.NODE_ENV;
-		let rspackCommandDefaultEnv =
+		const nodeEnv = process?.env?.NODE_ENV;
+		const rspackCommandDefaultEnv =
 			rspackCommand === "build" ? "production" : "development";
 		if (typeof options.nodeEnv === "string") {
 			process.env.NODE_ENV = nodeEnv || options.nodeEnv;
@@ -182,7 +182,7 @@ export class RspackCLI {
 				item.devtool = isBuild ? "source-map" : "cheap-module-source-map";
 			}
 			if (isServe) {
-				let installed = (item.plugins ||= []).find(
+				const installed = (item.plugins ||= []).find(
 					item => item instanceof rspackCore.ProgressPlugin
 				);
 				if (!installed) {

packages/rspack-dev-server/src/ansiHTML.js

@@ -288,7 +288,7 @@ function _setTags(colors) {
 		var color = _styles[code];
 		var oriColor = colors[color] || "000";
 		_openTags[code] = "color:#" + oriColor;
-		code = parseInt(code);
+		code = Number.parseInt(code);
 		_openTags[(code + 10).toString()] = "background:#" + oriColor;
 	}
 }

packages/rspack-dev-server/src/patch.ts

@@ -20,7 +20,7 @@ function applyDevServerPatch() {
 		// @ts-expect-error
 		...args
 	) {
-		let stats = args[1];
+		const stats = args[1];
 
 		if (!stats) {
 			return;

packages/rspack-lite-tapable/src/index.ts

@@ -327,8 +327,8 @@ export class HookBase<T, R, AdditionalOptions = UnsetAdditionalOptions>
 }
 
 export type StageRange = readonly [number, number];
-export const minStage = -Infinity;
-export const maxStage = Infinity;
+export const minStage = Number.NEGATIVE_INFINITY;
+export const maxStage = Number.POSITIVE_INFINITY;
 const allStageRange = [minStage, maxStage] as const;
 const i32MIN = -(2 ** 31);
 const i32MAX = 2 ** 31 - 1;
@@ -346,7 +346,7 @@ export class QueriedHook<T, R, AdditionalOptions = UnsetAdditionalOptions> {
 	constructor(stageRange: StageRange, hook: HookBase<T, R, AdditionalOptions>) {
 		const tapsInRange: typeof hook.taps = [];
 		const [from, to] = stageRange;
-		for (let tap of hook.taps) {
+		for (const tap of hook.taps) {
 			const stage = tap.stage ?? 0;
 			if (from <= stage && stage < to) {
 				tapsInRange.push(tap);
@@ -417,7 +417,7 @@ export class SyncHook<
 		if (from === minStage) {
 			this._runCallInterceptors(...args2);
 		}
-		for (let tap of tapsInRange) {
+		for (const tap of tapsInRange) {
 			this._runTapInterceptors(tap);
 			try {
 				tap.fn(...args2);
@@ -485,7 +485,7 @@ export class SyncBailHook<
 		if (from === minStage) {
 			this._runCallInterceptors(...args2);
 		}
-		for (let tap of tapsInRange) {
+		for (const tap of tapsInRange) {
 			this._runTapInterceptors(tap);
 			let r = undefined;
 			try {
@@ -566,7 +566,7 @@ export class SyncWaterfallHook<
 		if (from === minStage) {
 			this._runCallInterceptors(...args2);
 		}
-		for (let tap of tapsInRange) {
+		for (const tap of tapsInRange) {
 			this._runTapInterceptors(tap);
 			try {
 				const r = tap.fn(...args2);
@@ -646,7 +646,7 @@ export class AsyncParallelHook<
 		};
 		if (tapsInRange.length === 0) return done();
 		let counter = tapsInRange.length;
-		for (let tap of tapsInRange) {
+		for (const tap of tapsInRange) {
 			this._runTapInterceptors(tap);
 			if (tap.type === "promise") {
 				const promise = tap.fn(...args2);

packages/rspack-plugin-react-refresh/src/sockets/utils/getSocketUrlParts.ts

@@ -24,7 +24,7 @@ export default function getSocketUrlParts(
 	}
 
 	/** @type {SocketUrlParts} */
-	let urlParts: SocketUrlParts = {} as SocketUrlParts;
+	const urlParts: SocketUrlParts = {} as SocketUrlParts;
 
 	// If the resource query is available,
 	// parse it and ignore everything we received from the script host.

packages/rspack-plugin-react-refresh/src/utils/getAdditionalEntries.ts

@@ -14,7 +14,7 @@ export function getAdditionalEntries({
 	options: NormalizedPluginOptions;
 }): AdditionalEntries {
 	/** @type {Record<string, string | number>} */
-	let resourceQuery: Record<string, string | number> = {};
+	const resourceQuery: Record<string, string | number> = {};
 
 	if (devServer) {
 		const { client, https, http2, sockHost, sockPath, sockPort } = devServer;

packages/rspack-test-tools/src/case/diff.ts

@@ -65,7 +65,7 @@ export function createDiffCase(name: string, src: string, dist: string) {
 				compareMap.clear();
 				await tester.check(env);
 			});
-			for (let file of caseConfig.files!) {
+			for (const file of caseConfig.files!) {
 				describe(`Comparing "${file}"`, () => {
 					let moduleResults: TModuleCompareResult[] = [];
 					let runtimeResults: TModuleCompareResult[] = [];
@@ -137,7 +137,7 @@ function createDiffProcessor(config: IDiffProcessorOptions) {
 
 function checkBundleFiles(name: string, dist: string, files: string[]) {
 	describe(`Checking ${name} dist files`, () => {
-		for (let file of files) {
+		for (const file of files) {
 			it(`${name}: ${file} should be generated`, () => {
 				expect(fs.existsSync(path.join(dist, file))).toBeTruthy();
 			});
@@ -172,7 +172,7 @@ function checkCompareResults(
 			).toEqual([]);
 		});
 		it(`all modules should be the same`, () => {
-			for (let result of getResults().filter(
+			for (const result of getResults().filter(
 				i => i.type === ECompareResultType.Different
 			)) {
 				console.log(`${result.name}:\n${result.detail}`);

packages/rspack-test-tools/src/compare/comparator.ts

@@ -20,7 +20,7 @@ export interface IDiffComparatorOptions {
 export class DiffComparator {
 	constructor(private options: IDiffComparatorOptions) {}
 	async compare() {
-		for (let file of this.options.files!) {
+		for (const file of this.options.files!) {
 			try {
 				const result = compareFile(
 					path.join(this.options.rspackDist, file),
@@ -46,15 +46,15 @@ export class DiffComparator {
 						bootstrap: this.options.bootstrap
 					}
 				);
-				for (let reporter of this.options.reporters) {
+				for (const reporter of this.options.reporters) {
 					reporter.increment(file, result.modules["modules"] || []);
 				}
-				for (let reporter of this.options.reporters) {
+				for (const reporter of this.options.reporters) {
 					reporter.increment(file, result.modules["runtimeModules"] || []);
 				}
 			} catch (e) {
 				console.error(e);
-				for (let reporter of this.options.reporters) {
+				for (const reporter of this.options.reporters) {
 					reporter.failure(file);
 				}
 			}

packages/rspack-test-tools/src/compare/compare.ts

@@ -66,7 +66,7 @@ export function compareFile(
 		bootstrap: compareOptions.bootstrap
 	});
 
-	for (let type of ["modules", "runtimeModules"]) {
+	for (const type of ["modules", "runtimeModules"]) {
 		const t = type as "modules" | "runtimeModules";
 		let moduleList: string[] = [];
 		if (compareOptions[t] === true) {
@@ -99,7 +99,7 @@ export function compareModules(
 	compareOptions: ICompareOptions
 ) {
 	const compareResults: TModuleCompareResult[] = [];
-	for (let name of modules) {
+	for (const name of modules) {
 		const renamed = replaceRuntimeModuleName(name);
 		const sourceContent =
 			sourceModules.has(renamed) &&

packages/rspack-test-tools/src/compare/format-code.ts

@@ -163,7 +163,7 @@ export function formatCode(
 		},
 		ObjectExpression(path) {
 			if (options.ignoreObjectPropertySequence) {
-				let result = [];
+				const result = [];
 				let safe = [];
 				while (path.node.properties.length || safe.length) {
 					const cur = path.node.properties.shift()!;
@@ -210,7 +210,7 @@ export function formatCode(
 	}
 
 	if (options.replacements) {
-		for (let { from, to } of options.replacements) {
+		for (const { from, to } of options.replacements) {
 			result = result.replaceAll(from, to);
 		}
 	}

packages/rspack-test-tools/src/compare/replace-runtime-module-name.ts

@@ -73,7 +73,7 @@ const RUNTIME_MODULE_PARAM_REGEX = {
 };
 
 export function replaceRuntimeModuleName(raw: string) {
-	for (let [rspackName, webpackName] of Object.entries(
+	for (const [rspackName, webpackName] of Object.entries(
 		RUNTIME_MODULE_NAME_MAPPING
 	)) {
 		if (

packages/rspack-test-tools/src/helper/directory.ts

@@ -54,7 +54,7 @@ export function describeByWalk(
 						path.join(testId, caseName).split(".").shift()!
 					);
 					describeFn(name, () => {
-						let source = path.join(sourceBase, caseName);
+						const source = path.join(sourceBase, caseName);
 						let dist = "";
 						if (absoluteDist) {
 							dist = path.join(distBase, caseName);

packages/rspack-test-tools/src/helper/legacy/FakeDocument.js

@@ -36,7 +36,7 @@ export default class FakeDocument {
 
 	_onElementRemoved(element) {
 		const type = element._type;
-		let list = this._elementsByTagName.get(type);
+		const list = this._elementsByTagName.get(type);
 		const idx = list.indexOf(element);
 		list.splice(idx, 1);
 	}

packages/rspack-test-tools/src/helper/legacy/warmup-webpack.js

@@ -2,7 +2,7 @@
 describe("warmup", () => {
 	it("should warmup webpack", done => {
 		let webpack = require("@rspack/core");
-		let END = new Error("end warmup");
+		const END = new Error("end warmup");
 		webpack(
 			{
 				entry: "data:text/javascript,import 'data:text/javascript,'",

packages/rspack-test-tools/src/helper/util/checkSourceMap.js

@@ -18,7 +18,7 @@ export default async function checkSourceMap(
 	const path = require("path");
 
 	const sources = JSON.parse(outCodeMap).sources;
-	for (let source of sources) {
+	for (const source of sources) {
 		if (sources.filter(s => s === source).length > 1) {
 			throw new Error(
 				`Duplicate source ${JSON.stringify(source)} found in source map`

packages/rspack-test-tools/src/helper/util/expectWarningFactory.js

@@ -1,6 +1,6 @@
 // @ts-nocheck
 module.exports = () => {
-	let warnings = [];
+	const warnings = [];
 	let oldWarn;
 
 	beforeEach(done => {

packages/rspack-test-tools/src/plugin/webpack-module-placeholder-plugin.ts

@@ -32,7 +32,7 @@ function createRenderRuntimeModulesFn(Template) {
 				runtimeSource = codeGenResult.sources.get("runtime");
 			}
 			if (runtimeSource) {
-				let identifier = module.identifier();
+				const identifier = module.identifier();
 				source.add(Template.toNormalComment(`start::${identifier}`) + "\n");
 				if (!module.shouldIsolate()) {
 					source.add(runtimeSource);

packages/rspack-test-tools/src/processor/basic.ts

@@ -96,7 +96,7 @@ export class BasicProcessor<T extends ECompilerType> implements ITestProcessor {
 			return;
 		}
 
-		for (let bundle of bundles!) {
+		for (const bundle of bundles!) {
 			const runnerFactory = context.getRunnerFactory(this._options.name);
 			if (!runnerFactory) {
 				throw new Error(`Test case ${this._options.name} is not runable`);

packages/rspack-test-tools/src/processor/builtin.ts

@@ -141,7 +141,7 @@ export class BuiltinProcessor<
 				defaultOptions as TCompilerOptions<ECompilerType.Rspack>;
 			const testConfigFile = context.getSource("rspack.config.js");
 			if (fs.existsSync(testConfigFile)) {
-				let caseOptions = require(testConfigFile);
+				const caseOptions = require(testConfigFile);
 				if (caseOptions.entry) {
 					delete rspackDefaultOptions.entry;
 				}
@@ -166,7 +166,7 @@ export class BuiltinProcessor<
 			const htmlOptions = (rspackDefaultOptions as any).builtins?.html;
 			if (htmlOptions) {
 				if (Array.isArray(htmlOptions)) {
-					for (let item of htmlOptions) {
+					for (const item of htmlOptions) {
 						rspackDefaultOptions.plugins!.push(
 							new rspack.HtmlRspackPlugin(item)
 						);

packages/rspack-test-tools/src/processor/diff.ts

@@ -98,7 +98,7 @@ export class DiffProcessor implements ITestProcessor {
 		}
 
 		const dist = context.getDist();
-		for (let file of this.options.files!) {
+		for (const file of this.options.files!) {
 			const rspackDist = path.join(dist, ECompilerType.Rspack, file);
 			const webpackDist = path.join(dist, ECompilerType.Webpack, file);
 			const result = compareFile(rspackDist, webpackDist, {
@@ -173,7 +173,7 @@ export class DiffProcessor implements ITestProcessor {
 			ignoreCssFilePath: this.options.ignoreCssFilePath,
 			replacements: this.options.replacements || []
 		};
-		for (let hash of this.hashes) {
+		for (const hash of this.hashes) {
 			formatOptions.replacements!.push({ from: hash, to: "fullhash" });
 		}
 		return formatOptions;

packages/rspack-test-tools/src/processor/hook.ts

@@ -102,11 +102,11 @@ export class HookCasesContext extends TestContext {
 	 */
 	snapped(cb: (...args: unknown[]) => Promise<unknown>, prefix = "") {
 		// eslint-disable-next-line
-		let context = this;
+		const context = this;
 		return function SNAPPED_HOOK(this: any, ...args: unknown[]) {
-			let group = prefix ? prefix : context.count++;
+			const group = prefix ? prefix : context.count++;
 			context._addSnapshot(args, "input", group);
-			let output = cb.apply(this, args);
+			const output = cb.apply(this, args);
 			if (output && typeof output.then === "function") {
 				let resolve;
 				context.promises.push(new Promise(r => (resolve = r)));
@@ -157,11 +157,11 @@ export class HookCasesContext extends TestContext {
 		await Promise.allSettled(this.promises);
 		if (!this.snapshotsList.length) return;
 
-		let snapshots = this.snapshotsList.reduce((acc, group, index) => {
-			let block = this.snapshots[group || index].reduce(
+		const snapshots = this.snapshotsList.reduce((acc, group, index) => {
+			const block = this.snapshots[group || index].reduce(
 				(acc, [content, name]) => {
 					name = `## ${name || `test: ${index}`}\n\n`;
-					let block = "```javascript\n" + content + "\n```\n";
+					const block = "```javascript\n" + content + "\n```\n";
 					return (acc += name + block + "\n");
 				},
 				""