Skip to content

Update protocol and contracts docs to match current state#14

Merged
A1igator merged 1 commit into
mainfrom
A1igator/docs-update-protocol-contracts
Mar 19, 2026
Merged

Update protocol and contracts docs to match current state#14
A1igator merged 1 commit into
mainfrom
A1igator/docs-update-protocol-contracts

Conversation

@A1igator
Copy link
Copy Markdown
Contributor

Summary

  • Escrow scheme spec rewritten to align with coinbase/x402 PR #1425 and issue #1011 — adds settlement methods (authorize/charge), nonce derivation, 11-step verification logic, settlement logic, error codes, PaymentInfo struct with all fields, and expiry ordering
  • All contract addresses updated to unified CREATE3 addresses (same on every chain)
  • Missing contracts added: ArbiterRegistry, RefundRequestEvidence, ReceiverRefundCollector, SignatureCondition/SignatureRefundRequest factories, and all condition/combinator factories
  • Supported chains expanded from 2 (Base + Base Sepolia) to 11 chains
  • PaymentInfo struct fixed in architecture docs (was missing preApprovalExpiry, refundExpiry, salt, had wrong field ordering)
  • Payment flow diagrams fixed to route through operator (not escrow directly)
  • Roadmap updated with escrow scheme spec submission status and multi-chain deployment

Test plan

  • Preview with npx mint dev and verify all pages render
  • Verify all contract addresses match @x402r/core config
  • Verify escrow scheme spec matches PR #1425 content
  • Check all internal links work

🤖 Generated with Claude Code

@A1igator A1igator force-pushed the A1igator/docs-update-protocol-contracts branch from deac253 to 9c3881a Compare March 19, 2026 07:42
Comment thread contracts/core-contracts.mdx Outdated
Comment thread contracts/overview.mdx

<Steps>
<Step title="Authorize">
Lock funds in escrow at request time. Client signs ERC-3009 authorization, facilitator calls `escrow.authorize()` to pull tokens into escrow contract.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

improve wording, um dashes

@A1igator A1igator force-pushed the A1igator/docs-update-protocol-contracts branch 2 times, most recently from 345b23b to e729a1a Compare March 19, 2026 07:46
Comment thread contracts/core-contracts.mdx Outdated

**Use case:** Enable clients and operators to discover and verify arbiters on-chain.

### ERC3009PaymentCollector
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should we break down this file into multiple section/pages. Just suggestion

@A1igator A1igator force-pushed the A1igator/docs-update-protocol-contracts branch 2 times, most recently from 854a7f0 to f59d467 Compare March 19, 2026 07:57
- Rewrite escrow scheme spec to align with x402 PR #1425 and issue #1011
  (settlement methods, nonce derivation, verification/settlement logic,
  error codes, PaymentInfo struct, expiry ordering)
- Update all contract addresses to unified CREATE3 (same on every chain)
- Add missing contracts: ArbiterRegistry, RefundRequestEvidence,
  ReceiverRefundCollector, SignatureCondition, SignatureRefundRequest
- Add all 11 supported chains to index page
- Fix PaymentInfo struct in architecture docs (was missing fields)
- Fix payment flow diagrams to route through operator (not escrow directly)
- Update factories with CREATE3 addresses and additional factory list
- Update roadmap with escrow scheme spec submission status
- Fix comparison hybrid example with correct escrow extra fields

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
@A1igator A1igator force-pushed the A1igator/docs-update-protocol-contracts branch from f59d467 to 04097ba Compare March 19, 2026 08:00
@A1igator A1igator merged commit 448483c into main Mar 19, 2026
1 check passed
@A1igator A1igator deleted the A1igator/docs-update-protocol-contracts branch March 19, 2026 08:05
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