fix(x/tx): default to using gogoproto.HybridResolver wherever possible

store/database.go

@@ -18,20 +18,20 @@ type Reader interface {
 	//
 	// Note: <key> is safe to modify and read after calling Get.
 	// The returned byte slice is safe to read, but cannot be modified.
-	Get(storeKey []byte, key []byte) ([]byte, error)
+	Get(storeKey, key []byte) ([]byte, error)
 }
 
 // Writer wraps the Set method of a backing data store.
 type Writer interface {
 	// Set inserts the given value into the key-value data store.
 	//
 	// Note: <key, value> are safe to modify and read after calling Set.
-	Set(storeKey []byte, key, value []byte) error
+	Set(storeKey, key, value []byte) error
 
 	// Delete removes the key from the backing key-value data store.
 	//
 	// Note: <key> is safe to modify and read after calling Delete.
-	Delete(storeKey []byte, key []byte) error
+	Delete(storeKey, key []byte) error
 }
 
 // Database contains all the methods required to allow handling different

store/internal/encoding/changeset_test.go

@@ -3,8 +3,9 @@ package encoding
 import (
 	"testing"
 
-	corestore "cosmossdk.io/core/store"
 	"github.com/stretchr/testify/require"
+
+	corestore "cosmossdk.io/core/store"
 )
 
 func TestChangesetMarshal(t *testing.T) {

store/storage/pebbledb/batch.go

@@ -57,11 +57,11 @@ func (b *Batch) set(storeKey []byte, tombstone uint64, key, value []byte) error
 	return nil
 }
 
-func (b *Batch) Set(storeKey []byte, key, value []byte) error {
+func (b *Batch) Set(storeKey, key, value []byte) error {
 	return b.set(storeKey, 0, key, value)
 }
 
-func (b *Batch) Delete(storeKey []byte, key []byte) error {
+func (b *Batch) Delete(storeKey, key []byte) error {
 	return b.set(storeKey, b.version, key, []byte(tombstoneVal))
 }
 

store/storage/pebbledb/db.go

@@ -319,7 +319,7 @@ func storePrefix(storeKey []byte) []byte {
 	return append([]byte(StorePrefixTpl), storeKey...)
 }
 
-func prependStoreKey(storeKey []byte, key []byte) []byte {
+func prependStoreKey(storeKey, key []byte) []byte {
 	return append(storePrefix(storeKey), key...)
 }
 
@@ -362,7 +362,7 @@ func valTombstoned(value []byte) bool {
 	return true
 }
 
-func getMVCCSlice(db *pebble.DB, storeKey []byte, key []byte, version uint64) ([]byte, error) {
+func getMVCCSlice(db *pebble.DB, storeKey, key []byte, version uint64) ([]byte, error) {
 	// end domain is exclusive, so we need to increment the version by 1
 	if version < math.MaxUint64 {
 		version++

store/storage/sqlite/batch.go

@@ -62,13 +62,13 @@ func (b *Batch) Reset() error {
 	return nil
 }
 
-func (b *Batch) Set(storeKey []byte, key, value []byte) error {
+func (b *Batch) Set(storeKey, key, value []byte) error {
 	b.size += len(key) + len(value)
 	b.ops = append(b.ops, batchOp{action: batchActionSet, storeKey: storeKey, key: key, value: value})
 	return nil
 }
 
-func (b *Batch) Delete(storeKey []byte, key []byte) error {
+func (b *Batch) Delete(storeKey, key []byte) error {
 	b.size += len(key)
 	b.ops = append(b.ops, batchOp{action: batchActionDel, storeKey: storeKey, key: key})
 	return nil

store/storage/sqlite/db_test.go

@@ -13,9 +13,7 @@ import (
 	"cosmossdk.io/store/v2/storage"
 )
 
-var (
-	storeKey1 = []byte("store1")
-)
+var storeKey1 = []byte("store1")
 
 func TestStorageTestSuite(t *testing.T) {
 	s := &storage.StorageTestSuite{

x/tx/signing/aminojson/aminojson.go

@@ -5,6 +5,7 @@ import (
 	"errors"
 	"fmt"
 
+	gogoproto "github.com/cosmos/gogoproto/proto"
 	"google.golang.org/protobuf/reflect/protoregistry"
 
 	signingv1beta1 "cosmossdk.io/api/cosmos/tx/signing/v1beta1"
@@ -31,7 +32,7 @@ type SignModeHandlerOptions struct {
 func NewSignModeHandler(options SignModeHandlerOptions) *SignModeHandler {
 	h := &SignModeHandler{}
 	if options.FileResolver == nil {
-		h.fileResolver = protoregistry.GlobalFiles
+		h.fileResolver = gogoproto.HybridResolver
 	} else {
 		h.fileResolver = options.FileResolver
 	}

x/tx/signing/aminojson/json_marshal.go

@@ -7,6 +7,7 @@ import (
 	"io"
 	"sort"
 
+	gogoproto "github.com/cosmos/gogoproto/proto"
 	"github.com/pkg/errors"
 	"google.golang.org/protobuf/proto"
 	"google.golang.org/protobuf/reflect/protoreflect"
@@ -55,7 +56,7 @@ type Encoder struct {
 // rules.
 func NewEncoder(options EncoderOptions) Encoder {
 	if options.FileResolver == nil {
-		options.FileResolver = protoregistry.GlobalFiles
+		options.FileResolver = gogoproto.HybridResolver
 	}
 	if options.TypeResolver == nil {
 		options.TypeResolver = protoregistry.GlobalTypes

x/tx/signing/context.go

@@ -5,6 +5,7 @@ import (
 	"fmt"
 
 	cosmos_proto "github.com/cosmos/cosmos-proto"
+	gogoproto "github.com/cosmos/gogoproto/proto"
 	"google.golang.org/protobuf/proto"
 	"google.golang.org/protobuf/reflect/protodesc"
 	"google.golang.org/protobuf/reflect/protoreflect"
@@ -79,7 +80,7 @@ type ProtoFileResolver interface {
 func NewContext(options Options) (*Context, error) {
 	protoFiles := options.FileResolver
 	if protoFiles == nil {
-		protoFiles = protoregistry.GlobalFiles
+		protoFiles = gogoproto.HybridResolver
 	}
 
 	protoTypes := options.TypeResolver

x/tx/signing/textual/handler.go

@@ -8,6 +8,7 @@ import (
 	"reflect"
 
 	cosmos_proto "github.com/cosmos/cosmos-proto"
+	gogoproto "github.com/cosmos/gogoproto/proto"
 	"google.golang.org/protobuf/proto"
 	"google.golang.org/protobuf/reflect/protoreflect"
 	"google.golang.org/protobuf/reflect/protoregistry"
@@ -71,7 +72,7 @@ func NewSignModeHandler(o SignModeOptions) (*SignModeHandler, error) {
 		return nil, errors.New("coinMetadataQuerier must be non-empty")
 	}
 	if o.FileResolver == nil {
-		o.FileResolver = protoregistry.GlobalFiles
+		o.FileResolver = gogoproto.HybridResolver
 	}
 	if o.TypeResolver == nil {
 		o.TypeResolver = protoregistry.GlobalTypes