webpack 5.83.1 → 5.85.0

package/README.md

@@ -32,8 +32,8 @@
 	<a href="https://github.com/webpack/webpack/graphs/contributors">
 		<img src="https://img.shields.io/github/contributors/webpack/webpack.svg">
 	</a>
-	<a href="https://gitter.im/webpack/webpack">
-		<img src="https://badges.gitter.im/webpack/webpack.svg">
+	<a href="https://github.com/webpack/webpack/discussions">
+		<img src="https://img.shields.io/github/discussions/webpack/webpack">
 	</a>
   <a href="https://twitter.com/Webpack">
 		<img src="https://img.shields.io/twitter/follow/Webpack?style=social">

package/lib/AsyncDependenciesBlock.js

@@ -39,14 +39,14 @@ class AsyncDependenciesBlock extends DependenciesBlock {
 	}
 
 	/**
-	 * @returns {string} The name of the chunk
+	 * @returns {string | undefined} The name of the chunk
 	 */
 	get chunkName() {
 		return this.groupOptions.name;
 	}
 
 	/**
-	 * @param {string} value The new chunk name
+	 * @param {string | undefined} value The new chunk name
 	 * @returns {void}
 	 */
 	set chunkName(value) {

package/lib/AutomaticPrefetchPlugin.js

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

package/lib/BannerPlugin.js

@@ -24,6 +24,10 @@ const validate = createSchemaValidation(
 	}
 );
 
+/**
+ * @param {string} str string to wrap
+ * @returns {string} wrapped string
+ */
 const wrapComment = str => {
 	if (!str.includes("\n")) {
 		return Template.toComment(str);

package/lib/CacheFacade.js

@@ -55,6 +55,10 @@ class MultiItemCache {
 	 * @returns {Promise<T>} promise with the data
 	 */
 	getPromise() {
+		/**
+		 * @param {number} i index
+		 * @returns {Promise<T>} promise with the data
+		 */
 		const next = i => {
 			return this._items[i].getPromise().then(result => {
 				if (result !== undefined) return result;

package/lib/ChunkGroup.js

@@ -353,7 +353,7 @@ class ChunkGroup {
 	}
 
 	/**
-	 * @returns {Array} an array containing the blocks
+	 * @returns {Array<AsyncDependenciesBlock>} an array containing the blocks
 	 */
 	getBlocks() {
 		return this._blocks.getFromCache(getArray);
@@ -363,6 +363,10 @@ class ChunkGroup {
 		return this._blocks.size;
 	}
 
+	/**
+	 * @param {AsyncDependenciesBlock} block block
+	 * @returns {boolean} true, if block exists
+	 */
 	hasBlock(block) {
 		return this._blocks.has(block);
 	}

package/lib/CleanPlugin.js

@@ -78,7 +78,8 @@ const getDiffToFs = (fs, outputPath, currentAssets, callback) => {
 		directories,
 		10,
 		(directory, callback) => {
-			fs.readdir(join(fs, outputPath, directory), (err, entries) => {
+			/** @type {NonNullable<OutputFileSystem["readdir"]>} */
+			(fs.readdir)(join(fs, outputPath, directory), (err, entries) => {
 				if (err) {
 					if (err.code === "ENOENT") return callback();
 					if (err.code === "ENOTDIR") {
@@ -128,7 +129,8 @@ const getDiffToOldAssets = (currentAssets, oldAssets) => {
  */
 const doStat = (fs, filename, callback) => {
 	if ("lstat" in fs) {
-		fs.lstat(filename, callback);
+		/** @type {NonNullable<OutputFileSystem["lstat"]>} */
+		(fs.lstat)(filename, callback);
 	} else {
 		fs.stat(filename, callback);
 	}
@@ -145,6 +147,9 @@ const doStat = (fs, filename, callback) => {
  * @returns {void}
  */
 const applyDiff = (fs, outputPath, dry, logger, diff, isKept, callback) => {
+	/**
+	 * @param {string} msg message
+	 */
 	const log = msg => {
 		if (dry) {
 			logger.info(msg);
@@ -165,6 +170,10 @@ const applyDiff = (fs, outputPath, dry, logger, diff, isKept, callback) => {
 		jobs,
 		10,
 		({ type, filename, parent }, push, callback) => {
+			/**
+			 * @param {Error & { code?: string }} err error
+			 * @returns {void}
+			 */
 			const handleError = err => {
 				if (err.code === "ENOENT") {
 					log(`${filename} was removed during cleaning by something else`);
@@ -195,7 +204,9 @@ const applyDiff = (fs, outputPath, dry, logger, diff, isKept, callback) => {
 							});
 							return callback();
 						}
-						fs.readdir(path, (err, entries) => {
+
+						/** @type {NonNullable<OutputFileSystem["readdir"]>} */
+						(fs.readdir)(path, (err, entries) => {
 							if (err) return handleError(err);
 							/** @type {Job} */
 							const deleteJob = {
@@ -317,9 +328,17 @@ class CleanPlugin {
 			typeof keep === "function"
 				? keep
 				: typeof keep === "string"
-				? path => path.startsWith(keep)
+				? /**
+				   * @param {string} path path
+				   * @returns {boolean} true, if the path should be kept
+				   */
+				  path => path.startsWith(keep)
 				: typeof keep === "object" && keep.test
-				? path => keep.test(path)
+				? /**
+				   * @param {string} path path
+				   * @returns {boolean} true, if the path should be kept
+				   */
+				  path => keep.test(path)
 				: () => false;
 
 		// We assume that no external modification happens while the compiler is active
@@ -371,6 +390,10 @@ class CleanPlugin {
 
 				const outputPath = compilation.getPath(compiler.outputPath, {});
 
+				/**
+				 * @param {string} path path
+				 * @returns {boolean} true, if needs to be kept
+				 */
 				const isKept = path => {
 					const result = hooks.keep.call(path);
 					if (result !== undefined) return result;
@@ -378,7 +401,7 @@ class CleanPlugin {
 				};
 
 				/**
-				 * @param {Error=} err err
+				 * @param {(Error | null)=} err err
 				 * @param {Set<string>=} diff diff
 				 */
 				const diffCallback = (err, diff) => {
@@ -392,7 +415,7 @@ class CleanPlugin {
 						outputPath,
 						dry,
 						logger,
-						diff,
+						/** @type {Set<string>} */ (diff),
 						isKept,
 						(err, keptAssets) => {
 							if (err) {

package/lib/CompatibilityPlugin.js

@@ -10,6 +10,7 @@ const {
 	JAVASCRIPT_MODULE_TYPE_DYNAMIC,
 	JAVASCRIPT_MODULE_TYPE_ESM
 } = require("./ModuleTypeConstants");
+const RuntimeGlobals = require("./RuntimeGlobals");
 const ConstDependency = require("./dependencies/ConstDependency");
 
 /** @typedef {import("./Compiler")} Compiler */
@@ -79,7 +80,7 @@ class CompatibilityPlugin {
 						if (
 							statement.type === "FunctionDeclaration" &&
 							statement.id &&
-							statement.id.name === "__webpack_require__"
+							statement.id.name === RuntimeGlobals.require
 						) {
 							const newName = `__nested_webpack_require_${statement.range[0]}__`;
 							parser.tagVariable(
@@ -98,7 +99,7 @@ class CompatibilityPlugin {
 						}
 					});
 					parser.hooks.pattern
-						.for("__webpack_require__")
+						.for(RuntimeGlobals.require)
 						.tap(PLUGIN_NAME, pattern => {
 							const newName = `__nested_webpack_require_${pattern.range[0]}__`;
 							parser.tagVariable(pattern.name, nestedWebpackIdentifierTag, {
@@ -112,7 +113,7 @@ class CompatibilityPlugin {
 							return true;
 						});
 					parser.hooks.pattern
-						.for("__webpack_exports__")
+						.for(RuntimeGlobals.exports)
 						.tap(PLUGIN_NAME, pattern => {
 							parser.tagVariable(pattern.name, nestedWebpackIdentifierTag, {
 								name: "__nested_webpack_exports__",

package/lib/Compilation.js

@@ -681,7 +681,7 @@ BREAKING CHANGE: Asset processing hooks in Compilation has been merged into a si
 			optimizeChunkModules: new AsyncSeriesBailHook(["chunks", "modules"]),
 			/** @type {SyncHook<[Iterable<Chunk>, Iterable<Module>]>} */
 			afterOptimizeChunkModules: new SyncHook(["chunks", "modules"]),
-			/** @type {SyncBailHook<[], boolean>} */
+			/** @type {SyncBailHook<[], boolean | undefined>} */
 			shouldRecord: new SyncBailHook([]),
 
 			/** @type {SyncHook<[Chunk, Set<string>, RuntimeRequirementsContext]>} */
@@ -5024,13 +5024,13 @@ This prevents using hashes of each other and should be avoided.`);
 									};
 									const interceptModuleExecution = (__webpack_require__[
 										RuntimeGlobals.interceptModuleExecution.replace(
-											"__webpack_require__.",
+											`${RuntimeGlobals.require}.`,
 											""
 										)
 									] = []);
 									const moduleCache = (__webpack_require__[
 										RuntimeGlobals.moduleCache.replace(
-											"__webpack_require__.",
+											`${RuntimeGlobals.require}.`,
 											""
 										)
 									] = {});

package/lib/Compiler.js

@@ -126,7 +126,7 @@ class Compiler {
 			/** @type {SyncHook<[]>} */
 			initialize: new SyncHook([]),
 
-			/** @type {SyncBailHook<[Compilation], boolean>} */
+			/** @type {SyncBailHook<[Compilation], boolean | undefined>} */
 			shouldEmit: new SyncBailHook(["compilation"]),
 			/** @type {AsyncSeriesHook<[Stats]>} */
 			done: new AsyncSeriesHook(["stats"]),

package/lib/ContextModule.js

@@ -640,7 +640,7 @@ class ContextModule extends Module {
 
 	getReturn(type, asyncModule) {
 		if (type === 9) {
-			return "__webpack_require__(id)";
+			return `${RuntimeGlobals.require}(id)`;
 		}
 		return `${RuntimeGlobals.createFakeNamespaceObject}(id, ${type}${
 			asyncModule ? " | 16" : ""
@@ -799,7 +799,7 @@ module.exports = webpackAsyncContext;`;
 				? `${arrow ? "id =>" : "function(id)"} {
 		${this.getReturnModuleObjectSource(fakeMap)}
 	}`
-				: "__webpack_require__";
+				: RuntimeGlobals.require;
 		return `var map = ${JSON.stringify(map, null, "\t")};
 ${this.getFakeMapInitStatement(fakeMap)}
 
@@ -850,7 +850,7 @@ module.exports = webpackAsyncContext;`;
 				? `${arrow ? "id =>" : "function(id)"} {
 		${this.getReturnModuleObjectSource(fakeMap, true)};
 	}`
-				: "__webpack_require__";
+				: RuntimeGlobals.require;
 
 		return `var map = ${JSON.stringify(map, null, "\t")};
 ${this.getFakeMapInitStatement(fakeMap)}

package/lib/ContextModuleFactory.js

@@ -80,7 +80,7 @@ module.exports = class ContextModuleFactory extends ModuleFactory {
 
 	/**
 	 * @param {ModuleFactoryCreateData} data data object
-	 * @param {function(Error=, ModuleFactoryResult=): void} callback callback
+	 * @param {function((Error | null)=, ModuleFactoryResult=): void} callback callback
 	 * @returns {void}
 	 */
 	create(data, callback) {

package/lib/DefinePlugin.js

@@ -463,7 +463,7 @@ class DefinePlugin {
 								});
 							parser.hooks.expression.for(key).tap(PLUGIN_NAME, expr => {
 								addValueDependency(originalKey);
-								const strCode = toCode(
+								let strCode = toCode(
 									code,
 									parser,
 									compilation.valueCacheVersions,
@@ -473,6 +473,11 @@ class DefinePlugin {
 									!parser.isAsiPosition(expr.range[0]),
 									parser.destructuringAssignmentPropertiesFor(expr)
 								);
+
+								if (parser.scope.inShorthand) {
+									strCode = parser.scope.inShorthand + ":" + strCode;
+								}
+
 								if (WEBPACK_REQUIRE_FUNCTION_REGEXP.test(strCode)) {
 									return toConstantDependency(parser, strCode, [
 										RuntimeGlobals.require
@@ -564,7 +569,7 @@ class DefinePlugin {
 							);
 						parser.hooks.expression.for(key).tap(PLUGIN_NAME, expr => {
 							addValueDependency(key);
-							const strCode = stringifyObj(
+							let strCode = stringifyObj(
 								obj,
 								parser,
 								compilation.valueCacheVersions,
@@ -575,6 +580,10 @@ class DefinePlugin {
 								parser.destructuringAssignmentPropertiesFor(expr)
 							);
 
+							if (parser.scope.inShorthand) {
+								strCode = parser.scope.inShorthand + ":" + strCode;
+							}
+
 							if (WEBPACK_REQUIRE_FUNCTION_REGEXP.test(strCode)) {
 								return toConstantDependency(parser, strCode, [
 									RuntimeGlobals.require

package/lib/DllEntryPlugin.js

@@ -10,6 +10,11 @@ const DllEntryDependency = require("./dependencies/DllEntryDependency");
 const EntryDependency = require("./dependencies/EntryDependency");
 
 class DllEntryPlugin {
+	/**
+	 * @param {string} context context
+	 * @param {string[]} entries entry names
+	 * @param {TODO} options options
+	 */
 	constructor(context, entries, options) {
 		this.context = context;
 		this.entries = entries;

package/lib/DllModule.js

@@ -89,7 +89,7 @@ class DllModule extends Module {
 		const sources = new Map();
 		sources.set(
 			"javascript",
-			new RawSource("module.exports = __webpack_require__;")
+			new RawSource(`module.exports = ${RuntimeGlobals.require};`)
 		);
 		return {
 			sources,

package/lib/DllModuleFactory.js

@@ -19,7 +19,7 @@ class DllModuleFactory extends ModuleFactory {
 	}
 	/**
 	 * @param {ModuleFactoryCreateData} data data object
-	 * @param {function(Error=, ModuleFactoryResult=): void} callback callback
+	 * @param {function((Error | null)=, ModuleFactoryResult=): void} callback callback
 	 * @returns {void}
 	 */
 	create(data, callback) {

package/lib/EvalSourceMapDevToolPlugin.js

@@ -48,7 +48,9 @@ class EvalSourceMapDevToolPlugin {
 			options = inputOptions;
 		}
 		this.sourceMapComment =
-			options.append || "//# sourceURL=[module]\n//# sourceMappingURL=[url]";
+			options.append && typeof options.append !== "function"
+				? options.append
+				: "//# sourceURL=[module]\n//# sourceMappingURL=[url]";
 		this.moduleFilenameTemplate =
 			options.moduleFilenameTemplate ||
 			"webpack://[namespace]/[resource-path]?[hash]";

package/lib/HookWebpackError.js

@@ -85,7 +85,7 @@ const tryRunOrWebpackError = (fn, hook) => {
 		if (err instanceof WebpackError) {
 			throw err;
 		}
-		throw new HookWebpackError(err, hook);
+		throw new HookWebpackError(/** @type {Error} */ (err), hook);
 	}
 	return r;
 };

package/lib/IgnoreErrorModuleFactory.js

@@ -26,7 +26,7 @@ class IgnoreErrorModuleFactory extends ModuleFactory {
 
 	/**
 	 * @param {ModuleFactoryCreateData} data data object
-	 * @param {function(Error=, ModuleFactoryResult=): void} callback callback
+	 * @param {function((Error | null)=, ModuleFactoryResult=): void} callback callback
 	 * @returns {void}
 	 */
 	create(data, callback) {

package/lib/MainTemplate.js

@@ -306,8 +306,8 @@ class MainTemplate {
 
 Object.defineProperty(MainTemplate.prototype, "requireFn", {
 	get: util.deprecate(
-		() => "__webpack_require__",
-		'MainTemplate.requireFn is deprecated (use "__webpack_require__")',
+		() => RuntimeGlobals.require,
+		`MainTemplate.requireFn is deprecated (use "${RuntimeGlobals.require}")`,
 		"DEP_WEBPACK_MAIN_TEMPLATE_REQUIRE_FN"
 	)
 });

package/lib/Module.js

@@ -131,13 +131,13 @@ const deprecatedNeedRebuild = util.deprecate(
 class Module extends DependenciesBlock {
 	/**
 	 * @param {ModuleTypes | ""} type the module type, when deserializing the type is not known and is an empty string
-	 * @param {string=} context an optional context
-	 * @param {string=} layer an optional layer in which the module is
+	 * @param {(string | null)=} context an optional context
+	 * @param {(string | null)=} layer an optional layer in which the module is
 	 */
 	constructor(type, context = null, layer = null) {
 		super();
 
-		/** @type {ModuleTypes | ""} */
+		/** @type {ModuleTypes} */
 		this.type = type;
 		/** @type {string | null} */
 		this.context = context;
@@ -151,7 +151,7 @@ class Module extends DependenciesBlock {
 		this.debugId = debugId++;
 
 		// Info from Factory
-		/** @type {ResolveOptions} */
+		/** @type {ResolveOptions | undefined} */
 		this.resolveOptions = EMPTY_RESOLVE_OPTIONS;
 		/** @type {object | undefined} */
 		this.factoryMeta = undefined;
@@ -167,9 +167,9 @@ class Module extends DependenciesBlock {
 		this._warnings = undefined;
 		/** @type {WebpackError[] | undefined} */
 		this._errors = undefined;
-		/** @type {BuildMeta} */
+		/** @type {BuildMeta | undefined} */
 		this.buildMeta = undefined;
-		/** @type {Record<string, any>} */
+		/** @type {Record<string, any> | undefined} */
 		this.buildInfo = undefined;
 		/** @type {Dependency[] | undefined} */
 		this.presentationalDependencies = undefined;
@@ -331,6 +331,10 @@ class Module extends DependenciesBlock {
 		);
 	}
 
+	/**
+	 * @param {Chunk} chunk the chunk
+	 * @returns {boolean} true, when the module was added
+	 */
 	addChunk(chunk) {
 		const chunkGraph = ChunkGraph.getChunkGraphForModule(
 			this,
@@ -342,6 +346,10 @@ class Module extends DependenciesBlock {
 		return true;
 	}
 
+	/**
+	 * @param {Chunk} chunk the chunk
+	 * @returns {void}
+	 */
 	removeChunk(chunk) {
 		return ChunkGraph.getChunkGraphForModule(
 			this,
@@ -350,6 +358,10 @@ class Module extends DependenciesBlock {
 		).disconnectChunkAndModule(chunk, this);
 	}
 
+	/**
+	 * @param {Chunk} chunk the chunk
+	 * @returns {boolean} true, when the module is in the chunk
+	 */
 	isInChunk(chunk) {
 		return ChunkGraph.getChunkGraphForModule(
 			this,
@@ -435,7 +447,7 @@ class Module extends DependenciesBlock {
 			case "namespace":
 				return "namespace";
 			case "default":
-				switch (this.buildMeta.defaultObject) {
+				switch (/** @type {BuildMeta} */ (this.buildMeta).defaultObject) {
 					case "redirect":
 						return "default-with-named";
 					case "redirect-warn":
@@ -447,7 +459,7 @@ class Module extends DependenciesBlock {
 				if (strict) return "default-with-named";
 				// Try to figure out value of __esModule by following reexports
 				const handleDefault = () => {
-					switch (this.buildMeta.defaultObject) {
+					switch (/** @type {BuildMeta} */ (this.buildMeta).defaultObject) {
 						case "redirect":
 						case "redirect-warn":
 							return "default-with-named";
@@ -664,7 +676,7 @@ class Module extends DependenciesBlock {
 		] of moduleGraph.getIncomingConnectionsByOriginModule(this)) {
 			if (!connections.some(c => c.isTargetActive(chunk.runtime))) continue;
 			for (const originChunk of chunkGraph.getModuleChunksIterable(
-				fromModule
+				/** @type {Module} */ (fromModule)
 			)) {
 				// return true if module this is not reachable from originChunk when ignoring chunk
 				if (!this.isAccessibleInChunk(chunkGraph, originChunk, chunk))

package/lib/ModuleFactory.js

@@ -38,7 +38,7 @@ class ModuleFactory {
 	/**
 	 * @abstract
 	 * @param {ModuleFactoryCreateData} data data object
-	 * @param {function(Error=, ModuleFactoryResult=): void} callback callback
+	 * @param {function((Error | null)=, ModuleFactoryResult=): void} callback callback
 	 * @returns {void}
 	 */
 	create(data, callback) {

package/lib/ModuleTypeConstants.js

@@ -135,7 +135,8 @@ const WEBPACK_MODULE_TYPE_LAZY_COMPILATION_PROXY = "lazy-compilation-proxy";
 /** @typedef {"css" | "css/global" | "css/module"} CSSModuleTypes */
 /** @typedef {"asset" | "asset/inline" | "asset/resource" | "asset/source" | "asset/raw-data-url"} AssetModuleTypes */
 /** @typedef {"runtime" | "fallback-module" | "remote-module" | "provide-module" | "consume-shared-module" | "lazy-compilation-proxy"} WebpackModuleTypes */
-/** @typedef {JavaScriptModuleTypes | JSONModuleType | WebAssemblyModuleTypes | CSSModuleTypes | AssetModuleTypes | WebpackModuleTypes} ModuleTypes */
+/** @typedef {string} UnknownModuleTypes */
+/** @typedef {JavaScriptModuleTypes | JSONModuleType | WebAssemblyModuleTypes | CSSModuleTypes | AssetModuleTypes | WebpackModuleTypes | UnknownModuleTypes} ModuleTypes */
 
 exports.ASSET_MODULE_TYPE = ASSET_MODULE_TYPE;
 exports.ASSET_MODULE_TYPE_RAW_DATA_URL = ASSET_MODULE_TYPE_RAW_DATA_URL;

package/lib/NormalModuleFactory.js

@@ -753,7 +753,7 @@ class NormalModuleFactory extends ModuleFactory {
 
 	/**
 	 * @param {ModuleFactoryCreateData} data data object
-	 * @param {function(Error=, ModuleFactoryResult=): void} callback callback
+	 * @param {function((Error | null)=, ModuleFactoryResult=): void} callback callback
 	 * @returns {void}
 	 */
 	create(data, callback) {

package/lib/NormalModuleReplacementPlugin.js

@@ -43,7 +43,9 @@ class NormalModuleReplacementPlugin {
 				});
 				nmf.hooks.afterResolve.tap("NormalModuleReplacementPlugin", result => {
 					const createData = result.createData;
-					if (resourceRegExp.test(createData.resource)) {
+					if (
+						resourceRegExp.test(/** @type {string} */ (createData.resource))
+					) {
 						if (typeof newResource === "function") {
 							newResource(result);
 						} else {
@@ -56,7 +58,7 @@ class NormalModuleReplacementPlugin {
 							} else {
 								createData.resource = join(
 									fs,
-									dirname(fs, createData.resource),
+									dirname(fs, /** @type {string} */ (createData.resource)),
 									newResource
 								);
 							}

package/lib/NullFactory.js

@@ -13,7 +13,7 @@ const ModuleFactory = require("./ModuleFactory");
 class NullFactory extends ModuleFactory {
 	/**
 	 * @param {ModuleFactoryCreateData} data data object
-	 * @param {function(Error=, ModuleFactoryResult=): void} callback callback
+	 * @param {function((Error | null)=, ModuleFactoryResult=): void} callback callback
 	 * @returns {void}
 	 */
 	create(data, callback) {

package/lib/PrefetchPlugin.js

@@ -10,6 +10,10 @@ const PrefetchDependency = require("./dependencies/PrefetchDependency");
 /** @typedef {import("./Compiler")} Compiler */
 
 class PrefetchPlugin {
+	/**
+	 * @param {string} context context or request if context is not set
+	 * @param {string} [request] request
+	 */
 	constructor(context, request) {
 		if (request) {
 			this.context = context;

package/lib/RecordIdsPlugin.js

@@ -69,7 +69,7 @@ class RecordIdsPlugin {
 			compilation.hooks.recordModules.tap(
 				"RecordIdsPlugin",
 				/**
-				 * @param {Module[]} modules the modules array
+				 * @param {Iterable<Module>} modules the modules array
 				 * @param {Records} records the records object
 				 * @returns {void}
 				 */
@@ -92,7 +92,7 @@ class RecordIdsPlugin {
 			compilation.hooks.reviveModules.tap(
 				"RecordIdsPlugin",
 				/**
-				 * @param {Module[]} modules the modules array
+				 * @param {Iterable<Module>} modules the modules array
 				 * @param {Records} records the records object
 				 * @returns {void}
 				 */
@@ -166,7 +166,7 @@ class RecordIdsPlugin {
 			compilation.hooks.recordChunks.tap(
 				"RecordIdsPlugin",
 				/**
-				 * @param {Chunk[]} chunks the chunks array
+				 * @param {Iterable<Chunk>} chunks the chunks array
 				 * @param {Records} records the records object
 				 * @returns {void}
 				 */
@@ -192,7 +192,7 @@ class RecordIdsPlugin {
 			compilation.hooks.reviveChunks.tap(
 				"RecordIdsPlugin",
 				/**
-				 * @param {Chunk[]} chunks the chunks array
+				 * @param {Iterable<Chunk>} chunks the chunks array
 				 * @param {Records} records the records object
 				 * @returns {void}
 				 */

package/lib/RuntimeModule.js

@@ -38,15 +38,15 @@ class RuntimeModule extends Module {
 		this.stage = stage;
 		this.buildMeta = {};
 		this.buildInfo = {};
-		/** @type {Compilation} */
+		/** @type {Compilation | undefined} */
 		this.compilation = undefined;
-		/** @type {Chunk} */
+		/** @type {Chunk | undefined} */
 		this.chunk = undefined;
-		/** @type {ChunkGraph} */
+		/** @type {ChunkGraph | undefined} */
 		this.chunkGraph = undefined;
 		this.fullHash = false;
 		this.dependentHash = false;
-		/** @type {string} */
+		/** @type {string | undefined} */
 		this._cachedGeneratedCode = undefined;
 	}
 
@@ -117,7 +117,7 @@ class RuntimeModule extends Module {
 				hash.update(this.getGeneratedCode());
 			}
 		} catch (err) {
-			hash.update(err.message);
+			hash.update(/** @type {Error} */ (err).message);
 		}
 		super.updateHash(hash, context);
 	}

package/lib/RuntimePlugin.js

@@ -99,6 +99,10 @@ class RuntimePlugin {
 	apply(compiler) {
 		compiler.hooks.compilation.tap("RuntimePlugin", compilation => {
 			const globalChunkLoading = compilation.outputOptions.chunkLoading;
+			/**
+			 * @param {Chunk} chunk chunk
+			 * @returns {boolean} true, when chunk loading is disabled for the chunk
+			 */
 			const isChunkLoadingDisabledForChunk = chunk => {
 				const options = chunk.getEntryOptions();
 				const chunkLoading =