fix: manually fix createidempotentAssociatedToken data serialization - #17
fix: manually fix createidempotentAssociatedToken data serialization#17swimricky wants to merge 1 commit into
Conversation
|
@swimricky is attempting to deploy a commit to the Metaplex Foundation Team on Vercel. A member of the Team first needs to authorize it. |
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Manually fixes the data serialization for createAssociatedTokenIdempotent
prior to this fix, the instruction is processed as the plain
createAssociatedTokenAccountsince the data is empty (https://github.com/solana-program/associated-token-account/blob/0b867b5340cd001e5980d8ca7928effc4e10015c/program/src/processor.rs#L42) and if you pass in an ATA account that's already initialized(or any account not owned by the system program), the transaction will fail.https://github.com/solana-program/associated-token-account/blob/program%40v8.0.0/program/src/processor.rs#L107-L109
I ran the
pnpm generate:clientand it didn't update anything. Didn't have time to look into kinobi (dont know if it's even still actively maintained) to see where the root cause was from that side.