fix(ext/node): resolve exports even if parent module filename isn't present

ext/node/ops/require.rs

@@ -532,12 +532,16 @@ where
     return Ok(None);
   };
 
-  let referrer = Url::from_file_path(parent_path).unwrap();
+  let referrer = if parent_path.is_empty() {
+    None
+  } else {
+    Some(Url::from_file_path(parent_path).unwrap())
+  };
   let r = node_resolver.package_exports_resolve(
     &pkg.path,
     &format!(".{expansion}"),
     exports,
-    Some(&referrer),
+    referrer.as_ref(),
     NodeModuleKind::Cjs,
     REQUIRE_CONDITIONS,
     NodeResolutionMode::Execution,

ext/node/polyfills/01_require.js

@@ -523,17 +523,13 @@ function resolveExports(
     return;
   }
 
-  if (!parentPath) {
-    return false;
-  }
-
   return op_require_resolve_exports(
     usesLocalNodeModulesDir,
     modulesPath,
     request,
     name,
     expansion,
-    parentPath,
+    parentPath ?? "",
   ) ?? false;
 }
 

tests/registry/npm/@denotest/cjs-multiple-exports/1.0.0/package.json

@@ -0,0 +1,8 @@
+{
+  "name": "@denotest/cjs-multiple-exports",
+  "version": "1.0.0",
+  "exports": {
+    ".": "./src/index.js",
+    "./add": "./src/add.js"
+  }
+}

tests/registry/npm/@denotest/cjs-multiple-exports/1.0.0/src/add.js

@@ -0,0 +1,3 @@
+module.exports = function add(a, b) {
+  return a + b;
+};

tests/registry/npm/@denotest/cjs-multiple-exports/1.0.0/src/index.js

@@ -0,0 +1,3 @@
+module.exports = {
+  hello: "world"
+};

tests/specs/node/require_export_from_parent_with_no_filename/__test__.jsonc

@@ -0,0 +1,13 @@
+{
+  "tempDir": true,
+  "steps": [
+    {
+      "args": "install",
+      "output": "[WILDCARD]"
+    },
+    {
+      "args": "run -A main.cjs",
+      "output": "3\n"
+    }
+  ]
+}

tests/specs/node/require_export_from_parent_with_no_filename/main.cjs

@@ -0,0 +1,16 @@
+const path = require("node:path");
+const Module = require("node:module");
+function requireFromString(code, filename) {
+  const paths = Module._nodeModulePaths((0, path.dirname)(filename));
+  const m = new Module(filename, module.parent);
+  m.paths = paths;
+  m._compile(code, filename);
+  return m.exports;
+}
+
+const code = `
+const add = require("@denotest/cjs-multiple-exports/add");
+
+console.log(add(1, 2));
+`;
+requireFromString(code, "fake.js");

tests/specs/node/require_export_from_parent_with_no_filename/package.json

@@ -0,0 +1,5 @@
+{
+  "dependencies": {
+    "@denotest/cjs-multiple-exports": "1.0.0"
+  }
+}