Add ERC-4337 and ERC-7702 account implementations

[ernestognw]

Description

After multiple rounds of iterations, the Account Implementations were reduced to an AccountCore, a opinionated Account contract and a couple of AbtractSigners that implement signature validation logic.

The documentation will later include how to compose these contracts and they should be also customizable in Wizard

[netlify]

✅ Deploy Preview for community-contracts ready!

Name	Link
🔨 Latest commit	dcdae8d
🔍 Latest deploy log	https://app.netlify.com/sites/community-contracts/deploys/67576a213ab09d000803210d
😎 Deploy Preview	https://deploy-preview-25--community-contracts.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[Amxx]

There is a lot of duplicated code between AccountECDSA, AccountP256 and AccountRSA:

• ERC7739Signer
• ERC721Holder
• ERC1155HolderLean

maybe we should bundle all that in some way that all variants can get easily.

[ernestognw]

@frangio and I consider that the Base Account should not inherit from ERC7339Signer since ERC-7339 is a workaround with important limitations.

[frangio]

important limitations

I was thinking of composability. This made me think it's not good enough to be a default.

On the other hand, 1271 without 7739 is unsafe... possibly with bigger impact than the lack of composability.

So let's discuss it a bit more.

[ernestognw]

I updated moving the EIP 712 user op typed hash logic to AccountBase.

One thing is that AccountBase will need EIP-712 to sign the typed user op hash, anyway. In that case, I'm reconsidering having ERC-7739 by default.

On the other hand, 1271 without 7739 is unsafe... possibly with bigger impact than the lack of composability.

Right. I agree we should be opinionated on the ERC-1271 implementation. iiuc, implementing ERC-7803 would allow for better composability and works better as a default, is that correct?

[frangio]

ERC-7803 solves composability but it's not usable today, it needs to be accepted and deployed across wallets, there is no guarantee that will happen at this point.

[Amxx]

IMO this is good to merge.

There is still work to do, but that can be done in parallele in separate PR, such that the review works remains acceptable (many small PR are easier to process)

Upcomming work (some of which as aready started)

• Documentation
• Testing (particularly EIP7702)
• Extensions (7579, ...)
• Signer constructor/initialization
• UserOperation -> hash for signature.

[ernestognw]

I reviewed the changes and saw they required a couple of documentation (NatSpec) updates so I committed them myself. Basically, I fixed multiple broken links in the documentation (due to #30). I think it's fine to hardcode the OpenZeppelin docs for now so that the correct source is linked.

I'm merging this since there are no controversial changes, so I can focus on ERC7702 testing, documentation, an ERC7579 implementation.

Signer constructor/initialization

What are your thoughts here?