Skip to content

fix(execution): universal whole-share retention rule for small accounts#209

Merged
Pigbibi merged 1 commit into
mainfrom
fix/universal-whole-share-retention
Jul 8, 2026
Merged

fix(execution): universal whole-share retention rule for small accounts#209
Pigbibi merged 1 commit into
mainfrom
fix/universal-whole-share-retention

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

问题

小账户整数股兼容层使用硬编码白名单决定是否保留已有持仓,每加新标的都要手动维护。

修复

通用规则:如果账户已持有该标的且 target >= 1 股价的 85%,就保留不卖。

详见 LongBridgePlatform#307

🤖 Generated with Claude Code

@chatgpt-codex-connector

This comment has been minimized.

@cursor

cursor Bot commented Jul 8, 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.

Replace the per-symbol hardcoded whitelist with a universal rule:
if the account already holds a symbol (>0 shares) AND the strategy
wants to keep >= 85% of 1 share's value, retain the position.

Co-Authored-By: Claude <noreply@anthropic.com>
@Pigbibi Pigbibi force-pushed the fix/universal-whole-share-retention branch from 055be8a to ac934b7 Compare July 8, 2026 21:50
@cursor

cursor Bot commented Jul 8, 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 enabled auto-merge (squash) July 8, 2026 21:51
@Pigbibi Pigbibi merged commit 296da01 into main Jul 8, 2026
2 checks passed
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