Skip to content

Add DDO on-chain contract adapter#643

Merged
parkan merged 5 commits intodata-preservation-programs:feat/ddo-deal-typefrom
anjor:fix/ddo-contract-adapter
Mar 12, 2026
Merged

Add DDO on-chain contract adapter#643
parkan merged 5 commits intodata-preservation-programs:feat/ddo-deal-typefrom
anjor:fix/ddo-contract-adapter

Conversation

@anjor
Copy link
Collaborator

@anjor anjor commented Mar 11, 2026

Summary

  • add a local DDO scheduling/tracking interface and concrete on-chain adapter backed by ddo-client
  • implement piece-granular payment setup, tx confirmation-depth polling, receipt event parsing, and allocation status reads
  • add focused adapter tests and record the new ddo-client dependency

Testing

  • go test ./service/dealpusher

Notes

  • upstream ddo-client still requires a raw private key for write clients, so this adapter currently enforces signer/address matching at runtime until signer-based constructors exist

@anjor anjor requested a review from parkan March 11, 2026 11:39
//
// caveat: ddo-client (pinned at 2fff1a5b168a) overwrites auth.Context with
// context.Background() in its write methods, so ctx cancellation does not
// propagate to in-flight EnsurePayments/CreateAllocations calls.
Copy link
Collaborator

Choose a reason for hiding this comment

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

we will fix this upstream

@parkan parkan changed the base branch from main to feat/ddo-deal-type March 12, 2026 17:18
@parkan parkan force-pushed the fix/ddo-contract-adapter branch from f947bde to eea1c84 Compare March 12, 2026 17:18
@parkan parkan merged commit 9f7182d into data-preservation-programs:feat/ddo-deal-type Mar 12, 2026
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