ci: show report

.github/workflows/test.yml

@@ -55,6 +55,8 @@ jobs:
         run: yarn --frozen-lockfile
         if: matrix.node-version != '10.x'
       - name: Run tests with coverage
-        run: npm run test:coverage -- --ci
-      - if: ${{ matrix.os != 'windows-latest' }}
-        uses: codecov/codecov-action@v5
+        run: yarn test:coverage -- --ci
+      - uses: codecov/codecov-action@v5
+        with:
+          flags: integration
+          token: ${{ secrets.CODECOV_TOKEN }}

.gitignore

@@ -3,7 +3,10 @@
 /.nyc_output
 *.log
 .eslintcache
+
+# IDE
 .idea
 .vscode
+
 .DS_Store
 package-lock.json

.prettierignore

@@ -1,3 +1,6 @@
 package.json
 test/**/*.*
 !test/*.js
+
+# Generated files
+types.d.ts

.prettierrc.js

@@ -3,7 +3,6 @@ module.exports = {
 	useTabs: true,
 	tabWidth: 2,
 	trailingComma: "none",
-	arrowParens: "avoid",
 	overrides: [
 		{
 			files: "*.json",
@@ -13,7 +12,7 @@ module.exports = {
 			}
 		},
 		{
-			files: "*.ts",
+			files: "*.{cts,mts,ts}",
 			options: {
 				parser: "typescript"
 			}

README.md

@@ -72,18 +72,21 @@ const context = {};
 const lookupStartPath = "/Users/webpack/some/root/dir";
 const request = "./path/to-look-up.js";
 const resolveContext = {};
-myResolver.resolve(context, lookupStartPath, request, resolveContext, (
-	err /*Error*/,
-	filepath /*string*/
-) => {
-	// Do something with the path
-});
+myResolver.resolve(
+	context,
+	lookupStartPath,
+	request,
+	resolveContext,
+	(err /*Error*/, filepath /*string*/) => {
+		// Do something with the path
+	}
+);
 ```
 
 #### Resolver Options
 
 | Field            | Default                     | Description                                                                                                                                               |
-|------------------|-----------------------------| --------------------------------------------------------------------------------------------------------------------------------------------------------- |
+| ---------------- | --------------------------- | --------------------------------------------------------------------------------------------------------------------------------------------------------- |
 | alias            | []                          | A list of module alias configurations or an object which maps key to value                                                                                |
 | aliasFields      | []                          | A list of alias fields in description files                                                                                                               |
 | extensionAlias   | {}                          | An object which maps extension to extension aliases                                                                                                       |
@@ -94,7 +97,7 @@ myResolver.resolve(context, lookupStartPath, request, resolveContext, (
 | enforceExtension | false                       | Enforce that a extension from extensions must be used                                                                                                     |
 | exportsFields    | ["exports"]                 | A list of exports fields in description files                                                                                                             |
 | extensions       | [".js", ".json", ".node"]   | A list of extensions which should be tried for files                                                                                                      |
-| fallback         | []                          | Same as `alias`, but only used if default resolving fails                                                                                                |                                                                                                                                                                                         
+| fallback         | []                          | Same as `alias`, but only used if default resolving fails                                                                                                 |
 | fileSystem       |                             | The file system which should be used                                                                                                                      |
 | fullySpecified   | false                       | Request passed to resolve is already fully specified and extensions or main files are not resolved for it (they are still resolved for internal requests) |
 | mainFields       | ["main"]                    | A list of main fields in description files                                                                                                                |

jest.config.js

@@ -1,3 +1,3 @@
 module.exports = {
-    moduleFileExtensions: ["js", "mjs", "cjs", "ts"]
+	moduleFileExtensions: ["js", "mjs", "cjs", "ts"]
 };

lib/AliasPlugin.js

@@ -36,7 +36,7 @@ module.exports = class AliasPlugin {
 		 * @param {string} maybeAbsolutePath path
 		 * @returns {null|string} absolute path with slash ending
 		 */
-		const getAbsolutePathWithSlashEnding = maybeAbsolutePath => {
+		const getAbsolutePathWithSlashEnding = (maybeAbsolutePath) => {
 			const type = getType(maybeAbsolutePath);
 			if (type === PathType.AbsolutePosix || type === PathType.AbsoluteWin) {
 				return resolver.join(maybeAbsolutePath, "_").slice(0, -1);

lib/CachedInputFileSystem.js

@@ -22,7 +22,7 @@ const nextTick = require("process").nextTick;
  * @param {string} path path
  * @returns {string} dirname
  */
-const dirname = path => {
+const dirname = (path) => {
 	let idx = path.length - 1;
 	while (idx >= 0) {
 		const c = path.charCodeAt(idx);

lib/ModulesInHierarchicalDirectoriesPlugin.js

@@ -37,8 +37,8 @@ module.exports = class ModulesInHierarchicalDirectoriesPlugin {
 				(request, resolveContext, callback) => {
 					const fs = resolver.fileSystem;
 					const addrs = getPaths(/** @type {string} */ (request.path))
-						.paths.map(p => {
-							return this.directories.map(d => resolver.join(p, d));
+						.paths.map((p) => {
+							return this.directories.map((d) => resolver.join(p, d));
 						})
 						.reduce((array, p) => {
 							array.push.apply(array, p);

lib/Resolver.js

@@ -347,7 +347,7 @@ const {
  * @returns {string} in camel case
  */
 function toCamelCase(str) {
-	return str.replace(/-([a-z])/g, str => str.slice(1).toUpperCase());
+	return str.replace(/-([a-z])/g, (str) => str.slice(1).toUpperCase());
 }
 
 class Resolver {
@@ -519,15 +519,15 @@ class Resolver {
 			/**
 			 * @param {ResolveRequest} obj object
 			 */
-			yield_ = obj => {
+			yield_ = (obj) => {
 				old(obj);
 				yieldCalled = true;
 			};
 			/**
 			 * @param {ResolveRequest} result result
 			 * @returns {void}
 			 */
-			finishYield = result => {
+			finishYield = (result) => {
 				if (result) {
 					/** @type {ResolveContextYield} */ (yield_)(result);
 				}
@@ -541,7 +541,7 @@ class Resolver {
 		 * @param {ResolveRequest} result result
 		 * @returns {void}
 		 */
-		const finishResolved = result => {
+		const finishResolved = (result) => {
 			return callback(
 				null,
 				result.path === false
@@ -557,7 +557,7 @@ class Resolver {
 		 * @param {string[]} log logs
 		 * @returns {void}
 		 */
-		const finishWithoutResolve = log => {
+		const finishWithoutResolve = (log) => {
 			/**
 			 * @type {ErrorWithDetail}
 			 */
@@ -577,7 +577,7 @@ class Resolver {
 				obj,
 				message,
 				{
-					log: msg => {
+					log: (msg) => {
 						parentLog(msg);
 						log.push(msg);
 					},
@@ -639,7 +639,7 @@ class Resolver {
 						obj,
 						message,
 						{
-							log: msg => log.push(msg),
+							log: (msg) => log.push(msg),
 							yield: yield_,
 							stack: resolveContext.stack
 						},

lib/ResolverFactory.js

@@ -157,7 +157,7 @@ function processPnpApiOption(option) {
  */
 function normalizeAlias(alias) {
 	return typeof alias === "object" && !Array.isArray(alias) && alias !== null
-		? Object.keys(alias).map(key => {
+		? Object.keys(alias).map((key) => {
 				/** @type {AliasOptionEntry} */
 				const obj = { name: key, onlyModule: false, alias: alias[key] };
 
@@ -226,7 +226,7 @@ function createOptions(options) {
 				: options.enforceExtension,
 		extensions: new Set(options.extensions || [".js", ".json", ".node"]),
 		extensionAlias: options.extensionAlias
-			? Object.keys(options.extensionAlias).map(k => ({
+			? Object.keys(options.extensionAlias).map((k) => ({
 					extension: k,
 					alias: /** @type {ExtensionAliasOptions} */ (options.extensionAlias)[
 						k
@@ -252,7 +252,7 @@ function createOptions(options) {
 				: options.modules
 				? [options.modules]
 				: ["node_modules"],
-			item => {
+			(item) => {
 				const type = getType(item);
 				return type === PathType.Normal || type === PathType.Relative;
 			}
@@ -394,10 +394,10 @@ exports.createResolver = function (options) {
 	if (alias.length > 0) {
 		plugins.push(new AliasPlugin("raw-resolve", alias, "internal-resolve"));
 	}
-	aliasFields.forEach(item => {
+	aliasFields.forEach((item) => {
 		plugins.push(new AliasFieldPlugin("raw-resolve", item, "internal-resolve"));
 	});
-	extensionAlias.forEach(item =>
+	extensionAlias.forEach((item) =>
 		plugins.push(
 			new ExtensionAliasPlugin("raw-resolve", item, "normal-resolve")
 		)
@@ -437,7 +437,7 @@ exports.createResolver = function (options) {
 	}
 
 	// internal
-	importsFields.forEach(importsField => {
+	importsFields.forEach((importsField) => {
 		plugins.push(
 			new ImportsFieldPlugin(
 				"internal",
@@ -450,18 +450,18 @@ exports.createResolver = function (options) {
 	});
 
 	// raw-module
-	exportsFields.forEach(exportsField => {
+	exportsFields.forEach((exportsField) => {
 		plugins.push(
 			new SelfReferencePlugin("raw-module", exportsField, "resolve-as-module")
 		);
 	});
-	modules.forEach(item => {
+	modules.forEach((item) => {
 		if (Array.isArray(item)) {
 			if (item.includes("node_modules") && pnpApi) {
 				plugins.push(
 					new ModulesInHierarchicalDirectoriesPlugin(
 						"raw-module",
-						item.filter(i => i !== "node_modules"),
+						item.filter((i) => i !== "node_modules"),
 						"module"
 					)
 				);
@@ -531,7 +531,7 @@ exports.createResolver = function (options) {
 	);
 
 	// resolve-in-package
-	exportsFields.forEach(exportsField => {
+	exportsFields.forEach((exportsField) => {
 		plugins.push(
 			new ExportsFieldPlugin(
 				"resolve-in-package",
@@ -603,7 +603,7 @@ exports.createResolver = function (options) {
 				"existing-directory"
 			)
 		);
-		mainFiles.forEach(item => {
+		mainFiles.forEach((item) => {
 			plugins.push(
 				new UseFilePlugin(
 					"undescribed-existing-directory",
@@ -614,7 +614,7 @@ exports.createResolver = function (options) {
 		});
 
 		// described-existing-directory
-		mainFields.forEach(item => {
+		mainFields.forEach((item) => {
 			plugins.push(
 				new MainFieldPlugin(
 					"existing-directory",
@@ -623,7 +623,7 @@ exports.createResolver = function (options) {
 				)
 			);
 		});
-		mainFiles.forEach(item => {
+		mainFiles.forEach((item) => {
 			plugins.push(
 				new UseFilePlugin("existing-directory", item, "undescribed-raw-file")
 			);
@@ -653,14 +653,14 @@ exports.createResolver = function (options) {
 		if (!enforceExtension) {
 			plugins.push(new TryNextPlugin("raw-file", "no extension", "file"));
 		}
-		extensions.forEach(item => {
+		extensions.forEach((item) => {
 			plugins.push(new AppendPlugin("raw-file", item, "file"));
 		});
 
 		// file
 		if (alias.length > 0)
 			plugins.push(new AliasPlugin("file", alias, "internal-resolve"));
-		aliasFields.forEach(item => {
+		aliasFields.forEach((item) => {
 			plugins.push(new AliasFieldPlugin("file", item, "internal-resolve"));
 		});
 		plugins.push(new NextPlugin("file", "final-file"));

lib/ResultPlugin.js

@@ -27,7 +27,7 @@ module.exports = class ResultPlugin {
 				const obj = { ...request };
 				if (resolverContext.log)
 					resolverContext.log("reporting result " + obj.path);
-				resolver.hooks.result.callAsync(obj, resolverContext, err => {
+				resolver.hooks.result.callAsync(obj, resolverContext, (err) => {
 					if (err) return callback(err);
 					if (typeof resolverContext.yield === "function") {
 						resolverContext.yield(obj);

lib/SyncAsyncFileSystemDecorator.js

@@ -174,7 +174,7 @@
 			);
 		this.readJsonSync =
 			/** @type {SyncFileSystem["readJsonSync"]} */
-			(arg => readJsonSync.call(fs, arg));
+			((arg) => readJsonSync.call(fs, arg));
 	}
 
 	this.realpath = undefined;

lib/UnsafeCachePlugin.js

@@ -82,7 +82,7 @@ module.exports = class UnsafeCachePlugin {
 				const yieldResult = [];
 				if (isYield) {
 					yieldFn = resolveContext.yield;
-					yield_ = result => {
+					yield_ = (result) => {
 						yieldResult.push(result);
 					};
 				}

lib/createInnerContext.js

@@ -20,7 +20,7 @@ module.exports = function createInnerContext(options, message) {
 			/**
 			 * @param {string} msg message
 			 */
-			innerLog = msg => {
+			innerLog = (msg) => {
 				if (!messageReported) {
 					/** @type {(function(string): void)} */
 					(options.log)(message);

lib/util/entrypoints.js

@@ -87,7 +87,7 @@ module.exports.processExportsField = function processExportsField(
 ) {
 	return createFieldProcessor(
 		buildExportsField(exportsField),
-		request => (request.length === 0 ? "." : "./" + request),
+		(request) => (request.length === 0 ? "." : "./" + request),
 		assertExportsFieldRequest,
 		assertExportTarget
 	);
@@ -102,7 +102,7 @@ module.exports.processImportsField = function processImportsField(
 ) {
 	return createFieldProcessor(
 		importsField,
-		request => "#" + request,
+		(request) => "#" + request,
 		assertImportsFieldRequest,
 		assertImportTarget
 	);

lib/util/path.js

@@ -45,7 +45,7 @@ exports.deprecatedInvalidSegmentRegEx = deprecatedInvalidSegmentRegEx;
  * @param {string} p a path
  * @returns {PathType} type of path
  */
-const getType = p => {
+const getType = (p) => {
 	switch (p.length) {
 		case 0:
 			return PathType.Empty;
@@ -129,7 +129,7 @@ exports.getType = getType;
  * @param {string} p a path
  * @returns {string} the normalized path
  */
-const normalize = p => {
+const normalize = (p) => {
 	switch (getType(p)) {
 		case PathType.Empty:
 			return p;