Merge pull request #108 from JakobJingleheimer/patch-1

fix: loader stomping context from other loaders
testdouble · Mar 16, 2024 · fdf9156 · fdf9156
2 parents 0cfc7b6 + e8f1211
commit fdf9156
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/lib/quibble.mjs b/lib/quibble.mjs
@@ -55,15 +55,15 @@ export async function resolve (specifier, context, nextResolve) {
   const { parentURL } = context
 
   try {
-    const { url } = await resolve()
+    const { url, ...ctx } = await resolve()
 
     const quibbledUrl = addQueryToUrl(url, '__quibble', stubModuleGeneration)
 
     if (url.startsWith('node:') && !getStubsInfo(quibbledUrl)) {
-      return { url }
+      return { ...ctx, url } // It's allowed to change ctx for a builtin (but unlikely)
     }
 
-    return { url: quibbledUrl }
+    return { ...ctx, url: quibbledUrl }
   } catch (error) {
     if (error.code === 'ERR_MODULE_NOT_FOUND') {
       return {