chore: address review comments

Author: stevennevins

src/RegistryCoordinator.sol

@@ -65,10 +65,9 @@ contract RegistryCoordinator is
         IStakeRegistry _stakeRegistry,
         IBLSApkRegistry _blsApkRegistry,
         IIndexRegistry _indexRegistry,
-        IAVSDirectory _avsDirectory,
         IPauserRegistry _pauserRegistry
     )
-        RegistryCoordinatorStorage(_serviceManager, _stakeRegistry, _blsApkRegistry, _indexRegistry, _avsDirectory)
+        RegistryCoordinatorStorage(_serviceManager, _stakeRegistry, _blsApkRegistry, _indexRegistry)
         EIP712("AVSRegistryCoordinator", "v0.0.1")
         Pausable(_pauserRegistry)
     {

src/RegistryCoordinatorStorage.sol

@@ -72,21 +72,24 @@ abstract contract RegistryCoordinatorStorage is IRegistryCoordinator {
     /// @notice the delay in seconds before an operator can reregister after being ejected
     uint256 public ejectionCooldown;
 
+    /// @notice Whether this AVS uses operator sets for registration
+    /// @dev If true, operators must register to operator sets via the AllocationManager
     bool public isOperatorSetAVS;
+
+    /// @notice Mapping from quorum number to whether the quorum is an M2 quorum
+    /// @dev M2 quorums are pre-operator sets and track total delegated stake only
     mapping(uint8 => bool) public isM2Quorum;
 
     constructor(
         IServiceManager _serviceManager,
         IStakeRegistry _stakeRegistry,
         IBLSApkRegistry _blsApkRegistry,
-        IIndexRegistry _indexRegistry,
-        IAVSDirectory _avsDirectory
+        IIndexRegistry _indexRegistry
     ) {
         serviceManager = _serviceManager;
         stakeRegistry = _stakeRegistry;
         blsApkRegistry = _blsApkRegistry;
         indexRegistry = _indexRegistry;
-        avsDirectory = _avsDirectory;
     }
 
     // storage gap for upgradeability

test/harnesses/RegistryCoordinatorHarness.t.sol

@@ -12,9 +12,8 @@ contract RegistryCoordinatorHarness is RegistryCoordinator, Test {
         IStakeRegistry _stakeRegistry,
         IBLSApkRegistry _blsApkRegistry,
         IIndexRegistry _indexRegistry,
-        IAVSDirectory _avsDirectory,
         IPauserRegistry _pauserRegistry
-    ) RegistryCoordinator(_serviceManager, _stakeRegistry, _blsApkRegistry, _indexRegistry, _avsDirectory, _pauserRegistry) {
+    ) RegistryCoordinator(_serviceManager, _stakeRegistry, _blsApkRegistry, _indexRegistry, _pauserRegistry) {
         _transferOwnership(msg.sender);
     }
 

test/integration/CoreRegistration.t.sol

@@ -91,7 +91,6 @@ contract Test_CoreRegistration is MockAVSDeployer {
             stakeRegistry,
             blsApkRegistry,
             indexRegistry,
-            avsDirectory,
             pauserRegistry
         );
 

test/integration/IntegrationDeployer.t.sol

@@ -376,7 +376,7 @@ abstract contract IntegrationDeployer is Test, IUserDeployer {
         uint32[] memory slashableStakeQuorumLookAheadPeriods = new uint32[](0);
 
         RegistryCoordinator registryCoordinatorImplementation =
-            new RegistryCoordinator(serviceManager, stakeRegistry, blsApkRegistry, indexRegistry, avsDirectory, pauserRegistry);
+            new RegistryCoordinator(serviceManager, stakeRegistry, blsApkRegistry, indexRegistry, pauserRegistry);
         proxyAdmin.upgradeAndCall(
             TransparentUpgradeableProxy(payable(address(registryCoordinator))),
             address(registryCoordinatorImplementation),

test/unit/StakeRegistryUnit.t.sol

@@ -49,7 +49,6 @@ contract StakeRegistryUnitTests is MockAVSDeployer, IStakeRegistryEvents {
             stakeRegistry,
             IBLSApkRegistry(blsApkRegistry),
             IIndexRegistry(indexRegistry),
-            IAVSDirectory(avsDirectory),
             pauserRegistry
         );
 

test/utils/MockAVSDeployer.sol

@@ -274,7 +274,7 @@ contract MockAVSDeployer is Test {
         }
 
         registryCoordinatorImplementation = new RegistryCoordinatorHarness(
-            serviceManager, stakeRegistry, blsApkRegistry, indexRegistry, avsDirectory, pauserRegistry
+            serviceManager, stakeRegistry, blsApkRegistry, indexRegistry, pauserRegistry
         );
         {
             delete operatorSetParams;