[Miner] feat: add TxClient

[bryanchriswhite]

Summary

Human Summary

---
title: TxClient Dependency Graph
---


flowchart

    subgraph tclient[TxClient]
        tctx[TxContext]
        builder[TxBuilder]
        keyring[Keyring]

        bclient[BlockClient]
        eclient[EventsQueryClient]

        tclient_internal((_))
    end

    chain[Blockchain]

    tctx --"#GetKeyring()"--> keyring
tctx --"#SignTx()"--> builder
tctx --"#EncodeTx()"--> builder
tctx -."#BroadcastTx()".-> chain
tctx -."#QueryTx()".-> chain

eclient -."websocket connection".-> chain
bclient --"committed  block subscription"--> eclient
tclient_internal --"own tx subscription"--> eclient

Loading

AI Summary

Summary generated by Reviewpad on 07 Nov 23 07:56 UTC

This pull request includes several changes across multiple files.

• A new file godoc.go has been added to the pkg/client directory. This file defines a package called client that provides interfaces and types for interacting with blockchain functionalities.

• The go.mod file has been modified to update dependencies and remove unused dependencies.

• A new file services.go has been added to the pkg/client package. This file contains a function NewTestApplicationServiceConfig that returns a slice of pointers to sharedtypes.ApplicationServiceConfig structs.

• The context.go file has been modified to update comments and function names related to the cosmos-sdk transaction handling.

• A new file README.md has been added, which provides an overview of the client package, installation instructions, usage examples, and other sections.

• The errors.go file has been added to the pkg/client/tx package. It defines error variables related to transaction handling.

• The README.md file has been modified to remove certain sections.

• Several files have been modified or added to the pkg/client/tx package, including the addition of a new mock client for the BlockClient interface and changes to the TxClient and TxContext interfaces.

• The go.sum file has been updated to include changes to the versions of certain dependencies.

• Files related to test client functionality have been modified or added.

Overall, this pull request includes various additions, modifications, and updates across multiple files to enhance the functionality and documentation of the codebase.

Issue

Relates to:

• [Miner] Foundation for the Miner submodule #13

TxClient & TxContext are used to build, encode, sign, and broadcast transactions from an off-chain context.

Type of change

Select one or more:

• New feature, functionality or library
• Bug fix
• Code health or cleanup
• Documentation
• Other (specify)

Testing

• Run all unit tests: make go_test
• Verify Localnet manually: See the instructions [here](TODO: add link to instructions)

Sanity Checklist

• I have tested my changes using the available tooling
• I have performed a self-review of my own code
• I have commented my code, updated documentation and left TODOs throughout the codebase

[Olshansk]

@bryanchriswhite Checking in whether this is ready for another review yet or not?

[Olshansk]

Ty so much for simplifying the comments!