docs(staking): expand Owner/Operator/ValidatorWallet key model section#440
Conversation
- Add diagram, per-key comparison table, and key rotation / loss scenarios (operator rotation incl. two-step transfer; owner key loss is unrecoverable) - Clarify that the node only ever holds the operator key and map community terms (node/validator/rewards key) to it - Setup guide: warn to back up the owner wallet (wizard export only covers the operator key), link to the key model section - Fix misleading config comment: operator performs consensus duties for the ValidatorWallet, it does not own it Rotation functions verified present on deployed Bradbury ValidatorWallet implementation (0x5f9DCdcEfAB114da8387d6Bf8cFd75E8d3bb6BB4).
✅ Deploy Preview for genlayer-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughDocumentation-only changes clarify the roles of owner and operator keys relative to the ValidatorWallet. The staking concepts page adds a Mermaid diagram, a responsibilities table, and key rotation/loss guidance. The setup guide and config.yaml comments are updated to correct the operator address description and add owner key backup instructions. ChangesOwner/Operator Terminology Clarification
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Why
Validators repeatedly ask which keys their node actually holds and how the owner/operator/ValidatorWallet relate — most recently someone discovered their previous devops never handed over the owner key. The current section is three sentences and doesn't answer the questions people actually have (which keys live on the server, what happens if a key is lost, where funds go).
What changed
staking.mdx— expanded Owner, Operator, and ValidatorWallet (heading/anchor unchanged):setOperatoror the two-stepinitiateOperatorTransferflow; owner key loss is unrecoverable (no admin override) — with a warning callout for inherited validatorssetup-guide.mdx+content/validators/config.yaml:content/validators/config.yaml, the source the setup-guide block is regenerated from)Verification
genlayer-consensusv0.6 (ValidatorWalletBlueprint.sol,StakingValidators.sol, spec04-staking-system.md) andgenlayer-node(config/keystore/signer wiring)setOperator,initiateOperatorTransfer/completeOperatorTransfer/cancelOperatorTransfer,transferOwnership) confirmed present on the deployed Bradbury ValidatorWallet implementation (0x5f9DCdcEfAB114da8387d6Bf8cFd75E8d3bb6BB4) via read-onlycastcalls; confirmed no recovery path exists for a lost owner keypnpm dev: Mermaid diagram renders, callouts/tables/anchors and cross-page links workSummary by CodeRabbit