From ddb8617a194ae8483d6ca0444bef15cf313f5757 Mon Sep 17 00:00:00 2001 From: bing Date: Mon, 22 Jun 2026 21:38:24 +0200 Subject: [PATCH] fix: do not set start index for deposit request in non-electra forks This was causing a spec test failure in v1.7.0-alpha.10 spec tests, reproducible with `zig build test:spec_tests -Dpreset=minimal -Dspec_tests.filters="fulu transition transition_with_deposit_request_right_after_fork"`. See: https://github.com/ChainSafe/lodestar/blob/f6b2af6879db8092cdaf0dad41b3a098ff391a4a/packages/state-transition/src/block/processDepositRequest.ts#L133 --- src/state_transition/block/process_deposit_request.zig | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/state_transition/block/process_deposit_request.zig b/src/state_transition/block/process_deposit_request.zig index 175fb5b2c..3315f9ac1 100644 --- a/src/state_transition/block/process_deposit_request.zig +++ b/src/state_transition/block/process_deposit_request.zig @@ -6,9 +6,11 @@ const PendingDeposit = types.electra.PendingDeposit.Type; const c = @import("constants"); pub fn processDepositRequest(comptime fork: ForkSeq, state: *BeaconState(fork), deposit_request: *const DepositRequest) !void { - const deposit_requests_start_index = try state.depositRequestsStartIndex(); - if (deposit_requests_start_index == c.UNSET_DEPOSIT_REQUESTS_START_INDEX) { - try state.setDepositRequestsStartIndex(deposit_request.index); + if (comptime fork == .electra) { + const deposit_requests_start_index = try state.depositRequestsStartIndex(); + if (deposit_requests_start_index == c.UNSET_DEPOSIT_REQUESTS_START_INDEX) { + try state.setDepositRequestsStartIndex(deposit_request.index); + } } const pending_deposit = PendingDeposit{