feat(ts/fast-strip): Distinguish invalid vs unsupported

[kdy1]

Description:

I tried adding a property to the error thrown, but wasm_bindgen does not support adding property to an error nor creating different kinds of errors.

So I throw an object with code and message instead.

[changeset-bot]

🦋 Changeset detected

Latest commit: 050b9ae

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[codspeed-hq]

CodSpeed Performance Report

Merging #9846 will not alter performance

_{Comparing kdy1:amaro-error (5fc3dc5) with main (ae53a35)}

Summary

✅ 206 untouched benchmarks

[marco-ippolito]

So when it has unsupported syntax it will have the label help: unsupported syntax.
That means I have to match with a regex which I think is not very efficient.
I'd rather not have the longer message, the error message is descriptive enough for users.
If its not possible to have it easily accessible in js I'd just remove the cause and leave it as is

[kdy1]

@marco-ippolito Is it fine to throw an object? I think wasm_bindgen would allow creating object with arbitrary properties.

(Not sure if it supports throwing them, though)

[marco-ippolito]

@marco-ippolito Is it fine to throw an object? I think wasm_bindgen would allow creating object with arbitrary properties.

(Not sure if it supports throwing them, though)

I think so since we always wrap what is thrown into a Node error. Let's give it a try

[kdy1]

@marco-ippolito Can you take a look on the new error output?

[marco-ippolito]

@marco-ippolito Can you take a look on the new error output?

So I'd expect always an object with
code and message
Code could be either InvalidSyntax or UnsupportedSyntax
Message the usual error message

[kdy1]

Yeap, I added some test cases

[robpalme]

Thanks for adding the new tests.

My comments are non-blocking.

[robpalme]

The ^^^ should point to the public keyword only.

[marco-ippolito]

Is this change included in the latest nightly?

[kdy1]

I triggered a full publish action for 1.10.7 (stable).

https://github.com/swc-project/swc/actions/runs/12703698875

[kdy1]

@marco-ippolito It's published