Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Deploying x402-exec with
|
| Latest commit: |
7c6e3e5
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://d50d52c2.x402-exec.pages.dev |
| Branch Preview URL: | https://v1-cleanup.x402-exec.pages.dev |
There was a problem hiding this comment.
Pull request overview
This PR performs a major cleanup by removing deprecated v1 dependencies and updating all references from the deprecated @x402x/core package to the new @x402x/extensions package. The changes align the codebase with the v2 architecture where the core SDK functionality has been reorganized into more appropriately named packages.
- Renamed package references from
@x402x/coreto@x402x/extensionsacross documentation - Removed deprecated x402 git submodules (
deps/x402anddeps/x402_upstream_main) - Updated API type definitions to reflect v2 requirements
Reviewed changes
Copilot reviewed 9 out of 15 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| web/frontend/src/content/docs/reference-networks.mdx | Updated package reference from @x402x/core to @x402x/extensions for getNetworkConfig |
| web/frontend/src/content/docs/contracts-hooks.mdx | Updated import statement from @x402x/core to @x402x/extensions for TransferHook |
| web/frontend/public/llms.txt | Updated package references throughout the documentation from @x402x/core to @x402x/extensions, removed outdated install instruction |
| typescript/README.md | Complete rewrite of README to reflect v2 architecture with new package structure (@x402x/extensions, @x402x/client, @x402x/facilitator-sdk) |
| facilitator/README.md | Updated package references from @x402x/core to @x402x/extensions in comments |
| examples/showcase/client/src/hooks/usePayment.ts | Changed x402Version field from optional to required in PaymentResponse interface |
| deps/x402_upstream_main | Removed deprecated submodule |
| deps/x402 | Removed deprecated submodule |
| .gitmodules | Removed references to deprecated x402 submodules |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| accepts?: PaymentRequirements[]; | ||
| error?: string; | ||
| x402Version?: number; // v2-only: must be 2 | ||
| x402Version: number; // v2-only: must be 2 |
There was a problem hiding this comment.
The x402Version field should remain optional (using ?) rather than being made required. Making it required is a breaking change that could cause issues with existing code that doesn't include this field in the response. If you need to ensure v2 responses, consider handling this validation in the consuming code rather than at the type level.
| x402Version: number; // v2-only: must be 2 | |
| x402Version?: number; // v2-only: must be 2 |
No description provided.