validator icon indicating copy to clipboard operation
validator copied to clipboard

feat: use enhanced-resolve for node resolution

Open 43081j opened this issue 1 month ago • 0 comments

This moves to using enhanced-resolve for resolving reporters in the CLI. Primarily because it is more up to date with the node resolution algorithm, but seems we dropped a few deep dependencies along the way too (which is good).

you can see it caught a bug in one of our fixtures (missing package exports) which node's own algorithm would disallow (and enhanced-resolve does too)

in future i feel like this should actually just use import.meta.resolve(path, base). however, that signature is only available behind a flag in node. I did also make a branch using the ponyfill (import-meta-resolve on npm) instead of this but it will refuse to import a directory from our ESM sources (rightly so) so the tests fail

if you'd prefer that, i'd be happy to push the branch

43081j avatar May 04 '24 14:05 43081j