Skip to content
This repository has been archived by the owner on Oct 7, 2024. It is now read-only.

feat: add support for ERC-4337 accounts #213

Merged
merged 17 commits into from
Dec 6, 2023
Merged

Conversation

danroc
Copy link
Contributor

@danroc danroc commented Nov 29, 2023

Description

This PR adds support to ERC-4337 accounts. These new accounts should implement the following signing flow:

image

Changes

This PR adds a new EthKeyring interface to be used internally by MetaMask.

It also adds the follwoing methods that are required to implement the flow described above:

  • prepareUserOperation
  • patchUserOperation
  • signUserOperation

@danroc danroc requested a review from a team as a code owner November 29, 2023 11:52
gantunesr
gantunesr previously approved these changes Dec 4, 2023
docs/evm-methods.md Show resolved Hide resolved
docs/evm_methods_userOp.md Show resolved Hide resolved
@danroc danroc added this pull request to the merge queue Dec 6, 2023
Merged via the queue into main with commit 43d90b2 Dec 6, 2023
17 checks passed
@danroc danroc deleted the feature/erc-4337-support branch December 6, 2023 13:53
legobeat added a commit that referenced this pull request Dec 6, 2023
legobeat added a commit that referenced this pull request Dec 6, 2023
danroc pushed a commit that referenced this pull request Dec 7, 2023
danroc pushed a commit that referenced this pull request Dec 7, 2023
github-merge-queue bot pushed a commit that referenced this pull request Dec 7, 2023
* 2.0.0

* update changelog

* format changelog

* add changelog entries for #204 and #213

---------

Co-authored-by: github-actions <[email protected]>
Co-authored-by: legobt <[email protected]>
danroc added a commit to MetaMask/core that referenced this pull request Jan 22, 2024
## Description

**Depends on:**

- [x] <MetaMask/keyring-api#213>
- [x] <MetaMask/KeyringController#315>
- [x] <#3617>

This PR adds the following new methods available in the `EthKeyring`
interface:

- `prepareUserOperation`
- `patchUserOperation`
- `signUserOperation`

See the [PR](MetaMask/keyring-api#213) in the
`keyring-api` for more details.

## Checklist

- [ ] I've updated the test suite for new or updated code as appropriate
- [ ] I've updated documentation for new or updated code as appropriate
(note: this will usually be JSDoc)
- [ ] I've highlighted breaking changes using the "BREAKING" category
above as appropriate
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
team-accounts This should be handled by the Accounts Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants