feat: native wrap/unwrap CLI + Morpho marketId registry + max-repay-by-shares - #27
Merged
Conversation
….deposit/withdraw) The CLI had no surface for the WETH9-shape `deposit()` / `withdraw(uint256)` selectors. Anything that needed the wrapped form (Morpho Blue collateral on Monad, perp DEX collateral on HyperEVM, etc.) required users to hand-craft the calldata or rely on a DEX. Adds a first-class `defi token wrap` / `unwrap` pair that reads the chain's `wrapped_native` from chains.toml so the same command works on every supported chain (HYPE/MNT/MON/BNB/ETH). Changes: - defi-cli/src/commands/token.ts: register `wrap` and `unwrap` subcommands. wrap encodes deposit() with value=amount (payable). unwrap encodes withdraw(uint256) with value=0 (non-payable). Both target `chain.wrapped_native`; rejects with a clear error when the chain doesn't register one. - token.test.ts: 3 new regressions pinning the selector bytes (0xd0e30db0 / 0x2e1a7d4d), the value semantics (deposit carries value, withdraw forces 0), and the per-chain target lookup (different `wrapped_native` on monad / hyperevm / bnb). Live verify on Monad mainnet (round-trip): - wrap 1 MON → 1 WMON tx 0x552ef59061b388a249aa295573fa596ee0aa2497d0cd48a6ce7479cd716ec586 - unwrap 1 WMON → 1 MON tx 0x379f248ba43cc9ce01e7c7ca103f5b6fa2a8b360a27f7ab723e51f52f6a3ecff This unblocks the Monad Morpho borrow lifecycle (US-B5) which needs WMON as collateral. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Pre-PR users had to query the Morpho GraphQL API every time they wanted to
hit a direct Morpho Blue market — `--market` only accepted a 32-byte hex.
Now each Morpho protocol TOML can register a `[[protocol.markets]]` array
with friendly names, and the CLI resolves `--market WMON-AUSD` to the
registered hex automatically.
Schema:
- defi-core/registry/protocol.ts: add MarketInfo {name, id, loan_asset?,
collateral_asset?, lltv?} and `markets?: MarketInfo[]` on ProtocolEntry.
Adapter:
- morpho.ts: read entry.markets in the constructor, build a case-insensitive
name → id map, and expose `resolveMarketIdByName(name)` /
`listNamedMarkets()`. Adapter behaviour for hex marketIds is unchanged.
CLI:
- commands/lending.ts: new `resolveMarketInput(adapter, raw)` helper.
32-byte hex passes through verbatim; otherwise look up via the adapter
and surface a helpful error listing valid choices on miss.
Registry data:
- ts/config/protocols/lending/morpho_blue_monad.toml: add 2 markets
WMON-AUSD (lltv 77%) and TETH-TUSD (lltv 86%).
- ts/config/protocols/lending/felix_morpho.toml: add 2 HyperEVM markets
WHYPE-USDT0 (lltv 77%) and wstHYPE-USDT0 (lltv 62%).
- All ids verified via blue-api.morpho.org on 2026-05-07.
Tests:
- morpho.test.ts: 4 new regressions for the named-market path —
case-insensitive match, unknown-name returns null, listNamedMarkets,
empty registry preserves backward compatibility. defi-protocols suite
goes 55 → 59 tests.
Verified: the Monad WMON-AUSD lookup resolves the right marketId and the
on-chain `idToMarketParams` returns the canonical AUSD/WMON tuple, with
calldata encoded against the right Morpho contract — see the dry-run
preview captured during US-B4 verification.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…ad lifecycle Morpho Blue's repay() reverts when `assets > borrowed_assets` because toSharesUp underflows the user's borrowShares. Cleanly closing a position therefore requires passing `shares = position.borrowShares, assets = 0`, not an asset amount. Pre-PR the adapter only emitted the assets path, so `defi lending repay --amount max` would always revert and leave dust. Change: - morpho.ts buildRepay: when params.amount === maxUint256, query Morpho.position(id, user) for the live borrowShares and encode repay with (shares=borrowShares, assets=0). Approve max-uint256 since the contract pulls toAssetsUp(shares) which is unknown at calldata-build time. Behaviour for explicit amounts is unchanged. Live US-B5 verification on Monad mainnet — full WMON/AUSD borrow lifecycle: 1. wrap 1 MON → 1 WMON 0xa7915193ef7e10139d61a57d7936c2c2adf09f817d061e32626f9ab697c93481 2. supply-collat 1 WMON (named market WMON-AUSD) 0x825c66814572f4e85dd56d45f86c73adb835b074acc4825dbebafb780ccd75bd 3. borrow 0.02 AUSD 0x02ecac84c92ca53e0f13c1c7aa6974b39139c553ad74c0a972504f7cbfab107b 4. repay 0.02 AUSD (assets path) 0x73aecc33eb01d2d2cab9729334382f55fd25426790be2fe9fa771ebac8ad1075 5. withdraw-collat 0.2 WMON 0x9b655b7bfe285d23d8f9388409d54597b0e6037c6c496300fe167cf117cb7650 6. unwrap 0.2 WMON → 0.2 MON 0xba54e042af1988df44e53d03201e17efee1ed17b6fe8a6bbe1aa7f2639c834dc Documented limitation: - Cleanly clearing a borrow via `--amount max` requires +1 wei of the loan asset beyond the original borrow amount, because Morpho's toAssetsUp() rounds up by 1 wei. When the user only ever funded the position via the same borrow, the wallet is short by that 1 wei and the share-mode repay still reverts at IERC20.transferFrom. Lifecycle shown above terminates with ~0.798 WMON collateral securing a 0.020101-wei AUSD dust position; clearing the dust requires acquiring ≥1 wei of AUSD externally (DEX swap, transfer in). Out of scope for US-B5 — file as a follow-up if it ever matters in practice. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…hape guard
US-B4 added 32-byte Morpho Blue marketIds to the protocol TOMLs as
`[[protocol.markets]]` entries — first non-address `0x...` literals in
config/. The address-shape regression test that pinned every config hex
literal at exactly 42 chars (0x + 40 hex) was tripping on those. The
guard's intent is to catch malformed addresses (37/39-char gauges,
truncated rewards_distributor); 64-char marketIds aren't malformed,
they're a different on-chain shape (bytes32 vs address).
Change: extend the allowed-length set from {42} to {42, 66}. Rename test
description to reflect the broader scope. Add a sanity case that pins
the 66-char marketId path so the rule can't silently regress to
addresses-only.
Test count: defi-cli 105 → 106 (+1 sanity). Suite stays green.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…ap-cli # Conflicts: # ts/packages/defi-cli/dist/index.js # ts/packages/defi-cli/dist/index.js.map # ts/packages/defi-cli/dist/main.js # ts/packages/defi-cli/dist/main.js.map # ts/packages/defi-cli/dist/mcp-server.js # ts/packages/defi-cli/dist/mcp-server.js.map # ts/packages/defi-core/dist/index.d.ts # ts/packages/defi-core/dist/index.js.map # ts/packages/defi-protocols/dist/index.d.ts # ts/packages/defi-protocols/dist/index.js # ts/packages/defi-protocols/dist/index.js.map
8 tasks
Hiksang
added a commit
that referenced
this pull request
May 8, 2026
…ion + adapter fixes (#32) * build: regenerate dist + sync prebuild package configs for v1.0.13 Re-runs `pnpm -r build` from current main HEAD and syncs the package- local config mirror so npm-published artifacts match the source tree. Reason: across PRs #23 / #26 / #27 we resolved dist conflict markers by taking main's stale dist (`git checkout --theirs`) — that kept the PRs mergeable but left committed dist files out of sync with the post-merge source. This single rebuild commit reconciles them. Affected: defi-cli/dist/{index,main,mcp-server}.js (~+700 lines source-derived) defi-cli/dist/*.map defi-protocols/dist/{index.js, index.d.ts, *.map} defi-core/dist/{index.d.ts, *.map} defi-cli/config/protocols/lending/venus_flux_bnb.toml (prebuild sync from workspace root: vusdt → vlisusd rename) Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * Release v1.0.13: bridge feature + production-grade verification + adapter fixes Highlights since v1.0.12 (30 commits across 8 PRs): == Bridge feature complete == - Relay added as 4th provider (~3s native bridge, executor-wired live) - LiFi + deBridge + CCTP wired to Executor — `--broadcast` no longer silently ignored - CCTP V2 `--auto-receive` — burn → poll Iris attestation → auto-receiveMessage on dest - Live-verified Base→Arbitrum 0.1 USDC E2E (burn 0x6b5d41fd, receive 0x9b5bd5c2) - Per-chain src/dst token resolution for cross-chain stables - Relay errorCode parsing → actionable hints (AMOUNT_TOO_LOW, NO_QUOTES, etc) - MCP `defi_bridge` 3 latent fixes: dst-token, Relay symbol→native, LiFi fromAddress == Adapter bug fixes == - compound_v2: uint256.max withdraw + outstanding-borrow guard (Venus, Compound V2 forks) - venus-flux-bnb: rename mislabeled `vusdt` → `vlisusd` (actual underlying is lisUSD) - Ramses CL mint encoding + MerchantMoe/TraderJoe LB token-order realign - Curve target the pool not router + StableswapNG dynamic uint256[] ABI - uniswap-v2 / solidly LP-token approval on remove - Hybra V4 --redeem-type CLI flag for instant-exit vs 2-year veHYBR lock - Aave V3 collateral toggle + eMode for borrow lifecycle (#21) - Morpho marketId-based supply/borrow + supplyCollateral (#22) - Compound V2 enterMarkets toggle for Venus borrow lifecycle (#26) - Native wrap/unwrap CLI + Morpho marketId registry + max-repay-by-shares (#27) == Production-grade verification sweep == - HyperEVM 11/11 protocols full lifecycle (emission tokens received) - Mantle: Aave V3 + UniV3 + MerchantMoe LB (MOE 1.009 received) - Base: UniV3 + Aerodrome V2 + Aerodrome CL (AERO emission received) - BNB upgraded to 🟢 production: 13/16 protocols verified live - Monad partial: UniV3 + UniV2 (mainnet TBD for the rest) - ve(3,3) emission tokens live-received: RAM, KITTEN, MOE, AERO, THE - ULTRAQA sandbox script (51-test sweep across 4 chains) == Hardening == - ip-address >=10.1.1 via pnpm.overrides (GHSA-v2v4-37r5-5v8g moderate XSS) - viem wallet/public clients anchored to chainId at construction (#29) - CLI handler unit-test coverage for status/schema (#28) - @vitest/coverage-v8 baseline measurement script (#30) - post-baseline test-foundation QA report (#31) == Test plan == - pnpm -r test: 189/189 pass (defi-core 32, defi-protocols 55, defi-cli 102) - pnpm -r build: clean across all 3 packages - pnpm audit: 0 vulnerabilities - ULTRAQA sandbox: 51/51 pass + 8/8 ve(3,3) lp pipeline - CCTP --auto-receive E2E: live broadcast verified - MCP defi_bridge smoke: LiFi USDC + Relay USDC + Relay native — all OK Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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
1 participant
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.
Summary
Three Morpho/native-wrap improvements stacked on this branch:
1. `defi token wrap` / `unwrap` (commit d28702f)
First-class CLI surface for the WETH9-shape `deposit()` / `withdraw(uint256)` selectors. Reads each chain's `wrapped_native` from chains.toml so the same command works on every supported chain.
2. Per-chain Morpho marketIds TOML registry (commit eb56803)
Pre-PR users had to query the Morpho GraphQL API every time. Now each Morpho protocol TOML can register a `[[protocol.markets]]` array with friendly names.
3. Max-repay-by-shares for clean Morpho position close (commit 2fa7052)
Morpho `repay()` reverts when `assets > borrowed_assets`. Cleanly closing a position requires passing `shares = position.borrowShares, assets = 0`.
4. Address-shape test allows 32-byte bytes32 (commit 19ae2b4)
US-B4 added 64-char marketId hex to TOMLs; the address-shape regression test that pinned every `0x...` literal at 42 chars was tripping. Extended allowed-length set from {42} to {42, 66}.
Live txs
Monad WMON/AUSD lifecycle:
Test plan
Documented limitation
Morpho `repay --amount max` requires +1 wei of the loan asset beyond the original borrow because `toAssetsUp()` rounds up. When the user only ever funded the position via the same borrow, the wallet is short by that 1 wei. Out of scope for this PR — file as a follow-up.
🤖 Generated with Claude Code