-
Notifications
You must be signed in to change notification settings - Fork 577
feat(sdks): sealevel fee support svm sdk [IGNORE FOR NOW] #8647
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
Draft
xeno097
wants to merge
34
commits into
main
Choose a base branch
from
xeno/sealevel-fee-support-svm-sdk
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from all commits
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
d24d23a
chore(svm-sdk): [DROPPABLE] inject fee program bytes from xeno/sealev…
xeno097 aca9374
feat(svm-sdk): add fee program PDA derivation functions
xeno097 9b97c67
feat(svm-sdk): add fee SDK types, constants, and signer helpers
xeno097 a9aa7e1
feat(svm-sdk): add fee codec types and Borsh encoding functions
xeno097 43cb049
feat(svm-sdk): add fee account decoders and i64 codec support
xeno097 2e26b77
feat(svm-sdk): add fee instruction builders for Leaf management
xeno097 4cd180c
feat(svm-sdk): add fee account query helpers and type detection
xeno097 ff9f367
feat(svm-sdk): add linear fee reader and writer
xeno097 07b041a
feat(svm-sdk): add linear fee E2E tests
xeno097 5b205ee
feat(svm-sdk): extract leaf fee base and add regressive fee reader/wr…
xeno097 fef07e8
feat(svm-sdk): add shared leaf fee test suite and regressive fee E2E …
xeno097 1a8515a
feat(svm-sdk): add progressive fee reader/writer and E2E tests
xeno097 0410f41
feat(svm-sdk): add AddQuoteSigner and RemoveQuoteSigner instruction b…
xeno097 3f76710
feat(svm-sdk): add offchainQuotedLinear fee reader and writer
xeno097 a5ec941
feat(svm-sdk): add offchainQuotedLinear fee E2E tests
xeno097 d8cc8bd
feat(svm-sdk): add SetRoute, RemoveRoute, and SetWildcardQuoteSigners…
xeno097 088d6b8
feat(svm-sdk): add fetchRouteDomain query helper
xeno097 dd22e00
feat(svm-sdk): add routing fee reader and writer
xeno097 3947b12
feat(svm-sdk): add routing fee E2E tests
xeno097 128c08d
feat(svm-sdk): add SetCrossCollateralRoute and RemoveCrossCollateralR…
xeno097 1817fa1
feat(svm-sdk): add fetchCrossCollateralRoute query helper
xeno097 e67c833
feat(svm-sdk): add cross-collateral routing fee reader/writer and ext…
xeno097 18304a2
feat(svm-sdk): add cross-collateral routing fee E2E tests
xeno097 7e1630f
feat(svm-sdk): add runtime fee instruction builders
xeno097 a728d74
feat(svm-sdk): add SvmFeeArtifactManager
xeno097 ffd4f0c
feat(svm-sdk): wire SvmFeeArtifactManager into protocol provider and …
xeno097 dece6a6
feat(svm-sdk): add fee stress tests to discover on-chain limits
xeno097 0598a74
feat(svm-sdk): add fee stress tests to discover on-chain limits
xeno097 709a33f
fix(svm-sdk): address code review findings on fee SDK
xeno097 c4f3efe
fix(provider-sdk, deploy-sdk, svm-sdk): thread FeeReadContext through…
xeno097 769eab2
chore: add changeset for SVM fee SDK support
xeno097 ed0247b
test(svm-sdk): add test for removing last signer preserving offchainQ…
xeno097 ac0459c
ci: add fee E2E tests to SVM SDK CI matrix
xeno097 22a40cd
fix(svm-sdk): address PR review comments
xeno097 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,20 @@ | ||
| --- | ||
| '@hyperlane-xyz/provider-sdk': major | ||
| '@hyperlane-xyz/deploy-sdk': major | ||
| '@hyperlane-xyz/sealevel-sdk': minor | ||
| '@hyperlane-xyz/aleo-sdk': patch | ||
| '@hyperlane-xyz/cosmos-sdk': patch | ||
| '@hyperlane-xyz/radix-sdk': patch | ||
| '@hyperlane-xyz/starknet-core': patch | ||
| '@hyperlane-xyz/tron-sdk': patch | ||
| --- | ||
|
|
||
| SVM fee program SDK support was added, enabling deployment and management of all 6 fee types (linear, regressive, progressive, offchainQuotedLinear, routing, crossCollateralRouting) on Solana/SVM chains. | ||
|
|
||
| **Breaking change:** `ProtocolProvider.createFeeArtifactManager()` now requires a `FeeReadContext` parameter. This ensures routed fee types (routing, CC routing) receive the domain/router context needed to discover non-enumerable route PDAs. All protocol SDK implementations were updated. | ||
|
|
||
| The SVM fee SDK includes: | ||
| - Readers and writers for all 6 fee types with full create/update support. | ||
| - Instruction builders for all 18 fee program instructions including runtime operations (QuoteFee, SubmitQuote, etc.). | ||
| - PDA derivation, Borsh codecs, and account decoders for the fee program's on-chain state. | ||
| - SvmFeeArtifactManager implementing IRawFeeArtifactManager, wired into SvmProtocolProvider. |
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
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
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
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
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
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
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
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
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
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
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
Binary file not shown.
Oops, something went wrong.
Oops, something went wrong.
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.
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.
🚩 Fee program binary added to program-bytes generator with fallback path mechanism
In
scripts/generate-program-bytes.mjs, the newtokenFeeentry was added at line 28, and lines 36-48 introduce a fallback mechanism that checksLOCAL_SO_DIR(so-binaries/) when the primary path (rust/sealevel/target/deploy/) doesn't have the binary. This is a deviation from the existing pattern where all programs came from the same build directory. The fallback could mask build issues if a stale local binary is used instead of a freshly compiled one. Thecheck-program-bytes-hash.mjsscript should catch staleness via the source hash, but only if it covers the fee program sources.Was this helpful? React with 👍 or 👎 to provide feedback.