Add USDV supply adapters for Tron, Base, Polygon and BNB Chain#869
Add USDV supply adapters for Tron, Base, Polygon and BNB Chain#869valtorum wants to merge 3 commits into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughUSDV support now includes additional network contract mappings, shared peg typing for minted balance aggregation, revised Stellar supply calculation, simplified chain handlers, and metadata describing issuance and redemption across supported networks. ChangesUSDV multi-network support
Estimated code review effort: 3 (Moderate) | ~20 minutes Sequence Diagram(s)sequenceDiagram
participant Adapter
participant XRPLGatewayBalances
participant StellarAPI
participant BalanceAggregator
Adapter->>XRPLGatewayBalances: Request gateway_balances
XRPLGatewayBalances-->>Adapter: Return obligation balances
Adapter->>StellarAPI: Read balance record
StellarAPI-->>Adapter: Return balance fields
Adapter->>BalanceAggregator: Sum supply with pegType
Possibly related PRs
🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (3 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
|
hey @valtorum are all of these tokens circulating? for example on base 89% of the supply is in this wallet: |
Thanks for flagging this. The Base address you linked is an issuer-controlled/non-circulating balance and should be excluded from the reported USDV supply. I will update the adapter accordingly and review the other supported networks so the same treatment is applied consistently where issuer-controlled balances exist. |
|
Updated the adapter to exclude confirmed issuer-controlled balances from reported circulating USDV supply. The Base balance flagged above is now marked as unreleased, and I also added confirmed controlled balances for Polygon, BNB Chain, and TRON where applicable. |
Updates the existing Valtorum USD (USDV) listing.
Changes included:
The adapter uses DefiLlama's existing
addChainExportshelper for EVM/TRON supply reads and keeps the custom XRP Ledger/Stellar logic already used for those networks.Summary by CodeRabbit
New Features
Documentation