Skip to content

Comments

Add PDP scheduling API boundary interfaces#616

Merged
anjor merged 4 commits intomainfrom
feat/pdp-scheduling-api-boundary
Feb 20, 2026
Merged

Add PDP scheduling API boundary interfaces#616
anjor merged 4 commits intomainfrom
feat/pdp-scheduling-api-boundary

Conversation

@anjor
Copy link
Collaborator

@anjor anjor commented Feb 19, 2026

Next after this:

  1. Add a real schedule-level deal-type source (DB field or deterministic resolver rule), replacing test-only resolver behavior.
  2. Implement PDPProofSetManager (ensure proof set + queue add-roots tx).
  3. Implement PDPTransactionConfirmer (receipt polling + confirmation depth).
  4. Wire runPDPSchedule to:
    • pick wallet/car batch,
    • ensure proof set,
    • queue tx,
    • wait for confirmation,
    • only then create/update PDP deal records.
  5. Add persistence for on-chain workflow state (tx hash, confirmation status) so retries/restarts are safe.
  6. Add integration tests for PDP sequencing and failure recovery.

Let me know if you would like me to include any of this in the same PR. @parkan

@anjor anjor marked this pull request as draft February 19, 2026 12:36
@anjor anjor force-pushed the feat/pdp-scheduling-api-boundary branch from e8a7958 to 0d2e70f Compare February 19, 2026 12:38
@anjor anjor marked this pull request as ready for review February 19, 2026 12:41
@anjor anjor requested a review from parkan February 19, 2026 12:41
Copy link
Collaborator

@parkan parkan left a comment

Choose a reason for hiding this comment

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

overall lgtm, minor nit on gas check, maybe add a comment as to why the deal type is resolved at runtime

@anjor anjor enabled auto-merge February 20, 2026 12:54
@anjor anjor merged commit b2a0167 into main Feb 20, 2026
1 check passed
@anjor anjor deleted the feat/pdp-scheduling-api-boundary branch February 20, 2026 13:01
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.

2 participants