fix: Add check for whether account is deployed - Githubissues

stackup-wallet / userop.js

A simple JS library for building ERC-4337 UserOperations.

MIT License

113 stars 32 forks source link

fix: Add check for whether account is deployed #83

Open SilasZhr opened 1 year ago

SilasZhr commented 1 year ago

What?

This PR updates both simpleAccount.ts and gasLimit.ts with initCode issue.

Why?

There is an issue when I want to send the first UO if I've deployed account proxy by calling senderCreator or AccountFactory.
It always checks the nonce in EntryPoint in Here. But I think user should be able to use EntryPoint with deployed account.

How?

check the account code instead of ctx.op.nonce when setting initCode for UO
check initCode length before getting factory address from initCode, and return zero for CreationGas if initCode is 0x

Testing?

Passed