support native Node.js Typescript type-stripping

[mcollina]

Given that Node.js v23 has unflagged type stripping, I think we should support it out of the box

[mcollina]

cc @SamSalvatico @jean-michelet

[jean-michelet]

We should wait for #440 to be merged.
Do you want to do it @SamSalvatico, or I take this one?

[SamSalvatico]

I can take It @jean-michelet

[SamSalvatico]

@jean-michelet I gave a look, but unfortunately I am not able to progress into this (missing knowledge on my side)

[jean-michelet]

Have you had a look at runtime module?

I think we should detect the use here. But I haven't tested this new Node feature yet, I'll have a look.

[SamSalvatico]

runtime module

I gave a look to those links, but I cannot find a way in which the auto enable type stripping is checked, because it only looks for checkEnvVariable('FASTIFY_AUTOLOAD_TYPESCRIPT') or external tools, not node version

[jean-michelet]

it only looks for checkEnvVariable('FASTIFY_AUTOLOAD_TYPESCRIPT') or external tools, not node version

You can have access Node version on process.

I think supportTypeScript should return true if node version >= 23.
Regarding testing, maybe we can create a script unit-typescript-native-type-strippping.js that exec ts test suite only if node version >= 23.

[jean-michelet]

@mcollina
I started to work on this, I am positive I can push a PR soon, but this error will be unreachable from Node 23:
https://github.com/fastify/fastify-autoload/blob/master/lib/find-plugins.js#L148

So it does break the test that expect this error: https://github.com/fastify/fastify-autoload/tree/master/test/issues/369

For now, all I can think of is a wonky combination of a conditional test execution based on runtime Node version and ignoring coverage for L148.

[jean-michelet]

Should we offer a way to enforce supportTypeScript = false? We can also try to check if the flag --no-experimental-strip-types is passed?

[mcollina]

No, I don't think we should enforce it to false.

[mcollina]

I would not try to check the cli option either.

[Fdawgs]

Closed by #442