feat: add native program examples #8
Closed
tilo-14 wants to merge 12 commits into
Closed
Conversation
tilo-14
changed the title
tilo-14
commented
Jan 17, 2026
tilo-14
force-pushed
the
feat/native-program-examples
branch
from
dabdc4e to
2fffdbf
Compare
tilo-14
force-pushed
the
feat/native-program-examples
branch
from
9d21014 to
59f9ff7
Compare
tilo-14
commented
Jan 23, 2026
tilo-14
commented
Jan 23, 2026
- Format account destructuring across multiple lines for readability - Remove program-keypair.json (should not be committed) - Remove AI index line from README - Use authority_seeds! macro consistently in create_token_account.rs and create_mint.rs
tilo-14
commented
Jan 23, 2026
tilo-14
commented
Jan 23, 2026
tilo-14
commented
Jan 23, 2026
tilo-14
commented
Jan 23, 2026
added 4 commits
January 23, 2026 12:50
Replace manual proof fields (proof_a, proof_b, proof_c, address_merkle_tree_root_index) with CreateAccountsProof struct from light-compressible crate for cleaner serialization.
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
- Add transfer_checked CPI instruction with decimal validation - Add mint_to_checked CPI instruction with decimal validation - Update Cargo.toml to use public crates from crates.io - Refactor tests to use COMPRESSIBLE_CONFIG_V1 constant - Update README with new instructions
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
…lify test comments
| [workspace.dependencies] | ||
| # Pin to match light-protocol Cargo.lock | ||
| constant_time_eq = "=0.3.1" | ||
| blake3 = "=1.5.5" |
Contributor
There was a problem hiding this comment.
Suggested change
| blake3 = "=1.5.5" | |
| blake3 = "=1.8.2" |
|
|
||
| - **create-mint** - Create a light-token mint via CPI | ||
| - [Instruction](program/src/instructions/create_mint.rs) | [Test](program/tests/create_mint.rs) | ||
| - **create-ata** - Create an associated light-token account via CPI |
Contributor
There was a problem hiding this comment.
Suggested change
| - **create-ata** - Create an associated light-token account via CPI | |
| - **create-associated** - Create an associated light-token account via CPI |
| - **thaw** - Thaw token account via CPI | ||
| - [Instruction](program/src/instructions/thaw.rs) | [Test](program/tests/thaw.rs) | ||
| - **close** - Close token account via CPI | ||
| - [Instruction](program/src/instructions/close.rs) | [Test](program/tests/close.rs) |
Comment on lines
+16
to
+17
| light-token-interface = "0.3.0" | ||
| light-compressible = "0.4.0" |
Contributor
There was a problem hiding this comment.
we shouldn't depend on these crates directly light-token should reexport everything. If something is not reexported we should add it.
Member
Author
There was a problem hiding this comment.
light-sdk
light-token
light-client
light program test
Member
Author
There was a problem hiding this comment.
light token interface should be reexported out of light token interface
| return Err(ProgramError::NotEnoughAccountKeys); | ||
| }; | ||
|
|
||
| // Close token account. Must be empty (balance = 0) |
Contributor
There was a problem hiding this comment.
Suggested change
| // Close token account. Must be empty (balance = 0) | |
| // Close token account. Must be empty (balance == 0) |
| } | ||
|
|
||
| #[derive(BorshDeserialize)] | ||
| struct CreateMintSignedData { |
Contributor
There was a problem hiding this comment.
can't we use the same instruction data for both?
|
|
||
| #[tokio::test(flavor = "multi_thread")] | ||
| async fn approve_cpi() { | ||
| // Setup: create mint and ATA with tokens |
Contributor
There was a problem hiding this comment.
atas -> associated_token_accounts pls
|
|
||
| // Build instruction data | ||
| #[derive(BorshSerialize)] | ||
| struct CreateMintDataTest { |
Contributor
There was a problem hiding this comment.
why dont we import the program defined struct?
|
|
||
| // Build instruction data | ||
| #[derive(BorshSerialize)] | ||
| struct CreateMintSignedDataTest { |
Contributor
There was a problem hiding this comment.
pls use the program defined struct
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Test plan
cd native/program && cargo test-sbfcd native && pnpm test