feat: enable transaction history for all networks #172
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
Move binary search transaction discovery from EVMAdapter to NetworkAdapter base class, enabling it for all networks (Arbitrum, Optimism, Base, Polygon, BNB) without code duplication. - Add initTxSearch() method to NetworkAdapter base class - Move getAddressTransactions() implementation to base class - Update all adapters to call initTxSearch() in constructor - Remove stub implementations from network-specific adapters
|
🚀 Preview: https://pr-172--openscan.netlify.app |
MatiasOS
approved these changes
Jan 21, 2026
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Description
Enable binary search transaction discovery for all networks by moving the implementation from EVMAdapter to the NetworkAdapter base class. Previously, only Ethereum mainnet had working transaction history - now it works on Arbitrum, Optimism, Base, Polygon, and BNB.
Related Issue
N/A
Type of Change
Changes Made
txSearchproperty andinitTxSearch()method to NetworkAdapter base classgetAddressTransactions()implementation from EVMAdapter to NetworkAdapterinitTxSearch(client)in their constructors:getAddressTransactions()implementations from all network-specific adaptersResult: 126 lines of duplicated code removed (93 added, 219 deleted)
Screenshots (if applicable)
N/A
Checklist
npm run format:fixandnpm run lint:fixnpm run typecheckwith no errorsnpm run test:runAdditional Notes
The binary search algorithm uses nonce/balance state changes to discover transactions without relying on indexers. This approach works on any EVM-compatible network with standard RPC methods (
eth_getTransactionCount,eth_getBalance).