Repository navigation
Conversation
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Automations to automatically generate PRs for you. |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
courtney918
left a comment
There was a problem hiding this comment.
- Visa needs
merchant_country: step 2 inkernel.mdxlists merchant, amount, and currency, but Visa card requests are rejected withoutspec.merchant_country. Worth calling out in step 2 or the Visa table row. - No code examples in
kernel.mdx:AGENTS.mdasks for TS + Python, and the link/agentcard guides have CLI commands. A minimal create-wallet / create-card snippet would help. - Diagram arrow:
C -->|fill| Tpoints from the card to the trusted controller, which implies card data reaches the controller. Pointing bothCandTintoF(as before) is more accurate.
courtney918
left a comment
There was a problem hiding this comment.
Optional / non-blocking:
- Overview comparison: the new table drops test vs live mode (link is live-only, agentcard has
test_mode), which affects how people build. Might be worth keeping that one detail. vaults/overview.mdx: the "Payment items" section still only links the link/agentcard item references; could add KERNEL card.- PR body: the Pylon customer issue link probably shouldn't be in a public repo's PR description.
Visa card requests are rejected without spec.merchant_country, so the guide now says so in the purchase steps and approval table. Add TypeScript, Python, and CLI examples for creating the wallet and authorizing a card. The wallet chooser notes each provider's environment, and the vaults overview links the KERNEL card guide. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
On the diagram arrow: the 1, 2, 4, 5 and the PR body are addressed in ff853b7. Thanks for catching the Visa country requirement. |
Description
The wallet chooser omits the Kernel card provider, leaving developers to piece together enrollment, card custody, and purchase approvals from the API reference and embedded capture guide.
Add a short provider guide, a three-row provider chooser, and links from payment setup and embedded capture. The guide uses numbered steps and an approval table, with fallback behavior behind an expandable section. Detailed limits and expiry values stay in the API reference. Note that Visa purchases require
merchant_country, and show each provider's environment in the chooser. Also clarify cross-origin payment-field support and link the existing pricing answer. Lay out the payments diagram vertically with shared setup and both checkout branches visible together.Testing
bunx mint@4.2.930 broken-links— no broken links.bunx mint@4.2.930 dev --port 3000— all five affected pages return HTTP 200 without MDX compilation errors.git diff --checkpasses.Documentation only; no application behavior changed. The KERNEL card guide adds TypeScript, Python, and CLI examples, checked against the API and CLI.
Note
Low Risk
Documentation-only updates to MDX and site navigation; no product code or API behavior changes.
Overview
Adds a dedicated KERNEL card provider guide covering
provider: "kernel"enrollment (hosted or embedded), purchase authorization, network-token vs fallback behavior, and TypeScript/Python/CLI examples. The new page is wired into the Payments nav and linked from payments setup, vault overview, embedded card capture, and the wallets overview.The wallets overview drops the two-card chooser and long Link vs Agentcard comparison in favor of a compact three-integration table (KERNEL card, Link, Agentcard) and points readers at each provider guide.
Payments on KERNEL retools the lifecycle Mermaid diagram to a top-down layout that shows shared wallet setup plus separate
filland Agentcard checkout branches, and adds Get started links for KERNEL card and pricing.vaults/fillnow states that selectors can target inputs in cross-origin payment iframes, without implying processor acceptance.Reviewed by Cursor Bugbot for commit ff853b7. Bugbot is set up for automated code reviews on this repo. Configure here.