webpack 5.90.3 → 5.92.0

package/README.md

@@ -315,7 +315,7 @@ a question to [StackOverflow with the webpack tag](https://stackoverflow.com/tag
 
 If you are twitter savvy you can tweet #webpack with your question and someone should be able to reach out and help also.
 
-If you have discovered a 🐜 or have a feature suggestion, feel free to create an issue on Github.
+If you have discovered a 🐜 or have a feature suggestion, feel free to create an issue on GitHub.
 
 ### License
 
@@ -405,7 +405,7 @@ src="https://static.monei.net/monei-logo.svg" height="30" alt="MONEI"></a>
 
 <h2 align="center">Gold Sponsors</h2>
 
-[Become a gold sponsor](https://opencollective.com/webpack#sponsor) and get your logo on our README on Github with a link to your site.
+[Become a gold sponsor](https://opencollective.com/webpack#sponsor) and get your logo on our README on GitHub with a link to your site.
 
 <div align="center">
 
@@ -444,7 +444,7 @@ src="https://static.monei.net/monei-logo.svg" height="30" alt="MONEI"></a>
 
 <h2 align="center">Silver Sponsors</h2>
 
-[Become a silver sponsor](https://opencollective.com/webpack#sponsor) and get your logo on our README on Github with a link to your site.
+[Become a silver sponsor](https://opencollective.com/webpack#sponsor) and get your logo on our README on GitHub with a link to your site.
 
 <div align="center">
 
@@ -483,7 +483,7 @@ src="https://static.monei.net/monei-logo.svg" height="30" alt="MONEI"></a>
 
 <h2 align="center">Bronze Sponsors</h2>
 
-[Become a bronze sponsor](https://opencollective.com/webpack#sponsor) and get your logo on our README on Github with a link to your site.
+[Become a bronze sponsor](https://opencollective.com/webpack#sponsor) and get your logo on our README on GitHub with a link to your site.
 
 <div align="center">
 
@@ -593,7 +593,7 @@ src="https://static.monei.net/monei-logo.svg" height="30" alt="MONEI"></a>
 
 <h2 align="center">Backers</h2>
 
-[Become a backer](https://opencollective.com/webpack#backer) and get your image on our README on Github with a link to your site.
+[Become a backer](https://opencollective.com/webpack#backer) and get your image on our README on GitHub with a link to your site.
 
 <a href="https://opencollective.com/webpack/backer/0/website?requireActive=false" target="_blank"><img src="https://opencollective.com/webpack/backer/0/avatar.svg?requireActive=false"></a>
 <a href="https://opencollective.com/webpack/backer/1/website?requireActive=false" target="_blank"><img src="https://opencollective.com/webpack/backer/1/avatar.svg?requireActive=false"></a>

package/bin/webpack.js

@@ -79,7 +79,6 @@ const runCli = cli => {
 	const pkg = require(pkgPath);
 
 	if (pkg.type === "module" || /\.mjs/i.test(pkg.bin[cli.binName])) {
-		// eslint-disable-next-line n/no-unsupported-features/es-syntax
 		import(path.resolve(path.dirname(pkgPath), pkg.bin[cli.binName])).catch(
 			error => {
 				console.error(error);
@@ -92,7 +91,7 @@ const runCli = cli => {
 };
 
 /**
- * @typedef {Object} CliOption
+ * @typedef {object} CliOption
  * @property {string} name display name
  * @property {string} package npm package name
  * @property {string} binName name of the executable file
@@ -119,6 +118,7 @@ if (!cli.installed) {
 
 	console.error(notify);
 
+	/** @type {string | undefined} */
 	let packageManager;
 
 	if (fs.existsSync(path.resolve(process.cwd(), "yarn.lock"))) {
@@ -171,7 +171,10 @@ if (!cli.installed) {
 			}')...`
 		);
 
-		runCommand(packageManager, installOptions.concat(cli.package))
+		runCommand(
+			/** @type {string} */ (packageManager),
+			installOptions.concat(cli.package)
+		)
 			.then(() => {
 				runCli(cli);
 			})

package/lib/APIPlugin.js

@@ -25,6 +25,7 @@ const GetFullHashRuntimeModule = require("./runtime/GetFullHashRuntimeModule");
 
 /** @typedef {import("./Compiler")} Compiler */
 /** @typedef {import("./Dependency").DependencyLocation} DependencyLocation */
+/** @typedef {import("./Module").BuildInfo} BuildInfo */
 /** @typedef {import("./javascript/JavascriptParser")} JavascriptParser */
 /** @typedef {import("./javascript/JavascriptParser").Range} Range */
 
@@ -132,7 +133,7 @@ function getReplacements(module, importMetaName) {
 const PLUGIN_NAME = "APIPlugin";
 
 /**
- * @typedef {Object} APIPluginOptions
+ * @typedef {object} APIPluginOptions
  * @property {boolean} [module] the output filename
  */
 
@@ -187,10 +188,14 @@ class APIPlugin {
 				hooks.renderModuleContent.tap(
 					PLUGIN_NAME,
 					(source, module, renderContext) => {
-						if (module.buildInfo.needCreateRequire) {
+						if (/** @type {BuildInfo} */ (module.buildInfo).needCreateRequire) {
+							const needPrefix =
+								renderContext.runtimeTemplate.supportNodePrefixForCoreModules();
 							const chunkInitFragments = [
 								new InitFragment(
-									'import { createRequire as __WEBPACK_EXTERNAL_createRequire } from "module";\n',
+									`import { createRequire as __WEBPACK_EXTERNAL_createRequire } from "${
+										needPrefix ? "node:" : ""
+									}module";\n`,
 									InitFragment.STAGE_HARMONY_IMPORTS,
 									0,
 									"external module node-commonjs"
@@ -214,7 +219,8 @@ class APIPlugin {
 							const dep = toConstantDependency(parser, info.expr, info.req);
 
 							if (key === "__non_webpack_require__" && this.options.module) {
-								parser.state.module.buildInfo.needCreateRequire = true;
+								/** @type {BuildInfo} */
+								(parser.state.module.buildInfo).needCreateRequire = true;
 							}
 
 							return dep(expression);
@@ -267,7 +273,8 @@ class APIPlugin {
 					parser.hooks.expression
 						.for("__webpack_module__.id")
 						.tap(PLUGIN_NAME, expr => {
-							parser.state.module.buildInfo.moduleConcatenationBailout =
+							/** @type {BuildInfo} */
+							(parser.state.module.buildInfo).moduleConcatenationBailout =
 								"__webpack_module__.id";
 							const dep = new ConstDependency(
 								parser.state.module.moduleArgument + ".id",
@@ -282,7 +289,8 @@ class APIPlugin {
 					parser.hooks.expression
 						.for("__webpack_module__")
 						.tap(PLUGIN_NAME, expr => {
-							parser.state.module.buildInfo.moduleConcatenationBailout =
+							/** @type {BuildInfo} */
+							(parser.state.module.buildInfo).moduleConcatenationBailout =
 								"__webpack_module__";
 							const dep = new ConstDependency(
 								parser.state.module.moduleArgument,

package/lib/AutomaticPrefetchPlugin.js

@@ -27,7 +27,7 @@ class AutomaticPrefetchPlugin {
 				);
 			}
 		);
-		/** @type {{context: string, request: string}[] | null} */
+		/** @type {{context: string | null, request: string}[] | null} */
 		let lastModules = null;
 		compiler.hooks.afterCompile.tap("AutomaticPrefetchPlugin", compilation => {
 			lastModules = [];

package/lib/BannerPlugin.js

@@ -83,12 +83,14 @@ class BannerPlugin {
 			options
 		);
 		const cache = new WeakMap();
+		const stage =
+			this.options.stage || Compilation.PROCESS_ASSETS_STAGE_ADDITIONS;
 
 		compiler.hooks.compilation.tap("BannerPlugin", compilation => {
 			compilation.hooks.processAssets.tap(
 				{
 					name: "BannerPlugin",
-					stage: Compilation.PROCESS_ASSETS_STAGE_ADDITIONS
+					stage
 				},
 				() => {
 					for (const chunk of compilation.chunks) {

package/lib/Cache.js

@@ -14,14 +14,14 @@ const {
 /** @typedef {import("./WebpackError")} WebpackError */
 
 /**
- * @typedef {Object} Etag
+ * @typedef {object} Etag
  * @property {function(): string} toString
  */
 
 /**
  * @template T
  * @callback CallbackCache
- * @param {(WebpackError | null)=} err
+ * @param {WebpackError | null} err
  * @param {T=} result
  * @returns {void}
  */
@@ -33,6 +33,11 @@ const {
  * @returns {void}
  */
 
+/**
+ * @param {number} times times
+ * @param {function(Error=): void} callback callback
+ * @returns {function(Error=): void} callback
+ */
 const needCalls = (times, callback) => {
 	return err => {
 		if (--times === 0) {
@@ -71,6 +76,7 @@ class Cache {
 	 * @returns {void}
 	 */
 	get(identifier, etag, callback) {
+		/** @type {GotHandler[]} */
 		const gotHandlers = [];
 		this.hooks.get.callAsync(identifier, etag, gotHandlers, (err, result) => {
 			if (err) {

package/lib/CacheFacade.js

@@ -19,8 +19,8 @@ const mergeEtags = require("./cache/mergeEtags");
 /**
  * @template T
  * @callback CallbackCache
- * @param {(WebpackError | null)=} err
- * @param {T=} result
+ * @param {(Error | null)=} err
+ * @param {(T | null)=} result
  * @returns {void}
  */
 
@@ -196,7 +196,7 @@ class CacheFacade {
 	/**
 	 * @param {Cache} cache the root cache
 	 * @param {string} name the child cache name
-	 * @param {string | HashConstructor} hashFunction the hash function to use
+	 * @param {(string | HashConstructor)=} hashFunction the hash function to use
 	 */
 	constructor(cache, name, hashFunction) {
 		this._cache = cache;

package/lib/Chunk.js

@@ -38,13 +38,13 @@ const { mergeRuntime } = require("./util/runtime");
 const ChunkFilesSet = createArrayToSetDeprecationSet("chunk.files");
 
 /**
- * @typedef {Object} WithId an object who has an id property *
+ * @typedef {object} WithId an object who has an id property *
  * @property {string | number} id the id of the object
  */
 
 /**
  * @deprecated
- * @typedef {Object} ChunkMaps
+ * @typedef {object} ChunkMaps
  * @property {Record<string|number, string>} hash
  * @property {Record<string|number, Record<string, string>>} contentHash
  * @property {Record<string|number, string>} name
@@ -52,7 +52,7 @@ const ChunkFilesSet = createArrayToSetDeprecationSet("chunk.files");
 
 /**
  * @deprecated
- * @typedef {Object} ChunkModuleMaps
+ * @typedef {object} ChunkModuleMaps
  * @property {Record<string|number, (string|number)[]>} id
  * @property {Record<string|number, string>} hash
  */
@@ -85,7 +85,10 @@ class Chunk {
 		this.filenameTemplate = undefined;
 		/** @type {(string | function(PathData, AssetInfo=): string) | undefined} */
 		this.cssFilenameTemplate = undefined;
-		/** @private @type {SortableSet<ChunkGroup>} */
+		/**
+		 * @private
+		 * @type {SortableSet<ChunkGroup>}
+		 */
 		this._groups = new SortableSet(undefined, compareChunkGroupsByIndex);
 		/** @type {RuntimeSpec} */
 		this.runtime = undefined;

package/lib/ChunkGraph.js

@@ -33,6 +33,7 @@ const {
 /** @typedef {import("./ChunkGroup")} ChunkGroup */
 /** @typedef {import("./Module")} Module */
 /** @typedef {import("./ModuleGraph")} ModuleGraph */
+/** @typedef {import("./ModuleGraphConnection").ConnectionState} ConnectionState */
 /** @typedef {import("./RuntimeModule")} RuntimeModule */
 /** @typedef {typeof import("./util/Hash")} Hash */
 /** @typedef {import("./util/runtime").RuntimeSpec} RuntimeSpec */
@@ -49,12 +50,16 @@ const compareModuleIterables = compareIterables(compareModulesByIdentifier);
 /** @typedef {[Module, Entrypoint | undefined]} EntryModuleWithChunkGroup */
 
 /**
- * @typedef {Object} ChunkSizeOptions
+ * @typedef {object} ChunkSizeOptions
  * @property {number=} chunkOverhead constant overhead for a chunk
  * @property {number=} entryChunkMultiplicator multiplicator for initial chunks
  */
 
 class ModuleHashInfo {
+	/**
+	 * @param {string} hash hash
+	 * @param {string} renderedHash  rendered hash
+	 */
 	constructor(hash, renderedHash) {
 		this.hash = hash;
 		this.renderedHash = renderedHash;
@@ -180,13 +185,17 @@ const isAvailableChunk = (a, b) => {
 	return true;
 };
 
+/** @typedef {Set<Chunk>} EntryInChunks */
+/** @typedef {Set<Chunk>} RuntimeInChunks */
+/** @typedef {string | number} ModuleId */
+
 class ChunkGraphModule {
 	constructor() {
 		/** @type {SortableSet<Chunk>} */
 		this.chunks = new SortableSet();
-		/** @type {Set<Chunk> | undefined} */
+		/** @type {EntryInChunks | undefined} */
 		this.entryInChunks = undefined;
-		/** @type {Set<Chunk> | undefined} */
+		/** @type {RuntimeInChunks | undefined} */
 		this.runtimeInChunks = undefined;
 		/** @type {RuntimeSpecMap<ModuleHashInfo> | undefined} */
 		this.hashes = undefined;
@@ -194,9 +203,9 @@ class ChunkGraphModule {
 		this.id = null;
 		/** @type {RuntimeSpecMap<Set<string>> | undefined} */
 		this.runtimeRequirements = undefined;
-		/** @type {RuntimeSpecMap<string>} */
+		/** @type {RuntimeSpecMap<string> | undefined} */
 		this.graphHashes = undefined;
-		/** @type {RuntimeSpecMap<string>} */
+		/** @type {RuntimeSpecMap<string> | undefined} */
 		this.graphHashesWithConnections = undefined;
 	}
 }
@@ -230,13 +239,25 @@ class ChunkGraph {
 	 * @param {string | Hash} hashFunction the hash function to use
 	 */
 	constructor(moduleGraph, hashFunction = "md4") {
-		/** @private @type {WeakMap<Module, ChunkGraphModule>} */
+		/**
+		 * @private
+		 * @type {WeakMap<Module, ChunkGraphModule>}
+		 */
 		this._modules = new WeakMap();
-		/** @private @type {WeakMap<Chunk, ChunkGraphChunk>} */
+		/**
+		 * @private
+		 * @type {WeakMap<Chunk, ChunkGraphChunk>}
+		 */
 		this._chunks = new WeakMap();
-		/** @private @type {WeakMap<AsyncDependenciesBlock, ChunkGroup>} */
+		/**
+		 * @private
+		 * @type {WeakMap<AsyncDependenciesBlock, ChunkGroup>}
+		 */
 		this._blockChunkGroups = new WeakMap();
-		/** @private @type {Map<string, string | number>} */
+		/**
+		 * @private
+		 * @type {Map<string, string | number>}
+		 */
 		this._runtimeIds = new Map();
 		/** @type {ModuleGraph} */
 		this.moduleGraph = moduleGraph;
@@ -284,6 +305,9 @@ class ChunkGraph {
 			findGraphRoots(set, module => {
 				/** @type {Set<Module>} */
 				const set = new Set();
+				/**
+				 * @param {Module} module module
+				 */
 				const addDependencies = module => {
 					for (const connection of moduleGraph.getOutgoingConnections(module)) {
 						if (!connection.module) continue;
@@ -417,7 +441,7 @@ class ChunkGraph {
 			}
 			for (const chunk of oldCgm.entryInChunks) {
 				const cgc = this._getChunkGraphChunk(chunk);
-				const old = cgc.entryModules.get(oldModule);
+				const old = /** @type {Entrypoint} */ (cgc.entryModules.get(oldModule));
 				/** @type {Map<Module, Entrypoint>} */
 				const newEntryModules = new Map();
 				for (const [m, cg] of cgc.entryModules) {
@@ -716,7 +740,7 @@ class ChunkGraph {
 		for (const asyncChunk of includeAllChunks
 			? chunk.getAllReferencedChunks()
 			: chunk.getAllAsyncChunks()) {
-			/** @type {(string|number)[]} */
+			/** @type {(string | number)[] | undefined} */
 			let array;
 			for (const module of this.getOrderedChunkModulesIterable(
 				asyncChunk,
@@ -755,7 +779,7 @@ class ChunkGraph {
 		for (const asyncChunk of includeAllChunks
 			? chunk.getAllReferencedChunks()
 			: chunk.getAllAsyncChunks()) {
-			/** @type {Record<string|number, string>} */
+			/** @type {Record<string|number, string> | undefined} */
 			let idToHashMap;
 			for (const module of this.getOrderedChunkModulesIterable(
 				asyncChunk,
@@ -1115,7 +1139,7 @@ class ChunkGraph {
 	 */
 	disconnectEntryModule(module) {
 		const cgm = this._getChunkGraphModule(module);
-		for (const chunk of cgm.entryInChunks) {
+		for (const chunk of /** @type {EntryInChunks} */ (cgm.entryInChunks)) {
 			const cgc = this._getChunkGraphChunk(chunk);
 			cgc.entryModules.delete(module);
 		}
@@ -1223,14 +1247,7 @@ class ChunkGraph {
 		const array = Array.from(cgc.runtimeModules);
 		array.sort(
 			concatComparators(
-				compareSelect(
-					/**
-					 * @param {RuntimeModule} r runtime module
-					 * @returns {number=} stage
-					 */
-					r => r.stage,
-					compareIds
-				),
+				compareSelect(r => /** @type {RuntimeModule} */ (r).stage, compareIds),
 				compareModulesByIdentifier
 			)
 		);
@@ -1275,7 +1292,7 @@ class ChunkGraph {
 
 	/**
 	 * @param {AsyncDependenciesBlock} depBlock the async block
-	 * @returns {ChunkGroup} the chunk group
+	 * @returns {ChunkGroup | undefined} the chunk group
 	 */
 	getBlockChunkGroup(depBlock) {
 		return this._blockChunkGroups.get(depBlock);
@@ -1305,7 +1322,7 @@ class ChunkGraph {
 
 	/**
 	 * @param {Module} module the module
-	 * @returns {string | number} the id of the module
+	 * @returns {ModuleId} the id of the module
 	 */
 	getModuleId(module) {
 		const cgm = this._getChunkGraphModule(module);
@@ -1314,7 +1331,7 @@ class ChunkGraph {
 
 	/**
 	 * @param {Module} module the module
-	 * @param {string | number} id the id of the module
+	 * @param {ModuleId} id the id of the module
 	 * @returns {void}
 	 */
 	setModuleId(module, id) {
@@ -1364,7 +1381,7 @@ class ChunkGraph {
 Caller might not support runtime-dependent code generation (opt-out via optimization.usedExports: "global").`
 				);
 			}
-			return first(hashInfoItems);
+			return /** @type {T} */ (first(hashInfoItems));
 		} else {
 			const hashInfo = hashes.get(runtime);
 			if (!hashInfo) {
@@ -1576,6 +1593,10 @@ Caller might not support runtime-dependent code generation (opt-out via optimiza
 		if (cgm.graphHashesWithConnections === undefined) {
 			cgm.graphHashesWithConnections = new RuntimeSpecMap();
 		}
+		/**
+		 * @param {ConnectionState} state state
+		 * @returns {"F" | "T" | "O"} result
+		 */
 		const activeStateToString = state => {
 			if (state === false) return "F";
 			if (state === true) return "T";
@@ -1646,6 +1667,9 @@ Caller might not support runtime-dependent code generation (opt-out via optimiza
 					? Array.from(connectedModules).sort(([a], [b]) => (a < b ? -1 : 1))
 					: connectedModules;
 			const hash = createHash(this._hashFunction);
+			/**
+			 * @param {Module} module module
+			 */
 			const addModuleToHash = module => {
 				hash.update(
 					this._getModuleGraphHashBigInt(
@@ -1655,6 +1679,9 @@ Caller might not support runtime-dependent code generation (opt-out via optimiza
 					).toString(16)
 				);
 			};
+			/**
+			 * @param {Set<Module>} modules modules
+			 */
 			const addModulesToHash = modules => {
 				let xor = ZERO_BIG_INT;
 				for (const m of modules) {

package/lib/ChunkGroup.js

@@ -25,7 +25,7 @@ const {
 /** @typedef {{module: Module, loc: DependencyLocation, request: string}} OriginRecord */
 
 /**
- * @typedef {Object} RawChunkGroupOptions
+ * @typedef {object} RawChunkGroupOptions
  * @property {number=} preloadOrder
  * @property {number=} prefetchOrder
  * @property {("low" | "high" | "auto")=} fetchPriority
@@ -70,7 +70,7 @@ const sortOrigin = (a, b) => {
 class ChunkGroup {
 	/**
 	 * Creates an instance of ChunkGroup.
-	 * @param {string|ChunkGroupOptions=} options chunk group options passed to chunkGroup
+	 * @param {string | ChunkGroupOptions=} options chunk group options passed to chunkGroup
 	 */
 	constructor(options) {
 		if (typeof options === "string") {
@@ -80,7 +80,7 @@ class ChunkGroup {
 		}
 		/** @type {number} */
 		this.groupDebugId = debugId++;
-		this.options = options;
+		this.options = /** @type {ChunkGroupOptions} */ (options);
 		/** @type {SortableSet<ChunkGroup>} */
 		this._children = new SortableSet(undefined, sortById);
 		/** @type {SortableSet<ChunkGroup>} */
@@ -93,10 +93,16 @@ class ChunkGroup {
 		/** @type {OriginRecord[]} */
 		this.origins = [];
 		/** Indices in top-down order */
-		/** @private @type {Map<Module, number>} */
+		/**
+		 * @private
+		 * @type {Map<Module, number>}
+		 */
 		this._modulePreOrderIndices = new Map();
 		/** Indices in bottom-up order */
-		/** @private @type {Map<Module, number>} */
+		/**
+		 * @private
+		 * @type {Map<Module, number>}
+		 */
 		this._modulePostOrderIndices = new Map();
 		/** @type {number | undefined} */
 		this.index = undefined;
@@ -108,18 +114,18 @@ class ChunkGroup {
 	 * @returns {void}
 	 */
 	addOptions(options) {
-		for (const key of Object.keys(options)) {
-			if (
-				this.options[/** @type {keyof ChunkGroupOptions} */ (key)] === undefined
-			) {
-				this.options[key] =
-					options[/** @type {keyof ChunkGroupOptions} */ (key)];
-			} else if (
-				this.options[/** @type {keyof ChunkGroupOptions} */ (key)] !==
-				options[/** @type {keyof ChunkGroupOptions} */ (key)]
-			) {
+		for (const _key of Object.keys(options)) {
+			const key = /** @type {keyof ChunkGroupOptions} */ (_key);
+			if (this.options[key] === undefined) {
+				/** @type {TODO} */
+				(this.options)[key] = options[key];
+			} else if (this.options[key] !== options[key]) {
 				if (key.endsWith("Order")) {
-					this.options[key] = Math.max(this.options[key], options[key]);
+					/** @type {TODO} */
+					(this.options)[key] = Math.max(
+						/** @type {number} */ (this.options[key]),
+						/** @type {number} */ (options[key])
+					);
 				} else {
 					throw new Error(
 						`ChunkGroup.addOptions: No option merge strategy for ${key}`
@@ -219,7 +225,7 @@ class ChunkGroup {
 	/**
 	 * @param {Chunk} oldChunk chunk to be replaced
 	 * @param {Chunk} newChunk New chunk that will be replaced with
-	 * @returns {boolean} returns true if the replacement was successful
+	 * @returns {boolean | undefined} returns true if the replacement was successful
 	 */
 	replaceChunk(oldChunk, newChunk) {
 		const oldIdx = this.chunks.indexOf(oldChunk);

package/lib/CleanPlugin.js

@@ -15,6 +15,7 @@ const processAsyncTree = require("./util/processAsyncTree");
 /** @typedef {import("../declarations/WebpackOptions").CleanOptions} CleanOptions */
 /** @typedef {import("./Compiler")} Compiler */
 /** @typedef {import("./logging/Logger").Logger} Logger */
+/** @typedef {import("./util/fs").IStats} IStats */
 /** @typedef {import("./util/fs").OutputFileSystem} OutputFileSystem */
 /** @typedef {import("./util/fs").StatsCallback} StatsCallback */
 
@@ -23,7 +24,7 @@ const processAsyncTree = require("./util/processAsyncTree");
 /** @typedef {function(IgnoreItem): void} AddToIgnoreCallback */
 
 /**
- * @typedef {Object} CleanPluginCompilationHooks
+ * @typedef {object} CleanPluginCompilationHooks
  * @property {SyncBailHook<[string], boolean>} keep when returning true the file/directory will be kept during cleaning, returning false will clean it and ignore the following plugins and config
  */
 
@@ -88,8 +89,8 @@ const getDiffToFs = (fs, outputPath, currentAssets, callback) => {
 					}
 					return callback(err);
 				}
-				for (const entry of entries) {
-					const file = /** @type {string} */ (entry);
+				for (const entry of /** @type {string[]} */ (entries)) {
+					const file = entry;
 					const filename = directory ? `${directory}/${file}` : file;
 					if (!directories.has(filename) && !currentAssets.has(filename)) {
 						diff.add(filename);
@@ -196,7 +197,7 @@ const applyDiff = (fs, outputPath, dry, logger, diff, isKept, callback) => {
 					}
 					doStat(fs, path, (err, stats) => {
 						if (err) return handleError(err);
-						if (!stats.isDirectory()) {
+						if (!(/** @type {IStats} */ (stats).isDirectory())) {
 							push({
 								type: "unlink",
 								filename,
@@ -206,7 +207,7 @@ const applyDiff = (fs, outputPath, dry, logger, diff, isKept, callback) => {
 						}
 
 						/** @type {NonNullable<OutputFileSystem["readdir"]>} */
-						(fs.readdir)(path, (err, entries) => {
+						(fs.readdir)(path, (err, _entries) => {
 							if (err) return handleError(err);
 							/** @type {Job} */
 							const deleteJob = {
@@ -214,6 +215,7 @@ const applyDiff = (fs, outputPath, dry, logger, diff, isKept, callback) => {
 								filename,
 								parent
 							};
+							const entries = /** @type {string[]} */ (_entries);
 							if (entries.length === 0) {
 								push(deleteJob);
 							} else {
@@ -355,7 +357,7 @@ class CleanPlugin {
 			(compilation, callback) => {
 				const hooks = CleanPlugin.getCompilationHooks(compilation);
 				const logger = compilation.getLogger("webpack.CleanPlugin");
-				const fs = compiler.outputFileSystem;
+				const fs = /** @type {OutputFileSystem} */ (compiler.outputFileSystem);
 
 				if (!fs.readdir) {
 					return callback(