Feat/dynamic fees

README.md

@@ -129,11 +129,11 @@ It is recommended to setup the ide to work with solidity development. In case of
 | InvalidNonce()              | 0x756688fe |
 | MsgValueTooLow()            | 0x508aaf00 |
 | MsgValueTooHigh()           | 0x5dffc92f |
-| PayloadTooLarge()           | 0x492f620d |
 | InsufficientMsgValue()      | 0x78f38f76 |
 | InsufficientFees()          | 0x8d53e553 |
 | InvalidMsgValue()           | 0x1841b4e1 |
 | FeesTooHigh()               | 0xc9034e18 |
+| PayloadTooLarge()           | 0x492f620d |
 | InvalidBatchSize()          | 0x7862e959 |
 | InsufficientMessageLength() | 0xbcfdc01d |
 | InvalidPacketLength()       | 0x5a375a8e |

contracts/ExecutionManager.sol

@@ -15,7 +15,7 @@ import {FEES_UPDATE_SIG_IDENTIFIER, RELATIVE_NATIVE_TOKEN_PRICE_UPDATE_SIG_IDENT
  * managing execution and other fees. This contract also implements the AccessControl interface, allowing for role-based
  * access control.
  */
-contract ExecutionManager is IExecutionManager, AccessControlExtended {
+contract ExecutionManager is AccessControlExtended {
     ISignatureVerifier public immutable signatureVerifier__;
     ISocket public immutable socket__;
     uint32 public immutable chainSlug;
@@ -183,13 +183,7 @@ contract ExecutionManager is IExecutionManager, AccessControlExtended {
     function isExecutor(
         bytes32 packedMessage,
         bytes memory sig
-    )
-        external
-        view
-        virtual
-        override
-        returns (address executor, bool isValidExecutor)
-    {
+    ) external view virtual returns (address executor, bool isValidExecutor) {
         executor = signatureVerifier__.recoverSigner(packedMessage, sig);
         isValidExecutor = _hasRole(EXECUTOR_ROLE, executor);
     }
@@ -198,17 +192,11 @@ contract ExecutionManager is IExecutionManager, AccessControlExtended {
      * @notice updates the total fee used by an executor to execute a message
      * @dev to be used for accounting when onchain fee distribution for individual executors is implemented
      * @dev this function should be called by socket only
-     * @inheritdoc IExecutionManager
      */
-    function updateExecutionFees(
-        address,
-        uint128,
-        bytes32
-    ) external view override {
+    function updateExecutionFees(address, uint128, bytes32) external view {
         if (msg.sender != address(socket__)) revert OnlySocket();
     }
 
-    /// @inheritdoc IExecutionManager
     function payAndCheckFees(
         uint256 minMsgGasLimit_,
         uint256 payloadSize_,
@@ -223,7 +211,6 @@ contract ExecutionManager is IExecutionManager, AccessControlExtended {
     )
         external
         payable
-        override
         returns (uint128 executionFee, uint128 transmissionFees)
     {
         if (msg.value >= type(uint128).max) revert InvalidMsgValue();
@@ -282,7 +269,7 @@ contract ExecutionManager is IExecutionManager, AccessControlExtended {
         uint256 payloadSize_,
         bytes32 executionParams_,
         uint32 siblingChainSlug_
-    ) external view override returns (uint128 minExecutionFee) {
+    ) external view returns (uint128 minExecutionFee) {
         minExecutionFee = _getMinFees(
             gasLimit_,
             payloadSize_,
@@ -291,7 +278,6 @@ contract ExecutionManager is IExecutionManager, AccessControlExtended {
         );
     }
 
-    /// @inheritdoc IExecutionManager
     function getExecutionTransmissionMinFees(
         uint256 minMsgGasLimit_,
         uint256 payloadSize_,
@@ -302,7 +288,6 @@ contract ExecutionManager is IExecutionManager, AccessControlExtended {
     )
         external
         view
-        override
         returns (uint128 minExecutionFee, uint128 transmissionFees)
     {
         minExecutionFee = _getMinFees(
@@ -357,7 +342,7 @@ contract ExecutionManager is IExecutionManager, AccessControlExtended {
     function verifyParams(
         bytes32 executionParams_,
         uint256 msgValue_
-    ) external pure override {
+    ) external pure {
         uint256 params = uint256(executionParams_);
         uint8 paramType = uint8(params >> 248);
 
@@ -381,7 +366,7 @@ contract ExecutionManager is IExecutionManager, AccessControlExtended {
         uint32 siblingChainSlug_,
         uint128 executionFees_,
         bytes calldata signature_
-    ) external override {
+    ) external {
         address feesUpdater = signatureVerifier__.recoverSigner(
             keccak256(
                 abi.encode(
@@ -420,7 +405,7 @@ contract ExecutionManager is IExecutionManager, AccessControlExtended {
         uint32 siblingChainSlug_,
         uint256 relativeNativeTokenPrice_,
         bytes calldata signature_
-    ) external override {
+    ) external {
         address feesUpdater = signatureVerifier__.recoverSigner(
             keccak256(
                 abi.encode(
@@ -461,7 +446,7 @@ contract ExecutionManager is IExecutionManager, AccessControlExtended {
         uint32 siblingChainSlug_,
         uint256 msgValueMinThreshold_,
         bytes calldata signature_
-    ) external override {
+    ) external {
         address feesUpdater = signatureVerifier__.recoverSigner(
             keccak256(
                 abi.encode(
@@ -498,7 +483,7 @@ contract ExecutionManager is IExecutionManager, AccessControlExtended {
         uint32 siblingChainSlug_,
         uint256 msgValueMaxThreshold_,
         bytes calldata signature_
-    ) external override {
+    ) external {
         address feesUpdater = signatureVerifier__.recoverSigner(
             keccak256(
                 abi.encode(
@@ -526,12 +511,11 @@ contract ExecutionManager is IExecutionManager, AccessControlExtended {
     /**
      * @notice updates the transmission fee needed for transmission
      * @dev this function stores value against msg.sender hence expected to be called by transmit manager
-     * @inheritdoc IExecutionManager
      */
     function setTransmissionMinFees(
         uint32 remoteChainSlug_,
         uint128 fees_
-    ) external override {
+    ) external {
         transmissionMinFees[msg.sender][remoteChainSlug_] = fees_;
     }
 
@@ -568,7 +552,7 @@ contract ExecutionManager is IExecutionManager, AccessControlExtended {
         uint32 siblingChainSlug_,
         address switchboard_,
         uint128 amount_
-    ) external override {
+    ) external {
         if (totalSwitchboardFees[switchboard_][siblingChainSlug_] < amount_)
             revert InsufficientFees();
 
@@ -590,7 +574,7 @@ contract ExecutionManager is IExecutionManager, AccessControlExtended {
     function withdrawTransmissionFees(
         uint32 siblingChainSlug_,
         uint128 amount_
-    ) external override {
+    ) external {
         if (
             totalExecutionAndTransmissionFees[siblingChainSlug_]
                 .totalTransmissionFees < amount_