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

package/lib/container/ContainerEntryModule.js

@@ -32,11 +32,12 @@ const ContainerExposedDependency = require("./ContainerExposedDependency");
 /** @typedef {import("../Module").SourceTypes} SourceTypes */
 /** @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, ExposesList, string]>} ObjectDeserializerContext */
+/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext<[string, ExposesList, string]>} ObjectSerializerContext */
 /** @typedef {import("../util/fs").InputFileSystem} InputFileSystem */
 
 /**
+ * Defines the expose options type used by this module.
  * @typedef {object} ExposeOptions
  * @property {string[]} import requests to exposed modules (last one is exported)
  * @property {string} name custom chunk name for the exposed module
@@ -46,6 +47,7 @@ const ContainerExposedDependency = require("./ContainerExposedDependency");
 
 class ContainerEntryModule extends Module {
 	/**
+	 * Creates an instance of ContainerEntryModule.
 	 * @param {string} name container entry name
 	 * @param {ExposesList} exposes list of exposed modules
 	 * @param {string} shareScope name of the share scope
@@ -61,6 +63,7 @@ class ContainerEntryModule extends Module {
 	}
 
 	/**
+	 * Returns the source types this module can generate.
 	 * @returns {SourceTypes} types available (do not mutate)
 	 */
 	getSourceTypes() {
@@ -68,6 +71,7 @@ class ContainerEntryModule extends Module {
 	}
 
 	/**
+	 * Returns the unique identifier used to reference this module.
 	 * @returns {string} a unique identifier of the module
 	 */
 	identifier() {
@@ -77,6 +81,7 @@ class ContainerEntryModule extends Module {
 	}
 
 	/**
+	 * Returns a human-readable identifier for this module.
 	 * @param {RequestShortener} requestShortener the request shortener
 	 * @returns {string} a user readable identifier of the module
 	 */
@@ -85,6 +90,7 @@ class ContainerEntryModule extends Module {
 	}
 
 	/**
+	 * Gets the library identifier.
 	 * @param {LibIdentOptions} options options
 	 * @returns {LibIdent | null} an identifier for library inclusion
 	 */
@@ -95,6 +101,7 @@ class ContainerEntryModule 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}
@@ -104,6 +111,7 @@ class ContainerEntryModule 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
@@ -147,6 +155,7 @@ class ContainerEntryModule extends Module {
 	}
 
 	/**
+	 * Generates code and runtime requirements for this module.
 	 * @param {CodeGenerationContext} context context for code generation
 	 * @returns {CodeGenerationResult} result
 	 */
@@ -211,11 +220,12 @@ class ContainerEntryModule extends Module {
 			);
 		}
 
+		const cst = runtimeTemplate.renderConst();
 		const source = Template.asString([
-			"var moduleMap = {",
+			`${cst} moduleMap = {`,
 			Template.indent(getters.join(",\n")),
 			"};",
-			`var get = ${runtimeTemplate.basicFunction("module, getScope", [
+			`${cst} get = ${runtimeTemplate.basicFunction("module, getScope", [
 				`${RuntimeGlobals.currentRemoteGetScope} = getScope;`,
 				// reusing the getScope variable to avoid creating a new var (and module is also used later)
 				"getScope = (",
@@ -233,10 +243,10 @@ class ContainerEntryModule extends Module {
 				`${RuntimeGlobals.currentRemoteGetScope} = undefined;`,
 				"return getScope;"
 			])};`,
-			`var init = ${runtimeTemplate.basicFunction("shareScope, initScope", [
+			`${cst} init = ${runtimeTemplate.basicFunction("shareScope, initScope", [
 				`if (!${RuntimeGlobals.shareScopeMap}) return;`,
-				`var name = ${JSON.stringify(this._shareScope)}`,
-				`var oldScope = ${RuntimeGlobals.shareScopeMap}[name];`,
+				`${cst} name = ${JSON.stringify(this._shareScope)}`,
+				`${cst} oldScope = ${RuntimeGlobals.shareScopeMap}[name];`,
 				'if(oldScope && oldScope !== shareScope) throw new Error("Container initialization failed as it has already been initialized with a different share scope");',
 				`${RuntimeGlobals.shareScopeMap}[name] = shareScope;`,
 				`return ${RuntimeGlobals.initializeSharing}(name, initScope);`
@@ -265,6 +275,7 @@ class ContainerEntryModule 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)
 	 */
@@ -273,24 +284,26 @@ class ContainerEntryModule extends Module {
 	}
 
 	/**
+	 * Serializes this instance into the provided serializer context.
 	 * @param {ObjectSerializerContext} context context
 	 */
 	serialize(context) {
-		const { write } = context;
-		write(this._name);
-		write(this._exposes);
-		write(this._shareScope);
+		context.write(this._name).write(this._exposes).write(this._shareScope);
 		super.serialize(context);
 	}
 
 	/**
+	 * Restores this instance from the provided deserializer context.
 	 * @param {ObjectDeserializerContext} context context
 	 * @returns {ContainerEntryModule} deserialized container entry module
 	 */
 	static deserialize(context) {
-		const { read } = context;
-		const obj = new ContainerEntryModule(read(), read(), read());
-		obj.deserialize(context);
+		const name = context.read();
+		const c1 = context.rest;
+		const exposes = c1.read();
+		const c2 = c1.rest;
+		const obj = new ContainerEntryModule(name, exposes, c2.read());
+		obj.deserialize(c2.rest);
 		return obj;
 	}
 }

package/lib/container/ContainerEntryModuleFactory.js

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

package/lib/container/ContainerExposedDependency.js

@@ -8,11 +8,12 @@
 const ModuleDependency = require("../dependencies/ModuleDependency");
 const makeSerializable = require("../util/makeSerializable");
 
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext} ObjectDeserializerContext */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext} ObjectSerializerContext */
+/** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext<[string]>} ObjectDeserializerContext */
+/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext<[string]>} ObjectSerializerContext */
 
 class ContainerExposedDependency extends ModuleDependency {
 	/**
+	 * Creates an instance of ContainerExposedDependency.
 	 * @param {string} exposedName public name
 	 * @param {string} request request to module
 	 */
@@ -31,6 +32,7 @@ class ContainerExposedDependency extends ModuleDependency {
 	}
 
 	/**
+	 * Returns an identifier to merge equal requests.
 	 * @returns {string | null} an identifier to merge equal requests
 	 */
 	getResourceIdentifier() {
@@ -38,6 +40,7 @@ class ContainerExposedDependency extends ModuleDependency {
 	}
 
 	/**
+	 * Serializes this instance into the provided serializer context.
 	 * @param {ObjectSerializerContext} context context
 	 */
 	serialize(context) {
@@ -46,6 +49,7 @@ class ContainerExposedDependency extends ModuleDependency {
 	}
 
 	/**
+	 * Restores this instance from the provided deserializer context.
 	 * @param {ObjectDeserializerContext} context context
 	 */
 	deserialize(context) {

package/lib/container/ContainerPlugin.js

@@ -23,6 +23,7 @@ const PLUGIN_NAME = "ContainerPlugin";
 
 class ContainerPlugin {
 	/**
+	 * Creates an instance of ContainerPlugin.
 	 * @param {ContainerPluginOptions} options options
 	 */
 	constructor(options) {
@@ -31,7 +32,7 @@ class ContainerPlugin {
 	}
 
 	/**
-	 * Apply the plugin
+	 * Applies the plugin by registering its hooks on the compiler.
 	 * @param {Compiler} compiler the compiler instance
 	 * @returns {void}
 	 */

package/lib/container/ContainerReferencePlugin.js

@@ -24,15 +24,16 @@ const PLUGIN_NAME = "ContainerReferencePlugin";
 
 class ContainerReferencePlugin {
 	/**
+	 * Creates an instance of ContainerReferencePlugin.
 	 * @param {ContainerReferencePluginOptions} options options
 	 */
 	constructor(options) {
-		/** @typedef {ContainerReferencePluginOptions} */
+		/** @type {ContainerReferencePluginOptions} */
 		this.options = options;
 	}
 
 	/**
-	 * Apply the plugin
+	 * Applies the plugin by registering its hooks on the compiler.
 	 * @param {Compiler} compiler the compiler instance
 	 * @returns {void}
 	 */

package/lib/container/FallbackDependency.js

@@ -9,11 +9,12 @@ const Dependency = require("../Dependency");
 const makeSerializable = require("../util/makeSerializable");
 
 /** @typedef {import("./RemoteModule").ExternalRequests} ExternalRequests */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext} ObjectDeserializerContext */
-/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext} ObjectSerializerContext */
+/** @typedef {import("../serialization/ObjectMiddleware").ObjectDeserializerContext<[ExternalRequests]>} ObjectDeserializerContext */
+/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext<[ExternalRequests]>} ObjectSerializerContext */
 
 class FallbackDependency extends Dependency {
 	/**
+	 * Creates an instance of FallbackDependency.
 	 * @param {ExternalRequests} requests requests
 	 */
 	constructor(requests) {
@@ -23,6 +24,7 @@ class FallbackDependency extends Dependency {
 	}
 
 	/**
+	 * Returns an identifier to merge equal requests.
 	 * @returns {string | null} an identifier to merge equal requests
 	 */
 	getResourceIdentifier() {
@@ -38,22 +40,22 @@ class FallbackDependency extends Dependency {
 	}
 
 	/**
+	 * Serializes this instance into the provided serializer context.
 	 * @param {ObjectSerializerContext} context context
 	 */
 	serialize(context) {
-		const { write } = context;
-		write(this.requests);
+		context.write(this.requests);
 		super.serialize(context);
 	}
 
 	/**
+	 * Restores this instance from the provided deserializer context.
 	 * @param {ObjectDeserializerContext} context context
 	 * @returns {FallbackDependency} deserialize fallback dependency
 	 */
 	static deserialize(context) {
-		const { read } = context;
-		const obj = new FallbackDependency(read());
-		obj.deserialize(context);
+		const obj = new FallbackDependency(context.read());
+		obj.deserialize(context.rest);
 		return obj;
 	}
 }

package/lib/container/FallbackItemDependency.js

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

package/lib/container/FallbackModule.js

@@ -32,8 +32,8 @@ const FallbackItemDependency = require("./FallbackItemDependency");
 /** @typedef {import("../Module").SourceTypes} SourceTypes */
 /** @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<[ExternalRequests]>} ObjectDeserializerContext */
+/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext<[ExternalRequests]>} ObjectSerializerContext */
 /** @typedef {import("../util/fs").InputFileSystem} InputFileSystem */
 /** @typedef {import("./RemoteModule").ExternalRequests} ExternalRequests */
 
@@ -41,6 +41,7 @@ const RUNTIME_REQUIREMENTS = new Set([RuntimeGlobals.module]);
 
 class FallbackModule extends Module {
 	/**
+	 * Creates an instance of FallbackModule.
 	 * @param {ExternalRequests} requests list of requests to choose one
 	 */
 	constructor(requests) {
@@ -52,6 +53,7 @@ class FallbackModule extends Module {
 	}
 
 	/**
+	 * Returns the unique identifier used to reference this module.
 	 * @returns {string} a unique identifier of the module
 	 */
 	identifier() {
@@ -59,6 +61,7 @@ class FallbackModule extends Module {
 	}
 
 	/**
+	 * Returns a human-readable identifier for this module.
 	 * @param {RequestShortener} requestShortener the request shortener
 	 * @returns {string} a user readable identifier of the module
 	 */
@@ -67,6 +70,7 @@ class FallbackModule extends Module {
 	}
 
 	/**
+	 * Gets the library identifier.
 	 * @param {LibIdentOptions} options options
 	 * @returns {LibIdent | null} an identifier for library inclusion
 	 */
@@ -77,6 +81,7 @@ class FallbackModule extends Module {
 	}
 
 	/**
+	 * Returns true if the module can be placed in the chunk.
 	 * @param {Chunk} chunk the chunk which condition should be checked
 	 * @param {Compilation} compilation the compilation
 	 * @returns {boolean} true if the module can be placed in the chunk
@@ -86,6 +91,7 @@ class FallbackModule 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}
@@ -95,6 +101,7 @@ class FallbackModule 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
@@ -117,6 +124,7 @@ class FallbackModule 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)
 	 */
@@ -125,6 +133,7 @@ class FallbackModule extends Module {
 	}
 
 	/**
+	 * Returns the source types this module can generate.
 	 * @returns {SourceTypes} types available (do not mutate)
 	 */
 	getSourceTypes() {
@@ -132,6 +141,7 @@ class FallbackModule extends Module {
 	}
 
 	/**
+	 * Generates code and runtime requirements for this module.
 	 * @param {CodeGenerationContext} context context for code generation
 	 * @returns {CodeGenerationResult} result
 	 */
@@ -139,10 +149,12 @@ class FallbackModule extends Module {
 		const ids = this.dependencies.map((dep) =>
 			chunkGraph.getModuleId(/** @type {Module} */ (moduleGraph.getModule(dep)))
 		);
+		const cst = runtimeTemplate.renderConst();
+		const lt = runtimeTemplate.renderLet();
 		const code = Template.asString([
-			`var ids = ${JSON.stringify(ids)};`,
-			"var error, result, i = 0;",
-			`var loop = ${runtimeTemplate.basicFunction("next", [
+			`${cst} ids = ${JSON.stringify(ids)};`,
+			`${lt} error, result, i = 0;`,
+			`${cst} loop = ${runtimeTemplate.basicFunction("next", [
 				"while(i < ids.length) {",
 				Template.indent([
 					`try { next = ${RuntimeGlobals.require}(ids[i++]); } catch(e) { return handleError(e); }`,
@@ -151,11 +163,11 @@ class FallbackModule extends Module {
 				"}",
 				"if(error) throw error;"
 			])}`,
-			`var handleResult = ${runtimeTemplate.basicFunction("result", [
+			`${cst} handleResult = ${runtimeTemplate.basicFunction("result", [
 				"if(result) return result;",
 				"return loop();"
 			])};`,
-			`var handleError = ${runtimeTemplate.basicFunction("e", [
+			`${cst} handleError = ${runtimeTemplate.basicFunction("e", [
 				"error = e;",
 				"return loop();"
 			])};`,
@@ -168,15 +180,16 @@ class FallbackModule extends Module {
 	}
 
 	/**
+	 * Serializes this instance into the provided serializer context.
 	 * @param {ObjectSerializerContext} context context
 	 */
 	serialize(context) {
-		const { write } = context;
-		write(this.requests);
+		context.write(this.requests);
 		super.serialize(context);
 	}
 
 	/**
+	 * Restores this instance from the provided deserializer context.
 	 * @param {ObjectDeserializerContext} context context
 	 * @returns {FallbackModule} deserialized fallback module
 	 */

package/lib/container/FallbackModuleFactory.js

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

package/lib/container/ModuleFederationPlugin.js

@@ -7,8 +7,8 @@
 
 const { SyncHook } = require("tapable");
 const isValidExternalsType = require("../../schemas/plugins/container/ExternalsType.check");
-const Compilation = require("../Compilation");
 const SharePlugin = require("../sharing/SharePlugin");
+const createHooksRegistry = require("../util/createHooksRegistry");
 const ContainerPlugin = require("./ContainerPlugin");
 const ContainerReferencePlugin = require("./ContainerReferencePlugin");
 const HoistContainerReferences = require("./HoistContainerReferencesPlugin");
@@ -19,17 +19,17 @@ const HoistContainerReferences = require("./HoistContainerReferencesPlugin");
 /** @typedef {import("../Dependency")} Dependency */
 
 /**
+ * Defines the compilation hooks type used by this module.
  * @typedef {object} CompilationHooks
  * @property {SyncHook<Dependency>} addContainerEntryDependency
  * @property {SyncHook<Dependency>} addFederationRuntimeDependency
  */
 
-/** @type {WeakMap<Compilation, CompilationHooks>} */
-const compilationHooksMap = new WeakMap();
 const PLUGIN_NAME = "ModuleFederationPlugin";
 
 class ModuleFederationPlugin {
 	/**
+	 * Creates an instance of ModuleFederationPlugin.
 	 * @param {ModuleFederationPluginOptions} options options
 	 */
 	constructor(options) {
@@ -38,29 +38,7 @@ class ModuleFederationPlugin {
 	}
 
 	/**
-	 * Get the compilation hooks associated with this plugin.
-	 * @param {Compilation} compilation The compilation instance.
-	 * @returns {CompilationHooks} The hooks for the compilation.
-	 */
-	static getCompilationHooks(compilation) {
-		if (!(compilation instanceof Compilation)) {
-			throw new TypeError(
-				"The 'compilation' argument must be an instance of Compilation"
-			);
-		}
-		let hooks = compilationHooksMap.get(compilation);
-		if (!hooks) {
-			hooks = {
-				addContainerEntryDependency: new SyncHook(["dependency"]),
-				addFederationRuntimeDependency: new SyncHook(["dependency"])
-			};
-			compilationHooksMap.set(compilation, hooks);
-		}
-		return hooks;
-	}
-
-	/**
-	 * Apply the plugin
+	 * Applies the plugin by registering its hooks on the compiler.
 	 * @param {Compiler} compiler the compiler instance
 	 * @returns {void}
 	 */
@@ -132,4 +110,12 @@ class ModuleFederationPlugin {
 	}
 }
 
+ModuleFederationPlugin.getCompilationHooks = createHooksRegistry(
+	() =>
+		/** @type {CompilationHooks} */ ({
+			addContainerEntryDependency: new SyncHook(["dependency"]),
+			addFederationRuntimeDependency: new SyncHook(["dependency"])
+		})
+);
+
 module.exports = ModuleFederationPlugin;

package/lib/container/RemoteModule.js

@@ -34,8 +34,8 @@ const RemoteToExternalDependency = require("./RemoteToExternalDependency");
 /** @typedef {import("../Module").ExportsType} ExportsType */
 /** @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, ExternalRequests, string, string]>} ObjectDeserializerContext */
+/** @typedef {import("../serialization/ObjectMiddleware").ObjectSerializerContext<[string, ExternalRequests, string, string]>} ObjectSerializerContext */
 /** @typedef {import("../util/fs").InputFileSystem} InputFileSystem */
 /** @typedef {import("../Module").BasicSourceTypes} BasicSourceTypes */
 
@@ -45,6 +45,7 @@ const RUNTIME_REQUIREMENTS = new Set([RuntimeGlobals.module]);
 
 class RemoteModule extends Module {
 	/**
+	 * Creates an instance of RemoteModule.
 	 * @param {string} request request string
 	 * @param {ExternalRequests} externalRequests list of external requests to containers
 	 * @param {string} internalRequest name of exposed module in container
@@ -67,6 +68,7 @@ class RemoteModule extends Module {
 	}
 
 	/**
+	 * Returns the unique identifier used to reference this module.
 	 * @returns {string} a unique identifier of the module
 	 */
 	identifier() {
@@ -74,6 +76,7 @@ class RemoteModule extends Module {
 	}
 
 	/**
+	 * Returns a human-readable identifier for this module.
 	 * @param {RequestShortener} requestShortener the request shortener
 	 * @returns {string} a user readable identifier of the module
 	 */
@@ -82,6 +85,7 @@ class RemoteModule extends Module {
 	}
 
 	/**
+	 * Gets the library identifier.
 	 * @param {LibIdentOptions} options options
 	 * @returns {LibIdent | null} an identifier for library inclusion
 	 */
@@ -92,6 +96,7 @@ class RemoteModule 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}
@@ -101,6 +106,7 @@ class RemoteModule 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
@@ -127,6 +133,7 @@ class RemoteModule 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)
 	 */
@@ -135,6 +142,7 @@ class RemoteModule extends Module {
 	}
 
 	/**
+	 * Returns the source types this module can generate.
 	 * @returns {SourceTypes} types available (do not mutate)
 	 */
 	getSourceTypes() {
@@ -155,6 +163,7 @@ class RemoteModule extends Module {
 	}
 
 	/**
+	 * Returns export type.
 	 * @param {ModuleGraph} moduleGraph the module graph
 	 * @param {boolean | undefined} strict the importing module is strict
 	 * @returns {ExportsType} export type
@@ -168,6 +177,7 @@ class RemoteModule extends Module {
 	}
 
 	/**
+	 * Returns the path used when matching this module against rule conditions.
 	 * @returns {NameForCondition | null} absolute path which should be used for condition matching (usually the resource path)
 	 */
 	nameForCondition() {
@@ -175,6 +185,7 @@ class RemoteModule extends Module {
 	}
 
 	/**
+	 * Generates code and runtime requirements for this module.
 	 * @param {CodeGenerationContext} context context for code generation
 	 * @returns {CodeGenerationResult} result
 	 */
@@ -197,25 +208,37 @@ class RemoteModule extends Module {
 	}
 
 	/**
+	 * Serializes this instance into the provided serializer context.
 	 * @param {ObjectSerializerContext} context context
 	 */
 	serialize(context) {
-		const { write } = context;
-		write(this.request);
-		write(this.externalRequests);
-		write(this.internalRequest);
-		write(this.shareScope);
+		context
+			.write(this.request)
+			.write(this.externalRequests)
+			.write(this.internalRequest)
+			.write(this.shareScope);
 		super.serialize(context);
 	}
 
 	/**
+	 * Restores this instance from the provided deserializer context.
 	 * @param {ObjectDeserializerContext} context context
 	 * @returns {RemoteModule} deserialized module
 	 */
 	static deserialize(context) {
-		const { read } = context;
-		const obj = new RemoteModule(read(), read(), read(), read());
-		obj.deserialize(context);
+		const request = context.read();
+		const c1 = context.rest;
+		const externalRequests = c1.read();
+		const c2 = c1.rest;
+		const internalRequest = c2.read();
+		const c3 = c2.rest;
+		const obj = new RemoteModule(
+			request,
+			externalRequests,
+			internalRequest,
+			c3.read()
+		);
+		obj.deserialize(c3.rest);
 		return obj;
 	}
 }