[zancas]

This PR fixes the five review findings on #1910. It targets the branch of #1910.

Changes

• app/wallet holds one module for each domain: address, balance, pool, send, transfer and keys. The types, enums and classes modules are deleted.
• app/wallet imports neither app/AppState nor app/walletBackend. Both of them import app/wallet.
• The @app/AppState barrel exports its own names only. Each wallet name has one import path, @app/wallet/<domain>.
• Each file imports a wallet module one time.
• The wallet enums are literal unions: ValueTransferKind, ValueTransferStatus, Pool, AddressKind, AddressScope. The string values are unchanged.
• WalletErrorInfo is a discriminated union. The 'chain' kind carries walletChain.
• WalletErrorInfo and DeleteWalletContext return to app/AppState/types. FilterKind moves to screens/History/components/Filters.tsx.
• The restore branch in LoadingApp calls the existing isViewingKey.
• TransactionType, TxDetailType, TransactionTypeEnum and AddressUnifiedTypeEnum had zero users and are deleted.

Tests

scripts/module_rules.ts holds five pure functions. Each function returns a discriminated union.

• __tests__/moduleRules.unit.ts tests the functions on inline sources.
• __tests__/walletModule.unit.ts applies the functions to the repository. Its five tests failed on the head of refactor(wallet): consolidate wallet enums and types into domain-grouped files #1910, one test for each finding.

tsc, eslint, prettier --check and all 285 Jest suites pass locally.

Out of scope

The five classes in app/wallet remain classes. The change of each class to a type with functions is a separate task.

🤖 Generated with Claude Code

[github-actions]

Visual review — ✅ matches baseline

Head vs visual/__baseline__. Open the review page — Current, Animations, and Image diffs tabs.