Skip to content

feat(solana): add BisonFi DEX integration#9283

Closed
karimhass wants to merge 3 commits intoduneanalytics:mainfrom
karimhass:karim/bisonfi-dex-integration
Closed

feat(solana): add BisonFi DEX integration#9283
karimhass wants to merge 3 commits intoduneanalytics:mainfrom
karimhass:karim/bisonfi-dex-integration

Conversation

@karimhass
Copy link
Copy Markdown
Contributor

  • Add BisonFi 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 bisonfi_solana_base_trades to dex_solana_base_trades union
  • Fix contributor name: Eekeguy → Eekeyguy

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:

- Add BisonFi 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 bisonfi_solana_base_trades to dex_solana_base_trades union
- Fix contributor name: Eekeguy → Eekeyguy
@cursor
Copy link
Copy Markdown

cursor Bot commented Feb 6, 2026

PR Summary

Medium Risk
Adds new incremental DEX trade extraction and joins against the large tokens_solana.transfers table, so correctness and performance of the semi-join/index arithmetic are the main risks; existing protocols are largely unaffected aside from the expanded union.

Overview
Adds a new BisonFi Solana DEX integration by introducing incremental dbt models to extract swap instructions from solana.instruction_calls and transform them into standardized base_trades rows.

The new base_trades model joins swaps to tokens_solana.transfers via a 3-step keyset + EXISTS semi-join pattern (with uniqueness enforced via surrogate_key) and exposes a bisonfi_solana_trades view filtered from dex_solana_trades, plus accompanying schema.yml tests/metadata. dex_solana_base_trades is updated to union in bisonfi_solana_base_trades.

Written by Cursor Bugbot for commit 9ea4ea1. Configure here.

@github-actions github-actions Bot marked this pull request as draft February 6, 2026 21:15
@github-actions github-actions Bot added WIP work in progress dbt: solana covers the Solana dbt subproject labels Feb 6, 2026
@karimhass karimhass marked this pull request as ready for review February 6, 2026 21:37
@github-actions github-actions Bot added ready-for-review this PR development is complete, please review and removed WIP work in progress labels Feb 6, 2026
Co-authored-by: Cursor <cursoragent@cursor.com>
@karimhass karimhass force-pushed the karim/bisonfi-dex-integration branch from fe7efce to 73b6cc1 Compare February 8, 2026 22:25
Co-authored-by: Cursor <cursoragent@cursor.com>
@karimhass karimhass closed this Feb 10, 2026
@github-actions github-actions Bot locked and limited conversation to collaborators Feb 10, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

dbt: solana covers the Solana dbt subproject ready-for-review this PR development is complete, please review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant