fixup! module: handle null source from async loader hooks in sync hooks

Author: joyeecheung

lib/internal/modules/cjs/loader.js

@@ -1794,9 +1794,7 @@ function loadSource(mod, filename, formatFromNode) {
 
   const defaultLoad = getDefaultLoad(mod[kURL], filename);
   const loadResult = loadWithHooks(mod[kURL], mod[kFormat], /* importAttributes */ undefined,
-                                   getCjsConditionsArray(),
-                                   defaultLoad,
-                                   validateLoadStrict);
+                                   getCjsConditionsArray(), defaultLoad, validateLoadStrict);
   // Reset the module properties with load hook results.
   if (loadResult.format !== undefined) {
     mod[kFormat] = loadResult.format;

lib/internal/modules/customization_hooks.js

@@ -275,7 +275,7 @@ function validateLoadStrict(url, context, result) {
 }
 
 function validateLoadSloppy(url, context, result) {
-  validateSourceSloppy(url, context, result);
+  validateSourcePermissive(url, context, result);
   validateFormat(url, context, result);
   return result;
 }
@@ -299,11 +299,12 @@ function validateSourceStrict(url, context, result) {
   }
 }
 
-function validateSourceSloppy(url, context, result) {
+function validateSourcePermissive(url, context, result) {
   const { source, format } = result;
   if (format === 'commonjs' && source == null) {
     // Accommodate the quirk in defaultLoad used by asynchronous loader hooks
     // which sets source to null for commonjs.
+    // See: https://github.com/nodejs/node/issues/57327#issuecomment-2701382020
     return;
   }
   validateSourceStrict(url, context, result);

lib/internal/modules/esm/translators.js

@@ -344,7 +344,7 @@ translators.set('commonjs', function commonjsStrategy(url, translateContext, par
   }
 
   // For backward-compatibility, it's possible to return a nullish value for
-  // CJS source associated with a file: URL - that usually means the source is not
+  // CJS source associated with a `file:` URL - that usually means the source is not
   // customized (is loaded by default load) or the hook author wants it to be reloaded
   // through CJS routine. In this case, the source is obtained by calling the
   // monkey-patchable CJS loader.