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:
|
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.