Refactor: W3APD-5238 - migrate from @solana/web3.js to @solana/kit#2347
Open
tanguyenvn wants to merge 3 commits intomasterfrom
Open
Refactor: W3APD-5238 - migrate from @solana/web3.js to @solana/kit#2347tanguyenvn wants to merge 3 commits intomasterfrom
tanguyenvn wants to merge 3 commits intomasterfrom
Conversation
- Replace Connection with RPC client from @solana/kit - Update transaction types to use @solana/kit's Transaction - Remove @toruslabs/bs58 in favor of @solana/kit's base58 utilities - Update demo app to use new Solana utilities BREAKING CHANGE: connection property renamed to rpc, transaction types now expect compiled Transaction from @solana/kit
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
tanguyenvn
force-pushed
the
feat/W3APD-5238-solana-kit
branch
from
8c378ae to
4c2564b
Compare
tanguyenvn
changed the title
tanguyenvn
commented
Jan 27, 2026
| } | ||
| }; | ||
|
|
||
| export const signAndSendTransaction = async (provider: IProvider, uiConsole: any) => { |
Contributor
Author
There was a problem hiding this comment.
clean up as these functions are not used
tanguyenvn
commented
Jan 27, 2026
| import { generateSolTransferInstruction, generateVersionedTransaction } from "../utils/solana"; | ||
|
|
||
| // EVM | ||
| export const walletSignPersonalMessage = async ( |
Contributor
Author
There was a problem hiding this comment.
clean up as these functions are not used
Contributor
Author
|
@cursor review |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
Contributor
Author
|
bugbot run |
Member
|
Is there a way to remove all our custom provider wrappers and use the custom signers? |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
BREAKING CHANGE: connection property renamed to rpc, transaction types now expect compiled Transaction from @solana/kit
Motivation and Context
Jira Link:
Description
How has this been tested?
Screenshots (if appropriate):
Types of changes
Checklist:
Note
Migrates Solana stack to
@solana/kit, standardizing RPC access and transaction types across packages and the demo app.@solana/web3.jswith@solana/kit; adds@solana-program/systemfor SOL transfer instructionsSolanaWalletto usecreateSolanaRpc, exposerpc(wasconnection), and require compiledTransactionfrom@solana/kit@solana/kitcodecs (base58/base64) and transaction (de)serialization helpersutils/solanato generate transfer instructions and legacy/versioned transactions; dashboard actions now userpc; simplifiedsolHandlers; removes legacywalletServiceHandlers@web3auth/modalto10.13.2, updates deps/peerDeps to@solana/kitBREAKING:
connection→rpc; Solana APIs now accept@solana/kitTransaction.Written by Cursor Bugbot for commit e8f9e45. This will update automatically on new commits. Configure here.