feat(tsx): use swc

[AugustinMauroy]

• reduce dependency size by 20KO
• it's supposed to perform better

https://swc.rs/docs/benchmarks

[JakobJingleheimer]

The benchmarks on that link appear to be broken: there's only 1 bar or line in the charts.

[AugustinMauroy]

The benchmarks on that link appear to be broken: there's only 1 bar or line in the charts.

did you opened an issue on their repo ?

[JakobJingleheimer]

No, not yet. Wanted to check with you because you saw it working before (so checking with you that what you saw before is not what you see now).

[AugustinMauroy]

@JakobJingleheimer should I also syncify tsx on this pr ????

My advice is to land this pr like this and make a refracto from behind to syncify all.

[JakobJingleheimer]

The benchmarks on that link appear to be broken: there's only 1 bar or line in the charts.

did you opened an issue on their repo ?

This was apparently a choice:

kdy1 commented on Apr 22
I removed the benchmark CI action because of the CI cost. I think we should document it

[JakobJingleheimer]

I would not combine those changes for a couple reasons:

1. Syncification has not landed in node, so this hook would be broken when used with another hook that is async (because this hook would fail to await the other).
2. They can't be backed out independently, and they're not related.

I already have a PR to syncify the hooks btw: #16. Syncifying introduces a non-trivial performance degradation (which I raised to node), which needs to be explored.

[AugustinMauroy]

The decision in the project is not to support/create a loader if one is already official.
https://github.com/swc-project/swc-node/tree/master/packages/register#swc-noderegister