feat(wallets): add support for wallet_addEthereumChain and wallet_switchEthereumChain

[gregfromstl]

PR-Codex overview

This PR enhances the WalletConnect integration by adding support for validating account addresses, switching Ethereum chains, and handling chain-related requests.

Detailed summary

• Added support for validating account addresses with checksum
• Implemented handling of switching Ethereum chains
• Updated request handlers for adding and switching Ethereum chains
• Added tests for wallet_addEthereumChain and wallet_switchEthereumChain requests

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
thirdweb_playground	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 19, 2024 2:46am

[changeset-bot]

⚠️ No Changeset found

Latest commit: d931436

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[graphite-app]

Your org requires the Graphite merge queue for merging into main

Add the label “merge-queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

[gregfromstl]

• feat(connect-wallet): support custom chains in WalletConnect #3372
• feat(wallets): add support for wallet_addEthereumChain and wallet_switchEthereumChain #3371 👈
• fix(wallets): cache walletConnectClient #3310
• refactor: update icon usage in ConnectWallet component #3309
• feat(react): Adds WalletConnect Receiver #3294
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @gregfromstl and the rest of your teammates on Graphite

[codspeed-hq]

CodSpeed Performance Report

Merging #3371 will not alter performance

_{Comparing feat/walletconnect-add-switch-chains (d931436) with 06-13-fix_wallets_cache_walletconnectclient (9266254)}

Summary

✅ 9 untouched benchmarks

[github-actions]

size-limit report 📦

Path	Size	Loading time (3g)	Running time (snapdragon)	Total time
thirdweb (esm)	40.65 KB (0%)	814 ms (0%)	5.5 s (+150.1% 🔺)	6.3 s
thirdweb (cjs)	91.92 KB (0%)	1.9 s (0%)	6.4 s (-12.41% 🔽)	8.3 s
thirdweb (minimal + tree-shaking)	4.74 KB (0%)	95 ms (0%)	184 ms (+32.49% 🔺)	279 ms
thirdweb/chains (tree-shaking)	423 B (0%)	10 ms (0%)	321 ms (+132.17% 🔺)	331 ms
thirdweb/react (minimal + tree-shaking)	13.46 KB (0%)	270 ms (0%)	405 ms (-46.57% 🔽)	674 ms

[codecov]

Codecov Report

Attention: Patch coverage is 76.19048% with 15 lines in your changes missing coverage. Please review.

Project coverage is 62.52%. Comparing base (9266254) to head (d931436).

Additional details and impacted files

@@                               Coverage Diff                               @@
##           06-13-fix_wallets_cache_walletconnectclient    #3371      +/-   ##
===============================================================================
- Coverage                                        62.55%   62.52%   -0.03%     
===============================================================================
  Files                                              870      872       +2     
  Lines                                            64595    64728     +133     
  Branches                                          3479     3487       +8     
===============================================================================
+ Hits                                             40405    40471      +66     
- Misses                                           23510    23577      +67     
  Partials                                           680      680              

Flag	Coverage Δ		*Carryforward flag
legacy_packages	65.60% <ø> (ø)		Carriedforward from 9266254
packages	61.89% <76.19%> (-0.04%)	⬇️

*This pull request uses carry forward flags. Click here to find out more.

Files	Coverage Δ
...s/wallet-connect/receiver/request-handlers/sign.ts	100.00% <100.00%> (ø)
...-connect/receiver/request-handlers/switch-chain.ts	100.00% <100.00%> (ø)
...rdweb/src/wallets/wallet-connect/receiver/utils.ts	96.87% <100.00%> (ø)
...wallets/wallet-connect/receiver/session-request.ts	91.22% <61.53%> (-6.15%)	⬇️

... and 4 files with indirect coverage changes

[graphite-app]

Merge activity

• Jun 18, 8:59 PM EDT: gregfromstl added this pull request to the Graphite merge queue.
• Jun 18, 9:01 PM EDT: The Graphite merge queue removed this pull request due to downstack failures on PR #3309.
• Jun 18, 9:01 PM EDT: The Graphite merge queue removed this pull request due to downstack failures on PR #3309.
• Jun 18, 10:38 PM EDT: gregfromstl added this pull request to the Graphite merge queue.
• Jun 18, 10:46 PM EDT: gregfromstl merged this pull request with the Graphite merge queue.