Skip to content

Fix runtime target status parsing for longbridge SG account#145

Merged
Pigbibi merged 1 commit into
mainfrom
fix/runtime-target-case-insensitive
Jul 2, 2026
Merged

Fix runtime target status parsing for longbridge SG account#145
Pigbibi merged 1 commit into
mainfrom
fix/runtime-target-case-insensitive

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

What changed

  • Make runtime target boolean parsing case/whitespace tolerant in strategy switch console worker and front-end.
  • Treat any enabled/disabled config parse failure as enabled when initializing runtime target mode.
  • Add regression tests for uppercase TRUE/FALSE values and worker parsing.

Why

  • Longbridge SG account showed state in backend but rendered as in UI when values were returned as case-variant strings.

Validation

  • node --test tests/test_cash_financing.js
  • node --test tests/strategy_switch_worker_validation.mjs

Co-Authored-By: Codex <noreply@openai.com>
@chatgpt-codex-connector

This comment has been minimized.

@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown

🤖 Codex PR Review

⚠️ Review skipped: The Codex review could not be completed.

No Codex service URL or API key configured. Set CODEX_AUDIT_SERVICE_URL, ANTHROPIC_API_KEY, or OPENAI_API_KEY.

Please ensure a human reviewer checks this PR before merging.

@Pigbibi Pigbibi enabled auto-merge (squash) July 2, 2026 17:58
@Pigbibi Pigbibi merged commit 564cfa8 into main Jul 2, 2026
8 checks passed
@Pigbibi Pigbibi deleted the fix/runtime-target-case-insensitive branch July 2, 2026 18:12
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