feat: implement Solana chain module (#22)#23
Conversation
Adds full Solana support behind the solana feature flag: - New src/chains/solana.rs with SolanaConnector and SolanaWallet - Support for SOL and SPL token transfers - Updated Chain enum to include Solana and SolanaDevnet - Updated README.md and examples/multi_chain.rs - Added integration tests in tests/solana_integration.rs Closes kcolbchain#22
|
Welcome to kcolbchain, @syutoutousai — glad you're here. 🌱 Here's what happens from this PR:
While you wait:
What happens after your first merge
Thanks for writing the code. We're building this to last. |
|
🤖 Audit verdict: Implementation code is benign and properly gated behind feature flags, but integration tests contain compilation errors that prevent CI from passing. Risks flagged:
Audited by the kcolbchain PR pipeline. See pipeline docs. |
|
Your PR was selected as the winner for the Solana chain module — nice work on the SolanaConnector, wallet support, and SPL token integration! One blocker before merge: there's an async/sync mismatch in the integration tests — Thank you! |
Summary
Implements the Solana chain module as requested in issue #22, adding support for Solana mainnet and devnet.
Changes
solanafeature flag to minimize dependency bloat. Enabling it addssolana-sdk,solana-client, andspl-token.src/chains/solana.rsprovidingSolanaConnectorfor RPC operations andSolanaWalletfor keypair management.SolanaandSolanaDevnetvariants to theChainenum with appropriate metadata (RPCs, symbols, explorers).examples/multi_chain.rsto demonstrate Solana usage and updatedREADME.md.tests/solana_integration.rsfor verifying core functionality against a local validator.Closes #22