Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate code generator .proto -> .ts from protobufjs to Telescope #219

Open
webmaster128 opened this issue Nov 6, 2023 · 0 comments
Open
Labels
enhancement New feature or request javascript Issues pertaining to the JavaScript implementation

Comments

@webmaster128
Copy link
Member

In #217 it was brought up the that current TypeScript code generator is not working properly anymore. My suggestion is to replace it instead of try to fix it.

There are basically 3 generations of TypeScript code generation we (Confio) used over the years

  1. protobufjs (pretty old school, TS types are generated out of JavaScript doc comments)
  2. ts-proto (quite nice, but a bit slow as it is a protoc plugin and does not allow interpreting Cosmos specific options)
  3. Telescope (most advanced solution)

It would be great to get the the TypeScript code generation migrated from 1. (used now) to 3. In addition to being the most powerful and convenient solution, it also means we can remove protobufjs runtime dependency, which is often reported as having security issues.

This affects the code in the generated folder, not the rest: https://github.com/cosmos/ics23/tree/master/js/src/generated. The generated types will be organized slightly differently but it should not be hard to adapt the JS codebase to it.

@crodriguezvega crodriguezvega added enhancement New feature or request javascript Issues pertaining to the JavaScript implementation labels Nov 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request javascript Issues pertaining to the JavaScript implementation
Projects
None yet
Development

No branches or pull requests

2 participants