feat: pectra compatibility

src/contracts/interfaces/IEigenPodManager.sol

@@ -140,13 +140,14 @@ interface IEigenPodManager is IEigenPodManagerErrors, IEigenPodManagerEvents, IS
 
     /**
      * @notice The Pectra hard fork timestamp used to determine which proof config to use for a checkpoint proof. 
-     * @dev This function returns type(uint64).max if the fork timestamp has not been set. The timestamp can never be set to 0.
+     * @dev This function returns type(uint64).max if the fork timestamp has not been set. 
      */
     function getPectraForkTimestamp() external view returns (uint64);
 
      /**
      * @notice Sets the pectra hard fork timestamp by the eigenPodManager owner
-     * @dev This function is callable only by the owner of the eigenPodManager only once
+     * @dev This function can only be called by the eigenPodManager owner
+     * @dev This function can only be called once
      */
     function setPectraForkTimestamp(uint64 pectraForkTimestamp) external;
 }

src/contracts/libraries/BeaconChainProofs.sol

@@ -332,9 +332,9 @@ library BeaconChainProofs {
     }
 
     /// @dev Gets the height of the beacon state tree based on the pectraForkTimestamp
-    /// @dev We subtract one from the pectraForkTimestamp to ensure that a `proofTimestamp` at the `pectraForkTimestamp` 
-    ///      is considered to be Pre-Pectra given the EIP-4788 returns the parent block. 
+    /// @dev We check if the proofTimestamp is <= pectraForkTimestamp because a `proofTimestamp` at the `pectraForkTimestamp` 
+    ///      is considered to be Pre-Pectra given the EIP-4788 oracle returns the parent block. 
     function getBeaconStateTreeHeight(uint64 proofTimestamp, uint64 pectraForkTimestamp) internal pure returns (uint256) {
-        return proofTimestamp < pectraForkTimestamp - 1 ? DENEB_BEACON_STATE_TREE_HEIGHT : PECTRA_BEACON_STATE_TREE_HEIGHT;
+        return proofTimestamp <= pectraForkTimestamp ? DENEB_BEACON_STATE_TREE_HEIGHT : PECTRA_BEACON_STATE_TREE_HEIGHT;
     }
 }

src/contracts/pods/EigenPodManager.sol

@@ -216,13 +216,13 @@ contract EigenPodManager is
     }
 
     /// @inheritdoc IEigenPodManager
-    function setPectraForkTimestamp(uint64 newPectraForkTimestamp) external onlyOwner {
+    function setPectraForkTimestamp(uint64 pectraForkTimestamp) external onlyOwner {
         uint64 currentPectraForkTimestamp = getPectraForkTimestamp();
         require(currentPectraForkTimestamp == type(uint64).max, ForkTimestampAlreadySet());
-        require(newPectraForkTimestamp != 0, InvalidForkTimestamp());
+        require(pectraForkTimestamp != type(uint64).max, InvalidForkTimestamp());
 
-        _pectraForkTimestamp = newPectraForkTimestamp;
-        emit PectraForkTimestampSet(newPectraForkTimestamp);
+        _pectraForkTimestamp = pectraForkTimestamp;
+        emit PectraForkTimestampSet(pectraForkTimestamp);
     }
 
     /// @inheritdoc IEigenPodManager