Skip to content

docs(sandbox): correct how a rule-based sweep reports an uncarriable amount - #1147

Draft
ls-bolt[bot] wants to merge 1 commit into
mainfrom
10-09-sandbox-sweep-amount-prose
Draft

ls-bolt[bot] wants to merge 1 commit into
mainfrom
10-09-sandbox-sweep-amount-prose

Conversation

@ls-bolt

@ls-bolt ls-bolt Bot commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

Summary

Documentation only. Three descriptions described a SWEEP_AMOUNT_OUT_OF_RANGE failure in terms that no longer hold.

Sandbox funding. POST /sandbox/internal-accounts/{accountId}/fund told integrators to "fund a rule-based account with a very small amount to rehearse that path". A small deposit no longer reproduces that failure, so the instruction sent people down a path they cannot follow. The surrounding description of what happens to the balance is unchanged: a sweep the corridor cannot carry still fails, and the funds still move to the account holder's own account in the same currency.

The two failure-reason descriptions. IncomingTransactionFailureReason and OutgoingTransactionFailureReason both said the rule's minimumAmount and maximumAmount "give the current band" for this reason. A returned SweepRule does not populate either field, so that pointed readers at values they would not find. Both now describe the corridor's own range without naming fields.

No enum member is added, removed, or renamed, and no behaviour changes.

Changes: 5 files

  • openapi/paths/sandbox/sandbox_internal_accounts_{accountId}_fund.yaml — drop the rehearsal instruction
  • openapi/components/schemas/transactions/IncomingTransactionFailureReason.yaml — drop the minimumAmount/maximumAmount citation
  • openapi/components/schemas/transactions/OutgoingTransactionFailureReason.yaml — same, in the reason table
  • openapi.yaml, mintlify/openapi.yaml — regenerated by make build

Verification

  • make build rebundles; the two generated copies are byte-identical to each other.
  • Parsed the bundle before and after: 721 schemas and 137 paths both times, and both failure-reason enums have exactly the same members in the same order.
  • make lint-openapi passes with 0 errors. The 909 pre-existing warnings and infos are unrelated to these files; none names a changed schema or path.

🤖 Generated with Claude Code

Requested by @bsiaotickchong

@mintlify

mintlify Bot commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

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

Project Status Preview Updated
Grid 🟢 Ready View Preview Oct 9, 2026, 1:04 AM

@vercel

vercel Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

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

3 Skipped Deployments
Project Deployment Actions Updated
grid-cards-demo Ignored Ignored Preview Oct 9, 2026 1:02am UTC
grid-flow-builder Ignored Ignored Preview Oct 9, 2026 1:02am UTC
grid-wallet-demo Ignored Ignored Preview Oct 9, 2026 1:02am UTC

Request Review

@ls-bolt ls-bolt Bot added the bolt label Oct 9, 2026
…amount

The sandbox fund endpoint told integrators to fund a rule-based account with a
very small amount to rehearse a SWEEP_AMOUNT_OUT_OF_RANGE failure. A small
deposit no longer takes that path, so the rehearsal does not reproduce it.

The two failure-reason descriptions cited the sweep rule's minimumAmount and
maximumAmount as the band that decides. Neither field is populated on a
returned rule, so the reasons now describe the corridor's own range.

Co-Authored-By: bsiaotickchong <bsiaotickchong@users.noreply.github.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@ls-bolt
ls-bolt Bot force-pushed the 10-09-sandbox-sweep-amount-prose branch from 1f8da8c to 6907a78 Compare October 9, 2026 01:02
@ls-bolt

ls-bolt Bot commented Oct 9, 2026

Copy link
Copy Markdown
Contributor Author

@greptile review

@github-actions github-actions Bot added the breaking-change Introduces a breaking change to the OpenAPI spec label Oct 9, 2026

Copy link
Copy Markdown

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

@github-actions github-actions Bot removed the breaking-change Introduces a breaking change to the OpenAPI spec label Oct 9, 2026
@greptile-apps

greptile-apps Bot commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

[Low impact] This documentation-only PR appears safe to merge.

Summary

This PR corrects three descriptions of failed rule-based sweeps.

  • Removes the sandbox instruction to trigger a failure with a tiny deposit.
  • Removes claims that minimumAmount and maximumAmount provide the current limits.
  • Updates both generated OpenAPI bundles without changing enum members, fields, or constraints.

No actionable issues were found.

Reviews (1) · Last reviewed commit: "docs(sandbox): correct how a rule-based ..." · Reviewed by Greptile

This branch was successfully deployed

1 active deployment
staging - mintlify — 6907a780 Deployed Oct 9, 2026 by mintlify[bot]
Sign up for free to 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