feat: track fallback hyper evm flow completed oft #434
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
…HyperEVMFlowCompleted tracking - Extract `getEventsFromTransactionReceipts` and `formatAndSaveEvents` into reusable generic utilities in `eventProcessing.ts` - Refactor `hyperEvmExecutor.ts` to export formatting functions instead of processing logic - Add `FallbackHyperEVMFlowCompleted` event tracking to OFT indexer alongside existing `SimpleTransferFlowCompleted` events - Replace duplicate event extraction methods in `CCTPIndexerDataHandler` with generic utilities - Update `EventDecoder` methods to use static references for consistency - Add integration test coverage for `FallbackHyperEVMFlowCompleted` event processing
NikolasHaimerl
changed the base branch from
master
to
nikolas/track-SimpleTransferCompleted-OFT
…/track-FallbackHyperEVMFlowCompleted-Oft
NikolasHaimerl
commented
Nov 19, 2025
| blocksTimestamps, | ||
| ), | ||
| formatAndSaveSimpleTransferFlowCompletedEvents( | ||
| formatAndSaveEvents( |
Contributor
Author
There was a problem hiding this comment.
We can make this change for all the other format functions too, but I wanted to keep the PR reviewable and limit it to the hyperEVMexecutor contract.
NikolasHaimerl
commented
Nov 19, 2025
| "0x8c5261668696ce22758910d05bab8f186d6eb247ceac2af2e82c7dc17669b036"; | ||
| const eventAbi = ["event MessageSent (bytes message)"]; | ||
| let events: MessageSentLog[] = this.decodeTransactionReceiptLogs( | ||
| let events: MessageSentLog[] = EventDecoder.decodeTransactionReceiptLogs( |
Contributor
Author
There was a problem hiding this comment.
These changes are necessary due to typescript compiler errors. The class is not defined if the static methods are called without initializing the class.
…exer data handlers
NikolasHaimerl
requested review from
amateima,
ashwinrava,
dohaki,
melisaguevara,
mrice32 and
nicholaspai
as code owners
…/track-FallbackHyperEVMFlowCompleted-Oft
…/track-FallbackHyperEVMFlowCompleted-Oft
amateima
reviewed
Nov 19, 2025
Contributor
amateima
left a comment
amateima
left a comment
There was a problem hiding this comment.
Big plus for the refactoring! Simple changes but very effective 👍
Comment on lines
+211
to
+215
| fallbackHyperEVMFlowCompletedEvents = getEventsFromTransactionReceipts( | ||
| transactionReceipts, | ||
| dstOftHandlerAddress, | ||
| EventDecoder.decodeFallbackHyperEVMFlowCompletedEvents, | ||
| ); |
Contributor
There was a problem hiding this comment.
This operation should be limited to HyperEVM only, right?
Contributor
Author
There was a problem hiding this comment.
I believe so, the OFTdstHandler only exists on hyperEVM right now AFAIK.
amateima
approved these changes
Nov 21, 2025
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.
Add FallbackHyperEVMFlowCompleted events from the OFT flow to the indexer.
Useful links: https://hyperevmscan.io/tx/0x05ccdbd44e8ffbed8f057762f40dee73fb218049347705d88f839dfe3c368c52#eventlog#5
Adds two generic functions to remove duplicate code:
formatAndSaveEventsandgetEventsFromTransactionReceipts