Skip to content

Commit

Permalink
chore: add note about bricked account
Browse files Browse the repository at this point in the history
  • Loading branch information
kopy-kat committed Oct 15, 2024
1 parent ce43c9a commit f405295
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions src/Safe7579Launchpad.sol
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,11 @@ import { IERC7579AccountEvents } from "./interfaces/IERC7579Account.sol";
/**
* Launchpad to deploy a Safe account and connect the Safe7579 adapter.
* Check Readme.md for more information.
* Note that (unlike some other smart accounts) the Safe7579 will only finish initializing the
* account during the execution phase of the first UserOperation in order to comply with ERC-4337.
* This means that if the exection of the first UserOperation fails, the account could be
* permanently bricked. To avoid this, make sure that all the initialization and call data are
* correctly set up and do not cause a revert.
* Special thanks to [nlordell (Safe)](https://github.com/nlordell), who came up with [this
* technique](https://github.com/safe-global/safe-modules/pull/184)
* @author rhinestone | zeroknots.eth
Expand Down

0 comments on commit f405295

Please sign in to comment.