Skip to content

Money fi#18591

Open
TuanTwendee wants to merge 11 commits intoDefiLlama:mainfrom
TuanTwendee:MoneyFi
Open

Money fi#18591
TuanTwendee wants to merge 11 commits intoDefiLlama:mainfrom
TuanTwendee:MoneyFi

Conversation

@TuanTwendee
Copy link
Copy Markdown
Contributor

@TuanTwendee TuanTwendee commented Mar 31, 2026

NOTE

Please enable "Allow edits by maintainers" while putting up the PR.


  1. If you would like to add a volume/fees/revenue adapter please submit the PR here.

  2. Once your adapter has been merged, it takes time to show on the UI. If more than 24 hours have passed, please let us know in Discord.

  3. Sorry, We no longer accept fetch adapter for new projects, we prefer the tvl to computed from blockchain data, if you have trouble with creating a the adapter, please hop onto our discord, we are happy to assist you.

  4. For updating listing info It is a different repo, you can find your listing in this file, you can edit it there and put up a PR

  5. Please do not add new npm dependencies, do not edit/push pnpm-lock.yaml file as part of your changes


(Needs to be filled only for new listings)

Name (to be shown on DefiLlama):
Twitter Link:
List of audit links if any:
Website Link:
Logo (High resolution, will be shown with rounded borders):
Current TVL:
Treasury Addresses (if the protocol has treasury)
Chain:
Coingecko ID (so your TVL can appear on Coingecko, leave empty if not listed): (https://api.coingecko.com/api/v3/coins/list)
Coinmarketcap ID (so your TVL can appear on Coinmarketcap, leave empty if not listed): (https://api.coinmarketcap.com/data-api/v3/map/all?listing_status=active,inactive,untracked&start=1&limit=10000)
Short Description (to be shown on DefiLlama):
Token address and ticker if any:
Category (full list at https://defillama.com/categories) *Please choose only one:
Oracle Provider(s): Specify the oracle(s) used (e.g., Chainlink, Band, API3, TWAP, etc.):
Implementation Details: Briefly describe how the oracle is integrated into your project:
Documentation/Proof: Provide links to documentation or any other resources that verify the oracle's usage:
forkedFrom (Does your project originate from another project):
methodology (what is being counted as tvl, how is tvl being calculated):
Github org/user (Optional, if your code is open source, we can track activity):
Does this project have a referral program?

Summary by CodeRabbit

  • Chores
    • Several blockchain networks (Ethereum, BSC, Arbitrum, Optimism, Core, Soneium) have been disabled in the platform configuration. Only the Base network is now active and supported. Users will only be able to interact with the Base blockchain at this time.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 31, 2026

📝 Walkthrough

Walkthrough

Multiple blockchain chain configurations (ethereum, bsc, arbitrum, optimism, core, soneium) were commented out in the MoneyFi index configuration file, disabling their fundVault, dexBridgeVault, strategies, and stablecoins definitions while keeping the base chain active for runtime use.

Changes

Cohort / File(s) Summary
Chain Configuration Disabling
projects/moneyfi/index.js
Commented out configuration blocks for six chains (ethereum, bsc, arbitrum, optimism, core, soneium), including their vault definitions, strategy configurations, and stablecoin arrays. Base chain configuration remains active.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Hop, skip, and commented code,
Six chains now rest on the road,
While base stands tall and bright,
We've tidied our config just right! 🌟

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is the unmodified template with no custom information filled in, leaving all required fields for new listings empty despite the PR making functional changes. Add a detailed description of the changes made (deactivating chains, adding Aptos support, fixing addresses) and fill in relevant template fields or note this is not a new listing.
Title check ❓ Inconclusive The title 'Money fi' is vague and generic, referring only to the project name without describing the specific changes made in this pull request. Replace with a descriptive title summarizing the main change, such as 'Deactivate network chains in MoneyFi adapter' or similar.
✅ Passed checks (1 passed)
Check name Status Explanation
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 unit tests (beta)
  • Create PR with unit tests

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.

@llamabutler
Copy link
Copy Markdown

The adapter at projects/moneyfi exports TVL:

aptos                     632.02 k
base                      19.45 k

total                    651.47 k 

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick comments (1)
projects/moneyfi/index.js (1)

18-98: Add a deactivation hallmark to explain the TVL step-change.

Lines 18–98 disable six chains at once, which will produce an abrupt TVL drop in charts. Add a dated hallmarks entry in module.exports so the change is explicitly documented for users.

Proposed patch
 module.exports = {
   ...chainExports,
   timetravel: true,
+  hallmarks: [
+    ["2026-03-31", "Disabled Ethereum/BSC/Arbitrum/Optimism/Core/Soneium adapters"],
+  ],
   methodology:
     "TVL counts stablecoins in fundVault, dexBridgeVault, and all strategies contracts.",
 };

Based on learnings: In DefiLlama adapters, when protocol activity changes in ways that affect reported TVL, hallmark annotations should be used to document the event date for chart interpretability.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@projects/moneyfi/index.js` around lines 18 - 98, Add a dated hallmark
documenting the mass deactivation to the exported adapter config: update the
module.exports object to include a hallmarks array (or append to existing
hallmarks) containing an entry with the YYYY-MM-DD timestamp and a short message
like "Disabled multiple chains (ethereum, bsc, arbitrum, optimism, core,
soneium) — causes TVL step-change"; ensure this hallmark is alongside the
existing exported symbols (e.g., the base/stablecoins config) so charts
consuming the adapter will show the annotation.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@projects/moneyfi/index.js`:
- Around line 18-98: Add a dated hallmark documenting the mass deactivation to
the exported adapter config: update the module.exports object to include a
hallmarks array (or append to existing hallmarks) containing an entry with the
YYYY-MM-DD timestamp and a short message like "Disabled multiple chains
(ethereum, bsc, arbitrum, optimism, core, soneium) — causes TVL step-change";
ensure this hallmark is alongside the existing exported symbols (e.g., the
base/stablecoins config) so charts consuming the adapter will show the
annotation.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 8026de5c-96ae-4738-84b9-8fe0c4887def

📥 Commits

Reviewing files that changed from the base of the PR and between 42539bc and 4ad9621.

📒 Files selected for processing (1)
  • projects/moneyfi/index.js

@RohanNero RohanNero self-assigned this Apr 1, 2026
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.

3 participants