Consensus Check: Upgrade Governance Contract to Governor Bravo

TL;DR:

Upgrade Uniswap’s current governance contract from Governor Alpha to Governor Bravo to improve governance upgradability and protocol safety.

Temperature Check
Temperature Check Snapshot Poll

Summary and Motivation:

Co-written with Getty Hill (@Getty), Eddy Lee (@elee), Yuan Han Li (@yuan-han-li), John Wang (@johnwang), and Ali Khambati (@alikhambati)

Governor Alpha, the current governance contract used, was a great start, but in light of Compound’s and other protocols upgrade to Governor Bravo, Uniswap should migrate given Bravo’s additional safety benefits and upgradability.

  1. Native upgradeability: Under Governor Alpha, changes to governance parameters require deploying a new contract and completely migrating to a new address. This is particularly damaging to governance participation as it introduces downtime and asynchronicity. Many governance tools and custodians use factory contracts which point to a specific contract address, meaning parties must manually upgrade their infrastructure every time governance parameters are changed under Governor Alpha. This includes tools for creating autonomous proposals like fish.vote ; front-ends such as Tally , Sybil, and Boardroom which aggregate and display governance results for various protocols; and professional custodians which are used by large token holders, delegates, and community members. Enabling a static contract address that has proxy upgradability patterns is paramount for maximizing participation, and maintaining an open and secure governance process.
  2. Voting reason string: Governor Bravo gives voters the opportunity to add free-form comments (text strings) along with their on-chain votes. Not only does this increase the transparency around the rationale people have behind their votes, but it also facilitates more in-depth and nuanced discussion.
  3. New ‘abstain’ voting option: Governor Bravo enables voters to formally abstain rather than forcing them to choose between voting yes/no or not voting at all. This will give voters more flexibility and also increase transparency into delegate behavior.
  4. Proposal numbers won’t reset: Under Governor Alpha, any upgrades to the contract resets the proposal number schema. Notice that ‘Proposal 0.4’ (which resulted in deployment and migration to a new Governor Alpha contract due to modifying the proposal submission threshold parameter) caused the following on-chain proposal from @HarvardLawBFI to be numbered ‘Proposal 1.1’ . Under Governor Bravo, this would not be an issue and proposal numbers would be continuous because the contract would be maintained at a single address.
  5. Proposal Cancellation: Bravo allows user-directed cancellations enabling erroneous proposals to be canceled if need be (rather than forcing people to vote no/abstain).
  6. Review Period: Governor Bravo allows governance to include a review/analysis period. Currently, Compound uses a 13140 block (~2-day) review period that allows holders to review the proposal. This means that users will have 2-days to prepare for voting (e.g., remove UNI from Aave), unlike Governor Alpha which instantly snapshots users’ voting power. A review period substantially improves the accessibility and safety of the governance process.

Implementation:

After speaking with the OpenZeppelin team about their reimplementation of Governor Bravo, we think using Compound’s Governor Bravo contract instead makes the most sense at this stage.

Although Open Zeppelin’s contract takes a simpler approach that can include all Bravo functionality, it does not do so by default. The contract uses similar logic, but the code is new. It has gone through an internal audit and review process but still needs to go through an external audit.

For the safety and simplicity of Uniswap, we think using Compound’s Governor Bravo contract is the best decision for the time being. Compound’s Bravo contract has already undergone an audit by OpenZeppelin and has been widely used. We can always change/upgrade to Open Zeppelin’s contract at a future date if their contract introduces new features/functionality the community is interested in. More details on OpenZeppelin’s code can be found here:
Github
Docs
More info

To allow the community to review our code, we have deployed the contract on the Ropsten test net. If this Consensus Check passes, we will then deploy the contract on mainnet for the formal on-chain proposal. Ropsten links:
Governor Bravo Delegator
Governor Bravo Delegate
Github for contracts

NB: The _initiate function has been slightly modified to take an initial proposal number as an input rather than pulling it from Governor Alpha. This way Uniswap can resume proper proposal numbering.

Snapshot:

The Snap Poll will be live for voting from 8AM EST on August 9th until 8PM EST on August 13th. If the poll passes with a minimum of 50,000 UNI in support, this proposal will move forward to the final stages.

A link to our snapshot proposal: Snapshot

Resource links:

  1. Governor Bravo Development - Protocol Development - Compound Community Forum
  2. Understanding Governor Bravo. A review of key changes versus… | by monetsupply | Tally | Jul, 2021 | Medium
  3. Compound | Proposal Detail #42
  4. Compound | Proposal Detail #43
6 Likes

Is there any particular reason quadratic voting hasn’t been included/considered for this proposal?

This might be the best time to be looking at implementing it if there ever was one…

Quadratic voting is a great mechanism used by projects like Gitcoin, however, it requires some level of KYC/verification of individuality. Without this, quadratic voting mechanisms are exposed to Sybil attacks where users could make multiple accounts and exploit the weighting of votes. With the open, anonymized state of Uniswap governance, it does not make sense to implement quadratic voting at this stage.

3 Likes