Skip to content

chore: remove validatorReexecute config option#22024

Merged
PhilWindle merged 2 commits into
merge-train/spartanfrom
palla/remove-validator-reexecute
Mar 26, 2026
Merged

chore: remove validatorReexecute config option#22024
PhilWindle merged 2 commits into
merge-train/spartanfrom
palla/remove-validator-reexecute

Conversation

@spalladino

@spalladino spalladino commented Mar 25, 2026

Copy link
Copy Markdown
Contributor

Removes the validatorReexecute config option, VALIDATOR_REEXECUTE env var, and all references across 22 files. Reexecution is now always enabled — the flag was true by default and no longer needs to be configurable.

🤖 Generated with Claude Code

Reexecution is now always enabled — the validatorReexecute flag was true
by default and no longer needs to be configurable. The shouldReexecute
logic now checks partOfCommittee and slashBroadcastedInvalidBlockPenalty
directly without gating on the removed flag.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@spalladino spalladino added the ci-no-fail-fast Sets NO_FAIL_FAST in the CI so the run is not aborted on the first failure label Mar 25, 2026
Move reexecution mocks into beforeEach since reexecution is now always
enabled, remove enableReexecution helper, and fix HA peer test to use
matching archive root and txHashes.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@PhilWindle PhilWindle merged commit 8ca3737 into merge-train/spartan Mar 26, 2026
11 checks passed
@PhilWindle PhilWindle deleted the palla/remove-validator-reexecute branch March 26, 2026 12:49
github-merge-queue Bot pushed a commit that referenced this pull request Mar 27, 2026
BEGIN_COMMIT_OVERRIDE
fix: only clear provenBlockNumber when it exceeds prune point (#21946)
chore: (A-779) load all accounts before calling
LogService.#getSecretsForSenders (#21923)
fix: align staging-public mana target with testnet/mainnet (#21983)
chore: (A-777) add warn logs for regressive path in block synchronizer
(#21925)
fix: fully validate txs retrieved from tx file store (#21988)
refactor: extract checkpoint proposal handling to ProposalHandler
(#21999)
fix: unbounded memory in calldataRetriever (#22004)
fix(p2p): check peer rate limit before global to prevent quota
starvation (#21997)
fix(p2p): evict expired failed-auth-handshake entries on heartbeat
(#21992)
chore: defensively handle skipPushProposedBlocksToArchiver (#22017)
chore: bump testnet prover resource profile to prod-hi-tps (#22019)
chore: (A-835) remove unused serializer (#22037)
fix(p2p): remove disconnected peers from scoring maps (#22009)
fix(e2e): set anvilSlotsInAnEpoch in slashing tests (#21869)
fix(ethereum): Audit fixes A-810, A-812 (nonce race, isEscapeHatchOpen
logging) (#21948)
chore: remove old TxPool implementation (#22028)
fix: Fix blob encoding when uploaded from proposals (#22045)
chore: Adds /cycle and /fix skills. Also configures linear mcp server
(#22043)
chore: remove validatorReexecute config option (#22024)
fix(sequencer): use last L1 slot of L2 slot as eth_simulateV1 timestamp
(#22023)
docs(simulator): clarify teardown gas billing is intentional (#22057)
chore: revert account loading optimization in log service (#22062)
fix: use DateProvider in PeerScoring (#22070)
fix(aztec.js): preserve extraHashedArgs in DeployMethod.with() (#22053)
fix(p2p): replace process.exit() with graceful shutdown in worker
cleanup (#22046)
chore: merge next (#22089)
fix(stdlib): correct NoteDao size (#22068)
feat: improve blob download from filestores (#22096)
fix: remove stale tx_pool v1 benchmark reference (#22104)
END_COMMIT_OVERRIDE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-no-fail-fast Sets NO_FAIL_FAST in the CI so the run is not aborted on the first failure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants