Upcoming refactors

[ValarDragon]

This is just a list of all the upcoming refactors that I am aware of. These are listed in the order I remembered them. Not all of these are written up into issues. Feel free to edit this issue directly. Lets try to get these all taken care of next iteration, so we can just punt all further big refactors until postlaunch / pre1.0? I no longer think these refactors are that important to have prelaunch

• client.CoreContext -> client.TxContext Client: Refactor CoreContext #1551
• Rename /x/ to something that implies what we mean. (Or alternatively document what exactly /x/ means)
• sdk.Context -> sdk.AppContext types: Rename context to AppContext #1747
• move auth somewhere else???
• move mock somewhere else / split up code between client and a simulation package
• decouple LCD and gaia
• multimsg support throughout cli. (is this prelaunch?)
• Reduce module dependence on each others type definitions. (@sunnya97 can you elaborate on this more)
• Remove modules from importing each other within their tests.
• New testing structure to only test handlers, without auth.
• Refactor Keybase api
• sdk.NewCoins should take in sdk.Int Default sdk.NewCoin to take sdk.Int  #1723
• Switch all error messages to be using error message tables Tabulate errors #1729 (may be pre1.0, not prelaunch)
• Change baseapp internal variables to use setters, and a seal method baseapp: Functional options should use setters and seal #1645
• Refactor tests with an integration test flag tests: Integration test flag #1665
• Switch coins.Amount to be sdk.Uint Switch coins.Amount to uint, consider using a deterministic map instead of a list #1273
• Remove baseapp.SetTxDecoder baseapp: NewBaseApp takes a Codec #1441
• Change staking keys to be unexposed Keys are declared as var and exposed #1261
• Switch certain tests to use golang sub-tests Define a test order in staking #1658
• use Cobra.ExactArgs in CLI UX: use Cobra.ExactArgs in CLI  #885 (mostly done I think?)
• Refactor client/context/helpers/{EnsureAccountNumber, EnsureSequenceNumber} to not re-get the account. (Just get it once)
• Rename unrevoke Rename revoke/unrevoke to jail/unjail #1305

[rigelrozanski]

• gov/cli/tx.go should be split up into flags.go/ query.go / tx.go

[alexanderbez]

Awesome! Thanks for this @ValarDragon

[rigelrozanski]

• Subpackages for x/modules #1779 Subpackages for x/modules

[ValarDragon]

Now that this stuff has been moved mostly to postlaunch, I don't think this summary issue remains helpful. Closing for now.