###### tags: `Cosmos SDK` # SDK Architecture Review - 2021-06-18 The **SDK Architecture Review** meeting is a bi-weekly meeting focused on architecture discussions of ADRs and features currently in development (any issues tagged with the label "Status: Needs Architecture Review"). ## Agenda - [x] Working Group Check In - https://github.com/cosmos/cosmos-sdk/wiki/Architecture-Design-Process - [x] Add URI x/bank denom metadata - https://github.com/cosmos/cosmos-sdk/issues/9502 - [x] Add ADR-043 NFT Module ? - https://github.com/cosmos/cosmos-sdk/pull/9329 - https://github.com/cosmos/cosmos-sdk/pull/9455 - [x] Minimal x/gov & x/group Alignment - https://github.com/cosmos/cosmos-sdk/issues/9438 - [x] Add transaction tips (aka "meta transactions") - https://github.com/cosmos/cosmos-sdk/issues/9406 - [x] ADR-33 and ADR-34 - [x] Client Applications - [x] Protobuf Versioning ## Notes ### Working Group Check In - Consensus Fees (#8224) - Gov/Group Module Alignment (#9066) - Module Wiring (#9182) - x/auth, x/bank and vesting re-design (#9336) - Protobuf golang v2 migration and codegen (#7488) - Offline signatures (#7896) ### Add URI x/bank denom metadata No major refactoring needed for the bank module to make this work. ### Minimal x/gov & x/group Alignment Currently in the gov module there are proposals with handlers and routers. We are planning to deprecate gov proposals and replace with regular messages because the auth proposal works with regular messages. We would add three new messages that would only be exucatable by the root of the gov module. Execute message on behalf of the gov module. Aaron, Marie, Calem, and Sam is current working group. If there are no major objections to this change, Calem will begin implementing. The proposal handlers might still be kept for one version but it might not be necessary. The gov propsal would include a type and description and a list of messages (same as group module). This should not require a larger refactoring of params. ### Add transaction tips (aka "meta transactions") We would be adding a new auxilary sign mode that would allow a signer to sign the body and to add a tip. This would solve two problems: - the ability for somebody to pay for transactions in a denom supported by a chain that they are not making the transaction on - any transaction with multiple signers is difficult because you need to know the other signers, you need all signers to sign, preventing the need for signers to sign again, only the final signer paying the fee needs to know about all the other signers This is high priority for Gravity Bridge. We would need to get this live by Fall for Cosmos Hub. ### Add ADR-043 NFT Module There is an ongoing NFT working group discussing ADR-043. The strategy is to utilize bank as much as possible as an asset manager. There is still no clear consensus on the strategy to utilize the bank module and the working group is open to opinion. This strategy still needs feedback from other community members who shared their opinion in the proposed ADR. ### ADR-33 and ADR-34 ### Client Applications ### Protocol Buffers Versioning We deleting reserved fields and reverted these changes. We will not change an existing proto file. We will be writing documentation to better describe best practices when writing and versioning proto files. There is a deprecated option that we will be using moving forward. We will be using `v1beta1` and holding off on bumping any proto files to `v1`. ## Follow Up - [ ] Add NFT Working Group - [ ] Draft ADR for metatransactions (ccoirdinate with Aaron)