chore: Prepare to use TypeScript natively

[JPeer264]

Description

This PR is a starting point of upgrading to TypeScript and preparing for future upgrades around testing and maintenance.

I've seen that couple of types between the *.js files and the index.d.ts were not 100% in sync, which is one of the reasons the move to TypeScript natively could help here.

What's new / changed

• moved js to lib in order to keep js as the same output folder which is delivered to NPM
• js/index.d.ts stays as type export for NPM and the types are re-exported via the jsdoc @typedef
• types.ts is actually the same as the previous index.d.ts excluding the SentryCli class
• TypeScript is partially used to keep the changes minimal
• I aligned the jsdoc types with the actual text from the previous index.d.ts

[JPeer264]

@andreiborza this is now ready to review. Had some issues with the postinstall, but this is solved now.

[szokeasaurusrex]

Added some comments, mostly containing some noob JS questions.

My one concern is: several of the changes here look like they might break the public JS API; am I understanding that correctly?

[JPeer264]

My one concern is: several of the changes here look like they might break the public JS API; am I understanding that correctly?

If you run npm install --ignore-scripts and npm run build locally - you'll see that the index.d.ts inside js-dist is exporting the same types as before (just a slightly different format). So there shouldn't be any breaking changes regarding the API.

[andreiborza]

Very nice!

[cursor]

Bug: NPM Postbuild Re-downloads Binary After Compile

The postbuild hook running npm run install-cli causes the CLI binary download script to execute after every TypeScript compilation. This creates unnecessary overhead during development since rebuilding TypeScript source doesn't require re-downloading or re-verifying the binary, and could fail in offline environments or slow down builds significantly with network latency.

 

[szokeasaurusrex]

I'll wait for @andreiborza to approve this, then I can provide a stamp (I think my approval is needed for some of the changes based on the code owner config)

[andreiborza]

@szokeasaurusrex approved :)

[szokeasaurusrex]

noice