fix(longbridge): change default strategy from tqqq_growth_income to soxl_soxx_trend_income#172
Conversation
…oxl_soxx_trend_income - Update platform-config.json: LongBridge default_strategy_profile tqqq → soxl - Regenerate derived config files (config.js, index.html, page_asset.js) - Update app.js hardcoded fallback to soxl_soxx_trend_income - Update test assertions for LongBridge default strategy drift checks Root cause: platform-config.json was the single source of truth but LongBridge still had tqqq_growth_income as default while IBKR/Schwab already used soxl_soxx_trend_income. The runtime reads RUNTIME_TARGET_JSON from GitHub Environment variables, which was initially set to tqqq and never updated via strategy switch. Co-Authored-By: Claude <noreply@anthropic.com>
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
🤖 Codex PR ReviewPlease ensure a human reviewer checks this PR before merging. |
Co-Authored-By: Claude <noreply@anthropic.com>
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b3f52225a6
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| } | ||
|
|
||
| function defaultStrategyForAccount(platform, account, fallback = "tqqq_growth_income") { | ||
| function defaultStrategyForAccount(platform, account, fallback = "soxl_soxx_trend_income") { |
There was a problem hiding this comment.
This updates the source app.js, but the Worker serves /app.js from the generated APP_JS module in web/strategy-switch-console/app_js.js; that generated file was not updated and still contains fallback = "tqqq_growth_income". In environments that use the checked-in Worker assets, the browser keeps running the old fallback, so this fallback change is not actually shipped until app_js.js is regenerated and committed.
Useful? React with 👍 / 👎.
1 participant
问题
LongBridge 平台的实际运行策略是
tqqq_growth_income（纳斯达克增长收益），但用户期望的是soxl_soxx_trend_income（半导体趋势收益）。根因
platform-config.json中 LongBridge 的default_strategy_profile自创建以来一直是tqqq_growth_income，从未更新为 SOXL 策略。而 IBKR 和 Schwab 已经正确设置为soxl_soxx_trend_income。运行时策略解析链：
platform-config.json是整个系统的单一真相来源，策略切换控制台和工作流都依赖它来确定默认策略。改动
platform-config.jsontqqq_growth_income→soxl_soxx_trend_incomeconfig.jsindex.htmlpage_asset.jsapp.jssoxl_soxx_trend_incometest_runtime_settings.py其他平台检查
tqqq→ soxl后续操作
合并后需要：
deploy-strategy-switch-console.yml）RUNTIME_TARGET_JSON中的strategy_profile更新为soxl_soxx_trend_income🤖 Generated with Claude Code