Skip to content

Strategy switch console review follow-ups - #98

Merged
Pigbibi merged 1 commit into
mainfrom
fix/console-review-followups
Jun 27, 2026
Merged

Strategy switch console review follow-ups#98
Pigbibi merged 1 commit into
mainfrom
fix/console-review-followups

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Persist cash_only_execution_mode in normalizeAccountOptions so KV account preferences reload correctly.
  • Align income layer summary with cash/option rules: no entry → 未读取; unset fields → 策略默认; use effective defaults in pending detection.
  • Replace invalid dl/dt/dd nesting with a semantic div summary list (role="list" / role="listitem").
  • Regroup reserve cash controls on the right column (策略 + 金额 + 比例); margin control stays on the left.
  • Remove unused noChangesNote copy; align copy button fallback text with i18n.
  • Add worker/UI validation for cash-only mode persistence and loadCurrentStrategies cash_only reads.

Test plan

  • python3 scripts/sync_strategy_switch_page_asset.py
  • node tests/strategy_switch_worker_validation.mjs
  • python3 -m unittest discover -s tests -q
  • Verify summary income layer shows 未读取 when current state unavailable
  • Verify account cash_only_execution_mode preference survives reload

Made with Cursor

Persist cash_only_execution_mode in account options, align income layer summary notRead/default handling, fix summary list semantics, regroup reserve cash controls, and extend worker validation coverage.

Co-authored-by: Cursor <cursoragent@cursor.com>
@Pigbibi
Pigbibi merged commit 2ab6bee into main Jun 27, 2026
1 of 2 checks passed
@Pigbibi
Pigbibi deleted the fix/console-review-followups branch June 27, 2026 17:46
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