lidofinance · eddort · Jun 13, 2025 · Jun 13, 2025
diff --git a/contracts/0.8.9/StakingRouter.sol b/contracts/0.8.9/StakingRouter.sol
@@ -364,22 +364,6 @@ contract StakingRouter is AccessControlEnumerable, BeaconChainDepositor, Version
         );
     }
 
-    /// @notice Updates the number of the refunded validators in the staking module with the given
-    /// node operator id.
-    /// @param _stakingModuleId Id of the staking module.
-    /// @param _nodeOperatorId Id of the node operator.
-    /// @param _refundedValidatorsCount New number of refunded validators of the node operator.
-    /// @dev The function is restricted to the `STAKING_MODULE_MANAGE_ROLE` role.
-    function updateRefundedValidatorsCount(
-        uint256 _stakingModuleId,
-        uint256 _nodeOperatorId,
-        uint256 _refundedValidatorsCount
-    ) external onlyRole(STAKING_MODULE_MANAGE_ROLE) {
-        _getIStakingModuleById(_stakingModuleId).updateRefundedValidatorsCount(
-            _nodeOperatorId, _refundedValidatorsCount
-        );
-    }
-
     /// @notice Reports the minted rewards to the staking modules with the specified ids.
     /// @param _stakingModuleIds Ids of the staking modules.
     /// @param _totalShares Total shares minted for the staking modules.
@@ -767,6 +751,7 @@ contract StakingRouter is AccessControlEnumerable, BeaconChainDepositor, Version
 
         /// @notice The number of validators that can't be withdrawn, but deposit costs were
         /// compensated to the Lido by the node operator.
+        /// @dev [deprecated] Refunded validators processing has been removed, this field is no longer used.
         uint256 refundedValidatorsCount;
 
         /// @notice A time when the penalty for stuck validators stops applying to node operator rewards.
@@ -819,15 +804,14 @@ contract StakingRouter is AccessControlEnumerable, BeaconChainDepositor, Version
             uint256 targetLimitMode,
             uint256 targetValidatorsCount,
             /* uint256 stuckValidatorsCount */,
-            uint256 refundedValidatorsCount,
+            /* uint256 refundedValidatorsCount */,
             /* uint256 stuckPenaltyEndTimestamp */,
             uint256 totalExitedValidators,
             uint256 totalDepositedValidators,
             uint256 depositableValidatorsCount
         ) = stakingModule.getNodeOperatorSummary(_nodeOperatorId);
         summary.targetLimitMode = targetLimitMode;
         summary.targetValidatorsCount = targetValidatorsCount;
-        summary.refundedValidatorsCount = refundedValidatorsCount;
         summary.totalExitedValidators = totalExitedValidators;
         summary.totalDepositedValidators = totalDepositedValidators;
         summary.depositableValidatorsCount = depositableValidatorsCount;

diff --git a/contracts/0.8.9/interfaces/IStakingModule.sol b/contracts/0.8.9/interfaces/IStakingModule.sol
@@ -160,11 +160,6 @@ interface IStakingModule {
         bytes calldata _exitedValidatorsCounts
     ) external;
 
-    /// @notice Updates the number of the refunded validators for node operator with the given id
-    /// @param _nodeOperatorId Id of the node operator
-    /// @param _refundedValidatorsCount New number of refunded validators of the node operator
-    function updateRefundedValidatorsCount(uint256 _nodeOperatorId, uint256 _refundedValidatorsCount) external;
-
     /// @notice Updates the limit of the validators that can be used for deposit
     /// @param _nodeOperatorId Id of the node operator
     /// @param _targetLimitMode target limit mode

diff --git a/test/0.8.9/contracts/StakingModule__MockForStakingRouter.sol b/test/0.8.9/contracts/StakingModule__MockForStakingRouter.sol
@@ -194,10 +194,6 @@ contract StakingModule__MockForStakingRouter is IStakingModule {
         emit Mock__ExitedValidatorsCountUpdated(_nodeOperatorIds, _stuckValidatorsCounts);
     }
 
-    function updateRefundedValidatorsCount(uint256 _nodeOperatorId, uint256 _refundedValidatorsCount) external {
-        emit Mock__RefundedValidatorsCountUpdated(_nodeOperatorId, _refundedValidatorsCount);
-    }
-
     function updateTargetValidatorsLimits(
         uint256 _nodeOperatorId,
         uint256 _targetLimitMode,

diff --git a/test/0.8.9/contracts/StakingModule__MockForTriggerableWithdrawals.sol b/test/0.8.9/contracts/StakingModule__MockForTriggerableWithdrawals.sol
@@ -99,10 +99,6 @@ contract StakingModule__MockForTriggerableWithdrawals is IStakingModule {
         return;
     }
 
-    function updateRefundedValidatorsCount(uint256, uint256) external pure override {
-        return;
-    }
-
     function updateExitedValidatorsCount(bytes calldata, bytes calldata) external pure override {
         return;
     }

diff --git a/test/0.8.9/stakingRouter/stakingRouter.module-sync.test.ts b/test/0.8.9/stakingRouter/stakingRouter.module-sync.test.ts
@@ -127,7 +127,7 @@ describe("StakingRouter.sol:module-sync", () => {
       1, // targetLimitMode
       100n, // targetValidatorsCount
       0n, // stuckValidatorsCount
-      5n, // refundedValidatorsCount
+      0n, // refundedValidatorsCount
       0n, // stuckPenaltyEndTimestamp
       50, // totalExitedValidators
       1000n, // totalDepositedValidators
@@ -364,25 +364,6 @@ describe("StakingRouter.sol:module-sync", () => {
     });
   });
 
-  context("updateRefundedValidatorsCount", () => {
-    const NODE_OPERATOR_ID = 0n;
-    const REFUNDED_VALIDATORS_COUNT = 10n;
-
-    it("Reverts if the caller does not have the role", async () => {
-      await expect(
-        stakingRouter
-          .connect(user)
-          .updateRefundedValidatorsCount(moduleId, NODE_OPERATOR_ID, REFUNDED_VALIDATORS_COUNT),
-      ).to.be.revertedWithOZAccessControlError(user.address, await stakingRouter.STAKING_MODULE_MANAGE_ROLE());
-    });
-
-    it("Redirects the call to the staking module", async () => {
-      await expect(stakingRouter.updateRefundedValidatorsCount(moduleId, NODE_OPERATOR_ID, REFUNDED_VALIDATORS_COUNT))
-        .to.emit(stakingModule, "Mock__RefundedValidatorsCountUpdated")
-        .withArgs(NODE_OPERATOR_ID, REFUNDED_VALIDATORS_COUNT);
-    });
-  });
-
   context("reportRewardsMinted", () => {
     it("Reverts if the caller does not have the role", async () => {
       await expect(
-Original file line number
+Diff line change
@@ Expand Up @@
             return;
         }
-        function updateRefundedValidatorsCount(uint256, uint256) external pure override {
-            return;
-        }
         function updateExitedValidatorsCount(bytes calldata, bytes calldata) external pure override {
             return;
         }
@@ Expand Down @@