2621 Commits (f36beb683109f620dfd36560e79b29f687fa4e5a)

 

All Branches   

Search

Author	SHA1	Message	Date
Danil Nemirovsky	86b1c7646c	feat: Add check if an address is a contract in Cosmos (#4369) ... ### Description Currently, integration into Cosmos make an assumption that the address where a message should be delivered is a contract. This change allows Relayer to actually check if the recipient is a contract. ### Backward compatibility Yes ### Testing * Tested using E2E tests for Cosmos. * Tested using auxiliary unit test (it is marked as ignored since it connects to real gRPC of Neutron). --------- Co-authored-by: Danil Nemirovsky <4614623+ameten@users.noreply.github.com>	3 months ago
Mohammed Hussan	25c235a8e2	feat(infra): support renzo warp route for checker tooling (#4353) ... ### Description - enable checker tooling to check renzo warp route ### Drive-by changes - pass `mailboxAddress` to `moduleMatchesConfig`, needed for `FALLBACK_ROUTER` checks <!-- Are there any minor or drive-by changes also included? --> ### Testing Manual	3 months ago
Mohammed Hussan	0a311642f7	feat(infra): Update renzo warp config in infra used for monitoring (#4376) ... ### Description - update infra config to include latest renzo warp router addresses - update the image tag with new build	3 months ago
Paul Balaji	e51cd1d438	feat: misc improvements to infra deployment (#4374) ... feat: misc improvements to infra deployments These are all things that I've done somewhat ad-hoc before, but now upstreaming into the monorepo. - write the deploy plan to a file instead of stdout - to avoid cluttering terminal with 1000+ line configs - `getMultiProviderForRole` should only be populated with the requested set of chains or the supported chains for given environmen - to avoid fetching 50+ more keys than you need to on any given deploy - remove the agent config write-back from the post-deploy - this is already covered by the update-agent-config.ts script which is usually ran manually anyway - also moved the helper function into the update-agent-config.ts file, as that is the only place it is being used right now - in `infra/scripts` instead of `infra/src` to avoid a dependency issue - add `writeYamlAtPath` helper method to utils - with small refactor of existing `writeJsonAtPath` - streamline hit detection on `readCache` in legacy deployer - improve logging of legacy hook deployer when deploying an aggregation hook - better config matching detection when deploying routing hooks - extension of https://github.com/hyperlane-xyz/hyperlane-monorepo/pull/4305 --------- Signed-off-by: pbio <10051819+paulbalaji@users.noreply.github.com>	3 months ago
Paul Balaji	1094ddfa91	chore: update key-funder values (#4375) ... chore: update key-funder values based on observed usage	3 months ago
Paul Balaji	2ffb78f5c2	feat: improve mailbox initialized check (#4371) ... resolves https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/4233 - feat: improve mailbox initialized check - drive-by: update pinned registry + agent config	3 months ago
Daniel Savu	feb1d390c4	chore: lower log noisiness by removing `ret` from `PendingOperation::… (#4372) ... …perpare` ### Description <!-- What's included in this PR? --> ### Drive-by changes <!-- Are there any minor or drive-by changes also included? --> ### Related issues <!-- - Fixes #[issue number here] --> ### Backward compatibility <!-- Are these changes backward compatible? Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling? Yes/No --> ### Testing <!-- What kind of testing have these changes undergone? None/Manual/Unit Tests -->	3 months ago
Paul Balaji	240d24d06d	chore: add zircuit safe address (#4340) ... - chore: add zircuit safe address - drive-by update to include latest zircuit metadata --------- Signed-off-by: pbio <10051819+paulbalaji@users.noreply.github.com>	3 months ago
Daniel Savu	2909e81a55	feat(sealevel): charge protocol fees (#4355) ... ### Description Adds protocol fees to the Sealevel implementation, by embedding the logic / custody into the mailbox program & PDA. The logic is very similar to what already exists in the IGP implementation. The instructions to interact with the protocol fees implementation are: - `Init` (now requires specifying the `protocol_fee` and `beneficiary` in addition to the other mailbox params) - `ClaimProtocolFees` - invokable by anyone, and sending accumulated lamports to the `beneficiary` - `SetProtocolFeeConfig` - invokable by the owner, sets new values for `protocol_fee` and / or `beneficiary` ### Drive-by changes - The restriction applied on max message size has been removed to mirror the Solidity V3 implementation ### Related issues - Fixes https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/4356 ### Backward compatibility No - requires redeploying the mailbox ### Testing Sealevel program integration tests, as is done for the other programs	3 months ago
Mohammed Hussan	3449fc76b1	feat(infra): add check-warp-deply script that checks all AW managed w… (#4357) ... ### Description - add a script that allows use to check the warp route deployments for all AW managed warp routes ### Drive-by changes - split up check-deploy script into reusable method - move check-deploy into check directory in infra ### Testing Manual	3 months ago
Kunal Arora	203084df2d	feat: add sdk support for `IStaticWeightedMultisigIsm` (#4331) ... ### Description Add WeightedMerkleRootMultisigIsm and WeightedMessageIdMultisigIsm support to the proxy factory deployer and the hyperlaneIsmFactory ### Drive-by changes none ### Related issues - fixes https://github.com/hyperlane-xyz/issues/issues/1345 ### Backward compatibility Yes ### Testing Unit	3 months ago
github-actions[bot]	3010e36ec9	Version Packages (#4364) ... This PR was opened by the [Changesets release](https://github.com/changesets/action) GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated. # Releases ## @hyperlane-xyz/core@5.1.0 ### Minor Changes - 013f19c64: Added SDK support for ArbL2ToL1Hook/ISM for selfrelay - 013f19c64: Added hook/ism for using the Optimism native bridge for L2->L1 calls - 013f19c64: Added yield route with yield going to message recipient. - 013f19c64: feat: attributable fraud for signers - 013f19c64: Implement checkpoint fraud proofs for use in slashing ### Patch Changes - 013f19c64: fix: only evaluate dynamic revert reasons in reverting branch - @hyperlane-xyz/utils@5.1.0 ## @hyperlane-xyz/cli@5.1.0 ### Minor Changes - 013f19c64: Update to registry v2.5.0 - 013f19c64: Added SDK support for ArbL2ToL1Hook/ISM for selfrelay - 013f19c64: Add output of hyperlane warp read to ./configs/warp-route-deployment.yaml - 013f19c64: Remove registry.getUri() from core read logging to prevent registry error - 013f19c64: Fixes the new chain message to display the correct command - 013f19c64: Add check & confirm for existing mailbox to core deploy to allow users to decide if they want to deploy a new mailbox ### Patch Changes - 013f19c64: Require at least 1 chain selection in warp init - 013f19c64: feat: Add long-running CLI relayer - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [19f7d4fd9] - @hyperlane-xyz/sdk@5.1.0 - @hyperlane-xyz/utils@5.1.0 ## @hyperlane-xyz/helloworld@5.1.0 ### Minor Changes - 013f19c64: Update to registry v2.5.0 ### Patch Changes - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [19f7d4fd9] - @hyperlane-xyz/sdk@5.1.0 - @hyperlane-xyz/core@5.1.0 ## @hyperlane-xyz/sdk@5.1.0 ### Minor Changes - 013f19c64: Add ether's error reasoning handling to SmartProvider to show clearer error messages - 013f19c64: Support proxiedFactories in HypERC20App and extend HypERC20Checker with ProxiedRouterChecker - 013f19c64: Deploy to arbitrumsepolia, basesepolia, ecotestnet, optimismsepolia, polygonamoy - 013f19c64: Deploy to zircuit - 013f19c64: Update cosmos zod schema and enroll new validators for cheesechain, xlayer, zircuit, worldchain. - 013f19c64: Added SDK support for ArbL2ToL1Hook/ISM for selfrelay - 013f19c64: Support proxyAdmin checks for non AW owned warp router contracts - 013f19c64: Add stride validators to default multisig ism - 013f19c64: Adds CollateralFiat to token mapping which will output the correct standard to the warp deploy artifact. - 013f19c64: Deploy to solana + eclipse - 013f19c64: Added yield route with yield going to message recipient. - 19f7d4fd9: Support passing foreignDeployments to HypERC20App constructor ### Patch Changes - 013f19c64: feat: Add long-running CLI relayer - 013f19c64: Support xERC20Lockbox in checkToken - 013f19c64: Update ProxyAdminViolation interface to include proxyAdmin and proxy contract fields - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - @hyperlane-xyz/core@5.1.0 - @hyperlane-xyz/utils@5.1.0 ## @hyperlane-xyz/widgets@5.1.0 ### Minor Changes - 013f19c64: Update to registry v2.5.0 ### Patch Changes - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [19f7d4fd9] - @hyperlane-xyz/sdk@5.1.0 ## @hyperlane-xyz/utils@5.1.0 ## @hyperlane-xyz/infra@5.1.0 ### Minor Changes - 013f19c64: Update to registry v2.5.0 ### Patch Changes - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [013f19c64] - Updated dependencies [19f7d4fd9] - @hyperlane-xyz/sdk@5.1.0 - @hyperlane-xyz/helloworld@5.1.0 - @hyperlane-xyz/utils@5.1.0 ## @hyperlane-xyz/ccip-server@5.1.0 Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>	3 months ago
Lee	013f19c64c	fix: Revert "Version Packages (#4259)" (#4365) ... This reverts commit aff57bfa67.	3 months ago
github-actions[bot]	aff57bfa67	Version Packages (#4259) ... This PR was opened by the [Changesets release](https://github.com/changesets/action) GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated. # Releases ## @hyperlane-xyz/core@5.1.0 ### Minor Changes - e151b5f9a: Added SDK support for ArbL2ToL1Hook/ISM for selfrelay - 6f377f174: Added hook/ism for using the Optimism native bridge for L2->L1 calls - 63abe564d: Added yield route with yield going to message recipient. - 76f7ecaff: feat: attributable fraud for signers - cb404cb85: Implement checkpoint fraud proofs for use in slashing ### Patch Changes - 2edfa4043: fix: only evaluate dynamic revert reasons in reverting branch - @hyperlane-xyz/utils@5.1.0 ## @hyperlane-xyz/cli@5.1.0 ### Minor Changes - d274c6b3f: Update to registry v2.5.0 - e151b5f9a: Added SDK support for ArbL2ToL1Hook/ISM for selfrelay - dd12e93e0: Add output of hyperlane warp read to ./configs/warp-route-deployment.yaml - 9087bfcbb: Remove registry.getUri() from core read logging to prevent registry error - 4a1972903: Fixes the new chain message to display the correct command - 7d3b5ceac: Add check & confirm for existing mailbox to core deploy to allow users to decide if they want to deploy a new mailbox ### Patch Changes - 3d4e6436d: Require at least 1 chain selection in warp init - c2c5bb9bb: feat: Add long-running CLI relayer - Updated dependencies [d408b0d6f] - Updated dependencies [1516e7f51] - Updated dependencies [2d922805c] - Updated dependencies [103e6b3e1] - Updated dependencies [c2c5bb9bb] - Updated dependencies [1abefa60e] - Updated dependencies [279516f28] - Updated dependencies [ff0171677] - Updated dependencies [e151b5f9a] - Updated dependencies [cab86f2f9] - Updated dependencies [4f19b9fae] - Updated dependencies [19ea05384] - Updated dependencies [a49f52cc9] - Updated dependencies [63abe564d] - Updated dependencies [19f7d4fd9] - @hyperlane-xyz/sdk@5.1.0 - @hyperlane-xyz/utils@5.1.0 ## @hyperlane-xyz/helloworld@5.1.0 ### Minor Changes - d274c6b3f: Update to registry v2.5.0 ### Patch Changes - Updated dependencies [d408b0d6f] - Updated dependencies [1516e7f51] - Updated dependencies [2d922805c] - Updated dependencies [103e6b3e1] - Updated dependencies [c2c5bb9bb] - Updated dependencies [1abefa60e] - Updated dependencies [279516f28] - Updated dependencies [ff0171677] - Updated dependencies [e151b5f9a] - Updated dependencies [cab86f2f9] - Updated dependencies [6f377f174] - Updated dependencies [4f19b9fae] - Updated dependencies [19ea05384] - Updated dependencies [a49f52cc9] - Updated dependencies [2edfa4043] - Updated dependencies [63abe564d] - Updated dependencies [76f7ecaff] - Updated dependencies [cb404cb85] - Updated dependencies [19f7d4fd9] - @hyperlane-xyz/sdk@5.1.0 - @hyperlane-xyz/core@5.1.0 ## @hyperlane-xyz/sdk@5.1.0 ### Minor Changes - d408b0d6f: Add ether's error reasoning handling to SmartProvider to show clearer error messages - 1516e7f51: Support proxiedFactories in HypERC20App and extend HypERC20Checker with ProxiedRouterChecker - 2d922805c: Deploy to arbitrumsepolia, basesepolia, ecotestnet, optimismsepolia, polygonamoy - 103e6b3e1: Deploy to zircuit - 279516f28: Update cosmos zod schema and enroll new validators for cheesechain, xlayer, zircuit, worldchain. - e151b5f9a: Added SDK support for ArbL2ToL1Hook/ISM for selfrelay - cab86f2f9: Support proxyAdmin checks for non AW owned warp router contracts - 4f19b9fae: Add stride validators to default multisig ism - 19ea05384: Adds CollateralFiat to token mapping which will output the correct standard to the warp deploy artifact. - a49f52cc9: Deploy to solana + eclipse - 63abe564d: Added yield route with yield going to message recipient. - 19f7d4fd9: Support passing foreignDeployments to HypERC20App constructor ### Patch Changes - c2c5bb9bb: feat: Add long-running CLI relayer - 1abefa60e: Support xERC20Lockbox in checkToken - ff0171677: Update ProxyAdminViolation interface to include proxyAdmin and proxy contract fields - Updated dependencies [e151b5f9a] - Updated dependencies [6f377f174] - Updated dependencies [2edfa4043] - Updated dependencies [63abe564d] - Updated dependencies [76f7ecaff] - Updated dependencies [cb404cb85] - @hyperlane-xyz/core@5.1.0 - @hyperlane-xyz/utils@5.1.0 ## @hyperlane-xyz/widgets@5.1.0 ### Minor Changes - d274c6b3f: Update to registry v2.5.0 ### Patch Changes - Updated dependencies [d408b0d6f] - Updated dependencies [1516e7f51] - Updated dependencies [2d922805c] - Updated dependencies [103e6b3e1] - Updated dependencies [c2c5bb9bb] - Updated dependencies [1abefa60e] - Updated dependencies [279516f28] - Updated dependencies [ff0171677] - Updated dependencies [e151b5f9a] - Updated dependencies [cab86f2f9] - Updated dependencies [4f19b9fae] - Updated dependencies [19ea05384] - Updated dependencies [a49f52cc9] - Updated dependencies [63abe564d] - Updated dependencies [19f7d4fd9] - @hyperlane-xyz/sdk@5.1.0 ## @hyperlane-xyz/utils@5.1.0 ## @hyperlane-xyz/infra@5.1.0 ### Minor Changes - d274c6b3f: Update to registry v2.5.0 ### Patch Changes - Updated dependencies [d408b0d6f] - Updated dependencies [1516e7f51] - Updated dependencies [d274c6b3f] - Updated dependencies [2d922805c] - Updated dependencies [103e6b3e1] - Updated dependencies [c2c5bb9bb] - Updated dependencies [1abefa60e] - Updated dependencies [279516f28] - Updated dependencies [ff0171677] - Updated dependencies [e151b5f9a] - Updated dependencies [cab86f2f9] - Updated dependencies [4f19b9fae] - Updated dependencies [19ea05384] - Updated dependencies [a49f52cc9] - Updated dependencies [63abe564d] - Updated dependencies [19f7d4fd9] - @hyperlane-xyz/sdk@5.1.0 - @hyperlane-xyz/helloworld@5.1.0 - @hyperlane-xyz/utils@5.1.0 ## @hyperlane-xyz/ccip-server@5.1.0 --------- Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Le Yu <6251863+ltyu@users.noreply.github.com>	3 months ago
Lee	7f8a68de72	fix: changeset typo (#4363)	3 months ago
Trevor Porter	68dc989e8e	feat: basic script to help generate validator sets for Cosmos tooling (#4346) ... ### Description - As part of a recent endeavor to update validator sets on Neutron, this script helped generate validator sets in a format compatible with the `cw-hyperlane` deploy tooling - Some context: - the output of this script is copied into a config that's then used with e.g. `CONFIG=./ism-config-aug-13.yaml yarn cw-hpl update ism-multisig -n neutron-1` - a full writeup of everything I did here https://www.notion.so/hyperlanexyz/Updating-Neutron-default-ISM-091a3528343c4e7b98453768eb950b38 ### Drive-by changes <!-- Are there any minor or drive-by changes also included? --> ### Related issues <!-- - Fixes #[issue number here] --> ### Backward compatibility <!-- Are these changes backward compatible? Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling? Yes/No --> ### Testing <!-- What kind of testing have these changes undergone? None/Manual/Unit Tests -->	3 months ago
Nam Chu Hoai	0779a23db6	refactor: Rename InterchainAccountRouter tests for easier reuse (#4336) ... ### Description Minor refactor of test	3 months ago
Mohammed Hussan	cab86f2f97	feat(sdk): Support proxyAdmin checks for admins not owned by AW (#4338) ... ### Description - Support checking proxyAdmin owners for non AW proxy admins - Stop assuming all proxyAdmins will be owned by AW - This change was motivated by the need to have our checker tooling check Renzo Wapr routes, where Renzo owns the proxy routers ### Drive-by changes - Clean up the interface for `ProxyAdminViolation` ### Testing Manual	3 months ago
Lee	da55df5b4a	fix(sdk): Update smart provider logging to debug (#4344) ... ### Description This stops deriving errors such as `hyperlane ism read --address 0xc6608b648111020719d85163017a0dd2c9943d69 --chain fraxtal` ### Testing Manual	3 months ago
Lee	a6d3d12ce2	fix: Update renzo zircuit validator set (#4343) ... Should match default Ism: 1abefa60e2/typescript/sdk/src/consts/multisigIsm.ts (L539-L542)	3 months ago
Mohammed Hussan	1abefa60e2	feat(checkToken): Support checking token config for xERC20Lockbox type (#4339) ... ### Description - When setting up the warp configuration for an `xERC20Lockbox` contract, we pass the lockbox address in the token field for the config and we lookup the underlying ERC20 token in the smart contract constructor 1579ca221c/solidity/contracts/token/extensions/HypXERC20Lockbox.sol (L17) - This change accounts for this setup when checking for `CollateralTokenMismatch` violations ### Testing Manual	3 months ago
Paul Balaji	4f19b9fae6	feat: add stride validators to default multisig (#4315) ... feat: add stride validators to default multisig	3 months ago
Trevor Porter	6c2f01f83a	chore: add back Osmosis relaying / validator (#4335) ... ### Description - seems I accidentally disabled optimism relaying before, yikes - adding back for Skip to play around with ### Drive-by changes <!-- Are there any minor or drive-by changes also included? --> ### Related issues <!-- - Fixes #[issue number here] --> ### Backward compatibility <!-- Are these changes backward compatible? Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling? Yes/No --> ### Testing <!-- What kind of testing have these changes undergone? None/Manual/Unit Tests -->	3 months ago
Kunal Arora	6f377f1743	feat: implement `OPL2ToL1Ism` contracts (#4319) ... ### Description Implementing the `OPL2ToL1Hook` and OPL2ToL1Ism` contracts for using the Optimism native bridge for sending messages from the L2 to L1. - Optimism needs double call encoding for the direct portal call because the portal itself calls the l1Messenger which then directs it to the ism. ### Drive-by changes None ### Related issues - fixes https://github.com/hyperlane-xyz/issues/issues/1346 ### Backward compatibility Yes ### Testing Unit testing with mocks for OP contracts	3 months ago
Trevor Porter	cc8624d687	chore: fix key funder errors, lower some thresholds (#4333) ... ### Description - updated the image to include the sei gas price fix #4321 - lowered a bunch of thresholds for some newer chains that could be lower based off looking at the fees / available balances. Also bridged some funds over to them ### Drive-by changes <!-- Are there any minor or drive-by changes also included? --> ### Related issues <!-- - Fixes #[issue number here] --> ### Backward compatibility <!-- Are these changes backward compatible? Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling? Yes/No --> ### Testing <!-- What kind of testing have these changes undergone? None/Manual/Unit Tests -->	3 months ago
J M Rossy	06180b060b	chore: Enable no-restricted-imports eslint rule (#4329) ... ### Description Incorrect imports from node-specific global libs, especially for `assert` are very common. This won't catch all possible mistakes but it will help avoid future problems like https://github.com/hyperlane-xyz/hyperlane-monorepo/pull/4327 ### Backward compatibility Yes	3 months ago
Lee	580f6c109e	fix(sdk): Remove console.assert (#4327) ... ### Description warp template UI throws  ### Backward compatibility Yes ### Testing Manual	3 months ago
Trevor Porter	57bbb16cc1	feat: panics upon startup when RPC issues occur include the offending chain (#4323) ... ### Description Some quick low hanging fruit to make debugging easier in the future. Today (https://discord.com/channels/935678348330434570/1273582453889699843) some chains were having RPC issues and it caused an agent deploy to panic upon startup. Debugging this was hard because the panic message just indicates an error is occurring, but doesn't say what the offending chain is! Examples: - https://cloudlogging.app.goo.gl/puQk7hm4fXyJhYKs8 - https://cloudlogging.app.goo.gl/5qZrbc2Ww4LWQXWC6 - https://cloudlogging.app.goo.gl/ZL7mKbBzzJaePtty9 All these panic happen when constructing the cursor, which isn't yet done in an indexing task. The quick and easy fix is to just use expect instead of unwrap We still unwrap in a few places, but mostly in places that aren't susceptible to flaky RPC issues, e.g. reading from a HashMap that we know should always have an entry, etc ### Drive-by changes <!-- Are there any minor or drive-by changes also included? --> ### Related issues <!-- - Fixes #[issue number here] --> ### Backward compatibility <!-- Are these changes backward compatible? Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling? Yes/No --> ### Testing <!-- What kind of testing have these changes undergone? None/Manual/Unit Tests -->	3 months ago
Trevor Porter	ddf6834c7b	chore: drop some labels in remote writes to lower cardinality (#4326) ... - Labels chosen by looking at https://abacusworks.grafana.net/d/cardinality-management/cardinality-management-1-overview?var-datasource=grafanacloud-prom&from=2024-08-15T11:34:34.928Z&to=2024-08-16T11:34:34.929Z - Motivating case: a GKE rolling upgrade recently resulting in some issues with the remote write where it complained about too many series https://abacusworks.grafana.net/d/cdn9jukg55bswb/usage-insights-5-metrics-ingestion?var-instance=$__all&viewPanel=panel-9&from=2024-08-16T10:04:52.962Z&to=2024-08-16T11:04:52.962Z (thread https://discord.com/channels/935678348330434570/1273952173037715458) - Many of these labels would change whenever a pod would be restarted, a new deploy occurred, new nodes set up, etc - Deployed to testnet4 and mainnet3 and observed the labels were dropped ### Description <!-- What's included in this PR? --> ### Drive-by changes <!-- Are there any minor or drive-by changes also included? --> ### Related issues <!-- - Fixes #[issue number here] --> ### Backward compatibility <!-- Are these changes backward compatible? Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling? Yes/No --> ### Testing <!-- What kind of testing have these changes undergone? None/Manual/Unit Tests -->	3 months ago
Mohammed Hussan	9680efc3b1	feat(infra): Update warp configs in infra (#4299) ... ### Description - update the warp configs in infra to reflect expected behavior or change behavior ### Drive-by changes - allow fork script to support passing warp route id ### Testing Manual	3 months ago
Mohammed Hussan	1516e7f51b	feat(HypERC20App): Support proxiedFactories in HypERC20App, extend HypERC20Checker with ProxiedRouterChecker (#4312) ... ### Description - Extend `HypERC20Checker` with `ProxiedRouterChecker` so we can check for proxyAdmin violations (`checkProxiedContracts` is call in `checkChain`) - Support passing `proxiedFactories` to `HypERC20App` ### Fixes - #4325 ### Testing Manual	3 months ago
Lee	19ea053845	fix(sdk): Add collateralFiat support into Token.ts & TokenStandard.ts (#4308) ... ### Description Adds CollateralFiat to token mapping which will output the correct `standard` to the warp deploy artifact. ### Related issues https://github.com/hyperlane-xyz/hyperlane-monorepo/issues/4307 ### Backward compatibility Yes ### Testing Manually tested with Beta (collateral) -> Gamma (CollateralFiat) 1. Deploy an ERC20 to represent USDC on beta ``` forge create contracts/test/ERC20Test.sol:ERC20Test \ --constructor-args "Test" "TS" 10000000000000000000000000 6 \ --rpc-url https://beta-op.rpc.caldera.xyz/http \ --private-key $HYP_KEY ``` 2. Deploy a CollateralFiat on gamma ``` forge create contracts/test/ERC20Test.sol:FiatTokenTest \ --constructor-args "FiatTest" "FTS" 0 6 \ --rpc-url https://rpc-gamma-2bgo7wnh9d.t.conduit.xyz \ --private-key $HYP_KEY ``` 3. warp init and deploy 4. Start up the ts relayer `hyperlane relayer --chains beta,gamma` 5. Add the deploy artifacts to UI ``` # warpRoutes.yaml tokens: - chainName: beta standard: EvmHypCollateral decimals: 6 symbol: TS name: Test addressOrDenom: "0x56D13Eb21a625EdA8438F55DF2C31dC3632034f5" collateralAddressOrDenom: "0x9BcC604D4381C5b0Ad12Ff3Bf32bEdE063416BC7" connections: - token: ethereum|gamma|0xA7c59f010700930003b33aB25a7a0679C860f29c - chainName: gamma standard: EvmHypCollateralFiat decimals: 6 symbol: TS name: Test addressOrDenom: "0xA7c59f010700930003b33aB25a7a0679C860f29c" collateralAddressOrDenom: "0xD5ac451B0c50B9476107823Af206eD814a2e2580" connections: - token: ethereum|beta|0x56D13Eb21a625EdA8438F55DF2C31dC3632034f5 # chains.yaml beta: displayName: Beta chainId: 7097927 domainId: 7097927 protocol: ethereum name: beta isTestnet: true rpcUrls: - http: https://beta-op.rpc.caldera.xyz/http nativeToken: symbol: ETH name: Ether decimals: 18 gamma: blockExplorers: - apiUrl: https://explorer-gamma-2bgo7wnh9d.t.conduit.xyz family: blockscout name: gamma explorer url: https://explorer-gamma-2bgo7wnh9d.t.conduit.xyz chainId: 69995 displayName: Gamma domainId: 69995 isTestnet: true name: gamma nativeToken: decimals: 18 name: Ether symbol: ETH protocol: ethereum rpcUrls: - http: https://rpc-gamma-2bgo7wnh9d.t.conduit.xyz ``` 7. Start up UI	3 months ago
Trevor Porter	beab39c43c	chore: update sei gas price, temporarily stop deploying to osmosis (#4321) ... ### Description Following https://seitrace.com/proposal/83?chain=pacific-1, need a higher gas price ### Drive-by changes <!-- Are there any minor or drive-by changes also included? --> ### Related issues <!-- - Fixes #[issue number here] --> ### Backward compatibility <!-- Are these changes backward compatible? Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling? Yes/No --> ### Testing <!-- What kind of testing have these changes undergone? None/Manual/Unit Tests -->	3 months ago
Paul Balaji	279516f280	chore: enroll worldchain, xlayer, cheesechain, zircuit validators (#4313) ... - chore: enroll worldchain, xlayer, cheesechain, zircuit validators - fix: update cosmos zod schema - drive-by: update core checker to check if the ism in registry matches the expected config	3 months ago
Yorke Rhodes	0d544d97e5	chore: use new forge feature for filtering coverage (#4318) ... ### Description - Remove lcov toolchain from coverage script in favor of new native forge features - Require strictly increasing project coverage ### Drive-by changes - Add coverage gutters to vscode recommendations for inline view ### Backward compatibility Yes ### Testing See codecov report comment	3 months ago
Lee	5dfa2d54e6	fix(infra): Sort Renzo validators (#4317) ... As requested by @Mo-Hussain to aid deploy checker	3 months ago
Trevor Porter	c4e7a4e5ee	fix: require thresholds to always be a set majority (#4314) ... ### Description - Thresholds must always be a set majority, enforced by a unit test - Updates thresholds for some chains that should've been updated as the set expanded ### Drive-by changes <!-- Are there any minor or drive-by changes also included? --> ### Related issues <!-- - Fixes #[issue number here] --> ### Backward compatibility <!-- Are these changes backward compatible? Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling? Yes/No --> ### Testing <!-- What kind of testing have these changes undergone? None/Manual/Unit Tests -->	3 months ago
Lee	4a1972903e	fix(cli): incorrect command for new chains (#4285) ... ### Description Fixes the new chain message to display the correct command	3 months ago
Yorke Rhodes	4425c69991	fix: solidity coverage failing due to fixture issue (#4304) ... ### Description Fixes solidity coverage reporting ### Testing CI Tests	3 months ago
Mohammed Hussan	ff0171677d	feat(governor): Support handling proxyAdmin violation in HyperlaneAppGovernor (#4311) ... ### Description - Support handling proxyAdmin violation, where the proxyAdmin of a proxy contract is not the expected proxyAdmin, `HyperlaneAppGovernor` - `HyperlaneCoreGovernor` and `ProxiedRouterGovernor` implementation will use it ### Testing Manual	3 months ago
Lee	d0ce06280f	feat(infra): Add Zircuit and Safes to Renzo deploy script (#4310) ... ### Description Adds Zircuit and Safes to Renzo deploy script ### Backward compatibility Yes ### Testing Manual	3 months ago
Paul Balaji	181259a8b0	fix: short-circuit known configs in legacy hook deployer (#4305) ... fix: short-circuit known configs in legacy hook deployer given we already do the process of recovering the same hooks for each domain, this short-circuits that even further to avoid the extra time taken to try, recover, move-on for each hook	3 months ago
Trevor Porter	9db5871f27	fix: use correct Neutron IGP (#4303) ... ### Description - We've been operating Neutron warp routes with a whitelist so far. It seems like in the initial shuffle of things, we configured the agents to index an IGP that was actually replaced 😬 by another one - This starts indexing the new one. Because of watermark indexing, we won't backfill old IGP payments, because from its perspective it thinks it's already indexed those. I wonder if we should tie watermark indexing cursors to the IGP address? Either way, this is probably fine as to date only a couple non-warp messages from Neutron have occurred ### Drive-by changes <!-- Are there any minor or drive-by changes also included? --> ### Related issues <!-- - Fixes #[issue number here] --> ### Backward compatibility <!-- Are these changes backward compatible? Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling? Yes/No --> ### Testing <!-- What kind of testing have these changes undergone? None/Manual/Unit Tests -->	3 months ago
Paul Balaji	d274c6b3f8	chore: update registry to v2.5.0 (#4302) ... chore: update registry to v2.5.0	3 months ago
Yorke Rhodes	76f7ecafff	feat: attributable fraud for signers (#4284) ... ### Description Implements fraud proof attributions for checkpoint signers (validators). It is important to have a record of attribution in storage for slashing replay protection. We also maintain a [whitelist of merkle tree hooks](b14f997810/solidity/contracts/libs/CheckpointLib.sol (L50-L54)) such that validators cannot simply lie about this part of the checkpoint and circumvent fraud proofs. ### Backward compatibility Yes ### Testing Unit Tests	3 months ago
Nam Chu Hoai	c2c5bb9bbc	feat: Add long-running CLI relayer (#4279) ... ### Description Adds the CLI equivalent of #3693. Should mostly be used only for demo purposes with reliable RPCs in the registry. Includes some drive-by changes to the existing single-message self-relay. ### Drive-by changes - Self-relay fixes ### Backward compatibility Yes ### Testing Manual --------- Co-authored-by: J M Rossy <jm.rossy@gmail.com>	3 months ago
Lee	9087bfcbb8	fix(cli): Remove getUri from core read logging (#4283) ... ### Description This removes the `getUri()` from a specific log because the function throws for MergeRegistry and ends up hiding the real error message. ### Backward compatibility Yes ### Testing Manual	3 months ago
Paul Balaji	8be75b6569	feat: core deploy to new chains (#4281) ... Deploy to new chains: 1. cyber 1. degenchain 1. kroma 1. lisk 1. lukso 1. merlin 1. metis 1. mint 1. proofofplay 1. real 1. sanko 1. tangle 1. xai drive-by: - fix bug in `set-rpc-urls-from-registry.ts` - add script to generate sql to add multiple chains to scraper db	3 months ago
Paul Balaji	2d922805c0	feat: add arbitrumsepolia, basesepolia, ecotestnet, optimismsepolia, polygonamoy to core chains (#4297) ... Deploy to arbitrumsepolia, basesepolia, ecotestnet, optimismsepolia, polygonamoy	4 months ago
Kunal Arora	63abe564d4	feat: implementation of yield rebasing warp route contracts (#4203) ... ### Description - To reflect the rebased underlying asset, `_transferRemote` is overridden because we need to pass in the ERC4626 vault shares which on the synthetic `HypERC4626VaultYield` which change in value as the underlying vault shares accrue yield or receive a drawdown. - The exchangeRate (i.e., assets/shares) is encoded as metadata and sent to the synthetic token to calculate the value of the synthetic token. - On synthetic transfers, the overridden `ERC20.transfer()` function calculates the needed shares for the amount and transfers them instead. Design decisions - Do we need to rebase frequently? Yes, otherwise the exchangeRate on the synthetic token will be stale compared to the the actual vault claimable yield, which means a user who deposits later and right before the infrequent rebase will get disproportionate share of the yield. You can rebase by calling `rebase()` on the collateral token which makes a transferRemote call with 0 amount. - Why exchangeRate and not underlyingDepositedAmount on the synthetic? If a user withdraws (ie calls transferRemote on the synthetic) while a rebase is inflight, the underlyingDepositedAmount will get an inaccurate update (will be overreported and hence will derive inaccurate balance amounts on the synthetic. The exchangeRate doesn't change if the user is depositing or withdrawing. - Why override the `transfer` on the synthetic? Otherwise if a user A sends 100 USDC worth of shares to user B which has a exchange rate of 1.05, user A would have unwittingly sent 105 #tokens (5 excess). - What happens in a drawdown of the value? Once the user calls withdraws the the withdrawal they get out is accordingly discounted. If a user transfers on the synthetic, it will still gets the stale exchangeRate until rebase. - What if the owner of the vault charges a fee on the yield? In that case, you need to override the totalAssets to reflect the assets - feesByOwner and the warp route implementation will stay the same. One example implementation is `MockERC4626YieldSharing` and we have adequate tests to cover the functionality. - Rounding error? We're rounding down the exchangeRate as per the general recommendation for ERC4626 vault math. - Why not make the synthetic token an ERC4626 itself, given that there's some overlapping functionality? Even though, the assetsToShares and assetsToShares functions are the same, the synthetic token itself doesn't have an underlying token so it doesn't confer to the 4626 interface. - What if there are multiple synthetic tokens for a single rebasing collateral contract and what happens is the exchange rate is not in sync between the synthetics? Things to keep in mind for - How do I keep the exchangeRate in sync with the underlying vault? Keep calling rebase as a cron job frequently, or make a hook call in deposit, redeem, and withdraw function to automatically resync the exchangeRate. - 4626 math pitfalls: https://www.zellic.io/blog/exploring-erc-4626/ ### Drive-by changes - Renaming `HypERC20CollateralVaultDeposit` -> `HypERC4626OwnerYieldCollateral` ### Related issues - fixes https://github.com/hyperlane-xyz/issues/issues/1334 ### Backward compatibility Yes ### Testing Unit tests	4 months ago