resolve to wrong module with aliasFields

[ahabhgk]

webpack-test/cases/resolving/browser-field should pass

[Boshen]

Here are the missing pieces:

1. node_modules is missing from https://github.com/web-infra-dev/rspack/tree/main/webpack-test/cases/resolving/browser-field

2. To pass this:

   rspack/webpack-test/cases/resolving/browser-field/index.js

   Line 2 in 6f85b03

   expect(require("replacing-module1")).toBe("new-module");

We need to set browserField: true in the webpack config

rspack/packages/rspack/src/config/zod.ts

Lines 321 to 325 in 6f85b03

	/**
	* This is `aliasField: ["browser"]` in webpack, because no one
	* uses aliasField other than "browser". ---@bvanjoi
	*/
	browserField: z.boolean().optional(),

But it seems like there is a big mismatch between all these configs - See https://www.rspack.dev/config/resolve.html#resolvealiasfield and https://webpack.js.org/configuration/resolve/#resolvealiasfields

---

FYI oxc_resolver reads alias_fields

rspack/crates/rspack_core/src/resolver/resolver_impl.rs

Lines 313 to 320 in 6f85b03

	let alias_fields = (if options.browser_field.unwrap_or(true) {
	vec!["browser".to_string()]
	} else {
	vec![]
	})
	.into_iter()
	.map(|x| vec![x])
	.collect();

[Boshen]

I'm still investigating on how to pass

rspack/webpack-test/cases/resolving/browser-field/index.js

Lines 40 to 43 in 6f85b03

	if (Math.random() < 0) require("recursive-file/a");
	if (Math.random() < 0) require("recursive-file/b");
	if (Math.random() < 0) require("recursive-file/c");
	if (Math.random() < 0) require("recursive-file/d");

but it seems the rspack suite commented it out:

rspack/packages/rspack/tests/cases/resolve/browser-field/index.js

Lines 38 to 43 in 6f85b03

	// TODO: missing module
	// if (Math.random() < 0) require("recursive-file/a");
	// if (Math.random() < 0) require("recursive-file/b");
	// if (Math.random() < 0) require("recursive-file/c");
	// if (Math.random() < 0) require("recursive-file/d");

---

enhanced-resolve will throw an infinite recursion error, while oxc_resolver will resolve it to webpack-test/cases/resolving/browser-field/node_modules/recursive-file/a.js

[Boshen]

For debugging what's being passed into oxc_resolver:

cd webpack-test
OXC_RESOLVER=TRACE pnpm t -- -t "resolving browser-field" | grep  alias_fields

and you should see alias_fields: [["browser"]]