feat(solana): add GoonFi v2 DEX integration

[eekey-cloud]

Summary

• Add GoonFi v2 DEX spellbook integration for Solana
• Includes staging table, base trades model, and trades view
• Updates existing goonfi schema with v2 models

Test plan

• CI tests pass for GoonFi v2 models
• Verify data accuracy against on-chain transactions

[cursor]

PR Summary

Medium Risk
Adds new incremental models that parse on-chain instructions/transfers and feeds them into the unified Solana DEX trades pipeline; risk is mainly around correctness/duplication due to surrogate key + instruction index assumptions and incremental merge boundaries.

Overview
Adds GoonFi v2 Solana DEX ingestion by introducing new goonfi_v2_solana models: a staging incremental model sourcing solana.instruction_calls, an incremental base_trades model that joins token transfers to derive bought/sold amounts/mints/vaults, and a trades view filtered to project='goonfi' + version=2.

Wires GoonFi v2 into the unified dex_solana_base_trades union so it flows through dex_solana_trades, updates goonfi_solana_trades to use expose_spells (instead of hide_spells), and extends goonfi/schema.yml with v2 model metadata and uniqueness/column tests.

^{Written by Cursor Bugbot for commit fac2b89. Configure here.}

[cursor]

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

[cursor]

Inconsistent incremental predicates mixing block_time and block_date

Low Severity

The goonfi_v2_solana_base_trades.sql model inconsistently mixes block_time and block_date for incremental predicates. The config (line 9) and swaps CTE (line 34) use block_time, while the transfers filter (line 68) uses block_date. This differs from the v1 goonfi model and the aquifer model, which consistently use block_date throughout. The inconsistency makes the code harder to maintain and could lead to subtle data synchronization issues around date boundaries.

 

[eekey-cloud]

Closing PR