Skip to content

Conversation

@tilo-14
Copy link
Contributor

@tilo-14 tilo-14 commented Jan 17, 2026

@tilo-14 tilo-14 force-pushed the add-anchor branch 2 times, most recently from 3772405 to 81db9df Compare January 17, 2026 17:55
.await
.unwrap();

// 7. Transfer Light Token → SPL (uses TransferInterface with auto-detection)

Choose a reason for hiding this comment

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

auto-detection is misleading you declare which tokens you are using by defining source and recipient owner program ids.

Copy link

@ananas-block ananas-block Jan 18, 2026

Choose a reason for hiding this comment

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

shouldnt the file be called spl_to_light_tokens.rs or similar?

Choose a reason for hiding this comment

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

dangerous these will easily become outdated

└── close.rs
```

## Instruction discriminators

Choose a reason for hiding this comment

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

misleading

Comment on lines 12 to 15
pub fn burn<'info>(
ctx: Context<'_, '_, '_, 'info, BurnAccounts<'info>>,
amount: u64,
) -> Result<()> {

Choose a reason for hiding this comment

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

Suggested change
pub fn burn<'info>(
ctx: Context<'_, '_, '_, 'info, BurnAccounts<'info>>,
amount: u64,
) -> Result<()> {
pub fn burn(ctx: Context<BurnAccounts>, amount: u64) -> Result<()> {

Choose a reason for hiding this comment

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

same for all other programs


Learn more [about Light-Token here](https://www.zkcompression.com/light-token/welcome).

For your AI to index this repo: [AI Index](CLAUDE.md), [Flow Diagram](flow.mermaid), [State Machine Diagram](states.mermaid)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

rm For your AI to index this repo: AI Index, Flow Diagram, State Machine Diagram

.await
.unwrap();

println!("Burned {} tokens via anchor program", burn_amount);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

should only print tx id. akin to typescript client prints

@@ -0,0 +1 @@
[196,52,241,96,50,167,183,75,251,111,133,107,26,185,35,101,19,119,253,19,35,164,230,219,57,197,157,114,110,187,41,190,170,36,163,213,228,94,249,171,13,74,217,229,218,63,175,39,24,177,247,204,140,120,73,245,204,205,44,199,159,204,122,189] No newline at end of file
Copy link
Contributor Author

Choose a reason for hiding this comment

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

rm

tilo-14 added 2 commits January 22, 2026 01:03
- Remove AI index line from README.md
- Delete keypair.json from create-token-account program
- Simplify test println statements to only output transaction signature

The transaction signature provides all necessary information for
verification and is more useful than verbose descriptive text.
Copy link

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 1 potential issue.

View issue and 4 additional flags in Devin Review.

Open in Devin Review

- Remove all TypeScript test files (tests/*.ts)
- Remove package.json and tsconfig.json
- Remove .claude/settings.json and BURN_ARCHITECTURE.md
- Add .gitignore for node_modules, target, TS files
- Update Anchor.toml test script to use cargo test-sbf

Rust integration tests remain in programs/*/tests/test.rs
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.

3 participants