sea: allow requiring core modules with the "node:" prefix

Previously, the `require()` function exposed to the embedded SEA code
was calling the internal `require()` function if the module name
belonged to the list of public core modules but the internal `require()`
function does not support loading modules with the "node:" prefix, so
this change forwards the calls to another `require()` function that
supports this.

Fixes: nodejs/single-executable#69
Signed-off-by: Darshan Sen <raisinten@gmail.com>

lib/internal/util/embedding.js

@@ -36,12 +36,18 @@ function embedderRunCjs(contents) {
     customDirname);
 }
 
+let embedderRequireInternal;
+
 function embedderRequire(path) {
   if (!Module.isBuiltin(path)) {
     throw new ERR_UNKNOWN_BUILTIN_MODULE(path);
   }
 
-  return require(path);
+  if (!embedderRequireInternal) {
+    embedderRequireInternal = Module.createRequire(process.execPath);
+  }
+
+  return embedderRequireInternal(path);
 }
 
 module.exports = { embedderRequire, embedderRunCjs };

test/fixtures/sea.js

@@ -10,7 +10,7 @@ expectWarning('ExperimentalWarning',
               'might change at any time');
 
 const { deepStrictEqual, strictEqual, throws } = require('assert');
-const { dirname } = require('path');
+const { dirname } = require('node:path');
 
 deepStrictEqual(process.argv, [process.execPath, process.execPath, '-a', '--b=c', 'd']);