throw recursion error when resolving cursed browser fields

[Boshen]

From webpack test suite: https://github.com/webpack/webpack/blob/87660921808566ef3b8796f8df61bd79fc026108/test/cases/resolving/browser-field/index.js#L40-L43

Given node_modules/recursive-file/package.json:

{
	"browser": {
		"a.js": "./a",
		"./b.js": "./b",
		"c.js": "./d.js",
		"./d.js": "./c.js"
	}
}

The following snippet should all throw recursion error.

require("recursive-file/a");
require("recursive-file/b");
require("recursive-file/c");
require("recursive-file/d");

Caveat:

require("module-c") should resolve to browser-module/node_modules/module-c.js given

https://github.com/oxc-project/oxc_resolver/blob/2baf418ab6f8c4cc5073d71dc9a628c43f1a0165/fixtures/enhanced_resolve/test/fixtures/browser-module/package.json#L7

[Boshen]

Current dependencies on/for this PR:

• main
  • PR throw recursion error when resolving cursed browser fields #17 👈

This stack of pull requests is managed by Graphite.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (009927b) 94.09% compared to head (dbdc251) 94.11%.
Report is 2 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #17      +/-   ##
==========================================
+ Coverage   94.09%   94.11%   +0.01%     
==========================================
  Files          11       11              
  Lines        2253     2259       +6     
==========================================
+ Hits         2120     2126       +6     
  Misses        133      133              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[codspeed-hq]

CodSpeed Performance Report

Merging #17 will not alter performance

_{⚠️ No base runs were found}

_{Falling back to comparing resolve-browser-field (dbdc251) with main (409f096)}

Summary

✅ 2 untouched benchmarks