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

WIP: Add prost support #1769

Closed
wants to merge 16 commits into from
Closed

WIP: Add prost support #1769

wants to merge 16 commits into from

Conversation

ethanfrey
Copy link
Member

We've had a lot of discussions here about how to add support for protobuf encoding in local contract storage. Forking cw-storage-plus is not that hard, but it requires that all core types in cosmwasm-std support both prost and serde encoding.

This is a demo of nicer apis to generate this alongside cw_serde as well as how to handle some patterns we have in the standard library.

It would be great to auto-generate the field tags as well to make this less work. That was demo'd in https://github.com/eira-fransham/autoproto but that code has no commit since 2021 and no tests on many lines of proc macros, so it probably requires a fork and a new maintainer before we could take that approach

@mergify mergify bot mentioned this pull request Jul 7, 2023
@mikedotexe
Copy link

from idea to real-and-working at an unparalleled velocity
grateful.

@webmaster128
Copy link
Member

The next iteration of storage library storey will use MessagePack for value serialization. This ensure we don't need escaping for string and have true binary support without loosing the convenience that serde brings. MessagePack is self-describing which makes schema-less debugging possible. In #2118 we'll make add first-class support for that in the standard library.

For this reason I'll close the protobuf route here

@webmaster128 webmaster128 deleted the add-prost-support branch May 26, 2024 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants