fix: avoid nonce gap in nonceManager (#3142)

[0xrouss]

Fixes #3142 by deferring the nonce consumption in prepareTransactionRequest until after gas estimation succeeds, avoiding a nonce gap when gas estimation fails.

Changes

• Reordered logic so nonceManager.consume(...) is called after gas estimation
• Added a changeset for a patch release

Details

• Old behavior: We called nonceManager.consume(...) before gas estimation, so if gas estimation failed, we never broadcast a transaction but had already incremented the nonce.
• New behavior: We’ve reordered the logic in prepareTransactionRequest so that all fees and gas estimation happen first. Only if those steps succeed do we call consume(...) from the nonce manager. This way, no extra nonce is burned on failure.
• Alternative considered: Rolling back the nonce in a try/catch: We tested that approach but decided to defer consume entirely until the end, which is simpler and avoids the need for a rollback

[changeset-bot]

🦋 Changeset detected

Latest commit: ec689f7

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]

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

A member of the Team first needs to authorize it.