feat(morpho-vault): replace morpho-builder with vault-integration skill - #29
Open
jinmel wants to merge 1 commit into
Open
feat(morpho-vault): replace morpho-builder with vault-integration skill#29jinmel wants to merge 1 commit into
jinmel wants to merge 1 commit into
Conversation
Remove the morpho-builder plugin and introduce morpho-vault, a plugin carrying the vault-integration skill for code-generation agents building Morpho Vault earn features. The skill pairs the technical wiring (GraphQL API fields verified against the live schema, morpho-sdk deposit/withdraw flows, ERC-4626 mechanics) with Morpho's Integrator Blueprint (product vocabulary, Powered by Morpho attribution, disclosures, APY composition, vault transparency). Also add a Promptfoo eval suite (evals/suites/vault-integration) modeled on Uniswap's uniswap-ai framework: three integrator-brief cases graded by deterministic checks plus checklist-scored LLM rubrics. In agentic benchmarks the skill scored 23/23 assertions vs 16/23 for a no-skill baseline, with baselines inventing GraphQL fields (netApyWithoutRewards, whitelisted) and omitting attribution/disclosure entirely.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Part of SDK-266
Summary
Replaces the experimental
morpho-builderplugin withmorpho-vault, a plugin carrying thevault-integrationskill: a full technical + UX guide for code-generation agents building Morpho Vault earn features (deposit apps, savings screens, treasury scripts).The skill fuses two halves that fail together in real integrations:
netApyExcludingRewards,state.allRewards,listed,state.curators,liquidity), morpho-sdk deposit/withdraw flows, ERC-4626 mechanics (redeemfor full exits, the Vault V2max*-returns-0 quirk), token quirks (USDC decimals, USDT allowance reset, DAI permit).Layout: a lean 93-line
SKILL.mdworkflow (discover → display → disclose → transact → review) with six progressive-disclosure references.Evals
Adds
evals/suites/vault-integration, a Promptfoo suite modeled on Uniswap's uniswap-ai eval framework: three realistic integrator briefs (React earn tab, treasury deposit bot, misleading-copy review) graded by deterministiccontainschecks plus checklist-scored LLM rubrics (blueprint compliance ≥ 0.85, technical correctness ≥ 0.8).Agentic benchmark during development (Claude with skill vs. no-skill baseline, one run each):
Baselines invented GraphQL fields (
netApyWithoutRewards, awhitelistedfilter), hardcoded memory-recalled vault addresses, and shipped no attribution or disclosure gate — the exact failure modes the skill's verified references close.Other changes
marketplace.json:morpho-builderentry →morpho-vaultREADME.md: overview + install instructions updatedmorpho-cliskill description now cross-referencesvault-integrationfor application-code workskills/vault-integrationsymlink follows the repo conventionTest plan
marketplace.json,plugin.json,promptfoo.yaml)npx promptfoo evalinevals/suites/vault-integration(needsANTHROPIC_API_KEY; not wired to CI yet)