Uh oh!
There was an error while loading. Please reload this page.
fix(sdk): set maxTopUp to u16::MAX in all instruction builders - #2291
Conversation
PR #2279 fixed mint-to but missed 7 other instruction builders that hardcoded maxTopUp: 0 (no top-ups allowed). The on-chain program rejects any rent top-up with MaxTopUpExceeded (0x467b) when maxTopUp is 0. Set to 65535 (u16::MAX = no limit) to match the Rust SDK default. Files changed: - wrap.ts, unwrap.ts, create-decompress-interface-instruction.ts - create-mint.ts, decompress-mint.ts, mint-to-compressed.ts - update-mint.ts, update-metadata.ts
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (8)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including 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 |
ananas-block
commented
Feb 15, 2026
looks correct to me |
Uh oh!
There was an error while loading. Please reload this page.
Summary
maxTopUp: 0(no top-ups allowed), causingMaxTopUpExceeded(0x467b) when writing to accounts with compressible extensionsmint-to.tsbut missed the rest65535(u16::MAX= no limit), matching the Rust SDK default (Transfer2Config::default(),MintActionCompressedInstructionData::new())Files changed
wrap.tsunwrap.tscreate-decompress-interface-instruction.tscreate-mint.tsdecompress-mint.tsmint-to-compressed.tsupdate-mint.tsupdate-metadata.tsTest plan
LIGHT_PROTOCOL_VERSION=V2 pnpm run build)vitest run tests/unit)