Merged
Conversation
…transactionadder-hook Partial viem migration
…7-cleanup-documentation
…getreceipt-hook Partial viem migration
…7-cleanup-documentation
…recoverfundsfromproxy-hook Partial viem migration
shoom3301
reviewed
Apr 13, 2026
shoom3301
reviewed
Apr 13, 2026
Collaborator
|
@brunota20 token search doesn't work as expected, try to seach for "SHO" in Mainnet. This PR:
|
shoom3301
reviewed
Apr 13, 2026
shoom3301
reviewed
Apr 13, 2026
shoom3301
reviewed
Apr 13, 2026
Collaborator
|
@brunota20 there is also an issue with wallet name and icon displaying.
|
shoom3301
reviewed
Apr 13, 2026
shoom3301
reviewed
Apr 13, 2026
…7-cleanup-documentation
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
shoom3301
approved these changes
Apr 14, 2026
Collaborator
shoom3301
left a comment
There was a problem hiding this comment.
Tested the main scenarios, it works well!
Thanks a lot @bsvalverde and @brunota20 !
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.




Summary
Preview links:
This PR removes
ethersentirely and migrates all blockchain interaction logic to awagmi+viem-centric architecture.This PR was built up incrementally through the following PRs. Note that a small number of commits were also made directly to this branch and are not associated with a separate PR.
What Changed
1. Dependency Removal
ethersmulticallweb3-react2. Contract Interaction Refactor
wagmireadContract/writeContractwagmi’s built-in multicall supportviemadapter instead of theethers v5adapterSigner resolution is now handled automatically by the
wagmiadapter (connected wallet signer or default permit account where applicable).3.
ethers→viemMigrationAll
ethersutilities have been replaced with theirviemequivalents:viemutilitiesviemencoding helpersBigNumber→ nativebigintviemhelpersThis removes the
BigNumberabstraction entirely and standardizes numeric handling onbigint.To Test
Here are recorded demos of:
Background
Previously, the project relied on:
ethersfor providers, signers, contract instances, encoding utilities, andBigNumberweb3-reactfor wallet/provider managementmulticalllibrary for batched readsThis created:
ethers,web3-react, andwagmiBigNumbervs native types)Since
wagmialready provides:readContract/writeContractAnd
viemprovides:bigint-based numeric handlingMaintaining
ethersandweb3-reactwas redundant.This refactor consolidates all Web3 logic under a single abstraction stack (
wagmi+viem), reduces complexity, removes redundant layers, and simplifies long-term maintenance.