Skip to content

Conversation

@hanzel98
Copy link
Contributor

What?

  • New version of the DeleGator contracts. This contract has been created to be used with EIP7702 Accounts. It deletes the UUPS proxy logic that we used to have in the previous DeleGators. it doesn't modify the previous contracts instead it creates a new core and a new implementation.
  • The new implementation is a stateless deleGator that doesn't allow changing the signer/owner of the contract, the only account that can control the smart account is the same EOA where the contracts run. The EOA can interact with the contract via submitting direct transactions to the contract functions, or via the entry point using signatures.

Why?

  • The EIP7702 does not require UUPS proxy functionality, because this EIP updates the smart contracts using a different mechanism.

How?

  • New set of contracts without touching the previous ones.
  • Deleted namespace storage because there is no storage

@hanzel98 hanzel98 requested a review from a team as a code owner January 31, 2025 18:47
@hanzel98 hanzel98 marked this pull request as draft January 31, 2025 18:47
Copy link
Contributor

@jeffsmale90 jeffsmale90 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like how simple the stateless implementation is.

@hanzel98 hanzel98 force-pushed the feat/eip7702-stateless-delegator branch from b3e996d to a0e2b54 Compare February 3, 2025 04:43
jeffsmale90
jeffsmale90 previously approved these changes Feb 5, 2025
Copy link
Contributor

@jeffsmale90 jeffsmale90 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very high level review - but LGTM.

One comment on the tests.

@hanzel98 hanzel98 marked this pull request as ready for review February 5, 2025 15:50
@hanzel98 hanzel98 force-pushed the feat/eip7702-stateless-delegator branch 2 times, most recently from 062d43b to d0023a5 Compare February 6, 2025 20:05
@hanzel98 hanzel98 force-pushed the feat/eip7702-stateless-delegator branch from d0023a5 to 85faea1 Compare February 7, 2025 01:36
@hanzel98 hanzel98 force-pushed the feat/eip7702-stateless-delegator branch from 85faea1 to efa7ca0 Compare February 7, 2025 02:47
@hanzel98 hanzel98 merged commit d522a38 into main Feb 7, 2025
4 checks passed
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.

4 participants