Skip to content

fix: sponsored access key gas estimation#4890

Merged
jxom merged 1 commit into
wevm:mainfrom
parvahuja:parv/tempo-sponsored-access-key-gas
Jul 23, 2026
Merged

fix: sponsored access key gas estimation#4890
jxom merged 1 commit into
wevm:mainfrom
parvahuja:parv/tempo-sponsored-access-key-gas

Conversation

@parvahuja

Copy link
Copy Markdown
Contributor

prepareTransactionRequest still runs Tempo's afterFillParameters hook when callers pass parameters: []. The hook currently turns missing gas into a 10,000 or 20,000 placeholder, so a later eth_fillTransaction treats that placeholder as an explicit gas limit instead of estimating gas.

Only apply the existing access-key/WebAuthn buffer when gas has already been filled. This preserves the original buffers and the relay-signed gas behavior.

Regression coverage verifies that:

  • missing gas remains missing for a sponsored access-key micro-prepare
  • an existing 33,000 gas estimate still receives the 10,000 access-key buffer

Live verification reproduced the failure as a 20,000 gas limit for a transaction requiring 43,828, then completed the same OpenClaw payment with HTTP 200 after applying this guard.

Validation:

  • focused regression tests: 2 passed
  • pnpm build:esm
  • Biome check on changed source and tests

@vercel

vercel Bot commented Jul 22, 2026

Copy link
Copy Markdown

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

A member of the Team first needs to authorize it.

@changeset-bot

changeset-bot Bot commented Jul 22, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 63481b2

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

@parvahuja parvahuja changed the title Fix sponsored access key gas estimation fix: sponsored access key gas estimation Jul 23, 2026
@parvahuja
parvahuja marked this pull request as ready for review July 23, 2026 01:08
@jxom
jxom merged commit 3078687 into wevm:main Jul 23, 2026
2 of 3 checks passed
@github-actions github-actions Bot mentioned this pull request Jul 23, 2026
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