Skip to content

Zircuit finance adaptor#2431

Merged
0xkr3p merged 2 commits intoDefiLlama:masterfrom
kbak:zircuit-finance
Feb 27, 2026
Merged

Zircuit finance adaptor#2431
0xkr3p merged 2 commits intoDefiLlama:masterfrom
kbak:zircuit-finance

Conversation

@kbak
Copy link
Contributor

@kbak kbak commented Feb 27, 2026

https://finance.zircuit.com

Related PR
TVL adapter: DefiLlama/DefiLlama-Adapters#18154

Summary by CodeRabbit

  • New Features
    • Added Zircuit Finance integration that computes APY and TVL for Z circuit vaults on the Base chain, including real-time on-chain data collection for vault metrics and underlying asset price tracking.

@coderabbitai
Copy link

coderabbitai bot commented Feb 27, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ca62718 and 8a2fec7.

📒 Files selected for processing (1)
  • src/adaptors/zircuit-finance/index.js

📝 Walkthrough

Walkthrough

A new adaptor is introduced for Zircuit Finance that calculates APY and TVL metrics for two vaults on the Base blockchain. The adaptor fetches vault data via multicalls, retrieves underlying token prices, and computes TVL in USD and APY based on vault rates across time periods.

Changes

Cohort / File(s) Summary
Zircuit Finance Adaptor
src/adaptors/zircuit-finance/index.js
New adaptor that retrieves on-chain vault data via multicalls (symbol, decimals, totalAssets, convertToAssets), fetches underlying token prices, computes TVL per vault and APY base from rate changes, and exports configuration with pool details, disabled timetravel, and project URL.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 A new vault upon the Base does gleam,
Zircuit's treasury fulfills our dream!
Multicalls dance, prices sing so bright,
APY and TVL now in sight! 💫

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Zircuit finance adaptor' directly describes the main change: adding a new adaptor for Zircuit Finance to compute APY and TVL metrics.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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.

@0xkr3p
Copy link
Contributor

0xkr3p commented Feb 27, 2026

hey @kbak, thanks for the pr!

@0xkr3p 0xkr3p merged commit df3a226 into DefiLlama:master Feb 27, 2026
2 checks passed
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.

2 participants