webpack 5.100.2 → 5.101.1

package/lib/Compilation.js

@@ -372,7 +372,7 @@ const { isSourceEqual } = require("./util/source");
  * @property {false | "none" | "error" | "warn" | "info" | "log" | "verbose"} logging
  * @property {((value: string) => boolean)[]} loggingDebug
  * @property {boolean} loggingTrace
- * @property {TODO} _env
+ * @property {EXPECTED_ANY} _env
  */
 
 /** @typedef {KnownNormalizedStatsOptions & Omit<StatsOptions, keyof KnownNormalizedStatsOptions> & Record<string, EXPECTED_ANY>} NormalizedStatsOptions */
@@ -403,7 +403,7 @@ const deprecatedNormalModuleLoaderHook = util.deprecate(
 	 * @param {Compilation} compilation compilation
 	 * @returns {NormalModuleCompilationHooks["loader"]} hooks
 	 */
-	compilation =>
+	(compilation) =>
 		require("./NormalModule").getCompilationHooks(compilation).loader,
 	"Compilation.hooks.normalModuleLoader was moved to NormalModule.getCompilationHooks(compilation).loader",
 	"DEP_WEBPACK_COMPILATION_NORMAL_MODULE_LOADER_HOOK"
@@ -413,7 +413,7 @@ const deprecatedNormalModuleLoaderHook = util.deprecate(
 /**
  * @param {ModuleTemplates | undefined} moduleTemplates module templates
  */
-const defineRemovedModuleTemplates = moduleTemplates => {
+const defineRemovedModuleTemplates = (moduleTemplates) => {
 	Object.defineProperties(moduleTemplates, {
 		asset: {
 			enumerable: false,
@@ -437,21 +437,24 @@ const defineRemovedModuleTemplates = moduleTemplates => {
 	moduleTemplates = undefined;
 };
 
-const byId = compareSelect(c => c.id, compareIds);
+const byId = compareSelect((c) => c.id, compareIds);
 
 const byNameOrHash = concatComparators(
-	compareSelect(c => c.name, compareIds),
-	compareSelect(c => c.fullHash, compareIds)
+	compareSelect((c) => c.name, compareIds),
+	compareSelect((c) => c.fullHash, compareIds)
 );
 
-const byMessage = compareSelect(err => `${err.message}`, compareStringsNumeric);
+const byMessage = compareSelect(
+	(err) => `${err.message}`,
+	compareStringsNumeric
+);
 
 const byModule = compareSelect(
-	err => (err.module && err.module.identifier()) || "",
+	(err) => (err.module && err.module.identifier()) || "",
 	compareStringsNumeric
 );
 
-const byLocation = compareSelect(err => err.loc, compareLocations);
+const byLocation = compareSelect((err) => err.loc, compareLocations);
 
 const compareErrors = concatComparators(byModule, byLocation, byMessage);
 
@@ -497,7 +500,7 @@ class Compilation {
 		 * @param {CompilationAssets} assets assets
 		 * @returns {CompilationAssets} new assets
 		 */
-		const popNewAssets = assets => {
+		const popNewAssets = (assets) => {
 			let newAssets;
 			for (const file of Object.keys(assets)) {
 				if (savedAssets.has(file)) continue;
@@ -514,7 +517,7 @@ class Compilation {
 			call: () => {
 				savedAssets = new Set(Object.keys(this.assets));
 			},
-			register: tap => {
+			register: (tap) => {
 				const { type, name } = tap;
 				const { fn, additionalAssets, ...remainingTap } = tap;
 				const additionalAssetsFn =
@@ -523,15 +526,30 @@ class Compilation {
 
 				/** @type {ProcessedAssets | undefined} */
 				const processedAssets = additionalAssetsFn ? new WeakSet() : undefined;
+				/**
+				 * @param {CompilationAssets} assets to be processed by additionalAssetsFn
+				 * @returns {CompilationAssets} available assets
+				 */
+				const getAvailableAssets = (assets) => {
+					/** @type {CompilationAssets} */
+					const availableAssets = {};
+					for (const file of Object.keys(assets)) {
+						// https://github.com/webpack-contrib/compression-webpack-plugin/issues/390
+						if (this.assets[file]) {
+							availableAssets[file] = assets[file];
+						}
+					}
+					return availableAssets;
+				};
 				switch (type) {
 					case "sync":
 						if (additionalAssetsFn) {
-							this.hooks.processAdditionalAssets.tap(name, assets => {
+							this.hooks.processAdditionalAssets.tap(name, (assets) => {
 								if (
 									/** @type {ProcessedAssets} */
 									(processedAssets).has(this.assets)
 								) {
-									additionalAssetsFn(assets);
+									additionalAssetsFn(getAvailableAssets(assets));
 								}
 							});
 						}
@@ -572,7 +590,10 @@ class Compilation {
 										/** @type {ProcessedAssets} */
 										(processedAssets).has(this.assets)
 									) {
-										return additionalAssetsFn(assets, callback);
+										return additionalAssetsFn(
+											getAvailableAssets(assets),
+											callback
+										);
 									}
 									callback();
 								}
@@ -592,7 +613,7 @@ class Compilation {
 									 * @param {Error} err err
 									 * @returns {void}
 									 */
-									err => {
+									(err) => {
 										if (err) return callback(err);
 										if (processedAssets !== undefined) {
 											processedAssets.add(this.assets);
@@ -612,12 +633,12 @@ class Compilation {
 						};
 					case "promise":
 						if (additionalAssetsFn) {
-							this.hooks.processAdditionalAssets.tapPromise(name, assets => {
+							this.hooks.processAdditionalAssets.tapPromise(name, (assets) => {
 								if (
 									/** @type {ProcessedAssets} */
 									(processedAssets).has(this.assets)
 								) {
-									return additionalAssetsFn(assets);
+									return additionalAssetsFn(getAvailableAssets(assets));
 								}
 								return Promise.resolve();
 							});
@@ -628,7 +649,7 @@ class Compilation {
 							 * @param {CompilationAssets} assets assets
 							 * @returns {Promise<CompilationAssets>} result
 							 */
-							fn: assets => {
+							fn: (assets) => {
 								const p = fn(assets);
 								if (!p || !p.then) return p;
 								return p.then(() => {
@@ -665,14 +686,15 @@ class Compilation {
 			 * @param {string} reason reason
 			 * @returns {string} error message
 			 */
-			const errorMessage =
-				reason => `Can't automatically convert plugin using Compilation.hooks.${name} to Compilation.hooks.processAssets because ${reason}.
+			const errorMessage = (
+				reason
+			) => `Can't automatically convert plugin using Compilation.hooks.${name} to Compilation.hooks.processAssets because ${reason}.
 BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a single Compilation.hooks.processAssets hook.`;
 			/**
 			 * @param {string | (import("tapable").TapOptions & { name: string; } & ProcessAssetsAdditionalOptions)} options hook options
 			 * @returns {import("tapable").TapOptions & { name: string; } & ProcessAssetsAdditionalOptions} modified options
 			 */
-			const getOptions = options => {
+			const getOptions = (options) => {
 				if (typeof options === "string") options = { name: options };
 				if (options.stage) {
 					throw new Error(errorMessage("it's using the 'stage' option"));
@@ -1075,7 +1097,7 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 		this.addModuleQueue = new AsyncQueue({
 			name: "addModule",
 			parent: this.processDependenciesQueue,
-			getKey: module => module.identifier(),
+			getKey: (module) => module.identifier(),
 			processor: this._addModule.bind(this)
 		});
 		/** @type {AsyncQueue<FactorizeModuleOptions, string, Module | ModuleFactoryResult>} */
@@ -1199,7 +1221,7 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 				 * @param {string} item item
 				 * @returns {LazySet<string>} file dependencies
 				 */
-				item => this.fileDependencies.add(item),
+				(item) => this.fileDependencies.add(item),
 				"Compilation.compilationDependencies is deprecated (used Compilation.fileDependencies instead)",
 				"DEP_WEBPACK_COMPILATION_COMPILATION_DEPENDENCIES"
 			)
@@ -1351,7 +1373,7 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 					/* eslint-enable no-console */
 				}
 			},
-			childName => {
+			(childName) => {
 				if (typeof name === "function") {
 					if (typeof childName === "function") {
 						return this.getLogger(() => {
@@ -1527,7 +1549,7 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 					this.resolverFactory.get("normal", module.resolveOptions),
 					/** @type {InputFileSystem} */
 					(this.inputFileSystem),
-					err => {
+					(err) => {
 						if (currentProfile !== undefined) {
 							currentProfile.markBuildingEnd();
 						}
@@ -1538,20 +1560,25 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 						if (currentProfile !== undefined) {
 							currentProfile.markStoringStart();
 						}
-						this._modulesCache.store(module.identifier(), null, module, err => {
-							if (currentProfile !== undefined) {
-								currentProfile.markStoringEnd();
-							}
-							if (err) {
-								this.hooks.failedModule.call(
-									module,
-									/** @type {WebpackError} */ (err)
-								);
-								return callback(new ModuleStoreError(module, err));
+						this._modulesCache.store(
+							module.identifier(),
+							null,
+							module,
+							(err) => {
+								if (currentProfile !== undefined) {
+									currentProfile.markStoringEnd();
+								}
+								if (err) {
+									this.hooks.failedModule.call(
+										module,
+										/** @type {WebpackError} */ (err)
+									);
+									return callback(new ModuleStoreError(module, err));
+								}
+								this.hooks.succeedModule.call(module);
+								return callback();
 							}
-							this.hooks.succeedModule.call(module);
-							return callback();
-						});
+						);
 					}
 				);
 			}
@@ -1575,7 +1602,7 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 		/**
 		 * @param {DependenciesBlock} block block
 		 */
-		const processDependenciesBlock = block => {
+		const processDependenciesBlock = (block) => {
 			if (block.dependencies) {
 				let i = 0;
 				for (const dep of block.dependencies) {
@@ -1625,7 +1652,7 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 		 * @param {WebpackError=} err error
 		 * @returns {void}
 		 */
-		const onDependenciesSorted = err => {
+		const onDependenciesSorted = (err) => {
 			if (err) return callback(err);
 
 			// early exit without changing parallelism back and forth
@@ -1639,7 +1666,7 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 			for (const item of sortedDependencies) {
 				inProgressTransitive++;
 				// eslint-disable-next-line no-loop-func
-				this.handleModuleCreation(item, err => {
+				this.handleModuleCreation(item, (err) => {
 					// In V8, the Error objects keep a reference to the functions on the stack. These warnings &
 					// errors are created inside closures that keep a reference to the Compilation, so errors are
 					// leaking the Compilation object.
@@ -1661,7 +1688,7 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 		 * @param {WebpackError=} err error
 		 * @returns {void}
 		 */
-		const onTransitiveTasksFinished = err => {
+		const onTransitiveTasksFinished = (err) => {
 			if (err) return callback(err);
 			this.processDependenciesQueue.decreaseParallelism();
 
@@ -1738,7 +1765,7 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 											module,
 											dep,
 											cachedModule,
-											err => {
+											(err) => {
 												if (err) {
 													if (inProgressTransitive <= 0) return;
 													inProgressTransitive = -1;
@@ -1783,7 +1810,7 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 		 * @param {Dependency} dep dependency
 		 * @returns {void}
 		 */
-		const processDependencyForResolving = dep => {
+		const processDependencyForResolving = (dep) => {
 			const resourceIdent = dep.getResourceIdentifier();
 			if (resourceIdent !== undefined && resourceIdent !== null) {
 				const category = dep.category;
@@ -1982,7 +2009,7 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 				};
 				if (err) {
 					if (factoryResult) applyFactoryResultDependencies();
-					if (dependencies.every(d => d.optional)) {
+					if (dependencies.every((d) => d.optional)) {
 						this.warnings.push(err);
 						return callback();
 					}
@@ -2136,7 +2163,7 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 			}
 		}
 
-		this.buildModule(module, err => {
+		this.buildModule(module, (err) => {
 			if (creatingModuleDuringBuildSet !== undefined) {
 				creatingModuleDuringBuildSet.delete(module);
 			}
@@ -2160,7 +2187,7 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 				return callback(null, module);
 			}
 
-			this.processModuleDependencies(module, err => {
+			this.processModuleDependencies(module, (err) => {
 				if (err) {
 					return callback(err);
 				}
@@ -2238,7 +2265,7 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 						originModule,
 						err,
 						/** @type {DependencyLocation} */
-						(dependencies.map(d => d.loc).find(Boolean))
+						(dependencies.map((d) => d.loc).find(Boolean))
 					);
 					return callback(notFoundError, factoryResult ? result : undefined);
 				}
@@ -2453,9 +2480,9 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 		const oldBlocks = [...module.blocks];
 		module.invalidateBuild();
 		this.buildQueue.invalidate(module);
-		this.buildModule(module, err => {
+		this.buildModule(module, (err) => {
 			if (err) {
-				return this.hooks.finishRebuildingModule.callAsync(module, err2 => {
+				return this.hooks.finishRebuildingModule.callAsync(module, (err2) => {
 					if (err2) {
 						callback(
 							makeWebpackError(err2, "Compilation.hooks.finishRebuildingModule")
@@ -2468,13 +2495,13 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 
 			this.processDependenciesQueue.invalidate(module);
 			this.moduleGraph.unfreeze();
-			this.processModuleDependencies(module, err => {
+			this.processModuleDependencies(module, (err) => {
 				if (err) return callback(err);
 				this.removeReasonsOfDependencyBlock(module, {
 					dependencies: oldDependencies,
 					blocks: oldBlocks
 				});
-				this.hooks.finishRebuildingModule.callAsync(module, err2 => {
+				this.hooks.finishRebuildingModule.callAsync(module, (err2) => {
 					if (err2) {
 						callback(
 							makeWebpackError(err2, "Compilation.hooks.finishRebuildingModule")
@@ -2511,7 +2538,7 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 		 * @param {Module} module module
 		 * @returns {WeakReferences | undefined} references
 		 */
-		const computeReferences = module => {
+		const computeReferences = (module) => {
 			/** @type {WeakReferences | undefined} */
 			let references;
 			for (const connection of moduleGraph.getOutgoingConnections(module)) {
@@ -2604,7 +2631,7 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 		 * @param {readonly ModuleGraphConnection[]} connections connections
 		 * @returns {symbol|boolean} result
 		 */
-		const reduceAffectType = connections => {
+		const reduceAffectType = (connections) => {
 			let affected = false;
 			for (const { dependency } of connections) {
 				if (!dependency) continue;
@@ -2684,7 +2711,7 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 		 * @param {Module} module module
 		 * @returns {References} references
 		 */
-		const computeReferences = module => {
+		const computeReferences = (module) => {
 			const id = /** @type {ModuleId} */ (chunkGraph.getModuleId(module));
 			/** @type {Map<Module, string | number | undefined> | undefined} */
 			let modules;
@@ -2804,27 +2831,27 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 				p.range(
 					profile.buildingStartTime,
 					profile.buildingEndTime,
-					f => (profile.buildingParallelismFactor = f)
+					(f) => (profile.buildingParallelismFactor = f)
 				);
 				p.range(
 					profile.factoryStartTime,
 					profile.factoryEndTime,
-					f => (profile.factoryParallelismFactor = f)
+					(f) => (profile.factoryParallelismFactor = f)
 				);
 				p.range(
 					profile.integrationStartTime,
 					profile.integrationEndTime,
-					f => (profile.integrationParallelismFactor = f)
+					(f) => (profile.integrationParallelismFactor = f)
 				);
 				p.range(
 					profile.storingStartTime,
 					profile.storingEndTime,
-					f => (profile.storingParallelismFactor = f)
+					(f) => (profile.storingParallelismFactor = f)
 				);
 				p.range(
 					profile.restoringStartTime,
 					profile.restoringEndTime,
-					f => (profile.restoringParallelismFactor = f)
+					(f) => (profile.restoringParallelismFactor = f)
 				);
 				if (profile.additionalFactoryTimes) {
 					for (const { start, end } of profile.additionalFactoryTimes) {
@@ -2832,7 +2859,7 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 						p.range(
 							start,
 							end,
-							f =>
+							(f) =>
 								(profile.additionalFactoriesParallelismFactor += f * influence)
 						);
 					}
@@ -2955,33 +2982,33 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 			};
 			logNormalSummary(
 				"resolve to new modules",
-				p => p.factory,
-				p => p.factoryParallelismFactor
+				(p) => p.factory,
+				(p) => p.factoryParallelismFactor
 			);
 			logNormalSummary(
 				"resolve to existing modules",
-				p => p.additionalFactories,
-				p => p.additionalFactoriesParallelismFactor
+				(p) => p.additionalFactories,
+				(p) => p.additionalFactoriesParallelismFactor
 			);
 			logNormalSummary(
 				"integrate modules",
-				p => p.restoring,
-				p => p.restoringParallelismFactor
+				(p) => p.restoring,
+				(p) => p.restoringParallelismFactor
 			);
 			logByLoadersSummary(
 				"build modules",
-				p => p.building,
-				p => p.buildingParallelismFactor
+				(p) => p.building,
+				(p) => p.buildingParallelismFactor
 			);
 			logNormalSummary(
 				"store modules",
-				p => p.storing,
-				p => p.storingParallelismFactor
+				(p) => p.storing,
+				(p) => p.storingParallelismFactor
 			);
 			logNormalSummary(
 				"restore modules",
-				p => p.restoring,
-				p => p.restoringParallelismFactor
+				(p) => p.restoring,
+				(p) => p.restoringParallelismFactor
 			);
 			this.logger.timeEnd("finish module profiles");
 		}
@@ -2990,7 +3017,7 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 		this.logger.timeEnd("compute affected modules");
 		this.logger.time("finish modules");
 		const { modules, moduleMemCaches } = this;
-		this.hooks.finishModules.callAsync(modules, err => {
+		this.hooks.finishModules.callAsync(modules, (err) => {
 			this.logger.timeEnd("finish modules");
 			if (err) return callback(/** @type {WebpackError} */ (err));
 
@@ -3061,7 +3088,7 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 		 * @param {WebpackError=} err err
 		 * @returns {void}
 		 */
-		const finalCallback = err => {
+		const finalCallback = (err) => {
 			this.factorizeQueue.clear();
 			this.buildQueue.clear();
 			this.rebuildQueue.clear();
@@ -3139,11 +3166,11 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 			 * @param {Dependency[]} deps deps
 			 * @returns {Module[]} sorted deps
 			 */
-			const mapAndSort = deps =>
+			const mapAndSort = (deps) =>
 				/** @type {Module[]} */
-				(deps.map(dep => this.moduleGraph.getModule(dep)).filter(Boolean)).sort(
-					compareModulesByIdentifier
-				);
+				(
+					deps.map((dep) => this.moduleGraph.getModule(dep)).filter(Boolean)
+				).sort(compareModulesByIdentifier);
 			const includedModules = [
 				...mapAndSort(this.globalEntry.includeDependencies),
 				...mapAndSort(includeDependencies)
@@ -3233,6 +3260,7 @@ Or do you want to use the entrypoints '${name}' and '${runtime}' independently o
 				entry.setRuntimeChunk(chunk);
 			}
 		}
+
 		buildChunkGraph(this, chunkGraphInit);
 		this.hooks.afterChunks.call(this.chunks);
 		this.logger.timeEnd("create chunks");
@@ -3250,7 +3278,27 @@ Or do you want to use the entrypoints '${name}' and '${runtime}' independently o
 		}
 		this.hooks.afterOptimizeChunks.call(this.chunks, this.chunkGroups);
 
-		this.hooks.optimizeTree.callAsync(this.chunks, this.modules, err => {
+		for (const [
+			name,
+			{
+				options: { dependOn }
+			}
+		] of this.entries) {
+			if (dependOn) {
+				const entry = /** @type {Entrypoint} */ (this.entrypoints.get(name));
+				for (const dep of dependOn) {
+					const depEntry = /** @type {Entrypoint} */ (
+						this.entrypoints.get(dep)
+					);
+					const runtimeChunk = depEntry.getRuntimeChunk();
+					if (runtimeChunk) {
+						runtimeChunk.addGroup(entry);
+					}
+				}
+			}
+		}
+
+		this.hooks.optimizeTree.callAsync(this.chunks, this.modules, (err) => {
 			if (err) {
 				return finalCallback(
 					makeWebpackError(err, "Compilation.hooks.optimizeTree")
@@ -3262,7 +3310,7 @@ Or do you want to use the entrypoints '${name}' and '${runtime}' independently o
 			this.hooks.optimizeChunkModules.callAsync(
 				this.chunks,
 				this.modules,
-				err => {
+				(err) => {
 					if (err) {
 						return finalCallback(
 							makeWebpackError(err, "Compilation.hooks.optimizeChunkModules")
@@ -3325,7 +3373,7 @@ Or do you want to use the entrypoints '${name}' and '${runtime}' independently o
 
 					this.logger.time("code generation");
 					this.hooks.beforeCodeGeneration.call();
-					this.codeGeneration(err => {
+					this.codeGeneration((err) => {
 						if (err) {
 							return finalCallback(err);
 						}
@@ -3344,7 +3392,7 @@ Or do you want to use the entrypoints '${name}' and '${runtime}' independently o
 						this.hooks.afterHash.call();
 						this.logger.timeEnd("hashing");
 
-						this._runCodeGenerationJobs(codeGenerationJobs, err => {
+						this._runCodeGenerationJobs(codeGenerationJobs, (err) => {
 							if (err) {
 								return finalCallback(err);
 							}
@@ -3367,7 +3415,7 @@ Or do you want to use the entrypoints '${name}' and '${runtime}' independently o
 
 							const cont = () => {
 								this.logger.time("process assets");
-								this.hooks.processAssets.callAsync(this.assets, err => {
+								this.hooks.processAssets.callAsync(this.assets, (err) => {
 									if (err) {
 										return finalCallback(
 											makeWebpackError(err, "Compilation.hooks.processAssets")
@@ -3403,7 +3451,7 @@ Or do you want to use the entrypoints '${name}' and '${runtime}' independently o
 										this.unseal();
 										return this.seal(callback);
 									}
-									return this.hooks.afterSeal.callAsync(err => {
+									return this.hooks.afterSeal.callAsync((err) => {
 										if (err) {
 											return finalCallback(
 												makeWebpackError(err, "Compilation.hooks.afterSeal")
@@ -3418,7 +3466,7 @@ Or do you want to use the entrypoints '${name}' and '${runtime}' independently o
 							this.logger.time("create chunk assets");
 							if (this.hooks.shouldGenerateChunkAssets.call() !== false) {
 								this.hooks.beforeChunkAssets.call();
-								this.createChunkAssets(err => {
+								this.createChunkAssets((err) => {
 									this.logger.timeEnd("create chunk assets");
 									if (err) {
 										return finalCallback(err);
@@ -3542,7 +3590,7 @@ Or do you want to use the entrypoints '${name}' and '${runtime}' independently o
 					if (
 						codeGenerationDependencies !== undefined &&
 						(notCodeGeneratedModules === undefined ||
-							codeGenerationDependencies.some(dep => {
+							codeGenerationDependencies.some((dep) => {
 								const referencedModule = /** @type {Module} */ (
 									moduleGraph.getModule(dep)
 								);
@@ -3574,7 +3622,7 @@ Or do you want to use the entrypoints '${name}' and '${runtime}' independently o
 						}
 					);
 				},
-				err => {
+				(err) => {
 					if (err) return callback(err);
 					if (delayedJobs.length > 0) {
 						if (delayedJobs.length === jobs.length) {
@@ -3583,7 +3631,7 @@ Or do you want to use the entrypoints '${name}' and '${runtime}' independently o
 									new Error(
 										`Unable to make progress during code generation because of circular code generation dependency: ${Array.from(
 											delayedModules,
-											m => m.identifier()
+											(m) => m.identifier()
 										).join(", ")}`
 									)
 								)
@@ -3597,7 +3645,7 @@ Or do you want to use the entrypoints '${name}' and '${runtime}' independently o
 					}
 					if (errors.length > 0) {
 						errors.sort(
-							compareSelect(err => err.module, compareModulesByIdentifier)
+							compareSelect((err) => err.module, compareModulesByIdentifier)
 						);
 						for (const error of errors) {
 							this.errors.push(error);
@@ -3644,7 +3692,7 @@ Or do you want to use the entrypoints '${name}' and '${runtime}' independently o
 	) {
 		let codeGenerated = false;
 		const cache = new MultiItemCache(
-			runtimes.map(runtime =>
+			runtimes.map((runtime) =>
 				this._codeGenerationCache.getItemCache(
 					`${module.identifier()}|${getRuntimeKey(runtime)}`,
 					`${hash}|${dependencyTemplates.getHash()}`
@@ -3683,7 +3731,7 @@ Or do you want to use the entrypoints '${name}' and '${runtime}' independently o
 				results.add(module, runtime, result);
 			}
 			if (!cachedResult) {
-				cache.store(result, err =>
+				cache.store(result, (err) =>
 					callback(/** @type {WebpackError} */ (err), codeGenerated)
 				);
 			} else {
@@ -4031,7 +4079,7 @@ Or do you want to use the entrypoints '${name}' and '${runtime}' independently o
 		 * @param {Module} module module for processing
 		 * @returns {void}
 		 */
-		const processModule = module => {
+		const processModule = (module) => {
 			if (!moduleGraph.setDepthIfLower(module, depth)) return;
 			queue.add(module);
 		};
@@ -4157,7 +4205,7 @@ Or do you want to use the entrypoints '${name}' and '${runtime}' independently o
 		/**
 		 * @param {Dependency} d dependency to (maybe) patch up
 		 */
-		const iteratorDependency = d => {
+		const iteratorDependency = (d) => {
 			const depModule = this.moduleGraph.getModule(d);
 			if (!depModule) {
 				return;
@@ -4190,7 +4238,7 @@ Or do you want to use the entrypoints '${name}' and '${runtime}' independently o
 		/**
 		 * @param {Entrypoint} ep an entrypoint
 		 */
-		const processEntrypoint = ep => {
+		const processEntrypoint = (ep) => {
 			const runtime = /** @type {string} */ (ep.options.runtime || ep.name);
 			const chunk = /** @type {Chunk} */ (ep.getRuntimeChunk());
 			chunkGraph.setRuntimeId(runtime, /** @type {ChunkId} */ (chunk.id));
@@ -4273,7 +4321,9 @@ Or do you want to use the entrypoints '${name}' and '${runtime}' independently o
 			}
 		}
 		if (errors.length > 0) {
-			errors.sort(compareSelect(err => err.module, compareModulesByIdentifier));
+			errors.sort(
+				compareSelect((err) => err.module, compareModulesByIdentifier)
+			);
 			for (const error of errors) {
 				this.errors.push(error);
 			}
@@ -4410,7 +4460,7 @@ Or do you want to use the entrypoints '${name}' and '${runtime}' independently o
 		for (const info of runtimeChunksMap.values()) {
 			for (const other of new Set(
 				[...info.chunk.getAllReferencedAsyncEntrypoints()].map(
-					e => e.chunks[e.chunks.length - 1]
+					(e) => e.chunks[e.chunks.length - 1]
 				)
 			)) {
 				const otherInfo =
@@ -4463,11 +4513,11 @@ Or do you want to use the entrypoints '${name}' and '${runtime}' independently o
 					circularRuntimeChunkInfo.push(info);
 				}
 			}
-			circularRuntimeChunkInfo.sort(compareSelect(i => i.chunk, byId));
+			circularRuntimeChunkInfo.sort(compareSelect((i) => i.chunk, byId));
 			const err =
 				new WebpackError(`Circular dependency between chunks with runtime (${Array.from(
 					circularRuntimeChunkInfo,
-					c => c.chunk.name || c.chunk.id
+					(c) => c.chunk.name || c.chunk.id
 				).join(", ")})
 This prevents using hashes of each other and should be avoided.`);
 			err.chunk = circularRuntimeChunkInfo[0].chunk;
@@ -4487,7 +4537,7 @@ This prevents using hashes of each other and should be avoided.`);
 		/**
 		 * @param {Chunk} chunk chunk
 		 */
-		const processChunk = chunk => {
+		const processChunk = (chunk) => {
 			// Last minute module hash generation for modules that depend on chunk hashes
 			this.logger.time("hashing: hash runtime modules");
 			const runtime = chunk.runtime;
@@ -4564,7 +4614,9 @@ This prevents using hashes of each other and should be avoided.`);
 		for (const chunk of runtimeChunks) processChunk(chunk);
 		for (const chunk of initialChunks) processChunk(chunk);
 		if (errors.length > 0) {
-			errors.sort(compareSelect(err => err.module, compareModulesByIdentifier));
+			errors.sort(
+				compareSelect((err) => err.module, compareModulesByIdentifier)
+			);
 			for (const error of errors) {
 				this.errors.push(error);
 			}
@@ -4671,7 +4723,7 @@ This prevents using hashes of each other and should be avoided.`);
 				/**
 				 * @param {string} name name
 				 */
-				const remove = name => {
+				const remove = (name) => {
 					const relatedIn = this._assetsRelatedIn.get(name);
 					if (relatedIn === undefined) return;
 					const entry = relatedIn.get(key);
@@ -4696,7 +4748,7 @@ This prevents using hashes of each other and should be avoided.`);
 				/**
 				 * @param {string} name name
 				 */
-				const add = name => {
+				const add = (name) => {
 					let relatedIn = this._assetsRelatedIn.get(name);
 					if (relatedIn === undefined) {
 						this._assetsRelatedIn.set(name, (relatedIn = new Map()));
@@ -4783,7 +4835,7 @@ This prevents using hashes of each other and should be avoided.`);
 					const entry = related[key];
 					let newEntry;
 					if (Array.isArray(entry)) {
-						newEntry = entry.map(x => (x === file ? newFile : x));
+						newEntry = entry.map((x) => (x === file ? newFile : x));
 					} else if (entry === file) {
 						newEntry = newFile;
 					} else {
@@ -4837,7 +4889,7 @@ This prevents using hashes of each other and should be avoided.`);
 				/**
 				 * @param {string} file file
 				 */
-				const checkUsedAndDelete = file => {
+				const checkUsedAndDelete = (file) => {
 					if (!this._assetsRelatedIn.has(file)) {
 						this.deleteAsset(file);
 					}
@@ -4987,7 +5039,7 @@ This prevents using hashes of each other and should be avoided.`);
 							 * @param {Error} err error
 							 * @returns {void}
 							 */
-							const errorAndCallback = err => {
+							const errorAndCallback = (err) => {
 								const filename =
 									file ||
 									(typeof file === "string"
@@ -5068,7 +5120,7 @@ This prevents using hashes of each other and should be avoided.`);
 									chunk
 								});
 								if (source !== sourceFromCache) {
-									assetCacheItem.store(source, err => {
+									assetCacheItem.store(source, (err) => {
 										if (err) return errorAndCallback(err);
 										inTry = false;
 										return callback();
@@ -5190,9 +5242,9 @@ This prevents using hashes of each other and should be avoided.`);
 			modules,
 			10,
 			(module, push, callback) => {
-				this.buildQueue.waitFor(module, err => {
+				this.buildQueue.waitFor(module, (err) => {
 					if (err) return callback(err);
-					this.processDependenciesQueue.waitFor(module, err => {
+					this.processDependenciesQueue.waitFor(module, (err) => {
 						if (err) return callback(err);
 						for (const { module: m } of this.moduleGraph.getOutgoingConnections(
 							module
@@ -5205,7 +5257,7 @@ This prevents using hashes of each other and should be avoided.`);
 					});
 				});
 			},
-			err => {
+			(err) => {
 				if (err) return callback(/** @type {WebpackError} */ (err));
 
 				// Create new chunk graph, chunk and entrypoint for the build time execution
@@ -5288,7 +5340,7 @@ This prevents using hashes of each other and should be avoided.`);
 				const reportErrors = () => {
 					if (errors.length > 0) {
 						errors.sort(
-							compareSelect(err => err.module, compareModulesByIdentifier)
+							compareSelect((err) => err.module, compareModulesByIdentifier)
 						);
 						for (const error of errors) {
 							this.errors.push(error);
@@ -5298,7 +5350,7 @@ This prevents using hashes of each other and should be avoided.`);
 				};
 
 				// Generate code for all aggregated modules
-				asyncLib.eachLimit(modules, 10, codeGen, err => {
+				asyncLib.eachLimit(modules, 10, codeGen, (err) => {
 					if (err) return callback(err);
 					reportErrors();
 
@@ -5334,7 +5386,7 @@ This prevents using hashes of each other and should be avoided.`);
 					}
 
 					// Generate code for all runtime modules
-					asyncLib.eachLimit(runtimeModules, 10, codeGen, err => {
+					asyncLib.eachLimit(runtimeModules, 10, codeGen, (err) => {
 						if (err) return callback(err);
 						reportErrors();
 
@@ -5410,7 +5462,7 @@ This prevents using hashes of each other and should be avoided.`);
 									callback
 								);
 							},
-							err => {
+							(err) => {
 								if (err) return callback(err);
 
 								/** @type {ExecuteModuleExports | undefined} */
@@ -5422,7 +5474,7 @@ This prevents using hashes of each other and should be avoided.`);
 									} = this.outputOptions;
 
 									/** @type {WebpackRequire} */
-									const __webpack_require__ = id => {
+									const __webpack_require__ = (id) => {
 										const cached = moduleCache[id];
 										if (cached !== undefined) {
 											if (cached.error) throw cached.error;
@@ -5656,7 +5708,7 @@ Object.defineProperty(compilationPrototype, "cache", {
 		/**
 		 * @param {EXPECTED_ANY} _v value
 		 */
-		_v => {},
+		(_v) => {},
 		"Compilation.cache was removed in favor of Compilation.getCache()",
 		"DEP_WEBPACK_COMPILATION_CACHE"
 	)