Skip to content

Fix runtime target key normalization for strategy switch#146

Merged
Pigbibi merged 1 commit into
mainfrom
fix-longbridge-runtime-target-key-normalization
Jul 2, 2026
Merged

Fix runtime target key normalization for strategy switch#146
Pigbibi merged 1 commit into
mainfrom
fix-longbridge-runtime-target-key-normalization

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Fix account runtime status lookup in strategy switch page to tolerate key case/whitespace mismatch.
  • Keep exact key precedence, add normalized fallback for key/target_name/label.
  • Preserve existing behavior for existing state entries while filling defaults safely.
  • Keep fallback path unchanged so missing state still uses defaults.
  • Added regression tests for normalized mapping and fallback behavior.

Validation

  • node tests/test_cash_financing.js
  • node 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 merged commit 3f7e50b into main Jul 2, 2026
8 checks passed
@Pigbibi Pigbibi deleted the fix-longbridge-runtime-target-key-normalization branch July 2, 2026 18:45
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