CIP: Add Hyperlane protocol #263
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jcstein
reviewed
Mar 6, 2025
rootulp
approved these changes
Mar 6, 2025
cips/cip-add-hyperlane.md
Outdated
|
|
||
| ### Detailed Design | ||
|
|
||
| Hyperlane on Celestia, more concretely, requires the addition of two SDK modules: `x/core` and `x/warp`. These are imported from https://github.com/bcp-innovations/hyperlane-cosmos. Token transfer will be enabled upon upgrade. |
There was a problem hiding this comment.
[optional]
Suggested change
| Hyperlane on Celestia, more concretely, requires the addition of two SDK modules: `x/core` and `x/warp`. These are imported from https://github.com/bcp-innovations/hyperlane-cosmos. Token transfer will be enabled upon upgrade. | |
| Hyperlane on Celestia, more concretely, requires the addition of two SDK modules: `x/core` and `x/warp`. These are imported from https://github.com/bcp-innovations/hyperlane-cosmos. These two modules will be added in the next upgrade which will enable token transfers to other Hyperlane enabled chains. |
There was a problem hiding this comment.
The last sentence refers to an actual parameter that needs to be set to actually switch on token transfers. Maybe that wasn't clear
There was a problem hiding this comment.
Yes that wasn't clear. Which parameter needs to be set?
cips/cip-add-hyperlane.md
Outdated
|
|
||
| ## Motivation | ||
|
|
||
| Hyperlane is a popular bridging protocol that has support for EVM, currently the most popular VM for Celestia Rollups, as well as several others, creating connections between some 130 networks. Hyperlane is already employed indirectly to transfer TIA around but is dependent on third party networks, expanding the trust assumptions. Adding Hyperlane natively allows for direct transferring of TIA and possibly other tokens in the future without expanding the trust assumptions beyond the DA network. |
There was a problem hiding this comment.
[optional] added a few sentences about how Hyperlane differs from IBC and why that might be part of the motivation here.
Suggested change
| Hyperlane is a popular bridging protocol that has support for EVM, currently the most popular VM for Celestia Rollups, as well as several others, creating connections between some 130 networks. Hyperlane is already employed indirectly to transfer TIA around but is dependent on third party networks, expanding the trust assumptions. Adding Hyperlane natively allows for direct transferring of TIA and possibly other tokens in the future without expanding the trust assumptions beyond the DA network. | |
| The only bridging protocol currently deployed on Celestia is IBC. Hyperlane is a widely used bridging protocol that supports EVM—the most popular virtual machine for Celestia Rollups—as well as several other environments. Since IBC doesn't currently support EVM chains, adding Hyperlane to Celestia will enable connections to 100+ networks, many of which are not serviced by IBC. Currently, Hyperlane is used indirectly to transfer TIA, but it relies on third-party networks, which increases trust assumptions. Integrating Hyperlane natively would enable direct transfers of TIA, and potentially other tokens in the future, without expanding trust assumptions beyond the DA network. |
There was a problem hiding this comment.
IDK about the 130 network number because the Hyperlane website claims 100+ and the Hyperlane registry has 108 chains listed.
Ref:
There was a problem hiding this comment.
I think I got this number from Jacob. I don't mind replacing it with 100+ - I guess the idea is just to explain the extent of it's connectivity
Co-authored-by: Josh Stein <[email protected]>
Co-authored-by: Rootul P <[email protected]>
cmwaters
commented
Mar 7, 2025
Co-authored-by: Rootul P <[email protected]>
Co-authored-by: Rootul P <[email protected]>
Co-authored-by: Rootul P <[email protected]>
Co-authored-by: Rootul P <[email protected]>
|
It looks like this was merged without assigning a CIP number and adding it to the table of contents. Do you mind doing that @rootulp when you get a moment? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Supersedes: #256
Rendered