Skip to content

Add native USDT wallet valuation to read-only account facts - #334

Merged
Pigbibi merged 2 commits into
mainfrom
codex/binance-wallet-total-20261003
Oct 2, 2026
Merged

Pigbibi merged 2 commits into
mainfrom
codex/binance-wallet-total-20261003

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Oct 2, 2026 •

Copy link
Copy Markdown
Collaborator

Changes

  • Add an optional USDT wallet valuation to the existing read-only account facts.
  • Use one fixed signed wallet-balance GET with a 15-second timeout, no redirects and no retries.
  • Sum each returned wallet row once with exact Decimal arithmetic; retain native Spot/Flexible Earn quantities when valuation is unavailable.
  • Keep existing identity, source and no-order protections unchanged. The valuation covers provider-returned wallet rows and is separate from account NAV or USD totals.

Validation

  • 144 focused synthetic tests passed, including the locked SDK's final request parameters, invalid responses, exact arithmetic and quantity fallback.
  • The full local suite passed: 1,259 tests and 91 subtests. The SDK wire test runs in a clean child process to avoid other suites' module stubs.
  • Ruff, compile checks and diff checks passed.
  • Independent source review passed within this scope.

No real wallet request or production reader adoption is included in this source change.

Pigbibi and others added 2 commits October 3, 2026 04:13
Co-Authored-By: Codex <noreply@openai.com>
Co-Authored-By: Codex <noreply@openai.com>
@Pigbibi
Pigbibi merged commit 2696cce into main Oct 2, 2026
1 check passed
@Pigbibi
Pigbibi deleted the codex/binance-wallet-total-20261003 branch October 2, 2026 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant