Add name property to TwirpError

[frederikprijck]

We are using TwirpError together with VError, more specifically the findCauseByName utility to find errors of specific kinds based on the .name property.

As twirp-ts does not set a custom name property on TwirpError, we have to rely on using instanceof TwirpError, which works but is a bit unexpected.

Wanted to open an issue, but figured I'd open this PR to propose + discuss. Feel free to close if this isn't something you want to add 👍

[cristiandouce]

@fenos would you think this is something we could merge and release?

[cristiandouce]

@nningego sorry to ping you, would you know if anyone is maintaining this repo? 🙏

[armandogmendoza]

Hello @cristiandouce @frederikprijck
This package is in maintenance mode and we don't plan to do any new releases.
Our suggestion is that you fork the repo or find another fork that has been actively maintained.
Anyway, the patch looks good and it makes sense.

[MikeMcC399]

@armandogmendoza

This package is in maintenance mode and we don't plan to do any new releases.
Our suggestion is that you fork the repo or find another fork that has been actively maintained.

Please add this information to the README file and consider marking the npm package twirp-ts as deprecated in the npm registry.

I'm not sure if the statement means the repo is really in maintenance mode. If there are no more releases, not even for bug-fixes, then this would be an end-of-life status. Since there is no issue list available, it is not possible to report any issues. It might also be appropriate to archive the repo in this situation.