Merge pull request #267 from barak007/master

Minimize footprint for browser bundling

Author: sokra

lib/CachedInputFileSystem.js

@@ -5,6 +5,8 @@
 
 "use strict";
 
+const nextTick = require("process").nextTick;
+
 /** @typedef {import("./Resolver").FileSystem} FileSystem */
 /** @typedef {import("./Resolver").SyncFileSystem} SyncFileSystem */
 
@@ -175,8 +177,8 @@ class CacheBackend {
 		// Check in cache
 		let cacheEntry = this._data.get(path);
 		if (cacheEntry !== undefined) {
-			if (cacheEntry.err) return process.nextTick(callback, cacheEntry.err);
-			return process.nextTick(callback, null, cacheEntry.result);
+			if (cacheEntry.err) return nextTick(callback, cacheEntry.err);
+			return nextTick(callback, null, cacheEntry.result);
 		}
 
 		// Check if there is already the same operation running

lib/Resolver.js

@@ -153,7 +153,7 @@ class Resolver {
 				["request", "resolveContext"],
 				"resolve"
 			),
-			/** @type {AsyncSeriesHook<[ResolveRequest, ResolveContext], void>} */
+			/** @type {AsyncSeriesHook<[ResolveRequest, ResolveContext]>} */
 			result: new AsyncSeriesHook(["result", "resolveContext"], "result")
 		};
 	}

lib/ResolverFactory.js

@@ -5,6 +5,7 @@
 
 "use strict";
 
+const versions = require("process").versions;
 const Resolver = require("./Resolver");
 const { getType, PathType } = require("./util/path");
 
@@ -112,7 +113,7 @@ const UseFilePlugin = require("./UseFilePlugin");
 function processPnpApiOption(option) {
 	if (
 		option === undefined &&
-		/** @type {NodeJS.ProcessVersions & {pnp: string}} */ (process.versions).pnp
+		/** @type {NodeJS.ProcessVersions & {pnp: string}} */ (versions).pnp
 	) {
 		// @ts-ignore
 		return require("pnpapi"); // eslint-disable-line node/no-missing-require

lib/util/process-browser.js

@@ -0,0 +1,16 @@
+/*
+	MIT License http://www.opensource.org/licenses/mit-license.php
+	Author Tobias Koppers @sokra
+*/
+
+"use strict";
+
+exports.process = {
+	versions: {},
+	nextTick(fn) {
+		const args = Array.prototype.slice.call(arguments, 1);
+		Promise.resolve().then(function () {
+			fn.apply(null, args);
+		});
+	}
+};

package.json

@@ -8,9 +8,13 @@
     "types.d.ts",
     "LICENSE"
   ],
+  "browser": {
+    "pnpapi": false,
+    "process": "./lib/util/process-browser.js"
+  },
   "dependencies": {
     "graceful-fs": "^4.2.4",
-    "tapable": "^2.0.0"
+    "tapable": "^2.2.0"
   },
   "license": "MIT",
   "devDependencies": {
@@ -28,7 +32,7 @@
     "nyc": "^15.1.0",
     "prettier": "^2.1.2",
     "should": "^13.2.3",
-    "tooling": "webpack/tooling#v1.8.0",
+    "tooling": "webpack/tooling#v1.11.0",
     "typescript": "^4.0.2"
   },
   "engines": {

types.d.ts

@@ -216,12 +216,10 @@ declare abstract class Resolver {
 					null | (BaseResolveRequest & Partial<ParsedIdentifier>)
 				>,
 				BaseResolveRequest & Partial<ParsedIdentifier>
-			],
-			void
+			]
 		>;
 		noResolve: SyncHook<
-			[BaseResolveRequest & Partial<ParsedIdentifier>, Error],
-			void
+			[BaseResolveRequest & Partial<ParsedIdentifier>, Error]
 		>;
 		resolve: AsyncSeriesBailHook<
 			[BaseResolveRequest & Partial<ParsedIdentifier>, ResolveContext],