[BCN] Exclude ERC20 transfers from Alchemy native tx history#4163
Open
leolambo wants to merge 4 commits into
Open
[BCN] Exclude ERC20 transfers from Alchemy native tx history#4163leolambo wants to merge 4 commits into
leolambo wants to merge 4 commits into
Conversation
Token receives were surfacing as tiny native-ETH receives because the default category list included erc20. Token sends still appear via the underlying external call (0 ETH, gas spent), matching Moralis and the EVM CSP convention where native and token histories are separate.
leolambo
force-pushed
the
alchemyNativeTxFilter
branch
from
5c3f874 to
181fe67
Compare
…myNativeTxFilter
Sends and receives now use different category sets so erc20 sends still surface (as 0-ETH txs with gas spent) while erc20 receives stay out of native history. Cross-category dedupe keeps the external row when a single hash appears under multiple categories (e.g. a defi tx). Adapter no longer sets a category; that's owned by EVMListTransactionsStream downstream, matching the Moralis convention.
The previous endpoint hits the public sepolia.base.org RPC, which rate- limits the 200-call burst getErc20Transfers makes when scanning a 10k block range in 100-block windows. drpc.org's public Base sepolia endpoint handles the burst cleanly (verified 100/100 sequential).
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Token receives were found as native-ETH receives because the default category list included erc20. Token sends still appear via the underlying external call (0 ETH, gas spent).
Changelog
Testing Notes
Add any helpful notes for reviewers to test your code here.
Checklist