Add Blobstream Client to rpc crate

[Ferret-san]

Adds the Blobstream trait for the Client to be able to make blobstream related requests. The blobstream service for celestia-node can be found here

#[rpc(client)]
pub trait Blobstream {
    /// GetDataRootTupleRoot retrieves the data root tuple root for a given range from start to end
    #[method(name = "blobstream.GetDataRootTupleRoot")]
    async fn get_data_root_tuple_root(
        &self,
        start: u64,
        end: u64,
    ) -> Result<DataRootTupleRoot, Error>;

    /// GetDataRootTupleInclusionProof returns a data root tuple inclusion proof for a given height
    /// between a range from start to end
    #[method(name = "blobstream.GetDataRootTupleInclusionProof")]
    async fn get_data_root_tuple_inclusion_proof(
        &self,
        height: u64,
        start: u64,
        end: u64,
    ) -> Result<DataRootTupleInclusionProof, Error>;
}

[zvolin]

thanks! left some ideas

[zvolin]

in this block, the account we use for blob submission should be founded with tia, why this change?

[Ferret-san]

when testing against mocha, if I had my node sync from a recent height and trusted hash, waiting for height 2 would trigger a warning on the node header not found (celestia-node), so removing it allowed me to test this, but we could have an env for this height and that way the test does not assume you have a node synced from genesis etc

[zvolin]

We can instead do a loop here, where we request network head and break if it is >2 and sleep otherwise

[zvolin]

if it is a hash, could we alias it to celestia_types::Hash?

[Ferret-san]

yes can def do that

[zvolin]

I think I would combine those tests into a single one where we request a proof and the root and just call proof.verify(root).unwrap()

[Ferret-san]

yeah I pushed and opened the PR before making the test more robust if you will, let me get this under a single test 🫡

[zvolin]

those tests seem to fail on CI.

called `Result::unwrap()` on an `Err` value: Call(ErrorObject { code: ServerError(1), message: "end block 4211300 is higher than local chain height 28. Wait for the node until it syncs up to 4211300", data: None })

could you instead make header_get_network_head() request and adjust height accordingly?

[Ferret-san]

noted!

[zvolin]

one more thing, since those will be implicitly exposed on any jsonrpsee client, we have a convention of prefixing methods with the modeule name. Could you rename those to start with blobstream_?

[Ferret-san]

so blobstream_get_data_root_tuple_root?