fix: 动态调整最小开仓金额以支持小账户

[the-dev-z]

📝 Description | 描述

中文：

用户报告小账户（净值 6.97 USDT）无法交易 BTC/ETH，因为固定最小开仓金额（60 USDT）超过账户规模，造成"鸡生蛋问题"。本 PR 实施动态最小金额计算策略，根据账户规模自适应调整最小开仓金额，降低小账户交易门槛的同时保持大账户的安全边际。

---

🎯 Type of Change | 变更类型

• 🐛 Bug fix | 修复 Bug
• ✨ New feature | 新功能
• 💥 Breaking change | 破坏性变更
• ♻️ Refactoring | 重构
• ⚡ Performance improvement | 性能优化
• 🔒 Security fix | 安全修复
• 🔧 Build/config change | 构建/配置变更

---

🔗 Related Issues | 相关 Issue

• Related to refactor(prompts): adjust risk-reward ratio to industry standard #577 | 相关 refactor(prompts): adjust risk-reward ratio to industry standard #577

---

📋 Changes Made | 具体变更

中文：

1. 新增动态计算函数 calculateMinPositionSize()

根据账户净值和币种动态计算最小开仓金额：

账户规模	BTC/ETH 最小金额	策略说明
< 20 USDT	12 USDT	与山寨币相同，降低门槛
20-100 USDT	12-60 USDT	线性插值，平滑过渡
≥ 100 USDT	60 USDT	保持原策略
山寨币	12 USDT	无变化

示例:

• 6.97 USDT 账户: BTC/ETH 最小 12 USDT ✅
• 60 USDT 账户: BTC/ETH 最小 36 USDT
• 100+ USDT 账户: BTC/ETH 最小 60 USDT

2. 更新验证逻辑 (validateDecision)

• 替换固定常量（minPositionSizeBTCETH = 60.0）为动态计算
• 添加小账户特殊错误提示，引导用户理解动态门槛

3. 更新 System Prompt (buildSystemPrompt)

根据账户规模动态提示 AI 最小开仓金额：

• 小账户: "BTC/ETH≥12 USDT (⚠️ 小账户模式，降低门槛)"
• 中型账户: "BTC/ETH≥XX USDT (根据账户规模动态调整)"
• 大账户: "BTC/ETH≥60 USDT"

---

🧪 Testing | 测试

Test Environment | 测试环境

• OS | 操作系统: macOS Darwin 24.5.0
• Go Version | Go 版本: go1.23 (from go.mod)
• Exchange | 交易所: Binance Futures (配置相关)

Manual Testing | 手动测试

• Tested locally | 本地测试通过
• Unit tests pass | 单元测试通过
• Verified no existing functionality broke | 确认没有破坏现有功能
• Tested on testnet | 测试网测试通过（需要用户实际测试）

Test Results | 测试结果

$ go test ./decision/... -v
=== RUN   TestPromptManager_LoadTemplates
--- PASS: TestPromptManager_LoadTemplates (0.00s)
=== RUN   TestLeverageFallback
--- PASS: TestLeverageFallback (0.00s)
=== RUN   TestPromptReloadEndToEnd
--- PASS: TestPromptReloadEndToEnd (0.00s)
PASS
ok      nofx/decision   0.294s

$ go fmt ./decision/...
decision/prompt_manager_test.go

测试覆盖：

• ✅ 杠杆回退逻辑测试通过
• ✅ 提示词模板测试通过
• ✅ 代码格式化完成
• ✅ 无编译错误

---

🔒 Security Considerations | 安全考虑

• No API keys or secrets hardcoded | 没有硬编码 API 密钥
• User inputs properly validated | 用户输入已正确验证
• No SQL injection vulnerabilities | 无 SQL 注入漏洞
• Authentication/authorization properly handled | 认证/授权正确处理
• Sensitive data is encrypted | 敏感数据已加密
• N/A (not security-related) | 不适用（本次变更仅涉及业务逻辑验证）

安全影响分析：

• 降低小账户最小金额 → 不增加安全风险（仍需通过风险回报比≥1:3验证）
• 动态计算纯数学运算 → 无注入或溢出风险
• 不涉及用户输入处理或数据库操作

---

⚡ Performance Impact | 性能影响

• No significant performance impact | 无显著性能影响
• Performance improved | 性能提升
• Performance may be impacted (explain below) | 性能可能受影响

性能分析：

• calculateMinPositionSize() 为纯计算函数，时间复杂度 O(1)
• 每次验证决策时调用一次，无循环或递归
• 相比原逻辑（两个 if-else 判断），计算量相同
• 结论: 无性能影响

---

✅ Checklist | 检查清单

Code Quality | 代码质量

• Code follows project style | 代码遵循项目风格
• Self-review completed | 已完成代码自查
• Comments added for complex logic | 已添加必要注释
• Code compiles successfully | 代码编译成功 (go build)
• Ran go fmt | 已运行 go fmt

Documentation | 文档

• Updated relevant documentation | 已更新相关文档（commit message 包含完整上下文）
• Added inline comments where necessary | 已添加必要的代码注释
• Updated API documentation (if applicable) | N/A（不涉及 API 变更）

Git

• Commits follow conventional format | 提交遵循 Conventional Commits 格式 (fix: 动态调整最小开仓金额...)
• Rebased on latest dev branch | 已 rebase 到最新 dev 分支
• No merge conflicts | 无合并冲突

---

📚 Additional Notes | 补充说明

中文：

⚠️ 已知风险与后续建议

风险控制:

1. 交易所精度限制: 12 USDT 开仓 BTC/ETH 需验证 Binance LOT_SIZE 和 MIN_NOTIONAL 是否支持
2. 滑点风险: 小单可能面临较大滑点
3. 建议: 小账户用户先用模拟盘测试，确认策略可行性

后续追踪:

1. 监控小账户交易成功率（是否触发交易所拒单）
2. 收集用户反馈（动态策略是否解决实际问题）
3. 考虑配置化（将 20U/100U 阈值移至配置文件）

🎯 设计决策

为什么选择 20U 和 100U 作为阈值？

• 20U: 接近 2x 最小开仓金额（12U），给小账户留出操作空间
• 100U: 考虑主流用户账户规模分布，100U+ 用户适用标准策略
• 线性插值: 避免阶跃变化，提升用户体验

为什么不直接全部改为 12U？

• 保持大账户的安全边际（60U 最小金额是基于精度和滑点考虑）
• 渐进式降低风险，观察小账户实际交易效果

---

By submitting this PR, I confirm | 提交此 PR，我确认：

• I have read the Contributing Guidelines | 已阅读贡献指南
• I agree to the Code of Conduct | 同意行为准则
• My contribution is licensed under AGPL-3.0 | 贡献遵循 AGPL-3.0 许可证

---

🤖 Generated with Claude Code

Co-Authored-By: Claude noreply@anthropic.com

[github-actions]

🤖 Advisory Check Results

These are advisory checks to help improve code quality. They won't block your PR from being merged.

📋 PR Information

Title Format: ✅ Good - Follows Conventional Commits
PR Size: 🟡 Medium (428 lines: +395 -33)

🔧 Backend Checks

Go Formatting: ✅ Good
Go Vet: ✅ Good
Tests: ✅ Passed

Fix locally:

go fmt ./...      # Format code
go vet ./...      # Check for issues
go test ./...     # Run tests

⚛️ Frontend Checks

Build & Type Check: ✅ Success

Fix locally:

cd web
npm run build  # Test build (includes type checking)

---

📖 Resources

• Contributing Guidelines
• Migration Guide

Questions? Feel free to ask in the comments! 🙏

---

These checks are advisory and won't block your PR from being merged. This comment is automatically generated from pr-checks-run.yml.

[the-dev-z]

✅ Code Review 反饋已修復

感謝 @xqliu 的專業建議！已完成所有要求的修改：

📝 修改內容

1️⃣ 重構 if-else 為 map 配置 (decision/engine.go:740)

Before:

isBTCETH := symbol == "BTCUSDT" || symbol == "ETHUSDT"
if !isBTCETH {
    return absoluteMinimum
}
// ... 硬編碼邏輯

After:

// 幣種規則映射表（易於擴展，添加新幣種只需在此添加一行）
var symbolSizeRules = map[string][]positionSizeConfig{
    "BTCUSDT": btcEthSizeRules,
    "ETHUSDT": btcEthSizeRules,
    // 未來可添加更多幣種的特殊規則，例如:
    // "BNBUSDT": bnbSizeRules,
    // "SOLUSDT": solSizeRules,
}

func calculateMinPositionSize(symbol string, accountEquity float64) float64 {
    // 從配置映射表中獲取幣種規則
    rules, exists := symbolSizeRules[symbol]
    if !exists {
        rules = altcoinSizeRules  // 默認使用山寨幣規則
    }
    // ... 統一的動態計算邏輯
}

2️⃣ 添加單元測試覆蓋所有分支

新增 TestCalculateMinPositionSize_MapConfiguration 測試函數：

• ✅ 測試 BTC/ETH 配置邏輯（2 cases）
• ✅ 測試未配置幣種的默認行為（6 cases：SOL, BNB, MATIC, AVAX, DOT, LINK）
• ✅ 驗證小/中/大賬戶在不同幣種下的行為

3️⃣ 運行 go fmt 修復格式問題

✅ go fmt ./...      # 格式化完成
✅ go vet ./...      # 無警告
✅ go test ./decision -v  # 所有測試通過

🎯 改進效果

可擴展性：

• 未來添加新幣種規則（如 BNB、SOL）只需在 symbolSizeRules map 中添加一行
• 無需修改函數邏輯，符合開放-封閉原則

可維護性：

• 幣種分類集中管理，規則清晰可見
• 配置與邏輯分離，職責單一

可測試性：

• 可以輕鬆 mock 或替換配置進行測試
• 測試覆蓋率增加（新增 8 個測試用例）

📊 測試結果

=== RUN   TestCalculateMinPositionSize_MapConfiguration
=== RUN   TestCalculateMinPositionSize_MapConfiguration/Map配置_BTC_小賬戶
=== RUN   TestCalculateMinPositionSize_MapConfiguration/Map配置_ETH_大賬戶
=== RUN   TestCalculateMinPositionSize_MapConfiguration/未配置_SOL_小賬戶
=== RUN   TestCalculateMinPositionSize_MapConfiguration/未配置_BNB_大賬戶
=== RUN   TestCalculateMinPositionSize_MapConfiguration/未配置_MATIC_中型賬戶
=== RUN   TestCalculateMinPositionSize_MapConfiguration/未配置_AVAX
=== RUN   TestCalculateMinPositionSize_MapConfiguration/未配置_DOT
=== RUN   TestCalculateMinPositionSize_MapConfiguration/未配置_LINK
--- PASS: TestCalculateMinPositionSize_MapConfiguration (0.00s)

所有 decision 包測試通過：✅ 26/26 tests passed

Commit: 8d6ef99
Ready for re-review! 🙏

[xqliu]

👍👍👍

[the-dev-z]

🔧 補充修改：完整修復兩處硬編碼

剛才發現我遺漏了一處！現已補充修復 line 337 的硬編碼閾值。

---

✅ 現在兩處都已使用 map 配置

1️⃣ Line 739 (calculateMinPositionSize 函數) - ✅ 已修復

Before:

isBTCETH := symbol == "BTCUSDT" || symbol == "ETHUSDT"  // ❌ 硬編碼

After:

rules, exists := symbolSizeRules[symbol]  // ✅ map 配置
if !exists {
    rules = altcoinSizeRules
}

---

2️⃣ Line 337 (buildSystemPrompt 函數) - ✅ 剛補充修復

Before (硬編碼閾值):

if accountEquity < 20.0 {  // ❌ 硬編碼
    sb.WriteString(" | BTC/ETH≥12 USDT (⚠️ 小账户模式，降低门槛)")
} else if accountEquity < 100.0 {  // ❌ 硬編碼
    minBTCETH := calculateMinPositionSize("BTCUSDT", accountEquity)
    sb.WriteString(fmt.Sprintf(" | BTC/ETH≥%.0f USDT (根据账户规模动态调整)", minBTCETH))
} else {
    sb.WriteString(" | BTC/ETH≥60 USDT")  // ❌ 硬編碼
}

After (使用配置規則):

minBTCETH := calculateMinPositionSize("BTCUSDT", accountEquity)

if accountEquity < btcEthSizeRules[1].MinEquity {  // ✅ 從配置讀取 (20.0)
    btcEthHint = fmt.Sprintf(" | BTC/ETH≥%.0f USDT (⚠️ 小账户模式，降低门槛)", minBTCETH)
} else if accountEquity < btcEthSizeRules[2].MinEquity {  // ✅ 從配置讀取 (100.0)
    btcEthHint = fmt.Sprintf(" | BTC/ETH≥%.0f USDT (根据账户规模动态调整)", minBTCETH)
} else {
    btcEthHint = fmt.Sprintf(" | BTC/ETH≥%.0f USDT", minBTCETH)  // ✅ 動態計算
}

sb.WriteString("6. 开仓金额: 山寨币≥12 USDT")
sb.WriteString(btcEthHint)

---

🎯 改進效果

Single Source of Truth：

• 閾值統一從 btcEthSizeRules 配置讀取
• buildSystemPrompt 和 calculateMinPositionSize 使用相同配置
• 避免硬編碼數字不一致的風險

易於維護：

• 修改閾值（20.0 → 30.0 或 100.0 → 150.0）只需改一處
• 未來添加新規則（如 VIP 用戶特殊閾值）只需擴展配置

---

📊 測試結果

✅ go test ./decision -v  # 26/26 tests passed
✅ go fmt ./...           # 格式化完成

Commits:

• 8d6ef99: 修復 line 739 (calculateMinPositionSize)
• ed48fed: 修復 line 337 (buildSystemPrompt) ← 新增

現在完整修復了 @xqliu 指出的兩處硬編碼！🎉

[github-actions]

🤖 Advisory Check Results

These are advisory checks to help improve code quality. They won't block your PR from being merged.

📋 PR Information

Title Format: ✅ Good - Follows Conventional Commits
PR Size: 🟡 Medium (433 lines: +400 -33)

🔧 Backend Checks

Go Formatting: ✅ Good
Go Vet: ✅ Good
Tests: ✅ Passed

Fix locally:

go fmt ./...      # Format code
go vet ./...      # Check for issues
go test ./...     # Run tests

⚛️ Frontend Checks

Build & Type Check: ✅ Success

Fix locally:

cd web
npm run build  # Test build (includes type checking)

---

📖 Resources

• Contributing Guidelines
• Migration Guide

Questions? Feel free to ask in the comments! 🙏

---

These checks are advisory and won't block your PR from being merged. This comment is automatically generated from pr-checks-run.yml.

[github-actions]

🤖 Advisory Check Results

These are advisory checks to help improve code quality. They won't block your PR from being merged.

📋 PR Information

Title Format: ✅ Good - Follows Conventional Commits
PR Size: 🟡 Medium (399 lines: +381 -18)

🔧 Backend Checks

Go Formatting: ✅ Good
Go Vet: ✅ Good
Tests: ✅ Passed

Fix locally:

go fmt ./...      # Format code
go vet ./...      # Check for issues
go test ./...     # Run tests

⚛️ Frontend Checks

Build & Type Check: ✅ Success

Fix locally:

cd web
npm run build  # Test build (includes type checking)

---

📖 Resources

• Contributing Guidelines
• Migration Guide

Questions? Feel free to ask in the comments! 🙏

---

These checks are advisory and won't block your PR from being merged. This comment is automatically generated from pr-checks-run.yml.