-
Notifications
You must be signed in to change notification settings - Fork 31
Cranker, CLI: Change the version of restaking, vault clients
#269
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
restaking, vault clientsrestaking, vault clients
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR addresses compatibility issues between the cranker code and mainnet by downgrading the jito-restaking-client and jito-vault-client dependencies to a specific commit that matches the deployed mainnet version. The changes resolve transaction simulation errors caused by instruction mismatches.
- Pinned
jito-restaking-clientandjito-vault-clientto commitbbc5abe64d5bbbeab1d6f33f5a63a97cc3b9638a - Fixed transaction size calculation in vault handler to use proper serialization
- Commented out incompatible secondary admin functionality in CLI
Reviewed Changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| crankers/Cargo.toml | Updated restaking and vault client dependencies to specific git commit |
| cli/Cargo.toml | Updated restaking and vault client dependencies to specific git commit |
| Cargo.toml | Added bincode dependency and cli to workspace members |
| crankers/src/vault_handler.rs | Fixed transaction size calculation and improved batch handling logic |
| cli/src/vault_handler.rs | Commented out incompatible secondary admin functionality and added hardcoded addresses |
| crankers/README.md | Fixed command line formatting with proper backslashes |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 6 out of 7 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Problem
Solution
jito-restaking-clientandjito-vault-clientthat is compatible with current mainnet