esm: syncify default path of ModuleLoader.load

PR-URL: #57419
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>

Author: JakobJingleheimer

lib/internal/modules/esm/load.js

@@ -26,32 +26,7 @@ const {
 
 /**
  * @param {URL} url URL to the module
- * @param {ESModuleContext} context used to decorate error messages
- * @returns {Promise<{ responseURL: string, source: string | BufferView }>}
- */
-asyncfunctiongetSource(url,context){
-const{ protocol, href }=url;
-constresponseURL=href;
-letsource;
-if(protocol==='file:'){
-const{readFile: readFileAsync}=require('internal/fs/promises').exports;
-source=awaitreadFileAsync(url);
-}elseif(protocol==='data:'){
-constresult=dataURLProcessor(url);
-if(result==='failure'){
-thrownewERR_INVALID_URL(responseURL,null);
-}
-source=BufferFrom(result.body);
-}else{
-constsupportedSchemes=['file','data'];
-thrownewERR_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 }}
  */
 functiongetSourceSync(url,context){
@@ -80,7 +55,7 @@ function getSourceSync(url, context) {
  * @param {LoadContext} context
  * @returns {LoadReturn}
  */
-asyncfunctiondefaultLoad(url,context=kEmptyObject){
+functiondefaultLoad(url,context=kEmptyObject){
 letresponseURL=url;
 let{
  importAttributes,
@@ -110,13 +85,13 @@ async function defaultLoad(url, context = kEmptyObject) {
 source=null;
 }elseif(format!=='commonjs'){
 if(source==null){
-({ responseURL, source }=awaitgetSource(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=awaitdefaultGetFormat(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

@@ -806,9 +806,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 }}
  */
-asyncload(url,context){
+load(url,context){
 if(loadHooks.length){
 // Has module.registerHooks() hooks, use the synchronous variant that can handle both hooks.
 returnthis.#loadSync(url,context);

lib/internal/modules/helpers.js

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