Skip to content

feat(pattern): add Atomic DvP Settlement pattern#56

Merged
oskarth merged 5 commits intomasterfrom
pr-009-dvp-atomic-settlement
Jan 20, 2026
Merged

feat(pattern): add Atomic DvP Settlement pattern#56
oskarth merged 5 commits intomasterfrom
pr-009-dvp-atomic-settlement

Conversation

@oskarth
Copy link
Copy Markdown
Collaborator

@oskarth oskarth commented Jan 16, 2026

What are you adding?

  • Vendor/Protocol
  • Enterprise Use Case
  • Update to existing content
  • Other

Description

Adds the Atomic DvP Settlement pattern documenting core payment locking and escrow mechanisms for delivery-versus-payment settlement:

  • New pattern: patterns/pattern-delivery-vs-payment-atomic.md
  • Documents HTLC, dual-approval escrow, and time-locked escrow mechanisms
  • Includes protocol steps, guarantees, trade-offs, and example flows
  • Updates CLAUDE.md with enhanced pattern template documentation
  • Marks PR-008 and PR-009 as complete in PRD

Checklist

  • I've checked this doesn't duplicate existing content
  • All links work
  • Info is accurate

@oskarth oskarth force-pushed the pr-009-dvp-atomic-settlement branch 2 times, most recently from 32f3de2 to 2d799ea Compare January 16, 2026 05:31
@oskarth oskarth marked this pull request as ready for review January 16, 2026 05:48
@oskarth oskarth requested review from Meyanis95 and mojalil January 16, 2026 05:50
@@ -0,0 +1,182 @@
---
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

These already exist pattern-dvp-erc7573.md.
Is this new pattern a replacement or an extension to the previous pattern?

5. **Settle**: If conditions satisfied, escrow atomically transfers asset to buyer, payment to seller.
6. **Timeout**: If conditions not met by deadline, escrow returns assets to original owners.

## Integration with EIP-6123 (Derivatives)
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This could be promoted to /approach.

@oskarth
Copy link
Copy Markdown
Collaborator Author

oskarth commented Jan 19, 2026

Addressed Feedback

@Meyanis95

Promoted to approach - Content documents multiple mechanisms (HTLC, escrow, oracle), which fits approach format better than pattern

Added relationship clarification - Overview now explains this is general atomic settlement, while ERC-7573 pattern handles cross-network specifically

Added persona TLDRs - Business, Technical, Legal summaries per approach template


Ready for re-review.


Settlement risk occurs when one party delivers an asset without receiving payment, or vice versa. This approach documents multiple mechanisms for achieving atomic settlement where both legs complete or neither does, eliminating counterparty risk.

**Relationship to [Atomic DvP via ERC-7573](../patterns/pattern-dvp-erc7573.md)**: The ERC-7573 pattern specifically addresses cross-network coordination using the ERC-7573 standard. This approach is more general, documenting the core atomicity mechanisms (HTLCs, escrow, oracles) that work on single networks or can be composed with ERC-7573 for cross-network scenarios.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

ERC-7573 is still a draft


## Overview

Settlement risk occurs when one party delivers an asset without receiving payment, or vice versa. This approach documents multiple mechanisms for achieving atomic settlement where both legs complete or neither does, eliminating counterparty risk.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Should distinguish:

  • Single network atomicity, blockchain core property vs. tradfi
  • Cross network trustless atomicity (L2<->L2) is not solved yet

@Meyanis95
Copy link
Copy Markdown
Collaborator

The content of the approach doc is good, but it just lacks context about the core issue.
Also, LLM tends to heavily index ERC-7573, while it's still a draft and not a proven solution for cross-network atomicity.

@oskarth
Copy link
Copy Markdown
Collaborator Author

oskarth commented Jan 20, 2026

Addressed Feedback (Round 2)

@Meyanis95

ERC-7573 draft status - Added "(draft; requires trusted oracle)" in overview and "(draft)" notation in recommended components

Single vs cross-network atomicity - Added dedicated paragraph distinguishing:

  • Single-network: inherent blockchain property, straightforward
  • Cross-network (L2↔L2): trustless atomicity is an open research problem
  • Current solutions require trusted intermediaries/oracles

Cross-network trust trade-off - Added explicit trade-off: "trustless cross-network DvP remains an unsolved problem"


Ready for re-review.

@Meyanis95
Copy link
Copy Markdown
Collaborator

Looks great!

Addresses reviewer feedback:
- Promotes pattern to approach format (better fits comprehensive content)
- Adds relationship clarification to ERC-7573 pattern
- Adds persona TLDRs per approach template
- Updates CHANGELOG and PRD references
@oskarth oskarth force-pushed the pr-009-dvp-atomic-settlement branch from ecdae80 to 6c67b00 Compare January 20, 2026 08:49
@oskarth oskarth merged commit aca38c0 into master Jan 20, 2026
4 checks passed
@oskarth oskarth deleted the pr-009-dvp-atomic-settlement branch January 20, 2026 08:49
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