Conversation
📝 WalkthroughWalkthroughThe adaptor moves from per-chain nested pool definitions to a flat, address-keyed Changes
Sequence Diagram(s)mermaid Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
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. Comment |
|
The symbiosis adapter exports pools: Test Suites: 1 passed, 1 total |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@src/adaptors/symbiosis/index.js`:
- Around line 20-43: The TVL computed in loadTvlData is currently
token-denominated because tvlUsd is derived only from raw token balances
(variable tvlUsd inside loadTvlData) and not multiplied by each token's USD
price; fix by fetching USD prices for the tokens in pools (prefer a batch price
lookup before the Promise.all, e.g., gather pool.token addresses and call the
existing price oracle/price-fetcher) and multiply the token amount
(balance.div(10**pool.decimals)) by the fetched USD price before assigning
tvlData[pool.sToken]; ensure the price lookup handles chain differences
(pool.chain / pool.isTron) and falls back or logs if price unavailable.
|
The symbiosis adapter exports pools: Test Suites: 1 passed, 1 total |
Summary by CodeRabbit
New Features
Improvements
Refactor
✏️ Tip: You can customize this high-level summary in your review settings.