Skip to content

Humanized wallet - #240

Open
3esmit wants to merge 2 commits into
mainfrom
humanized-wallet
Open

Humanized wallet#240
3esmit wants to merge 2 commits into
mainfrom
humanized-wallet

Conversation

@3esmit

@3esmit 3esmit commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

Summary

Humanizes the shared wallet experience in the AMM UI by presenting wallet
state, token holdings, and decoded program accounts as useful account data
instead of raw account records. Generic wallet infrastructure now lives in the
shared wallet module rather than AMM-specific sources.

Changes

  • extend the shared wallet provider, controller, account model, and QML
    controls for account creation, status, transaction feedback, copying, and
    account presentation
  • move generic sequencer settings, identity probing, IDL decoding, token
    definition caching, and portfolio processing into apps/shared/wallet
  • keep AMM responsible only for AMM deployment data and token configuration;
    reuse shared network identity and portfolio services
  • decode Token and AMM account data into token definitions, aggregated
    holdings, and program-account fields, using committed IDLs from artifacts/
  • clear stale presentation state and avoid zero balances when a public account
    read or token decode fails; reject stale probe and refresh responses
  • add focused C++ and QML coverage for network state, settings, retry/error
    handling, portfolio caching, stale state, and wallet presentation

Validation

  • git diff --check
  • focused shared-wallet C++ and QML tests
  • nix build .#amm-ui --no-link
  • cargo +nightly fmt --all -- --check
  • taplo fmt --check .
  • RISC0_SKIP_BUILD=1 cargo +1.94.0 clippy --workspace --all-targets -- -D warnings
  • RISC0_DEV_MODE=1 cargo +1.94.0 test --workspace --exclude integration_tests
  • RISC0_DEV_MODE=1 cargo +1.94.0 test -p integration_tests
  • IDL drift check

Copilot AI review requested due to automatic review settings July 29, 2026 14:14

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@3esmit

3esmit commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator Author

E2E verification

RISC0_DEV_MODE=1 cargo +1.94.0 test -p integration_tests --test amm39 passed.

Read-only testnet validation used the matching deployed AMM program. No transaction was submitted.

Trade and liquidity

Trade

Token selector

New liquidity position

Wallet

Wallet overview

All-assets wallet

The all-assets view below uses the fixture's initialized testnet token records.

All-assets token holdings

All-assets token holdings, continued

Accounts

Advanced decoded records

Create account

Rename account

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.

2 participants