refactor: share EVM chain list across dex and gas sector spells

[0xBoxer]

Thank you for contributing to Spellbook 🪄

Please open the PR in draft and mark as ready when you want to request a review.

Description:

Follow-up to #9895 addressing @jeff-dude's review note that the EVM chain list ends up defined in two paired sector models, which is easy to miss when adding a new chain.

Extracts the chain lists into two shared macros under dbt_macros/shared/:

• dex_evm_chains() — consumed by dex.trades (union) and dex_evm.trades (expose_spells post_hook)
• gas_evm_chains() — consumed by gas.fees (union of the EVM subset, tron appended once at the top of the model) and gas_evm.fees (expose_spells post_hook)

After this, adding or removing an EVM chain for either sector is a one-line change in the macro instead of a coordinated edit across both models.

Compiled outputs are unchanged for dex_trades, dex_evm_trades, gas_fees, and gas_evm_fees (same chains, same order — gas_fees had a minor linea/lens ordering inconsistency between its inline set chains and expose_spells args; the macro produces the alphabetical order the expose_spells list already used).

Verified with dbt compile --select dex_trades dex_evm_trades in the dex project and dbt compile --select gas_fees gas_evm_fees in hourly_spellbook.

---

quick links for more information:

• README.md
• spellbook docs
• CONTRIBUTING.md

Slack Thread

  

[jeff-dude]

@cursor move each shared macro into respective subproject (dex, hourly) macros directory directly (no need for shared naming standard)

[cursor]

Done — moved the two macros into each sub-project's own macros/ directory:

• dbt_macros/shared/dex_evm_chains.sql → dbt_subprojects/dex/macros/dex_evm_chains.sql
• dbt_macros/shared/gas_evm_chains.sql → dbt_subprojects/hourly_spellbook/macros/gas_evm_chains.sql

Re-compiled dex_trades / dex_evm_trades in the dex project and gas_fees / gas_evm_fees in hourly_spellbook — both resolve the macros from the sub-project macros/ path and produce the same compiled SQL as before (1817-line dex_trades, 1777-line gas_fees).

  

[jeff-dude]

@cursor review

[cursor]

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

^{Reviewed by Cursor Bugbot for commit 33c4fff. Configure here.}