Skip to content

feat: Pinocchio v10#73

Open
glowiedev wants to merge 5 commits into
codama-idl:mainfrom
glowiedev:feat-pinocchio-v10
Open

feat: Pinocchio v10#73
glowiedev wants to merge 5 commits into
codama-idl:mainfrom
glowiedev:feat-pinocchio-v10

Conversation

@glowiedev

Copy link
Copy Markdown

I've refactored the repo to generate the rust cpi client with the new changes from Pinocchio v9 to v10.1.

@changeset-bot

changeset-bot Bot commented Jan 14, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 49bc4f9

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@codama/renderers-rust-cpi Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@glowiedev

Copy link
Copy Markdown
Author

I've noticed there are many thing wrong here that have been missed when this repo was seperated from the main codama repo. For example types are not generated when they may be needed, also instruction data is erased as the set length in from_raw_parts is always 0 instead of the expected length of that instruction data. Adding to this the discriminator is never appended to the beginning. If it's alright I'd like to fix these issues before having this merged.

@glowiedev

Copy link
Copy Markdown
Author

Added support for anchor style discriminators by implementing bytes type visitor.

@glowiedev

Copy link
Copy Markdown
Author

@lorisleiva bump

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.

1 participant