Skip to content

rpc: align Mgt Pending block #19409

Merged
lupin012 merged 5 commits intomainfrom
align_mgt_pendingBlock
Mar 2, 2026
Merged

rpc: align Mgt Pending block #19409
lupin012 merged 5 commits intomainfrom
align_mgt_pendingBlock

Conversation

@lupin012
Copy link
Contributor

Align pending block behavior to be as close to Geth as possible

@lupin012 lupin012 force-pushed the align_mgt_pendingBlock branch from 432a2b9 to 4f4b8cd Compare February 23, 2026 21:07
@lupin012 lupin012 marked this pull request as ready for review February 23, 2026 21:43
@lupin012 lupin012 requested a review from canepat as a code owner February 23, 2026 21:43
lupin012 and others added 3 commits February 25, 2026 19:29
…torageAt

Remove early error returns for pending block number in GetBalance, GetCode
and GetStorageAt. rpchelper.GetBlockNumber already handles PendingBlockNumber
gracefully by falling back to the latest committed block when no pending block
is available, so the explicit error was unnecessary and broke callers like
bind.transact (PendingCodeAt) used during contract deployment in tests such
as TestShutterBlockBuilding.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@lupin012 lupin012 merged commit 84a5aad into main Mar 2, 2026
25 checks passed
@lupin012 lupin012 deleted the align_mgt_pendingBlock branch March 2, 2026 18:13
sudeepdino008 pushed a commit that referenced this pull request Mar 4, 2026
Align pending block behavior to be as close to Geth as possible

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
sudeepdino008 pushed a commit that referenced this pull request Mar 4, 2026
Align pending block behavior to be as close to Geth as possible

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants