Route bridge flows through QuotedCalls#2
Open
yorhodes wants to merge 9 commits intohyperlane-xyz:mainfrom
Open
Route bridge flows through QuotedCalls#2yorhodes wants to merge 9 commits intohyperlane-xyz:mainfrom
yorhodes wants to merge 9 commits intohyperlane-xyz:mainfrom
Conversation
* chore: update hl dependency * chore: remove foundry hl dependency, fix tests * refactor: itokenbridge => ixvelotokenbridge * chore: bump to node v22 * test: use icarouters in prod
* chore: update foundry version * style: fix lint
- Resolve merge conflicts favoring upstream values - Upgrade @hyperlane-xyz/core from 8.1.1 to 11.3.1 - Add @openzeppelin/contracts-v4 and contracts-upgradeable-v4 as aliased deps (workaround for core not declaring OZ as dependency) - Update BridgeRouter to use v11 transferRemote API (3 args) - Remove unused StandardHookMetadata import - Use ITokenFee for quoteTransferRemote (defined on parent interface) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Replace HypXERC20/HypERC20Collateral concrete imports with ITokenBridge - Remove redundant token() validation (bridge's transferRemote reverts on mismatch) - Remove unused transferRemote from IXVeloTokenBridge interface - Restore BaseForkFixture.t.sol to upstream/main - Fix stack-too-deep in Dispatcher by scoping bridge token variables - Update tests to expect generic reverts for wrong-token cases Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Make 12 RouterImmutables public→internal (removes external getter dispatch entries) and merge duplicate bridge execution functions into single executeHypBridge. Runtime bytecode drops from 24,641 to 23,759 bytes, leaving 817 bytes of margin under the 24,576 byte limit. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
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
4 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.
Summary
QUOTED_CALLSrouter entrypointQuotedCallsTesting
forge test --offline --match-path test/foundry-tests/bridge/QuoteExactInput.t.solRelated