feat: support message based proposals (backport #1372) (#1373)

* feat: support message based proposals (#1372) * feat: change MakeSwapProposal to foundation proposal * chore: refactor MsgMakeSwapProposal -> MsgSetSwap * chore: clean-up * chore: use original proto stringer interface * chore: remove unused proto annotations * chore: allow gov and foundation as authority address * chore: lint fix * chore: remove redundant part * chore: update changelog (cherry picked from commit 97570b2) # Conflicts: # x/fswap/keeper/proposal.go * chore: fix conflict --------- Co-authored-by: jaeseung-bae <[email protected]> Co-authored-by: jaeseung.bae <[email protected]> Co-authored-by: zemyblue <[email protected]>
Finschia · May 9, 2024 · fb1c2b0 · fb1c2b0
1 parent aaf3163
commit fb1c2b0
Show file tree

Hide file tree

Showing 23 changed files with 996 additions and 1,077 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -52,6 +52,7 @@ Ref: https://keepachangelog.com/en/1.0.0/
 * (x/fswap) [\#1363](https://github.com/Finschia/finschia-sdk/pull/1363) introduce new event for MakeSwapProposal
 * (x/fbridge) [\#1366](https://github.com/Finschia/finschia-sdk/pull/1366) Set target denom as module parameters
 * (x/fbridge) [\#1369](https://github.com/Finschia/finschia-sdk/pull/1369) Add the event of `SetBridgeStatus`
+* (x/fswap) [\#1372](https://github.com/Finschia/finschia-sdk/pull/1372) support message based proposals
 
 ### Bug Fixes
 * (x/auth) [#1281](https://github.com/Finschia/finschia-sdk/pull/1281) `ModuleAccount.Validate` now reports a nil `.BaseAccount` instead of panicking. (backport #1274)

diff --git a/docs/core/proto-docs.md b/docs/core/proto-docs.md
@@ -978,7 +978,6 @@
     - [Msg](#lbm.foundation.v1.Msg)
 
 - [lbm/fswap/v1/fswap.proto](#lbm/fswap/v1/fswap.proto)
-    - [MakeSwapProposal](#lbm.fswap.v1.MakeSwapProposal)
     - [Swap](#lbm.fswap.v1.Swap)
     - [SwapStats](#lbm.fswap.v1.SwapStats)
     - [Swapped](#lbm.fswap.v1.Swapped)
@@ -1002,6 +1001,8 @@
     - [Query](#lbm.fswap.v1.Query)
 
 - [lbm/fswap/v1/tx.proto](#lbm/fswap/v1/tx.proto)
+    - [MsgSetSwap](#lbm.fswap.v1.MsgSetSwap)
+    - [MsgSetSwapResponse](#lbm.fswap.v1.MsgSetSwapResponse)
     - [MsgSwap](#lbm.fswap.v1.MsgSwap)
     - [MsgSwapAll](#lbm.fswap.v1.MsgSwapAll)
     - [MsgSwapAllResponse](#lbm.fswap.v1.MsgSwapAllResponse)
@@ -14555,24 +14556,6 @@ Msg defines the foundation Msg service.
 
 
 
-<a name="lbm.fswap.v1.MakeSwapProposal"></a>
-
-### MakeSwapProposal
-From cosmos-sdk 0.46.0 they deprecated this way, but currently finschia-sdk based on 0.45.10
-
-
-| Field | Type | Label | Description |
-| ----- | ---- | ----- | ----------- |
-| `title` | [string](#string) |  |  |
-| `description` | [string](#string) |  |  |
-| `swap` | [Swap](#lbm.fswap.v1.Swap) |  |  |
-| `to_denom_metadata` | [cosmos.bank.v1beta1.Metadata](#cosmos.bank.v1beta1.Metadata) |  |  |
-
-
-
-
-
-
 <a name="lbm.fswap.v1.Swap"></a>
 
 ### Swap
@@ -14856,6 +14839,33 @@ GenesisState defines the fswap module's genesis state.
 
 
 
+<a name="lbm.fswap.v1.MsgSetSwap"></a>
+
+### MsgSetSwap
+
+
+
+| Field | Type | Label | Description |
+| ----- | ---- | ----- | ----------- |
+| `authority` | [string](#string) |  | authority is the address of the privileged account. |
+| `swap` | [Swap](#lbm.fswap.v1.Swap) |  |  |
+| `to_denom_metadata` | [cosmos.bank.v1beta1.Metadata](#cosmos.bank.v1beta1.Metadata) |  |  |
+
+
+
+
+
+
+<a name="lbm.fswap.v1.MsgSetSwapResponse"></a>
+
+### MsgSetSwapResponse
+
+
+
+
+
+
+
 <a name="lbm.fswap.v1.MsgSwap"></a>
 
 ### MsgSwap
@@ -14925,6 +14935,7 @@ GenesisState defines the fswap module's genesis state.
 | ----------- | ------------ | ------------- | ------------| ------- | -------- |
 | `Swap` | [MsgSwap](#lbm.fswap.v1.MsgSwap) | [MsgSwapResponse](#lbm.fswap.v1.MsgSwapResponse) |  | |
 | `SwapAll` | [MsgSwapAll](#lbm.fswap.v1.MsgSwapAll) | [MsgSwapAllResponse](#lbm.fswap.v1.MsgSwapAllResponse) |  | |
+| `SetSwap` | [MsgSetSwap](#lbm.fswap.v1.MsgSetSwap) | [MsgSetSwapResponse](#lbm.fswap.v1.MsgSetSwapResponse) |  | |
 
  <!-- end services -->
 

diff --git a/proto/lbm/fswap/v1/fswap.proto b/proto/lbm/fswap/v1/fswap.proto
@@ -5,12 +5,8 @@ option go_package = "github.com/Finschia/finschia-sdk/x/fswap/types";
 
 import "gogoproto/gogo.proto";
 import "cosmos/base/v1beta1/coin.proto";
-import "cosmos/bank/v1beta1/bank.proto";
 
 message Swap {
-  option (gogoproto.equal)            = true;
-  option (gogoproto.goproto_stringer) = false;
-
   string from_denom              = 1;
   string to_denom                = 2;
   string amount_cap_for_to_denom = 3
@@ -20,25 +16,10 @@ message Swap {
 }
 
 message SwapStats {
-  option (gogoproto.equal)            = true;
-  option (gogoproto.goproto_stringer) = false;
-
   int32 swap_count = 1;
 }
 
-// From cosmos-sdk 0.46.0 they deprecated this way, but currently finschia-sdk based on 0.45.10
-message MakeSwapProposal {
-  option (gogoproto.goproto_stringer) = false;
-
-  string                       title             = 1;
-  string                       description       = 2;
-  Swap                         swap              = 3 [(gogoproto.nullable) = false];
-  cosmos.bank.v1beta1.Metadata to_denom_metadata = 4
-      [(gogoproto.moretags) = "yaml:\"denom_metadata\"", (gogoproto.nullable) = false];
-}
-
 message Swapped {
-  option (gogoproto.goproto_stringer)       = false;
   cosmos.base.v1beta1.Coin from_coin_amount = 1
       [(gogoproto.castrepeated) = "github.com/Finschia/finschia-sdk/types.Coin", (gogoproto.nullable) = false];
   cosmos.base.v1beta1.Coin to_coin_amount = 2

diff --git a/proto/lbm/fswap/v1/tx.proto b/proto/lbm/fswap/v1/tx.proto
@@ -5,10 +5,13 @@ option go_package = "github.com/Finschia/finschia-sdk/x/fswap/types";
 
 import "gogoproto/gogo.proto";
 import "cosmos/base/v1beta1/coin.proto";
+import "cosmos/bank/v1beta1/bank.proto";
+import "lbm/fswap/v1/fswap.proto";
 
 service Msg {
   rpc Swap(MsgSwap) returns (MsgSwapResponse);
   rpc SwapAll(MsgSwapAll) returns (MsgSwapAllResponse);
+  rpc SetSwap(MsgSetSwap) returns (MsgSetSwapResponse);
 }
 
 message MsgSwap {
@@ -30,3 +33,13 @@ message MsgSwapAll {
 }
 
 message MsgSwapAllResponse {}
+
+message MsgSetSwap {
+  // authority is the address of the privileged account.
+  string  authority = 1;
+  Swap                         swap              = 2 [(gogoproto.nullable) = false];
+  cosmos.bank.v1beta1.Metadata to_denom_metadata = 3
+  [(gogoproto.moretags) = "yaml:\"denom_metadata\"", (gogoproto.nullable) = false];
+}
+
+message MsgSetSwapResponse {}
diff --git a/simapp/app.go b/simapp/app.go
@@ -78,7 +78,6 @@ import (
 	foundationkeeper "github.com/Finschia/finschia-sdk/x/foundation/keeper"
 	foundationmodule "github.com/Finschia/finschia-sdk/x/foundation/module"
 	"github.com/Finschia/finschia-sdk/x/fswap"
-	fswapclient "github.com/Finschia/finschia-sdk/x/fswap/client"
 	fswapkeeper "github.com/Finschia/finschia-sdk/x/fswap/keeper"
 	fswaptypes "github.com/Finschia/finschia-sdk/x/fswap/types"
 	"github.com/Finschia/finschia-sdk/x/genutil"
@@ -139,7 +138,6 @@ var (
 			upgradeclient.ProposalHandler,
 			upgradeclient.CancelProposalHandler,
 			foundationclient.ProposalHandler,
-			fswapclient.ProposalHandler,
 		),
 		params.AppModuleBasic{},
 		crisis.AppModuleBasic{},
@@ -349,16 +347,15 @@ func NewSimApp(
 	app.AuthzKeeper = authzkeeper.NewKeeper(keys[authzkeeper.StoreKey], appCodec, app.BaseApp.MsgServiceRouter())
 
 	fswapConfig := fswaptypes.DefaultConfig()
-	app.FswapKeeper = fswapkeeper.NewKeeper(appCodec, keys[fswaptypes.StoreKey], fswapConfig, app.BankKeeper)
+	app.FswapKeeper = fswapkeeper.NewKeeper(appCodec, keys[fswaptypes.StoreKey], fswapConfig, fswaptypes.DefaultAuthority().String(), app.BankKeeper)
 
 	// register the proposal types
 	govRouter := govtypes.NewRouter()
 	govRouter.AddRoute(govtypes.RouterKey, govtypes.ProposalHandler).
 		AddRoute(paramproposal.RouterKey, params.NewParamChangeProposalHandler(app.ParamsKeeper)).
 		AddRoute(distrtypes.RouterKey, distr.NewCommunityPoolSpendProposalHandler(app.DistrKeeper)).
 		AddRoute(upgradetypes.RouterKey, upgrade.NewSoftwareUpgradeProposalHandler(app.UpgradeKeeper)).
-		AddRoute(foundation.RouterKey, foundationkeeper.NewFoundationProposalsHandler(app.FoundationKeeper)).
-		AddRoute(fswaptypes.RouterKey, fswap.NewSwapHandler(app.FswapKeeper))
+		AddRoute(foundation.RouterKey, foundationkeeper.NewFoundationProposalsHandler(app.FoundationKeeper))
 
 	govKeeper := govkeeper.NewKeeper(
 		appCodec, keys[govtypes.StoreKey], app.GetSubspace(govtypes.ModuleName), app.AccountKeeper, app.BankKeeper,

diff --git a/x/fswap/client/cli/tx.go b/x/fswap/client/cli/tx.go
@@ -13,8 +13,6 @@ import (
 	sdkerrors "github.com/Finschia/finschia-sdk/types/errors"
 	bank "github.com/Finschia/finschia-sdk/x/bank/types"
 	"github.com/Finschia/finschia-sdk/x/fswap/types"
-	govcli "github.com/Finschia/finschia-sdk/x/gov/client/cli"
-	gov "github.com/Finschia/finschia-sdk/x/gov/types"
 )
 
 const (
@@ -36,6 +34,7 @@ func GetTxCmd() *cobra.Command {
 	cmd.AddCommand(
 		CmdTxMsgSwap(),
 		CmdTxMsgSwapAll(),
+		CmdMsgSetSwap(),
 	)
 
 	return cmd
@@ -115,21 +114,21 @@ func CmdTxMsgSwapAll() *cobra.Command {
 	return cmd
 }
 
-// NewCmdMakeSwapProposal implements a command handler for submitting a swap init proposal transaction.
-func NewCmdMakeSwapProposal() *cobra.Command {
+// CmdMsgSetSwap implements a command handler for submitting a swap init proposal transaction.
+func CmdMsgSetSwap() *cobra.Command {
 	cmd := &cobra.Command{
-		Use:   "make-swap [messages-json]",
-		Args:  cobra.ExactArgs(1),
-		Short: "todo",
+		Use:   "set-swap [authority] [metadata-json]",
+		Args:  cobra.ExactArgs(2),
+		Short: "Set a swap",
 		Long: `
 Parameters:
-    messages-json: messages in json format that will be executed if the proposal is accepted.
+    metadata-json: messages in json format that will be executed if the proposal is accepted.
 
-Example of the content of messages-json:
+Example of the content of metadata-json:
 
 {
   "metadata": {
-    "description": "the base coin of Finschia mainnet",
+    "description": "example of to-denom is finschia cony",
     "denom_units": [
       {
         "denom": "cony",
@@ -152,19 +151,11 @@ Example of the content of messages-json:
 }
 `,
 		RunE: func(cmd *cobra.Command, args []string) error {
-			clientCtx, err := client.GetClientTxContext(cmd)
-			if err != nil {
-				return err
-			}
-
-			from := clientCtx.GetFromAddress()
-
-			title, err := cmd.Flags().GetString(govcli.FlagTitle)
-			if err != nil {
+			if err := validateGenerateOnly(cmd); err != nil {
 				return err
 			}
 
-			description, err := cmd.Flags().GetString(govcli.FlagDescription)
+			clientCtx, err := client.GetClientTxContext(cmd)
 			if err != nil {
 				return err
 			}
@@ -178,10 +169,12 @@ Example of the content of messages-json:
 			if err != nil {
 				return err
 			}
+
 			amountCapStr, err := cmd.Flags().GetString(FlagAmountCapForToDenom)
 			if err != nil {
 				return err
 			}
+
 			amountCap, ok := sdk.NewIntFromString(amountCapStr)
 			if !ok {
 				return sdkerrors.ErrInvalidRequest.Wrapf("failed to parse %s %s", FlagAmountCapForToDenom, amountCap.String())
@@ -202,42 +195,42 @@ Example of the content of messages-json:
 				SwapRate:            swapRateDec,
 			}
 
-			toDenomMetadata, err := parseToDenomMetadata(args[0])
-			if err != nil {
-				return err
-			}
-
-			content := types.NewMakeSwapProposal(title, description, swap, toDenomMetadata)
-
-			depositStr, err := cmd.Flags().GetString(govcli.FlagDeposit)
-			if err != nil {
-				return err
-			}
-			deposit, err := sdk.ParseCoinsNormalized(depositStr)
+			authority := args[0]
+			toDenomMetadata, err := parseToDenomMetadata(args[1])
 			if err != nil {
 				return err
 			}
 
-			msg, err := gov.NewMsgSubmitProposal(content, deposit, from)
-			if err != nil {
-				return err
+			msg := types.MsgSetSwap{
+				Authority:       authority,
+				Swap:            swap,
+				ToDenomMetadata: toDenomMetadata,
 			}
 
-			return tx.GenerateOrBroadcastTxCLI(clientCtx, cmd.Flags(), msg)
+			return tx.GenerateOrBroadcastTxCLI(clientCtx, cmd.Flags(), &msg)
 		},
 	}
 
-	cmd.Flags().String(govcli.FlagTitle, "", "title of proposal")
-	cmd.Flags().String(govcli.FlagDescription, "", "description of proposal")
-	cmd.Flags().String(govcli.FlagDeposit, "", "deposit of proposal")
-	cmd.Flags().String(FlagFromDenom, "", "cony")
-	cmd.Flags().String(FlagToDenom, "", "PDT")
-	cmd.Flags().String(FlagAmountCapForToDenom, "0", "tbd")
-	cmd.Flags().String(FlagSwapRate, "0", "tbd")
+	cmd.Flags().String(FlagFromDenom, "", "set fromDenom string, ex) cony")
+	cmd.Flags().String(FlagToDenom, "", "set toDenom string, ex) peb")
+	cmd.Flags().String(FlagAmountCapForToDenom, "0", "set integer value for limit cap for the amount to swap to to-denom, ex 1000000000")
+	cmd.Flags().String(FlagSwapRate, "0", "set swap rate for swap from fromDenom to toDenom, ex(rate for cony to peb)  148079656000000")
 
+	flags.AddTxFlagsToCmd(cmd)
 	return cmd
 }
 
+func validateGenerateOnly(cmd *cobra.Command) error {
+	generateOnly, err := cmd.Flags().GetBool(flags.FlagGenerateOnly)
+	if err != nil {
+		return err
+	}
+	if !generateOnly {
+		return fmt.Errorf("you must use it with the flag --%s", flags.FlagGenerateOnly)
+	}
+	return nil
+}
+
 func parseToDenomMetadata(jsonDenomMetadata string) (bank.Metadata, error) {
 	type toDenomMeta struct {
 		Metadata bank.Metadata `json:"metadata"`

diff --git a/x/fswap/client/proposal_handler.go b/x/fswap/client/proposal_handler.go
diff --git a/x/fswap/codec/codec.go b/x/fswap/codec/codec.go
diff --git a/x/fswap/handler.go b/x/fswap/handler.go