docs(skill): refresh the grid-api skill for the current API spec - #707
Closed
ls-bolt[bot] wants to merge 1 commit into
Closed
docs(skill): refresh the grid-api skill for the current API spec#707ls-bolt[bot] wants to merge 1 commit into
ls-bolt[bot] wants to merge 1 commit into
Conversation
The skill drives the API with curl (not the CLI), so this brings its details
current with the spec:
- account-types.md: remove the bogus paymentRails create field (it's response-
only) from all bodies; fix swiftBic->swiftCode, GBP sortCode, XOF countries->
region, UGX/MWK phone patterns, EUR/AED required beneficiary fields, SGD
bankName optional; add INR NEFT/RTGS path, platformAccountId/
defaultUmaDepositAccount, 11 missing account types, and beneficiary optionals.
- SKILL.md/workflows.md: drop currency from ACCOUNT destinations (UMA-only);
fix kyc-link (GET query -> POST /customers/{id}/kyc-link with body); remove
invalid PENDING_APPROVAL status filter; receiver id -> lookupId; add top-level
purposeOfPayment/remittanceInformation, cryptoNetwork for stablecoin funding,
transfer-out remittanceInformation/paymentRail, a BUSINESS customer example,
and an SCA/authorize (PENDING_AUTHORIZATION) subsection.
- endpoints.md: fix 2 stale entries, add ~68 missing operations across 12 new
domains (agents, auth, sca, cards, tokens, documents, verifications, …), and
complete the response-codes table.
|
The latest updates on your projects. Learn more about Vercel for GitHub. 2 Skipped Deployments
|
jklein24
added a commit
that referenced
this pull request
Jul 20, 2026
## Summary
Brings the bundled `grid-api` Claude skill (`.claude/skills/grid-api/`)
up to date with the current `2025-10-13` spec. The skill drives the API
with `curl` (not the CLI), so this is a spec-freshness pass — audited
all four files against `openapi/` and fixed the drift.
## What was stale / fixed
**account-types.md**
- Removed the bogus `paymentRails` field from every create body — it's a
**response-only** attribute, not a create input (was in all ~29 curl
examples + every "required fields" list).
- `swiftBic` → `swiftCode`; GBP `sortCode` `12-34-56` → `123456`
(`^[0-9]{6}$`); XOF `countries` → `region` (BJ/CI/SN/TG); UGX/MWK phone
patterns → generic; EUR/AED beneficiary required fields
(`countryOfResidence`/`address`); SGD `bankName` optional.
- Added the INR NEFT/RTGS bank-transfer path, top-level
`platformAccountId`/`defaultUmaDepositAccount`, 11 missing account types
(BDT/CNY/COP/EGP/GHS/GTQ/HTG/JMD/PKR/SLV/SWIFT), and beneficiary
optional fields.
**SKILL.md / workflows.md**
- Dropped `currency` from ACCOUNT destinations (it's UMA-only; ACCOUNT
uses `paymentRail`).
- Fixed kyc-link (was `GET /customers/kyc-link?query`; now `POST
/customers/{customerId}/kyc-link` with a `{redirectUri}` body).
- Removed the invalid `PENDING_APPROVAL` transaction-status filter;
receiver-lookup `id` → `lookupId` (+ enumerated
`supportedCurrencies`/`requiredPayerDataFields`).
- Added top-level `purposeOfPayment`/`remittanceInformation`,
`cryptoNetwork` for stablecoin realtime funding, transfer-out
`remittanceInformation`/`paymentRail`, a BUSINESS customer create
example (required `legalName`/`taxId`/`incorporatedOn`), and an SCA /
`PENDING_AUTHORIZATION` / authorize subsection.
**endpoints.md**
- Fixed the 2 stale entries (`GET /quotes` list removed; kyc-link
GET→POST), added ~68 missing operations across 12 newer domains (agents,
auth, sca, cards, tokens, documents, verifications, discoveries,
beneficial-owners, stablecoins, internal-accounts update/export, …), and
completed the response-codes table (202/204/403/405/410/423/429).
Auth, base URL, and the discriminated source/destination shapes were
already current — unchanged.
## Test plan
Documentation-only. Verified every fix against
`openapi/components/schemas/**` and `openapi/paths/**`; confirmed all
curl JSON bodies still parse and no stale tokens remain
(`paymentRails`/`swiftBic`/`countries`).
Original PR: #707
Co-authored-by: Jeremy <jeremy@lightspark.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Brings the bundled
grid-apiClaude skill (.claude/skills/grid-api/) up to date with the current2025-10-13spec. The skill drives the API withcurl(not the CLI), so this is a spec-freshness pass — audited all four files againstopenapi/and fixed the drift.What was stale / fixed
account-types.md
paymentRailsfield from every create body — it's a response-only attribute, not a create input (was in all ~29 curl examples + every "required fields" list).swiftBic→swiftCode; GBPsortCode12-34-56→123456(^[0-9]{6}$); XOFcountries→region(BJ/CI/SN/TG); UGX/MWK phone patterns → generic; EUR/AED beneficiary required fields (countryOfResidence/address); SGDbankNameoptional.platformAccountId/defaultUmaDepositAccount, 11 missing account types (BDT/CNY/COP/EGP/GHS/GTQ/HTG/JMD/PKR/SLV/SWIFT), and beneficiary optional fields.SKILL.md / workflows.md
currencyfrom ACCOUNT destinations (it's UMA-only; ACCOUNT usespaymentRail).GET /customers/kyc-link?query; nowPOST /customers/{customerId}/kyc-linkwith a{redirectUri}body).PENDING_APPROVALtransaction-status filter; receiver-lookupid→lookupId(+ enumeratedsupportedCurrencies/requiredPayerDataFields).purposeOfPayment/remittanceInformation,cryptoNetworkfor stablecoin realtime funding, transfer-outremittanceInformation/paymentRail, a BUSINESS customer create example (requiredlegalName/taxId/incorporatedOn), and an SCA /PENDING_AUTHORIZATION/ authorize subsection.endpoints.md
GET /quoteslist removed; kyc-link GET→POST), added ~68 missing operations across 12 newer domains (agents, auth, sca, cards, tokens, documents, verifications, discoveries, beneficial-owners, stablecoins, internal-accounts update/export, …), and completed the response-codes table (202/204/403/405/410/423/429).Auth, base URL, and the discriminated source/destination shapes were already current — unchanged.
Test plan
Documentation-only. Verified every fix against
openapi/components/schemas/**andopenapi/paths/**; confirmed all curl JSON bodies still parse and no stale tokens remain (paymentRails/swiftBic/countries).