ChainSafe / ChainSafe/lodestar

Clean up post EIP-8282

Open
#9,558 0 comments 0 reactions 0 assignees View on GitHub
spec-gloas
Dominant language
TypeScript
Stars
1.4k
Forks
483
Avg merge
1d 16h
Merged PRs (30d)
150

Description

> We need to remove all references to the builder voluntary exit mechanism that is now removed. Eg. remove getBuilderVoluntaryExitSignatureSet and merge getVoluntaryExitsSignatureSets and getVoluntaryExitSignatureSet into one.
>
> Could be a future refactoring opportunity

yes there is more stuff we might wanna clean up, eg. there is no need to validate pending deposits anymore so we can probably remove the cache or at least keep it only for the fork transition, this branch still needs to be cleaned up carefully

_Originally posted by @nflaig in https://github.com/ChainSafe/lodestar/issues/9541#issuecomment-4776788126_

Contributor guide

Open the contributing guide

Research direction

Start by searching the Lodestar repository for getBuilderVoluntaryExitSignatureSet, getVoluntaryExitsSignatureSets, getVoluntaryExitSignatureSet, and pending-deposit validation or cache references. Trace the fork-transition paths before changing shared logic. Done means obsolete builder voluntary-exit references are removed, signature-set handling is consolidated, and pending-deposit caching is retained only where still needed.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
blockchain
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.