Skip to content

feat: resolve account via getConnectorClient in prepareTransactionRequest#5084

Merged
jxom merged 5 commits into
wevm:mainfrom
jxom:jxom/prepare-transaction-request-account
Apr 26, 2026
Merged

feat: resolve account via getConnectorClient in prepareTransactionRequest#5084
jxom merged 5 commits into
wevm:mainfrom
jxom:jxom/prepare-transaction-request-account

Conversation

@jxom
Copy link
Copy Markdown
Member

@jxom jxom commented Apr 26, 2026

Summary

prepareTransactionRequest now resolves the account via getConnectorClient when no account is provided. This lets connectors that implement getClient (e.g. webAuthn) supply a local signing account instead of falling back to a json-rpc account derived from the connection's address.

Also updated the webAuthn connector's getClient to return a signable account via provider.getAccount({ signable: true }).

Why

Previously prepareTransactionRequest used getConnection(config).address for the account, which gave a plain address. When the underlying viem action needed a local Account (e.g. for tempo feePayer/keyAuthorization flows), the call chain ended up with a json-rpc account, breaking signing flows that depend on the connector's local account.

Changes

  • packages/core/src/actions/prepareTransactionRequest.ts: resolve account via getConnectorClient(config, { assertChainId: false, chainId, account }) when none is supplied.
  • packages/core/src/tempo/Connectors.ts: webAuthn connector getClient now passes { signable: true } to provider.getAccount so the returned client carries a signable Account.

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 26, 2026

@jxom is attempting to deploy a commit to the Wevm Team on Vercel.

A member of the Team first needs to authorize it.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 26, 2026

🦋 Changeset detected

Latest commit: abad939

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

This PR includes changesets to release 5 packages
Name Type
@wagmi/core Patch
@wagmi/connectors Patch
wagmi Patch
@wagmi/solid Patch
@wagmi/vue 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

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Apr 26, 2026

Open in StackBlitz

@wagmi/cli

npm i https://pkg.pr.new/@wagmi/cli@5084

@wagmi/connectors

npm i https://pkg.pr.new/@wagmi/connectors@5084

@wagmi/core

npm i https://pkg.pr.new/@wagmi/core@5084

create-wagmi

npm i https://pkg.pr.new/create-wagmi@5084

wagmi

npm i https://pkg.pr.new/wagmi@5084

@wagmi/solid

npm i https://pkg.pr.new/@wagmi/solid@5084

@wagmi/vue

npm i https://pkg.pr.new/@wagmi/vue@5084

commit: a17d762

@jxom jxom merged commit 2058bd6 into wevm:main Apr 26, 2026
5 of 11 checks passed
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