Skip to content

Show broker account values in Firstrade idle heartbeats - #336

Merged
Pigbibi merged 1 commit into
mainfrom
codex/firstrade-heartbeat-account-values-20260925
Sep 25, 2026
Merged

Pigbibi merged 1 commit into
mainfrom
codex/firstrade-heartbeat-account-values-20260925

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Sep 25, 2026

Copy link
Copy Markdown
Contributor

Change

  • Capture broker reported full account equity independently from strategy subset equity.
  • Add current broker cash and account equity to no-order cycle notifications in Chinese and English. Missing or unverified values are labeled as such.
  • Keep strategy sizing, risk, and order paths unchanged.

Validation

  • python3 -m pytest -q tests/test_notifications_telegram.py tests/test_runtime_broker_adapters.py — 18 passed.
  • Production image deployment remains parked because the existing dry-run precheck found stale signal-source data. This PR does not change that guard.

Co-Authored-By: Codex <noreply@openai.com>
@Pigbibi
Pigbibi merged commit 62bc5f1 into main Sep 25, 2026
1 check passed
@Pigbibi
Pigbibi deleted the codex/firstrade-heartbeat-account-values-20260925 branch September 25, 2026 15:18
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