fix: remove mutable tos

Author: 0xrajath

src/contracts/cloud/ComputeRegistry.sol

@@ -2,7 +2,6 @@
 pragma solidity ^0.8.27;
 
 import "@openzeppelin-upgrades/contracts/proxy/utils/Initializable.sol";
-import "@openzeppelin-upgrades/contracts/access/OwnableUpgradeable.sol";
 
 import "../mixins/PermissionControllerMixin.sol";
 import "../mixins/SignatureUtilsMixin.sol";
@@ -15,13 +14,7 @@ import "./ComputeRegistryStorage.sol";
  * @notice This contract handles permissionless (de)registration of AVS operator sets to the EigenCompute Operator.
  * It enables AVSs to easily access managed operator infrastructure as part of EigenCloud for quick bootstrapping.
  */
-contract ComputeRegistry is
-    Initializable,
-    OwnableUpgradeable,
-    ComputeRegistryStorage,
-    PermissionControllerMixin,
-    SignatureUtilsMixin
-{
+contract ComputeRegistry is Initializable, ComputeRegistryStorage, PermissionControllerMixin, SignatureUtilsMixin {
     using OperatorSetLib for OperatorSet;
 
     /**
@@ -52,32 +45,23 @@ contract ComputeRegistry is
      * @param _releaseManager The ReleaseManager contract address
      * @param _allocationManager The AllocationManager contract address
      * @param _permissionController The PermissionController contract address
+     * @param _tosHash The immutable hash of the Terms of Service
      * @param _version The semantic version of the contract
      */
     constructor(
         IReleaseManager _releaseManager,
         IAllocationManager _allocationManager,
         IPermissionController _permissionController,
+        bytes32 _tosHash,
         string memory _version
     )
-        ComputeRegistryStorage(_releaseManager, _allocationManager)
+        ComputeRegistryStorage(_releaseManager, _allocationManager, _tosHash)
         PermissionControllerMixin(_permissionController)
         SignatureUtilsMixin(_version)
     {
         _disableInitializers();
     }
 
-    /**
-     * @notice Initializes the contract
-     * @param _owner The owner of the contract
-     * @param _tosHash The hash of the Terms of Service that AVS operators must sign
-     */
-    function initialize(address _owner, bytes32 _tosHash) external initializer {
-        __Ownable_init();
-        transferOwnership(_owner);
-        _setTosHash(_tosHash);
-    }
-
     /**
      *
      *                         EXTERNAL FUNCTIONS
@@ -110,9 +94,9 @@ contract ComputeRegistry is
         // Register the operator set
         isOperatorSetRegistered[operatorSetKey] = true;
         _operatorSetTosSignature[operatorSetKey] =
-            TOSSignature({signer: msg.sender, tosHash: tosHash, signature: signature});
+            TOSSignature({signer: msg.sender, tosHash: TOS_HASH, signature: signature});
 
-        emit OperatorSetRegistered(operatorSet, msg.sender, tosHash, signature);
+        emit OperatorSetRegistered(operatorSet, msg.sender, TOS_HASH, signature);
     }
 
     /**
@@ -130,38 +114,9 @@ contract ComputeRegistry is
         emit OperatorSetDeregistered(operatorSet);
     }
 
-    /**
-     * @notice Updates the Terms of Service hash
-     * @param _tosHash The new Terms of Service hash
-     * @dev Only callable by the contract owner
-     */
-    function setTosHash(
-        bytes32 _tosHash
-    ) external onlyOwner {
-        _setTosHash(_tosHash);
-    }
-
-    /**
-     *
-     *                         INTERNAL FUNCTIONS
-     *
-     */
-
-    /**
-     * @dev Internal function to set the Terms of Service hash
-     * @param _tosHash The new Terms of Service hash
-     */
-    function _setTosHash(
-        bytes32 _tosHash
-    ) internal {
-        tosHash = _tosHash;
-        emit TosHashSet(_tosHash);
-    }
-
     /**
      *
      *                         VIEW FUNCTIONS
-     *
      */
 
     /**
@@ -188,7 +143,7 @@ contract ComputeRegistry is
             keccak256(
                 abi.encode(
                     TOS_AGREEMENT_TYPEHASH,
-                    tosHash,
+                    TOS_HASH,
                     operatorSet.avs,
                     operatorSet.id,
                     signer,

src/contracts/cloud/ComputeRegistryStorage.sol

@@ -22,10 +22,10 @@ abstract contract ComputeRegistryStorage is IComputeRegistry {
     /// @notice The AllocationManager contract
     IAllocationManager public immutable ALLOCATION_MANAGER;
 
-    // Storage
-
     /// @notice The hash of the Terms of Service that AVS operators must sign
-    bytes32 public tosHash;
+    bytes32 public immutable TOS_HASH;
+
+    // Storage
 
     /// @notice Mapping to track if an operator set is registered for compute
     /// @dev operatorSetKey => isRegistered
@@ -40,10 +40,11 @@ abstract contract ComputeRegistryStorage is IComputeRegistry {
      * variables without shifting down storage in the inheritance chain.
      * See https://docs.openzeppelin.com/contracts/4.x/upgradeable#storage_gaps
      */
-    uint256[47] private __gap;
+    uint256[48] private __gap;
 
-    constructor(IReleaseManager _releaseManager, IAllocationManager _allocationManager) {
+    constructor(IReleaseManager _releaseManager, IAllocationManager _allocationManager, bytes32 _tosHash) {
         RELEASE_MANAGER = _releaseManager;
         ALLOCATION_MANAGER = _allocationManager;
+        TOS_HASH = _tosHash;
     }
 }

src/contracts/interfaces/IComputeRegistry.sol

@@ -41,10 +41,6 @@ interface IComputeRegistryEvents {
     /// @notice Emitted when an operator set is deregistered from compute
     /// @param operatorSet The operator set that was deregistered
     event OperatorSetDeregistered(OperatorSet indexed operatorSet);
-
-    /// @notice Emitted when the Terms of Service hash is updated
-    /// @param tosHash The new Terms of Service hash
-    event TosHashSet(bytes32 tosHash);
 }
 
 interface IComputeRegistry is IComputeRegistryErrors, IComputeRegistryEvents, IComputeRegistryTypes {
@@ -74,15 +70,6 @@ interface IComputeRegistry is IComputeRegistryErrors, IComputeRegistryEvents, IC
         OperatorSet calldata operatorSet
     ) external;
 
-    /**
-     * @notice Updates the Terms of Service hash
-     * @param tosHash The new Terms of Service hash
-     * @dev Only callable by the contract owner
-     */
-    function setTosHash(
-        bytes32 tosHash
-    ) external;
-
     /**
      *
      *                         VIEW FUNCTIONS
@@ -117,7 +104,7 @@ interface IComputeRegistry is IComputeRegistryErrors, IComputeRegistryEvents, IC
      * @notice Returns the hash of the Terms of Service
      * @return The hash of the Terms of Service that must be signed
      */
-    function tosHash() external view returns (bytes32);
+    function TOS_HASH() external view returns (bytes32);
 
     /**
      * @notice Checks if an operator set is registered for compute