Conversation
|
Thanks for the retryability fix — the catch path restoring provider = null / disabling the action button is the right shape. One gap across the shared const accounts = (await provider.request({ method: "eth_accounts", ... })) as string[];
return { provider, account: accounts[0] };If walletInfo.textContent = connection.account ?? "Connected";
sendButton.disabled = false; // or bridgeButton.disabled = false on EVM-only examplesSo the UI can show a fake "Connected" state and re-enable the action with no usable account. Pre-PR code had the same Suggested harden (same for Solana when
Happy to re-review once that lands. |
|
Good point. I pushed cc3debd to cover that path as part of this PR. The EVM helpers now use the account list returned by Verified locally with Node 22.12.0:
|
This PR: - addresses the issue raised in circlefin#6 and supersedes circlefin#7 and circlefin#8 - also clears the output panel on successful wallet connect - surfaces the actual `Error` throw message instead of generic `Unknown error` - adds potential troubleshooting guidance for Swap kit in case of bad RPC - switches out Base Sepolia for Avalanche Fuji for the unified balance kit demo and explains the situation of both EVM and Solana wallet prompts required during the spend operation - drops the estimate pre-step for `send()` and `swap()` - bumps dependencies to latest versions Closes circlefin#6
Summary
Fixes #6.
Testing
cd app-kit-send && npm test && npm run buildcd app-kit-swap && npm test && npm run buildcd app-kit-bridge-evm && npm test && npm run buildcd app-kit-bridge-solana && npm test && npm run buildAll 10 tests pass and all four Vite production builds complete successfully on Node.js v24.20.0.