fix(account-abstraction): prepare fees when estimating user operation gas#4768
Conversation
|
@nikhilkumar1612 is attempting to deploy a commit to the Wevm Team on Vercel. A member of the Team first needs to authorize it. |
🦋 Changeset detectedLatest commit: e287156 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
jxom
left a comment
There was a problem hiding this comment.
Verified locally against a forked-anvil harness: without fees, the outgoing eth_estimateUserOperationGas payload now includes maxFeePerGas/maxPriorityFeePerGas (previously absent since the invalid preparation keys made the fees step a no-op); userOperation.estimateFeesPerGas hook values reach the wire; explicit fee overrides pass through unchanged; no double fee-fetch in the sendUserOperation path (inner prepare short-circuits on resolved fees).
Description
Fixes #4726.
estimateUserOperationGaswas passingmaxFeePerGasandmaxPriorityFeePerGasin theprepareUserOperationparameterslist, but those are User Operation field names, not valid preparation keys.This updates the call to use the existing
feespreparation key instead, soprepareUserOperationruns theclient.userOperation.estimateFeesPerGashook before sendingeth_estimateUserOperationGas.This makes the outgoing gas estimation request include
maxFeePerGasandmaxPriorityFeePerGasfor strict bundlers.Included a patch changeset.
Types of changes
Checklist