hyperlane-monorepo

Commit Graph

Author	SHA1	Message	Date
Yorke Rhodes	92c86cca66	fix: forward value from ICA router to proxies (#4395 ) ### Description Value sent to the hyperlane message recipient (InterchainAccountRouter) would not be forwarded to the proxy account for use in calls with value. Only value sent to the proxy account was supported. ### Related Issues - Fixes https://github.com/chainlight-io/2024-08-hyperlane/issues/4 ### Backward compatibility No, ICA routers would need upgrade/redeploy ### Testing Unit Tests	2 months ago
Nam Chu Hoai	0779a23db6	refactor: Rename InterchainAccountRouter tests for easier reuse (#4336 ) ### Description Minor refactor of test	3 months ago
Yorke Rhodes	6620fe6367	fix: eliminate branching from `TokenRouter` (#3878 ) ### Description Fixes bug in `TokenRouter` which dispatches messages to token recipient rather than address in `Router` table when hook overrides are provided. ### Drive-by changes Simplifies inheritance tree of `TokenRouter` > `GasRouter` > `Router` > `MailboxClient` ### Backward compatibility Yes ### Testing Unit Tests	5 months ago
Kunal Arora	769f57d283	feat:ICA support in SDK (#3483 ) ### Description - Adding 'deployAccount` and `getCallRemote` helpers in InterchainAccount.ts for ease of use for the governor - Adding resolveAccountOwner/resolveInterchainAccountAsOwner wherever it's ambiguous if the owner is address (status quo) or AccountConfig (from this PR) - Added icaApp to HyperlaneDeployer for easier access to ICA ownership for any deployer - resolveAccountOwner vs resolveInterchainAccountAsOwner: the former is usable outside the deployers like the checker, etc whereas the latter is where we're already using the deployer and have the required localRouter already cached - includes the formatting change for the account verification file - owner needs casting in infra/config because of the type Address \| AccountConfig type (cannot use resolveAccountOwner here which requires multiprovider) - Add quoteGasPayment with gas limit override to the ICA router contract ### Drive-by changes - HyperlaneDeployer.cacheAddressesMap appends addresses two keys deep instead of overridding the whole cache - callData in utils now supports value to be compatible with CallData[] in the ICA Router contract and generally for V3 which allows msg.value on dispatch - `appFromAddressesMapHelper` doesn't intersect multiprovider with contractMap chains (this causes issues if the contracts map doesn't include chains we may want to read something like domainId from like ismfactory or ICADeployer) - HyperlaneHookDeployer is now using the owner as owner vs it was earlier using the deployer earlier for unknown reason - ### Related issues - fixes https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/3505 ### Backward compatibility No (call value) ### Testing Unit testing/e2e testing through governor --------- Signed-off-by: Paul Balaji <paul@hyperlane.xyz> Co-authored-by: Paul Balaji <paul@hyperlane.xyz> Co-authored-by: J M Rossy <jm.rossy@gmail.com> Co-authored-by: Trevor Porter <trkporter@ucdavis.edu> Co-authored-by: Daniel Savu <23065004+daniel-savu@users.noreply.github.com>	8 months ago
Kunal Arora	4f0ea4361e	feat: add quoteGasPayment to ICA router (#3460 ) ### Description - for easier offchain use, we can directly call the ICA router to fetch the quote needed for the dispatch. ### Drive-by changes None ### Related issues None ### Backward compatibility Yes ### Testing Unit	8 months ago
Kunal Arora	d224c060d4	fix: send_value fuzz test for ICA (#3469 ) ### Description make sure we assume `value <= balance - gasQuote` in ICA tests ### Drive-by changes none ### Related issues - fixes https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/3468 ### Testing fuzz	8 months ago
Kunal Arora	59e89afc5c	feat: add account -> owner mapping in ICA router (#3447 ) ### Description - added accountOwner mapping to the ICA router for easier tracking of remote owners for the transaction type inference by the AppGovernor ### Drive-by changes none ### Related issues none ### Backward compatibility yes ### Testing unit	8 months ago
Kunal Arora	f9d67aa0b5	feat: update ICA to v3 (#3314 ) ### Description - Update ICA to accept _hookMetadata per call and per batch of calls. - Allow ICA to accept msg.value ### Drive-by changes - Moved IGP from MockMailbox to the InterchainAccountRouterTest contract as it's not needed in the former and doesn't break any tests ### Related issues - Fixes https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/3197 ### Backward compatibility Yes with V3 ### Testing Unit tests with focus on igp as the hook as this will be the one used in production	9 months ago
Yorke Rhodes	b14f997810	Graduate to yarn 4 🧶 (#2899 ) Co-authored-by: J M Rossy <jm.rossy@gmail.com> Fixes https://github.com/hyperlane-xyz/issues/issues/687 Fixes #2228 Fixes https://github.com/hyperlane-xyz/issues/issues/704	1 year ago
Yorke Rhodes	1ecfc46ce2	v3 Router SDK changes (#2752 ) ### Description - Reorganize solidity directory structure for clarity - Updates router apps (token and helloworld) with changes from https://github.com/hyperlane-xyz/hyperlane-monorepo/pull/2749 ### Drive-by changes - Merges the hyperlane-token package to solidity and sdk ### Backward compatibility No (removed token package) ### Testing - Unit Tests - Instrumented coverage	1 year ago
Yorke Rhodes	fcfecdf250	Update router implementations for v3 (#2749 )	1 year ago
Yorke Rhodes	e582357de8	Make Domain Routing ISM enumerable (#2644 )	1 year ago
Yorke Rhodes	aa92fe37db	Fix crosschaincall duplicate warning (#2532 ) ### Description Fixes ``` duplicate definition - NotCrossChainCall() ```	1 year ago
Alex	0ddaf1c5cc	Fix incorrect getRemoteInterchainAccount derivation (#2446 ) ### Description A minor change to how we deploy InterchainAccountRouter's and a fix for remote ICA address derivation. The fix is outlined clearly in #2383 by Nam but essentially by assigning the `implementation` & `bytecode` variables in the constructor of `InterchainAccountRouter`, we are unable to correctly access them in `getRemoteInterchainAccount` if using a proxy. By moving initialisation of these variables to `initialize` we correctly store the values in the proxy storage, not the implementation storage. ### Drive-by changes The `InterchainAccountRouter` should no longer be used unproxied. ### Related issues - Fixes #2383 ### Backward compatibility No, new ICA deploy ### Testing - [x] Unit Tests --------- Co-authored-by: Yorke Rhodes <yorke@hyperlane.xyz>	1 year ago
Asa Oines	f8d645e580	Add RoutingIsm (#1985 ) ### Description This PR adds `IRoutingIsm` and three implementations: - AbstractRoutingIsm, which folks building RoutingIsms can inherit from - DomainRoutingIsm, which routes based on message origin - InterchainAccountIsm, which routes based on the ISM in the InterchainAccountMessage ### Drive-by changes - Some related testing changes ### Related issues - Fixes #1521 ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None ### Testing _What kind of testing have these changes undergone?_ Unit Tests	2 years ago
Asa Oines	f8b04029ac	Fix flaky solidity test (#2013 ) ### Description It's failing during certain fuzzer conditions b/c the assertion is made before the assumption.	2 years ago
Asa Oines	f5a74006ec	Add non-struct API for IQS and ICA middlewares (#1989 ) ### Description This PR adds functions to the ICA and IQS middlewares that do not take complex types. These simpler interfaces lower the barrier to entry for users, especially for those using tooling that does not support structs well (e.g. etherscan, tally) ### Drive-by changes - Rename destinationDomain -> destination - Remove messageId from InterchainCallDispatched, rename to RemoteCallDispatched ### Related issues None ### Backward compatibility _Are these changes backward compatible?_ Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None ### Testing _What kind of testing have these changes undergone?_ Unit Tests	2 years ago
Asa Oines	7f5d50ee9c	Interoperable Interchain Accounts (#1832 ) ### Description This PR introduces a version of Interchain Accounts that is fully interoperable with any compatible contract deployed on a remote chain. Rather than follow the previous `Router` pattern in which destination contracts were hard coded by the deployer, this version allows the user to optionally specify which destination contract should receive the interchain account message. In order to make this secure, interchain accounts are additionally name spaced by ISM and origin router. Note that this is not feature-complete until paired with a RoutingISM that switches based on the ISM included in the message ### Drive-by changes - Creates the `IRouter` and `IHyperlaneConnectionClient` interfaces - Moves middleware interfaces into their own directory - Moves middleware libs into their own directory - Changes `InterchainCallMessage` -> `InterchainQueryMessage` now that it's unused by anything else ### Related issues - Fixes #1747 - Fixes #1969 ### Backward compatibility _Are these changes backward compatible?_ No _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None ### Testing _What kind of testing have these changes undergone?_ Unit Tests	2 years ago
Yorke Rhodes	4ac6290886	Use bytecode deployer for ICA auth (#1914 ) ### Description - Replaces `OwnableMulticall` (`owner` in storage) with `DeployerMulticall` (`deployer` in bytecode) for optimizing ICA deployment costs ### Backward compatibility No, bytecode changes implies new ICA addresses ### Testing Unit Tests	2 years ago
Yorke Rhodes	655bee2da7	Allow value in ICAs (#1640 ) * ICA router changes * Add ICA tests with value * Use InterchainCallMessage lib * Simplify interfaces of middlewares * Avoid using for to improve library coverage * Tolerate 2% gas snapshot deviations * Address pr comments	2 years ago
Trevor Porter	aaafdee480	Update InterchainGasCalculator with on chain fee quoting (#1645 ) * IGP charges 1 wei * Don't pay for gas in LiquidityLayerRouter * Add GasOverheadIgp * Remove compiler silencing * Shift around some fns * nit var name * nit * Configure gas overhead IGP (#1588) * Instrument multisigIsm.verify gas costs * Add deployment tooling for IGP gas overhead * Include mailbox.process and max body in overhead * Restore multisig tests * GasOverheadIgp -> OverheadIgp, fix infra build * Prettier, change some infra / deploy tooling * run yarn hyperlane * yarn gas, update test contracts * rm OwnableUpgradeable for Overhead ISM * Move gas instrumentation into its own describe * back to TestRecipient__factory for normal verify tests * deploy test * BaseInterchainGasPaymaster -> InterchainGasPaymaster * trying to deploy * Deploying with deployerOwnedProxyAdmin * Deployed to testnet3 * Cleaning up * cleaning up * clean * everything * Clean up, remove more coingecko stuff * Use actual quoted value in kathy * Fix yarn lint-ts * Fix flaky test * yarn gas * Add back gas payment estimates * fix test * reset src/index.ts * revert infra/package.json * clean up verification.json * Update yarn.lock * Fix build * update kathy, some nits * Rebase had some incorrect code; fixed that * ran yarn hyperlane * update test * reset test_config.json Co-authored-by: Yorke Rhodes <yorke@hyperlane.xyz>	2 years ago
Yorke Rhodes	f21da232de	Fix bubbling up revert reason (#1597 ) * Fix bubbling up revert reason * Add Call lib * Add coverage for revert reasons to middlewares * Address pr comments * Update gas snapshot	2 years ago
Yorke Rhodes	1651c78ead	Deploy v2 middlewares (#1540 ) * Update contracts with initialize owner changes * Sdk updates * Deploy ICAs and IQS to testnet3 * Deploy ICAs and IQS to mainnet2 * Update artifacts with names * Fix celo IGP * Redeploy middlewares to testnet3 * Fix alfajores verification * Redeploy middlewares to mainnet2 * Use helloworld main * Address pr comments * Simplify expect violations * Update gas snapshot	2 years ago
Yorke Rhodes	68dc546ee0	Middleware cleanup (#1501 ) * Improve docs and comments * Cleanup interchain queries tests * Update gas snapshot * Improve middleware test coverage * Add forge coverage report * Update gas snapshot * Add coverage report to PR comments * Use latest version of coverage report action	2 years ago
Yorke Rhodes	ffe107387b	Use minimal proxies in ICA router (#1440 ) * Use minimal proxies in ICA router * Update optimized gas snapshots * Add instructions for gas snapshot update * Match solc config of hardhat and foundry * Pull out constants and add useful comments * Optimize with immutables	2 years ago
Yorke Rhodes	c25cd842b2	Run forge tests in CI (#1355 ) * Move foundry workflow so it is picked up by github * Require ISM in router config Co-authored-by: Asa Oines <asaoines@gmail.com>	2 years ago
Nam Chu Hoai	bb01859bad	TokenBridgeRouter and CircleBridgeAdapter (#1212 ) * Compiles * nit * Add initializer * Add some setters * Got it working * Don't track nonces in contract state * Nit * Starting tests * Tests * Use actual token behavior in mocktoken * Complete MockTokenBridgeAdapter * Fix up * fix * Fix test * Remove relayer * PR review * PR review Co-authored-by: Trevor Porter <trkporter@ucdavis.edu>	2 years ago
Nam Chu Hoai	f829c6f350	Add foundry to core (#1171 )	2 years ago

28 Commits (32d0a67c2130f5f893bd824ccfb05007418cf191)