module: use sync cjs when importing cts

PR-URL: #60072
Backport-PR-URL: #62029Fixes: #59963
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com>
Fixes: #61801

Author: marco-ippolito

lib/internal/modules/esm/loader.js

@@ -493,7 +493,9 @@ class ModuleLoader {
 constloadResult=this.#loadSync(url,{ format, importAttributes });
 
 // Use the synchronous commonjs translator which can deal with cycles.
-constfinalFormat=loadResult.format==='commonjs' ? 'commonjs-sync' : loadResult.format;
+constfinalFormat=
+loadResult.format==='commonjs'||
+loadResult.format==='commonjs-typescript' ? 'commonjs-sync' : loadResult.format;
 
 if(finalFormat==='wasm'){
 assert.fail('WASM is currently unsupported by require(esm)');

test/es-module/test-typescript-commonjs.mjs

@@ -186,3 +186,13 @@ test('expect failure of a .cts file requiring esm in node_modules', async () =>
 match(result.stderr,/ERR_UNSUPPORTED_NODE_MODULES_TYPE_STRIPPING/);
 strictEqual(result.code,1);
 });
+
+test('cts -> require mts -> import cts',async()=>{
+constresult=awaitspawnPromisified(process.execPath,[
+fixtures.path('typescript/cts/issue-59963/a.cts'),
+]);
+
+strictEqual(result.stderr,'');
+strictEqual(result.stdout,'Hello from c.cts\n');
+strictEqual(result.code,0);
+});

test/es-module/test-typescript-module.mjs

@@ -133,3 +133,14 @@ test('execute .ts file importing a module', async () => {
 strictEqual(result.stdout,'Hello, TypeScript!\n');
 strictEqual(result.code,0);
 });
+
+test('mts -> import cts -> require mts',async()=>{
+constresult=awaitspawnPromisified(process.execPath,[
+'--no-warnings',
+fixtures.path('typescript/mts/issue-59963/a.mts'),
+]);
+
+strictEqual(result.stderr,'');
+strictEqual(result.stdout,'Hello from c.mts\n');
+strictEqual(result.code,0);
+});

test/fixtures/typescript/cts/issue-59963/a.cts

@@ -0,0 +1,3 @@
+const{ message }=require("./b.mts");
+interfaceFoo{};
+console.log(message);

test/fixtures/typescript/cts/issue-59963/b.mts

@@ -0,0 +1,2 @@
+interfaceFoo{};
+export{message}from"./c.cts";

test/fixtures/typescript/cts/issue-59963/c.cts

@@ -0,0 +1,2 @@
+constmessage: string="Hello from c.cts";
+module.exports={ message };

test/fixtures/typescript/mts/issue-59963/a.mts

@@ -0,0 +1,3 @@
+import{message}from"./b.cts";
+interfaceFoo{};
+console.log(message);

test/fixtures/typescript/mts/issue-59963/b.cts

@@ -0,0 +1,3 @@
+const{ message }=require("./c.mts");
+interfaceFoo{};
+module.exports={ message };

test/fixtures/typescript/mts/issue-59963/c.mts

@@ -0,0 +1 @@
+exportconstmessage: string="Hello from c.mts";