esm: syncify default path of ModuleLoader.load

PR-URL: #57419
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>

Author: JakobJingleheimer

lib/internal/modules/esm/load.js

@@ -30,32 +30,7 @@ const {
 
 /**
  * @param {URL} url URL to the module
- * @param {ESModuleContext} context used to decorate error messages
- * @returns {Promise<{ responseURL: string, source: string | BufferView }>}
- */
-async function getSource(url, context) {
-  const { protocol, href } = url;
-  const responseURL = href;
-  let source;
-  if (protocol === 'file:') {
-    const { readFile: readFileAsync } = require('internal/fs/promises').exports;
-    source = await readFileAsync(url);
-  } else if (protocol === 'data:') {
-    const result = dataURLProcessor(url);
-    if (result === 'failure') {
-      throw new ERR_INVALID_URL(responseURL, null);
-    }
-    source = BufferFrom(result.body);
-  } else {
-    const supportedSchemes = ['file', 'data'];
-    throw new ERR_UNSUPPORTED_ESM_URL_SCHEME(url, supportedSchemes);
-  }
-  return { __proto__: null, responseURL, source };
-}
-
-/**
- * @param {URL} url URL to the module
- * @param {ESModuleContext} context used to decorate error messages
+ * @param {LoadContext} context used to decorate error messages
  * @returns {{ responseURL: string, source: string | BufferView }}
  */
 function getSourceSync(url, context) {
@@ -84,7 +59,7 @@ function getSourceSync(url, context) {
  * @param {LoadContext} context
  * @returns {LoadReturn}
  */
-async function defaultLoad(url, context = kEmptyObject) {
+function defaultLoad(url, context = kEmptyObject) {
   let responseURL = url;
   let {
     importAttributes,
@@ -111,13 +86,13 @@ async function defaultLoad(url, context = kEmptyObject) {
     format ??= 'builtin';
   } else if (format !== 'commonjs' || defaultType === 'module') {
     if (source == null) {
-      ({ responseURL, source } = await getSource(urlInstance, context));
+      ({ responseURL, source } = getSourceSync(urlInstance, context));
       context = { __proto__: context, source };
     }
 
     if (format == null) {
       // Now that we have the source for the module, run `defaultGetFormat` to detect its format.
-      format = await defaultGetFormat(urlInstance, context);
+      format = defaultGetFormat(urlInstance, context);
 
       if (format === 'commonjs') {
         // For backward compatibility reasons, we need to discard the source in

lib/internal/modules/esm/loader.js

@@ -785,9 +785,9 @@ class ModuleLoader {
    * if any.
    * @param {string} url The URL of the module to be loaded.
    * @param {object} context Metadata about the module
-   * @returns {Promise<{ format: ModuleFormat, source: ModuleSource }>}
+   * @returns {Promise<{ format: ModuleFormat, source: ModuleSource }> | { format: ModuleFormat, source: ModuleSource }}
    */
-  async load(url, context) {
+  load(url, context) {
     if (loadHooks.length) {
       // Has module.registerHooks() hooks, use the synchronous variant that can handle both hooks.
       return this.#loadSync(url, context);

lib/internal/modules/helpers.js

@@ -264,7 +264,7 @@ function normalizeReferrerURL(referrerName) {
     }
 
     if (StringPrototypeStartsWith(referrerName, 'file://') ||
-        URLCanParse(referrerName)) {
+    URLCanParse(referrerName)) {
       return referrerName;
     }