Skip to content

[codex] Fix combo bot display name#104

Merged
Pigbibi merged 1 commit into
mainfrom
codex/fix-combo-bot-display
Jul 4, 2026
Merged

[codex] Fix combo bot display name#104
Pigbibi merged 1 commit into
mainfrom
codex/fix-combo-bot-display

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Jul 4, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • fix periodic bot status fallback for crypto_equity_combo
  • add English/Chinese display name translations for combo profile
  • cover combo heartbeat display name with regression tests

Root cause

main.maybe_send_periodic_btc_status_report() resolved the default status strategy name with a hard-coded Crypto Live Pool Rotation fallback when no display name was passed. The combo profile also lacked legacy i18n keys, so direct heartbeat calls could show the old/default strategy name.

Validation

  • PYTHONPATH=/Users/lisiyi/Projects/QuantPlatformKit/src:/Users/lisiyi/Projects/CryptoStrategies/src:. STRATEGY_PROFILE=crypto_equity_combo NOTIFY_LANG=zh python3 - <<'PY' ... PY -> 🧭 策略: 加密动量组合
  • PYTHONPATH=/Users/lisiyi/Projects/QuantPlatformKit/src:/Users/lisiyi/Projects/CryptoStrategies/src:. python3 -m pytest tests/test_notify_i18n.py tests/test_status_reports.py tests/test_runtime_config_support.py -q -> 22 passed
  • PYTHONPATH=/Users/lisiyi/Projects/QuantPlatformKit/src:/Users/lisiyi/Projects/CryptoStrategies/src:. python3 -m pytest tests/test_strategy_runtime.py tests/test_cycle_service.py tests/test_runtime_support.py -q -> 18 passed
  • git diff --check

Co-Authored-By: Codex <noreply@openai.com>
@cursor

cursor Bot commented Jul 4, 2026

Copy link
Copy Markdown

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.

@Pigbibi Pigbibi merged commit dfefa1f into main Jul 4, 2026
2 checks passed
@Pigbibi Pigbibi deleted the codex/fix-combo-bot-display branch July 4, 2026 11:14
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