Fix/align lend amount types

[thinktanktom]

Summary

Aligns the lend module's amount and position types with the pattern established by the swap module in #284. The core principle: developers pass human-readable numbers in, and receive human-readable numbers out with the same field names. Raw bigint values are always suffixed with Raw.

Closes #303

---

Problem

The lend module had an inconsistent type pattern where the same field name changed type between input and output:

// Input
amount: number         // human-readable (e.g. 100)

// Output — same name, completely different type
amount: bigint         // raw on-chain value (e.g. 100000000n)
balanceFormatted: string  // awkward Formatted suffix

This was a footgun for developers consuming the SDK: passing amount: 100 in and receiving amount: 100000000n back from the same field.

---

Changes

SDK types (packages/sdk/src/types/lend/base.ts)

Interface	Before	After
LendTransaction	amount: bigint	amount: number + amountRaw: bigint
LendMarketPosition	balance: bigint	balance: number + balanceRaw: bigint
LendMarketPosition	balanceFormatted: string	Removed (replaced by balance: number)
LendMarketPosition	shares: bigint	shares: number + sharesRaw: bigint
LendMarketPosition	sharesFormatted: string	Removed (replaced by shares: number)
LendClosePositionParams	amount: bigint	amountRaw: bigint (consistency with LendOpenPositionInternalParams)

SDK providers

• LendProvider.ts: closePosition passes amountRaw to _closePosition
• MorphoLendProvider: _openPosition, _closePosition, _getPosition updated to return new field shape using formatAssetAmount
• AaveLendProvider: _openETHPosition, _openERC20Position, _closeETHPosition, _closeERC20Position, _getPosition updated
• MockLendProvider: All mock helpers updated to produce correct amount/amountRaw/balanceRaw/sharesRaw fields

Demo backend

• packages/demo/backend/src/types/lend.ts — no change needed; serializeBigInt handles the new *Raw bigint fields automatically

Demo frontend

• packages/demo/frontend/src/types/api.ts — PositionResponse updated
• packages/demo/frontend/src/api/actionsApi.ts — getPosition deserializes balanceRaw/sharesRaw from string, passes balance/shares through as numbers
• packages/demo/frontend/src/hooks/useWalletBalance.ts — depositedAmount derived from position.balance (number) instead of position.balanceFormatted (string)
• packages/demo/frontend/src/hooks/useLendProvider.ts — position balance comparison updated

Tests

• All assertions updated across LendProvider.test.ts, WalletLendNamespace.spec.ts, AaveLendProvider.test.ts, MorphoLendProvider.test.ts, useActivityLogging.spec.ts
• All 375 SDK unit tests passing, all frontend tests passing

---

Before / After

// BEFORE
const position = await wallet.lend.getPosition({ marketId })
console.log(position.balance)         // 100000000n  ← bigint, confusing
console.log(position.balanceFormatted) // "100.00"    ← string, extra field

const tx = await wallet.lend.openPosition({ amount: 100, ... })
console.log(tx.amount) // 100000000n  ← bigint, doesn't match what you passed in

// AFTER
const position = await wallet.lend.getPosition({ marketId })
console.log(position.balance)    // 100      ← number, matches input type
console.log(position.balanceRaw) // 100000000n ← raw value, clearly named

const tx = await wallet.lend.openPosition({ amount: 100, ... })
console.log(tx.amount)    // 100        ← number, matches what you passed in
console.log(tx.amountRaw) // 100000000n ← raw value, clearly named

---

Notes

• No changes to the public openPosition/closePosition call signatures — amount: number input is unchanged
• formatAssetAmount from packages/sdk/src/utils/assets.ts is used for all bigint → number conversions (already existed in the codebase)
• This is a breaking change for any consumer currently reading amount, balance, shares, balanceFormatted, or sharesFormatted off lend types — a changeset should be added before merge

[netlify]

👷 Deploy request for actions-ui pending review.

Visit the deploys page to approve it

Name	Link
🔨 Latest commit	98d2926