fix: add fallback handler for getProposerDuties#9518
Conversation
There was a problem hiding this comment.
Code Review
This pull request addresses a regression (#9380) where requesting proposer duties for the next epoch could throw an error. It updates proposerShufflingDecisionRoot to return null if the decision slot is in the future relative to the state's slot, preventing out-of-bounds state access. Additionally, the validator API is updated to handle fallback logic gracefully when the proposer shuffling decision root is not yet in the state, and improved error messages and logging are introduced. Unit tests have been added to verify these changes. There are no review comments, so I have no feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## unstable #9518 +/- ##
============================================
- Coverage 52.58% 52.58% -0.01%
============================================
Files 848 848
Lines 60875 60871 -4
Branches 4483 4483
============================================
- Hits 32013 32009 -4
Misses 28800 28800
Partials 62 62 🚀 New features to boost your workflow:
|
Performance Report🚀🚀 Significant benchmark improvement detected
Full benchmark results
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 918490971a
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
Motivation
get_proposer_duties()v1Description
=> fallback to
get_proposer_duties()v1, which is how it worked for v1.43AI Assistance Disclosure