feat(recurring-payment): contract deployment - #1767
Conversation
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
Greptile SummaryThe PR promotes ERC20RecurringPaymentProxy v0.2.0 as the default artifact, registers its Sepolia deployment, and preserves v0.1.0 behavior where required.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[Recurring-payment helper] --> B{Version explicitly supplied?}
B -->|Yes| C[Use requested version]
B -->|No| D{v0.2.0 deployed on network?}
D -->|Yes| E[Use v0.2.0]
D -->|No| F[Use v0.1.0]
C --> G[Resolve ABI and deployment]
E --> G
F --> G
Reviews (3): Last reviewed commit: "fix(recurring-payment): fall back to 0.1..." | Re-trigger Greptile |
There was a problem hiding this comment.
Approved.
This registers the Sepolia v0.2 deployment and makes version resolution safe: v0.2 is selected where deployed, while other networks retain v0.1. The legacy trigger helpers explicitly target v0.1, so promoting the artifact default does not change their ABI or destination.

Promote ERC20RecurringPaymentProxy v0.2.0 as default with Sepolia deployment
Deploys
ERC20RecurringPaymentProxyv0.2.0 to Sepolia at0xD7b1553ffE25491377a505f97f92cc44427D80A0and promotes it to the default artifact version.Version resolution
A
resolveRecurringPaymentProxyVersionhelper is introduced so that functions likegetRecurringPaymentProxyAddress,connectRecurringPaymentProxy, andgetPayerRecurringPaymentAllowanceautomatically select0.2.0on networks where it is deployed, and fall back to0.1.0elsewhere. Callers that explicitly need v1 (e.g.encodeRecurringPaymentTriggerandtriggerRecurringPayment) now passRECURRING_PROXY_V1directly.Constructor args
The
ERC20RecurringPaymentProxyv0.2.0 deployment script now resolves the admin address from the Safe admin artifact when available, falling back to theRECURRING_PAYMENT_ADMIN_ADDRESSenvironment variable for networks where no Safe is registered.Tests
version: RECURRING_PROXY_V1explicitly where v1 behaviour is expected.triggerRecurringPaymentBatch(v0.2.0) and that v0.1.0 still exposestriggerRecurringPayment.