feat: meta operator registry

AGENTS.md

@@ -6,8 +6,9 @@
 - `test/`: Forge tests (`*.t.sol`), plus `test/fork/*` for fork/integration and deployment tests.
 - `script/`: Forge scripts (deploy/verify, per-chain variants).
 - `artifacts/`, `broadcast/`, `out/`, `cache/`: build, deploy, and fork outputs.
-- `lib/`, `node_modules/`: dependencies; see `remappings.txt`.
-- `docs/`, `digest/`: documentation and design notes.
+- `gists/`: small code examples related to the module.
+- `node_modules/`: dependencies; see `remappings.txt`.
+- `docs/`: documentation and design notes.
 
 ## Build, Test, and Development Commands
 
@@ -18,21 +19,32 @@
 - `just test-local`: spins up anvil fork, deploys, runs deployment+integration tests.
 - `just coverage` | `just coverage-lcov`: coverage (LCOV saved; see `lcov.html`).
 - Linting: `yarn lint:check` (prettier + solhint), `yarn lint:fix`, `yarn lint:solhint`.
+- Diff: `git diff --no-ext-diff`
 
 ## Coding Style & Naming Conventions
 
 - Formatting: Prettier + `prettier-plugin-solidity` (Solidity `printWidth=80`, `tabWidth=4`, spaces only).
 - Linting: Solhint (`.solhint.json`) with `solhint:recommended` and `solhint-plugin-lido-csm`.
 - Versions: enforce `pragma solidity 0.8.33` (`compiler-version` rule).
 - Naming: contracts/libraries `CamelCase` (e.g., `CSModule`, `AssetRecovererLib`), interfaces `IName` (rule: `interface-starts-with-i`).
+- The if statement body should be enclosed in a block, for example: `if (cond) { do(); }`
+- Inline assembly should be well documented, preferably every non-trivial line with its own comment.
 - Conventions: prefer custom errors, calldata parameters, and struct packing (gas rules). Immutable vars styled as constants.
+- Keep things in one function unless composable or reusable.
+- Prefer short variable names where possible. Prefer same length variable names for related things.
+- Prefer early returns to else statements.
+- While refactoring keep comments added from existing implementations where applicable.
+- Make sure external functions in contracts and interfaces have proper natspec.
+- Avoid using magic numbers, prefer re-using or defining constants.
+- When last I looked, the year was 2026.
 
 ## Testing Guidelines
 
 - Framework: Foundry/Forge with fuzzing (`fuzz.runs=256`).
 - Structure: unit tests in `test/*.t.sol`; fork suites under `test/fork/*` (deployment/integration).
 - Run: `just test-unit` for fast cycles; `CHAIN`/`RPC_URL` required for fork tests. Example: `export CHAIN=hoodi && export RPC_URL=<https-url>`.
 - Coverage: `just coverage-lcov` produces LCOV output (commit if relevant).
+- After making changes to the source code make sure you've either ran build command or unit tests.
 
 ## Commit & Pull Request Guidelines
 

Justfile

@@ -48,7 +48,7 @@ import "curated.just"
 default: clean deps build test-all
 
 build *args:
-    forge build --skip test --force {{args}}
+    forge build --skip test --skip script {{args}}
 
 clean:
     forge clean

foundry.toml

@@ -1,11 +1,11 @@
 [profile.default]
-solc = "0.8.33"
 evm_version = "osaka"
 optimizer = true
 optimizer_runs = 0 # REVISE IN THE FUTURE
 bytecode_hash = "none" # The metadata hash removed from the bytecode (not the metadata itself).
 # uncomment this to inspect storage layouts in build artifacts
 # extra_output = ["storageLayout"]
+verbosity = 3
 
 src = "src"
 out = "out"

script/csm/DeployBase.s.sol

@@ -91,7 +91,6 @@ struct DeployParams {
     uint256 defaultAllowedExitDelay;
     uint256 defaultExitDelayFee;
     uint256 defaultMaxElWithdrawalRequestFee;
-    uint256 defaultDepositAllocationWeight;
     // VettedGate
     address identifiedCommunityStakersGateManager;
     uint256 identifiedCommunityStakersGateCurveId;
@@ -115,7 +114,6 @@ struct DeployParams {
     uint256 identifiedCommunityStakersGateAllowedExitDelay;
     uint256 identifiedCommunityStakersGateExitDelayFee;
     uint256 identifiedCommunityStakersGateMaxElWithdrawalRequestFee;
-    uint256 identifiedCommunityStakersGateDepositAllocationWeight;
     // GateSeal
     address gateSealFactory;
     address sealingCommittee;
@@ -268,9 +266,7 @@ abstract contract DeployBase is Script {
                     defaultAllowedExitDelay: config.defaultAllowedExitDelay,
                     defaultExitDelayFee: config.defaultExitDelayFee,
                     defaultMaxElWithdrawalRequestFee: config
-                        .defaultMaxElWithdrawalRequestFee,
-                    defaultDepositAllocationWeight: config
-                        .defaultDepositAllocationWeight
+                        .defaultMaxElWithdrawalRequestFee
                 })
             });
 
@@ -469,15 +465,6 @@ abstract contract DeployBase is Script {
                 identifiedCommunityStakersGateBondCurveId,
                 config.identifiedCommunityStakersGateMaxElWithdrawalRequestFee
             );
-            if (
-                config.identifiedCommunityStakersGateDepositAllocationWeight !=
-                0
-            ) {
-                parametersRegistry.setDepositAllocationWeight(
-                    identifiedCommunityStakersGateBondCurveId,
-                    config.identifiedCommunityStakersGateDepositAllocationWeight
-                );
-            }
 
             feeDistributor.initialize({
                 admin: address(deployer),

script/curated/DeployBase.s.sol

@@ -16,7 +16,7 @@ import { FeeOracle } from "../../src/FeeOracle.sol";
 import { Verifier } from "../../src/Verifier.sol";
 import { ParametersRegistry } from "../../src/ParametersRegistry.sol";
 import { ExitPenalties } from "../../src/ExitPenalties.sol";
-import { OperatorsData } from "../../src/OperatorsData.sol";
+import { MetaOperatorRegistry } from "../../src/MetaOperatorRegistry.sol";
 import { CuratedGate } from "../../src/CuratedGate.sol";
 import { CuratedGateFactory } from "../../src/CuratedGateFactory.sol";
 
@@ -51,7 +51,6 @@ struct GateCurveParams {
     uint256 allowedExitDelay;
     uint256 exitDelayFee;
     uint256 maxElWithdrawalRequestFee;
-    uint256 depositAllocationWeight;
 }
 
 struct CuratedGateConfig {
@@ -115,8 +114,6 @@ struct CuratedDeployParams {
     uint256 defaultAllowedExitDelay;
     uint256 defaultExitDelayFee;
     uint256 defaultMaxElWithdrawalRequestFee;
-    uint256 defaultDepositAllocationWeight;
-    uint256 identifiedCommunityStakersGateDepositAllocationWeight;
     // Curated gates
     CuratedGateConfig[] curatedGates;
     // GateSeal
@@ -149,7 +146,7 @@ abstract contract DeployBase is Script {
     Verifier public verifier;
     HashConsensus public hashConsensus;
     ParametersRegistry public parametersRegistry;
-    OperatorsData public operatorsData;
+    MetaOperatorRegistry public metaRegistry;
     CuratedGateFactory public curatedGateFactory;
     address[] public curatedGateInstances;
     address internal curatedGateImpl;
@@ -218,8 +215,10 @@ abstract contract DeployBase is Script {
             );
 
             accounting = Accounting(_deployProxy(deployer, address(dummyImpl)));
-
             oracle = FeeOracle(_deployProxy(deployer, address(dummyImpl)));
+            metaRegistry = MetaOperatorRegistry(
+                _deployProxy(deployer, address(dummyImpl))
+            );
 
             FeeDistributor feeDistributorImpl = new FeeDistributor({
                 stETH: locator.lido(),
@@ -277,9 +276,7 @@ abstract contract DeployBase is Script {
                     defaultAllowedExitDelay: config.defaultAllowedExitDelay,
                     defaultExitDelayFee: config.defaultExitDelayFee,
                     defaultMaxElWithdrawalRequestFee: config
-                        .defaultMaxElWithdrawalRequestFee,
-                    defaultDepositAllocationWeight: config
-                        .defaultDepositAllocationWeight
+                        .defaultMaxElWithdrawalRequestFee
                 })
             });
 
@@ -386,12 +383,6 @@ abstract contract DeployBase is Script {
                     curveId,
                     params.maxElWithdrawalRequestFee
                 );
-                if (params.depositAllocationWeight != 0) {
-                    parametersRegistry.setDepositAllocationWeight(
-                        curveId,
-                        params.depositAllocationWeight
-                    );
-                }
             }
             accounting.revokeRole(
                 accounting.MANAGE_BOND_CURVES_ROLE(),
@@ -407,7 +398,8 @@ abstract contract DeployBase is Script {
                 lidoLocator: config.lidoLocatorAddress,
                 parametersRegistry: address(parametersRegistry),
                 accounting: address(accounting),
-                exitPenalties: address(exitPenalties)
+                exitPenalties: address(exitPenalties),
+                metaOperatorsRegistry: address(metaRegistry)
             });
 
             {
@@ -420,6 +412,20 @@ abstract contract DeployBase is Script {
 
             curatedModule.initialize({ admin: deployer });
 
+            MetaOperatorRegistry metaRegistryImpl = new MetaOperatorRegistry(
+                address(curatedModule),
+                locator.stakingRouter()
+            );
+
+            {
+                OssifiableProxy metaRegistryProxy = OssifiableProxy(
+                    payable(address(metaRegistry))
+                );
+                metaRegistryProxy.proxy__upgradeTo(address(metaRegistryImpl));
+                metaRegistryProxy.proxy__changeAdmin(config.proxyAdmin);
+            }
+            metaRegistry.initialize({ admin: deployer });
+
             ValidatorStrikes strikesImpl = new ValidatorStrikes({
                 module: address(curatedModule),
                 oracle: address(oracle),
@@ -454,20 +460,8 @@ abstract contract DeployBase is Script {
 
             strikes.initialize(deployer, address(ejector));
 
-            address operatorsDataImpl = address(
-                new OperatorsData(locator.stakingRouter())
-            );
-            operatorsData = OperatorsData(
-                _deployProxy(config.proxyAdmin, operatorsDataImpl)
-            );
-            operatorsData.initialize(deployer);
-
             curatedGateImpl = address(
-                new CuratedGate(
-                    address(curatedModule),
-                    config.stakingModuleId,
-                    address(operatorsData)
-                )
+                new CuratedGate(config.stakingModuleId, address(curatedModule))
             );
 
             curatedGateFactory = new CuratedGateFactory(curatedGateImpl);
@@ -638,12 +632,12 @@ abstract contract DeployBase is Script {
                 gate.revokeRole(gate.DEFAULT_ADMIN_ROLE(), deployer);
             }
 
-            operatorsData.grantRole(
-                operatorsData.DEFAULT_ADMIN_ROLE(),
+            metaRegistry.grantRole(
+                metaRegistry.DEFAULT_ADMIN_ROLE(),
                 config.aragonAgent
             );
-            operatorsData.revokeRole(
-                operatorsData.DEFAULT_ADMIN_ROLE(),
+            metaRegistry.revokeRole(
+                metaRegistry.DEFAULT_ADMIN_ROLE(),
                 deployer
             );
 
@@ -687,6 +681,11 @@ abstract contract DeployBase is Script {
             deployJson.set("ChainId", chainId);
             deployJson.set("CuratedModule", address(curatedModule));
             deployJson.set("CuratedModuleImpl", address(curatedModuleImpl));
+            deployJson.set("MetaOperatorRegistry", address(metaRegistry));
+            deployJson.set(
+                "MetaOperatorRegistryImpl",
+                address(metaRegistryImpl)
+            );
             deployJson.set("ParametersRegistry", address(parametersRegistry));
             deployJson.set(
                 "ParametersRegistryImpl",
@@ -705,8 +704,6 @@ abstract contract DeployBase is Script {
             deployJson.set("ValidatorStrikesImpl", address(strikesImpl));
             deployJson.set("HashConsensus", address(hashConsensus));
             deployJson.set("Verifier", address(verifier));
-            deployJson.set("OperatorsData", address(operatorsData));
-            deployJson.set("OperatorsDataImpl", address(operatorsDataImpl));
             deployJson.set("CuratedGateFactory", address(curatedGateFactory));
             deployJson.set("CuratedGates", curatedGateInstances);
             deployJson.set("LidoLocator", config.lidoLocatorAddress);
@@ -766,7 +763,10 @@ abstract contract DeployBase is Script {
                     address(gate)
                 );
             }
-            operatorsData.grantRole(operatorsData.SETTER_ROLE(), address(gate));
+            metaRegistry.grantRole(
+                metaRegistry.SET_OPERATOR_INFO_ROLE(),
+                address(gate)
+            );
             gate.grantRole(gate.PAUSE_ROLE(), config.resealManager);
             gate.grantRole(gate.RESUME_ROLE(), config.resealManager);
             gate.grantRole(
@@ -847,8 +847,8 @@ abstract contract DeployBase is Script {
             parametersRegistry.DEFAULT_ADMIN_ROLE(),
             config.secondAdminAddress
         );
-        operatorsData.grantRole(
-            operatorsData.DEFAULT_ADMIN_ROLE(),
+        metaRegistry.grantRole(
+            metaRegistry.DEFAULT_ADMIN_ROLE(),
             config.secondAdminAddress
         );
         for (uint256 i = 0; i < curatedGateInstances.length; i++) {

script/curated/DeployHoodi.s.sol

@@ -112,7 +112,6 @@ contract DeployHoodi is DeployBase {
             primaryGate.params.allowedExitDelay = 3 days; // TODO
             primaryGate.params.exitDelayFee = 0.02 ether; // TODO
             primaryGate.params.maxElWithdrawalRequestFee = 0.05 ether; // TODO
-            primaryGate.params.depositAllocationWeight = 1; // TODO: reconsider
         }
 
         // GateSeal

script/curated/DeployLocalDevNet.s.sol

@@ -126,7 +126,6 @@ contract DeployLocalDevNet is DeployBase {
             primaryGate.params.allowedExitDelay = 8 days; // TODO
             primaryGate.params.exitDelayFee = 0.05 ether; // TODO
             primaryGate.params.maxElWithdrawalRequestFee = 0.05 ether; // TODO
-            primaryGate.params.depositAllocationWeight = 1; // TODO: reconsider
         }
 
         // GateSeal

script/curated/DeployMainnet.s.sol

@@ -114,7 +114,6 @@ contract DeployMainnet is DeployBase {
             primaryGate.params.allowedExitDelay = 5 days; // TODO
             primaryGate.params.exitDelayFee = 0.05 ether; // TODO
             primaryGate.params.maxElWithdrawalRequestFee = 0.1 ether; // TODO
-            primaryGate.params.depositAllocationWeight = 1; // TODO: reconsider
         }
 
         // GateSeal

script/fork-helpers/SimulateVote.s.sol

@@ -195,14 +195,6 @@ contract SimulateVote is Script, ForkHelpersCommon {
             );
             vm.stopBroadcast();
         }
-        {
-            vm.startBroadcast(parametersRegistryAdmin);
-            parametersRegistry.setDefaultDepositAllocationWeight(
-                deployParams.defaultDepositAllocationWeight
-            );
-            vm.stopBroadcast();
-        }
-
         {
             OssifiableProxy oracleProxy = OssifiableProxy(
                 payable(deploymentConfig.oracle)

src/Accounting.sol

@@ -182,7 +182,7 @@ contract Accounting is
     ) external onlyRole(SET_BOND_CURVE_ROLE) {
         _onlyExistingNodeOperator(nodeOperatorId);
         BondCurve._setBondCurve(nodeOperatorId, curveId);
-        MODULE.updateDepositableValidatorsCount(nodeOperatorId);
+        MODULE.onNodeOperatorBondCurveUpdated(nodeOperatorId);
     }
 
     /// @inheritdoc IAccounting

src/CSModule.sol

@@ -386,15 +386,25 @@
     /// @dev The function does nothing in CSM, since the information about the operator balances is not used in the
     ///      module. If it becomes needed in the future, the method should be implemented and the oracle should deliver
     ///      the actual balances.
     function updateOperatorBalances(
         uint256[] calldata,
         uint256[] calldata,
         uint256[] calldata,
         uint256
     ) external {
         // NOTE: The function does nothing in CSM, see the docstring.
     }
 
+    /// @inheritdoc IBaseModule
+    function onNodeOperatorBondCurveUpdated(
+        uint256 nodeOperatorId
+    ) external override(IBaseModule) {
+        _updateDepositableValidatorsCount({
+            nodeOperatorId: nodeOperatorId,
+            incrementNonceIfUpdated: true
+        });
+    }
+
     /// @inheritdoc IStakingModule
     function getStakingModuleSummary()
         external
@@ -467,8 +477,8 @@
         uint256 nodeOperatorId,
         uint256 newCount,
         bool incrementNonceIfUpdated
-    ) internal override {
-        super._applyDepositableValidatorsCount(
+    ) internal override returns (bool changed) {
+        changed = super._applyDepositableValidatorsCount(
             no,
             nodeOperatorId,
             newCount,