Skip to content

Conversation

hdang-circle
Copy link

@hdang-circle hdang-circle commented Aug 6, 2025

Summary

This PR is to deploy EPv0.6 paymasters for World, Sonic, Celo mainnet and testnet.

Detail

Added SponsorPaymasterProxy_constructor_args and SponsorPaymasterProxy.json file which is used to verify the proxy contract after the deployment is completed successfully.

Checklist

  • Did you add new tests and confirm all tests pass? (yarn test)
  • Did you ensure any new Solidity source code files meet minimum test coverage requirements? (yarn coverage)
  • Did you update relevant docs? (docs are found in the docs folder)
  • Do your commits follow the Conventional Commits standard?
  • Does your PR title also follow the Conventional Commits standard?
  • Did you run lint (yarn lint) and fix any issues?
  • Did you run formatter (yarn format:check) and fix any issues (yarn format:write)?

Testing

Checked Block Explorer for World, Sonic, and Celo testnet and mainnet for funding, adding verifying signers, and staking. In this case, only World and Celo requires staking.

Documentation

N/A

@hdang-circle hdang-circle changed the title [AA-2733] Deploy EPv0.6 paymasters for World, Sonic, Celo mainnet and testnet feat: Deploy EPv0.6 paymasters for World, Sonic, Celo mainnet and testnet Aug 6, 2025
Copy link
Contributor

@stigter-circle stigter-circle Aug 7, 2025

Choose a reason for hiding this comment

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

@hieule-circle looks like the content didn't get committed.

Copy link
Author

Choose a reason for hiding this comment

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

@stigter-circle constructor args added

Copy link
Contributor

@huaweigu huaweigu left a comment

Choose a reason for hiding this comment

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

Have we also verified these contracts?

// A list of chains that need to run setup work after deployment. Please adjust the list based on the chain
// expansion needs.
function getChainsForSetup() internal pure returns (string[4] memory) {
return ["base-sepolia", "base", "op-sepolia", "op"];
Copy link
Contributor

Choose a reason for hiding this comment

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

Why removing testnets, op and base?

Copy link
Author

Choose a reason for hiding this comment

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

so I deployed the contracts for celo, sonic, and world testnet first and then celo, sonic, and world mainnet afterwards. that's why you are seeing those three chains in the getChainsForSetup() function

Copy link
Contributor

Choose a reason for hiding this comment

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

Okay, so these are the chains currently being deployed, we might want to consider renaming the function to better reflect that.

Copy link
Author

Choose a reason for hiding this comment

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

@huaweigu yeah that makes sense, do you have any suggestions in mind? I'm thinking maybe something like getCurrentChainsForSetup(), getCurrentDeploymentChains(), etc.

Copy link
Contributor

Choose a reason for hiding this comment

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

getCurrentDeploymentChains sounds good!

@@ -0,0 +1 @@
00000000000000000000000026f77c6455518caf29a5414004c5a2ef441228b100000000000000000000000000000000000000000000000000000000000000400000000000000000000000000000000000000000000000000000000000000044485cc9550000000000000000000000007e99c7725463a858c34d0fd69a09b11fd75087cc0000000000000000000000007e99c7725463a858c34d0fd69a09b11fd75087cc00000000000000000000000000000000000000000000000000000000 No newline at end of file
Copy link
Contributor

Choose a reason for hiding this comment

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

nit missing new line

@hdang-circle
Copy link
Author

@huaweigu Yes, the contracts have been verified. Confirmed with @stigter-circle

@hdang-circle hdang-circle requested a review from huaweigu August 8, 2025 20:00
@weikangsong weikangsong merged commit 5d528b8 into circlefin:entry-point-v0.6 Aug 11, 2025
6 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