Skip to content

feat: add protocol entry for stride-hyperlane #15541

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

0xn0va
Copy link

@0xn0va 0xn0va commented Jul 17, 2025

Closes #13710

This adapter tracks the amount of TIA bridged from Celestia to Stride via Hyperlane, using a lock-and-mint mechanism.
The TVL reflects the TIA tokens currently locked on Stride, representing liquidity originally from Celestia.

@waynebruce0x
Copy link
Collaborator

This is already counted in the stride-hyperlane adapter though? Why do we need another adapter file?

@waynebruce0x waynebruce0x self-assigned this Jul 18, 2025
@0xn0va
Copy link
Author

0xn0va commented Jul 18, 2025

The issue is that this adapter wasn't linked to any protocol via the protocols/ directory, so the TVL wasn't being surfaced on the site (e.g. Celestia chain page still showed $0 - reference issue #13710 ).

This file just connects the working adapter to the Celestia chain so the value gets properly aggregated and shown on the frontend. It doesn't duplicate anything — it only makes the existing TVL visible.

@0xn0va
Copy link
Author

0xn0va commented Jul 18, 2025

To be honest, the description I wrote in the PR could have been clearer.

It focuses on what the adapter itself is doing, but doesn’t really explain why this new protocols/ file is needed.

The real purpose of this PR is just to make sure the existing TVL from the adapter is surfaced on the site — the data was already being calculated, it just wasn’t being shown.

Thanks for pointing it out!

@waynebruce0x
Copy link
Collaborator

The reason for this is that we dont include bridge tvl in the chain aggregate tvls, this is the same across all chains

@0xn0va
Copy link
Author

0xn0va commented Jul 18, 2025

Got it. Thanks for clarifying. That makes sense now.

I had assumed bridged TVL would count towards the origin chain, but I understand now.

Thanks for the explanation. I am gonna to delete the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix celestia tvl
2 participants