fix: wire up stake registry to call add and rm strategies for op set

Author: stevennevins

src/StakeRegistry.sol

@@ -254,6 +254,16 @@ contract StakeRegistry is StakeRegistryStorage {
         StrategyParams[] memory _strategyParams
     ) public virtual onlyCoordinatorOwner quorumExists(quorumNumber) {
         _addStrategyParams(quorumNumber, _strategyParams);
+
+        uint256 numStratsToAdd = _strategyParams.length;
+
+        if (isOperatorSetQuorum(quorumNumber)){
+            IStrategy[] memory strategiesToAdd = new IStrategy[](numStratsToAdd);
+            for (uint256 i = 0; i < numStratsToAdd; i++) {
+                strategiesToAdd[i] = _strategyParams[i].strategy;
+            }
+            serviceManager.addStrategyToOperatorSet(quorumNumber, strategiesToAdd);
+        }
     }
 
     /**
@@ -270,17 +280,25 @@ contract StakeRegistry is StakeRegistryStorage {
 
         StrategyParams[] storage _strategyParams = strategyParams[quorumNumber];
         IStrategy[] storage _strategiesPerQuorum = strategiesPerQuorum[quorumNumber];
+        IStrategy[] memory _strategiesToRemove = new IStrategy[](toRemoveLength);
 
         for (uint256 i = 0; i < toRemoveLength; i++) {
+            _strategiesToRemove[i]=_strategyParams[indicesToRemove[i]].strategy;
             emit StrategyRemovedFromQuorum(quorumNumber, _strategyParams[indicesToRemove[i]].strategy);
             emit StrategyMultiplierUpdated(quorumNumber, _strategyParams[indicesToRemove[i]].strategy, 0);
 
+
+
             // Replace index to remove with the last item in the list, then pop the last item
             _strategyParams[indicesToRemove[i]] = _strategyParams[_strategyParams.length - 1];
             _strategyParams.pop();
             _strategiesPerQuorum[indicesToRemove[i]] = _strategiesPerQuorum[_strategiesPerQuorum.length - 1];
             _strategiesPerQuorum.pop();
         }
+
+        if (isOperatorSetQuorum(quorumNumber)){
+            serviceManager.removeStrategiesFromOperatorSet(quorumNumber, _strategiesToRemove);
+        }
     }
 
     /**
@@ -560,7 +578,7 @@ contract StakeRegistry is StakeRegistryStorage {
      * @param quorumNumber The quorum number to check
      * @return True if the quorum is an operator set quorum
      */
-    function isOperatorSetQuorum(uint8 quorumNumber) external view returns (bool) {
+    function isOperatorSetQuorum(uint8 quorumNumber) public view returns (bool) {
         bool isM2 = IRegistryCoordinator(registryCoordinator).isM2Quorum(quorumNumber);
         bool isOperatorSet = IRegistryCoordinator(registryCoordinator).isOperatorSetAVS();
         return isOperatorSet && !isM2;