[IBC] Add ICS-02 Client Interfaces

[h5law]

Description

Summary generated by Reviewpad on 26 Jul 23 09:37 UTC

This pull request includes the following changes:

1. Makefile:

   • Added lines to generate go structures for the ibc/client/types/proto/*.proto and ibc/client/light_clients/types/proto/*.proto files using the PROTOC_SHARED command.

2. ibc_host_module.go:

   • Removed the IBCHandler interface.
   • Updated the IBCHostSubmodule interface to only extend the Submodule and ibcHostFactory interfaces.
   • Removed the ibc_host constant.
   • Modified the return type of the GetTimestamp method.
   • Modified the return type of the GetProvableStore method.
   • Removed various IBC-related methods.

3. New file: pocket.proto:

   • Added a new proto file called "pocket.proto" in the "ibc/client/light_clients/types/proto" directory.
   • Defined multiple message types related to the Pocket network's IBC client.

4. bus_module.go:

   • Added a new method, "GetClientManager()", to the "Bus" interface.

5. New file: wasm.proto:

   • Added a new proto file called "wasm.proto" in the "ibc/client/types/proto" directory.
   • Defined several message types related to client and consensus states.

6. ibc_client_module.go:

   • Added a new Go package named "modules".
   • Defined constants, types, and interfaces for managing IBC light clients.
   • Included methods for client lifecycle management, client queries, and client state verification.

These changes cover updates to Makefile, interface definitions, addition of new files, and modifications to existing files.

Issue

Fixes #894

Type of change

Please mark the relevant option(s):

• New feature, functionality or library
• Bug fix
• Code health or cleanup
• Major breaking change
• Documentation
• Other

List of changes

• Create the Client submodule interface
  • Add the ClientManager interface
• Add the different interfaces for client types
• Add proto definitions for WASM client generic types and Pocket specific definitons

Testing

• make develop_test; if any code changes were made
• make test_e2e on k8s LocalNet; if any code changes were made
• e2e-devnet-test passes tests on DevNet; if any code was changed
• Docker Compose LocalNet; if any major functionality was changed or introduced
• k8s LocalNet; if any infrastructure or configuration changes were made

Required Checklist

• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have added, or updated, godoc format comments on touched members (see: tip.golang.org/doc/comment)
• I have tested my changes using the available tooling
• I have updated the corresponding CHANGELOG

If Applicable Checklist

• I have updated the corresponding README(s); local and/or global
• I have added tests that prove my fix is effective or that my feature works
• I have added, or updated, mermaid.js diagrams in the corresponding README(s)
• I have added, or updated, documentation and mermaid.js diagrams in shared/docs/* if I updated shared/*README(s)

[Olshansk]

TIL some stuff from the parametesr in this proto