Factor out base module and interfaces (#641)

## Description

All modules now inherit from the `Module` contract, implementing the
shared `IBaseModule` interface. Satellite contracts rely on the shared
interface.

## Checklist

- [x] Appropriate PR labels applied
- [x] Test coverage maintained (`just coverage`)
  - [x] No need to add/update tests
  - [ ] Tests are added/updated
- [ ] Documentation maintained
  - [ ] No need to update
  - [ ] Updated

Author: madlabman

script/curated/DeployBase.s.sol

@@ -395,7 +395,7 @@ abstract contract DeployBase is Script {
                 moduleType: config.moduleType,
                 lidoLocator: config.lidoLocatorAddress,
                 parametersRegistry: address(parametersRegistry),
-                _accounting: address(accounting),
+                accounting: address(accounting),
                 exitPenalties: address(exitPenalties)
             });
 

script/fork-helpers/NodeOperators.s.sol

@@ -4,12 +4,15 @@
 pragma solidity 0.8.24;
 
 import { Script } from "forge-std/Script.sol";
+
+import { IVEBO } from "src/interfaces/IVEBO.sol";
+import { IStakingRouter } from "src/interfaces/IStakingRouter.sol";
+import { NodeOperator, WithdrawnValidatorInfo } from "src/interfaces/IBaseModule.sol";
+
 import { DeploymentFixtures } from "test/helpers/Fixtures.sol";
+import { Utilities } from "test/helpers/Utilities.sol";
+
 import { ForkHelpersCommon } from "./Common.sol";
-import { IVEBO } from "../../src/interfaces/IVEBO.sol";
-import { Utilities } from "../../test/helpers/Utilities.sol";
-import { IStakingRouter } from "../../src/interfaces/IStakingRouter.sol";
-import { NodeOperator, WithdrawnValidatorInfo } from "../../src/interfaces/ICSModule.sol";
 
 contract NodeOperators is
     Script,

src/Accounting.sol

@@ -14,7 +14,7 @@ import { AssetRecovererLib } from "./lib/AssetRecovererLib.sol";
 import { FeeSplits } from "./lib/FeeSplits.sol";
 
 import { IStakingModule } from "./interfaces/IStakingModule.sol";
-import { ICSModule, NodeOperatorManagementProperties } from "./interfaces/ICSModule.sol";
+import { IBaseModule, NodeOperatorManagementProperties } from "./interfaces/IBaseModule.sol";
 import { IAccounting } from "./interfaces/IAccounting.sol";
 import { IFeeDistributor } from "./interfaces/IFeeDistributor.sol";
 import { IERC20Permit } from "./interfaces/IERC20Permit.sol";
@@ -39,7 +39,7 @@ contract Accounting is
         keccak256("SET_BOND_CURVE_ROLE");
     bytes32 public constant RECOVERER_ROLE = keccak256("RECOVERER_ROLE");
 
-    ICSModule public immutable MODULE;
+    IBaseModule public immutable MODULE;
     IFeeDistributor public immutable FEE_DISTRIBUTOR;
     /// @dev DEPRECATED
     /// @custom:oz-renamed-from feeDistributor
@@ -77,7 +77,7 @@ contract Accounting is
             revert ZeroFeeDistributorAddress();
         }
 
-        MODULE = ICSModule(module);
+        MODULE = IBaseModule(module);
         FEE_DISTRIBUTOR = IFeeDistributor(feeDistributor);
 
         _disableInitializers();