Skip to content

Finish canoto Migration #2037

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

Open
wants to merge 33 commits into
base: main
Choose a base branch
from
Open

Finish canoto Migration #2037

wants to merge 33 commits into from

Conversation

RodrigoVillar
Copy link
Contributor

@RodrigoVillar RodrigoVillar commented Apr 27, 2025

This PR closes #1976 by migrating the following to use canoto serialization:

  • Actions
  • Outputs
  • Auths
  • ABI
    • The ABI now utilizes a different format which is coupled to the canoto specification.
  • Universal CLI
    • The Universal CLI uses canoto.Any to marshal actions and to unmarshal output values.

@RodrigoVillar RodrigoVillar self-assigned this Apr 27, 2025
@RodrigoVillar RodrigoVillar marked this pull request as ready for review April 28, 2025 21:35
}

// Unmarshal unmarshals a value of type [T] from the reader by unpacking
// the typeID and invoking the corresponding decoder function.
func (p *TypeParser[T]) Unmarshal(bytes []byte) (T, error) {
func (p *CanotoParser[T]) Unmarshal(bytes []byte) (T, error) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Talked offline about moving this into the canoto repo.

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.

Migrate User Defined Types to Canoto + Dynamic ABI
2 participants