Luganodes recovery on mainnet at electra1 fork

[fridrik01]

See proposal in https://forum.berachain.com/t/validator-state-recovery-following-deposit-signature-mismatch-incident/196

[codecov]

Codecov Report

❌ Patch coverage is 0% with 22 lines in your changes missing coverage. Please review.
✅ Project coverage is 60.64%. Comparing base (e474a72) to head (0e959e7).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
state-transition/core/state_processor_fixes.go	0.00%	22 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2822      +/-   ##
==========================================
- Coverage   60.75%   60.64%   -0.12%     
==========================================
  Files         354      354              
  Lines       16652    16673      +21     
==========================================
- Hits        10117    10111       -6     
- Misses       5747     5773      +26     
- Partials      788      789       +1     

Files with missing lines	Coverage Δ
state-transition/core/state_processor_fixes.go	8.00% <0.00%> (-5.80%)	⬇️

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

Pull Request Overview

This PR introduces automated recovery for Luganode validators by injecting a hardcoded deposit during the Electra1 fork on mainnet.

• Inserted a new processElectra1Fixes hook into the fork processing flow
• Added processLuganodesRecovery to create a one-time BLS deposit for Luganode validators
• Defined constants and credentials for the hardcoded deposit in a new file

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
state-transition/core/state_processor_forks.go	Calls processElectra1Fixes before logging the Electra1 fork
state-transition/core/state_processor_fixes.go	Implements fork fixes including Luganode deposit recovery

Comments suppressed due to low confidence (2)

state-transition/core/state_processor_fixes.go:55

• This new recovery logic should be covered by unit tests to ensure the deposit is only added once and errors are handled correctly.

func (sp *StateProcessor) processLuganodesRecovery(st *state.StateDB) error {

state-transition/core/state_processor_fixes.go:41

• There's a TODO placeholder for withdrawal credentials. Ensure this is updated to the correct value before release to avoid injecting invalid credentials.

var fixCredentials = ctypes.WithdrawalCredentials{0x00} // TODO: update this to a correct value

[rezzmah]

Did the math on my side and this value looks correct

[rezzmah]

How can we validate the credentials?

[codingwithmanny]

@rezbera we could ask them to verify their comet credentials.

[rezzmah]

lgtm pending someone tripple checking the credentials

[calbera]

utACK