Skip to content

feat(1474): expose ledger-emitted events from the node#1487

Draft
m2ux wants to merge 1 commit into
mainfrom
feat/1474-expose-ledger-events
Draft

feat(1474): expose ledger-emitted events from the node#1487
m2ux wants to merge 1 commit into
mainfrom
feat/1474-expose-ledger-events

Conversation

@m2ux
Copy link
Copy Markdown
Contributor

@m2ux m2ux commented May 11, 2026

Summary

Expose ledger-emitted events from the node so downstream consumers (notably the indexer) no longer have to re-execute transactions to reconstruct them.

🐛 Issue 📐 Engineering


Motivation

The node currently discards every event emitted by the ledger when processing a transaction (see ledger/src/versions/common/api/ledger.rs). To recover this data, the indexer re-applies every transaction locally — duplicating compute the node has already done and creating a divergence risk between the node and any consumer that needs to know what actually happened in a block. Exposing these events from the node removes the duplication and gives every downstream service a single, trustworthy source of truth.

The work package will also assess the network-traffic and architectural implications of publishing all events, as flagged on the originating issue, before settling on a publication mechanism.


Changes

Implementation (coming next):

  • Surface ledger-emitted events through the node's existing event/state APIs (mechanism TBD during plan-prepare)
  • Update the call-site in ledger/src/versions/common/api/ledger.rs so events are propagated rather than discarded
  • Add coverage that asserts events are observable end-to-end from the node

📌 Submission Checklist

  • Changes are backward-compatible (or flagged if breaking)
  • Pull request description explains why the change is needed
  • Self-reviewed the diff
  • I have included a change file, or skipped for this reason: [reason]
  • If the changes introduce a new feature, I have bumped the node minor version
  • Update documentation (if relevant)
  • No new todos introduced

🔱 Fork Strategy

  • Node Runtime Update
  • Node Client Update
  • Other
  • N/A

🗹 TODO before merging

  • Ready for review

Signed-off-by: Mike Clay <mike.clay@shielded.io>
@m2ux m2ux self-assigned this May 11, 2026
@m2ux m2ux requested a review from gilescope May 11, 2026 13:44
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