webpack 5.90.3 → 5.92.0

package/lib/optimize/InnerGraph.js

@@ -9,6 +9,7 @@ const { UsageState } = require("../ExportsInfo");
 
 /** @typedef {import("estree").Node} AnyNode */
 /** @typedef {import("../Dependency")} Dependency */
+/** @typedef {import("../Module")} Module */
 /** @typedef {import("../ModuleGraph")} ModuleGraph */
 /** @typedef {import("../ModuleGraphConnection")} ModuleGraphConnection */
 /** @typedef {import("../ModuleGraphConnection").ConnectionState} ConnectionState */
@@ -20,7 +21,7 @@ const { UsageState } = require("../ExportsInfo");
 /** @typedef {function(boolean | Set<string> | undefined): void} UsageCallback */
 
 /**
- * @typedef {Object} StateObject
+ * @typedef {object} StateObject
  * @property {InnerGraph} innerGraph
  * @property {TopLevelSymbol=} currentTopLevelSymbol
  * @property {Map<TopLevelSymbol, Set<UsageCallback>>} usageCallbackMap
@@ -295,7 +296,9 @@ exports.isDependencyUsedByExports = (
 ) => {
 	if (usedByExports === false) return false;
 	if (usedByExports !== true && usedByExports !== undefined) {
-		const selfModule = moduleGraph.getParentModule(dependency);
+		const selfModule =
+			/** @type {Module} */
+			(moduleGraph.getParentModule(dependency));
 		const exportsInfo = moduleGraph.getExportsInfo(selfModule);
 		let used = false;
 		for (const exportName of usedByExports) {
@@ -320,7 +323,9 @@ exports.getDependencyUsedByExportsCondition = (
 ) => {
 	if (usedByExports === false) return false;
 	if (usedByExports !== true && usedByExports !== undefined) {
-		const selfModule = moduleGraph.getParentModule(dependency);
+		const selfModule =
+			/** @type {Module} */
+			(moduleGraph.getParentModule(dependency));
 		const exportsInfo = moduleGraph.getExportsInfo(selfModule);
 		return (connections, runtime) => {
 			for (const exportName of usedByExports) {

package/lib/optimize/InnerGraphPlugin.js

@@ -19,8 +19,10 @@ const InnerGraph = require("./InnerGraph");
 /** @typedef {import("../../declarations/WebpackOptions").JavascriptParserOptions} JavascriptParserOptions */
 /** @typedef {import("../Compiler")} Compiler */
 /** @typedef {import("../Dependency")} Dependency */
+/** @typedef {import("../Dependency").DependencyLocation} DependencyLocation */
 /** @typedef {import("../dependencies/HarmonyImportSpecifierDependency")} HarmonyImportSpecifierDependency */
 /** @typedef {import("../javascript/JavascriptParser")} JavascriptParser */
+/** @typedef {import("../javascript/JavascriptParser").Range} Range */
 /** @typedef {import("./InnerGraph").InnerGraph} InnerGraph */
 /** @typedef {import("./InnerGraph").TopLevelSymbol} TopLevelSymbol */
 
@@ -123,7 +125,10 @@ class InnerGraphPlugin {
 						if (parser.scope.topLevelScope === true) {
 							if (
 								statement.type === "ClassDeclaration" &&
-								parser.isPure(statement, statement.range[0])
+								parser.isPure(
+									statement,
+									/** @type {Range} */ (statement.range)[0]
+								)
 							) {
 								const name = statement.id ? statement.id.name : "*default*";
 								const fn = InnerGraph.tagTopLevelSymbol(parser, name);
@@ -137,10 +142,12 @@ class InnerGraphPlugin {
 								if (
 									(decl.type === "ClassExpression" ||
 										decl.type === "ClassDeclaration") &&
-									parser.isPure(decl, decl.range[0])
+									parser.isPure(decl, /** @type {Range} */ (decl.range)[0])
 								) {
 									classWithTopLevelSymbol.set(decl, fn);
-								} else if (parser.isPure(decl, statement.range[0])) {
+								} else if (
+									parser.isPure(decl, /** @type {Range} */ (statement.range)[0])
+								) {
 									statementWithTopLevelSymbol.set(statement, fn);
 									if (
 										!decl.type.endsWith("FunctionExpression") &&
@@ -164,11 +171,19 @@ class InnerGraphPlugin {
 							const name = decl.id.name;
 							if (
 								decl.init.type === "ClassExpression" &&
-								parser.isPure(decl.init, decl.id.range[1])
+								parser.isPure(
+									decl.init,
+									/** @type {Range} */ (decl.id.range)[1]
+								)
 							) {
 								const fn = InnerGraph.tagTopLevelSymbol(parser, name);
 								classWithTopLevelSymbol.set(decl.init, fn);
-							} else if (parser.isPure(decl.init, decl.id.range[1])) {
+							} else if (
+								parser.isPure(
+									decl.init,
+									/** @type {Range} */ (decl.id.range)[1]
+								)
+							) {
 								const fn = InnerGraph.tagTopLevelSymbol(parser, name);
 								declWithTopLevelSymbol.set(decl, fn);
 								if (
@@ -214,9 +229,11 @@ class InnerGraphPlugin {
 												return;
 											default: {
 												const dep = new PureExpressionDependency(
-													purePart.range
+													/** @type {Range} */ (purePart.range)
 												);
-												dep.loc = statement.loc;
+												dep.loc =
+													/** @type {DependencyLocation} */
+													(statement.loc);
 												dep.usedByExports = usedByExports;
 												parser.state.module.addDependency(dep);
 												break;
@@ -238,7 +255,9 @@ class InnerGraphPlugin {
 									fn &&
 									parser.isPure(
 										expr,
-										statement.id ? statement.id.range[1] : statement.range[0]
+										statement.id
+											? /** @type {Range} */ (statement.id.range)[1]
+											: /** @type {Range} */ (statement.range)[0]
 									)
 								) {
 									InnerGraph.setTopLevelSymbol(parser.state, fn);
@@ -272,7 +291,9 @@ class InnerGraphPlugin {
 										!element.static ||
 										parser.isPure(
 											expression,
-											element.key ? element.key.range[1] : element.range[0]
+											element.key
+												? /** @type {Range} */ (element.key.range)[1]
+												: /** @type {Range} */ (element.range)[0]
 										)
 									) {
 										InnerGraph.setTopLevelSymbol(parser.state, fn);
@@ -284,9 +305,11 @@ class InnerGraphPlugin {
 														return;
 													default: {
 														const dep = new PureExpressionDependency(
-															expression.range
+															/** @type {Range} */ (expression.range)
 														);
-														dep.loc = expression.loc;
+														dep.loc =
+															/** @type {DependencyLocation} */
+															(expression.loc);
 														dep.usedByExports = usedByExports;
 														parser.state.module.addDependency(dep);
 														break;
@@ -321,9 +344,9 @@ class InnerGraphPlugin {
 												return;
 											default: {
 												const dep = new PureExpressionDependency(
-													decl.init.range
+													/** @type {Range} */ (decl.init.range)
 												);
-												dep.loc = decl.loc;
+												dep.loc = /** @type {DependencyLocation} */ (decl.loc);
 												dep.usedByExports = usedByExports;
 												parser.state.module.addDependency(dep);
 												break;

package/lib/optimize/LimitChunkCountPlugin.js

@@ -24,7 +24,7 @@ const validate = createSchemaValidation(
 );
 
 /**
- * @typedef {Object} ChunkCombination
+ * @typedef {object} ChunkCombination
  * @property {boolean} deleted this is set to true when combination was removed
  * @property {number} sizeDiff
  * @property {number} integratedSize
@@ -159,7 +159,6 @@ class LimitChunkCountPlugin {
 					const modifiedChunks = new Set();
 
 					let changed = false;
-					// eslint-disable-next-line no-constant-condition
 					loop: while (true) {
 						const combination = combinations.popFirst();
 						if (combination === undefined) break;

package/lib/optimize/ModuleConcatenationPlugin.js

@@ -23,11 +23,12 @@ const ConcatenatedModule = require("./ConcatenatedModule");
 /** @typedef {import("../Compilation")} Compilation */
 /** @typedef {import("../Compiler")} Compiler */
 /** @typedef {import("../Module")} Module */
+/** @typedef {import("../Module").BuildInfo} BuildInfo */
 /** @typedef {import("../RequestShortener")} RequestShortener */
 /** @typedef {import("../util/runtime").RuntimeSpec} RuntimeSpec */
 
 /**
- * @typedef {Object} Statistics
+ * @typedef {object} Statistics
  * @property {number} cached
  * @property {number} alreadyInConfig
  * @property {number} invalidModule
@@ -49,6 +50,9 @@ const formatBailoutReason = msg => {
 };
 
 class ModuleConcatenationPlugin {
+	/**
+	 * @param {TODO} options options
+	 */
 	constructor(options) {
 		if (typeof options !== "object") options = {};
 		this.options = options;
@@ -171,7 +175,7 @@ class ModuleConcatenationPlugin {
 						}
 
 						// Must be in strict mode
-						if (!module.buildInfo.strict) {
+						if (!(/** @type {BuildInfo} */ (module.buildInfo).strict)) {
 							setBailoutReason(module, `Module is not in strict mode`);
 							continue;
 						}
@@ -242,7 +246,10 @@ class ModuleConcatenationPlugin {
 					// this improves performance, because modules already selected as inner are skipped
 					logger.time("sort relevant modules");
 					relevantModules.sort((a, b) => {
-						return moduleGraph.getDepth(a) - moduleGraph.getDepth(b);
+						return (
+							/** @type {number} */ (moduleGraph.getDepth(a)) -
+							/** @type {number} */ (moduleGraph.getDepth(b))
+						);
 					});
 					logger.timeEnd("sort relevant modules");
 
@@ -927,6 +934,9 @@ class ConcatConfiguration {
 		return this.modules.size;
 	}
 
+	/**
+	 * @param {number} snapshot snapshot
+	 */
 	rollback(snapshot) {
 		const modules = this.modules;
 		for (const m of modules) {

package/lib/optimize/RealContentHashPlugin.js

@@ -87,7 +87,7 @@ const toCachedSource = source => {
 /** @typedef {Set<string>} Hashes */
 
 /**
- * @typedef {Object} AssetInfoForRealContentHash
+ * @typedef {object} AssetInfoForRealContentHash
  * @property {string} name
  * @property {AssetInfo} info
  * @property {Source} source
@@ -102,7 +102,7 @@ const toCachedSource = source => {
  */
 
 /**
- * @typedef {Object} CompilationHooks
+ * @typedef {object} CompilationHooks
  * @property {SyncBailHook<[Buffer[], string], string>} updateHash
  */
 
@@ -131,7 +131,7 @@ class RealContentHashPlugin {
 	}
 
 	/**
-	 * @param {Object} options options object
+	 * @param {object} options options object
 	 * @param {string | Hash} options.hashFunction the hash function to use
 	 * @param {string} options.hashDigest the hash digest to use
 	 */

package/lib/optimize/RemoveParentModulesPlugin.js

@@ -77,6 +77,7 @@ class RemoveParentModulesPlugin {
 
 				let nextModuleMask = ONE_BIGINT;
 				const maskByModule = new WeakMap();
+				/** @type {Module[]} */
 				const ordinalModules = [];
 
 				/**

package/lib/optimize/RuntimeChunkPlugin.js

@@ -10,6 +10,9 @@
 /** @typedef {import("../Entrypoint")} Entrypoint */
 
 class RuntimeChunkPlugin {
+	/**
+	 * @param {{ name?: (entrypoint: { name: string }) => string }} options options
+	 */
 	constructor(options) {
 		this.options = {
 			/**
@@ -37,7 +40,9 @@ class RuntimeChunkPlugin {
 						(compilation.entries.get(entryName));
 					if (data.options.runtime === undefined && !data.options.dependOn) {
 						// Determine runtime chunk name
-						let name = this.options.name;
+						let name =
+							/** @type {string | ((entrypoint: { name: string }) => string)} */
+							(this.options.name);
 						if (typeof name === "function") {
 							name = name({ name: entryName });
 						}

package/lib/optimize/SideEffectsFlagPlugin.js

@@ -20,23 +20,28 @@ const formatLocation = require("../formatLocation");
 /** @typedef {import("estree").Statement} Statement */
 /** @typedef {import("../Compiler")} Compiler */
 /** @typedef {import("../Dependency")} Dependency */
+/** @typedef {import("../Dependency").DependencyLocation} DependencyLocation */
 /** @typedef {import("../Module")} Module */
+/** @typedef {import("../Module").BuildMeta} BuildMeta */
 /** @typedef {import("../javascript/JavascriptParser")} JavascriptParser */
+/** @typedef {import("../javascript/JavascriptParser").Range} Range */
 
 /**
- * @typedef {Object} ExportInModule
+ * @typedef {object} ExportInModule
  * @property {Module} module the module
  * @property {string} exportName the name of the export
  * @property {boolean} checked if the export is conditional
  */
 
 /**
- * @typedef {Object} ReexportInfo
+ * @typedef {object} ReexportInfo
  * @property {Map<string, ExportInModule[]>} static
  * @property {Map<Module, Set<string>>} dynamic
  */
 
-/** @type {WeakMap<any, Map<string, RegExp>>} */
+/** @typedef {Map<string, RegExp>} CacheItem */
+
+/** @type {WeakMap<any, CacheItem>} */
 const globToRegexpCache = new WeakMap();
 
 /**
@@ -96,7 +101,7 @@ class SideEffectsFlagPlugin {
 							const hasSideEffects = SideEffectsFlagPlugin.moduleHasSideEffects(
 								resolveData.relativePath,
 								sideEffects,
-								cache
+								/** @type {CacheItem} */ (cache)
 							);
 							module.factoryMeta.sideEffectFree = !hasSideEffects;
 						}
@@ -132,7 +137,10 @@ class SideEffectsFlagPlugin {
 								switch (statement.type) {
 									case "ExpressionStatement":
 										if (
-											!parser.isPure(statement.expression, statement.range[0])
+											!parser.isPure(
+												statement.expression,
+												/** @type {Range} */ (statement.range)[0]
+											)
 										) {
 											sideEffectsStatement = statement;
 										}
@@ -140,27 +148,35 @@ class SideEffectsFlagPlugin {
 									case "IfStatement":
 									case "WhileStatement":
 									case "DoWhileStatement":
-										if (!parser.isPure(statement.test, statement.range[0])) {
+										if (
+											!parser.isPure(
+												statement.test,
+												/** @type {Range} */ (statement.range)[0]
+											)
+										) {
 											sideEffectsStatement = statement;
 										}
 										// statement hook will be called for child statements too
 										break;
 									case "ForStatement":
 										if (
-											!parser.isPure(statement.init, statement.range[0]) ||
+											!parser.isPure(
+												statement.init,
+												/** @type {Range} */ (statement.range)[0]
+											) ||
 											!parser.isPure(
 												statement.test,
 												statement.init
-													? statement.init.range[1]
-													: statement.range[0]
+													? /** @type {Range} */ (statement.init.range)[1]
+													: /** @type {Range} */ (statement.range)[0]
 											) ||
 											!parser.isPure(
 												statement.update,
 												statement.test
-													? statement.test.range[1]
+													? /** @type {Range} */ (statement.test.range)[1]
 													: statement.init
-														? statement.init.range[1]
-														: statement.range[0]
+														? /** @type {Range} */ (statement.init.range)[1]
+														: /** @type {Range} */ (statement.range)[0]
 											)
 										) {
 											sideEffectsStatement = statement;
@@ -169,7 +185,10 @@ class SideEffectsFlagPlugin {
 										break;
 									case "SwitchStatement":
 										if (
-											!parser.isPure(statement.discriminant, statement.range[0])
+											!parser.isPure(
+												statement.discriminant,
+												/** @type {Range} */ (statement.range)[0]
+											)
 										) {
 											sideEffectsStatement = statement;
 										}
@@ -178,14 +197,22 @@ class SideEffectsFlagPlugin {
 									case "VariableDeclaration":
 									case "ClassDeclaration":
 									case "FunctionDeclaration":
-										if (!parser.isPure(statement, statement.range[0])) {
+										if (
+											!parser.isPure(
+												statement,
+												/** @type {Range} */ (statement.range)[0]
+											)
+										) {
 											sideEffectsStatement = statement;
 										}
 										break;
 									case "ExportNamedDeclaration":
 									case "ExportDefaultDeclaration":
 										if (
-											!parser.isPure(statement.declaration, statement.range[0])
+											!parser.isPure(
+												statement.declaration,
+												/** @type {Range} */ (statement.range)[0]
+											)
 										) {
 											sideEffectsStatement = statement;
 										}
@@ -208,7 +235,8 @@ class SideEffectsFlagPlugin {
 						);
 						parser.hooks.finish.tap(PLUGIN_NAME, () => {
 							if (sideEffectsStatement === undefined) {
-								parser.state.module.buildMeta.sideEffectFree = true;
+								/** @type {BuildMeta} */
+								(parser.state.module.buildMeta).sideEffectFree = true;
 							} else {
 								const { loc, type } = sideEffectsStatement;
 								moduleGraph
@@ -216,7 +244,7 @@ class SideEffectsFlagPlugin {
 									.push(
 										() =>
 											`Statement (${type}) with side effects in source code at ${formatLocation(
-												loc
+												/** @type {DependencyLocation} */ (loc)
 											)}`
 									);
 							}
@@ -246,6 +274,9 @@ class SideEffectsFlagPlugin {
 
 						const optimizedModules = new Set();
 
+						/**
+						 * @param {Module} module module
+						 */
 						const optimizeIncomingConnections = module => {
 							if (optimizedModules.has(module)) return;
 							optimizedModules.add(module);

package/lib/optimize/SplitChunksPlugin.js

@@ -52,7 +52,7 @@ const MinMaxSizeWarning = require("./MinMaxSizeWarning");
  */
 
 /**
- * @typedef {Object} CacheGroupSource
+ * @typedef {object} CacheGroupSource
  * @property {string=} key
  * @property {number=} priority
  * @property {GetName=} getName
@@ -75,7 +75,7 @@ const MinMaxSizeWarning = require("./MinMaxSizeWarning");
  */
 
 /**
- * @typedef {Object} CacheGroup
+ * @typedef {object} CacheGroup
  * @property {string} key
  * @property {number=} priority
  * @property {GetName=} getName
@@ -101,7 +101,7 @@ const MinMaxSizeWarning = require("./MinMaxSizeWarning");
  */
 
 /**
- * @typedef {Object} FallbackCacheGroup
+ * @typedef {object} FallbackCacheGroup
  * @property {ChunkFilterFunction} chunksFilter
  * @property {SplitChunksSizes} minSize
  * @property {SplitChunksSizes} maxAsyncSize
@@ -110,7 +110,7 @@ const MinMaxSizeWarning = require("./MinMaxSizeWarning");
  */
 
 /**
- * @typedef {Object} CacheGroupsContext
+ * @typedef {object} CacheGroupsContext
  * @property {ModuleGraph} moduleGraph
  * @property {ChunkGraph} chunkGraph
  */
@@ -131,7 +131,7 @@ const MinMaxSizeWarning = require("./MinMaxSizeWarning");
  */
 
 /**
- * @typedef {Object} SplitChunksOptions
+ * @typedef {object} SplitChunksOptions
  * @property {ChunkFilterFunction} chunksFilter
  * @property {string[]} defaultSizeTypes
  * @property {SplitChunksSizes} minSize
@@ -153,14 +153,14 @@ const MinMaxSizeWarning = require("./MinMaxSizeWarning");
  */
 
 /**
- * @typedef {Object} ChunksInfoItem
+ * @typedef {object} ChunksInfoItem
  * @property {SortableSet<Module>} modules
  * @property {CacheGroup} cacheGroup
  * @property {number} cacheGroupIndex
  * @property {string} name
  * @property {Record<string, number>} sizes
  * @property {Set<Chunk>} chunks
- * @property {Set<Chunk>} reuseableChunks
+ * @property {Set<Chunk>} reusableChunks
  * @property {Set<bigint | Chunk>} chunksKeys
  */
 
@@ -1023,7 +1023,7 @@ module.exports = class SplitChunksPlugin {
 						getExportsCombinationsFactory()(key);
 
 					/**
-					 * @typedef {Object} SelectedChunksResult
+					 * @typedef {object} SelectedChunksResult
 					 * @property {Chunk[]} chunks the list of chunks
 					 * @property {bigint | Chunk} key a key of the list
 					 */
@@ -1177,7 +1177,7 @@ module.exports = class SplitChunksPlugin {
 									name,
 									sizes: {},
 									chunks: new Set(),
-									reuseableChunks: new Set(),
+									reusableChunks: new Set(),
 									chunksKeys: new Set()
 								})
 							);
@@ -1322,7 +1322,7 @@ module.exports = class SplitChunksPlugin {
 					}
 
 					/**
-					 * @typedef {Object} MaxSizeQueueItem
+					 * @typedef {object} MaxSizeQueueItem
 					 * @property {SplitChunksSizes} minSize
 					 * @property {SplitChunksSizes} maxAsyncSize
 					 * @property {SplitChunksSizes} maxInitialSize

package/lib/performance/SizeLimitsPlugin.js

@@ -13,18 +13,19 @@ const NoAsyncChunksWarning = require("./NoAsyncChunksWarning");
 /** @typedef {import("webpack-sources").Source} Source */
 /** @typedef {import("../../declarations/WebpackOptions").PerformanceOptions} PerformanceOptions */
 /** @typedef {import("../ChunkGroup")} ChunkGroup */
+/** @typedef {import("../Compilation").Asset} Asset */
 /** @typedef {import("../Compiler")} Compiler */
 /** @typedef {import("../Entrypoint")} Entrypoint */
 /** @typedef {import("../WebpackError")} WebpackError */
 
 /**
- * @typedef {Object} AssetDetails
+ * @typedef {object} AssetDetails
  * @property {string} name
  * @property {number} size
  */
 
 /**
- * @typedef {Object} EntrypointDetails
+ * @typedef {object} EntrypointDetails
  * @property {string} name
  * @property {number} size
  * @property {string[]} files
@@ -32,6 +33,12 @@ const NoAsyncChunksWarning = require("./NoAsyncChunksWarning");
 
 const isOverSizeLimitSet = new WeakSet();
 
+/**
+ * @param {Asset["name"]} name the name
+ * @param {Asset["source"]} source the source
+ * @param {Asset["info"]} info the info
+ * @returns {boolean} result
+ */
 const excludeSourceMap = (name, source, info) => !info.development;
 
 module.exports = class SizeLimitsPlugin {
@@ -104,6 +111,10 @@ module.exports = class SizeLimitsPlugin {
 				}
 			}
 
+			/**
+			 * @param {Asset["name"]} name the name
+			 * @returns {boolean | undefined} result
+			 */
 			const fileFilter = name => {
 				const asset = compilation.getAsset(name);
 				return asset && assetFilter(asset.name, asset.source, asset.info);

package/lib/rules/ObjectMatcherRulePlugin.js

@@ -7,11 +7,18 @@
 
 /** @typedef {import("./RuleSetCompiler")} RuleSetCompiler */
 /** @typedef {import("./RuleSetCompiler").RuleCondition} RuleCondition */
+/** @typedef {import("./RuleSetCompiler").RuleConditionFunction} RuleConditionFunction */
 
 class ObjectMatcherRulePlugin {
-	constructor(ruleProperty, dataProperty) {
+	/**
+	 * @param {string} ruleProperty the rule property
+	 * @param {string=} dataProperty the data property
+	 * @param {RuleConditionFunction=} additionalConditionFunction need to check
+	 */
+	constructor(ruleProperty, dataProperty, additionalConditionFunction) {
 		this.ruleProperty = ruleProperty;
 		this.dataProperty = dataProperty || ruleProperty;
+		this.additionalConditionFunction = additionalConditionFunction;
 	}
 
 	/**
@@ -32,6 +39,13 @@ class ObjectMatcherRulePlugin {
 							`${path}.${ruleProperty}.${property}`,
 							value[property]
 						);
+						if (this.additionalConditionFunction) {
+							result.conditions.push({
+								property: [dataProperty],
+								matchWhenEmpty: condition.matchWhenEmpty,
+								fn: this.additionalConditionFunction
+							});
+						}
 						result.conditions.push({
 							property: [dataProperty, ...nestedDataProperties],
 							matchWhenEmpty: condition.matchWhenEmpty,

package/lib/rules/RuleSetCompiler.js

@@ -7,21 +7,23 @@
 
 const { SyncHook } = require("tapable");
 
+/** @typedef {function(string): boolean} RuleConditionFunction */
+
 /**
- * @typedef {Object} RuleCondition
+ * @typedef {object} RuleCondition
  * @property {string | string[]} property
  * @property {boolean} matchWhenEmpty
- * @property {function(string): boolean} fn
+ * @property {RuleConditionFunction} fn
  */
 
 /**
- * @typedef {Object} Condition
+ * @typedef {object} Condition
  * @property {boolean} matchWhenEmpty
- * @property {function(string): boolean} fn
+ * @property {RuleConditionFunction} fn
  */
 
 /**
- * @typedef {Object} CompiledRule
+ * @typedef {object} CompiledRule
  * @property {RuleCondition[]} conditions
  * @property {(Effect|function(object): Effect[])[]} effects
  * @property {CompiledRule[]=} rules
@@ -29,13 +31,13 @@ const { SyncHook } = require("tapable");
  */
 
 /**
- * @typedef {Object} Effect
+ * @typedef {object} Effect
  * @property {string} type
  * @property {any} value
  */
 
 /**
- * @typedef {Object} RuleSet
+ * @typedef {object} RuleSet
  * @property {Map<string, any>} references map of references in the rule set (may grow over time)
  * @property {function(object): Effect[]} exec execute the rule set
  */

package/lib/runtime/EnsureChunkRuntimeModule.js

@@ -9,10 +9,11 @@ const RuntimeModule = require("../RuntimeModule");
 const Template = require("../Template");
 
 /** @typedef {import("../Compilation")} Compilation */
+/** @typedef {import("../Module").ReadOnlyRuntimeRequirements} ReadOnlyRuntimeRequirements */
 
 class EnsureChunkRuntimeModule extends RuntimeModule {
 	/**
-	 * @param {ReadonlySet<string>} runtimeRequirements runtime requirements
+	 * @param {ReadOnlyRuntimeRequirements} runtimeRequirements runtime requirements
 	 */
 	constructor(runtimeRequirements) {
 		super("ensure chunk");