Skip to content

chore: Use solana-rpc-client crate which is minimal#94

Open
Arrowana wants to merge 3 commits intocodama-idl:mainfrom
Arrowana:chore/solana-rpc-client-minimal-dep
Open

chore: Use solana-rpc-client crate which is minimal#94
Arrowana wants to merge 3 commits intocodama-idl:mainfrom
Arrowana:chore/solana-rpc-client-minimal-dep

Conversation

@Arrowana
Copy link
Copy Markdown

@Arrowana Arrowana commented Mar 31, 2026

solana-client has an extensive amount of bloat that is absolutely not necessary to use a rpc client, use the underlying solana-rpc-client

It brought a bajillion unnecessary dependencies

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Mar 31, 2026

🦋 Changeset detected

Latest commit: d0b3f6b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@codama/renderers-rust Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Copy Markdown
Member

@lorisleiva lorisleiva left a comment

Choose a reason for hiding this comment

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

Nice, thanks for that! Just pinging @febo to make sure that's okay with him too. 🙏

Comment thread src/utils/cargoToml.ts Outdated
'solana-account-info': '~3.1',
'solana-address': { features: ['borsh', 'copy', 'curve25519', 'decode'], version: '~2.2' },
'solana-client': { optional: true, version: '^3.0' },
'solana-rpc-client': { optional: true, version: '^3.0' },
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think this just needs reordering to make Eslint happy.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

yes, done

@lorisleiva lorisleiva requested a review from febo March 31, 2026 15:30
Copy link
Copy Markdown

@febo febo left a comment

Choose a reason for hiding this comment

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

Looks great – thanks!

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