Skip to content

docs: declare the 429 daily-volume response on /sandbox/send - #866

Draft
ls-bolt[bot] wants to merge 1 commit into
mainfrom
08-25-sandbox-send-429-daily-volume
Draft

docs: declare the 429 daily-volume response on /sandbox/send#866
ls-bolt[bot] wants to merge 1 commit into
mainfrom
08-25-sandbox-send-429-daily-volume

Conversation

@ls-bolt

@ls-boltls-boltBot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Summary

/sandbox/send can refuse a request with DAILY_VOLUME_LIMIT_EXCEEDED when funding a quote would take the customer past the daily withdrawal limit, but its response list did not declare a 429. Generated clients therefore had no typed error for a status the endpoint really returns.

  • Declare the 429 / Error429 response on /sandbox/send.
  • Widen the /quotes429 description: the daily limit is checked when a quote is created, not only when immediatelyExecute is set.

Edited in openapi/ (the source) and rebundled with make build, so the generated openapi.yaml and mintlify/openapi.yaml stay in sync.

No enum changes — Error429 already carries DAILY_VOLUME_LIMIT_EXCEEDED, and Error400 already carries TRANSACTION_SIZE_LIMIT_EXCEEDED. This only documents the responses on the endpoints that emit them, so /transfer-out, /quotes, /quotes/{quoteId}/execute, and /sandbox/send now all declare the 429.

Compatibility

Additive. A newly documented error response on an endpoint that already returns it — no request or success-response shape changes, and no existing field is removed or renamed.

Test plan

  • npm run build:openapi re-run; git diff --exit-code openapi.yaml mintlify/openapi.yaml is clean, which is the check CI runs
  • npx spectral lint openapi.yaml --fail-severity=error — passes (exit 0)
  • npx @redocly/cli lint openapi/openapi.yaml — 7 errors, all pre-existing in the card-simulation paths; identical count on a clean main
  • Verified all four endpoints that can emit the limit now declare 429 with the Error429 schema

@vercel

vercelBot commented Aug 25, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
ProjectDeploymentActionsUpdated (UTC)
grid-flow-builderIgnoredIgnoredPreviewAug 25, 2026 11:57pm
grid-wallet-demoIgnoredIgnoredPreviewAug 25, 2026 11:57pm

Request Review

@ls-boltls-boltBot added the bolt label Aug 25, 2026
@lightspark-botGraphite App

Copy link
Copy Markdown

This stack of pull requests is managed by Graphite. Learn more about stacking.

@github-actions

github-actionsBot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

✱ Stainless preview builds for grid

This PR will update the grid SDKs with the following commit messages.

cli

chore(internal): regenerate SDK with no functional changes

go

chore(internal): regenerate SDK with no functional changes

kotlin

chore(internal): regenerate SDK with no functional changes

openapi

feat(api): add 429 error response to quote funding method

php

chore(internal): regenerate SDK with no functional changes

python

chore(internal): regenerate SDK with no functional changes

ruby

chore(internal): regenerate SDK with no functional changes

typescript

chore(internal): regenerate SDK with no functional changes

Edit this comment to update them. They will appear in their respective SDK's changelogs.

grid-typescriptstudio · code · diff

Your SDK build had at least one "warning" diagnostic, but this did not represent a regression.
generate ⚠️build ⏭️ (prev: build ✅) → lint ⏭️ (prev: lint ❗) → test ✅

grid-openapistudio · code · diff

Your SDK build had at least one "warning" diagnostic, but this did not represent a regression.
generate ⚠️

grid-rubystudio · code · diff

Your SDK build had at least one "warning" diagnostic, but this did not represent a regression.
generate ⚠️build ⏭️ (prev: build ✅) → lint ✅test ✅

grid-gostudio · code · diff

Your SDK build had at least one "warning" diagnostic, but this did not represent a regression.
generate ⚠️build ⏭️ (prev: build ✅) → lint ❗test ❗

go get github.com/stainless-sdks/grid-go@91ab1d69fbe33549f6f11a7a693caab048394470
grid-kotlinstudio · code · diff

Your SDK build had at least one "warning" diagnostic, but this did not represent a regression.
generate ⚠️build ⏭️ (prev: build ✅) → lint ⏭️ (prev: lint ✅) → test ❗

grid-pythonstudio · code · diff

Your SDK build had at least one "warning" diagnostic, but this did not represent a regression.
generate ⚠️build ⏭️ (prev: build ✅) → lint ⏭️ (prev: lint ❗) → test ❗

grid-phpstudio · code · diff

Your SDK build had at least one "warning" diagnostic, but this did not represent a regression.
generate ⚠️lint ✅test ✅

grid-clistudio · code · diff

Your SDK build had at least one "warning" diagnostic, but this did not represent a regression.
generate ⚠️build ❗lint ❗test ❗


This comment is auto-generated by GitHub Actions and is automatically kept up to date as you push.
If you push custom code to the preview branch, re-run this workflow to update the comment.
Last updated: 2026-08-26 00:05:20 UTC

/sandbox/send can refuse with DAILY_VOLUME_LIMIT_EXCEEDED when funding a
quote would take the customer past the daily withdrawal limit, but its
response list did not document a 429, so generated clients had no typed
error for it.
Also widen the /quotes 429 description: the limit is checked when a quote
is created, not only when immediatelyExecute is set.
Co-Authored-By: jacklatourette <jacklatourette@users.noreply.github.com>
@ls-bolt
ls-boltBotforce-pushed the 08-25-sandbox-send-429-daily-volume branch from b1a602c to c378f97CompareAugust 26, 2026 00:00
@mintlify

mintlifyBot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

ProjectStatusPreviewUpdated (UTC)
Grid🟢 ReadyView PreviewAug 26, 2026, 12:02 AM

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@lightspark-bot@jacklatourette