Skip to content

Conversation

grod220
Copy link
Member

@grod220 grod220 commented Sep 2, 2025

A follow up to #262

Updates the IDL to support newly added instructions: SyncMetadataToSplToken & SyncMetadataToToken2022.

@grod220 grod220 requested a review from lorisleiva September 2, 2025 20:00
Comment on lines 1 to 5
---
"@solana-program/token-wrap": patch
---

Update CreateMint helper to support token-2022 extension sizing
Copy link
Member

Choose a reason for hiding this comment

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

That's because we use git cliff on these program repos right? (Changeset is unfortunately JavaScript only).

Copy link
Member Author

Choose a reason for hiding this comment

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

This update was due to a program change where extensions were added to the wrapped token-2022 versions. This required a new conditional: https://github.com/solana-program/token-wrap/blob/main/clients/js/src/create-mint.ts#L70-L72

Copy link
Member

Choose a reason for hiding this comment

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

Okay no worries. I was not really questioning the content of the changeset but more why this repo is using changesets altogether when the other repos in the solana-program org are not. Not because it shouldn't but to understand if we should do the same on the other repos for consistency. I believe the other repos are using git cliff since it allows changelogs based on folders and not JS packages which is useful in a multi-language repo. This is out of scope for this PR anyway but I was curious. ☺️

Copy link
Member Author

@grod220 grod220 Sep 3, 2025

Choose a reason for hiding this comment

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

Oh! I used Kit as inspiration for using changesets and assumed it was the standard across the board for web projects at anza. My bad, didn't realize this. It doesn't feel urgent to fix now, but will definitely revisit this.

Copy link
Member

@lorisleiva lorisleiva left a comment

Choose a reason for hiding this comment

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

Perfect, thank you!

@grod220 grod220 merged commit 4c60e2c into main Sep 3, 2025
10 checks passed
@grod220 grod220 deleted the update-idl branch September 3, 2025 12:40
@grod220 grod220 mentioned this pull request Sep 3, 2025
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.

2 participants