Add Arbitrum (ARBITRUM_WALLET) to Grid public API schema - #907
Draft
jklein24 wants to merge 1 commit into
Draft
Conversation
- Add ARBITRUM to CryptoNetwork.yaml enum (between PLASMA and SPARK) - Add ARBITRUM_WALLET to PaymentAccountType.yaml and ExternalAccountType.yaml - Add 3 new schema files: ArbitrumWalletInfo, PaymentArbitrumWalletInfo (USDC+USDT), ArbitrumWalletExternalAccountInfo - Wire discriminator mappings in PaymentInstructions, ExternalAccountCreateInfoOneOf, ExternalAccountInfoOneOf - Update .stainless/stainless.yml with Arbitrum model entries and allOf transform targets - Update docs: crypto-funding.mdx (supported blockchains table), external-accounts.mdx, internal-accounts.mdx - Update yaml examples: RealtimeFundingQuoteSource, EstimateCryptoWithdrawalFeeRequest, crypto_estimate-withdrawal-fee path, external-account-status webhook - Update skill files: SKILL.md and references/account-types.md - Update samples: frontend CreateUsdcExternalAccount + CreateQuote, Kotlin ExternalAccounts (incl. missing import fix), grid-visualizer crypto.ts + account-types.ts - Regenerate openapi.yaml and mintlify/openapi.yaml (npm run build:openapi) Plan ref: lightsparkdev/webdev#34482
ls-bolt
Bot
added
bolt
breaking-change
Contributor
|
Contributor
✱ Stainless preview builds for gridThis PR will update the cli go kotlin openapi php python ruby typescript Edit this comment to update them. They will appear in their respective SDK's changelogs. ✅ grid-typescript studio · code · diff
✅ grid-openapi studio · code · diff
✅ grid-ruby studio · code · diff
✅ grid-kotlin studio · code · diff
✅ grid-go studio · code · diff
✅ grid-python studio · code · diff
✅ grid-php studio · code · diff
✅ grid-cli studio · code · diff
This comment is auto-generated by GitHub Actions and is automatically kept up to date as you push. |
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
Adds
ARBITRUMto theCryptoNetworkenum andARBITRUM_WALLETto the account types, wiring Arbitrum (EVM L2) as a new blockchain supported by the Grid public API. Arbitrum supports both USDC and USDT via Fireblocks, unlike Plasma (USDT-only).Slack thread: https://lightsparkgroup.slack.com/archives/D0AHLSKKV8R/p1788759710100209
Plan PR: https://github.com/lightsparkdev/webdev/pull/34482
Changes
3 new schema files:
openapi/components/schemas/common/ArbitrumWalletInfo.yaml— wallet shape (mirrorsPlasmaWalletInfo)openapi/components/schemas/common/PaymentArbitrumWalletInfo.yaml— payment instruction schema;assetTypeenum is[USDC, USDT](Plasma carries only USDT)openapi/components/schemas/external_accounts/ArbitrumWalletExternalAccountInfo.yaml— external account schema (mirrorsPlasmaWalletExternalAccountInfo)Enum / discriminator updates (6 files):
CryptoNetwork.yaml—ARBITRUMadded betweenPLASMAandSPARKPaymentAccountType.yaml,ExternalAccountType.yaml—ARBITRUM_WALLETadded afterPLASMA_WALLETPaymentInstructions.yaml,ExternalAccountCreateInfoOneOf.yaml,ExternalAccountInfoOneOf.yaml— discriminator refs + mappings wiredSupporting files:
.stainless/stainless.yml—arbitrum_wallet_infomodel ref + allOf strip targets for both Payment and ExternalAccount variantsopenapi/components/schemas/quotes/RealtimeFundingQuoteSource.yaml—ARBITRUMadded tocryptoNetworkexample listopenapi/components/schemas/crypto/EstimateCryptoWithdrawalFeeRequest.yaml— description updatedopenapi/paths/crypto/crypto_estimate-withdrawal-fee.yaml— newestimateArbitrumWithdrawalexampleopenapi/webhooks/external-account-status.yaml—arbitrumWalletActiveexample addedDocs:
mintlify/snippets/sending/crypto-funding.mdx— Plasma + Arbitrum rows in supported-blockchain tablemintlify/snippets/external-accounts.mdx— Arbitrum section (USDC + USDT cURL examples); EVM network note updatedmintlify/snippets/internal-accounts.mdx— Arbitrum Stablecoin Funding accordionSkill files:
.claude/skills/grid-api/SKILL.md— supported networks + realtime-funding crypto list updated.claude/skills/grid-api/references/account-types.md—ARBITRUM_WALLETrow added to tableSamples:
samples/frontend/src/steps/CreateUsdcExternalAccount.tsx—ARBITRUMentry inNETWORK_CONFIGSsamples/frontend/src/steps/CreateQuote.tsx—ARBITRUMinCRYPTO_NETWORKSsamples/kotlin/.../ExternalAccounts.kt—ARBITRUM_WALLETbranch +ArbitrumWalletInfoimportcomponents/grid-visualizer/src/data/crypto.ts— Arbitrum entries for USDC and USDTcomponents/grid-visualizer/src/data/account-types.ts—ARBITRUM_WALLETspecGenerated mirrors (A8):
openapi.yamlandmintlify/openapi.yaml— rebuilt vianpm run build:openapiVerification
npm run build:openapi— ✅ bundle succeedsnpm run lint:openapi— ✅ 0 errors (pre-existing warnings/infos unchanged)ArbitrumWalletInfoimport); no other findingsGenerated model class names (for webdev client regen)
ArbitrumWalletInfoPaymentArbitrumWalletInfoArbitrumWalletExternalAccountInfoThese names are used by the webdev Python client regen (
update_schema.sh) and by the Kotlin SDK builder.Context
assetTypeaccepts bothUSDCandUSDT(unlike Plasma which is USDT-only)Requested by @jklein24
Original PR: #906