fix(core): skip TS support detection via require.extensions (#6488)

B4nan · web-flow · commit 3efdcd0a00d0 · 2025-03-07T09:46:42.000+01:00
This check was problematic because of Node.js type stripping uses it now too, and it does fail to parse decorators. It was already removed from v7, but since the type stripping will be backported to Node.js 22, I am removing it from v6 too, otherwise things break when executing the compiled code. Related to nodejs/node#57298 (comment)
diff --git a/packages/core/src/utils/Utils.ts b/packages/core/src/utils/Utils.ts
@@ -801,8 +801,7 @@ export class Utils {
       || !!process.env.VITEST // check if vitest is used
       || !!process.versions.bun // check if bun is used
       || process.argv.slice(1).some(arg => arg.includes('ts-node')) // registering ts-node runner
-      || process.execArgv.some(arg => arg === 'ts-node/esm') // check for ts-node/esm module loader
-      || (require.extensions && !!require.extensions['.ts']); // check if the extension is registered
+      || process.execArgv.some(arg => arg === 'ts-node/esm'); // check for ts-node/esm module loader
   }
 
   /**

Original file line number	Diff line number	Diff line change
`@@ -801,8 +801,7 @@ export class Utils {`
`801`	`801`	`\|\| !!process.env.VITEST // check if vitest is used`
`802`	`802`	`\|\| !!process.versions.bun // check if bun is used`
`803`	`803`	`\|\| process.argv.slice(1).some(arg => arg.includes('ts-node')) // registering ts-node runner`
`804`		`- \|\| process.execArgv.some(arg => arg === 'ts-node/esm') // check for ts-node/esm module loader`
`805`		`- \|\| (require.extensions && !!require.extensions['.ts']); // check if the extension is registered`
	`804`	`+ \|\| process.execArgv.some(arg => arg === 'ts-node/esm'); // check for ts-node/esm module loader`
`806`	`805`	`}`
`807`	`806`
`808`	`807`	`/**`