Skip to content

Fix/deposit ux improvements#92

Merged
JamesLawton merged 4 commits intomainfrom
fix/deposit-ux-improvements
Apr 17, 2026
Merged

Fix/deposit ux improvements#92
JamesLawton merged 4 commits intomainfrom
fix/deposit-ux-improvements

Conversation

@JamesLawton
Copy link
Copy Markdown
Collaborator

No description provided.

…messages

Deposits send two txs: approve() on the token contract + the pool call.
Both need session whitelisting. This caused repeated re-authorization loops
when the token contract (e.g. USDC) wasn't whitelisted alongside the pool.

- Add USDC/USDT/WETH token contracts to AUTO_WHITELISTED_CONTRACTS in
  both wallet.ts (--print-url flow) and wallet-ui.tsx (interactive flow)
- Sync wallet-ui.tsx AUTO_WHITELISTED_CONTRACTS with wallet.ts (was missing
  all DeFi vault contracts — only had the 3 core contracts)
- Fix dry-run note and session rejection error in deposit command to name
  both --contract flags needed (token + pool)
- Update SKILL.md and polygon-defi/SKILL.md to document the two-contract
  requirement

Entire-Checkpoint: ada5950cb14d
Both the token contract (approve) and pool contract must be whitelisted
when creating a wallet for deposits. Update SKILL.md and polygon-defi/SKILL.md
to document the two-contract requirement and list common Polygon token addresses.

Entire-Checkpoint: 8754e23b78a8
…Y are the same key

Agents were hitting Trails "permission wall" errors not knowing that
SEQUENCE_PROJECT_ACCESS_KEY (auto-loaded from disk) is the same value
as TRAILS_API_KEY. Make the one-key/many-names relationship explicit.

Entire-Checkpoint: 4fd4e4cce22b
@JamesLawton JamesLawton merged commit 97b8c2d into main Apr 17, 2026
3 checks passed
@JamesLawton JamesLawton deleted the fix/deposit-ux-improvements branch April 17, 2026 10:03
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