Skip to content

Switch to viem - #584

Open
sirpy wants to merge 1 commit into
masterfrom
switch-to-viem
Open

sirpy wants to merge 1 commit into
masterfrom
switch-to-viem

Conversation

@sirpy

@sirpy sirpy commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

Description

Switch wallet to viem with fallback rpcs from chainlist use @goodwidget/core createviemfallbackclient

Summary by Sourcery

Migrate blockchain wallet operations from Web3 to viem with cached fallback RPC clients while preserving transaction, contract, and KMS workflows.

New Features:

  • Add viem-based wallet clients with GoodWidget fallback RPC discovery and chain configuration.
  • Support viem-based contract reads, event queries, transaction encoding, and local or KMS transaction submission across wallet workflows.

Bug Fixes:

  • Improve transaction recovery and receipt handling for nonce errors, insufficient funds, timeouts, and reverted transactions.

Enhancements:

  • Replace Web3 wallet and contract interactions with viem clients while preserving wallet queues, callbacks, balances, fees, and KMS signing behavior.
  • Update staking, fishing, verification, and wallet integrations for viem contract addresses, bigint values, event arguments, and account management.

Build:

  • Add viem and @goodwidget/core dependencies and update the lockfile.

Tests:

  • Update blockchain tests for viem APIs and add coverage for fallback RPC setup, transaction submission, KMS signing, nonce recovery, receipts, and bigint handling.

Chores:

  • Adjust lint configuration and related formatting for the new wallet implementation.

@sourcery-ai

sourcery-ai Bot commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

Sorry @sirpy, you've used your own review budget of 250,000 diff characters for the last 7 days.

You can request another review in 1 day and 1 hour by commenting @sourcery-ai review. Upgrade to get a review now.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant