refactor!: change fee distribution events

lidofinance · Jan 13, 2025 · 852d7e0 · 852d7e0
1 parent 846c0df
commit 852d7e0
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 6 deletions.
diff --git a/src/CSFeeDistributor.sol b/src/CSFeeDistributor.sol
@@ -82,7 +82,7 @@ contract CSFeeDistributor is
         }
 
         STETH.transferShares(ACCOUNTING, sharesToDistribute);
-        emit FeeDistributed(nodeOperatorId, sharesToDistribute);
+        emit OperatorFeeDistributed(nodeOperatorId, sharesToDistribute);
     }
 
     /// @inheritdoc ICSFeeDistributor
@@ -121,7 +121,7 @@ contract CSFeeDistributor is
             );
         }
 
-        emit LastDistributed(distributed);
+        emit ModuleFeeDistributed(distributed);
 
         // NOTE: Make sure off-chain tooling provides a distinct CID of a log even for empty reports, e.g. by mixing
         // in a frame identifier such as reference slot to a file.

diff --git a/src/interfaces/ICSFeeDistributor.sol b/src/interfaces/ICSFeeDistributor.sol
@@ -8,7 +8,10 @@ pragma solidity 0.8.24;
 
 interface ICSFeeDistributor is IAssetRecovererLib {
     /// @dev Emitted when fees are distributed
-    event FeeDistributed(uint256 indexed nodeOperatorId, uint256 shares);
+    event OperatorFeeDistributed(
+        uint256 indexed nodeOperatorId,
+        uint256 shares
+    );
 
     /// @dev Emitted when distribution data is updated
     event DistributionDataUpdated(
@@ -21,7 +24,7 @@ interface ICSFeeDistributor is IAssetRecovererLib {
     event DistributionLogUpdated(string logCid);
 
     /// @dev It logs how many shares were distributed in the latest report
-    event LastDistributed(uint256 shares);
+    event ModuleFeeDistributed(uint256 shares);
 
     error ZeroAccountingAddress();
     error ZeroStEthAddress();

diff --git a/test/CSFeeDistributor.t.sol b/test/CSFeeDistributor.t.sol
@@ -161,7 +161,7 @@ contract CSFeeDistributorTest is CSFeeDistributorTestBase {
         );
 
         vm.expectEmit(true, true, true, true, address(feeDistributor));
-        emit ICSFeeDistributor.FeeDistributed(nodeOperatorId, shares);
+        emit ICSFeeDistributor.OperatorFeeDistributed(nodeOperatorId, shares);
 
         vm.prank(address(accounting));
         feeDistributor.distributeFees({
@@ -420,7 +420,7 @@ contract CSFeeDistributorTest is CSFeeDistributorTestBase {
         );
 
         vm.expectEmit(true, true, true, true, address(feeDistributor));
-        emit ICSFeeDistributor.LastDistributed(shares);
+        emit ICSFeeDistributor.ModuleFeeDistributed(shares);
 
         vm.expectEmit(true, true, true, true, address(feeDistributor));
         emit ICSFeeDistributor.DistributionLogUpdated(logCid);