feat: meta operator registry

Author: madlabman

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,31 @@
 - `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.
+- 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

@@ -54,7 +54,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

script/DeployBase.s.sol

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

script/curated/DeployBase.s.sol

@@ -17,6 +17,7 @@ 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 +52,6 @@ struct GateCurveParams {
     uint256 allowedExitDelay;
     uint256 exitDelayFee;
     uint256 maxWithdrawalRequestFee;
-    uint256 depositAllocationWeight;
 }
 
 struct CuratedGateConfig {
@@ -64,6 +64,7 @@ struct CuratedGateConfig {
 struct CuratedDeployParams {
     // Lido addresses
     address lidoLocatorAddress;
+    address norAddress;
     address aragonAgent;
     address easyTrackEVMScriptExecutor;
     address proxyAdmin;
@@ -115,8 +116,6 @@ struct CuratedDeployParams {
     uint256 defaultAllowedExitDelay;
     uint256 defaultExitDelayFee;
     uint256 defaultMaxWithdrawalRequestFee;
-    uint256 defaultDepositAllocationWeight;
-    uint256 identifiedCommunityStakersGateDepositAllocationWeight;
     // Curated gates
     CuratedGateConfig[] curatedGates;
     // GateSeal
@@ -149,6 +148,7 @@ abstract contract DeployBase is Script {
     Verifier public verifier;
     HashConsensus public hashConsensus;
     ParametersRegistry public parametersRegistry;
+    MetaOperatorRegistry public metaOperatorsRegistry;
     OperatorsData public operatorsData;
     CuratedGateFactory public curatedGateFactory;
     address[] public curatedGateInstances;
@@ -219,6 +219,18 @@ abstract contract DeployBase is Script {
 
             accounting = Accounting(_deployProxy(deployer, address(dummyImpl)));
 
+            MetaOperatorRegistry metaOperatorsRegistryImpl = new MetaOperatorRegistry(
+                    address(curatedModule),
+                    config.norAddress
+                );
+            metaOperatorsRegistry = MetaOperatorRegistry(
+                _deployProxy(
+                    config.proxyAdmin,
+                    address(metaOperatorsRegistryImpl)
+                )
+            );
+            metaOperatorsRegistry.initialize(deployer);
+
             oracle = FeeOracle(_deployProxy(deployer, address(dummyImpl)));
 
             FeeDistributor feeDistributorImpl = new FeeDistributor({
@@ -277,9 +289,7 @@ abstract contract DeployBase is Script {
                     defaultAllowedExitDelay: config.defaultAllowedExitDelay,
                     defaultExitDelayFee: config.defaultExitDelayFee,
                     defaultMaxWithdrawalRequestFee: config
-                        .defaultMaxWithdrawalRequestFee,
-                    defaultDepositAllocationWeight: config
-                        .defaultDepositAllocationWeight
+                        .defaultMaxWithdrawalRequestFee
                 })
             });
 
@@ -386,12 +396,6 @@ abstract contract DeployBase is Script {
                     curveId,
                     params.maxWithdrawalRequestFee
                 );
-                if (params.depositAllocationWeight != 0) {
-                    parametersRegistry.setDepositAllocationWeight(
-                        curveId,
-                        params.depositAllocationWeight
-                    );
-                }
             }
             accounting.revokeRole(
                 accounting.MANAGE_BOND_CURVES_ROLE(),
@@ -407,7 +411,8 @@ abstract contract DeployBase is Script {
                 lidoLocator: config.lidoLocatorAddress,
                 parametersRegistry: address(parametersRegistry),
                 accounting: address(accounting),
-                exitPenalties: address(exitPenalties)
+                exitPenalties: address(exitPenalties),
+                metaOperatorsRegistry: address(metaOperatorsRegistry)
             });
 
             {
@@ -687,6 +692,14 @@ abstract contract DeployBase is Script {
             deployJson.set("ChainId", chainId);
             deployJson.set("CuratedModule", address(curatedModule));
             deployJson.set("CuratedModuleImpl", address(curatedModuleImpl));
+            deployJson.set(
+                "MetaOperatorRegistry",
+                address(metaOperatorsRegistry)
+            );
+            deployJson.set(
+                "MetaOperatorRegistryImpl",
+                address(metaOperatorsRegistryImpl)
+            );
             deployJson.set("ParametersRegistry", address(parametersRegistry));
             deployJson.set(
                 "ParametersRegistryImpl",

script/curated/DeployHoodi.s.sol

@@ -10,6 +10,7 @@ contract DeployHoodi is DeployBase {
     constructor() DeployBase("hoodi", 560048) {
         // Lido addresses
         config.lidoLocatorAddress = 0xe2EF9536DAAAEBFf5b1c130957AB3E80056b06D8;
+        config.norAddress = 0x5cDbE1590c083b5A2A64427fAA63A7cfDB91FbB5;
         config.aragonAgent = 0x0534aA41907c9631fae990960bCC72d75fA7cfeD;
         config
             .easyTrackEVMScriptExecutor = 0x79a20FD0FA36453B2F45eAbab19bfef43575Ba9E;

script/curated/DeployLocalDevNet.s.sol

@@ -10,6 +10,7 @@ contract DeployLocalDevNet is DeployBase {
     constructor() DeployBase("local-devnet", vm.envUint("DEVNET_CHAIN_ID")) {
         // Lido addresses
         config.lidoLocatorAddress = vm.envAddress("CSM_LOCATOR_ADDRESS");
+        config.norAddress = vm.envAddress("CSM_NOR_ADDRESS");
         config.aragonAgent = vm.envAddress("CSM_ARAGON_AGENT_ADDRESS");
         config.easyTrackEVMScriptExecutor = vm.envAddress(
             "EVM_SCRIPT_EXECUTOR_ADDRESS"

script/curated/DeployMainnet.s.sol

@@ -10,6 +10,7 @@ contract DeployMainnet is DeployBase {
     constructor() DeployBase("mainnet", 1) {
         // Lido addresses
         config.lidoLocatorAddress = 0xC1d0b3DE6792Bf6b4b37EccdcC24e45978Cfd2Eb;
+        config.norAddress = 0x55032650b14df07b85bF18A3a3eC8E0Af2e028d5;
         config.aragonAgent = 0x3e40D73EB977Dc6a537aF587D48316feE66E9C8c;
         config
             .easyTrackEVMScriptExecutor = 0xFE5986E06210aC1eCC1aDCafc0cc7f8D63B3F977;

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

@@ -9,7 +9,7 @@ import { SafeCast } from "@openzeppelin/contracts/utils/math/SafeCast.sol";
 import { BaseModule } from "./abstract/BaseModule.sol";
 
 import { IStakingModule, IStakingModuleV2 } from "./interfaces/IStakingModule.sol";
-import { NodeOperator } from "./interfaces/IBaseModule.sol";
+import { IBaseModule, NodeOperator } from "./interfaces/IBaseModule.sol";
 import { ICSModule } from "./interfaces/ICSModule.sol";
 
 import { TopUpQueueLib, TopUpQueueItem, newTopUpQueueItem } from "./lib/TopUpQueueLib.sol";
@@ -332,6 +332,16 @@ contract CSModule is ICSModule, BaseModule {
         }
     }
 
+    /// @inheritdoc IBaseModule
+    function onNodeOperatorBondCurveUpdated(
+        uint256 nodeOperatorId
+    ) external override(IBaseModule) {
+        _updateDepositableValidatorsCount({
+            nodeOperatorId: nodeOperatorId,
+            incrementNonceIfUpdated: true
+        });
+    }
+
     /// @inheritdoc IStakingModule
     function getStakingModuleSummary()
         external
@@ -401,8 +411,8 @@ contract CSModule is ICSModule, BaseModule {
         uint256 nodeOperatorId,
         uint256 newCount,
         bool incrementNonceIfUpdated
-    ) internal override {
-        super._applyDepositableValidatorsCount(
+    ) internal override returns (bool changed) {
+        changed = super._applyDepositableValidatorsCount(
             nodeOperatorId,
             newCount,
             incrementNonceIfUpdated