webpack 5.87.0 → 5.88.0

package/lib/APIPlugin.js

@@ -29,7 +29,7 @@ const GetFullHashRuntimeModule = require("./runtime/GetFullHashRuntimeModule");
 /**
  * @param {boolean} module true if ES module
  * @param {string} importMetaName `import.meta` name
- * @returns {Record<string, {expr: string, req: string[], type?: string, assign: boolean}>} replacements
+ * @returns {Record<string, {expr: string, req: string[] | null, type?: string, assign: boolean}>} replacements
  */
 function getReplacements(module, importMetaName) {
 	return {

package/lib/AsyncDependenciesBlock.js

@@ -21,9 +21,9 @@ const makeSerializable = require("./util/makeSerializable");
 
 class AsyncDependenciesBlock extends DependenciesBlock {
 	/**
-	 * @param {ChunkGroupOptions & { entryOptions?: EntryOptions }} groupOptions options for the group
-	 * @param {DependencyLocation=} loc the line of code
-	 * @param {string=} request the request
+	 * @param {(ChunkGroupOptions & { entryOptions?: EntryOptions }) | null} groupOptions options for the group
+	 * @param {(DependencyLocation | null)=} loc the line of code
+	 * @param {(string | null)=} request the request
 	 */
 	constructor(groupOptions, loc, request) {
 		super();

package/lib/ChunkGraph.js

@@ -188,7 +188,7 @@ class ChunkGraphModule {
 		this.entryInChunks = undefined;
 		/** @type {Set<Chunk> | undefined} */
 		this.runtimeInChunks = undefined;
-		/** @type {RuntimeSpecMap<ModuleHashInfo>} */
+		/** @type {RuntimeSpecMap<ModuleHashInfo> | undefined} */
 		this.hashes = undefined;
 		/** @type {string | number} */
 		this.id = null;
@@ -1388,7 +1388,7 @@ Caller might not support runtime-dependent code generation (opt-out via optimiza
 	 */
 	hasModuleHashes(module, runtime) {
 		const cgm = this._getChunkGraphModule(module);
-		const hashes = cgm.hashes;
+		const hashes = /** @type {RuntimeSpecMap<ModuleHashInfo>} */ (cgm.hashes);
 		return hashes && hashes.has(runtime);
 	}
 
@@ -1399,7 +1399,7 @@ Caller might not support runtime-dependent code generation (opt-out via optimiza
 	 */
 	getModuleHash(module, runtime) {
 		const cgm = this._getChunkGraphModule(module);
-		const hashes = cgm.hashes;
+		const hashes = /** @type {RuntimeSpecMap<ModuleHashInfo>} */ (cgm.hashes);
 		return this._getModuleHashInfo(module, hashes, runtime).hash;
 	}
 
@@ -1410,7 +1410,7 @@ Caller might not support runtime-dependent code generation (opt-out via optimiza
 	 */
 	getRenderedModuleHash(module, runtime) {
 		const cgm = this._getChunkGraphModule(module);
-		const hashes = cgm.hashes;
+		const hashes = /** @type {RuntimeSpecMap<ModuleHashInfo>} */ (cgm.hashes);
 		return this._getModuleHashInfo(module, hashes, runtime).renderedHash;
 	}
 

package/lib/ChunkGroup.js

@@ -109,9 +109,15 @@ class ChunkGroup {
 	 */
 	addOptions(options) {
 		for (const key of Object.keys(options)) {
-			if (this.options[key] === undefined) {
-				this.options[key] = options[key];
-			} else if (this.options[key] !== options[key]) {
+			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)]
+			) {
 				if (key.endsWith("Order")) {
 					this.options[key] = Math.max(this.options[key], options[key]);
 				} else {

package/lib/CompatibilityPlugin.js

@@ -13,8 +13,11 @@ const {
 const RuntimeGlobals = require("./RuntimeGlobals");
 const ConstDependency = require("./dependencies/ConstDependency");
 
+/** @typedef {import("estree").CallExpression} CallExpression */
 /** @typedef {import("./Compiler")} Compiler */
+/** @typedef {import("./Dependency").DependencyLocation} DependencyLocation */
 /** @typedef {import("./javascript/JavascriptParser")} JavascriptParser */
+/** @typedef {import("./javascript/JavascriptParser").Range} Range */
 
 const nestedWebpackIdentifierTag = Symbol("nested webpack identifier");
 const PLUGIN_NAME = "CompatibilityPlugin";
@@ -43,31 +46,41 @@ class CompatibilityPlugin {
 						)
 							return;
 
-						parser.hooks.call.for("require").tap(PLUGIN_NAME, expr => {
-							// support for browserify style require delegator: "require(o, !0)"
-							if (expr.arguments.length !== 2) return;
-							const second = parser.evaluateExpression(expr.arguments[1]);
-							if (!second.isBoolean()) return;
-							if (second.asBool() !== true) return;
-							const dep = new ConstDependency("require", expr.callee.range);
-							dep.loc = expr.loc;
-							if (parser.state.current.dependencies.length > 0) {
-								const last =
-									parser.state.current.dependencies[
-										parser.state.current.dependencies.length - 1
-									];
-								if (
-									last.critical &&
-									last.options &&
-									last.options.request === "." &&
-									last.userRequest === "." &&
-									last.options.recursive
-								)
-									parser.state.current.dependencies.pop();
+						parser.hooks.call.for("require").tap(
+							PLUGIN_NAME,
+							/**
+							 * @param {CallExpression} expr call expression
+							 * @returns {boolean | void} true when need to handle
+							 */
+							expr => {
+								// support for browserify style require delegator: "require(o, !0)"
+								if (expr.arguments.length !== 2) return;
+								const second = parser.evaluateExpression(expr.arguments[1]);
+								if (!second.isBoolean()) return;
+								if (second.asBool() !== true) return;
+								const dep = new ConstDependency(
+									"require",
+									/** @type {Range} */ (expr.callee.range)
+								);
+								dep.loc = /** @type {DependencyLocation} */ (expr.loc);
+								if (parser.state.current.dependencies.length > 0) {
+									const last =
+										parser.state.current.dependencies[
+											parser.state.current.dependencies.length - 1
+										];
+									if (
+										last.critical &&
+										last.options &&
+										last.options.request === "." &&
+										last.userRequest === "." &&
+										last.options.recursive
+									)
+										parser.state.current.dependencies.pop();
+								}
+								parser.state.module.addPresentationalDependency(dep);
+								return true;
 							}
-							parser.state.module.addPresentationalDependency(dep);
-							return true;
-						});
+						);
 					});
 
 				/**
@@ -82,7 +95,9 @@ class CompatibilityPlugin {
 							statement.id &&
 							statement.id.name === RuntimeGlobals.require
 						) {
-							const newName = `__nested_webpack_require_${statement.range[0]}__`;
+							const newName = `__nested_webpack_require_${
+								/** @type {Range} */ (statement.range)[0]
+							}__`;
 							parser.tagVariable(
 								statement.id.name,
 								nestedWebpackIdentifierTag,
@@ -101,7 +116,9 @@ class CompatibilityPlugin {
 					parser.hooks.pattern
 						.for(RuntimeGlobals.require)
 						.tap(PLUGIN_NAME, pattern => {
-							const newName = `__nested_webpack_require_${pattern.range[0]}__`;
+							const newName = `__nested_webpack_require_${
+								/** @type {Range} */ (pattern.range)[0]
+							}__`;
 							parser.tagVariable(pattern.name, nestedWebpackIdentifierTag, {
 								name: newName,
 								declaration: {
@@ -135,8 +152,11 @@ class CompatibilityPlugin {
 								parser.state.module.addPresentationalDependency(dep);
 								declaration.updated = true;
 							}
-							const dep = new ConstDependency(name, expr.range);
-							dep.loc = expr.loc;
+							const dep = new ConstDependency(
+								name,
+								/** @type {Range} */ (expr.range)
+							);
+							dep.loc = /** @type {DependencyLocation} */ (expr.loc);
 							parser.state.module.addPresentationalDependency(dep);
 							return true;
 						});
@@ -145,11 +165,11 @@ class CompatibilityPlugin {
 					parser.hooks.program.tap(PLUGIN_NAME, (program, comments) => {
 						if (comments.length === 0) return;
 						const c = comments[0];
-						if (c.type === "Line" && c.range[0] === 0) {
+						if (c.type === "Line" && /** @type {Range} */ (c.range)[0] === 0) {
 							if (parser.state.source.slice(0, 2).toString() !== "#!") return;
 							// this is a hashbang comment
 							const dep = new ConstDependency("//", 0);
-							dep.loc = c.loc;
+							dep.loc = /** @type {DependencyLocation} */ (c.loc);
 							parser.state.module.addPresentationalDependency(dep);
 						}
 					});

package/lib/Compilation.js

@@ -3208,6 +3208,7 @@ Or do you want to use the entrypoints '${name}' and '${runtime}' independently o
 		const { chunkGraph, moduleGraph, dependencyTemplates, runtimeTemplate } =
 			this;
 		const results = this.codeGenerationResults;
+		/** @type {WebpackError[]} */
 		const errors = [];
 		/** @type {Set<Module> | undefined} */
 		let notCodeGeneratedModules = undefined;
@@ -3303,7 +3304,7 @@ Or do you want to use the entrypoints '${name}' and '${runtime}' independently o
 	 * @param {RuntimeTemplate} runtimeTemplate runtimeTemplate
 	 * @param {WebpackError[]} errors errors
 	 * @param {CodeGenerationResults} results results
-	 * @param {function(WebpackError=, boolean=): void} callback callback
+	 * @param {function((WebpackError | null)=, boolean=): void} callback callback
 	 */
 	_codeGenerationModule(
 		module,

package/lib/ContextModule.js

@@ -63,7 +63,7 @@ const makeSerializable = require("./util/makeSerializable");
  * @property {RawChunkGroupOptions=} groupOptions
  * @property {string=} typePrefix
  * @property {string=} category
- * @property {string[][]=} referencedExports exports referenced from modules (won't be mangled)
+ * @property {(string[][] | null)=} referencedExports exports referenced from modules (won't be mangled)
  * @property {string=} layer
  */
 
@@ -231,6 +231,9 @@ class ContextModule extends Module {
 		} else if (this.options.namespaceObject) {
 			identifier += "|namespace object";
 		}
+		if (this.layer) {
+			identifier += `|layer: ${this.layer}`;
+		}
 
 		return identifier;
 	}

package/lib/ContextReplacementPlugin.js

@@ -8,7 +8,15 @@
 const ContextElementDependency = require("./dependencies/ContextElementDependency");
 const { join } = require("./util/fs");
 
+/** @typedef {import("./Compiler")} Compiler */
+
 class ContextReplacementPlugin {
+	/**
+	 * @param {RegExp} resourceRegExp A regular expression that determines which files will be selected
+	 * @param {TODO=} newContentResource A new resource to replace the match
+	 * @param {TODO=} newContentRecursive If true, all subdirectories are searched for matches
+	 * @param {TODO=} newContentRegExp A regular expression that determines which files will be selected
+	 */
 	constructor(
 		resourceRegExp,
 		newContentResource,
@@ -49,6 +57,11 @@ class ContextReplacementPlugin {
 		}
 	}
 
+	/**
+	 * Apply the plugin
+	 * @param {Compiler} compiler the compiler instance
+	 * @returns {void}
+	 */
 	apply(compiler) {
 		const resourceRegExp = this.resourceRegExp;
 		const newContentCallback = this.newContentCallback;

package/lib/DelegatedModule.js

@@ -42,6 +42,13 @@ const RUNTIME_REQUIREMENTS = new Set([
 ]);
 
 class DelegatedModule extends Module {
+	/**
+	 * @param {string} sourceRequest source request
+	 * @param {TODO} data data
+	 * @param {"require" | "object"} type type
+	 * @param {string} userRequest user request
+	 * @param {string | Module} originalRequest original request
+	 */
 	constructor(sourceRequest, data, type, userRequest, originalRequest) {
 		super(JAVASCRIPT_MODULE_TYPE_DYNAMIC, null);
 
@@ -51,7 +58,7 @@ class DelegatedModule extends Module {
 		this.delegationType = type;
 		this.userRequest = userRequest;
 		this.originalRequest = originalRequest;
-		/** @type {ManifestModuleData} */
+		/** @type {ManifestModuleData | undefined} */
 		this.delegateData = data;
 
 		// Build info
@@ -110,7 +117,8 @@ class DelegatedModule extends Module {
 	 * @returns {void}
 	 */
 	build(options, compilation, resolver, fs, callback) {
-		this.buildMeta = { ...this.delegateData.buildMeta };
+		const delegateData = /** @type {ManifestModuleData} */ (this.delegateData);
+		this.buildMeta = { ...delegateData.buildMeta };
 		this.buildInfo = {};
 		this.dependencies.length = 0;
 		this.delegatedSourceDependency = new DelegatedSourceDependency(
@@ -118,7 +126,7 @@ class DelegatedModule extends Module {
 		);
 		this.addDependency(this.delegatedSourceDependency);
 		this.addDependency(
-			new StaticExportsDependency(this.delegateData.exports || true, false)
+			new StaticExportsDependency(delegateData.exports || true, false)
 		);
 		callback();
 	}
@@ -202,6 +210,10 @@ class DelegatedModule extends Module {
 		super.serialize(context);
 	}
 
+	/**
+	 * @param {ObjectDeserializerContext} context context\
+	 * @returns {DelegatedModule} DelegatedModule
+	 */
 	static deserialize(context) {
 		const { read } = context;
 		const obj = new DelegatedModule(

package/lib/DelegatedModuleFactoryPlugin.js

@@ -7,6 +7,8 @@
 
 const DelegatedModule = require("./DelegatedModule");
 
+/** @typedef {import("./NormalModuleFactory")} NormalModuleFactory */
+
 // options.source
 // options.type
 // options.context
@@ -20,6 +22,10 @@ class DelegatedModuleFactoryPlugin {
 		options.extensions = options.extensions || ["", ".js", ".json", ".wasm"];
 	}
 
+	/**
+	 * @param {NormalModuleFactory} normalModuleFactory the normal module factory
+	 * @returns {void}
+	 */
 	apply(normalModuleFactory) {
 		const scope = this.options.scope;
 		if (scope) {

package/lib/Dependency.js

@@ -89,9 +89,9 @@ const getIgnoredModule = memoize(() => {
 
 class Dependency {
 	constructor() {
-		/** @type {Module} */
+		/** @type {Module | undefined} */
 		this._parentModule = undefined;
-		/** @type {DependenciesBlock} */
+		/** @type {DependenciesBlock | undefined} */
 		this._parentDependenciesBlock = undefined;
 		/** @type {number} */
 		this._parentDependenciesBlockIndex = -1;
@@ -174,6 +174,12 @@ class Dependency {
 		this._loc = loc;
 	}
 
+	/**
+	 * @param {number} startLine start line
+	 * @param {number} startColumn start column
+	 * @param {number} endLine end line
+	 * @param {number} endColumn end column
+	 */
 	setLoc(startLine, startColumn, endLine, endColumn) {
 		this._locSL = startLine;
 		this._locSC = startColumn;
@@ -247,7 +253,7 @@ class Dependency {
 	/**
 	 * Returns warnings
 	 * @param {ModuleGraph} moduleGraph module graph
-	 * @returns {WebpackError[]} warnings
+	 * @returns {WebpackError[] | null | undefined} warnings
 	 */
 	getWarnings(moduleGraph) {
 		return null;
@@ -256,7 +262,7 @@ class Dependency {
 	/**
 	 * Returns errors
 	 * @param {ModuleGraph} moduleGraph module graph
-	 * @returns {WebpackError[]} errors
+	 * @returns {WebpackError[] | null | undefined} errors
 	 */
 	getErrors(moduleGraph) {
 		return null;
@@ -288,7 +294,7 @@ class Dependency {
 
 	/**
 	 * @param {string} context context directory
-	 * @returns {Module} a module
+	 * @returns {Module | null} a module
 	 */
 	createIgnoredModule(context) {
 		return getIgnoredModule();

package/lib/DllEntryPlugin.js

@@ -9,6 +9,8 @@ const DllModuleFactory = require("./DllModuleFactory");
 const DllEntryDependency = require("./dependencies/DllEntryDependency");
 const EntryDependency = require("./dependencies/EntryDependency");
 
+/** @typedef {import("./Compiler")} Compiler */
+
 class DllEntryPlugin {
 	/**
 	 * @param {string} context context
@@ -21,6 +23,11 @@ class DllEntryPlugin {
 		this.options = options;
 	}
 
+	/**
+	 * Apply the plugin
+	 * @param {Compiler} compiler the compiler instance
+	 * @returns {void}
+	 */
 	apply(compiler) {
 		compiler.hooks.compilation.tap(
 			"DllEntryPlugin",
@@ -51,7 +58,10 @@ class DllEntryPlugin {
 					this.options.name
 				),
 				this.options,
-				callback
+				error => {
+					if (error) return callback(error);
+					callback();
+				}
 			);
 		});
 	}

package/lib/DllReferencePlugin.js

@@ -16,6 +16,7 @@ const makePathsRelative = require("./util/identifier").makePathsRelative;
 /** @typedef {import("../declarations/WebpackOptions").Externals} Externals */
 /** @typedef {import("../declarations/plugins/DllReferencePlugin").DllReferencePluginOptions} DllReferencePluginOptions */
 /** @typedef {import("../declarations/plugins/DllReferencePlugin").DllReferencePluginOptionsManifest} DllReferencePluginOptionsManifest */
+/** @typedef {import("./Compiler")} Compiler */
 
 const validate = createSchemaValidation(
 	require("../schemas/plugins/DllReferencePlugin.check.js"),
@@ -37,6 +38,11 @@ class DllReferencePlugin {
 		this._compilationData = new WeakMap();
 	}
 
+	/**
+	 * Apply the plugin
+	 * @param {Compiler} compiler the compiler instance
+	 * @returns {void}
+	 */
 	apply(compiler) {
 		compiler.hooks.compilation.tap(
 			"DllReferencePlugin",
@@ -140,7 +146,9 @@ class DllReferencePlugin {
 						// If there was an error parsing the manifest file, add the
 						// error as a compilation error to make the compilation fail.
 						if (data.error) {
-							compilation.errors.push(data.error);
+							compilation.errors.push(
+								/** @type {DllManifestError} */ (data.error)
+							);
 						}
 						compilation.fileDependencies.add(manifest);
 					}
@@ -151,6 +159,10 @@ class DllReferencePlugin {
 }
 
 class DllManifestError extends WebpackError {
+	/**
+	 * @param {string} filename filename of the manifest
+	 * @param {string} message error message
+	 */
 	constructor(filename, message) {
 		super();
 

package/lib/Entrypoint.js

@@ -83,7 +83,7 @@ class Entrypoint extends ChunkGroup {
 	 * @returns {Chunk} chunk
 	 */
 	getEntrypointChunk() {
-		return this._entrypointChunk;
+		return /** @type {Chunk} */ (this._entrypointChunk);
 	}
 
 	/**

package/lib/ErrorHelpers.js

@@ -45,6 +45,7 @@ const cutOffMultilineMessage = (stack, message) => {
 	const stackSplitByLines = stack.split("\n");
 	const messageSplitByLines = message.split("\n");
 
+	/** @type {string[]} */
 	const result = [];
 
 	stackSplitByLines.forEach((line, idx) => {

package/lib/EvalSourceMapDevToolPlugin.js

@@ -82,6 +82,10 @@ class EvalSourceMapDevToolPlugin {
 							return cachedSource;
 						}
 
+						/**
+						 * @param {Source} r result
+						 * @returns {Source} result
+						 */
 						const result = r => {
 							cache.set(source, r);
 							return r;

package/lib/ExportsInfo.js

@@ -14,6 +14,8 @@ const { forEachRuntime } = require("./util/runtime");
 /** @typedef {import("./Module")} Module */
 /** @typedef {import("./ModuleGraph")} ModuleGraph */
 /** @typedef {import("./ModuleGraphConnection")} ModuleGraphConnection */
+/** @typedef {import("./serialization/ObjectMiddleware").ObjectDeserializerContext} ObjectDeserializerContext */
+/** @typedef {import("./serialization/ObjectMiddleware").ObjectSerializerContext} ObjectSerializerContext */
 /** @typedef {import("./util/Hash")} Hash */
 
 /** @typedef {typeof UsageState.OnlyPropertiesUsed | typeof UsageState.NoInfo | typeof UsageState.Unknown | typeof UsageState.Used} RuntimeUsageStateType */
@@ -44,6 +46,9 @@ class RestoreProvidedData {
 		this.otherTerminalBinding = otherTerminalBinding;
 	}
 
+	/**
+	 * @param {ObjectSerializerContext} context context
+	 */
 	serialize({ write }) {
 		write(this.exports);
 		write(this.otherProvided);
@@ -51,6 +56,10 @@ class RestoreProvidedData {
 		write(this.otherTerminalBinding);
 	}
 
+	/**
+	 * @param {ObjectDeserializerContext} context context
+	 * @returns {RestoreProvidedData} RestoreProvidedData
+	 */
 	static deserialize({ read }) {
 		return new RestoreProvidedData(read(), read(), read(), read());
 	}
@@ -301,7 +310,12 @@ class ExportsInfo {
 				changed = true;
 			}
 			if (targetKey) {
-				exportInfo.setTarget(targetKey, targetModule, [exportInfo.name], -1);
+				exportInfo.setTarget(
+					targetKey,
+					/** @type {ModuleGraphConnection} */ (targetModule),
+					[exportInfo.name],
+					-1
+				);
 			}
 		}
 		if (this._redirectTo !== undefined) {
@@ -331,7 +345,7 @@ class ExportsInfo {
 			if (targetKey) {
 				this._otherExportsInfo.setTarget(
 					targetKey,
-					targetModule,
+					/** @type {ModuleGraphConnection} */ (targetModule),
 					undefined,
 					priority
 				);
@@ -1226,7 +1240,7 @@ class ExportInfo {
 	/**
 	 * @param {ModuleGraph} moduleGraph the module graph
 	 * @param {function(Module): boolean} validTargetModuleFilter a valid target module
-	 * @param {Set<ExportInfo> | undefined} alreadyVisited set of already visited export info to avoid circular references
+	 * @param {Set<ExportInfo>} alreadyVisited set of already visited export info to avoid circular references
 	 * @returns {{ module: Module, export: string[] | undefined } | undefined | false} the target, undefined when there is no target, false when no target is valid
 	 */
 	_findTarget(moduleGraph, validTargetModuleFilter, alreadyVisited) {

package/lib/ExportsInfoApiPlugin.js

@@ -14,7 +14,9 @@ const ConstDependency = require("./dependencies/ConstDependency");
 const ExportsInfoDependency = require("./dependencies/ExportsInfoDependency");
 
 /** @typedef {import("./Compiler")} Compiler */
+/** @typedef {import("./Dependency").DependencyLocation} DependencyLocation */
 /** @typedef {import("./javascript/JavascriptParser")} JavascriptParser */
+/** @typedef {import("./javascript/JavascriptParser").Range} Range */
 
 const PLUGIN_NAME = "ExportsInfoApiPlugin";
 
@@ -43,20 +45,27 @@ class ExportsInfoApiPlugin {
 							const dep =
 								members.length >= 2
 									? new ExportsInfoDependency(
-											expr.range,
+											/** @type {Range} */ (expr.range),
 											members.slice(0, -1),
 											members[members.length - 1]
 									  )
-									: new ExportsInfoDependency(expr.range, null, members[0]);
-							dep.loc = expr.loc;
+									: new ExportsInfoDependency(
+											/** @type {Range} */ (expr.range),
+											null,
+											members[0]
+									  );
+							dep.loc = /** @type {DependencyLocation} */ (expr.loc);
 							parser.state.module.addDependency(dep);
 							return true;
 						});
 					parser.hooks.expression
 						.for("__webpack_exports_info__")
 						.tap(PLUGIN_NAME, expr => {
-							const dep = new ConstDependency("true", expr.range);
-							dep.loc = expr.loc;
+							const dep = new ConstDependency(
+								"true",
+								/** @type {Range} */ (expr.range)
+							);
+							dep.loc = /** @type {DependencyLocation} */ (expr.loc);
 							parser.state.module.addPresentationalDependency(dep);
 							return true;
 						});

package/lib/ExternalModule.js

@@ -384,6 +384,11 @@ const getSourceForDefaultCase = (optional, request, runtimeTemplate) => {
 };
 
 class ExternalModule extends Module {
+	/**
+	 * @param {string | string[] | Record<string, string | string[]>} request request
+	 * @param {TODO} type type
+	 * @param {string} userRequest user request
+	 */
 	constructor(request, type, userRequest) {
 		super(JAVASCRIPT_MODULE_TYPE_DYNAMIC, null);
 

package/lib/FlagAllModulesAsUsedPlugin.js

@@ -8,10 +8,14 @@
 const { getEntryRuntime, mergeRuntimeOwned } = require("./util/runtime");
 
 /** @typedef {import("./Compiler")} Compiler */
+/** @typedef {import("./Module").FactoryMeta} FactoryMeta */
 /** @typedef {import("./util/runtime").RuntimeSpec} RuntimeSpec */
 
 const PLUGIN_NAME = "FlagAllModulesAsUsedPlugin";
 class FlagAllModulesAsUsedPlugin {
+	/**
+	 * @param {string} explanation explanation
+	 */
 	constructor(explanation) {
 		this.explanation = explanation;
 	}
@@ -40,7 +44,8 @@ class FlagAllModulesAsUsedPlugin {
 					if (module.factoryMeta === undefined) {
 						module.factoryMeta = {};
 					}
-					module.factoryMeta.sideEffectFree = false;
+					/** @type {FactoryMeta} */
+					(module.factoryMeta).sideEffectFree = false;
 				}
 			});
 		});

package/lib/FlagDependencyUsagePlugin.js

@@ -332,7 +332,9 @@ class FlagDependencyUsagePlugin {
 					}
 
 					while (queue.length) {
-						const [module, runtime] = queue.dequeue();
+						const [module, runtime] = /** @type {[Module, RuntimeSpec]} */ (
+							queue.dequeue()
+						);
 						processModule(module, runtime, false);
 					}
 					logger.timeEnd("trace exports usage in graph");