refactor: slashing registry coordinator

refactor: registry coordinator

refactor: remove sm calls

fix: tests wip

Author: 8sunyuan

src/BLSApkRegistry.sol

@@ -3,7 +3,7 @@ pragma solidity ^0.8.27;
 
 import {BLSApkRegistryStorage} from "./BLSApkRegistryStorage.sol";
 
-import {IRegistryCoordinator} from "./interfaces/IRegistryCoordinator.sol";
+import {ISlashingRegistryCoordinator} from "./interfaces/ISlashingRegistryCoordinator.sol";
 
 import {BN254} from "./libraries/BN254.sol";
 
@@ -18,8 +18,8 @@ contract BLSApkRegistry is BLSApkRegistryStorage {
 
     /// @notice Sets the (immutable) `registryCoordinator` address
     constructor(
-        IRegistryCoordinator _registryCoordinator
-    ) BLSApkRegistryStorage(_registryCoordinator) {}
+        ISlashingRegistryCoordinator _slashingRegistryCoordinator
+    ) BLSApkRegistryStorage(_slashingRegistryCoordinator) {}
 
     /*******************************************************************************
                       EXTERNAL FUNCTIONS - REGISTRY COORDINATOR

src/BLSApkRegistryStorage.sol

@@ -2,7 +2,7 @@
 pragma solidity ^0.8.27;
 
 import {IBLSApkRegistry} from "./interfaces/IBLSApkRegistry.sol";
-import {IRegistryCoordinator} from "./interfaces/IRegistryCoordinator.sol";
+import {ISlashingRegistryCoordinator} from "./interfaces/ISlashingRegistryCoordinator.sol";
 
 import {Initializable} from "@openzeppelin-upgrades/contracts/proxy/utils/Initializable.sol";
 
@@ -29,8 +29,8 @@ abstract contract BLSApkRegistryStorage is Initializable, IBLSApkRegistry {
     /// @notice maps quorumNumber => current aggregate pubkey of quorum
     mapping(uint8 => BN254.G1Point) public currentApk;
 
-    constructor(IRegistryCoordinator _registryCoordinator) {
-        registryCoordinator = address(_registryCoordinator);
+    constructor(ISlashingRegistryCoordinator _slashingRegistryCoordinator) {
+        registryCoordinator = address(_slashingRegistryCoordinator);
         // disable initializers so that the implementation contract cannot be initialized
         _disableInitializers();
     }

src/BLSSignatureChecker.sol

@@ -2,7 +2,7 @@
 pragma solidity ^0.8.27;
 
 import {IBLSSignatureChecker} from "./interfaces/IBLSSignatureChecker.sol";
-import {IRegistryCoordinator} from "./interfaces/IRegistryCoordinator.sol";
+import {ISlashingRegistryCoordinator} from "./interfaces/ISlashingRegistryCoordinator.sol";
 import {IBLSApkRegistry} from "./interfaces/IBLSApkRegistry.sol";
 import {IStakeRegistry, IDelegationManager} from "./interfaces/IStakeRegistry.sol";
 
@@ -23,7 +23,7 @@ contract BLSSignatureChecker is IBLSSignatureChecker {
     // gas cost of multiplying 2 pairings
     uint256 internal constant PAIRING_EQUALITY_CHECK_GAS = 120_000;
 
-    IRegistryCoordinator public immutable registryCoordinator;
+    ISlashingRegistryCoordinator public immutable registryCoordinator;
     IStakeRegistry public immutable stakeRegistry;
     IBLSApkRegistry public immutable blsApkRegistry;
     IDelegationManager public immutable delegation;
@@ -35,7 +35,7 @@ contract BLSSignatureChecker is IBLSSignatureChecker {
         _;
     }
 
-    constructor(IRegistryCoordinator _registryCoordinator) {
+    constructor(ISlashingRegistryCoordinator _registryCoordinator) {
         registryCoordinator = _registryCoordinator;
         stakeRegistry = _registryCoordinator.stakeRegistry();
         blsApkRegistry = _registryCoordinator.blsApkRegistry();

src/EjectionManager.sol

@@ -3,7 +3,7 @@ pragma solidity ^0.8.27;
 
 import {OwnableUpgradeable} from "@openzeppelin-upgrades/contracts/access/OwnableUpgradeable.sol";
 import {IEjectionManager} from "./interfaces/IEjectionManager.sol";
-import {IRegistryCoordinator} from "./interfaces/IRegistryCoordinator.sol";
+import {ISlashingRegistryCoordinator} from "./interfaces/ISlashingRegistryCoordinator.sol";
 import {IStakeRegistry} from "./interfaces/IStakeRegistry.sol";
 
 /**
@@ -19,7 +19,7 @@ contract EjectionManager is IEjectionManager, OwnableUpgradeable {
     uint8 internal constant MAX_QUORUM_COUNT = 192;
 
     /// @notice the RegistryCoordinator contract that is the entry point for ejection
-    IRegistryCoordinator public immutable registryCoordinator;
+    ISlashingRegistryCoordinator public immutable registryCoordinator;
     /// @notice the StakeRegistry contract that keeps track of quorum stake
     IStakeRegistry public immutable stakeRegistry;
 
@@ -32,7 +32,7 @@ contract EjectionManager is IEjectionManager, OwnableUpgradeable {
     mapping(uint8 => QuorumEjectionParams) public quorumEjectionParams;
 
     constructor(
-        IRegistryCoordinator _registryCoordinator,
+        ISlashingRegistryCoordinator _registryCoordinator,
         IStakeRegistry _stakeRegistry
     ) {
         registryCoordinator = _registryCoordinator;

src/IndexRegistry.sol

@@ -2,7 +2,7 @@
 pragma solidity ^0.8.27;
 
 import {IndexRegistryStorage} from "./IndexRegistryStorage.sol";
-import {IRegistryCoordinator} from "./interfaces/IRegistryCoordinator.sol";
+import {ISlashingRegistryCoordinator} from "./interfaces/ISlashingRegistryCoordinator.sol";
 
 /**
  * @title A `Registry` that keeps track of an ordered list of operators for each quorum
@@ -18,8 +18,8 @@ contract IndexRegistry is IndexRegistryStorage {
 
     /// @notice sets the (immutable) `registryCoordinator` address
     constructor(
-        IRegistryCoordinator _registryCoordinator
-    ) IndexRegistryStorage(_registryCoordinator) {}
+        ISlashingRegistryCoordinator _slashingRegistryCoordinator
+    ) IndexRegistryStorage(_slashingRegistryCoordinator) {}
 
     /*******************************************************************************
                       EXTERNAL FUNCTIONS - REGISTRY COORDINATOR

src/IndexRegistryStorage.sol

@@ -3,7 +3,7 @@ pragma solidity ^0.8.27;
 
 import {Initializable} from "@openzeppelin-upgrades/contracts/proxy/utils/Initializable.sol";
 
-import {IRegistryCoordinator} from "./interfaces/IRegistryCoordinator.sol";
+import {ISlashingRegistryCoordinator} from "./interfaces/ISlashingRegistryCoordinator.sol";
 import {IIndexRegistry} from "./interfaces/IIndexRegistry.sol";
 
 /**
@@ -31,9 +31,9 @@ abstract contract IndexRegistryStorage is Initializable, IIndexRegistry {
     mapping(uint8 => QuorumUpdate[]) internal _operatorCountHistory;
 
     constructor(
-        IRegistryCoordinator _registryCoordinator
+        ISlashingRegistryCoordinator _slashingRegistryCoordinator
     ){
-        registryCoordinator = address(_registryCoordinator);
+        registryCoordinator = address(_slashingRegistryCoordinator);
         // disable initializers so that the implementation contract cannot be initialized
         _disableInitializers();
     }

src/OperatorStateRetriever.sol

@@ -1,7 +1,7 @@
 // SPDX-License-Identifier: BUSL-1.1
 pragma solidity ^0.8.27;
 
-import {IRegistryCoordinator} from "./interfaces/IRegistryCoordinator.sol";
+import {ISlashingRegistryCoordinator} from "./interfaces/ISlashingRegistryCoordinator.sol";
 import {IBLSApkRegistry} from "./interfaces/IBLSApkRegistry.sol";
 import {IStakeRegistry} from "./interfaces/IStakeRegistry.sol";
 import {IIndexRegistry} from "./interfaces/IIndexRegistry.sol";
@@ -40,7 +40,7 @@ contract OperatorStateRetriever {
      *            was a part of at `blockNumber`, an ordered list of operators.
      */
     function getOperatorState(
-        IRegistryCoordinator registryCoordinator,
+        ISlashingRegistryCoordinator registryCoordinator,
         bytes32 operatorId,
         uint32 blockNumber
     ) external view returns (uint256, Operator[][] memory) {
@@ -64,7 +64,7 @@ contract OperatorStateRetriever {
      * @return 2d array of Operators. For each quorum, an ordered list of Operators
      */
     function getOperatorState(
-        IRegistryCoordinator registryCoordinator,
+        ISlashingRegistryCoordinator registryCoordinator,
         bytes memory quorumNumbers,
         uint32 blockNumber
     ) public view returns(Operator[][] memory) {
@@ -104,7 +104,7 @@ contract OperatorStateRetriever {
      *         4) the indices of the quorum apks for each of the provided quorums at the given blocknumber
      */
     function getCheckSignaturesIndices(
-        IRegistryCoordinator registryCoordinator,
+        ISlashingRegistryCoordinator registryCoordinator,
         uint32 referenceBlockNumber,
         bytes calldata quorumNumbers,
         bytes32[] calldata nonSignerOperatorIds
@@ -169,7 +169,7 @@ contract OperatorStateRetriever {
      * @param blockNumber is the block number to get the quorumBitmaps for
      */
     function getQuorumBitmapsAtBlockNumber(
-        IRegistryCoordinator registryCoordinator,
+        ISlashingRegistryCoordinator registryCoordinator,
         bytes32[] memory operatorIds,
         uint32 blockNumber
     ) external view returns (uint256[] memory) {
@@ -188,7 +188,7 @@ contract OperatorStateRetriever {
      * @dev if an operator is not registered, the operatorId will be 0
      */
     function getBatchOperatorId(
-        IRegistryCoordinator registryCoordinator,
+        ISlashingRegistryCoordinator registryCoordinator,
         address[] memory operators
     ) external view returns (bytes32[] memory operatorIds) {
         operatorIds = new bytes32[](operators.length);
@@ -204,7 +204,7 @@ contract OperatorStateRetriever {
      * @dev if an operator is not registered, the operator address will be 0
      */
     function getBatchOperatorFromId(
-        IRegistryCoordinator registryCoordinator,
+        ISlashingRegistryCoordinator registryCoordinator,
         bytes32[] memory operatorIds
     ) external view returns (address[] memory operators) {
         operators = new address[](operatorIds.length);