refactor: optimize zksync chain detection and add Creator Testnet test

[joaquim-verges]

Problem solved

Short description of the bug fixed or feature added

---

PR-Codex overview

This PR focuses on improving the handling of bytecode in the thirdweb library, particularly for zksync chains, and refactoring related functionalities for better clarity and performance.

Detailed summary

• Removed checks for publishedMetadata in favor of direct access to description and logo.
• Introduced fetchBytecodeFromCompilerMetadata for better bytecode retrieval.
• Updated tests for SmartWallet to include a new transaction test.
• Simplified isZkSyncChain logic for clarity.
• Adjusted several components to use the new bytecode fetching method.

✨ 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)
docs-v2	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 2, 2024 3:43am
thirdweb_playground	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 2, 2024 3:43am
thirdweb-www	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 2, 2024 3:43am
wallet-ui	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 2, 2024 3:43am

[changeset-bot]

🦋 Changeset detected

Latest commit: 454f063

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.

[joaquim-verges]

• refactor: optimize zksync chain detection and add Creator Testnet test #4883 👈
• main

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

Join @joaquim-verges and the rest of your teammates on Graphite

[github-actions]

size-limit report 📦

Path	Size	Loading time (3g)	Running time (snapdragon)	Total time
thirdweb (esm)	41.98 KB (0%)	840 ms (0%)	630 ms (+54.73% 🔺)	1.5 s
thirdweb (cjs)	101.58 KB (0%)	2.1 s (0%)	1.1 s (+18.26% 🔺)	3.1 s
thirdweb (minimal + tree-shaking)	4.82 KB (0%)	97 ms (0%)	44 ms (+17.06% 🔺)	141 ms
thirdweb/chains (tree-shaking)	498 B (0%)	10 ms (0%)	19 ms (-1.16% 🔽)	29 ms
thirdweb/react (minimal + tree-shaking)	16.75 KB (0%)	335 ms (0%)	93 ms (+2.66% 🔺)	428 ms

[codecov]

Codecov Report

Attention: Patch coverage is 72.00000% with 21 lines in your changes missing coverage. Please review.

Project coverage is 47.60%. Comparing base (78cbc41) to head (454f063).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
...ages/thirdweb/src/utils/any-evm/deploy-metadata.ts	79.41%	7 Missing ⚠️
...hirdweb/src/contract/deployment/utils/bootstrap.ts	16.66%	5 Missing ⚠️
...rdweb/src/extensions/prebuilts/deploy-published.ts	54.54%	5 Missing ⚠️
...thirdweb/src/contract/actions/compiler-metadata.ts	0.00%	2 Missing ⚠️
...thirdweb/src/utils/any-evm/zksync/isZkSyncChain.ts	87.50%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4883      +/-   ##
==========================================
+ Coverage   47.59%   47.60%   +0.01%     
==========================================
  Files        1054     1054              
  Lines       57144    57164      +20     
  Branches     3914     3917       +3     
==========================================
+ Hits        27196    27212      +16     
- Misses      29260    29263       +3     
- Partials      688      689       +1     

Flag	Coverage Δ		*Carryforward flag
legacy_packages	65.68% <ø> (ø)		Carriedforward from 78cbc41
packages	43.35% <72.00%> (+0.01%)	⬆️

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

Files with missing lines	Coverage Δ
...ages/thirdweb/src/contract/deployment/publisher.ts	79.31% <ø> (-0.18%)	⬇️
.../any-evm/compute-published-contract-deploy-info.ts	100.00% <100.00%> (ø)
...thirdweb/src/contract/actions/compiler-metadata.ts	87.09% <0.00%> (-0.41%)	⬇️
...thirdweb/src/utils/any-evm/zksync/isZkSyncChain.ts	94.59% <87.50%> (+3.16%)	⬆️
...hirdweb/src/contract/deployment/utils/bootstrap.ts	72.72% <16.66%> (-1.45%)	⬇️
...rdweb/src/extensions/prebuilts/deploy-published.ts	78.68% <54.54%> (-1.24%)	⬇️
...ages/thirdweb/src/utils/any-evm/deploy-metadata.ts	80.82% <79.41%> (+5.82%)	⬆️

[graphite-app]

Merge activity

• Oct 1, 11:40 PM EDT: jnsdls added this pull request to the Graphite merge queue.
• Oct 1, 11:44 PM EDT: jnsdls merged this pull request with the Graphite merge queue.