Skip to content

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

Draft
prestwich wants to merge 1 commit intofeat/host-notifier-traitfrom
feat/host-rpc
Draft

[hackathon] feat: add signet-host-rpc crate with RpcHostNotifier#107
prestwich wants to merge 1 commit intofeat/host-notifier-traitfrom
feat/host-rpc

Conversation

@prestwich
Copy link
Member

@prestwich prestwich commented Mar 13, 2026

Summary

  • Add new signet-host-rpc crate providing RPC WebSocket-based HostNotifier implementation
  • RpcHostNotifier<P>: follows host chain via newHeads subscription
  • RpcChainSegment + RpcBlock: wrapper types for RPC-fetched blocks/receipts
  • Ring buffer reorg detection via VecDeque block buffer
  • Epoch-aware safe/finalized block tag caching
  • RpcHostNotifierBuilder: fluent builder API
  • PR [hackathon] refactor: decouple signet-node from reth via HostNotifier trait #100 review feedback: MissingBlock error variant, tighter field visibility, zero-guard on backfill_batch_size

PR 4 of 5 in the host context adapter refactor.
Precursor: #105 (independent of #106)

Test plan

  • cargo clippy -p signet-host-rpc --all-features --all-targets
  • cargo +nightly fmt

🤖 Generated with Claude Code

@prestwich prestwich requested a review from a team as a code owner March 13, 2026 19:55
@prestwich prestwich marked this pull request as draft March 13, 2026 19:56
@prestwich prestwich changed the title feat: add signet-host-rpc crate with RpcHostNotifier [hackathon] feat: add signet-host-rpc crate with RpcHostNotifier Mar 13, 2026
@prestwich prestwich force-pushed the feat/host-notifier-trait branch from 1654673 to 09aad33 Compare March 13, 2026 20:25
@prestwich
Copy link
Member Author

[Claude Code]

Note: CI failures here are expected — test fixes land in #108 (fix/node-tests-host-notifier). PRs #104, #105, #106, #107, and #108 are intended to be merged as a unit; they are broken out into separate PRs for review isolation.

@prestwich prestwich force-pushed the feat/host-notifier-trait branch from 57bf22c to 17492c1 Compare March 17, 2026 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant