refactor: interface structure

src/interfaces/IBLSSignatureChecker.sol

@@ -30,15 +30,7 @@ interface IBLSSignatureCheckerErrors {
     error InvalidBLSSignature();
 }
 
-/**
- * @title Used for checking BLS aggregate signatures from the operators of a EigenLayer AVS with the RegistryCoordinator/BLSApkRegistry/StakeRegistry architechture.
- * @author Layr Labs, Inc.
- * @notice Terms of Service: https://docs.eigenlayer.xyz/overview/terms-of-service
- * @notice This is the contract for checking the validity of aggregate operator signatures.
- */
-interface IBLSSignatureChecker is IBLSSignatureCheckerErrors {
-    // DATA STRUCTURES
-
+interface IBLSSignatureCheckerTypes {
     struct NonSignerStakesAndSignature {
         uint32[] nonSignerQuorumBitmapIndices; // is the indices of all nonsigner quorum bitmaps
         BN254.G1Point[] nonSignerPubkeys; // is the G1 pubkeys of all nonsigners
@@ -60,18 +52,31 @@ interface IBLSSignatureChecker is IBLSSignatureCheckerErrors {
         // total amount staked by all operators in each quorum
         uint96[] totalStakeForQuorum;
     }
+}
 
-    // EVENTS
-
+interface IBLSSignatureCheckerEvents is IBLSSignatureCheckerTypes {
     /// @notice Emitted when `staleStakesForbiddenUpdate` is set
     event StaleStakesForbiddenUpdate(bool value);
+}
 
-    // CONSTANTS & IMMUTABLES
+interface IBLSSignatureChecker is IBLSSignatureCheckerErrors, IBLSSignatureCheckerEvents {
+    /// STATE
 
     function registryCoordinator() external view returns (IRegistryCoordinator);
+
     function stakeRegistry() external view returns (IStakeRegistry);
+
     function blsApkRegistry() external view returns (IBLSApkRegistry);
+
     function delegation() external view returns (IDelegationManager);
+
+    function staleStakesForbidden() external view returns (bool);
+
+    // ACTIONS
+
+    function setStaleStakesForbidden(bool value) external;
+
+    /// VIEW
 
     /**
      * @notice This function is called by disperser when it has aggregated all the signatures of the operators
@@ -95,4 +100,11 @@ interface IBLSSignatureChecker is IBLSSignatureCheckerErrors {
         uint32 referenceBlockNumber,
         NonSignerStakesAndSignature memory nonSignerStakesAndSignature
     ) external view returns (QuorumStakeTotals memory, bytes32);
-}
+
+    function trySignatureAndApkVerification(
+        bytes32 msgHash,
+        BN254.G1Point memory apk,
+        BN254.G2Point memory apkG2,
+        BN254.G1Point memory sigma
+    ) external view returns (bool pairingSuccessful, bool siganatureIsValid);
+}

test/unit/BLSSignatureCheckerUnit.t.sol

@@ -3,7 +3,7 @@ pragma solidity ^0.8.27;
 
 import "../../src/BLSSignatureChecker.sol";
 import "../utils/BLSMockAVSDeployer.sol";
-import {IBLSSignatureCheckerErrors} from "../../src/interfaces/IBLSSignatureChecker.sol";
+import {IBLSSignatureCheckerErrors, IBLSSignatureCheckerTypes} from "../../src/interfaces/IBLSSignatureChecker.sol";
 import {IBLSApkRegistryErrors} from "../../src/interfaces/IBLSApkRegistry.sol";
 import {QuorumBitmapHistoryLib} from "../../src/libraries/QuorumBitmapHistoryLib.sol";
 import {IStakeRegistryErrors} from "../../src/interfaces/IStakeRegistry.sol";
@@ -201,8 +201,8 @@ contract BLSSignatureCheckerUnitTests is BLSMockAVSDeployer {
             1, numNonSigners, quorumBitmap
         );
 
-        IBLSSignatureChecker.NonSignerStakesAndSignature memory incorrectLengthInputs =
-        IBLSSignatureChecker.NonSignerStakesAndSignature({
+        IBLSSignatureCheckerTypes.NonSignerStakesAndSignature memory incorrectLengthInputs =
+        IBLSSignatureCheckerTypes.NonSignerStakesAndSignature({
             nonSignerQuorumBitmapIndices: nonSignerStakesAndSignature.nonSignerQuorumBitmapIndices,
             nonSignerPubkeys: nonSignerStakesAndSignature.nonSignerPubkeys,
             quorumApks: nonSignerStakesAndSignature.quorumApks,