Skip to content

feat: integrate journal chain into node#146

Open
Fraser999 wants to merge 1 commit into
mainfrom
fraser/eng-2015/add-journal-chain
Open

feat: integrate journal chain into node#146
Fraser999 wants to merge 1 commit into
mainfrom
fraser/eng-2015/add-journal-chain

Conversation

@Fraser999
Copy link
Copy Markdown
Contributor

@Fraser999 Fraser999 commented May 20, 2026

Wires signet-journal-chain into SignetNode: every committed block is encoded as a Journal::V1, pushed into a store-less chain task, and streamed out over the new /journal WebSocket.

Summary

  • New JournalConfig in signet-node-config (env-var configurable; defaults 64 MiB / 200 entries / 100-journal subscriber lag).
  • SignetNode owns the chain handle, a bounded mpsc sender, and an in-memory rolling previous_journal_hash. Main loop uses tokio::select! to observe the chain ingestion task alongside host notifications.
  • signet-rpc serving layer takes an Option<JournalChainHandle>. When set, HTTP and WS transports merge journal_router and shut down via a shared CancellationToken with a 5s watchdog.

Follow-up (ENG-2017)

previous_journal_hash is in-memory only - not persisted, not rewound on revert. First journal after any restart or host revert is rejected by the chain. Ten reorg tests are #[ignore]d until this is fixed.

@Fraser999 Fraser999 requested a review from a team as a code owner May 20, 2026 15:05
Copy link
Copy Markdown
Contributor Author

Fraser999 commented May 20, 2026

@Fraser999 Fraser999 marked this pull request as draft May 20, 2026 15:07
@Fraser999 Fraser999 changed the title integrate journal chain into node feat: integrate journal chain into node May 20, 2026
@Fraser999 Fraser999 force-pushed the fraser/eng-2015/add-journal-chain branch 2 times, most recently from dba28ba to 6da1ac1 Compare May 26, 2026 12:56
@Fraser999 Fraser999 force-pushed the fraser/eng-2015/add-journal-chain branch from 6da1ac1 to 52a4458 Compare May 30, 2026 16:43
@Fraser999 Fraser999 force-pushed the fraser/eng-2015/add-journal-chain branch from 52a4458 to 627967d Compare June 1, 2026 13:27
@Fraser999 Fraser999 marked this pull request as ready for review June 2, 2026 11:48
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