RPC Standards #029
Transcript
Ask about this call
Answered from this call's summary and transcript only.
AI-generated — verify against the recording.
Call summary
Decisions
- •Glamsterdam/Amsterdam spec changes without tests will use a tracking issue or long-running branch; tests backfilled once fork spec is final — 00:27:52
- •Byte-type regex PR (#81415) approved to merge — restricting to lowercase aligns with all other byte types in spec — 00:39:32
- •Error code rework still in progress; no merge of current PR until Sina's updated positive-number proposal is ready; broader ACD discussion planned — 01:00:07
Highlights
- Pr Reviews:
- ·execution-apis PR #814 (eth_getRawTransaction endpoints): all clients merged except ethrex (PR approved, not merged); CI spell-check failure is upstream noise — rerun to green, ready to merge — 00:23:13
- ·PR adding slot number to block header schema (Glamsterdam): marked for Amsterdam label; needs tests deferred until fork spec is final — 00:24:43
- ·PR adding 3 missing eth_getRaw* endpoints (by hash/block): all EL clients already implement these; blocked on Go Ethereum PR #35227 merge + test regeneration — 00:33:28
- ·Regex fix for single-byte type (PR #81415): change to allow single zero in decimal numbers is correct; uppercase hex oddity in byte type resolved — approved to merge — 00:37:22
- Error Codes:
- ·New error code direction: define all new codes as strictly positive integers; positive = reliable/new, negative = legacy/fuzzy; Go Ethereum considering a legacy-mode mapper for transition period — 00:55:58
- ·Sina's updated proposal pending; grouping debate: user-friendly categories (gas errors, nonce errors) vs. implementation-layer categories (execution, simulation, validation); to be brought to ACD for broader input — 01:00:16
- Debug Execution Witness:
- ·debug_executionWitness has no cross-client spec agreement: Geth uses JSON header with empty key; Reth/Nethermind/Erigon use RLP bytes with populated key; Geth and Reth also expose a separate by-hash method — needs spec decision — 01:06:07
Action Items
- •Geth (Felix) — Merge Go Ethereum PR #35227, cut new release, regenerate execution-apis I/O tests for eth_getRaw* PR — 00:33:52
- •Boma — Open a tracked issue for debug_executionWitness wire format disagreement; tag relevant stakeholders (Kev) — 01:08:45
Meeting chat7
- Boma
- Felix (Geth)
sorry I have some wifi trouble
- Boma
- Boma
- Chase Wright
GitHub was just being dumb, https://github.com/ethereum/execution-apis/pull/814 passes CI now
- Chase Wright
We can close https://github.com/ethereum/execution-apis/pull/812 too
- Manuel Arto
Reacted to "We can close https..." with 👍
Key decisions
Glamsterdam/Amsterdam spec changes without tests will use a tracking issue or long-running branch; tests backfilled once fork spec is final
spec changes without tests will use a tracking issue or long-running branchByte-type regex PR (#81415) approved to merge — restricting to lowercase aligns with all other byte types in spec
Byte-type regex PR approved to mergeError code rework still in progress; no merge of current PR until Sina's updated positive-number proposal is ready; broader ACD discussion planned
Error code rework still in progress

