Skip to content

docs(spec): align scheme_exact_algo with v2 amount and verification#1879

Open
temptemp3 wants to merge 2 commits intocoinbase:mainfrom
temptemp3:docs/scheme-exact-algo-v2-verify
Open

docs(spec): align scheme_exact_algo with v2 amount and verification#1879
temptemp3 wants to merge 2 commits intocoinbase:mainfrom
temptemp3:docs/scheme-exact-algo-v2-verify

Conversation

@temptemp3
Copy link
Copy Markdown

Summary

Updates specs/schemes/exact/scheme_exact_algo.md for Coinbase review alignment (x402 v2):

  • Use amount consistently (not maxAmountRequired).
  • Document verification order: cross-validation (x402Version, scheme, network, accepted vs 402), genesis hash (gh) vs CAIP-2, asset amount/receiver/asset ID, fee-payer checks, simulate.
  • Point machine-readable errors at @x402/avm VerifyErrorReason.
  • Settlement: confirm transaction committed before reporting success.

Docs-only.

Made with Cursor

@cb-heimdall
Copy link
Copy Markdown

cb-heimdall commented Mar 30, 2026

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 30, 2026

@temptemp3 is attempting to deploy a commit to the Coinbase Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions bot added the specs Spec changes or additions label Mar 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

specs Spec changes or additions

Development

Successfully merging this pull request may close these issues.

2 participants