feat: add signet-host-reth crate with RethHostNotifier by prestwich · Pull Request #106 · init4tech/node-components

prestwich · 2026-03-13T19:53:59Z

[Claude Code]

Summary

Adds the signet-host-reth crate — a reth ExEx-backed implementation of the HostNotifier trait introduced in #105.

RethHostNotifier: wraps reth's ExExContext notification stream, provider, and event sender
RethChain: owning wrapper around reth Chain implementing Extractable with O(1) metadata access
RethAliasOracle + RethAliasOracleFactory: moved from signet-node to this crate
decompose_exex_context(): splits an ExExContext into a RethHostNotifier + plain config types
rpc_config_from_args() / serve_config_from_args(): extract reth-free RPC config from reth CLI args
Updates signet-node-tests to use the new SignetNodeBuilder API (explicit with_notifier, with_blob_cacher, with_serve_config, with_rpc_config)
Simplifies SignetNodeBuilder generics from 6 typestate params to 3
RevertRange: Replaces Arc<C> with lightweight RevertRange { first, tip } in ChainReverted and ChainReorged variants — only committed chain segments carry full block/receipt data (ENG-2069)
Adds node-side validation that revert range tip covers stored rollup tip, ensuring notification consistency

PR stack

This is part of the host context adapter refactor. PRs #104 and #105 are already merged.

Merge in order:

feat: add signet-host-reth crate with RethHostNotifier #106 ← signet-host-reth (this PR) — ready for review

feat: add signet-host-rpc crate with RpcHostNotifier #107 ← signet-host-rpc (based on feat: add signet-host-reth crate with RethHostNotifier #106)

fix(node-tests): adapt test infra to HostNotifier trait #108 ← test infra cleanup (based on feat: add signet-host-rpc crate with RpcHostNotifier #107)

Test plan

cargo clippy -p signet-host-reth --all-targets — clean
cargo clippy -p signet-node-types --all-targets — clean
cargo clippy -p signet-node --all-features --all-targets — clean
cargo clippy -p signet-node-tests --all-targets — clean
cargo +nightly fmt — clean
cargo t -p signet-node-types — 3 unit + 3 doc-tests pass

🤖 Generated with Claude Code

- Add compile-time size/align assertions for transmute safety (chain.rs) - Add tracing::error before set_head panics for operator observability - Log swallowed provider errors when reading safe/finalized block numbers - Replace `as` casts with `From` in config conversion for type safety - Restore condensed safety rationale for Latest alias oracle - Add unit tests for config conversion helpers (5 tests) - Add rustdoc usage examples to RethChain and decompose_exex_context - Extract test_blob_cacher helper to reduce duplication in node-tests eyre convention violation tracked in ENG-2041. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

alias.rs was modified in the base branch but deleted here; it moves to signet-host-reth in PR #106. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

- Delete unused ExtractableChainShim from blobber (RethChain replaces it) - Replace From<eyre::Report> with named RethHostError::notification() - Handle set_backfill_thresholds(None) by restoring defaults - Add blank line between module groups in host-reth lib.rs Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

…tions Reverted chain segments in HostNotificationKind (ChainReverted and ChainReorged) now carry a lightweight RevertRange (first, tip block numbers) instead of Arc<C>. No consumer reads block bodies or receipts from reverted segments — only block number range is needed. Adds node-side validation that the revert range tip covers the stored rollup tip, ensuring notification consistency with stored state. This unblocks signet-host-rpc where old/reverted blocks may not be available from the RPC node. Closes ENG-2069 Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

Fraser999

One tiny, non-blocking suggestion.

Fraser999 · 2026-03-19T14:42:00Z

+        // state: the range tip must be at or above the host block that
+        // produced our current rollup tip.
+        let rollup_tip = self.last_rollup_block()?;
+        let range_tip_ru = self.constants.host_block_to_rollup_block_num(tip).unwrap_or_default();


We'd get a clearer error message if we did something like .ok_or_else(|| eyre!(...))? instead of falling through to the ensure! below with range_tip_ru set to 0.

Instead of silently defaulting to 0 via unwrap_or_default(), return an explicit error when the revert range tip doesn't map to a rollup block. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

prestwich requested a review from a team as a code owner March 13, 2026 19:54

prestwich mentioned this pull request Mar 13, 2026

feat: add signet-host-rpc crate with RpcHostNotifier #107

Merged

3 tasks

prestwich marked this pull request as draft March 13, 2026 19:56

prestwich changed the title ~~feat: add signet-host-reth crate with RethHostNotifier~~ [hackathon] feat: add signet-host-reth crate with RethHostNotifier Mar 13, 2026

prestwich mentioned this pull request Mar 13, 2026

[hackathon] refactor: decouple signet-node from reth via HostNotifier trait #100

Closed

4 tasks

prestwich force-pushed the feat/host-notifier-trait branch from fdae9f3 to 1654673 Compare March 13, 2026 20:11

prestwich force-pushed the feat/host-reth branch from 466023f to b835214 Compare March 13, 2026 20:11

prestwich force-pushed the feat/host-notifier-trait branch from 1654673 to 09aad33 Compare March 13, 2026 20:25

prestwich force-pushed the feat/host-reth branch from b835214 to e46fd40 Compare March 13, 2026 20:44

prestwich mentioned this pull request Mar 14, 2026

[hackathon] feat: add HostNotifier trait and decouple node from ExExContext #105

Merged

8 tasks

prestwich mentioned this pull request Mar 15, 2026

fix(node-tests): adapt test infra to HostNotifier trait #108

Merged

3 tasks

prestwich force-pushed the feat/host-reth branch from d8ec66c to df37553 Compare March 17, 2026 16:03

prestwich force-pushed the feat/host-notifier-trait branch 2 times, most recently from 0758243 to b01e622 Compare March 17, 2026 16:20

prestwich force-pushed the feat/host-reth branch from df37553 to d8ec66c Compare March 17, 2026 16:20

prestwich force-pushed the feat/host-notifier-trait branch 2 times, most recently from 4697dff to b01e622 Compare March 17, 2026 16:23

prestwich force-pushed the feat/host-reth branch from d8ec66c to 812357c Compare March 17, 2026 16:41

prestwich force-pushed the feat/host-notifier-trait branch from b01e622 to 57bf22c Compare March 17, 2026 16:41

prestwich force-pushed the feat/host-reth branch from 812357c to 737a3bd Compare March 17, 2026 17:03

prestwich force-pushed the feat/host-notifier-trait branch from 57bf22c to 17492c1 Compare March 17, 2026 17:03

prestwich force-pushed the feat/host-reth branch from 737a3bd to 67bd023 Compare March 18, 2026 12:21

Fraser999 reviewed Mar 18, 2026

View reviewed changes

prestwich force-pushed the feat/host-reth branch 2 times, most recently from 7d86ca0 to 384fc12 Compare March 18, 2026 13:50

prestwich marked this pull request as ready for review March 18, 2026 13:55

prestwich requested a review from Fraser999 March 18, 2026 13:55

prestwich changed the base branch from feat/host-notifier-trait to develop March 18, 2026 13:56

prestwich closed this Mar 18, 2026

prestwich reopened this Mar 18, 2026

prestwich changed the title ~~[hackathon] feat: add signet-host-reth crate with RethHostNotifier~~ feat: add signet-host-reth crate with RethHostNotifier Mar 18, 2026

feat: add signet-host-reth crate with RethHostNotifier

087ee7e

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

prestwich force-pushed the feat/host-reth branch from 384fc12 to 087ee7e Compare March 18, 2026 15:51

Fraser999 approved these changes Mar 19, 2026

View reviewed changes

prestwich requested a review from Fraser999 March 19, 2026 14:22

Fraser999 approved these changes Mar 19, 2026

View reviewed changes

fix: use ok_or_eyre for clearer error on unmapped revert range tip

3a0f4d5

Instead of silently defaulting to 0 via unwrap_or_default(), return an explicit error when the revert range tip doesn't map to a rollup block. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

prestwich enabled auto-merge (squash) March 19, 2026 14:50

prestwich merged commit b5281c5 into develop Mar 19, 2026
6 checks passed

prestwich mentioned this pull request Mar 20, 2026

refactor(blobber): abstract blob sources and remove reth deps #112

Merged

10 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat: add signet-host-reth crate with RethHostNotifier#106

feat: add signet-host-reth crate with RethHostNotifier#106
prestwich merged 4 commits into
developfrom
feat/host-reth

prestwich commented Mar 13, 2026 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Fraser999 left a comment

Uh oh!

Fraser999 Mar 19, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

prestwich commented Mar 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

PR stack

Test plan

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Fraser999 left a comment

Choose a reason for hiding this comment

Uh oh!

Fraser999 Mar 19, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

prestwich commented Mar 13, 2026 •

edited

Loading