@depup/webpack 5.106.1-depup.0 → 5.109.2-depup.0

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;
 	}
 }

package/lib/container/RemoteRuntimeModule.js

@@ -22,6 +22,7 @@ class RemoteRuntimeModule extends RuntimeModule {
 	}
 
 	/**
+	 * Generates runtime code for this runtime module.
 	 * @returns {string | null} runtime code
 	 */
 	generate() {
@@ -59,13 +60,15 @@ class RemoteRuntimeModule extends RuntimeModule {
 				idToExternalAndNameMapping[id] = [shareScope, name, externalModuleId];
 			}
 		}
+		const cst = runtimeTemplate.renderConst();
+		const lt = runtimeTemplate.renderLet();
 		return Template.asString([
-			`var chunkMapping = ${JSON.stringify(
+			`${cst} chunkMapping = ${JSON.stringify(
 				chunkToRemotesMapping,
 				null,
 				"\t"
 			)};`,
-			`var idToExternalAndNameMapping = ${JSON.stringify(
+			`${cst} idToExternalAndNameMapping = ${JSON.stringify(
 				idToExternalAndNameMapping,
 				null,
 				"\t"
@@ -76,13 +79,13 @@ class RemoteRuntimeModule extends RuntimeModule {
 				`if(${RuntimeGlobals.hasOwnProperty}(chunkMapping, chunkId)) {`,
 				Template.indent([
 					`chunkMapping[chunkId].forEach(${runtimeTemplate.basicFunction("id", [
-						`var getScope = ${RuntimeGlobals.currentRemoteGetScope};`,
+						`${lt} getScope = ${RuntimeGlobals.currentRemoteGetScope};`,
 						"if(!getScope) getScope = [];",
-						"var data = idToExternalAndNameMapping[id];",
+						`${cst} data = idToExternalAndNameMapping[id];`,
 						"if(getScope.indexOf(data) >= 0) return;",
 						"getScope.push(data);",
 						"if(data.p) return promises.push(data.p);",
-						`var onError = ${runtimeTemplate.basicFunction("error", [
+						`${cst} onError = ${runtimeTemplate.basicFunction("error", [
 							'if(!error) error = new Error("Container missing");',
 							'if(typeof error.message === "string")',
 							Template.indent(
@@ -93,15 +96,15 @@ class RemoteRuntimeModule extends RuntimeModule {
 							}[id] = ${runtimeTemplate.basicFunction("", ["throw error;"])}`,
 							"data.p = 0;"
 						])};`,
-						`var handleFunction = ${runtimeTemplate.basicFunction(
+						`${cst} handleFunction = ${runtimeTemplate.basicFunction(
 							"fn, arg1, arg2, d, next, first",
 							[
 								"try {",
 								Template.indent([
-									"var promise = fn(arg1, arg2);",
-									"if(promise && promise.then) {",
+									`${cst} promise = fn(arg1, arg2);`,
+									`if(${runtimeTemplate.optionalChaining("promise", "then")}) {`,
 									Template.indent([
-										`var p = promise.then(${runtimeTemplate.returningFunction(
+										`${cst} p = promise.then(${runtimeTemplate.returningFunction(
 											"next(result, d)",
 											"result"
 										)}, onError);`,
@@ -116,15 +119,15 @@ class RemoteRuntimeModule extends RuntimeModule {
 								"}"
 							]
 						)}`,
-						`var onExternal = ${runtimeTemplate.returningFunction(
+						`${cst} onExternal = ${runtimeTemplate.returningFunction(
 							`external ? handleFunction(${RuntimeGlobals.initializeSharing}, data[0], 0, external, onInitialized, first) : onError()`,
 							"external, _, first"
 						)};`,
-						`var onInitialized = ${runtimeTemplate.returningFunction(
+						`${cst} onInitialized = ${runtimeTemplate.returningFunction(
 							"handleFunction(external.get, data[1], getScope, 0, onFactory, first)",
 							"_, external, first"
 						)};`,
-						`var onFactory = ${runtimeTemplate.basicFunction("factory", [
+						`${cst} onFactory = ${runtimeTemplate.basicFunction("factory", [
 							"data.p = 1;",
 							`${
 								RuntimeGlobals.moduleFactories

package/lib/container/RemoteToExternalDependency.js

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

package/lib/container/options.js

@@ -6,16 +6,19 @@
 "use strict";
 
 /**
+ * Defines the item type used by this module.
  * @template T
  * @typedef {Record<string, string | string[] | T>} Item
  */
 
 /**
+ * Defines the container options format type used by this module.
  * @template T
  * @typedef {(string | Item<T>)[] | Item<T>} ContainerOptionsFormat
  */
 
 /**
+ * Processes the provided t.
  * @template T
  * @template N
  * @param {ContainerOptionsFormat<T>} options options passed by the user
@@ -26,6 +29,7 @@
  */
 const process = (options, normalizeSimple, normalizeOptions, fn) => {
 	/**
+	 * Processes the provided item.
 	 * @param {(string | Item<T>)[]} items items
 	 */
 	const array = (items) => {
@@ -40,6 +44,7 @@ const process = (options, normalizeSimple, normalizeOptions, fn) => {
 		}
 	};
 	/**
+	 * Processes the provided obj.
 	 * @param {Item<T>} obj an object
 	 */
 	const object = (obj) => {
@@ -63,6 +68,7 @@ const process = (options, normalizeSimple, normalizeOptions, fn) => {
 };
 
 /**
+ * Returns parsed options.
  * @template T
  * @template R
  * @param {ContainerOptionsFormat<T>} options options passed by the user
@@ -80,6 +86,7 @@ const parseOptions = (options, normalizeSimple, normalizeOptions) => {
 };
 
 /**
+ * Returns options to spread or pass.
  * @template T
  * @param {string} scope scope name
  * @param {ContainerOptionsFormat<T>} options options passed by the user