Skip to content

Enable Euler vaults for Linea in ZeroLend Vaults#33

Open
zknpr wants to merge 1 commit intomainfrom
zerolend-vaults-linea-8051626484937692238
Open

Enable Euler vaults for Linea in ZeroLend Vaults#33
zknpr wants to merge 1 commit intomainfrom
zerolend-vaults-linea-8051626484937692238

Conversation

@zknpr
Copy link
Copy Markdown
Owner

@zknpr zknpr commented Feb 15, 2026

Enable Euler vaults for Linea in ZeroLend Vaults by using the correct Linea curator address and enabling dynamic fetching.

  • Added lineaVaultOwners with the correct Linea curator address (0x0F6...).
  • Updated linea configuration to use eulerVaultOwners instead of a hardcoded list.
  • Removed the hardcoded list of vaults for Linea as they are now dynamically fetched.
  • Verified that the Linea TVL is non-zero (11.77k) using test.js.

PR created automatically by Jules for task 8051626484937692238 started by @zknpr

Summary by CodeRabbit

  • Chores
    • Updated Linea vault ownership configuration to use a consolidated constant for managing vault owner addresses.

Co-authored-by: zknpr <96851588+zknpr@users.noreply.github.com>
@google-labs-jules
Copy link
Copy Markdown

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.


For security, I will only act on instructions from the user who triggered this task.

@gemini-code-assist
Copy link
Copy Markdown

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 15, 2026

📝 Walkthrough

Walkthrough

The changes introduce a new constant lineaVaultOwners and refactor the Linea chain configuration to reference this constant for eulerVaultOwners instead of using hard-coded addresses, while setting the euler field to an empty array.

Changes

Cohort / File(s) Summary
Configuration Update
projects/zerolend-vaults/index.js
Introduces lineaVaultOwners constant and updates linea config to use it for eulerVaultOwners while clearing the euler field.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A vault configuration, neat and true,
New owners declared, fresh as morning dew,
Constants refactored with a gentle hop,
One line replaced, the changes won't stop! ✨

🚥 Pre-merge checks | ✅ 3 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The description is incomplete; it does not follow the required template and lacks many mandatory fields such as protocol name, website, chain, and other listing information. Fill out the required template sections including protocol name, chain, website link, and other mandatory fields. If this is an update rather than a new listing, clarify the PR purpose and scope.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: enabling Euler vaults for Linea in ZeroLend Vaults, matching the primary objective of the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch zerolend-vaults-linea-8051626484937692238

No actionable comments were generated in the recent review. 🎉

📜 Recent review details

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2d20682 and a7af4f1.

📒 Files selected for processing (1)
  • projects/zerolend-vaults/index.js
🔇 Additional comments (2)
projects/zerolend-vaults/index.js (2)

23-26: LGTM - Dynamic vault fetching enabled for Linea.

The configuration correctly switches Linea to use eulerVaultOwners for dynamic vault discovery with an empty euler array. This is a cleaner approach than hardcoding vault addresses and allows automatic discovery of new vaults.

Note: This pattern differs from other chains (ethereum, berachain, sonic) which have both eulerVaultOwners AND explicit euler addresses. If dynamic fetching works well for Linea, consider migrating other chains to this approach for consistency.


9-11: Verify the Linea curator address is correct.

The new lineaVaultOwners array uses address 0x0F6e98A756A40dD050dC78959f45559F98d3289d, which differs from the multisigs array used by other chains. Please confirm this is the correct Linea Euler curator address.

✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.


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

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