Skip to content

Conversation

@jtang17
Copy link

@jtang17 jtang17 commented Jan 2, 2026

Description

Adds the exact payment scheme specification for Aptos blockchain:

Uses Aptos fungible asset transfer (0x1::primary_fungible_store::transfer)
Supports sponsored transactions via extra.sponsored field
Facilitators can implement as direct fee payer or gas station service
Follows x402 v2 protocol with CAIP-2 network identifiers (aptos:1, aptos:2)
Includes security notes on fee payer mechanism

Tests

Checklist

  • I have formatted and linted my code
  • All new and existing tests pass
  • My commits are signed (required for merge) -- you may need to rebase if you initially pushed unsigned commits

Adds the exact payment scheme specification for Aptos blockchain:
- Uses Aptos fungible asset transfer (0x1::primary_fungible_store::transfer)
- Supports sponsored transactions via extra.sponsored field
- Facilitators can implement as direct fee payer or gas station service
- Follows x402 v2 protocol with CAIP-2 network identifiers (aptos:1, aptos:2)
- Includes security notes on fee payer mechanism
- Replace gasStation URL with simple sponsored: true boolean in extra
- Clarify facilitator can use direct fee payer or gas station service internally
- Add security note about fee payer mechanism
- Link to Aptos sponsored transactions documentation
- Remove version field from PAYMENT-RESPONSE (consistent with other chains)
- Separate sponsored vs non-sponsored flows in appendix
@cb-heimdall
Copy link

cb-heimdall commented Jan 2, 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 1
Sum 2

@vercel
Copy link

vercel bot commented Jan 2, 2026

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

A member of the Team first needs to authorize it.

@phdargen
Copy link
Contributor

phdargen commented Jan 2, 2026

This is a continuation of #640 @erikreppel-cb @jillxuu

Not sure what happened there, seems when @jtang17 rebased your own commits got lost and the PR was then auto-closed

@phdargen
Copy link
Contributor

phdargen commented Jan 2, 2026

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants