Skip to content

Commit 9045e9e

Browse files
committed
fix: Don't use import.meta.resolve
import.meta.resolve was unflagged in Node.js 20.6 by nodejs/node#49028 Unfortunately, the unflagged version does not support the parentURL argument. Rather than attempting to detect whether the argument is supported (which is not as simple as .length testing) stop using it until a reliable version is unflagged. Signed-off-by: Kevin Locke <[email protected]>
1 parent 86eb4de commit 9045e9e

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

index.js

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,11 @@ function makeResolver() {
8080
// Could use resolve, if/when it supports package.json#exports:
8181
// https://github.com/browserify/resolve/issues/222
8282
// https://github.com/browserify/resolve/pull/224
83-
const resolveTransformer = import.meta.resolve || makeResolver();
83+
//
84+
// import.meta.resolve() was unflagged without the parentURL argument in
85+
// Node.js 20.6.0: https://github.com/nodejs/node/pull/49028
86+
// TODO[engine:node@>=22]: Use import.meta.resolve if it supports parentURL
87+
const resolveTransformer = makeResolver();
8488

8589
async function loadTransformer(name, parent) {
8690
const resolved = await resolveTransformer(name, parent);

0 commit comments

Comments
 (0)