module: load source maps in commonjs translator

PR-URL: #51033 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Geoffrey Booth <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: James M Snell <[email protected]>

Author: privatenumber

lib/internal/modules/esm/translators.js

@@ -289,6 +289,8 @@ function createCJSModuleWrap(url, source, isMain, loadCJS = loadCJSModule){
 // In case the source was not provided by the `load` step, we need fetch it now.
 source=stringify(source??getSource(newURL(url)).source);
 
+maybeCacheSourceMap(url,source);
+
 const{ exportNames, module }=cjsPreparseModuleExports(filename,source);
 cjsCache.set(url,module);
 constnamesWithDefault=exportNames.has('default') ?

test/es-module/test-esm-loader-hooks.mjs

@@ -737,6 +737,39 @@ describe('Loader hooks',{concurrency: true }, () =>{
 assert.strictEqual(signal,null);
 });
 
+it('should support source maps in commonjs translator',async()=>{
+constreadFile=async()=>{};
+consthook=`
+ import{readFile } from 'node:fs/promises'
+ export ${
+asyncfunctionload(url,context,nextLoad){
+constresolved=awaitnextLoad(url,context);
+if(context.format==='commonjs'){
+resolved.source=awaitreadFile(newURL(url));
+}
+returnresolved;
+}
+}`;
+
+const{ code, signal, stdout, stderr }=awaitspawnPromisified(execPath,[
+'--no-warnings',
+'--enable-source-maps',
+'--import',
+`data:text/javascript,${encodeURIComponent(`
+ import{register } from "node:module"
+ register(${
+JSON.stringify('data:text/javascript,'+encodeURIComponent(hook))
+});
+ `)}`,
+fixtures.path('source-map/throw-on-require.js'),
+]);
+
+assert.strictEqual(stdout,'');
+assert.match(stderr,/throw-on-require\.ts:9:9/);
+assert.strictEqual(code,1);
+assert.strictEqual(signal,null);
+});
+
 it('should handle mixed of opt-in modules and non-opt-in ones',async()=>{
 const{ code, signal, stdout, stderr }=awaitspawnPromisified(execPath,[
 '--no-warnings',