webpack 5.85.0 → 5.88.1

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/DependencyTemplates.js

@@ -27,7 +27,7 @@ class DependencyTemplates {
 
 	/**
 	 * @param {DependencyConstructor} dependency Constructor of Dependency
-	 * @returns {DependencyTemplate} template for this dependency
+	 * @returns {DependencyTemplate | undefined} template for this dependency
 	 */
 	get(dependency) {
 		return this._map.get(dependency);

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/DllModule.js

@@ -15,6 +15,7 @@ const makeSerializable = require("./util/makeSerializable");
 /** @typedef {import("../declarations/WebpackOptions").WebpackOptionsNormalized} WebpackOptions */
 /** @typedef {import("./ChunkGraph")} ChunkGraph */
 /** @typedef {import("./Compilation")} Compilation */
+/** @typedef {import("./Dependency")} Dependency */
 /** @typedef {import("./Dependency").UpdateHashContext} UpdateHashContext */
 /** @typedef {import("./DependencyTemplates")} DependencyTemplates */
 /** @typedef {import("./Module").CodeGenerationContext} CodeGenerationContext */
@@ -37,6 +38,11 @@ const RUNTIME_REQUIREMENTS = new Set([
 ]);
 
 class DllModule extends Module {
+	/**
+	 * @param {string} context context path
+	 * @param {Dependency[]} dependencies dependencies
+	 * @param {string} name name
+	 */
 	constructor(context, dependencies, name) {
 		super(JAVASCRIPT_MODULE_TYPE_DYNAMIC, context);
 

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;
@@ -160,7 +164,8 @@ class EvalSourceMapDevToolPlugin {
 						}
 						sourceMap.sourceRoot = options.sourceRoot || "";
 						const moduleId = chunkGraph.getModuleId(m);
-						sourceMap.file = `${moduleId}.js`;
+						sourceMap.file =
+							typeof moduleId === "number" ? `${moduleId}.js` : moduleId;
 
 						const footer =
 							this.sourceMapComment.replace(

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) {
@@ -1350,7 +1364,11 @@ class ExportInfo {
 			if (t === null) return undefined;
 			if (t.module !== target.module) return undefined;
 			if (!t.export !== !target.export) return undefined;
-			if (target.export && !equals(t.export, target.export)) return undefined;
+			if (
+				target.export &&
+				!equals(/** @type {ArrayLike<string>} */ (t.export), target.export)
+			)
+				return undefined;
 			result = values.next();
 		}
 		return target;

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

@@ -104,9 +104,13 @@ const getSourceForCommonJsExternal = moduleAndSpecifiers => {
 
 /**
  * @param {string|string[]} moduleAndSpecifiers the module request
+ * @param {string} importMetaName import.meta name
  * @returns {SourceData} the generated source
  */
-const getSourceForCommonJsExternalInNodeModule = moduleAndSpecifiers => {
+const getSourceForCommonJsExternalInNodeModule = (
+	moduleAndSpecifiers,
+	importMetaName
+) => {
 	const chunkInitFragments = [
 		new InitFragment(
 			'import { createRequire as __WEBPACK_EXTERNAL_createRequire } from "module";\n',
@@ -117,18 +121,18 @@ const getSourceForCommonJsExternalInNodeModule = moduleAndSpecifiers => {
 	];
 	if (!Array.isArray(moduleAndSpecifiers)) {
 		return {
-			expression: `__WEBPACK_EXTERNAL_createRequire(import.meta.url)(${JSON.stringify(
+			chunkInitFragments,
+			expression: `__WEBPACK_EXTERNAL_createRequire(${importMetaName}.url)(${JSON.stringify(
 				moduleAndSpecifiers
-			)})`,
-			chunkInitFragments
+			)})`
 		};
 	}
 	const moduleName = moduleAndSpecifiers[0];
 	return {
-		expression: `__WEBPACK_EXTERNAL_createRequire(import.meta.url)(${JSON.stringify(
+		chunkInitFragments,
+		expression: `__WEBPACK_EXTERNAL_createRequire(${importMetaName}.url)(${JSON.stringify(
 			moduleName
-		)})${propertyAccess(moduleAndSpecifiers, 1)}`,
-		chunkInitFragments
+		)})${propertyAccess(moduleAndSpecifiers, 1)}`
 	};
 };
 
@@ -380,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);
 
@@ -557,7 +566,10 @@ class ExternalModule extends Module {
 				return getSourceForCommonJsExternal(request);
 			case "node-commonjs":
 				return this.buildInfo.module
-					? getSourceForCommonJsExternalInNodeModule(request)
+					? getSourceForCommonJsExternalInNodeModule(
+							request,
+							runtimeTemplate.outputOptions.importMetaName
+					  )
 					: getSourceForCommonJsExternal(request);
 			case "amd":
 			case "amd-require":

package/lib/FileSystemInfo.js

@@ -99,8 +99,8 @@ const INVALID = Symbol("invalid");
  * @typedef {Object} SnapshotOptimizationEntry
  * @property {Snapshot} snapshot
  * @property {number} shared
- * @property {Set<string>} snapshotContent
- * @property {Set<SnapshotOptimizationEntry>} children
+ * @property {Set<string> | undefined} snapshotContent
+ * @property {Set<SnapshotOptimizationEntry> | undefined} children
  */
 
 /**
@@ -115,6 +115,12 @@ const INVALID = Symbol("invalid");
  * @property {Set<string>} resolveDependencies.missing list of missing entries
  */
 
+/**
+ * @typedef {Object} SnapshotOptions
+ * @property {boolean=} hash should use hash to snapshot
+ * @property {boolean=} timestamp should use timestamp to snapshot
+ */
+
 const DONE_ITERATOR_RESULT = new Set().keys().next();
 
 // cspell:word tshs
@@ -137,7 +143,7 @@ class SnapshotIterable {
 		let state = 0;
 		/** @type {IterableIterator<string>} */
 		let it;
-		/** @type {(Snapshot) => (Map<string, any> | Set<string>)[]} */
+		/** @type {(snapshot: Snapshot) => (Map<string, any> | Set<string>)[]} */
 		let getMaps;
 		/** @type {(Map<string, any> | Set<string>)[]} */
 		let maps;
@@ -552,7 +558,7 @@ class SnapshotOptimization {
 			}
 		};
 
-		/** @type {SnapshotOptimizationEntry} */
+		/** @type {SnapshotOptimizationEntry | undefined} */
 		let newOptimizationEntry = undefined;
 
 		const capturedFilesSize = capturedFiles.size;
@@ -757,10 +763,9 @@ const mergeMaps = (a, b) => {
 
 /**
  * @template T
- * @template K
- * @param {Set<T, K>} a source map
- * @param {Set<T, K>} b joining map
- * @returns {Set<T, K>} joined map
+ * @param {Set<T>} a source map
+ * @param {Set<T>} b joining map
+ * @returns {Set<T>} joined map
  */
 const mergeSets = (a, b) => {
 	if (!b || b.size === 0) return a;
@@ -858,8 +863,8 @@ const getManagedItem = (managedPath, path) => {
 
 /**
  * @template {ContextFileSystemInfoEntry | ContextTimestampAndHash} T
- * @param {T} entry entry
- * @returns {T["resolved"] | undefined} the resolved entry
+ * @param {T | null} entry entry
+ * @returns {T["resolved"] | null | undefined} the resolved entry
  */
 const getResolvedTimestamp = entry => {
 	if (entry === null) return null;
@@ -868,8 +873,8 @@ const getResolvedTimestamp = entry => {
 };
 
 /**
- * @param {ContextHash} entry entry
- * @returns {string | undefined} the resolved entry
+ * @param {ContextHash | null} entry entry
+ * @returns {string | null | undefined} the resolved entry
  */
 const getResolvedHash = entry => {
 	if (entry === null) return null;
@@ -877,6 +882,11 @@ const getResolvedHash = entry => {
 	return entry.symlinks === undefined ? entry.hash : undefined;
 };
 
+/**
+ * @template T
+ * @param {Set<T>} source source
+ * @param {Set<T>} target target
+ */
 const addAll = (source, target) => {
 	for (const key of source) target.add(key);
 };
@@ -1145,6 +1155,11 @@ class FileSystemInfo {
 		);
 	}
 
+	/**
+	 * @param {string} path path
+	 * @param {string} reason reason
+	 * @param {any[]} args arguments
+	 */
 	_log(path, reason, ...args) {
 		const key = path + reason;
 		if (this._loggedPaths.has(key)) return;
@@ -1258,7 +1273,7 @@ class FileSystemInfo {
 
 	/**
 	 * @param {string} path file path
-	 * @param {function((WebpackError | null)=, string=): void} callback callback function
+	 * @param {function((WebpackError | null)=, (string | null)=): void} callback callback function
 	 * @returns {void}
 	 */
 	getFileHash(path, callback) {
@@ -1289,7 +1304,7 @@ class FileSystemInfo {
 
 	/**
 	 * @param {string} path context path
-	 * @param {function((WebpackError | null)=, ContextHash=): void} callback callback function
+	 * @param {function((WebpackError | null)=, (ContextHash | null)=): void} callback callback function
 	 * @returns {void}
 	 */
 	_getUnresolvedContextHash(path, callback) {
@@ -1320,7 +1335,7 @@ class FileSystemInfo {
 
 	/**
 	 * @param {string} path context path
-	 * @param {function((WebpackError | null)=, ContextTimestampAndHash=): void} callback callback function
+	 * @param {function((WebpackError | null)=, (ContextTimestampAndHash | null)=): void} callback callback function
 	 * @returns {void}
 	 */
 	_getUnresolvedContextTsh(path, callback) {
@@ -1383,7 +1398,7 @@ class FileSystemInfo {
 		const resolveDirectories = new Set();
 		/** @type {Set<string>} */
 		const resolveMissing = new Set();
-		/** @type {Map<string, string | false>} */
+		/** @type {Map<string, string | false | undefined>} */
 		const resolveResults = new Map();
 		const invalidResolveResults = new Set();
 		const resolverContext = {
@@ -1391,6 +1406,10 @@ class FileSystemInfo {
 			contextDependencies: resolveDirectories,
 			missingDependencies: resolveMissing
 		};
+		/**
+		 * @param {string} expected expected result
+		 * @returns {string} expected result
+		 */
 		const expectedToString = expected => {
 			return expected ? ` (expected ${expected})` : "";
 		};
@@ -1610,7 +1629,7 @@ class FileSystemInfo {
 							break;
 						}
 						// Check commonjs cache for the module
-						/** @type {NodeModule} */
+						/** @type {NodeModule | undefined} */
 						const module = require.cache[path];
 						if (module && Array.isArray(module.children)) {
 							children: for (const child of module.children) {
@@ -1910,13 +1929,11 @@ class FileSystemInfo {
 
 	/**
 	 *
-	 * @param {number} startTime when processing the files has started
+	 * @param {number | null | undefined} startTime when processing the files has started
 	 * @param {Iterable<string>} files all files
 	 * @param {Iterable<string>} directories all directories
 	 * @param {Iterable<string>} missing all missing files or directories
-	 * @param {Object} options options object (for future extensions)
-	 * @param {boolean=} options.hash should use hash to snapshot
-	 * @param {boolean=} options.timestamp should use timestamp to snapshot
+	 * @param {SnapshotOptions | null | undefined} options options object (for future extensions)
 	 * @param {function((WebpackError | null)=, (Snapshot | null)=): void} callback callback function
 	 * @returns {void}
 	 */
@@ -2053,6 +2070,9 @@ class FileSystemInfo {
 			}
 			return capturedItems;
 		};
+		/**
+		 * @param {Set<string>} capturedFiles captured files
+		 */
 		const processCapturedFiles = capturedFiles => {
 			switch (mode) {
 				case 3:
@@ -2639,6 +2659,10 @@ class FileSystemInfo {
 				}
 			}
 		}
+		/**
+		 * @param {string} path file path
+		 * @param {string} hash hash
+		 */
 		const processFileHashSnapshot = (path, hash) => {
 			const cache = this._fileHashes.get(path);
 			if (cache !== undefined) {
@@ -2921,7 +2945,7 @@ class FileSystemInfo {
 
 			const hash = createHash(this._hashFunction);
 
-			hash.update(content);
+			hash.update(/** @type {string | Buffer} */ (content));
 
 			const digest = /** @type {string} */ (hash.digest("hex"));
 
@@ -2985,7 +3009,7 @@ class FileSystemInfo {
 	 * @param {function(string, IStats, function(Error=, ItemType=): void): void} options.fromFile called when context item is a file
 	 * @param {function(string, IStats, function(Error=, ItemType=): void): void} options.fromDirectory called when context item is a directory
 	 * @param {function(string[], ItemType[]): T} options.reduce called from all context items
-	 * @param {function((Error | null)=, (T)=): void} callback callback
+	 * @param {function((Error | null)=, (T | null)=): void} callback callback
 	 */
 	_readContext(
 		{
@@ -3172,6 +3196,7 @@ class FileSystemInfo {
 	 * @returns {void}
 	 */
 	_resolveContextTimestamp(entry, callback) {
+		/** @type {string[]} */
 		const hashes = [];
 		let safeTime = 0;
 		processAsyncTree(
@@ -3280,6 +3305,7 @@ class FileSystemInfo {
 	 * @returns {void}
 	 */
 	_resolveContextHash(entry, callback) {
+		/** @type {string[]} */
 		const hashes = [];
 		processAsyncTree(
 			entry.symlinks,
@@ -3436,7 +3462,9 @@ class FileSystemInfo {
 	 * @returns {void}
 	 */
 	_resolveContextTsh(entry, callback) {
+		/** @type {string[]} */
 		const hashes = [];
+		/** @type {string[]} */
 		const tsHashes = [];
 		let safeTime = 0;
 		processAsyncTree(
@@ -3554,7 +3582,7 @@ class FileSystemInfo {
 				}
 				let data;
 				try {
-					data = JSON.parse(content.toString("utf-8"));
+					data = JSON.parse(/** @type {Buffer} */ (content).toString("utf-8"));
 				} catch (e) {
 					return callback(e);
 				}

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");

package/lib/FlagEntryExportAsUsedPlugin.js

@@ -12,6 +12,10 @@ const { getEntryRuntime } = require("./util/runtime");
 const PLUGIN_NAME = "FlagEntryExportAsUsedPlugin";
 
 class FlagEntryExportAsUsedPlugin {
+	/**
+	 * @param {boolean} nsObjectUsed true, if the ns object is used
+	 * @param {string} explanation explanation for the reason
+	 */
 	constructor(nsObjectUsed, explanation) {
 		this.nsObjectUsed = nsObjectUsed;
 		this.explanation = explanation;