Skip to content

fix(207): switch pump.fun from fee wallet to dex_solana.trades - #1940

Merged
Flotapponnier merged 2 commits into
devfrom
fix/bench-207-pumpfun-dex-trades
Aug 13, 2026
Merged

fix(207): switch pump.fun from fee wallet to dex_solana.trades#1940
Flotapponnier merged 2 commits into
devfrom
fix/bench-207-pumpfun-dex-trades

Conversation

@Flotapponnier

Copy link
Copy Markdown
Collaborator

Summary

  • pump.fun cut bonding-curve fees to 0% on 2026-08-07; fee wallets still got micro-SOL on every tx from rent/account-creation side-effects, inflating unique tx count by ~60% (9.7M → was wrong)
  • Switch pump.fun to dex_solana.trades WHERE project IN ('pumpdotfun','pumpswap') which indexes actual swap events from both programs
  • Validated on Dune query 8318587: 6.0M unique tx vs 9.7M via fee wallet
  • All other providers (GMGN, Axiom, Fomo, Trojan) unchanged — fee wallet detection still accurate since they charge fees on every trade
  • Updated YAML: title, metric label, methodology bullets, disclaimer, FAQ entry

Coherence check

All providers now measure COUNT(DISTINCT tx_id) over 24h of real swap activity. Data source differs (fee wallet vs Spellbook) but the underlying event is the same.

Test plan

  • VPS harness rebuild with DUNE_QUERY_ID=8318587
  • materialize-worker rebuild to pick up updated YAML
  • Staging: verify bench 207 shows ~6M for pump.fun

After the 0% bonding-curve fee cut (2026-08-07), pump.fun fee wallets still
received micro-SOL on every tx (rent side-effects), inflating unique tx count
by ~60%. Switch to dex_solana.trades WHERE project IN ('pumpdotfun','pumpswap')
which indexes actual swap events. Validated: 6.0M vs 9.7M (Dune query 8318587).

Also updates bench YAML: title, metric label, methodology notes, disclaimer,
and FAQ entry explaining the data-source switch.
@Flotapponnier
Flotapponnier merged commit b3a90e1 into dev Aug 13, 2026
1 check failed
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