feat: lazy execution #578
Merged
feat: lazy execution #578
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
NikolasHaimerl
requested review from
amateima,
ashwinrava,
dohaki,
melisaguevara,
mrice32 and
nicholaspai
as code owners
NikolasHaimerl
changed the base branch from
stage
to
nikolas/ws-specific-rpc-urls
NikolasHaimerl
commented
Jan 8, 2026
| ): Promise<boolean> => { | ||
| // Check if receipt is present in payload | ||
| const receipt = payload.transactionReceipt; | ||
| const receipt = await payload.transactionReceipt; |
Contributor
Author
There was a problem hiding this comment.
This is where the rpc call is actually made, at the place where we need it.
amateima
approved these changes
Jan 8, 2026
NikolasHaimerl
merged commit 64b93d1
into
nikolas/ws-specific-rpc-urls
7 of 9 checks passed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The websocket indexer uses the transaction receipt during the processing of events. However, not every event needs the receipt during its processing. To avoid unnecessary RPC calls we make the fetching of the transaction receipt a lazy execution, meaning we only make the call when we actually need it. We use the package https://www.npmjs.com/package/p-lazy?activeTab=readme for this purpose.