-
Notifications
You must be signed in to change notification settings - Fork 299
fix: add support for seievm for tx acceleration #7616
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
Conversation
12644be to
aa1de7a
Compare
abhi-bitgo
left a comment
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.
lgtm
Ranjna-G
left a comment
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.
looks good
|
@claude review this PR |
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 adds support for transaction acceleration on the Sei EVM blockchain by including 'seievm' in the list of EVM-compatible chains that support acceleration intents. The change enables users to accelerate stuck transactions on Sei EVM by submitting replacement transactions with higher gas fees.
Key Changes:
- Added
'seievm'to the EVM family check inmpcUtils.tsto enable acceleration support - Added comprehensive test coverage for seievm acceleration functionality in wallet tests
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| modules/sdk-core/src/bitgo/utils/mpcUtils.ts | Added 'seievm' to the list of EVM chains that support transaction acceleration and nonce filling |
| modules/bitgo/test/v2/unit/wallet.ts | Added two test cases for seievm acceleration: one for prebuildTransaction and one for populateIntent functionality |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
lokesh-bitgo
left a comment
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.
Looks good
45c9c52
aa1de7a to
45c9c52
Compare
45c9c52 to
16be18e
Compare
Ranjna-G
left a comment
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.
lgtm
TICKET: WIN-8040