Repository navigation
InsufficientBalanceError for 403 INSUFFICIENT_BALANCE - #47
Merged
Merged
Conversation
added 4 commits
October 19, 2026 11:13
…equest only asks to top up
This was referenced Oct 19, 2026
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
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.
Part 1 of #45, split so the behavior that is already live can ship now.
The SDK raises
InsufficientBalanceError(withrequested_records,max_affordable_recordsin the request's own size unit) for a 403INSUFFICIENT_BALANCE, and the CLI prints it. The refusal shipped with discolike-platform !994, which is on platformmain.Commits are #45's four balance commits, cherry-picked onto current
developmentunchanged. ruff clean, 942 tests pass; ty reports the same 4 diagnostics asdevelopment.Docs: discolike-api-docs !49. Part 2 (free plan, no-credentials 401 object) is stacked on this branch.


Confidence Score: 5/5
This PR appears safe to merge.Summary
Adds
InsufficientBalanceErrorfor 403 responses carryingINSUFFICIENT_BALANCE.No actionable issues found. Review checks were based on code inspection; tests were not run.
Reviews (1) · Last reviewed commit: "docs(sdk): max_affordable_records is 0 w..." · Reviewed by Greptile