hyperlane-monorepo

Commit Graph

Author	SHA1	Message	Date
Mattie Conover	1722f4ee7e	Better Agent Configuration Parsing (#2070 ) ### Description This is a significant change to how we parse user configs. The main goal of this PR is to give a clear path and env key to any errors that are encountered when parsing configuration. In order to accomplish this, I have crated a new trait `FromRawConf` which defines a set of steps to validate and convert individual fields. Also for every major config struct, there is now a `Raw` variant of it which is used by the `config` library to reduce errors that occur during deserialization itself. `Raw` types should always use the most basic form of a value, such as `String`, `StrOrInt`, `bool`, and ideally optional in all cases. Missing values should be handled during the raw conversion and not by `config`. I also needed to make changes to a number of types stored in the parsed config types to force validation forward to the parsing step instead of doing it when we read the config value. This also required me to create a filter to prevent trying to validate certain configs that we would not ever need to load. These changes can also be built on later to support something other than `config` if we choose to, or add support for merging configs from multiple sources since everything is optional. ### Drive-by changes - Default to `http` for connection type - Default to no gas payment enforcement if not specified instead of failing - `ChainSetup` -> `ChainConf` - `GasPaymentEnforcementConfig` -> `GasPaymentEnforcementConf` - Made ethereum connection configuration more forgiving - Moved hyperlane base settings from `mod.rs` to `base.rs` - Moved config chain tests in hyperlane core to `tests` dir to fix a cyclical import problem - Extension traits to help with config in hyperlane core - Moved `StrOrInt` to new hyperlane core `config` module - Support for parsing `U256` from `StrOrInt` - Removed `HexString` type which is now redundant - Updated base settings to use hyperlane domain - Use `heyKey` as signer type if `type` is not specified and `key` is - Moved hyperlane ethereum chain config to a new module ### Related issues - Fixes #2033 - Fixes #2012 ### Backward compatibility _Are these changes backward compatible?_ Yes - This should take in configs in the same shape but be more forgiving in a few places _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?_ Manual Unit Test	2 years ago
Trevor Porter	e226a5a276	Fetch with timeout in Liquidity Layer app (#2071 ) ### Description Noticed that the portal relayer had hung on some async task, causing portal messages to stop being relayed. I'm not confident that fetches were to blame, but based off the logs it seems most likely that one of the fetches went wrong and an error was never thrown. So this adds a 10 second timeout for each fetch ### Drive-by changes n/a ### Related issues n/a ### 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?_ Ran locally, everything still worked	2 years ago
Asa Oines	267cf80a32	Publish 1.3.2 (#2086 ) ### Description Publishes 1.3.2 for utils/core/sdk, updates helloworld and token to their respective main branches and then moves them to 1.3.2 ### Drive-by changes None	2 years ago
Asa Oines	1df5ac7cfe	Fix logging when explorer URL not present (#2082 ) ### Description Logs the signer address when explorer url is not present	2 years ago
Nam Chu Hoai	d4a6dc0e7b	Fix address url in contractverifier (#2083 ) ### Description Fixes the path	2 years ago
Asa Oines	32407592e6	Redeploy Interchain Accounts (#2081 ) ### Description This PR contains the deployment artifacts from a fresh deploy of PI Interchain Account middleware to testnet and mainnet. The deployment hit some strange issues with polygon where it was complaining about underpriced txs. I was able to fix by submitting the failed transactions manually with cast. ### Drive-by changes - Changes the return type of HyperlaneDeployer.readCache, which was accidentally omitted from #2078 - Logs completion of HyperlaneConnectionClient initialization in deployer - Fixes proxy admin fetching in `changeAdmin` - Uses transaction overrides in `upgradeAndInitialize` - Allows for caching of `InterchainAccountRouter` if it has been deployed completely but adminship hasn't been changed - Submit initialization and ownership transfer txs sequentially in HyperlaneRouterDeployer, which hopefully fixes an issue in which logs were not showing up - Return early in governor if no violations ### Related issues - Fixes #2107 ### Backward compatibility _Are these changes backward compatible?_ No, the ICA docs need to be updated _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?_ Checker is passing	2 years ago
Asa Oines	179921b2bb	Support restoration of partial deployments in deployer (#2078 ) ### Description This PR allows the deployer to be instantiated with addresses even if not all contracts were deployed to a chain. This allows e.g. reuse of the ProxyAdmin contract when deploying multiple modules ### Drive-by changes - Minor improvements to objMap and objMapEntries types, so as to cast less in callers	2 years ago
Asa Oines	5de2bfdd0e	Programmatic ownership transfer of ownables in deployer (#2077 ) ### Description Moves the Deployer classes to detect ownable contracts programmatically when doing ownership transfer	2 years ago
Asa Oines	a4785f6592	Fix SDK attachContractsMap bug (#2076 ) ### Description attachContractsMap should filter out chains for which a complete set of addresses is not present ### Drive-by changes None	2 years ago
Nam Chu Hoai	864689a8a5	Update gnosis validators (#2060 ) ### Description Updates the validator set in the SDK for gnosis Co-authored-by: Asa Oines <asaoines@gmail.com>	2 years ago
Yorke Rhodes	25dfcbaed7	Bump to 1.3.1 (#2072 )	2 years ago
Yorke Rhodes	7250c1c14d	Move deployer config from constructor to deploy time (#2065 ) ### Description - Move config from constructor to deploy - Fix sdk tests - Update helloworld and token to reflect changes ### Drive-by Changes - Adds per chain deployment timeouts - Removes env subset deployer as SDK tests already cover ### Related issues - Fixes #[issue number here] ### Testing Unit Tests	2 years ago
Asa Oines	96e03b1681	Remove Create2Factory and associated deployer code (#2064 ) ### Description As discussed offline, the complexity of maintaining consistent addresses across chains is no longer worth it, especially as consistent addresses cannot be maintained across PI chains. ### Drive-by changes - Move governors into infra/src/govern - Take initialization args list as opposed to abi encoded args in deployer - Do not initialize IGP in the constructor since we need to initialize the proxy ### Related issues - Fixes #2040 ### 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
Trevor Porter	8fb6384e34	Catch more errors in Portal relayer (#2049 ) ### Description Sometimes get annoying stuff like: ``` {"chain":"goerli","level":"info","message":"Processing chain"} TypeError: response.result.map is not a function at LiquidityLayerApp.<anonymous> (/hyperlane-monorepo/typescript/sdk/src/middleware/liquidity-layer/LiquidityLayerApp.ts:93:28) at Generator.next (<anonymous>) at fulfilled (/hyperlane-monorepo/typescript/sdk/dist/middleware/liquidity-layer/LiquidityLayerApp.js:5:58) at runMicrotasks (<anonymous>) at processTicksAndRejections (node:internal/process/task_queues:96:5) ``` ### Drive-by changes n/a ### Related issues n/a ### 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?_ ran locally	2 years ago
J M Rossy	dbaac351d3	Make getDispatchedMessages static (#2055 ) - Make getDispatchedMessages available statically - Clarify types and comments for some message utils	2 years ago
Asa Oines	5a6a753de6	Fix fork tests 2 (#2063 ) - Fix an issue where `pick` inserts a key into an object. - Skip fork testing ICAs/IQS due to the IGP mismatch error	2 years ago
Asa Oines	e87095dc18	Release 1.3.0 (#2056 ) ### Description This PR releases v1.3.0 of utils, core, and SDK.	2 years ago
Asa Oines	aea77a10f7	Do not check StorageGasOracle's owner (#2052 ) ### Description We recently started checking the owner for every ownable contract. This broke CI since StorageGasOracle's owner is not expected to match the configured config owner. ### Drive-by changes None	2 years ago
Asa Oines	031f0a9692	Remove ProxiedContract and infer types (#2042 ) ### Description This PR removes the ProxiedContract type and moves to inferring class types from the factory type. ### Drive-by changes None ### Related issues - Fixes #2037 ### 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?_ Yes ### Testing _What kind of testing have these changes undergone?_ Unit Tests --------- Co-authored-by: Yorke Rhodes <yorke@hyperlane.xyz>	2 years ago
Asa Oines	17b400806e	Add checkers for middlewares (#2030 ) ### Description This PR adds apps and checkers for the ICA and IQS middlewares. ### Drive-by changes - Remove redundant lines from writeJSON - Automatically detect ownables in checkers - Automatically detect proxied contracts in checkers - Deploy ICA and IQS to testnet3 ### 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 --------- Co-authored-by: Yorke Rhodes <yorke@hyperlane.xyz>	2 years ago
Nam Chu Hoai	117aa93b61	Portal (#1970 ) ### Description Log when wrapped token has not been created and catch exceptions --------- Co-authored-by: Trevor Porter <trkporter@ucdavis.edu> Co-authored-by: J M Rossy <jm.rossy@gmail.com>	2 years ago
Yorke Rhodes	0b60a32e9c	Merge deploy scripting (#2015 ) ### Description - Merges all flavors of `infra/XXX/deploy.ts` to `infra/deploy.ts --module XXX` - Use `deployWithArtifacts` for consistent artifact management and dedupes ### Drive-by changes Adjusts ICA deployer and adds fork tests ### Backward compatibility No ### Testing Fork tests	2 years ago
Asa Oines	97d17570cc	Refactor IGP related code out of HyperlaneCore (#1907 ) ### Description A number of changes intended to make the SDK more easily consumed by hyperlane-deploy - Splits the IGP related code out of HyperlaneCore* into HyperlaneIgp* - Moves agent config building from infra to SDK so that it can be accessed by hyperlane-deploy - Modifies CoreConfig to specify a ChainMap<MultisigIsmConfig> (i.e. specify everything that you need to know to deploy to a chain in that chain's CoreConfig) - Removes core.extendWithConnectionClientConfig - Moves common logic into HyperlaneAppGovernor to be shared between HyperlaneCoreGovernor and HyperlaneIgpGovernor - Adds TestRecipient contract addresses to the SDK addresses for consumption by hyperlane-deploy - Allow buildContracts to build when address keys are a superset of factory keys (via filterContracts). This is useful as it allows us to just throw all the addresses at a HyperlaneApp constructor without needing to remove the ones that aren't relevant - Removes InterchainGasCalculator, the logic we want to keep now lives in HyperlaneIgp - Allows chains to be added to a MultiProvider ### Drive-by changes - Merges the infra govern script into the check script (to dedupe) - Some minor renaming where "Core" was being used liberally - Default to using whatever is configured in `owners.ts` in the router configs for infra (for ICA, IQS, HelloWorld, and LL deployment) ### Related issues - Fixes #[issue number here] ### Backward compatibility _Are these changes backward compatible?_ No, existing consumers of the SDK may need to adjust. _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?_ Manual / Unit Tests	2 years ago
Asa Oines	e8d90775f7	Add Aggregation ISM (#1737 ) ### Description This PR introduces the Aggregation ISM, which requires interchain messages to be verified by an m-of-n ISM set. This PR also introduces a new MultisigISM, which does not make use of the "commitment" optimization in its metadata. The previous MultisigISM has been moved to LegacyMultisigISM. Both AggregationISM and MultisigISM have abstract contracts that require `valuesAndThreshold` implementations. This PR implements versions with static m-of-n sets, using the EIP-3448 MetaProxy standard. These implementations are intended to be combined with a future RoutingISM implementation to replace LegacyMultisigISM. ### Drive-by changes - genhtml in solidity code coverage ### Related issues - Fixes #1522 ### Backward compatibility _Are these changes backward compatible?_ Yes ### Testing _What kind of testing have these changes undergone? Unit Tests	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
Nam Chu Hoai	c854784be6	Deploy 1.2.3 (#1971 ) ### Description Deploy packages at 1.2.3	2 years ago
Nam Chu Hoai	8f791a5eb6	Fix explorerAddressURL when no signer (#1962 ) ### Description Currently, if the MP doesn't have a signer, the function will always return null	2 years ago
Yorke Rhodes	5b021c6242	Add fork testing to CI (#1931 ) ### Description - Add fork tests to CI - [x] check - [x] deploy - [x] govern - [x] check (after govern) ### Related issues - Fixes https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/1653 ### Backward compatibility Yes ### Testing CI --------- Co-authored-by: nambrot <nambrot@googlemail.com>	2 years ago
Nam Chu Hoai	80005f01e9	Add Sepolia (#1930 ) ### Description This PR deploys Hyperlane to Sepolia. Updates packages to 1.2.2 to include sepolia ### Drive-by changes - Fixes ContractVerifier - Adds multiple bytecode hashes that are valid in HyperlaneCoreChecker ### Related issues - Fixes #1878 ### Backward compatibility Yes _Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?_ None	2 years ago
Trevor Porter	20e6857ac1	Support DefaultIsmInterchainGasPaymaster in checker & govern (#1926 ) ### Description * Adds DefaultIsmInterchainGasPaymaster to the checker and govern logic * I'm aware this conflicts with https://github.com/hyperlane-xyz/hyperlane-monorepo/pull/1907 -- I wrote this mostly so I could propose new multisig txs to update the overhead gas amounts on mainnet. Happy to either merge this before or after https://github.com/hyperlane-xyz/hyperlane-monorepo/pull/1907, whatever works easiest ### Drive-by changes * Some fixes in the wake of the sdk refactor, e.g.: * if some tx params like `maxPriorityFeePerGas` or `maxFeePerGas` were specified when we estimate gas, it can fail if the balance of the sender is insufficient. For cases where we want to estimate gas from the owner multisig which may not have native tokens, this caused issues. * We should've been passing the `from` address into the MultiProvider.estimateGas fn as a separate param ### Related issues n/a ### 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?_ Ran & proposed some txs on mainnet to update the destination gas overheads	2 years ago
Asa Oines	8c456c97f4	Minor SDK refactor (#1904 ) ### Description Move `sdk/src/deploy/X` to `sdk/src/X`, where appropriate Consumers of the SDK should not need to make any changes ### Drive-by changes None ### Related issues Using this as a base for moving some of the IGP deployment tooling out of `HyperlaneCoreDeployer` into `HyperlaneIgpDeployer` ### 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 --------- Co-authored-by: Trevor Porter <trkporter@ucdavis.edu>	2 years ago
Trevor Porter	cd75c25f40	Check IGP ownership, transfer IGP ownership in the initializer (#1895 ) ### Description * Adds the IGP & DefaultIsmIgp to the HyperlaneCoreChecker's list of ownables * Transfers ownership in the `initialize` function of the IGP - this way we won't ever get in a scenario where the proxy admin is the owner for fresh deploys ### Drive-by changes * The provided default mumbai rpc `https://rpc-mumbai.maticvigil.com` was giving problems for eth_getStorageAt, so changed it * Removed the DefaultIsmInterchainGasPaymaster from the list of proxies because it's not proxied (this is in #1765, but made this change in here too) ### Related issues n/a ### 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
J M Rossy	406d365e03	Bump versions for 1.2.1 (#1915 ) - Increment package versions to 1.2.1 - Update submodules for their version bumps	2 years ago
Nam Chu Hoai	5152da508b	Validator metadata name matches key (#1918 ) ### Description If you use a different name, then MP lookup can fail. Thus we should validate that the key matches the name	2 years ago
Trevor Porter	1cbe5fd0a0	Update gas oracle exchange rates & IGP bytecode hash (#1919 ) ### Description Just updates mainnet gas oracle token exchange rates Gas prices were the same ### Drive-by changes none ### Related issues n/a ### 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?_ Deployed	2 years ago
J M Rossy	319838e871	Improve MultiProvider explorer url handling (#1898 ) - Improve MultiProvider explorer url handling - Add ApiKey to ChainMetadata - Add zod schema validation for ChainMetadata - Fix bug in HyperlaneCore message fetching	2 years ago
Trevor Porter	f5d1fbbd03	On chain gas oracles in the IGP (#1765 ) * Rough implementations of all gas oracles, done with StaticGasOracle & tests * more tests, beef up StorageGasOracle * Tests cleaned up, moved to Foundry test for IGP * rm StaticGasOracle, fix all tests * cleaning up * comment * yarn gas, better comments * Fix sdk test * Fix e2e * Rm LZ stuff * fix router test * Rm MockInterchainGasPaymaster * pr comments * Move to setGasConfigs instead of setGasConfig * fix test * nit * yarn gas * IGP deploy & checker tooling, script to update on-chain StorageGasOracle data (#1794) * Add IGP gas oracle to checker * Checker works * Support different gas oracle contracts in IGP deploy tooling too * Export some types * Build out set-storage-gas-oracle-values * nit * nit * rm comment * Add pricing data * nits * nearly there * haven't tested e2e yet, but added new proxy upgrade logic to set gas oracles in IGP in upgradeAndCall * pr comments * lots * Move away from upgradeAndCall for IGP upgrade, trying to debug annoying dependency issue * Got working, cleaned a little * Add test to ensure a new implementation is deployed * yarn hyperlane * nits * cleaning * Map in description * Dedupe submission type logic * yarn gas * Separate beneficiary in IGP (#1801) * Add IGP gas oracle to checker * Checker works * Support different gas oracle contracts in IGP deploy tooling too * Export some types * Build out set-storage-gas-oracle-values * nit * nit * Add separate beneficiary to IGP * Checker / govern / deploy tooling * nits * Add compare-storage-gas-oracle * successful test upgrade, minor changes * Nits * Fix tests * Performed testne3 upgrade. Need to clean up code, but should probably use this for the mainnet upgrade * nit * update prices * Update gas prices and token exchange rates for mainnet * mainnet upgrade. needs lots of cleaning, but here it is in all its glory * Starting to clean up * more cleanup * Use updated submodules * yarn gas * nit * PR comments * yarn hyperlane * yarn gas	2 years ago
Trevor Porter	5d9372603e	Fund kathy more on mainnet, add IGP claiming to key funder (#1883 ) Fund kathy more on mainnet, add claiming to key funder	2 years ago
Yorke Rhodes	582d45d029	Proxy middlewares in SDK deployers (#1838 ) * Proxy middlewares in SDK deployers * Small mods * Fix build	2 years ago
J M Rossy	31a102a0a0	PI SDK support, Multiprovider refactor, Type loosening (#1804 ) Breaking change to SDK types and Multiprovider interface - Add support for PI chains in SDK classes - Modify ChainMetadata schema - Re-implement Multiprovider to use ChainMetadata - Remove ChainConnection - Remove strict chain typing throughout SDK	2 years ago
Yorke Rhodes	ef7309e580	Add ISM to RouterDeployer.initConnectionClient (#1829 ) * Add ISM to RouterDeployer.initConnectionClient * Add unit tests * Revert ism default changes * Simplify ism 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
Nam Chu Hoai	38ef67005c	Check bytecodes against constants (#1595 ) * Check bytecodes against constants * Check ProxyAdmin * Fix for polygon	2 years ago
Asa Oines	a836ce21d3	Use ValidatorAnnounce in relayer (#1569 )	2 years ago
Yorke Rhodes	aeb056336c	Implement GasRouter (#1621 ) * Implement GasRouter * Add gas router deployer * Add quote gas helper to gas router * Add coverage to GasRouter * Update gas snapshot	2 years ago
J M Rossy	a0444ec787	Gnosis logos and metadata touchups (#1775 ) - Add gnosis chain logos - Modify default goerli rpc url - Export chainMetadataToWagmiChain util - Update wagmi chains dep	2 years ago
Trevor Porter	d3a19d5abb	Add moonbeam IQS & ICA addys (#1745 )	2 years ago
Nam Chu Hoai	8b4dd4ac06	Deploy SDK 1.1.3 (#1711 ) * Deploy SDK 1.1.3 * Update submodule references * Yarn.lock * Readme updates --------- Co-authored-by: Asa Oines <asaoines@gmail.com>	2 years ago

1 2 3 4 5

240 Commits (79669902ded862c93f067effbb47752b2b344ed8)