feat(solana): add WhaleStreet DEX integration#9286
feat(solana): add WhaleStreet DEX integration#9286karimhass wants to merge 3 commits intoduneanalytics:mainfrom
Conversation
- Add WhaleStreet DEX models (stg_raw_swaps, base_trades, trades, schema) - Uses optimized 3-step semi-join pattern for tokens_solana.transfers - project_start_date 2026-02-03 with 1-day cap for CI efficiency - Add whalestreet_solana_base_trades to dex_solana_base_trades union
PR SummaryMedium Risk Overview The new pipeline extracts swaps from Written by Cursor Bugbot for commit 9c5313d. Configure here. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
Comment @cursor review or bugbot run to trigger another review on this PR
| post_hook='{{ expose_spells(\'["solana"]\', | ||
| "project", | ||
| "whalestreet", | ||
| \'["Eekeyguy"]\') }}') |
There was a problem hiding this comment.
Trades view uses expose_spells instead of hide_spells
Medium Severity
The whalestreet_solana_trades view uses expose_spells as its post_hook, but every other per-project DEX trades view (aquifer, goonfi, humidifi, obric, tessera, goosefx) uses hide_spells(). Only the main dex_solana_trades table uses expose_spells. This will incorrectly make the WhaleStreet convenience view publicly visible in Dune's data explorer (setting dune.public to true), when it is meant to be a hidden internal view that filters from the already-public dex_solana_trades.
Co-authored-by: Cursor <cursoragent@cursor.com>
79cf3d0 to
ade3eb3
Compare
Co-authored-by: Cursor <cursoragent@cursor.com>
…tics#9286) Co-authored-by: Cursor <cursoragent@cursor.com>


Thank you for contributing to Spellbook 🪄
Please open the PR in draft and mark as ready when you want to request a review.
Description:
[...]
quick links for more information: