Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Introduce BuilderInternals trait #14592

Open
fgimenez opened this issue Feb 19, 2025 · 0 comments
Open

Introduce BuilderInternals trait #14592

fgimenez opened this issue Feb 19, 2025 · 0 comments
Assignees
@fgimenez
Labels
A-sdk Related to reth's use as a library

Comments

@fgimenez
Copy link
Member

The node builder architecture currently uses BuilderComponentsAdapter as the internal state holder for NodeBuilderWithComponents. This adapter manages:

  • Node configuration (NodeConfig with chain spec)
  • Type system adaptation (NodeTypesAdapter)
  • Component building (through a generic ComponentsBuilder)
  • Node extensions (through AddOns)

Introducing a trait boundary could simplify the interaction between these components.
@fgimenez fgimenez added the A-sdk Related to reth's use as a library label Feb 19, 2025
@fgimenez fgimenez self-assigned this Feb 19, 2025
@fgimenez fgimenez moved this from Todo to In Progress in Reth Tracker Feb 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fgimenez fgimenez

Labels
A-sdk Related to reth's use as a library
Projects
Reth Tracker
Status: In Progress
Milestone
No milestone
Development

No branches or pull requests
1 participant
@fgimenez