Skip to content

fix(schwab): preserve bounded native account facts - #642

Merged
Pigbibi merged 2 commits into
mainfrom
codex/schwab-native-facts-20261001
Oct 1, 2026
Merged

Pigbibi merged 2 commits into
mainfrom
codex/schwab-native-facts-20261001

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

Change

  • Retain optional cashBalance decimal text and securitiesAccount.type from the existing selected-account response.
  • Preserve the current sizing, buying power, equity fallback, digest, requests and retry behavior.
  • Bound optional decimal parsing before fixed-point formatting. Missing or invalid optional facts are omitted.
  • Keep raw account-type tokens without assigning currency or interpreting broker account categories.

Validation

  • 21 Schwab portfolio tests and 72 subtests passed using the existing development environment.
  • Ruff, compileall and diff checks passed.
  • Independent material review passed after bounding decimal formatting.

The consumer will adopt this exact head on its existing dependency baseline. This does not enable trading or establish current broker balances.

Pigbibi and others added 2 commits October 1, 2026 11:19
Co-Authored-By: Codex <noreply@openai.com>
…acts-20261001

Co-Authored-By: Codex <noreply@openai.com>
@Pigbibi
Pigbibi merged commit 6150a3d into main Oct 1, 2026
1 check passed
@Pigbibi
Pigbibi deleted the codex/schwab-native-facts-20261001 branch October 1, 2026 03:27
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