Skip to content

fix: unhide tokens with low occurrences for some chains#9359

Open
maxime-oe wants to merge 1 commit into
mainfrom
fix/unhide-tokens-low-occurrences
Open

fix: unhide tokens with low occurrences for some chains#9359
maxime-oe wants to merge 1 commit into
mainfrom
fix/unhide-tokens-low-occurrences

Conversation

@maxime-oe

@maxime-oe maxime-oe commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Explanation

References

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed
  • I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them

Note

Low Risk
Narrow change to token visibility on one chain’s spam heuristic; no auth, persistence, or API contract changes.

Overview
EVM ERC-20 spam filtering in TokenDataSource now uses a per-chain minimum aggregator occurrence instead of a single global threshold of 3 for every chain.

A new MIN_TOKEN_OCCURRENCES_PER_CHAIN_OVERRIDE map sets eip155:5042 (Arc) to 1, because the chain is not fully released yet and legitimate tokens (e.g. EURC) can report fewer aggregator hits. Other chains still use MIN_TOKEN_OCCURRENCES (3). Custom-import bypass and the MUSD allowlist are unchanged; only the comparison uses minOccurrences from the override or default.

evmErc20Ids is typed as Caip19AssetId[] for consistency with parseCaipAssetType.

Reviewed by Cursor Bugbot for commit 94a157a. Bugbot is set up for automated code reviews on this repo. Configure here.

@maxime-oe maxime-oe force-pushed the fix/unhide-tokens-low-occurrences branch from bdf45cd to 94a157a Compare July 2, 2026 07:14
@maxime-oe maxime-oe marked this pull request as ready for review July 2, 2026 07:14
@maxime-oe maxime-oe requested a review from a team as a code owner July 2, 2026 07:14
@maxime-oe maxime-oe temporarily deployed to default-branch July 2, 2026 07:14 — with GitHub Actions Inactive
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