informalsystems / hermes-sdk

Apache License 2.0
9 stars 2 forks source link

Implement Tendermint `LightClient` #204

Open ljoss17 opened 7 months ago

ljoss17 commented 7 months ago

Summary

Building headers for Cosmos chain's is using the LightClient implementation from the current Hermes:

Which requires a AnyClientState https://github.com/informalsystems/hermes/blob/v1.7.4/crates/relayer/src/client_state.rs#L50. But the Sovereign Client state only has a TendermintParams field:

Some slight refactoring of the LightClient could allow decoupling it from Cosmos SDK, which would make it possible to build the headers without relying on the current Hermes implementation.


For Admin Use

thaodt commented 6 months ago

hi @ljoss17 can I take this? If yes, my first question is we still keep hermes v1.7.4 version for this feature?

soareschen commented 6 months ago

Hi @thaodt, it might be a little tricky for you to work on this as a first issue, as currently the direction is not very clear on how we should approach this re-implementation.

I have created #214 as an alternative good first issue that you can look into.