Merged
Conversation
Licensing section was not used and adding all the authors to a file is redundant given the git history.
axic
pushed a commit
to ipsilon/solidity
that referenced
this pull request
Apr 22, 2025
* ✨ Receiver (argotorg#622) * ⚡📬 callback fallback * 🤏 scrunch 🤏 scrunch comments a bit * 👌 Fallback function triggered when upper 224 bits of 0x40 are mostly zero. 👌 Fallback function triggered when upper 224 bits of 0x40 are mostly zero. * ⚡ optimize / formatting ⚡ optimize / formatting * ✍️ Write some tests n stuff * ⚡ zimplify codesize * 🥢 prefer less gas * 🥢 nit that dbl comment 🥢 nit that dbl comment * 🥌 Simpler and future-proofed is also cheaper ofc * 👌 Back to known magic vals. * 📁 File into accounts * 📍 Doc acct contracts in README * 🤏 header 🤏 header * 🤏 header 🤏 header * 🪡 Stitch in comment on fallback pass cond * ✨ EIP4337 Simple Account (argotorg#623) * 🪄 EIP4337 Base Account * 📁 Make and refile into `accounts` * ⚡ Pinion owned clone base & optimize & typo nits * 🥌 Simplify and support contract signing owner * Tidy * Update README * Tidy * Optimize * Add tests * Tidy * Optimize * Tidy * Add comment * Add deposit functions * Fix * Tidy * Use gas() instead of not(0) * Edit comments * Fix comments delegatecall -> call * Use SignatureCheckerLib to validate signature * Add signature check tests * Edit tests * ✏️ Txt nits * Add delegateExecute * Add delegateExecute test * Tidy * 🏧 As in Ownable, onlyOwner functions are payable * Make initialize payable * Edit comments * Edit comments * Futureproof * Strengthen tests * Change to use Execution struct * Execution -> Call * Optimize * Optimize and strengthen tests * ✨ ERC4337: 1️⃣2️⃣7️⃣1️⃣ (argotorg#642) * 1️⃣2️⃣7️⃣1️⃣ * 🤌 Pull branch * ✔️ EOA case (1o1) use hybrid validation method * ✔️ EOA case test * 📸 snapshot * Optimize * Edit comments * Add comment * Optimize * Copypasta toEthSignedMessageHash to SignatureCheckerLib * Add delegateGuard * Add comment * Tidy * Add missing restore part of free memory pointer * 🥢 Typo nit Remove extra `be` in comment and also use `so` as coordinating conjunction removing `that`. So it's not a subordinating conjunction. * 🤏 fmt 🤏 arrange imports (libzip is w/n receiver so can come next and ++ aesthetics) and remove re-explanatory comment. * ✨ ERC4337Factory (argotorg#644) * Add ERC4337Factory * Update README * Edit comment * 🤏 Nits Nit comment and payable constructor (why not it's a factory right) * fix offset in _call() * removed funcs & add tests * ✂️ Remove unused internal func `_call` no longer used * checkStartsWithCaller -> checkStartsWith * Simplify tests --------- Co-authored-by: ross <92001561+z0r0z@users.noreply.github.com> Co-authored-by: atarpara <akpatel0618@gmail.com> Co-authored-by: 0xlgtm <simon.tan.yu.jing@gmail.com> * delegateGuard -> storageGuard * Tidy test * ✨ ERC4337 direct storage (argotorg#645) * Add test * ~ snap * assert optimizes & goes harder in storguard * mock EP receive() to match StakeManager.sol * ⚡ optimize deposit via EP receive() * ⚡ optimize balance getter w/ STL fmt * 🤏 assert -> require * Add comments and extcodesize check on addDeposit * Tidy * Move back extcodesize check to deposit functions, make entryPoint view * Snapshot --------- Co-authored-by: ross <92001561+z0r0z@users.noreply.github.com> Co-authored-by: atarpara <akpatel0618@gmail.com> Co-authored-by: 0xlgtm <simon.tan.yu.jing@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Licensing section was not used and adding all the authors to a file is redundant given the git history.