feat(x/slashing): add autocli options for tx (cosmos#17967)

client/v2/autocli/flag/builder.go

@@ -23,6 +23,12 @@ import (
 	"github.com/cosmos/cosmos-sdk/runtime"
 )
 
+const (
+	AddressStringScalarType          = "cosmos.AddressString"
+	ValidatorAddressStringScalarType = "cosmos.ValidatorAddressString"
+	ConsensusAddressStringScalarType = "cosmos.ConsensusAddressString"
+)
+
 // Builder manages options for building pflag flags for protobuf messages.
 type Builder struct {
 	// TypeResolver specifies how protobuf types will be resolved. If it is
@@ -61,9 +67,9 @@ func (b *Builder) init() {
 
 	if b.scalarFlagTypes == nil {
 		b.scalarFlagTypes = map[string]Type{}
-		b.scalarFlagTypes["cosmos.AddressString"] = addressStringType{}
-		b.scalarFlagTypes["cosmos.ValidatorAddressString"] = validatorAddressStringType{}
-		b.scalarFlagTypes["cosmos.ConsensusAddressString"] = consensusAddressStringType{}
+		b.scalarFlagTypes[AddressStringScalarType] = addressStringType{}
+		b.scalarFlagTypes[ValidatorAddressStringScalarType] = validatorAddressStringType{}
+		b.scalarFlagTypes[ConsensusAddressStringScalarType] = consensusAddressStringType{}
 	}
 }
 
@@ -387,10 +393,10 @@ func (b *Builder) resolveFlagType(field protoreflect.FieldDescriptor) Type {
 }
 
 func (b *Builder) resolveFlagTypeBasic(field protoreflect.FieldDescriptor) Type {
-	scalar := proto.GetExtension(field.Options(), cosmos_proto.E_Scalar)
-	if scalar != nil {
+	scalar, ok := GetScalarType(field)
+	if ok {
 		b.init()
-		if typ, ok := b.scalarFlagTypes[scalar.(string)]; ok {
+		if typ, ok := b.scalarFlagTypes[scalar]; ok {
 			return typ
 		}
 	}
@@ -413,6 +419,13 @@ func (b *Builder) resolveFlagTypeBasic(field protoreflect.FieldDescriptor) Type
 	}
 }
 
+// GetScalarType gets scalar type of a field.
+func GetScalarType(field protoreflect.FieldDescriptor) (string, bool) {
+	scalar := proto.GetExtension(field.Options(), cosmos_proto.E_Scalar)
+	scalarStr, ok := scalar.(string)
+	return scalarStr, ok
+}
+
 // GetSignerFieldName gets signer field name of a message.
 // AutoCLI supports only one signer field per message.
 func GetSignerFieldName(descriptor protoreflect.MessageDescriptor) string {

client/v2/autocli/msg.go

@@ -131,8 +131,21 @@ func (b *Builder) BuildMsgMethodCommand(descriptor protoreflect.MethodDescriptor
 		// set signer to signer field if empty
 		fd := input.Descriptor().Fields().ByName(protoreflect.Name(flag.GetSignerFieldName(input.Descriptor())))
 		if addr := input.Get(fd).String(); addr == "" {
+			addressCodec := b.Builder.AddressCodec
+
+			scalarType, ok := flag.GetScalarType(fd)
+			if ok {
+				// override address codec if validator or consensus address
+				switch scalarType {
+				case flag.ValidatorAddressStringScalarType:
+					addressCodec = b.Builder.ValidatorAddressCodec
+				case flag.ConsensusAddressStringScalarType:
+					addressCodec = b.Builder.ConsensusAddressCodec
+				}
+			}
+
 			signerFromFlag := clientCtx.GetFromAddress()
-			signer, err := b.ClientCtx.AddressCodec.BytesToString(signerFromFlag.Bytes())
+			signer, err := addressCodec.BytesToString(signerFromFlag.Bytes())
 			if err != nil {
 				return fmt.Errorf("failed to set signer on message, got %v: %w", signerFromFlag, err)
 			}

x/slashing/autocli.go

@@ -36,5 +36,20 @@ func (am AppModule) AutoCLIOptions() *autocliv1.ModuleOptions {
 				},
 			},
 		},
+		Tx: &autocliv1.ServiceCommandDescriptor{
+			Service: slashingv1beta.Msg_ServiceDesc.ServiceName,
+			RpcCommandOptions: []*autocliv1.RpcCommandOptions{
+				{
+					RpcMethod: "Unjail",
+					Use:       "unjail",
+					Short:     "Unjail a jailed validator",
+					Example:   fmt.Sprintf("%s tx slashing unjail --from [validator]", version.AppName),
+				},
+				{
+					RpcMethod: "UpdateParams",
+					Skip:      true, // skipped because authority gated
+				},
+			},
+		},
 	}
 }

x/slashing/module.go

@@ -6,7 +6,6 @@ import (
 	"fmt"
 
 	gwruntime "github.com/grpc-ecosystem/grpc-gateway/runtime"
-	"github.com/spf13/cobra"
 
 	modulev1 "cosmossdk.io/api/cosmos/slashing/module/v1"
 	"cosmossdk.io/core/appmodule"
@@ -20,7 +19,6 @@ import (
 	simtypes "github.com/cosmos/cosmos-sdk/types/simulation"
 	authtypes "github.com/cosmos/cosmos-sdk/x/auth/types"
 	govtypes "github.com/cosmos/cosmos-sdk/x/gov/types"
-	"github.com/cosmos/cosmos-sdk/x/slashing/client/cli"
 	"github.com/cosmos/cosmos-sdk/x/slashing/keeper"
 	"github.com/cosmos/cosmos-sdk/x/slashing/simulation"
 	"github.com/cosmos/cosmos-sdk/x/slashing/types"
@@ -83,11 +81,6 @@ func (AppModuleBasic) RegisterGRPCGatewayRoutes(clientCtx client.Context, mux *g
 	}
 }
 
-// GetTxCmd returns the root tx command for the slashing module.
-func (amb AppModuleBasic) GetTxCmd() *cobra.Command {
-	return cli.NewTxCmd(amb.cdc.InterfaceRegistry().SigningContext().ValidatorAddressCodec())
-}
-
 // AppModule implements an application module for the slashing module.
 type AppModule struct {
 	AppModuleBasic