Add Solana facilitator guide to docs #8511
Conversation
Introduces a new MDX guide for Solana facilitator integration under x402, detailing setup and API usage. Updates the sidebar to include a link to the new Solana guide for easier navigation.
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
4 Skipped Deployments
|
github-actions
bot
added
the
Portal
|
Caution Review failedThe pull request is closed. WalkthroughAdds a new MDX documentation page describing Solana Facilitator support for x402 payments and a sidebar entry linking to it; the page includes exported metadata, supported networks, endpoints, and an end-to-end integration guide with examples. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes
Possibly related PRs
Pre-merge checks and finishing touches❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro Disabled knowledge base sources:
📒 Files selected for processing (1)
Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8511 +/- ##
=======================================
Coverage 54.65% 54.65%
=======================================
Files 920 920
Lines 61114 61114
Branches 4145 4145
=======================================
Hits 33399 33399
Misses 27613 27613
Partials 102 102
🚀 New features to boost your workflow:
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (3)
apps/portal/src/app/x402/facilitator/solana/page.mdx (3)
30-50: Call out thatx-secret-keymust stay server-sideThe examples correctly show server-side usage, but it’s easy for readers to paste this into client code. I’d recommend adding a short note right after this first snippet that
x-secret-keymust never be exposed in browser/mobile clients and should only be used from trusted server environments.
114-147: Make the final step heading consistent with prior numbered stepsYou use numbered headings for steps 1–4, then switch to
### Fetch protected resourceswithout a step number. For consistency and skim-ability, consider renaming this to something like### 5. Fetch protected resources.
68-76: Clarify which Solana cluster thenetwork/chainIdexample targetsThe
networkandchainIdvalues ("solana:8E9rvCKLFQia2Y35HXjjpWzj8weVo44K") look like a specific Solana cluster identifier, but it’s not obvious to readers whether this is devnet, mainnet, or just an example. It may help to add a brief comment or note indicating the intended cluster and how to choose the correct value for other environments.Also applies to: 155-156
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
Disabled knowledge base sources:
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (2)
apps/portal/src/app/x402/facilitator/solana/page.mdx(1 hunks)apps/portal/src/app/x402/sidebar.tsx(1 hunks)
🧰 Additional context used
📓 Path-based instructions (3)
**/*.{ts,tsx}
📄 CodeRabbit inference engine (CLAUDE.md)
**/*.{ts,tsx}: Write idiomatic TypeScript with explicit function declarations and return types
Limit each TypeScript file to one stateless, single-responsibility function for clarity
Re-use shared types from@/typesor localtypes.tsbarrels
Prefer type aliases over interface except for nominal shapes in TypeScript
Avoidanyandunknownin TypeScript unless unavoidable; narrow generics when possible
Choose composition over inheritance; leverage utility types (Partial,Pick, etc.) in TypeScript
**/*.{ts,tsx}: Write idiomatic TypeScript with explicit function declarations and return types
Limit each file to one stateless, single-responsibility function for clarity and testability
Re-use shared types from @/types or local types.ts barrel exports
Prefer type aliases over interface except for nominal shapes
Avoid any and unknown unless unavoidable; narrow generics whenever possible
Choose composition over inheritance; leverage utility types (Partial, Pick, etc.)
Comment only ambiguous logic in TypeScript files; avoid restating TypeScript types and signatures in prose
Files:
apps/portal/src/app/x402/sidebar.tsx
**/*.{js,jsx,ts,tsx,json}
📄 CodeRabbit inference engine (AGENTS.md)
Biome governs formatting and linting; its rules live in biome.json. Run
pnpm fix&pnpm lintbefore committing, ensure there are no linting errors
Files:
apps/portal/src/app/x402/sidebar.tsx
**/*.{ts,tsx,js,jsx}
📄 CodeRabbit inference engine (AGENTS.md)
Lazy-import optional features; avoid top-level side-effects
Files:
apps/portal/src/app/x402/sidebar.tsx
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (5)
- GitHub Check: E2E Tests (pnpm, esbuild)
- GitHub Check: Size
- GitHub Check: Lint Packages
- GitHub Check: Vercel Agent Review
- GitHub Check: Analyze (javascript)
🔇 Additional comments (1)
apps/portal/src/app/x402/sidebar.tsx (1)
40-43: Solana guide link wiring looks correctThe href matches the new page route (
/x402/facilitator/solana), and the label fits well under the existing “Facilitator” guide group. No changes needed.
size-limit report 📦
|
Introduces a new MDX guide for Solana facilitator integration under x402, detailing setup and API usage. Updates the sidebar to include a link to the new Solana guide for easier navigation.
PR-Codex overview
This PR introduces support for the
Solananetwork in thex402payments system, adding new functionality and documentation to facilitate transactions through aSolana facilitator wallet.Detailed summary
Solanain the sidebar ofx402.page.mdxforSolana Facilitatorwith:Solanasupport for payments.Solana facilitator wallet.Summary by CodeRabbit
Documentation
New Features
✏️ Tip: You can customize this high-level summary in your review settings.