Add the deployerCoordinatorManager role to the HyperdriveFactory

contracts/src/factory/HyperdriveFactory.sol

@@ -35,9 +35,14 @@ contract HyperdriveFactory is IHyperdriveFactory {
     ///      received. Defaults to `RECEIVE_LOCKED`
     uint256 private receiveLockState = RECEIVE_LOCKED;
 
-    /// @notice The governance address that updates the factory's configuration.
+    /// @notice The governance address that updates the factory's configuration
+    ///         and can add or remove deployer coordinators.
     address public governance;
 
+    /// @notice The deployer coordinator manager that can add or remove deployer
+    ///         coordinators.
+    address public deployerCoordinatorManager;
+
     /// @notice The governance address used when new instances are deployed.
     address public hyperdriveGovernance;
 
@@ -105,6 +110,8 @@ contract HyperdriveFactory is IHyperdriveFactory {
     struct FactoryConfig {
         /// @dev The address which can update a factory.
         address governance;
+        /// @dev The address which can add and remove deployer coordinators.
+        address deployerCoordinatorManager;
         /// @dev The address which is set as the governor of hyperdrive.
         address hyperdriveGovernance;
         /// @dev The default addresses which will be set to have the pauser role.
@@ -304,6 +311,7 @@ contract HyperdriveFactory is IHyperdriveFactory {
 
         // Initialize the other parameters.
         governance = _factoryConfig.governance;
+        deployerCoordinatorManager = _factoryConfig.deployerCoordinatorManager;
         hyperdriveGovernance = _factoryConfig.hyperdriveGovernance;
         feeCollector = _factoryConfig.feeCollector;
         sweepCollector = _factoryConfig.sweepCollector;
@@ -322,6 +330,17 @@ contract HyperdriveFactory is IHyperdriveFactory {
         _;
     }
 
+    /// @dev Ensure that the sender is either the governance address or the
+    ///      deployer coordinator manager.
+    modifier onlyDeployerCoordinatorManager() {
+        if (
+            msg.sender != governance && msg.sender != deployerCoordinatorManager
+        ) {
+            revert IHyperdriveFactory.Unauthorized();
+        }
+        _;
+    }
+
     /// @notice Allows ether to be sent to the contract. This is gated by a lock
     ///         to prevent ether from becoming stuck in the contract.
     receive() external payable {
@@ -337,7 +356,18 @@ contract HyperdriveFactory is IHyperdriveFactory {
         emit GovernanceUpdated(_governance);
     }
 
-    /// @notice Allows governance to change the hyperdrive governance address
+    /// @notice Allows governance to change the deployer coordinator manager
+    ///         address.
+    /// @param _deployerCoordinatorManager The new deployer coordinator manager
+    ///        address.
+    function updateDeployerCoordinatorManager(
+        address _deployerCoordinatorManager
+    ) external onlyGovernance {
+        deployerCoordinatorManager = _deployerCoordinatorManager;
+        emit DeployerCoordinatorManagerUpdated(_deployerCoordinatorManager);
+    }
+
+    /// @notice Allows governance to change the hyperdrive governance address.
     /// @param _hyperdriveGovernance The new hyperdrive governance address.
     function updateHyperdriveGovernance(
         address _hyperdriveGovernance
@@ -648,7 +678,7 @@ contract HyperdriveFactory is IHyperdriveFactory {
     /// @param _deployerCoordinator The new deployer coordinator.
     function addDeployerCoordinator(
         address _deployerCoordinator
-    ) external onlyGovernance {
+    ) external onlyDeployerCoordinatorManager {
         if (isDeployerCoordinator[_deployerCoordinator]) {
             revert IHyperdriveFactory.DeployerCoordinatorAlreadyAdded();
         }
@@ -663,7 +693,7 @@ contract HyperdriveFactory is IHyperdriveFactory {
     function removeDeployerCoordinator(
         address _deployerCoordinator,
         uint256 _index
-    ) external onlyGovernance {
+    ) external onlyDeployerCoordinatorManager {
         if (!isDeployerCoordinator[_deployerCoordinator]) {
             revert IHyperdriveFactory.DeployerCoordinatorNotAdded();
         }

contracts/src/interfaces/IHyperdriveFactory.sol

@@ -33,6 +33,11 @@ interface IHyperdriveFactory {
     /// @notice Emitted when the factory's governance is updated.
     event GovernanceUpdated(address indexed governance);
 
+    /// @notice Emitted when the deployer coordinator manager is updated.
+    event DeployerCoordinatorManagerUpdated(
+        address indexed deployerCoordinatorManager
+    );
+
     /// @notice Emitted when the governance address used in new deployments is
     ///         updated.
     event HyperdriveGovernanceUpdated(address indexed hyperdriveGovernance);

test/instances/erc4626/ERC4626Hyperdrive.t.sol

@@ -74,6 +74,7 @@ contract ERC4626HyperdriveTest is HyperdriveTest {
         factory = new HyperdriveFactory(
             HyperdriveFactory.FactoryConfig({
                 governance: alice,
+                deployerCoordinatorManager: celine,
                 hyperdriveGovernance: bob,
                 feeCollector: feeCollector,
                 sweepCollector: sweepCollector,

test/instances/erc4626/ERC4626Validation.t.sol

@@ -53,6 +53,7 @@ abstract contract ERC4626ValidationTest is HyperdriveTest {
         factory = new HyperdriveFactory(
             HyperdriveFactory.FactoryConfig({
                 governance: alice,
+                deployerCoordinatorManager: celine,
                 hyperdriveGovernance: bob,
                 feeCollector: feeCollector,
                 sweepCollector: sweepCollector,

test/instances/erc4626/UsdcERC4626.t.sol

@@ -70,6 +70,7 @@ contract UsdcERC4626 is ERC4626ValidationTest {
         factory = new HyperdriveFactory(
             HyperdriveFactory.FactoryConfig({
                 governance: alice,
+                deployerCoordinatorManager: celine,
                 hyperdriveGovernance: bob,
                 feeCollector: feeCollector,
                 sweepCollector: sweepCollector,

test/integrations/deployers/ERC4626DeployerCoordinator.t.sol

@@ -60,6 +60,7 @@ contract ERC4626DeployerCoordinatorTest is DeployerCoordinatorTest {
             new HyperdriveFactory(
                 HyperdriveFactory.FactoryConfig({
                     governance: alice,
+                    deployerCoordinatorManager: celine,
                     hyperdriveGovernance: bob,
                     feeCollector: feeCollector,
                     sweepCollector: sweepCollector,

test/integrations/deployers/RethDeployerCoordinator.t.sol

@@ -52,6 +52,7 @@ contract RethDeployerCoordinatorTest is DeployerCoordinatorTest {
             new HyperdriveFactory(
                 HyperdriveFactory.FactoryConfig({
                     governance: alice,
+                    deployerCoordinatorManager: celine,
                     hyperdriveGovernance: bob,
                     feeCollector: feeCollector,
                     sweepCollector: sweepCollector,

test/integrations/deployers/StethDeployerCoordinator.t.sol

@@ -52,6 +52,7 @@ contract StethDeployerCoordinatorTest is DeployerCoordinatorTest {
             new HyperdriveFactory(
                 HyperdriveFactory.FactoryConfig({
                     governance: alice,
+                    deployerCoordinatorManager: celine,
                     hyperdriveGovernance: bob,
                     feeCollector: feeCollector,
                     sweepCollector: sweepCollector,