Skip to content

chore(client): Re-export relevant Solana crates from anchor-client#4213

Closed
jamie-osec wants to merge 1 commit intosolana-foundation:masterfrom
jamie-osec:client-fixes
Closed

chore(client): Re-export relevant Solana crates from anchor-client#4213
jamie-osec wants to merge 1 commit intosolana-foundation:masterfrom
jamie-osec:client-fixes

Conversation

@jamie-osec
Copy link
Copy Markdown
Collaborator

  • Re-export crates relevant to basic clients from anchor_client (and update examples to match)
  • Update client doctest to work

@vercel
Copy link
Copy Markdown

vercel bot commented Feb 2, 2026

@jamie-osec is attempting to deploy a commit to the Solana Foundation Team on Vercel.

A member of the Team first needs to authorize it.

@nutafrost nutafrost moved this to Security Review Required in Anchor 1.0 Feb 2, 2026
@jamie-osec jamie-osec changed the title chore(client): Update doctest and API for working examples chore(client): Re-export relevant Solana crates from anchor-client Feb 2, 2026
@jamie-osec
Copy link
Copy Markdown
Collaborator Author

cc @acheroncrypto

@acheroncrypto acheroncrypto added the dependencies Pull requests that update a dependency file label Feb 8, 2026
Comment on lines +115 to +119
pub use solana_commitment_config;
pub use solana_keypair;
pub use solana_pubkey;
pub use solana_signer;
pub use solana_system_interface;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I had already made a PR (#4211) that exports the types used by our public API. Whether we should export crates that the client doesn't actually use (but people might find convenient) is another question, and the answer is not immediately clear to me when we consider our new approach of minimizing bulk exports.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I'm inclined to say it makes sense to do this, at least for the bare minimum types which are necessary to create a client (especially when these crates are already in anchor-clients dep graph, unlike in the case of re-exporting the entire solana-sdk)

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Closing in favour of #4211

@jamie-osec jamie-osec closed this Feb 13, 2026
@jamie-osec jamie-osec reopened this Feb 13, 2026
@github-project-automation github-project-automation bot moved this from Security Review Required to Done in Anchor 1.0 Feb 13, 2026
@jamie-osec jamie-osec moved this from Done to Security Review Done in Anchor 1.0 Feb 13, 2026
@jamie-osec jamie-osec closed this Feb 13, 2026
@github-project-automation github-project-automation bot moved this from Security Review Done to Done in Anchor 1.0 Feb 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

client dependencies Pull requests that update a dependency file

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants