[SDK] Fix: Pay not respecting ecosystem AA

[gregfromstl]

PR-Codex overview

This PR focuses on improving the detection of ecosystem wallets in the Pay modal by modifying the logic in the hasSponsoredTransactionsEnabled function and adding comprehensive tests for various wallet configurations.

Detailed summary

• Updated the condition in hasSponsoredTransactionsEnabled to check for ecosystem wallets using isEcosystemWallet.
• Added tests for hasSponsoredTransactionsEnabled covering:
  • Undefined wallets
  • Smart wallets with sponsorGas and gasless configurations
  • In-app wallets with smartAccount configurations
  • Regular wallets without smart account configurations

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

[linear]

CNCT-2629 Fix TransactionButton issue with gas sponsorship in ecosystem wallet

[changeset-bot]

🦋 Changeset detected

Latest commit: 5b24faf

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
thirdweb	Patch

Not sure what this means? Click here to learn what changesets are.

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

[graphite-app]

Your org has enabled 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.

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
docs-v2	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Dec 11, 2024 0:07am
thirdweb_playground	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Dec 11, 2024 0:07am
thirdweb-www	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Dec 11, 2024 0:07am
wallet-ui	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Dec 11, 2024 0:07am

[github-actions]

size-limit report 📦

Path	Size	Loading time (3g)	Running time (snapdragon)	Total time
thirdweb (esm)	44.93 KB (0%)	899 ms (0%)	277 ms (+21.07% 🔺)	1.2 s
thirdweb (cjs)	110.71 KB (0%)	2.3 s (0%)	589 ms (-27.19% 🔽)	2.9 s
thirdweb (minimal + tree-shaking)	5.58 KB (0%)	112 ms (0%)	78 ms (+462.18% 🔺)	190 ms
thirdweb/chains (tree-shaking)	506 B (0%)	10 ms (0%)	6 ms (+8.51% 🔺)	16 ms
thirdweb/react (minimal + tree-shaking)	19.01 KB (0%)	381 ms (0%)	161 ms (+234.53% 🔺)	541 ms

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 51.27%. Comparing base (714602d) to head (5b24faf).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5682      +/-   ##
==========================================
+ Coverage   51.23%   51.27%   +0.04%     
==========================================
  Files        1092     1092              
  Lines       57346    57346              
  Branches     4677     4686       +9     
==========================================
+ Hits        29383    29406      +23     
+ Misses      27247    27224      -23     
  Partials      716      716              

Flag	Coverage Δ		*Carryforward flag
legacy_packages	65.68% <ø> (ø)		Carriedforward from 714602d
packages	47.90% <100.00%> (+0.04%)	⬆️

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

Files with missing lines	Coverage Δ
packages/thirdweb/src/react/core/utils/wallet.ts	37.24% <100.00%> (+18.62%)	⬆️

... and 1 file with indirect coverage changes

[graphite-app]

Merge activity

• Dec 10, 6:55 PM EST: A user added this pull request to the Graphite merge queue.
• Dec 10, 7:08 PM EST: A user merged this pull request with the Graphite merge queue.