Skip to content

Conversation

@TateB
Copy link
Contributor

@TateB TateB commented Apr 29, 2025

Currently when using sendTransaction with MetaMask as the client, if accessList is specified alongside maxFeePerGas and maxPriorityFeePerGas the call will fail. This is due to MetaMask incorrectly deriving a transaction type of 0x01 (see MetaMask/core#5720), but sendTransaction can easily be tweaked to ensure compatibility with all existing MetaMask clients. Making the transaction type explicit ensures that it isn't derived, and bypasses the issue entirely.

Changes:

  • Added type to the formatted transaction in sendTransaction.

@changeset-bot
Copy link

changeset-bot bot commented Apr 29, 2025

🦋 Changeset detected

Latest commit: 1ae0f86

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

This PR includes changesets to release 1 package
Name Type
viem 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

@vercel
Copy link

vercel bot commented Apr 29, 2025

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

A member of the Team first needs to authorize it.

@jxom jxom merged commit 198504d into wevm:main May 1, 2025
11 of 25 checks passed
waynezircuit added a commit to zircuit-labs/zkr-viem that referenced this pull request May 8, 2025
* tests: up snaps

* feat: sync to latest EIP-5792 spec (wevm#3542)

* feat: sync to latest 5792 spec

* chore: changeset

* up

* up

* up

* chore: use calculated version for snapshots

* fix: type

* chore: version package (wevm#3543)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* tests: up

* fix(op-stack): `unproven` is contextually a ready-to-prove case (wevm#3545)

* fix(op-stack): `unproven` is contextually a ready-to-prove case

Signed-off-by: bennett <[email protected]>

* Create olive-kings-burn.md

---------

Signed-off-by: bennett <[email protected]>
Co-authored-by: jxom <[email protected]>

* chore: version package (wevm#3546)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* docs: up cache

* chore: up lockfile

* feat: export `withCache`

* src/index.test

* chore: version package (wevm#3547)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* docs: fix wrong chain in initiateWitdrawal.md (wevm#3548)

* fix: resolves wevm#3551

* chore: version package (wevm#3552)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* feat: add lens mainnet (wevm#3553)

add lens

* doc(types): remarks for block tag and its usage (wevm#3556)

* doc(types): remarks for block tag and its usage

genesis is technically an accepted block tag, yet not sure its practical usage

* Update block.ts

---------

Co-authored-by: jxom <[email protected]>

* chore: pin foundry nightly

* chore: version package (wevm#3554)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* chore: up snapshot

* chore: up fork block number (wevm#3557)

* chore: up fork block number

* chore: up alto

* chore: up

* chore: up

* feat: add 5792 errors (wevm#3558)

* feat: 5792 error types

* chore: changeset

* chore: version package (wevm#3559)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* feat: Local CCIP-Read Batch Gateway (wevm#3440)

* add batched gateway

* fix import

* rename stuff, add tests

* rename from Batched to Batch

* use solidifyError

* chore: tweaks

* chore: up

* change detection method

* restore lock file

* chore: changeset

---------

Co-authored-by: jxom <[email protected]>

* chore: fix types

* fix: 5792 failure status codes

* feat: add multicall3 contract to Hoodi chain (wevm#3561)

* chore: change order of peaq public RPCs (wevm#3563)

Change order of peaq public RPCs

We've done extensive testing while supporting Peaq at Aragon and the publicnode endpoint has been significantly more reliable.

* feat: add tron shasta testnet (wevm#3564)

shasta

Co-authored-by: abs3ntdev <[email protected]>

* feat: add Lestnet chain (wevm#3565)

* fix: omit salt from erc7739

* docs: add note about using encodeFunctionData to data of estimateGas (wevm#3566)

* add note about using encodeFunctionData to data of estimateGas

* update text to be clearer

* chore: version package (wevm#3560)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* fix: newton chain rpc url typo (wevm#3567)

* fix: correct RPC URL configuration for Metis chains (wevm#3568)

fix: wrongly configured rpcUrls Metis chains

* feat: updated SKALE Network Chains (wevm#3570)

* Update and Deprecte SKALE Network Chains

* Changeset

* Update brown-ducks-smell.md

---------

Co-authored-by: jxom <[email protected]>

* feat: add core Testnet2 and Legacy Testnet details (wevm#3571)

* add core tesnet details

* updated changeset

* chore: changeset

* chore: format

* chore: lockfile

* feat: prefer `signTypedData` on owners for `toCoinbaseSmartWallet` (wevm#3579)

feat: prefer `signTypedData` on owners for cbsw

Co-authored-by: Jainil Sutaria <[email protected]>

* chore: version package (wevm#3572)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* chore: version package

* docs: update Optimism contract doc links for accuracy (wevm#3582)

* Update estimateFinalizeWithdrawalGas.md

* Update estimateProveWithdrawalGas.md

* Update getTimeToFinalize.md

* Update getTimeToProve.md

* Update getWithdrawalStatus.md

* fix: dedupe noble (wevm#3588)

* fix: dedupe noble

* Create thick-carrots-refuse.md

---------

Co-authored-by: jxom <[email protected]>

* feat: add Gunz Mainnet (wevm#3581)

* chore: version package (wevm#3589)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* chore: bump size

* fix: update EIP-5792 version to latest

* chore: up

* feat(4337): entrypoint 0.8 + 7702 auths (wevm#3580)

* Add entrypoint 0.8 & 7702 support

* Export entryPoint08Abi and entryPoint08Address

* Add 7702 support to older entrypoints as well

* Add tests

* Fix test

* Fix tests

* No need for deploySimulationsContract

* Add tests for getUserOperationReceipt

* Fic types for prepare user operation

* Add more tests and introduce new anvil for AA tests

* Fix toSimple7702SmartAccount

* Reset anvil for AA tests

* Fix todo

* Fix toCoinbaseSmartAccount tests

* tests: tweak

* tests: tweak

* tests: up

* chore: tweaks

* chore: tweaks

* chore: tweaks

* chore: up

* chore: changeset

* chore: up

* feat: tweaks

* feat: support 7702 on EP 0.6

* chore: up

* chore: tweaks

* chore: tweaks

* chore: up

* chore: up

* chore: up

* chore: tweaks

* chore: tweaks

* chore: add paymaster cases

* chore: changeset

* chore: tweaks

* Update thirty-boats-tell.md

* chore: up

---------

Co-authored-by: Garvit Khatri <[email protected]>

* feat: stabilize EIP-5792 (wevm#3592)

* feat: stabilize eip5792

* Update three-socks-thank.md

* chore: tweaks

* chore: tweaks

* chore: tweaks

* chore: tweaks

* chore: add exports

* chore: remove redundant changeset

* feat: export ERC-1155 abi (wevm#3531)

* chore: add the 1155 abi, expose abi in src/index

* chore: restore lockfile

* chore: up the wildcard import bundle size

* chore: another 0.2kb to appease the worfklow environments

* Create green-steaks-mate.md

---------

Co-authored-by: jxom <[email protected]>

* chore: version package (wevm#3591)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* chore: up

* chore: up size

* docs: up

* docs: up

* docs: fix docs regarding the `wallet_switchEthereumChain` method. (wevm#3597)

* Update switchChain.md

* Update switchChain.ts

* Update sendTransaction.md

* Update prepareTransactionRequest.md

* Update signTransaction.md

* feat: add MemeCore mainnet and testnet (wevm#3596)

* feat: add MemeCore mainnet and testnet

* Update index.ts

* Update nine-hats-itch.md

---------

Co-authored-by: jxom <[email protected]>

* fix: resolves wevm#3593

* feat: add Shardeum mainnet (wevm#3599)

* Add Shardeum mainnet

* Update shardeum.ts

* Update shardeum.ts

---------

Co-authored-by: jxom <[email protected]>

* chore: version package (wevm#3598)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* fix: ENS batch gateway function signatures (wevm#3603)

* fix: ENS batch gateway function signatures

* Use ABI that's currently on mainnet

* Add changeset

* chore: tweaks

* chore: tweaks

* Update tiny-rabbits-watch.md

---------

Co-authored-by: jxom <[email protected]>

* revert: "fix: ENS batch gateway function signatures" (wevm#3605)

Revert "fix: ENS batch gateway function signatures (wevm#3603)"

This reverts commit 1c5326f.

* feat: add `type` to `sendTransaction` (wevm#3600)

* feat: add transaction type to formatted transaction requests

* Create khaki-nails-eat.md

---------

Co-authored-by: jxom <[email protected]>

* feat: up Fraxtal and Fraxtal testnet gas token to FRAX  (wevm#3607)

* Update fraxtal.ts with new gas token

* Update fraxtalTestnet.ts to new gas token

* Create purple-dingos-swim.md

---------

Co-authored-by: jxom <[email protected]>

* chore: version package (wevm#3604)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* tests: update snapshots

* fix: `wallet_getCapabilities` params (wevm#3611)

* chore: version package (wevm#3612)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* feat: typed eip5792 capabilities (wevm#3615)

* wip: typed capabilities

* w

* w

* docs

* w

* w

* w

* chore: update plumenetwork.xyz to plume.org (wevm#3620)

* chore: version package (wevm#3618)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* tests: up snapshots

* feat(experimental): add `connect` + `disconnect` actions (wevm#3619)

* wip: checkpoint

* chore: changeset

* wip: checkpoint

* chore: tweaks

* u

* w

* wip: checkpoint

* Update weak-wasps-battle.md

* w

* Update thin-flowers-impress.md

* w

* w

* wip: checkpoint

* w

* feat(experimental): add `addSubAccount` action (wevm#3621)

* wip: checkpoint

* chore: changeset

* wip: checkpoint

* chore: tweaks

* u

* w

* wip: checkpoint

* Update weak-wasps-battle.md

* w

* Update thin-flowers-impress.md

* w

* w

* wip: checkpoint

* w

* w

* wip: checkpoint

* wip: checkpoint

* w

* fix: exports

* fix: lockfile

* fix: make watchBlocks with `emitMissed` work from genesis block (wevm#3626)

* chore: add internal flag

* make watchBlocks with `emitMissed` work from genesis block

* up

* Update spotty-beds-grow.md

* Update spotty-beds-grow.md

---------

Co-authored-by: jxom <[email protected]>

* feat: sync erc7895

* chore: version package (wevm#3622)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* docs: up cache

* fix: resolves wevm#3627

* test: up

* docs: up

* feat: remove celo peculiarities from block structure (wevm#3623)

* remove celo peculiarities from block structure

* Update types.ts

* Update thick-mayflies-itch.md

* Update thick-mayflies-itch.md

---------

Co-authored-by: jxom <[email protected]>

* feat: add arenaz chain (wevm#3625)

* chore: add internal flag

* Added arenaz chain

* Added arenaz chain

* Update register.ts

---------

Co-authored-by: jxom <[email protected]>

* fix: types

* chore: up snaps

* chore: up disputegameFactory for World Sepolia (wevm#3632)

* Fix disputegameFactory for World Sepolia

Please see: https://docs.world.org/world-chain/developers/world-chain-contracts

* Create stupid-weeks-complain.md

---------

Co-authored-by: jxom <[email protected]>

* feat: up toHex.md (wevm#3630)

removing the duplicate value word.

* feat: support block overrides on simulate/call (wevm#3631)

* feat: support block overrides on simulate/call

* chore: tweak

* Update big-humans-learn.md

---------

Co-authored-by: jxom <[email protected]>

* fix: resolves wevm#3629

* chore: up size

* docs: up cache

* chore: up

* chore: changeset

* chore: up size

* chore: version package (wevm#3628)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* chore: check fix

* ci: dependency fix

* chore: formatting

---------

Signed-off-by: bennett <[email protected]>
Co-authored-by: jxom <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: bmzig <[email protected]>
Co-authored-by: Gonzalo D'Elia <[email protected]>
Co-authored-by: Yutaro-Mori-eng <[email protected]>
Co-authored-by: sam bacha <[email protected]>
Co-authored-by: Andrew Raffensperger <[email protected]>
Co-authored-by: Ilies Zareb <[email protected]>
Co-authored-by: Evan Aronson <[email protected]>
Co-authored-by: abs3ntdev <[email protected]>
Co-authored-by: abs3ntdev <[email protected]>
Co-authored-by: Ravi Singh Lodhi <[email protected]>
Co-authored-by: SethTenenbaum <[email protected]>
Co-authored-by: Markeljan <[email protected]>
Co-authored-by: Manuel Barbas <[email protected]>
Co-authored-by: Akshaya Gangatharan <[email protected]>
Co-authored-by: Jainil Sutaria <[email protected]>
Co-authored-by: Noisy <[email protected]>
Co-authored-by: v1rtl <[email protected]>
Co-authored-by: Garvit Khatri <[email protected]>
Co-authored-by: Robby Uitbeijerse <[email protected]>
Co-authored-by: 0xsanson <[email protected]>
Co-authored-by: tech-memecore <[email protected]>
Co-authored-by: Dan Cortes <[email protected]>
Co-authored-by: tate <[email protected]>
Co-authored-by: Ryan Adami <[email protected]>
Co-authored-by: Eugene Y. Q. Shen <[email protected]>
Co-authored-by: Sebastian Lorenz <[email protected]>
Co-authored-by: Aaron DeRuvo <[email protected]>
Co-authored-by: Ali Hosseini <[email protected]>
Co-authored-by: Zain Bacchus <[email protected]>
Co-authored-by: Bahador <[email protected]>
Co-authored-by: danilo neves cruz <[email protected]>
@tmm tmm mentioned this pull request Oct 23, 2025
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.

2 participants