hyperlane-monorepo

Commit Graph

Author	SHA1	Message	Date
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	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	af88ae66c3	Fix rebase	1 year ago
Yorke Rhodes	e582357de8	Make Domain Routing ISM enumerable (#2644 )	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	63558af85d	Use interfaces in agents (#2022 ) ### Description Move agents to use contract interfaces artifacts rather than the artifacts themselves. This ensures we don't anything that we can't guarantee is in every implementation of that interface. Particularly relevant for IGPs and ISMs. ### Drive-by changes - Moves `solidity/interfaces` to `solidity/contracts/interfaces` so that their artifacts can be used in `update_abis.sh` ### Related issues - #1757 ### 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?_ None	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
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
Nam Chu Hoai	6c40b382b8	Make ISM optional in RouterConfig (#1457 )	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
Asa Oines	e6a3373979	Fix build	2 years ago
Nam Chu Hoai	551af4c12e	Support ICA/IQS functions without Call struct (#1307 ) * Support ICA/IQS functions without Call struct * Fix	2 years ago
Asa Oines	4a82f13dac	Adopt V2 throughout the solidity/typescript repo (#1196 )	2 years ago
Nam Chu Hoai	0c9979e80a	Quickstart fixes (#1244 ) * Add IInterchainAccountRouter and MockInterchainAccountRouter * Have InterchainQueryRouter implement IInterchainQueryRouter and return leaf indices * Make ChainNameToDomainID more forgiving * Make hyperlaneCoreAddresses more forgiving * PR review	2 years ago
rabib	e31fc0d6d4	adding message leaf index to output of InterchainAccountRouter.dispatch(...) (#1236 )	2 years ago
Nam Chu Hoai	16374dc2ef	Merge ICA package (#1170 ) * Merge ICA package * Remove reference in dockerfile * Lint * Rename to hardhat tests	2 years ago
Nam Chu Hoai	6bb358a0e3	Merge core and app packages (#1169 ) * Merge core and app packages * Fix test * Fix test * Dockerfile * Fix test	2 years ago
Yorke Rhodes	a2f4810027	Add `InterchainQueryRouter` implementation (#1137 ) * Add InterchainQueryRouter implementation * Add e2e query and callback test * Add comments to query contracts * Add deploy script to infra * Add salt config * Deploy IQS to mainnet Co-authored-by: nambrot <nambrot@googlemail.com>	2 years ago
Nam Chu Hoai	68c394a2a8	Deploy consistent Create2Factory, TestRecipient and InterchainAccountRouters (#1123 ) * Deploy consistent Create2Factory, TestRecipient and InterchainAccountRouters * .gitignore * PR Review	2 years ago
J M Rossy	0d7639895c	Hyperlane packages and typescript rebrand (#1077 ) - Rename Typescript packages hyperlane-xyz org - Rename Abacus references in SDK export names - Bump packages versions to 0.5.0-beta0 Breaking change due to SDK export name changes	2 years ago
Yorke Rhodes	53593ef6f3	Add interchain accounts package (#1057 ) * Push initial ICA package * Update README * Remove generated files * Make _salt pure * Rename package to interchain accounts	2 years ago

29 Commits (12840964b98910f00c909b7e943b3904dd6476f9)