@depup/webpack 5.106.0-depup.0 → 5.109.0-depup.0

package/lib/sharing/ConsumeSharedRuntimeModule.js

@@ -8,6 +8,7 @@
 const RuntimeGlobals = require("../RuntimeGlobals");
 const RuntimeModule = require("../RuntimeModule");
 const Template = require("../Template");
+const { compareModulesById } = require("../util/comparators");
 const {
 	parseVersionRuntimeCode,
 	rangeToStringRuntimeCode,
@@ -31,10 +32,12 @@ class ConsumeSharedRuntimeModule extends RuntimeModule {
 	 */
 	constructor(runtimeRequirements) {
 		super("consumes", RuntimeModule.STAGE_ATTACH);
+		/** @type {ReadOnlyRuntimeRequirements} */
 		this._runtimeRequirements = runtimeRequirements;
 	}
 
 	/**
+	 * Generates runtime code for this runtime module.
 	 * @returns {string | null} runtime code
 	 */
 	generate() {
@@ -70,12 +73,14 @@ class ConsumeSharedRuntimeModule extends RuntimeModule {
 				);
 			}
 		};
+		const byId = compareModulesById(chunkGraph);
 		for (const chunk of /** @type {Chunk} */ (
 			this.chunk
 		).getAllReferencedChunks()) {
-			const modules = chunkGraph.getChunkModulesIterableBySourceType(
+			const modules = chunkGraph.getOrderedChunkModulesIterableBySourceType(
 				chunk,
-				"consume-shared"
+				"consume-shared",
+				byId
 			);
 			if (!modules) continue;
 			addModules(
@@ -87,27 +92,29 @@ class ConsumeSharedRuntimeModule extends RuntimeModule {
 		for (const chunk of /** @type {Chunk} */ (
 			this.chunk
 		).getAllInitialChunks()) {
-			const modules = chunkGraph.getChunkModulesIterableBySourceType(
+			const modules = chunkGraph.getOrderedChunkModulesIterableBySourceType(
 				chunk,
-				"consume-shared"
+				"consume-shared",
+				byId
 			);
 			if (!modules) continue;
 			addModules(modules, chunk, initialConsumes);
 		}
 		if (moduleIdToSourceMapping.size === 0) return null;
+		const cst = runtimeTemplate.renderConst();
 		return Template.asString([
 			parseVersionRuntimeCode(runtimeTemplate),
 			versionLtRuntimeCode(runtimeTemplate),
 			rangeToStringRuntimeCode(runtimeTemplate),
 			satisfyRuntimeCode(runtimeTemplate),
-			`var exists = ${runtimeTemplate.basicFunction("scope, key", [
+			`${cst} exists = ${runtimeTemplate.basicFunction("scope, key", [
 				`return scope && ${RuntimeGlobals.hasOwnProperty}(scope, key);`
 			])}`,
-			`var get = ${runtimeTemplate.basicFunction("entry", [
+			`${cst} get = ${runtimeTemplate.basicFunction("entry", [
 				"entry.loaded = 1;",
 				"return entry.get()"
 			])};`,
-			`var eagerOnly = ${runtimeTemplate.basicFunction("versions", [
+			`${cst} eagerOnly = ${runtimeTemplate.basicFunction("versions", [
 				`return Object.keys(versions).reduce(${runtimeTemplate.basicFunction(
 					"filtered, version",
 					Template.indent([
@@ -118,10 +125,10 @@ class ConsumeSharedRuntimeModule extends RuntimeModule {
 					])
 				)}, {});`
 			])};`,
-			`var findLatestVersion = ${runtimeTemplate.basicFunction(
+			`${cst} findLatestVersion = ${runtimeTemplate.basicFunction(
 				"scope, key, eager",
 				[
-					"var versions = eager ? eagerOnly(scope[key]) : scope[key];",
+					`${cst} versions = eager ? eagerOnly(scope[key]) : scope[key];`,
 					`var key = Object.keys(versions).reduce(${runtimeTemplate.basicFunction(
 						"a, b",
 						["return !a || versionLt(a, b) ? b : a;"]
@@ -129,10 +136,10 @@ class ConsumeSharedRuntimeModule extends RuntimeModule {
 					"return key && versions[key];"
 				]
 			)};`,
-			`var findSatisfyingVersion = ${runtimeTemplate.basicFunction(
+			`${cst} findSatisfyingVersion = ${runtimeTemplate.basicFunction(
 				"scope, key, requiredVersion, eager",
 				[
-					"var versions = eager ? eagerOnly(scope[key]) : scope[key];",
+					`${cst} versions = eager ? eagerOnly(scope[key]) : scope[key];`,
 					`var key = Object.keys(versions).reduce(${runtimeTemplate.basicFunction(
 						"a, b",
 						[
@@ -143,26 +150,26 @@ class ConsumeSharedRuntimeModule extends RuntimeModule {
 					"return key && versions[key]"
 				]
 			)};`,
-			`var findSingletonVersionKey = ${runtimeTemplate.basicFunction(
+			`${cst} findSingletonVersionKey = ${runtimeTemplate.basicFunction(
 				"scope, key, eager",
 				[
-					"var versions = eager ? eagerOnly(scope[key]) : scope[key];",
+					`${cst} versions = eager ? eagerOnly(scope[key]) : scope[key];`,
 					`return Object.keys(versions).reduce(${runtimeTemplate.basicFunction(
 						"a, b",
 						["return !a || (!versions[a].loaded && versionLt(a, b)) ? b : a;"]
 					)}, 0);`
 				]
 			)};`,
-			`var getInvalidSingletonVersionMessage = ${runtimeTemplate.basicFunction(
+			`${cst} getInvalidSingletonVersionMessage = ${runtimeTemplate.basicFunction(
 				"scope, key, version, requiredVersion",
 				[
 					'return "Unsatisfied version " + version + " from " + (version && scope[key][version].from) + " of shared singleton module " + key + " (required " + rangeToString(requiredVersion) + ")"'
 				]
 			)};`,
-			`var getInvalidVersionMessage = ${runtimeTemplate.basicFunction(
+			`${cst} getInvalidVersionMessage = ${runtimeTemplate.basicFunction(
 				"scope, scopeName, key, requiredVersion, eager",
 				[
-					"var versions = scope[key];",
+					`${cst} versions = scope[key];`,
 					'return "No satisfying version (" + rangeToString(requiredVersion) + ")" + (eager ? " for eager consumption" : "") + " of shared module " + key + " found in shared scope " + scopeName + ".\\n" +',
 					`\t"Available versions: " + Object.keys(versions).map(${runtimeTemplate.basicFunction(
 						"key",
@@ -170,26 +177,29 @@ class ConsumeSharedRuntimeModule extends RuntimeModule {
 					)}).join(", ");`
 				]
 			)};`,
-			`var fail = ${runtimeTemplate.basicFunction("msg", [
+			`${cst} fail = ${runtimeTemplate.basicFunction("msg", [
 				"throw new Error(msg);"
 			])}`,
-			`var failAsNotExist = ${runtimeTemplate.basicFunction("scopeName, key", [
-				'return fail("Shared module " + key + " doesn\'t exist in shared scope " + scopeName);'
-			])}`,
-			`var warn = /*#__PURE__*/ ${
+			`${cst} failAsNotExist = ${runtimeTemplate.basicFunction(
+				"scopeName, key",
+				[
+					'return fail("Shared module " + key + " doesn\'t exist in shared scope " + scopeName);'
+				]
+			)}`,
+			`${cst} warn = /*#__PURE__*/ ${
 				compilation.outputOptions.ignoreBrowserWarnings
 					? runtimeTemplate.basicFunction("", "")
 					: runtimeTemplate.basicFunction("msg", [
 							'if (typeof console !== "undefined" && console.warn) console.warn(msg);'
 						])
 			};`,
-			`var init = ${runtimeTemplate.returningFunction(
+			`${cst} init = ${runtimeTemplate.returningFunction(
 				Template.asString([
 					"function(scopeName, key, eager, c, d) {",
 					Template.indent([
-						`var promise = ${RuntimeGlobals.initializeSharing}(scopeName);`,
+						`${cst} promise = ${RuntimeGlobals.initializeSharing}(scopeName);`,
 						// if we require eager shared, we expect it to be already loaded before it requested, no need to wait the whole scope loaded.
-						"if (promise && promise.then && !eager) { ",
+						`if (${runtimeTemplate.optionalChaining("promise", "then")} && !eager) { `,
 						Template.indent([
 							`return promise.then(fn.bind(fn, scopeName, ${RuntimeGlobals.shareScopeMap}[scopeName], key, false, c, d));`
 						]),
@@ -201,50 +211,50 @@ class ConsumeSharedRuntimeModule extends RuntimeModule {
 				"fn"
 			)};`,
 			"",
-			`var useFallback = ${runtimeTemplate.basicFunction(
+			`${cst} useFallback = ${runtimeTemplate.basicFunction(
 				"scopeName, key, fallback",
 				["return fallback ? fallback() : failAsNotExist(scopeName, key);"]
 			)}`,
-			`var load = /*#__PURE__*/ init(${runtimeTemplate.basicFunction(
+			`${cst} load = /*#__PURE__*/ init(${runtimeTemplate.basicFunction(
 				"scopeName, scope, key, eager, fallback",
 				[
 					"if (!exists(scope, key)) return useFallback(scopeName, key, fallback);",
 					"return get(findLatestVersion(scope, key, eager));"
 				]
 			)});`,
-			`var loadVersion = /*#__PURE__*/ init(${runtimeTemplate.basicFunction(
+			`${cst} loadVersion = /*#__PURE__*/ init(${runtimeTemplate.basicFunction(
 				"scopeName, scope, key, eager, requiredVersion, fallback",
 				[
 					"if (!exists(scope, key)) return useFallback(scopeName, key, fallback);",
-					"var satisfyingVersion = findSatisfyingVersion(scope, key, requiredVersion, eager);",
+					`${cst} satisfyingVersion = findSatisfyingVersion(scope, key, requiredVersion, eager);`,
 					"if (satisfyingVersion) return get(satisfyingVersion);",
 					"warn(getInvalidVersionMessage(scope, scopeName, key, requiredVersion, eager))",
 					"return get(findLatestVersion(scope, key, eager));"
 				]
 			)});`,
-			`var loadStrictVersion = /*#__PURE__*/ init(${runtimeTemplate.basicFunction(
+			`${cst} loadStrictVersion = /*#__PURE__*/ init(${runtimeTemplate.basicFunction(
 				"scopeName, scope, key, eager, requiredVersion, fallback",
 				[
 					"if (!exists(scope, key)) return useFallback(scopeName, key, fallback);",
-					"var satisfyingVersion = findSatisfyingVersion(scope, key, requiredVersion, eager);",
+					`${cst} satisfyingVersion = findSatisfyingVersion(scope, key, requiredVersion, eager);`,
 					"if (satisfyingVersion) return get(satisfyingVersion);",
 					"if (fallback) return fallback();",
 					"fail(getInvalidVersionMessage(scope, scopeName, key, requiredVersion, eager));"
 				]
 			)});`,
-			`var loadSingleton = /*#__PURE__*/ init(${runtimeTemplate.basicFunction(
+			`${cst} loadSingleton = /*#__PURE__*/ init(${runtimeTemplate.basicFunction(
 				"scopeName, scope, key, eager, fallback",
 				[
 					"if (!exists(scope, key)) return useFallback(scopeName, key, fallback);",
-					"var version = findSingletonVersionKey(scope, key, eager);",
+					`${cst} version = findSingletonVersionKey(scope, key, eager);`,
 					"return get(scope[key][version]);"
 				]
 			)});`,
-			`var loadSingletonVersion = /*#__PURE__*/ init(${runtimeTemplate.basicFunction(
+			`${cst} loadSingletonVersion = /*#__PURE__*/ init(${runtimeTemplate.basicFunction(
 				"scopeName, scope, key, eager, requiredVersion, fallback",
 				[
 					"if (!exists(scope, key)) return useFallback(scopeName, key, fallback);",
-					"var version = findSingletonVersionKey(scope, key, eager);",
+					`${cst} version = findSingletonVersionKey(scope, key, eager);`,
 					"if (!satisfy(requiredVersion, version)) {",
 					Template.indent([
 						"warn(getInvalidSingletonVersionMessage(scope, key, version, requiredVersion));"
@@ -253,11 +263,11 @@ class ConsumeSharedRuntimeModule extends RuntimeModule {
 					"return get(scope[key][version]);"
 				]
 			)});`,
-			`var loadStrictSingletonVersion = /*#__PURE__*/ init(${runtimeTemplate.basicFunction(
+			`${cst} loadStrictSingletonVersion = /*#__PURE__*/ init(${runtimeTemplate.basicFunction(
 				"scopeName, scope, key, eager, requiredVersion, fallback",
 				[
 					"if (!exists(scope, key)) return useFallback(scopeName, key, fallback);",
-					"var version = findSingletonVersionKey(scope, key, eager);",
+					`${cst} version = findSingletonVersionKey(scope, key, eager);`,
 					"if (!satisfy(requiredVersion, version)) {",
 					Template.indent([
 						"fail(getInvalidSingletonVersionMessage(scope, key, version, requiredVersion));"
@@ -266,8 +276,8 @@ class ConsumeSharedRuntimeModule extends RuntimeModule {
 					"return get(scope[key][version]);"
 				]
 			)});`,
-			"var installedModules = {};",
-			"var moduleToHandlerMapping = {",
+			`${cst} installedModules = {};`,
+			`${cst} moduleToHandlerMapping = {`,
 			Template.indent(
 				Array.from(
 					moduleIdToSourceMapping,
@@ -278,7 +288,7 @@ class ConsumeSharedRuntimeModule extends RuntimeModule {
 
 			initialConsumes.length > 0
 				? Template.asString([
-						`var initialConsumes = ${JSON.stringify(initialConsumes)};`,
+						`${cst} initialConsumes = ${JSON.stringify(initialConsumes)};`,
 						`initialConsumes.forEach(${runtimeTemplate.basicFunction("id", [
 							`${
 								RuntimeGlobals.moduleFactories
@@ -286,7 +296,7 @@ class ConsumeSharedRuntimeModule extends RuntimeModule {
 								"// Handle case when module is used sync",
 								"installedModules[id] = 0;",
 								`delete ${RuntimeGlobals.moduleCache}[id];`,
-								"var factory = moduleToHandlerMapping[id]();",
+								`${cst} factory = moduleToHandlerMapping[id]();`,
 								'if(typeof factory !== "function") throw new Error("Shared module is not available for eager consumption: " + id);',
 								"module.exports = factory();"
 							])}`
@@ -295,12 +305,12 @@ class ConsumeSharedRuntimeModule extends RuntimeModule {
 				: "// no consumes in initial chunks",
 			this._runtimeRequirements.has(RuntimeGlobals.ensureChunkHandlers)
 				? Template.asString([
-						`var chunkMapping = ${JSON.stringify(
+						`${cst} chunkMapping = ${JSON.stringify(
 							chunkToModuleMapping,
 							null,
 							"\t"
 						)};`,
-						"var startedInstallModules = {};",
+						`${cst} startedInstallModules = {};`,
 						`${
 							RuntimeGlobals.ensureChunkHandlers
 						}.consumes = ${runtimeTemplate.basicFunction("chunkId, promises", [
@@ -311,7 +321,7 @@ class ConsumeSharedRuntimeModule extends RuntimeModule {
 									[
 										`if(${RuntimeGlobals.hasOwnProperty}(installedModules, id)) return promises.push(installedModules[id]);`,
 										"if(!startedInstallModules[id]) {",
-										`var onFactory = ${runtimeTemplate.basicFunction(
+										`${cst} onFactory = ${runtimeTemplate.basicFunction(
 											"factory",
 											[
 												"installedModules[id] = 0;",
@@ -324,7 +334,7 @@ class ConsumeSharedRuntimeModule extends RuntimeModule {
 											]
 										)};`,
 										"startedInstallModules[id] = true;",
-										`var onError = ${runtimeTemplate.basicFunction("error", [
+										`${cst} onError = ${runtimeTemplate.basicFunction("error", [
 											"delete installedModules[id];",
 											`${
 												RuntimeGlobals.moduleFactories
@@ -335,7 +345,7 @@ class ConsumeSharedRuntimeModule extends RuntimeModule {
 										])};`,
 										"try {",
 										Template.indent([
-											"var promise = moduleToHandlerMapping[id]();",
+											`${cst} promise = moduleToHandlerMapping[id]();`,
 											"if(promise.then) {",
 											Template.indent(
 												"promises.push(installedModules[id] = promise.then(onFactory)['catch'](onError));"

package/lib/sharing/ProvideForSharedDependency.js

@@ -10,6 +10,7 @@ const makeSerializable = require("../util/makeSerializable");
 
 class ProvideForSharedDependency extends ModuleDependency {
 	/**
+	 * Creates an instance of ProvideForSharedDependency.
 	 * @param {string} request request string
 	 */
 	constructor(request) {

package/lib/sharing/ProvideSharedDependency.js

@@ -8,11 +8,12 @@
 const Dependency = require("../Dependency");
 const makeSerializable = require("../util/makeSerializable");
 
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext} ObjectDeserializerContext */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext} ObjectSerializerContext */
+/** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext<[string, string, string, string | false, boolean]>} ObjectDeserializerContext */
+/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext<[string, string, string, string | false, boolean]>} ObjectSerializerContext */
 
 class ProvideSharedDependency extends Dependency {
 	/**
+	 * Creates an instance of ProvideSharedDependency.
 	 * @param {string} shareScope share scope
 	 * @param {string} name module name
 	 * @param {string | false} version version
@@ -21,10 +22,15 @@ class ProvideSharedDependency extends Dependency {
 	 */
 	constructor(shareScope, name, version, request, eager) {
 		super();
+		/** @type {string} */
 		this.shareScope = shareScope;
+		/** @type {string} */
 		this.name = name;
+		/** @type {string | false} */
 		this.version = version;
+		/** @type {string} */
 		this.request = request;
+		/** @type {boolean} */
 		this.eager = eager;
 	}
 
@@ -33,6 +39,7 @@ class ProvideSharedDependency extends Dependency {
 	}
 
 	/**
+	 * Returns an identifier to merge equal requests.
 	 * @returns {string | null} an identifier to merge equal requests
 	 */
 	getResourceIdentifier() {
@@ -42,32 +49,44 @@ class ProvideSharedDependency extends Dependency {
 	}
 
 	/**
+	 * Serializes this instance into the provided serializer context.
 	 * @param {ObjectSerializerContext} context context
 	 */
 	serialize(context) {
-		context.write(this.shareScope);
-		context.write(this.name);
-		context.write(this.request);
-		context.write(this.version);
-		context.write(this.eager);
+		context
+			.write(this.shareScope)
+			.write(this.name)
+			.write(this.request)
+			.write(this.version)
+			.write(this.eager);
 		super.serialize(context);
 	}
 
 	/**
+	 * Restores this instance from the provided deserializer context.
 	 * @param {ObjectDeserializerContext} context context
 	 * @returns {ProvideSharedDependency} deserialize fallback dependency
 	 */
 	static deserialize(context) {
-		const { read } = context;
+		// wire order is shareScope, name, request, version, eager;
+		// reorder into the constructor's (…, version, request, …) signature
+		const shareScope = context.read();
+		const c1 = context.rest;
+		const name = c1.read();
+		const c2 = c1.rest;
+		const request = c2.read();
+		const c3 = c2.rest;
+		const version = c3.read();
+		const c4 = c3.rest;
+		const eager = c4.read();
 		const obj = new ProvideSharedDependency(
-			read(),
-			read(),
-			read(),
-			read(),
-			read()
+			shareScope,
+			name,
+			version,
+			request,
+			eager
 		);
-		this.shareScope = context.read();
-		obj.deserialize(context);
+		obj.deserialize(c4.rest);
 		return obj;
 	}
 }

package/lib/sharing/ProvideSharedModule.js

@@ -27,12 +27,13 @@ const ProvideForSharedDependency = require("./ProvideForSharedDependency");
 /** @typedef {import("../Module").CodeGenerationResultData} CodeGenerationResultData */
 /** @typedef {import("../RequestShortener")} RequestShortener */
 /** @typedef {import("../ResolverFactory").ResolverWithOptions} ResolverWithOptions */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext} ObjectDeserializerContext */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext} ObjectSerializerContext */
+/** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext<[string, string, string | false, string, boolean]>} ObjectDeserializerContext */
+/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext<[string, string, string | false, string, boolean]>} ObjectSerializerContext */
 /** @typedef {import("../util/fs").InputFileSystem} InputFileSystem */
 
 class ProvideSharedModule extends Module {
 	/**
+	 * Creates an instance of ProvideSharedModule.
 	 * @param {string} shareScope shared scope name
 	 * @param {string} name shared key
 	 * @param {string | false} version version
@@ -41,21 +42,28 @@ class ProvideSharedModule extends Module {
 	 */
 	constructor(shareScope, name, version, request, eager) {
 		super(WEBPACK_MODULE_TYPE_PROVIDE);
+		/** @type {string} */
 		this._shareScope = shareScope;
+		/** @type {string} */
 		this._name = name;
+		/** @type {string | false} */
 		this._version = version;
+		/** @type {string} */
 		this._request = request;
+		/** @type {boolean} */
 		this._eager = eager;
 	}
 
 	/**
+	 * Returns the unique identifier used to reference this module.
 	 * @returns {string} a unique identifier of the module
 	 */
 	identifier() {
-		return `provide module (${this._shareScope}) ${this._name}@${this._version} = ${this._request}`;
+		return `provide module (${this._shareScope}) ${this._name}@${this._version}|${this._request}`;
 	}
 
 	/**
+	 * Returns a human-readable identifier for this module.
 	 * @param {RequestShortener} requestShortener the request shortener
 	 * @returns {string} a user readable identifier of the module
 	 */
@@ -66,6 +74,7 @@ class ProvideSharedModule extends Module {
 	}
 
 	/**
+	 * Gets the library identifier.
 	 * @param {LibIdentOptions} options options
 	 * @returns {LibIdent | null} an identifier for library inclusion
 	 */
@@ -76,6 +85,7 @@ class ProvideSharedModule extends Module {
 	}
 
 	/**
+	 * Checks whether the module needs to be rebuilt for the current build state.
 	 * @param {NeedBuildContext} context context info
 	 * @param {NeedBuildCallback} callback callback function, returns true, if the module needs a rebuild
 	 * @returns {void}
@@ -85,6 +95,7 @@ class ProvideSharedModule extends Module {
 	}
 
 	/**
+	 * Builds the module using the provided compilation context.
 	 * @param {WebpackOptions} options webpack options
 	 * @param {Compilation} compilation the compilation
 	 * @param {ResolverWithOptions} resolver the resolver
@@ -112,6 +123,7 @@ class ProvideSharedModule extends Module {
 	}
 
 	/**
+	 * Returns the estimated size for the requested source type.
 	 * @param {string=} type the source type for which the size should be estimated
 	 * @returns {number} the estimated size of the module (must be non-zero)
 	 */
@@ -120,6 +132,7 @@ class ProvideSharedModule extends Module {
 	}
 
 	/**
+	 * Returns the source types this module can generate.
 	 * @returns {SourceTypes} types available (do not mutate)
 	 */
 	getSourceTypes() {
@@ -127,6 +140,7 @@ class ProvideSharedModule extends Module {
 	}
 
 	/**
+	 * Generates code and runtime requirements for this module.
 	 * @param {CodeGenerationContext} context context for code generation
 	 * @returns {CodeGenerationResult} result
 	 */
@@ -164,26 +178,42 @@ class ProvideSharedModule extends Module {
 	}
 
 	/**
+	 * Serializes this instance into the provided serializer context.
 	 * @param {ObjectSerializerContext} context context
 	 */
 	serialize(context) {
-		const { write } = context;
-		write(this._shareScope);
-		write(this._name);
-		write(this._version);
-		write(this._request);
-		write(this._eager);
+		context
+			.write(this._shareScope)
+			.write(this._name)
+			.write(this._version)
+			.write(this._request)
+			.write(this._eager);
 		super.serialize(context);
 	}
 
 	/**
+	 * Restores this instance from the provided deserializer context.
 	 * @param {ObjectDeserializerContext} context context
 	 * @returns {ProvideSharedModule} deserialize fallback dependency
 	 */
 	static deserialize(context) {
-		const { read } = context;
-		const obj = new ProvideSharedModule(read(), read(), read(), read(), read());
-		obj.deserialize(context);
+		const shareScope = context.read();
+		const c1 = context.rest;
+		const name = c1.read();
+		const c2 = c1.rest;
+		const version = c2.read();
+		const c3 = c2.rest;
+		const request = c3.read();
+		const c4 = c3.rest;
+		const eager = c4.read();
+		const obj = new ProvideSharedModule(
+			shareScope,
+			name,
+			version,
+			request,
+			eager
+		);
+		obj.deserialize(c4.rest);
 		return obj;
 	}
 }

package/lib/sharing/ProvideSharedModuleFactory.js

@@ -14,6 +14,7 @@ const ProvideSharedModule = require("./ProvideSharedModule");
 
 class ProvideSharedModuleFactory extends ModuleFactory {
 	/**
+	 * Processes the provided data.
 	 * @param {ModuleFactoryCreateData} data data object
 	 * @param {ModuleFactoryCallback} callback callback
 	 * @returns {void}

package/lib/sharing/ProvideSharedPlugin.js

@@ -5,8 +5,9 @@
 
 "use strict";
 
-const WebpackError = require("../WebpackError");
 const { parseOptions } = require("../container/options");
+const WebpackError = require("../errors/WebpackError");
+const { ABSOLUTE_PATH_REGEXP } = require("../util/identifier");
 const ProvideForSharedDependency = require("./ProvideForSharedDependency");
 const ProvideSharedDependency = require("./ProvideSharedDependency");
 const ProvideSharedModuleFactory = require("./ProvideSharedModuleFactory");
@@ -17,6 +18,7 @@ const ProvideSharedModuleFactory = require("./ProvideSharedModuleFactory");
 /** @typedef {import("../NormalModuleFactory").NormalModuleCreateData} NormalModuleCreateData */
 
 /**
+ * Defines the provide options type used by this module.
  * @typedef {object} ProvideOptions
  * @property {string} shareKey
  * @property {string} shareScope
@@ -30,14 +32,16 @@ const PLUGIN_NAME = "ProvideSharedPlugin";
 
 class ProvideSharedPlugin {
 	/**
+	 * Creates an instance of ProvideSharedPlugin.
 	 * @param {ProvideSharedPluginOptions} options options
 	 */
 	constructor(options) {
+		/** @type {ProvideSharedPluginOptions} */
 		this.options = options;
 	}
 
 	/**
-	 * Apply the plugin
+	 * Applies the plugin by registering its hooks on the compiler.
 	 * @param {Compiler} compiler the compiler instance
 	 * @returns {void}
 	 */
@@ -98,13 +102,13 @@ class ProvideSharedPlugin {
 				/** @type {Map<string, ProvideOptions>} */
 				const prefixMatchProvides = new Map();
 				for (const [request, config] of provides) {
-					if (/^(?:\/|[A-Z]:\\|\\\\|\.\.?(?:\/|$))/i.test(request)) {
+					if (/^\.\.?(?:\/|$)/.test(request)) {
 						// relative request
 						resolvedProvideMap.set(request, {
 							config,
 							version: config.version
 						});
-					} else if (/^(?:\/|[A-Z]:\\|\\\\)/i.test(request)) {
+					} else if (ABSOLUTE_PATH_REGEXP.test(request)) {
 						// absolute path
 						resolvedProvideMap.set(request, {
 							config,
@@ -120,6 +124,7 @@ class ProvideSharedPlugin {
 				}
 				compilationData.set(compilation, resolvedProvideMap);
 				/**
+				 * Provide shared module.
 				 * @param {string} key key
 				 * @param {ProvideOptions} config config
 				 * @param {NormalModuleCreateData["resource"]} resource resource
@@ -166,7 +171,7 @@ class ProvideSharedPlugin {
 				normalModuleFactory.hooks.module.tap(
 					PLUGIN_NAME,
 					(module, { resource, resourceResolveData }, resolveData) => {
-						if (resolvedProvideMap.has(/** @type {string} */ (resource))) {
+						if (resolvedProvideMap.has(resource)) {
 							return module;
 						}
 						const { request } = resolveData;
@@ -176,7 +181,7 @@ class ProvideSharedPlugin {
 								provideSharedModule(
 									request,
 									config,
-									/** @type {string} */ (resource),
+									resource,
 									resourceResolveData
 								);
 								resolveData.cacheable = false;
@@ -186,12 +191,12 @@ class ProvideSharedPlugin {
 							if (request.startsWith(prefix)) {
 								const remainder = request.slice(prefix.length);
 								provideSharedModule(
-									/** @type {string} */ (resource),
+									resource,
 									{
 										...config,
 										shareKey: config.shareKey + remainder
 									},
-									/** @type {string} */ (resource),
+									resource,
 									resourceResolveData
 								);
 								resolveData.cacheable = false;