webpack 5.99.9 → 5.100.0

package/lib/serialization/FileMiddleware.js

@@ -7,11 +7,13 @@
 const { constants } = require("buffer");
 const { pipeline } = require("stream");
 const {
+	constants: zConstants,
+	// eslint-disable-next-line n/no-unsupported-features/node-builtins
 	createBrotliCompress,
+	// eslint-disable-next-line n/no-unsupported-features/node-builtins
 	createBrotliDecompress,
-	createGzip,
 	createGunzip,
-	constants: zConstants
+	createGzip
 } = require("zlib");
 const { DEFAULTS } = require("../config/defaults");
 const createHash = require("../util/createHash");
@@ -117,8 +119,9 @@ const serialize = async (
 	let lastBuffers;
 	for (const item of await data) {
 		if (typeof item === "function") {
-			if (!SerializerMiddleware.isLazy(item))
+			if (!SerializerMiddleware.isLazy(item)) {
 				throw new Error("Unexpected function");
+			}
 			if (!SerializerMiddleware.isLazy(item, middleware)) {
 				throw new Error(
 					"Unexpected lazy value with non-this target (can't pass through lazy values)"
@@ -551,7 +554,7 @@ class FileMiddleware extends SerializerMiddleware {
 								 * @param {(value?: undefined) => void} resolve resolve
 								 */
 								resolve => {
-									this.fs.rename(filename, `${filename}.old`, err => {
+									this.fs.rename(filename, `${filename}.old`, _err => {
 										resolve();
 									});
 								}

package/lib/serialization/ObjectMiddleware.js

@@ -175,7 +175,7 @@ if (exports.constructor !== Object) {
 	// eslint-disable-next-line n/exports-style
 	const Obj = /** @type {ObjectConstructor} */ (exports.constructor);
 	const Fn = /** @type {FunctionConstructor} */ (Obj.constructor);
-	for (const [type, config] of Array.from(jsTypes)) {
+	for (const [type, config] of jsTypes) {
 		if (type) {
 			const Type = new Fn(`return ${type.name};`)();
 			jsTypes.set(Type, config);
@@ -416,7 +416,7 @@ class ObjectMiddleware extends SerializerMiddleware {
 		 * @returns {string} stack
 		 */
 		const stackToString = item => {
-			const arr = Array.from(cycleStack);
+			const arr = [...cycleStack];
 			arr.push(item);
 			return arr
 				.map(item => {
@@ -439,11 +439,13 @@ class ObjectMiddleware extends SerializerMiddleware {
 					}
 					if (typeof item === "object" && item !== null) {
 						if (item.constructor) {
-							if (item.constructor === Object)
+							if (item.constructor === Object) {
 								return `Object { ${Object.keys(item).join(", ")} }`;
+							}
 							if (item.constructor === Map) return `Map { ${item.size} items }`;
-							if (item.constructor === Array)
+							if (item.constructor === Array) {
 								return `Array { ${item.length} items }`;
+							}
 							if (item.constructor === Set) return `Set { ${item.size} items }`;
 							if (item.constructor === RegExp) return item.toString();
 							return `${item.constructor.name}`;
@@ -472,8 +474,9 @@ class ObjectMiddleware extends SerializerMiddleware {
 					process(value);
 				} catch (err) {
 					if (err !== NOT_SERIALIZABLE) {
-						if (hasDebugInfoAttached === undefined)
+						if (hasDebugInfoAttached === undefined) {
 							hasDebugInfoAttached = new WeakSet();
+						}
 						if (!hasDebugInfoAttached.has(/** @type {Error} */ (err))) {
 							/** @type {Error} */
 							(err).message += `\nwhile serializing ${stackToString(value)}`;
@@ -596,8 +599,9 @@ class ObjectMiddleware extends SerializerMiddleware {
 
 				result.push(item);
 			} else if (typeof item === "function") {
-				if (!SerializerMiddleware.isLazy(item))
+				if (!SerializerMiddleware.isLazy(item)) {
 					throw new Error(`Unexpected function ${item}`);
+				}
 
 				/** @type {SerializedType | undefined} */
 				const serializedData =
@@ -658,14 +662,16 @@ class ObjectMiddleware extends SerializerMiddleware {
 	deserialize(data, context) {
 		let currentDataPos = 0;
 		const read = () => {
-			if (currentDataPos >= data.length)
+			if (currentDataPos >= data.length) {
 				throw new Error("Unexpected end of stream");
+			}
 
 			return data[currentDataPos++];
 		};
 
-		if (read() !== CURRENT_VERSION)
+		if (read() !== CURRENT_VERSION) {
 			throw new Error("Version mismatch, serializer changed");
+		}
 
 		let currentPos = 0;
 		/** @type {ReferenceableItem[]} */

package/lib/serialization/Serializer.js

@@ -20,8 +20,8 @@ class Serializer {
 	 * @param {Context=} context context
 	 */
 	constructor(middlewares, context) {
-		this.serializeMiddlewares = middlewares.slice();
-		this.deserializeMiddlewares = middlewares.slice().reverse();
+		this.serializeMiddlewares = [...middlewares];
+		this.deserializeMiddlewares = [...middlewares].reverse();
 		this.context = context;
 	}
 
@@ -49,7 +49,9 @@ class Serializer {
 				} catch (err) {
 					current = Promise.reject(err);
 				}
-			} else break;
+			} else {
+				break;
+			}
 		}
 		return /** @type {Promise<SerializedValue>} */ (current);
 	}

package/lib/serialization/SerializerMiddleware.js

@@ -35,6 +35,7 @@ class SerializerMiddleware {
 	 */
 	serialize(data, context) {
 		const AbstractMethodError = require("../AbstractMethodError");
+
 		throw new AbstractMethodError();
 	}
 
@@ -47,6 +48,7 @@ class SerializerMiddleware {
 	 */
 	deserialize(data, context) {
 		const AbstractMethodError = require("../AbstractMethodError");
+
 		throw new AbstractMethodError();
 	}
 

package/lib/sharing/ConsumeSharedModule.js

@@ -177,7 +177,7 @@ class ConsumeSharedModule extends Module {
 	 * @param {CodeGenerationContext} context context for code generation
 	 * @returns {CodeGenerationResult} result
 	 */
-	codeGeneration({ chunkGraph, moduleGraph, runtimeTemplate }) {
+	codeGeneration({ chunkGraph, runtimeTemplate }) {
 		const runtimeRequirements = new Set([RuntimeGlobals.shareScopeMap]);
 		const {
 			shareScope,

package/lib/sharing/ConsumeSharedPlugin.js

@@ -18,9 +18,9 @@ const ConsumeSharedRuntimeModule = require("./ConsumeSharedRuntimeModule");
 const ProvideForSharedDependency = require("./ProvideForSharedDependency");
 const { resolveMatchedConfigs } = require("./resolveMatchedConfigs");
 const {
-	isRequiredVersion,
 	getDescriptionFile,
-	getRequiredVersionFromDescriptionFile
+	getRequiredVersionFromDescriptionFile,
+	isRequiredVersion
 } = require("./utils");
 
 /** @typedef {import("../../declarations/plugins/sharing/ConsumeSharedPlugin").ConsumeSharedPluginOptions} ConsumeSharedPluginOptions */
@@ -32,7 +32,7 @@ const {
 /** @typedef {import("./utils").DescriptionFile} DescriptionFile */
 
 const validate = createSchemaValidation(
-	require("../../schemas/plugins/sharing/ConsumeSharedPlugin.check.js"),
+	require("../../schemas/plugins/sharing/ConsumeSharedPlugin.check"),
 	() => require("../../schemas/plugins/sharing/ConsumeSharedPlugin.json"),
 	{
 		name: "Consume Shared Plugin",

package/lib/sharing/ConsumeSharedRuntimeModule.js

@@ -10,9 +10,9 @@ const RuntimeModule = require("../RuntimeModule");
 const Template = require("../Template");
 const {
 	parseVersionRuntimeCode,
-	versionLtRuntimeCode,
 	rangeToStringRuntimeCode,
-	satisfyRuntimeCode
+	satisfyRuntimeCode,
+	versionLtRuntimeCode
 } = require("../util/semver");
 
 /** @typedef {import("webpack-sources").Source} Source */

package/lib/sharing/ProvideSharedModule.js

@@ -132,7 +132,7 @@ class ProvideSharedModule extends Module {
 	 * @param {CodeGenerationContext} context context for code generation
 	 * @returns {CodeGenerationResult} result
 	 */
-	codeGeneration({ runtimeTemplate, moduleGraph, chunkGraph }) {
+	codeGeneration({ runtimeTemplate, chunkGraph }) {
 		const runtimeRequirements = new Set([RuntimeGlobals.initializeSharing]);
 		const code = `register(${JSON.stringify(this._name)}, ${JSON.stringify(
 			this._version || "0"

package/lib/sharing/ProvideSharedPlugin.js

@@ -18,7 +18,7 @@ const ProvideSharedModuleFactory = require("./ProvideSharedModuleFactory");
 /** @typedef {import("../NormalModuleFactory").NormalModuleCreateData} NormalModuleCreateData */
 
 const validate = createSchemaValidation(
-	require("../../schemas/plugins/sharing/ProvideSharedPlugin.check.js"),
+	require("../../schemas/plugins/sharing/ProvideSharedPlugin.check"),
 	() => require("../../schemas/plugins/sharing/ProvideSharedPlugin.json"),
 	{
 		name: "Provide Shared Plugin",
@@ -49,8 +49,9 @@ class ProvideSharedPlugin {
 			parseOptions(
 				options.provides,
 				item => {
-					if (Array.isArray(item))
+					if (Array.isArray(item)) {
 						throw new Error("Unexpected array of provides");
+					}
 					/** @type {ProvideOptions} */
 					const result = {
 						shareKey: item,

package/lib/sharing/SharePlugin.js

@@ -27,8 +27,9 @@ class SharePlugin {
 		const sharedOptions = parseOptions(
 			options.shared,
 			(item, key) => {
-				if (typeof item !== "string")
+				if (typeof item !== "string") {
 					throw new Error("Unexpected array in shared");
+				}
 				/** @type {SharedConfig} */
 				const config =
 					item === key || !isRequiredVersion(item)

package/lib/sharing/ShareRuntimeModule.js

@@ -121,17 +121,15 @@ class ShareRuntimeModule extends RuntimeModule {
 					])}`,
 					"var promises = [];",
 					"switch(name) {",
-					...Array.from(initCodePerScope)
+					...[...initCodePerScope]
 						.sort(([a], [b]) => compareStrings(a, b))
 						.map(([name, stages]) =>
 							Template.indent([
 								`case ${JSON.stringify(name)}: {`,
 								Template.indent(
-									Array.from(stages)
+									[...stages]
 										.sort(([a], [b]) => a - b)
-										.map(([, initCode]) =>
-											Template.asString(Array.from(initCode))
-										)
+										.map(([, initCode]) => Template.asString([...initCode]))
 								),
 								"}",
 								"break;"

package/lib/sharing/utils.js

@@ -5,7 +5,7 @@
 
 "use strict";
 
-const { join, dirname, readJson } = require("../util/fs");
+const { dirname, join, readJson } = require("../util/fs");
 
 /** @typedef {import("../util/fs").InputFileSystem} InputFileSystem */
 /** @typedef {import("../util/fs").JsonObject} JsonObject */
@@ -292,34 +292,6 @@ function getGitUrlVersion(gitUrl) {
 	return getVersionFromHash(gitUrl);
 }
 
-/**
- * @param {string} str maybe required version
- * @returns {boolean} true, if it looks like a version
- */
-function isRequiredVersion(str) {
-	return VERSION_PATTERN_REGEXP.test(str);
-}
-
-module.exports.isRequiredVersion = isRequiredVersion;
-
-/**
- * @see https://docs.npmjs.com/cli/v7/configuring-npm/package-json#urls-as-dependencies
- * @param {string} versionDesc version to be normalized
- * @returns {string} normalized version
- */
-function normalizeVersion(versionDesc) {
-	versionDesc = (versionDesc && versionDesc.trim()) || "";
-
-	if (isRequiredVersion(versionDesc)) {
-		return versionDesc;
-	}
-
-	// add handle for URL Dependencies
-	return getGitUrlVersion(versionDesc.toLowerCase());
-}
-
-module.exports.normalizeVersion = normalizeVersion;
-
 /** @typedef {{ data: JsonObject, path: string }} DescriptionFile */
 
 /**
@@ -349,11 +321,9 @@ const getDescriptionFile = (
 		if (i >= descriptionFiles.length) {
 			const parentDirectory = dirname(fs, directory);
 			if (!parentDirectory || parentDirectory === directory) {
-				return callback(
-					null,
-					undefined,
-					Array.from(satisfiesDescriptionFileDataInternal.checkedFilePaths)
-				);
+				return callback(null, undefined, [
+					...satisfiesDescriptionFileDataInternal.checkedFilePaths
+				]);
 			}
 			return getDescriptionFile(
 				fs,
@@ -391,6 +361,7 @@ const getDescriptionFile = (
 	};
 	tryLoadCurrent();
 };
+
 module.exports.getDescriptionFile = getDescriptionFile;
 
 /**
@@ -421,5 +392,34 @@ const getRequiredVersionFromDescriptionFile = (data, packageName) => {
 		}
 	}
 };
+
 module.exports.getRequiredVersionFromDescriptionFile =
 	getRequiredVersionFromDescriptionFile;
+
+/**
+ * @param {string} str maybe required version
+ * @returns {boolean} true, if it looks like a version
+ */
+function isRequiredVersion(str) {
+	return VERSION_PATTERN_REGEXP.test(str);
+}
+
+module.exports.isRequiredVersion = isRequiredVersion;
+
+/**
+ * @see https://docs.npmjs.com/cli/v7/configuring-npm/package-json#urls-as-dependencies
+ * @param {string} versionDesc version to be normalized
+ * @returns {string} normalized version
+ */
+function normalizeVersion(versionDesc) {
+	versionDesc = (versionDesc && versionDesc.trim()) || "";
+
+	if (isRequiredVersion(versionDesc)) {
+		return versionDesc;
+	}
+
+	// add handle for URL Dependencies
+	return getGitUrlVersion(versionDesc.toLowerCase());
+}
+
+module.exports.normalizeVersion = normalizeVersion;

package/lib/stats/DefaultStatsFactoryPlugin.js

@@ -14,13 +14,13 @@ const AggressiveSplittingPlugin = require("../optimize/AggressiveSplittingPlugin
 const SizeLimitsPlugin = require("../performance/SizeLimitsPlugin");
 const { countIterable } = require("../util/IterableHelpers");
 const {
-	compareLocations,
 	compareChunksById,
-	compareNumbers,
 	compareIds,
-	concatComparators,
+	compareLocations,
+	compareModulesByIdentifier,
+	compareNumbers,
 	compareSelect,
-	compareModulesByIdentifier
+	concatComparators
 } = require("../util/comparators");
 const { makePathsRelative, parseResource } = require("../util/identifier");
 
@@ -358,7 +358,7 @@ const uniqueArray = (items, selector) => {
 			set.add(i);
 		}
 	}
-	return Array.from(set);
+	return [...set];
 };
 
 /**
@@ -495,7 +495,7 @@ const EXTRACT_ERROR = {
 	errorDetails: (
 		object,
 		error,
-		{ type, compilation, cachedGetErrors, cachedGetWarnings },
+		{ type, compilation, cachedGetErrors },
 		{ errorDetails }
 	) => {
 		if (
@@ -582,6 +582,7 @@ const SIMPLE_EXTRACTORS = {
 			const { logging, loggingDebug, loggingTrace } = options;
 			if (logging || (loggingDebug && loggingDebug.length > 0)) {
 				const util = require("util");
+
 				object.logging = {};
 				let acceptedTypes;
 				let collapsedGroups = false;
@@ -654,8 +655,9 @@ const SIMPLE_EXTRACTORS = {
 						if (
 							type === LogType.groupCollapsed &&
 							(debugMode || collapsedGroups)
-						)
+						) {
 							type = LogType.group;
+						}
 
 						if (depthInCollapsedGroup === 0) {
 							processedLogEntries++;
@@ -821,15 +823,11 @@ const SIMPLE_EXTRACTORS = {
 				}
 			}
 
-			const groupedAssets = factory.create(
-				`${type}.assets`,
-				Array.from(assets),
-				{
-					...context,
-					compilationFileToChunks,
-					compilationAuxiliaryFileToChunks
-				}
-			);
+			const groupedAssets = factory.create(`${type}.assets`, [...assets], {
+				...context,
+				compilationFileToChunks,
+				compilationAuxiliaryFileToChunks
+			});
 			const limited = spaceLimited(
 				groupedAssets,
 				/** @type {number} */ (options.assetsSpace)
@@ -841,13 +839,13 @@ const SIMPLE_EXTRACTORS = {
 			const { type } = context;
 			object.chunks = factory.create(
 				`${type}.chunks`,
-				Array.from(compilation.chunks),
+				[...compilation.chunks],
 				context
 			);
 		},
 		modules: (object, compilation, context, options, factory) => {
 			const { type } = context;
-			const array = Array.from(compilation.modules);
+			const array = [...compilation.modules];
 			const groupedModules = factory.create(`${type}.modules`, array, context);
 			const limited = spaceLimited(groupedModules, options.modulesSpace);
 			object.modules = limited.children;
@@ -980,9 +978,10 @@ const SIMPLE_EXTRACTORS = {
 					!warningsFilter &&
 					/** @type {KnownNormalizedStatsOptions["warningsFilter"]} */
 					(warningsFilter).length === 0
-				)
+				) {
 					// Type is wrong, because we don't need the real value for counting
 					return /** @type {EXPECTED_ANY[]} */ (cachedGetWarnings(c));
+				}
 				return factory
 					.create(`${type}${childType}.warnings`, cachedGetWarnings(c), context)
 					.filter(
@@ -1038,7 +1037,7 @@ const SIMPLE_EXTRACTORS = {
 		_: (
 			object,
 			asset,
-			{ compilation, compilationFileToChunks, compilationAuxiliaryFileToChunks }
+			{ compilationFileToChunks, compilationAuxiliaryFileToChunks }
 		) => {
 			const chunks = compilationFileToChunks.get(asset.name) || [];
 			const auxiliaryChunks =
@@ -1050,7 +1049,7 @@ const SIMPLE_EXTRACTORS = {
 			);
 			object.chunkIdHints = uniqueOrderedArray(
 				chunks,
-				c => Array.from(c.idNameHints),
+				c => [...c.idNameHints],
 				compareIds
 			);
 			object.auxiliaryChunkNames = uniqueOrderedArray(
@@ -1060,7 +1059,7 @@ const SIMPLE_EXTRACTORS = {
 			);
 			object.auxiliaryChunkIdHints = uniqueOrderedArray(
 				auxiliaryChunks,
-				c => Array.from(c.idNameHints),
+				c => [...c.idNameHints],
 				compareIds
 			);
 			object.filteredRelated = asset.related ? asset.related.length : undefined;
@@ -1200,7 +1199,7 @@ const SIMPLE_EXTRACTORS = {
 									}
 								}
 							}
-							return Array.from(set);
+							return [...set];
 						})
 					: undefined
 			};
@@ -1330,7 +1329,7 @@ const SIMPLE_EXTRACTORS = {
 			} = context;
 			const groupsReasons = factory.create(
 				`${type.slice(0, -8)}.reasons`,
-				Array.from(moduleGraph.getIncomingConnections(module)),
+				[...moduleGraph.getIncomingConnections(module)],
 				context
 			);
 			const limited = spaceLimited(
@@ -1352,7 +1351,7 @@ const SIMPLE_EXTRACTORS = {
 			} else if (typeof usedExports === "boolean") {
 				object.usedExports = usedExports;
 			} else {
-				object.usedExports = Array.from(usedExports);
+				object.usedExports = [...usedExports];
 			}
 		},
 		providedExports: (object, module, { compilation: { moduleGraph } }) => {
@@ -1506,15 +1505,15 @@ const SIMPLE_EXTRACTORS = {
 				size: chunkGraph.getChunkModulesSize(chunk),
 				sizes: chunkGraph.getChunkModulesSizes(chunk),
 				names: chunk.name ? [chunk.name] : [],
-				idHints: Array.from(chunk.idNameHints),
+				idHints: [...chunk.idNameHints],
 				runtime:
 					chunk.runtime === undefined
 						? undefined
 						: typeof chunk.runtime === "string"
 							? [makePathsRelative(chunk.runtime)]
 							: Array.from(chunk.runtime.sort(), makePathsRelative),
-				files: Array.from(chunk.files),
-				auxiliaryFiles: Array.from(chunk.auxiliaryFiles).sort(compareIds),
+				files: [...chunk.files],
+				auxiliaryFiles: [...chunk.auxiliaryFiles].sort(compareIds),
 				hash: /** @type {string} */ (chunk.renderedHash),
 				childrenByOrder: childIdByOrder
 			};
@@ -1523,7 +1522,7 @@ const SIMPLE_EXTRACTORS = {
 		ids: (object, chunk) => {
 			object.id = /** @type {ChunkId} */ (chunk.id);
 		},
-		chunkRelations: (object, chunk, { compilation: { chunkGraph } }) => {
+		chunkRelations: (object, chunk, _context) => {
 			/** @type {Set<string|number>} */
 			const parents = new Set();
 			/** @type {Set<string|number>} */
@@ -1543,13 +1542,14 @@ const SIMPLE_EXTRACTORS = {
 					}
 				}
 				for (const sibling of chunkGroup.chunks) {
-					if (sibling !== chunk)
+					if (sibling !== chunk) {
 						siblings.add(/** @type {ChunkId} */ (sibling.id));
+					}
 				}
 			}
-			object.siblings = Array.from(siblings).sort(compareIds);
-			object.parents = Array.from(parents).sort(compareIds);
-			object.children = Array.from(children).sort(compareIds);
+			object.siblings = [...siblings].sort(compareIds);
+			object.parents = [...parents].sort(compareIds);
+			object.children = [...children].sort(compareIds);
 		},
 		chunkModules: (object, chunk, context, options, factory) => {
 			const {
@@ -1624,15 +1624,13 @@ const SIMPLE_EXTRACTORS = {
 			object.originName = origin.readableIdentifier(requestShortener);
 			object.moduleIdentifier = module.identifier();
 			object.moduleName = module.readableIdentifier(requestShortener);
-			const dependencies = Array.from(
-				moduleGraph.getIncomingConnections(module)
-			)
+			const dependencies = [...moduleGraph.getIncomingConnections(module)]
 				.filter(c => c.resolvedOriginModule === origin && c.dependency)
 				.map(c => c.dependency);
 			object.dependencies = factory.create(
 				`${type}.dependencies`,
 				/** @type {Dependency[]} */
-				(Array.from(new Set(dependencies))),
+				([...new Set(dependencies)]),
 				context
 			);
 		},
@@ -1705,7 +1703,7 @@ const SORTERS = {
 	"chunk.modules": MODULES_SORTER,
 	"module.modules": MODULES_SORTER,
 	"module.reasons": {
-		_: (comparators, { compilation: { chunkGraph } }) => {
+		_: (comparators, _context) => {
 			comparators.push(
 				compareSelect(x => x.originModule, compareModulesByIdentifier)
 			);
@@ -1868,7 +1866,7 @@ const spaceLimited = (
 	if (groupsSize + items.length <= max) {
 		// The total size in the current state fits into the max
 		// keep all
-		children = groups.length > 0 ? groups.concat(items) : items;
+		children = groups.length > 0 ? [...groups, ...items] : items;
 	} else if (groups.length === 0) {
 		// slice items to max
 		// inner space marks that lines for filteredChildren already reserved
@@ -1931,7 +1929,7 @@ const spaceLimited = (
 					}
 				}
 			}
-			children = groups.concat(items);
+			children = [...groups, ...items];
 		} else if (limit === max) {
 			// If we have only enough space to show one line per group and one line for the filtered items
 			// collapse all groups and items
@@ -1956,7 +1954,7 @@ const errorsSpaceLimit = (errors, max) => {
 	let filtered = 0;
 	// Can not fit into limit
 	// print only messages
-	if (errors.length + 1 >= max)
+	if (errors.length + 1 >= max) {
 		return [
 			errors.map(error => {
 				if (typeof error === "string" || !error.details) return error;
@@ -1965,6 +1963,7 @@ const errorsSpaceLimit = (errors, max) => {
 			}),
 			filtered
 		];
+	}
 	let fullLength = errors.length;
 	let result = errors;
 
@@ -2123,12 +2122,13 @@ const ASSETS_GROUPERS = {
 					const keys = [];
 					if (groupAssetsByPath) {
 						keys.push(".");
-						if (extension)
+						if (extension) {
 							keys.push(
 								path.length
 									? `${path.join("/")}/*${extension}`
 									: `*${extension}`
 							);
+						}
 						while (path.length > 0) {
 							keys.push(`${path.join("/")}/`);
 							path.pop();
@@ -2147,7 +2147,7 @@ const ASSETS_GROUPERS = {
 			});
 		}
 	},
-	groupAssetsByInfo: (groupConfigs, context, options) => {
+	groupAssetsByInfo: (groupConfigs, _context, _options) => {
 		/**
 		 * @param {string} name name
 		 */
@@ -2168,7 +2168,7 @@ const ASSETS_GROUPERS = {
 		groupByAssetInfoFlag("development");
 		groupByAssetInfoFlag("hotModuleReplacement");
 	},
-	groupAssetsByChunk: (groupConfigs, context, options) => {
+	groupAssetsByChunk: (groupConfigs, _context, _options) => {
 		/**
 		 * @param {keyof KnownStatsAsset} name name
 		 */
@@ -2319,12 +2319,13 @@ const MODULES_GROUPERS = type => ({
 					const path = pathMatch ? pathMatch[1].split(/[/\\]/) : [];
 					const keys = [];
 					if (groupModulesByPath) {
-						if (extension)
+						if (extension) {
 							keys.push(
 								path.length
 									? `${path.join("/")}/*${extension}`
 									: `*${extension}`
 							);
+						}
 						while (path.length > 0) {
 							keys.push(`${path.join("/")}/`);
 							path.pop();
@@ -2511,8 +2512,9 @@ const iterateConfig = (config, options, fn) => {
 						value === false ||
 						value === undefined ||
 						(Array.isArray(value) && value.length === 0)
-					)
+					) {
 						continue;
+					}
 				}
 			}
 			fn(hookFor, subConfig[option]);
@@ -2684,4 +2686,5 @@ class DefaultStatsFactoryPlugin {
 		});
 	}
 }
+
 module.exports = DefaultStatsFactoryPlugin;