tools: update mainnet/calibnet PDPVerifier deploy helpers#255
Merged
Conversation
update mainnet/calibnet PDPVerifier deploy helpers
chore: bump PDPverifier version
make PDPFees.sybilFee externally readable
This reverts commit 3c4ab45.
fix: make PDPFees.sybilFee externally readable
ZenGround0
approved these changes
Mar 17, 2026
Contributor
ZenGround0
left a comment
ZenGround0
left a comment
There was a problem hiding this comment.
About to push a commit to fixup devnet deploy script. At which point this is ok to merge.
github-project-automation
bot
moved this from 🔎 Awaiting review
to ✔️ Approved by reviewer
in FOC
This was referenced Mar 17, 2026
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
2 participants
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.
Summary
Update the main PDPVerifier deployment helpers to use the current 4-argument constructor. This is needed to unblock next steps in #252 and #253
As of
v3.2.0,PDPVerifierno longer takes only the reinitializer version in its constructor. New implementations must also be deployed with the USDFC token address, the USDFC sybil-fee amount, and the payments contract address.This PR updates the scripts we need to unblock Calibration and Mainnet implementation deployment, plus the generic upgrade helper that also deploys replacement PDPVerifier implementations.
Why
PDPVerifiernow defines the following constructor insrc/PDPVerifier.sol:_initializerVersion_usdfcTokenAddress_usdfcSybilFee_paymentsContractAddressThose values are constructor-time configuration for the implementation contract. They are not proxy initializer inputs, and they cannot be filled in later by
initialize().That means any script that publishes a new PDPVerifier implementation must pass all four values at deploy time. The existing helpers still assumed the old single-argument constructor and were therefore stale for
v3.2.0.