Skip to content

Council support#86

Draft
thelostone-mc wants to merge 4 commits intomainfrom
council-support
Draft

Council support#86
thelostone-mc wants to merge 4 commits intomainfrom
council-support

Conversation

@thelostone-mc
Copy link
Contributor

@thelostone-mc thelostone-mc commented Jan 21, 2026

  • Add CouncilExecutor with AccessManager support
  • Adds CouncilExecutor and deployment scripts for council governance.

Use case:
Multiple council governors need to schedule and execute proposals through the same Compound timelock via the shim.

Proposal lifecycle flow:
CouncilGovernor → VetoGovernor → CouncilExecutor → Shim → CompoundTimelock

Note:

  • VetoGovernor uses OZ-style governance while the shim uses Compound-style.
  • CouncilExecutor bridges compatibility between the two.

Architecture:

  • All CouncilExecutors must be whitelisted as trusted executors on the Shim
  • All CouncilExecutors share a single AccessManager
  • Each CouncilExecutor must be configured with allowed (target, selector) pairs

Scripts:

  • 4_DeployAccessManager.s.sol - Deploy shared AccessManager (once)
  • 5_DeployCouncilExecutor.s.sol - Deploy CouncilExecutor (per governor)
  • And then configure access on AccessManager

@github-actions
Copy link

github-actions bot commented Feb 3, 2026

Coverage after merging council-support into main will be

77.50%

Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
src
   CouncilExecutor.sol0%0%0%0%106, 116, 116–117, 120, 129, 138, 138–139, 142, 150, 157, 160, 160–161, 165–166, 169, 171, 173, 175, 180, 183, 183–184, 188, 191, 196, 203, 206–207, 209, 209–210, 214, 214–215, 219, 219–220, 224–226, 230, 230–231, 235, 238, 238–239, 244, 252–253, 256, 256–257, 260, 263, 266, 269, 277–278, 280, 280–281, 71–72, 72–73, 82–83, 86–87, 96–97
   RollbackManager.sol100%100%100%100%
   RollbackManagerTimelockCompound.sol100%100%100%100%
   RollbackManagerTimelockControl.sol100%100%100%100%
   TimelockMultiAdminShim.sol100%100%100%100%

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant