fix(textual): Use HEX instead of base64

docs/architecture/adr-050-sign-mode-textual-annex1.md

@@ -214,7 +214,7 @@ Rendered as either ISO8601 (`2021-01-01T12:00:00Z`).
 
 ### bytes
 
-- Bytes are rendered in base64.
+- Bytes are rendered in hexadecimal, all capital letters, without the `0x` prefix.
 
 ### address bytes
 

docs/architecture/adr-050-sign-mode-textual.md

@@ -93,7 +93,7 @@ We define "transaction envelope" as all data in a transaction that is not in the
 ```
 Chain ID: <string>
 Account number: <uint64>
-*Public Key: <base64_string>
+*Public Key: <hex_string>
 Sequence: <uint64>
 <TxBody>                                                    // See #8.
 Fee: <coins>                                                // See value renderers for coins encoding.
@@ -112,7 +112,7 @@ Tip: <string>
 *<repeated Any>
 *This transaction has <int> other signers:                  // Skipped if there is only one signer
 *Signer (<int>/<int>):
-*Public Key: <base64_string>
+*Public Key: <hex_string>
 *Sequence: <uint64>
 *End of other signers
 ```
@@ -222,7 +222,7 @@ SIGN_MODE_TEXTUAL:
 ```
 Chain ID: simapp-1
 Account number: 10
-*Public Key: iQ...==        // Base64 pubkey
+*Public Key: iQ...==        // Hex pubkey
 Sequence: 2
 This transaction has 1 message:
 Message (1/1): bank v1beta1 send coins
@@ -481,7 +481,7 @@ SIGN_MODE_TEXTUAL is purely additive, and doesn't break any backwards compatibil
 
 ### Negative
 
-- Some fields are still encoded in non-human-readable ways, such as public keys in base64.
+- Some fields are still encoded in non-human-readable ways, such as public keys in hexadecimal.
 - New ledger app needs to be released, still unclear
 
 ### Neutral

tx/textual/valuerenderer/bytes.go

@@ -2,7 +2,7 @@ package valuerenderer
 
 import (
 	"context"
-	"encoding/base64"
+	"encoding/hex"
 	"io"
 
 	"google.golang.org/protobuf/reflect/protoreflect"
@@ -14,7 +14,7 @@ type bytesValueRenderer struct{}
 var _ ValueRenderer = bytesValueRenderer{}
 
 func (vr bytesValueRenderer) Format(ctx context.Context, v protoreflect.Value, w io.Writer) error {
-	_, err := io.WriteString(w, base64.StdEncoding.EncodeToString(v.Bytes()))
+	_, err := io.WriteString(w, hex.EncodeToString(v.Bytes()))
 	return err
 }
 
@@ -24,7 +24,7 @@ func (vr bytesValueRenderer) Parse(_ context.Context, r io.Reader) (protoreflect
 		return protoreflect.ValueOfBytes([]byte{}), err
 	}
 
-	data, err := base64.StdEncoding.DecodeString(string(formatted))
+	data, err := hex.DecodeString(string(formatted))
 	if err != nil {
 		return protoreflect.ValueOfBytes([]byte{}), err
 	}

tx/textual/valuerenderer/bytes_test.go

@@ -2,7 +2,7 @@ package valuerenderer_test
 
 import (
 	"context"
-	"encoding/hex"
+	"encoding/base64"
 	"encoding/json"
 	"os"
 	"strings"
@@ -14,14 +14,16 @@ import (
 
 func TestFormatBytes(t *testing.T) {
 	var testcases []bytesTest
+	// Bytes.json contains bytes that are represented in base64 format, and
+	// their expected results in hex.
 	raw, err := os.ReadFile("../internal/testdata/bytes.json")
 	require.NoError(t, err)
 
 	err = json.Unmarshal(raw, &testcases)
 	require.NoError(t, err)
 
 	for _, tc := range testcases {
-		data, err := hex.DecodeString(tc.hex)
+		data, err := base64.StdEncoding.DecodeString(tc.base64)
 		require.NoError(t, err)
 
 		r, err := valueRendererOf(data)
@@ -30,16 +32,16 @@ func TestFormatBytes(t *testing.T) {
 		b := new(strings.Builder)
 		err = r.Format(context.Background(), protoreflect.ValueOfBytes(data), b)
 		require.NoError(t, err)
-		require.Equal(t, tc.expRes, b.String())
+		require.Equal(t, tc.hex, b.String())
 	}
 }
 
 type bytesTest struct {
 	hex    string
-	expRes string
+	base64 string
 }
 
 func (t *bytesTest) UnmarshalJSON(b []byte) error {
-	a := []interface{}{&t.hex, &t.expRes}
+	a := []interface{}{&t.hex, &t.base64}
 	return json.Unmarshal(b, &a)
 }