refactor: remove bytes/HexBytes

[tac0turtle]

Summary

Throughout the cosmos-sdk we have always used bytes.HexBytes from cmtbytes "github.com/cometbft/cometbft/libs/bytes". I have always questioned the need for this type even within tendermint. Its not an issue, but it would be nice to reduce the reliance on comet as a whole and remove this type if its not needed

Problem Definition

Not a problem

Proposal

Remove usage of bytes.HexBytes in order to reduce reliance of types from comet.

Who ever picks up this issue should first look if this change is breaking.

• do we rely on the json encoding of the type or was this purely additive?

[chixiaowen]

Can i take this one? @tnachen @alexanderbez

[tac0turtle]

yea, part of the worskcope would need to be to double check encoding that depended on it and provide documentation on how it chnaged for clients

[chixiaowen]

yea, part of the worskcope would need to be to double check encoding that depended on it and provide documentation on how it chnaged for clients

ok, and where is the specific documentation that I can modify?

[tac0turtle]

You would add it to upgrading.md

[chixiaowen]

@tac0turtle I have submitted a pr that marked as Draft, you can now review my pr and give me some suggestions。