-
Notifications
You must be signed in to change notification settings - Fork 232
chore: Add Outbetestnet chain to Hyperlane registry (chainId 512512) #1208
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
📝 WalkthroughWalkthroughA new Outbe testnet chain configuration is introduced through two YAML files. The first defines contract address mappings for various infrastructure components, and the second specifies chain metadata including chain ID 512512, RPC endpoint, and native token details (COEN with 18 decimals). Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Suggested reviewers
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (2)
chains/outbetestnet/metadata.yaml (2)
1-1: Clear out that Cyrillic text in the comment.Line 1 has some Russian text ("для Intent Framework") that looks like it got left behind from a template. Better to keep comments clean and in English so everyone can read what this block does.
Apply this diff to clean up the comment:
-# Minimal Chain Config для Intent Framework +# Minimal Chain Config
8-9: Remove extra blank lines for cleaner formatting.There are a couple of blank lines here (8-9) before the
rpcUrlssection. Nothing breaks, but trimming them would tidy things up a bit. Like draining a swamp—sometimes you just gotta remove what's not needed.Apply this diff to remove the extra spacing:
isTestnet: true - - rpcUrls:
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
chains/outbetestnet/logo.svgis excluded by!**/*.svg
📒 Files selected for processing (2)
chains/outbetestnet/addresses.yaml(1 hunks)chains/outbetestnet/metadata.yaml(1 hunks)
🔇 Additional comments (2)
chains/outbetestnet/addresses.yaml (1)
1-13: All contract addresses present and properly formatted.Everything here layers together nicely—all 13 contract addresses are accounted for and formatted consistently with the 0x prefix and checksummed hex. No structural issues to worry about.
chains/outbetestnet/metadata.yaml (1)
3-20: Chain metadata structure and values look solid.The configuration is all there and correct: chainId and domainId both match the spec (512512), name aligns with the file path (outbe), protocol is ethereum, and the RPC endpoint, native token (COEN, 18 decimals), and block confirmations all match what's documented in the PR. The structure itself is valid YAML with no schema violations.
Description
Add Outbe chain to the Hyperlane registry.
outbetestnetOutbe Testnet512512512512ethereumhttps://eth.p.outbe.netOutbe (18 decimals)Core contracts:
domainRoutingIsmFactory: "0x1da7be0a2853B8e29df356d440aD9C4FB4d7deb0"
interchainAccountRouter: "0xe1f073901b6104ca5f93c817374D8d2cCF03DE1F"
mailbox: "0xAe6E952f092d5a944582E3E99a529aE8E97b077c"
merkleTreeHook: "0xE3AACA7b77c9C6B1791631AC12E592Bf8E01FB90"
proxyAdmin: "0xc01459C72F7E0eF80a54014Dd1985aF9Fb69a178"
staticAggregationHookFactory: "0x88060b27FA13c94D26929857Af4410cF009EDc9B"
staticAggregationIsmFactory: "0x559B52978C117514f84a45F325Fc67C6CEAfF3F2"
staticMerkleRootMultisigIsmFactory: "0xA7ffB9eEC28B64fe1495403922567Ac44cEF0272"
staticMerkleRootWeightedMultisigIsmFactory: "0x7a954D5659661aEa3672518cBCaD884384a5D6D1"
staticMessageIdMultisigIsmFactory: "0x2fdD5B36eAc3cA988af43F06e15e99f07C1C5e82"
staticMessageIdWeightedMultisigIsmFactory: "0x1AdEf7A458A7AF02626C7F82Cace41F4B8A99A50"
testRecipient: "0x945C85Ac7C4cBb6fB2E17d121648e69b221515f5"
validatorAnnounce: "0x4894B9F1F525c94bB5b38AD243EA82EeD43aA4b2"
Files added:
chains/outbe/metadata.yamlchains/outbe/addresses.yamlchains/outbe/logo.svgBackward compatibility
✅ Yes — this is an additive change (adds a new chain).
No existing configs are modified.
Testing
✅ Verified using Hyperlane CLI with local registry: