-
Notifications
You must be signed in to change notification settings - Fork 73
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
4337 Gas metering: Pimlico
VerifyingPaymaster
results, refactorings…
…, groundwork for `ERC20Paymaster` (#401) This PR updates the Pimlico benchmarking code to use the new v0.7 entrypoint and the new paymaster from Pimlico. Partially fixes #294 (Partially because there are blockers related to the ERC20 Paymaster; I will describe them below) ## Current Blockers For ERC20 Paymaster - They use the official Circle's USDC deployments on testnets with no liquidity, and the faucet drips 10 USDC daily. One user op costs around 7. - There are some problems with gas pricing. Currently, it wants roughly 7 USDC for a user operation on Base Sepolia (approximately 2x higher than the native token fee) and 537 USDC on Sepolia. The Pimlico team have been informed regarding the same at the time of writing this issue. Slack [Link](https://5afe.slack.com/archives/C05JA2HFQ1J/p1716898930935779?thread_ts=1715862769.219249&cid=C05JA2HFQ1J). ## Changes completed/in-progress - Using `base-sepolia` as well (`sepolia` sometimes has a high gas cost) - Using new signing struct with the latest 4337 Safe Module - Update of dependencies used in the Pimlico benchmark - Code refactors and type improvements - NPM scripts now explicitly contain the paymaster they should use - Addition of `VERBOSE` environment variable that outputs handy logging data to the console, which is very useful for debugging. [Example](https://dashboard.tenderly.co/mikhail-gnosis/project/simulator/8f7d3c77-b98c-49fd-badf-b35e4f5310c5?trace=0.1.0.0.2.4.2.1.4.0.0.2.0) --------- Co-authored-by: Mikhail <[email protected]>
- Loading branch information
Showing
18 changed files
with
1,762 additions
and
1,296 deletions.
There are no files selected for viewing
This file contains 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
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains 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
This file contains 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
This file contains 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
Oops, something went wrong.