Philippe Laferrière

yes I'm okay with keeping this issue open as the signature of `process_recv_packet_execute` indeed is not idiomatic.

Right, I thought some more about this since yesterday, and I think it's easily solvable if we document that "if `validate/execute/deliver()` returns an error (any top-level function), all state changes...

The short answer is that they are not essential to the IBC protocol, and the value of this feature could come if we had hundreds of IBC apps and it...

> I think I need a little bit more context around what an "IBC app" means here. The term is unfortunately overloaded. An "app" is the same as an "IBC...

> Does the ibc-rs library need to be protected against reentrancy attacks? That is currently an open question for me, and I currently don't have enough "concrete data" to respond....

Roughly yes. Basically an API over gRPC which would achieve the same as our current `ValidationContext`, `ExecutionContext`, etc.

The project has turned more recently into a chain that uses ibc-rs, to give us better insights into how using the ibc-rs API feels, as well as run some integration...

Hey @DaviRain-Su, what do you mean by a "local boot tutorial"? Currently the only docs we have is the [Usage section](https://github.com/informalsystems/basecoin-rs#usage) of the README (although those are not perfect either)....

Can you try with tendermint v0.34.21? This has solved issues for me in the past. Although planned, we are not yet ready to upgrade to CometBFT v0.37.0.

Yep makes a lot more sense. Updated the description. Although another alternative that might be best is to pass it by reference, and potentially have `type InitData: Clone;`?