Should erasableSyntaxOnly be enabled

[cjihrig]

TypeScript 5.8 added a new compiler option named erasableSyntaxOnly. This option is somewhat of a collaboration with the Node core team as it disallows TypeScript syntax that Node's type stripping does not support.

There are currently 49 errors reported when enabling this option, which is why I'm opening an issue instead of a PR.

If we don't want to work toward enabling it, that's fine, and we can carry on with the status quo. If we do want to enable it, it would lead to a future where we no longer need to transpile for anything except npm publish and running the transpiled tests in the CI.

[brendandburns]

I'll start working on fixing these, seems like a good idea.

[brendandburns]

I started looking into this. Mostly the errors are due to the use of automatic member variables via readonly constructor parameters. Those are easy to fix, though they do make the code a little more verbose.

Some of them are due to the use of enums in the generated code. That's going to require upstream changes in the code generator and a regeneration of the code.

I'll send a PR with all of the local fixes sometime in the next day or two, then work on the upstream generator (unless someone else in the community gets there first)

[brendandburns]

Generator changes are here:
OpenAPITools/openapi-generator#21040