feat(web3-compat): implement full Connection class replacement #140
Conversation
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
Transform @solana/web3-compat into a complete drop-in replacement for @solana/web3.js Connection class, powered by @solana/client and @solana/kit. Implementation includes: - 73 total methods (59 async RPC + 8 subscription + 6 remove listener) - Phase 1: Core high-frequency methods (getMultipleAccountsInfo, getTokenAccountsByOwner, sendTransaction, getTransaction, etc.) - Phase 2: Block and transaction history (getBlock, getBlockTime, getBlockHeight, isBlockhashValid, getFeeForMessage, etc.) - Phase 3: Parsed methods with jsonParsed encoding (getParsedAccountInfo, getParsedTransaction, getParsedBlock, etc.) - Phase 4-7: Slot, epoch, cluster, stake, inflation, and misc methods - WebSocket subscriptions (onAccountChange, onProgramAccountChange, onSignature, onSlotChange, onRootChange, onLogs, etc.) All methods follow strict web3.js API compatibility with automatic bigint-to-number conversion for seamless migration. Tests: 253 passing (12 new tests for Phase 3 and WebSocket subscriptions)
- Refactor Connection to use SolanaClient instead of SolanaRpcClient - Add `connection.client` getter for gradual migration to @solana/client - Fix confirmTransaction to accept object strategy and poll for confirmation - Fix getSignatureStatuses (commitment param not supported by RPC) - Revamp README with example-focused documentation - Add devnet integration tests with CI workflow - Remove experimental language from READMEs 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]>
Base58 encoded blockhashes and signatures have variable length (43-44 chars for hashes, 86-88 for signatures). Updated tests to use range checks instead of exact length assertions. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]>
Devnet integration tests are run in a dedicated CI job with TEST_PRIVATE_KEY. E2E tests require Playwright which is not installed in the base test environment. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]>
Contributor
|
tested on a few examples and works well! worth having more eyes for testing complex projects |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
Complete drop-in replacement for
@solana/web3.jsConnection class, powered by@solana/clientand@solana/kit.Key Features
connection.clientgetter provides access toSolanaClientfor gradual adoption of@solana/clientfeaturesAPI Coverage
Migration to @solana/client
Testing
TEST_PRIVATE_KEYsecret required)Changes in This PR
SolanaClientinstead ofSolanaRpcClientconnection.clientgetter for migration pathconfirmTransactionto accept object strategy and poll for confirmationgetSignatureStatusesparameter issue