Skip to content

chore: update treasury addresses#12266

Merged
kaladinlight merged 3 commits into
developfrom
update-treasury-addresses
Apr 9, 2026
Merged

chore: update treasury addresses#12266
kaladinlight merged 3 commits into
developfrom
update-treasury-addresses

Conversation

@kaladinlight
Copy link
Copy Markdown
Member

@kaladinlight kaladinlight commented Apr 9, 2026

Description

Update DAO treasury addresses across multiple chains:

  • Update Solana, Starknet, TON, Monad, and HyperEVM treasury addresses
  • Fix Avnu swapper to use starknetChainId instead of sellAsset.chainId for fee recipient lookup, and add integratorName
  • Fix Stonfi swapper to use tonChainId instead of sellAsset.chainId for treasury address lookup

Issue (if applicable)

Risk

Low — treasury address updates with targeted swapper fixes to use correct chain IDs for fee recipient lookups.

Affiliate fee routing for Avnu (Starknet) and Stonfi (TON) swappers.

Testing

Engineering

  • Verify Avnu quotes include the updated Starknet treasury address as fee recipient
  • Verify Stonfi quotes include the updated TON treasury address as referrer
  • Confirm no other swappers are affected

Operations

  • 🏁 My feature is behind a flag and doesn't require operations testing (yet)

Screenshots (if applicable)

Summary by CodeRabbit

  • Chores
    • Updated treasury address configurations across multiple blockchain networks (Solana, Starknet, TON, Monad, and HyperEVM) to ensure proper fee handling in swap operations.
    • Enhanced swap integrations with improved treasury address derivation logic.

@kaladinlight kaladinlight requested a review from a team as a code owner April 9, 2026 19:53
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 9, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f5ef0c43-d1e2-45ed-b3f0-48da04232c7f

📥 Commits

Reviewing files that changed from the base of the PR and between df28ed4 and 0e7617b.

📒 Files selected for processing (3)
  • packages/swapper/src/swappers/AvnuSwapper/swapperApi/getTradeQuote.ts
  • packages/swapper/src/swappers/StonfiSwapper/swapperApi/getTradeQuote.ts
  • packages/utils/src/treasury.ts

📝 Walkthrough

Walkthrough

The pull request updates treasury address constants across multiple blockchain networks and modifies two swapper implementations (Avnu and Stonfi) to use chain-specific treasury address identifiers instead of asset-based chain IDs when computing integrator fees and referrer addresses.

Changes

Cohort / File(s) Summary
Swapper Treasury Address Logic
packages/swapper/src/swappers/AvnuSwapper/swapperApi/getTradeQuote.ts, packages/swapper/src/swappers/StonfiSwapper/swapperApi/getTradeQuote.ts
Modified treasury address derivation in quote requests to use hardcoded chain IDs (starknetChainId, tonChainId) instead of sellAsset.chainId. Avnu swapper also adds integratorName: 'shapeshift' to request payload.
Treasury Constants Update
packages/utils/src/treasury.ts
Updated exported treasury address constants for Solana, Starknet, Ton, Monad, and Hyperevm networks to new on-chain addresses. Monad and Hyperevm now share a common address.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Poem

🐰 A rabbit hops through chains so grand,
Treasury addresses, updated and planned,
Starknet, Ton, and Solana too,
New addresses shiny, fresh and new!
With integrator names now in sight,
The swappers' treasury logic is right! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'chore: update treasury addresses' accurately and concisely summarizes the main change: updating treasury address constants across multiple chains and fixing swapper implementations to use correct chain IDs.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch update-treasury-addresses

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@kaladinlight kaladinlight enabled auto-merge (squash) April 9, 2026 19:58
@kaladinlight kaladinlight merged commit e33e742 into develop Apr 9, 2026
4 checks passed
@kaladinlight kaladinlight deleted the update-treasury-addresses branch April 9, 2026 20:05
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