From 2a3678d24d6cb717b9d205d02edbb9f8c30ddab4 Mon Sep 17 00:00:00 2001
From: Dan Lynch <pyramation@gmail.com>
Date: Wed, 20 Jul 2022 02:10:33 -0700
Subject: [PATCH] fix: add cosmos_proto implements (#12639)

It seems that we're missing the `cosmos_proto.implements_interface` on many proposal messages/structs

this is a draft to discuss adding options in protos for better codegen tooling. It seems that the following messages/structs are meant to implement `Content`:

```
option (cosmos_proto.implements_interface) = "cosmos.gov.v1beta1.Content"
```

Would this be a breaking change? Happy to improve/amend this so it's mergeable.

related issue: https://github.com/cosmos/cosmos-sdk/issues/12640

(cherry picked from commit 64674ba9e55c87ff8926730222426284f6001c88)
---
 proto/cosmos/distribution/v1beta1/distribution.proto | 2 ++
 proto/cosmos/params/v1beta1/params.proto             | 2 ++
 proto/cosmos/upgrade/v1beta1/upgrade.proto           | 3 +++
 3 files changed, 7 insertions(+)

diff --git a/proto/cosmos/distribution/v1beta1/distribution.proto b/proto/cosmos/distribution/v1beta1/distribution.proto
index ad12bd6049fa..1afe25ae4ffb 100644
--- a/proto/cosmos/distribution/v1beta1/distribution.proto
+++ b/proto/cosmos/distribution/v1beta1/distribution.proto
@@ -102,6 +102,7 @@ message CommunityPoolSpendProposal {
   option (gogoproto.equal)            = false;
   option (gogoproto.goproto_getters)  = false;
   option (gogoproto.goproto_stringer) = false;
+  option (cosmos_proto.implements_interface) = "cosmos.gov.v1beta1.Content";
 
   string   title                           = 1;
   string   description                     = 2;
@@ -143,6 +144,7 @@ message DelegationDelegatorReward {
 message CommunityPoolSpendProposalWithDeposit {
   option (gogoproto.goproto_getters)  = false;
   option (gogoproto.goproto_stringer) = true;
+  option (cosmos_proto.implements_interface) = "cosmos.gov.v1beta1.Content";
 
   string title       = 1;
   string description = 2;
diff --git a/proto/cosmos/params/v1beta1/params.proto b/proto/cosmos/params/v1beta1/params.proto
index 5382fd799964..e5aabfeca4ca 100644
--- a/proto/cosmos/params/v1beta1/params.proto
+++ b/proto/cosmos/params/v1beta1/params.proto
@@ -5,11 +5,13 @@ option go_package            = "github.com/cosmos/cosmos-sdk/x/params/types/prop
 option (gogoproto.equal_all) = true;
 
 import "gogoproto/gogo.proto";
+import "cosmos_proto/cosmos.proto";
 
 // ParameterChangeProposal defines a proposal to change one or more parameters.
 message ParameterChangeProposal {
   option (gogoproto.goproto_getters)  = false;
   option (gogoproto.goproto_stringer) = false;
+  option (cosmos_proto.implements_interface) = "cosmos.gov.v1beta1.Content";
 
   string               title       = 1;
   string               description = 2;
diff --git a/proto/cosmos/upgrade/v1beta1/upgrade.proto b/proto/cosmos/upgrade/v1beta1/upgrade.proto
index 4de7d01fc304..9831c894d235 100644
--- a/proto/cosmos/upgrade/v1beta1/upgrade.proto
+++ b/proto/cosmos/upgrade/v1beta1/upgrade.proto
@@ -4,6 +4,7 @@ package cosmos.upgrade.v1beta1;
 import "google/protobuf/any.proto";
 import "gogoproto/gogo.proto";
 import "google/protobuf/timestamp.proto";
+import "cosmos_proto/cosmos.proto";
 
 option go_package                      = "github.com/cosmos/cosmos-sdk/x/upgrade/types";
 option (gogoproto.goproto_getters_all) = false;
@@ -47,6 +48,7 @@ message Plan {
 // proposals, see MsgSoftwareUpgrade.
 message SoftwareUpgradeProposal {
   option deprecated                   = true;
+  option (cosmos_proto.implements_interface) = "cosmos.gov.v1beta1.Content";
   option (gogoproto.equal)            = true;
   option (gogoproto.goproto_stringer) = false;
 
@@ -61,6 +63,7 @@ message SoftwareUpgradeProposal {
 // proposals, see MsgCancelUpgrade.
 message CancelSoftwareUpgradeProposal {
   option deprecated                   = true;
+  option (cosmos_proto.implements_interface) = "cosmos.gov.v1beta1.Content";
   option (gogoproto.equal)            = true;
   option (gogoproto.goproto_stringer) = false;