[BUG][P1] 余额总盈亏这些显示错误

[xqliu]

⚠️ Before submitting: Please check the Troubleshooting Guide (中文版) to see if your issue can be resolved quickly.

🐛 Bug Description

红线框出的这些数据都是错的

🔍 Bug Category

• Trading execution (orders not executing, wrong position size, etc.)
• AI decision issues (unexpected decisions, only opening one direction, etc.)
• Exchange connection (API errors, authentication failures, etc.)
• UI/Frontend (display issues, buttons not working, data not updating, etc.)
• Backend/API (server errors, crashes, performance issues, etc.)
• Configuration (settings not saving, database errors, etc.)
• Other: _________________

📋 Steps to Reproduce

1. Go to '...'
2. Click on '...' / Run command '...'
3. Configure '...'
4. See error

✅ Expected Behavior

❌ Actual Behavior

📸 Screenshots & Logs

Frontend Error (if applicable)

Browser Console Screenshot:

Network Tab (failed requests):

Backend Logs (if applicable)

Docker users:

# View backend logs
docker compose logs backend --tail=100

# OR continuously follow logs
docker compose logs -f backend

Manual/PM2 users:

# Terminal output where you ran: ./nofx
# OR PM2 logs:
pm2 logs nofx --lines 100

Backend Log Output:

Paste backend logs here (last 50-100 lines around the error)

Trading/Decision Logs (if trading issue)

Decision Log Path: decision_logs/{trader_id}/{timestamp}.json

{
  "paste relevant decision log here if applicable"
}

💻 Environment

System:

• OS: [e.g. macOS 13, Ubuntu 22.04, Windows 11]
• Deployment: [Docker / Manual / PM2]

Backend:

• Go Version: [run: go version]
• NOFX Version: [run: git log -1 --oneline or check release tag]

Frontend:

• Browser: [e.g. Chrome 120, Firefox 121, Safari 17]
• Node.js Version: [run: node -v]

Trading Setup:

• Exchange: [Binance / Hyperliquid / Aster]
• Account Type: [Main Account / Subaccount]
• Position Mode: [Hedge Mode (Dual) / One-way Mode] ← Important for trading bugs!
• AI Model: [DeepSeek / Qwen / Custom]
• Number of Traders: [e.g. 1, 2, etc.]

🔧 Configuration (if relevant)

Leverage Settings:

{
  "btc_eth_leverage": 5,
  "altcoin_leverage": 5
}

Any custom settings:

📊 Additional Context

Frequency:

• Happens every time
• Happens randomly
• Happened once

Timeline:

• Did this work before? [ ] Yes [ ] No
• When did it break? [e.g. after upgrade to v3.0.0, after changing config, etc.]
• Recent changes? [e.g. updated dependencies, changed exchange, etc.]

Impact:

• System cannot start
• Trading stopped/broken
• UI broken but trading works
• Minor visual issue
• Other: _________________

💡 Possible Solution

---

📝 Quick Tips for Faster Resolution

For Trading Issues:

1. ✅ Check Binance position mode: Go to Futures → ⚙️ Preferences → Position Mode → Must be Hedge Mode
2. ✅ Verify API permissions: Futures trading must be enabled
3. ✅ Check decision logs in decision_logs/{trader_id}/ for AI reasoning

For Connection Issues:
4. ✅ Test API connectivity: curl http://localhost:8080/api/health
5. ✅ Check API rate limits on exchange
6. ✅ Verify API keys are not expired

For UI Issues:
7. ✅ Hard refresh: Ctrl+Shift+R (or Cmd+Shift+R on Mac)
8. ✅ Check browser console (F12) for errors
9. ✅ Verify backend is running: docker compose ps or ps aux | grep nofx

[xqliu]

🔍 问题根源分析

我已经定位到问题所在。盈亏百分比计算错误的原因是 initialBalance 被自动更新机制错误地修改了。

问题复现路径

在 trader/auto_trader.go:287-371 的 syncBalanceChanges() 函数中，存在一个自动检测余额变化的机制：

// 变化超过5%才更新
if math.Abs(changePercent) > 5.0 {
    log.Printf("🔔 [%s] 检测到余额大幅变化: %.2f → %.2f USDT (%.2f%%)",
        at.name, oldBalance, actualBalance, changePercent)
    
    // 更新内存中的 initialBalance
    at.initialBalance = actualBalance
    
    // 更新数据库...
}

实际发生的情况

从日志可以看到：

1. 初始余额: 200.00 USDT
2. 22:57:13 - 检测到余额变化 -16.35%，将 initialBalance 从 200.00 更新为 167.29 USDT
3. 23:09:13 - 再次检测到余额变化 -14.76%，将 initialBalance 从 167.29 更新为 142.61 USDT

2025/11/06 22:45:12 💰 初始余额: 200.00 USDT
2025/11/06 22:57:13 🔔 [招财猫] 检测到余额大幅变化: 200.00 → 167.29 USDT (-16.35%)
2025/11/06 23:09:13 🔔 [招财猫] 检测到余额大幅变化: 167.29 → 142.61 USDT (-14.76%)

计算验证

当前显示（23:12:21）：

• 净值 = 194.14 USDT
• 盈亏 = 51.53 USDT
• 百分比 = 36.13% ❌

错误的计算（使用被自动更新的 initialBalance = 142.61）：

totalPnL = 194.14 - 142.61 = 51.53 ✓
totalPnLPct = (51.53 / 142.61) × 100 = 36.13% ✓

正确的计算（应该使用真实的初始余额 = 200.00）：

totalPnL = 194.14 - 200.00 = -5.86
totalPnLPct = (-5.86 / 200.00) × 100 = -2.93% ✓

根本原因

syncBalanceChanges() 函数会在余额变化超过 5% 时，自动将 initialBalance 更新为当前的可用余额（availableBalance）。这个设计的初衷可能是为了应对手动出入金，但存在严重问题：

1. 交易亏损时：系统会错误地将亏损后的余额当成新的"初始余额"，导致盈亏计算归零
2. 使用错误的余额：它使用的是 availableBalance（可用余额），而不是 totalEquity（总净值 = 钱包余额 + 未实现盈亏）

这导致：

• 持仓有未实现亏损时，可用余额会减少
• 系统误以为发生了出金，将较低的可用余额作为新的基准
• 当持仓平仓或盈利时，相对于这个错误的基准计算出虚高的盈亏百分比

---

💡 修复方案

方案1：完全移除自动同步逻辑（推荐）

如果不需要支持交易过程中的手动出入金，直接禁用这个功能：

// 注释掉或删除 syncBalanceChanges() 的定时调用
// 在 Start() 方法中移除相关代码

方案2：修改同步条件，只在余额明显增加时才更新

只在余额明显增加（如增加超过 20%）时才认为是入金：

// 只在余额显著增加时才更新（入金场景）
if changePercent > 20.0 {  // 只处理增加的情况
    log.Printf("🔔 [%s] 检测到入金: %.2f → %.2f USDT (+%.2f%%)",
        at.name, oldBalance, actualBalance, changePercent)
    at.initialBalance = actualBalance
    // 更新数据库...
}

方案3：分离 initialBalance 和 lastKnownBalance 的概念（最佳）

• initialBalance: 永远不变的基准（用于计算总收益率）
• lastKnownBalance: 用于检测出入金事件

type AutoTrader struct {
    initialBalance    float64  // 永远不变，用于计算收益率
    lastKnownBalance  float64  // 用于检测出入金
    // ...
}

func (at *AutoTrader) syncBalanceChanges() {
    // ...
    
    changePercent := ((actualBalance - at.lastKnownBalance) / at.lastKnownBalance) * 100
    
    // 检测明显的入金/出金（>20%）
    if changePercent > 20.0 {
        log.Printf("🔔 [%s] 检测到入金事件", at.name)
        at.initialBalance = actualBalance  // 重置基准
    } else if changePercent < -20.0 {
        log.Printf("🔔 [%s] 检测到出金事件", at.name)
        at.initialBalance = actualBalance  // 重置基准
    }
    
    at.lastKnownBalance = actualBalance  // 总是更新追踪余额
}

方案4：使用 totalEquity 替代 availableBalance

当前代码使用的是 availableBalance（可用余额），应该改用 totalEquity（总净值）：

// 当前错误的实现
actualBalance := balance["availableBalance"].(float64)

// 应该改为
totalWalletBalance := balance["totalWalletBalance"].(float64)
totalUnrealizedProfit := balance["totalUnrealizedProfit"].(float64)
actualBalance := totalWalletBalance + totalUnrealizedProfit  // 使用总净值

---

📝 建议的修复优先级

1. 立即修复：使用 totalEquity 替代 availableBalance（方案4）
2. 短期修复：采用方案3（分离两个概念）
3. 长期优化：添加手动出入金记录功能，而不是依赖自动检测

---

🔗 相关代码位置

• trader/auto_trader.go:287-371 - syncBalanceChanges() 函数
• trader/auto_trader.go:1307-1310 - GetAccountInfo() 中的盈亏计算
• trader/auto_trader.go:90 - initialBalance 字段定义

[xqliu]

🔍 补充说明：余额变化的真正原因

我刚刚查看了日志，发现这两次"余额大幅变化"并不是因为开单或平仓，而是因为：

真正的原因：混淆了 钱包余额 和 可用余额

从日志可以看到：

22:55:21 - 开了一个 ETHUSDT 3x 空单（0.0249 ETH）

✓ 开空仓成功: ETHUSDT 数量: 0.0249
止损价设置: 3489.0000
止盈价设置: 2862.0000

22:57:13 - 第一次"检测到余额大幅变化"

⭐ 总资产: 194.86 USDC | Perp 可用: 167.29 USDC | Spot 余额: 0.00 USDC
🔔 [招财猫] 检测到余额大幅变化: 200.00 → 167.29 USDT (-16.35%)

关键问题：

• 总资产 = 194.86 USDC（钱包余额 + 未实现盈亏）
• 可用余额 = 167.29 USDC（钱包余额 - 占用保证金）
• 系统错误地使用了 可用余额 而不是 总资产 来检测变化

为什么可用余额会减少？

当开仓后：

1. 总资产（Total Equity） 几乎不变（只有小幅盈亏波动）：194.86 USDC
2. 钱包余额 也不变：194.80 USDC
3. 可用余额 大幅减少：从 200 → 167.29（因为保证金被占用了）

计算保证金占用：

• ETH 开仓价：3291.50
• 数量：0.0249 ETH
• 杠杆：3x
• 仓位价值：0.0249 × 3291.50 = 81.96 USDC
• 保证金占用：81.96 / 3 ≈ 27.32 USDC

类似地，在 23:09:13 第二次变化时，又开了新仓位，进一步占用保证金，导致可用余额从 167.29 → 142.61。

代码中的问题

在 trader/auto_trader.go:287-371 的 syncBalanceChanges() 中：

func (at *AutoTrader) syncBalanceChanges() {
    // ...
    
    // ❌ 错误：使用了 availableBalance（可用余额）
    actualBalance := 0.0
    if availableBalance, ok := balanceInfo["availableBalance"].(float64); ok && availableBalance > 0 {
        actualBalance = availableBalance  // 这个会随着持仓变化而变化！
    }
    
    // ...
    
    changePercent := ((actualBalance - at.initialBalance) / at.initialBalance) * 100
    
    // 变化超过5%就错误地认为发生了出入金
    if math.Abs(changePercent) > 5.0 {
        at.initialBalance = actualBalance  // 错误地更新基准
    }
}

应该改为使用 totalEquity（总资产）：

// ✅ 正确：应该使用总资产
totalWalletBalance := balanceInfo["totalWalletBalance"].(float64)
totalUnrealizedProfit := balanceInfo["totalUnrealizedProfit"].(float64)
actualBalance := totalWalletBalance + totalUnrealizedProfit  // 总资产才是真实的账户价值

---

📊 数据对比总结

时间点	总资产	钱包余额	可用余额	initialBalance (错误)	实际应该的 initialBalance
初始	~200	~200	~200	200	200
22:57:13 (开仓后)	194.86	194.80	167.29	167.29 ❌	200 ✅
23:09:13 (再次开仓)	194.80	194.80	142.61	142.61 ❌	200 ✅
当前	193.75	194.80	142.61	142.61 ❌	200 ✅

真实盈亏：193.75 - 200 = -6.25 USDC (-3.13%)
显示盈亏：193.75 - 142.61 = +51.14 USDC (+35.84%) ❌

---

✅ 修复方案（更新）

现在问题更明确了，修复方案应该是：

方案1：使用 totalEquity 替代 availableBalance（最简单）

在 syncBalanceChanges() 中：

// 获取总资产而不是可用余额
totalWalletBalance := 0.0
totalUnrealizedProfit := 0.0

if wallet, ok := balanceInfo["totalWalletBalance"].(float64); ok {
    totalWalletBalance = wallet
}
if unrealized, ok := balanceInfo["totalUnrealizedProfit"].(float64); ok {
    totalUnrealizedProfit = unrealized
}

actualBalance := totalWalletBalance + totalUnrealizedProfit

方案2：完全禁用自动同步（如果不需要支持出入金）

直接注释掉或删除 syncBalanceChanges() 的定时调用。

方案3：更严格的出入金检测条件

只在变化超过 50% 时才认为是真正的出入金：

if math.Abs(changePercent) > 50.0 {  // 提高阈值到50%
    // ...
}

---

总结：这个 bug 的核心是混淆了"可用余额"和"总资产"，导致正常的持仓保证金占用被误认为是出金，从而错误地重置了初始余额基准。

[xqliu]

💡 推荐的修复步骤

基于以上分析，我建议按以下优先级修复这个问题：

---

🔥 Priority 1: 修复 syncBalanceChanges() 使用错误的余额类型

文件: trader/auto_trader.go:287-371

当前错误代码:

func (at *AutoTrader) syncBalanceChanges() {
    // ...
    actualBalance := 0.0
    if availableBalance, ok := balanceInfo["availableBalance"].(float64); ok && availableBalance > 0 {
        actualBalance = availableBalance  // ❌ 错误：这会随持仓变化
    } else if totalBalance, ok := balanceInfo["balance"].(float64); ok && totalBalance > 0 {
        actualBalance = totalBalance
    }
    // ...
}

修复后代码:

func (at *AutoTrader) syncBalanceChanges() {
    // ...
    
    // ✅ 使用总资产（钱包余额 + 未实现盈亏）
    totalWalletBalance := 0.0
    totalUnrealizedProfit := 0.0
    
    if wallet, ok := balanceInfo["totalWalletBalance"].(float64); ok {
        totalWalletBalance = wallet
    }
    if unrealized, ok := balanceInfo["totalUnrealizedProfit"].(float64); ok {
        totalUnrealizedProfit = unrealized
    }
    
    actualBalance := totalWalletBalance + totalUnrealizedProfit
    
    // 对于 Hyperliquid，可能字段名不同，需要适配
    if actualBalance == 0 {
        // Hyperliquid 使用 marginSummary.accountValue
        if accountValue, ok := balanceInfo["accountValue"].(float64); ok {
            actualBalance = accountValue
        }
    }
    
    // ...后续检测逻辑保持不变
}

---

🔥 Priority 2: 提高出入金检测阈值

当前问题: 5% 的阈值太敏感，正常的交易盈亏就可能触发

建议修改:

// 从 5% 提高到 20% 或更高
if math.Abs(changePercent) > 20.0 {  // ✅ 提高阈值
    log.Printf("🔔 [%s] 检测到余额大幅变化: %.2f → %.2f USDT (%.2f%%)",
        at.name, oldBalance, actualBalance, changePercent)
    
    at.initialBalance = actualBalance
    // ...
}

或者，只在余额增加时才认为是入金：

// 只在余额显著增加时更新（入金场景）
if changePercent > 20.0 {  // 只处理增加 >20% 的情况
    log.Printf("💰 [%s] 检测到入金: %.2f → %.2f USDT (+%.2f%%)",
        at.name, oldBalance, actualBalance, changePercent)
    
    at.initialBalance = actualBalance
    // ...
} else if changePercent < -50.0 {  // 出金阈值更高（-50%）
    log.Printf("💸 [%s] 检测到出金: %.2f → %.2f USDT (%.2f%%)",
        at.name, oldBalance, actualBalance, changePercent)
    
    at.initialBalance = actualBalance
    // ...
}

---

🔥 Priority 3: 分离 initialBalance 和 lastKnownBalance 概念

目标: 让 initialBalance 真正"永远不变"，只用于计算收益率

修改 AutoTrader 结构体:

type AutoTrader struct {
    // ...
    initialBalance    float64  // 永远不变的基准（用于计算总收益率）
    lastKnownBalance  float64  // 上次检查的余额（用于检测出入金）
    // ...
}

修改 syncBalanceChanges():

func (at *AutoTrader) syncBalanceChanges() {
    // ... 获取 actualBalance（使用 totalEquity）
    
    if at.lastKnownBalance <= 0 {
        at.lastKnownBalance = actualBalance
        return
    }
    
    changePercent := ((actualBalance - at.lastKnownBalance) / at.lastKnownBalance) * 100
    
    // 检测明显的入金/出金（>20%）
    if changePercent > 20.0 {
        log.Printf("💰 [%s] 检测到入金事件: %.2f → %.2f (+%.2f%%)",
            at.name, at.lastKnownBalance, actualBalance, changePercent)
        
        at.initialBalance = actualBalance  // 重置收益率基准
        
        // 记录入金事件到数据库（可选）
        // ...
    } else if changePercent < -20.0 {
        log.Printf("💸 [%s] 检测到出金事件: %.2f → %.2f (%.2f%%)",
            at.name, at.lastKnownBalance, actualBalance, changePercent)
        
        at.initialBalance = actualBalance  // 重置收益率基准
        
        // 记录出金事件到数据库（可选）
        // ...
    }
    
    // 总是更新追踪余额（即使没有发生出入金）
    at.lastKnownBalance = actualBalance
}

---

📝 Priority 4: 添加手动重置功能（可选）

为用户提供手动重置 initialBalance 的 API：

API 端点: POST /api/traders/:id/reset-balance

func (s *Server) handleResetBalance(c *gin.Context) {
    _, traderID, err := s.getTraderFromQuery(c)
    if err != nil {
        c.JSON(http.StatusBadRequest, gin.H{"error": err.Error()})
        return
    }
    
    trader, err := s.traderManager.GetTrader(traderID)
    if err != nil {
        c.JSON(http.StatusNotFound, gin.H{"error": err.Error()})
        return
    }
    
    // 获取当前账户总资产
    account, err := trader.GetAccountInfo()
    if err != nil {
        c.JSON(http.StatusInternalServerError, gin.H{"error": err.Error()})
        return
    }
    
    newBalance := account["total_equity"].(float64)
    
    // 重置 initialBalance
    err = trader.ResetInitialBalance(newBalance)
    if err != nil {
        c.JSON(http.StatusInternalServerError, gin.H{"error": err.Error()})
        return
    }
    
    c.JSON(http.StatusOK, gin.H{
        "message": "Initial balance reset successfully",
        "new_balance": newBalance,
    })
}

---

🧪 Priority 5: 添加单元测试

为 syncBalanceChanges() 添加测试用例，确保修复后不会复发：

func TestSyncBalanceChanges(t *testing.T) {
    // 测试场景1: 开仓后可用余额减少，但总资产不变
    // 预期：不应该触发 initialBalance 更新
    
    // 测试场景2: 真实入金（总资产增加 >20%）
    // 预期：应该更新 initialBalance
    
    // 测试场景3: 真实出金（总资产减少 >20%）
    // 预期：应该更新 initialBalance
    
    // 测试场景4: 正常交易盈亏（±15%）
    // 预期：不应该触发 initialBalance 更新
}

---

🎯 立即行动建议

如果需要快速修复，建议：

1. 最快方案：直接禁用自动同步功能

   • 在 trader/auto_trader.go 的 Start() 方法中，注释掉 syncBalanceChanges() 的定时调用
   • 适用于：不需要支持交易过程中手动出入金的场景

2. 推荐方案：实施 Priority 1 + Priority 2

   • 修复余额类型（使用 totalEquity）
   • 提高阈值到 20% 或更高
   • 预计工作量：30-60 分钟

3. 完整方案：实施 Priority 1-5

   • 彻底解决问题并防止复发
   • 预计工作量：2-3 小时

---

📊 验证修复是否成功

修复后，观察日志应该看到：

✅ 正确的日志示例：
⭐ 总资产: 194.80 USDC | Perp 可用: 142.61 USDC
✓ [招财猫] 余额变化不大 (0.02%)，无需更新

❌ 不应该再出现：
🔔 [招财猫] 检测到余额大幅变化: 167.29 → 142.61 USDT (-14.76%)

以及前端显示的盈亏应该与实际相符：

• 初始余额：200 USDC
• 当前净值：194 USDC
• 显示盈亏：-6 USDC (-3%) ✅

而不是：

• 显示盈亏：+51 USDC (+36%) ❌

[Tring-green]

官方动作太慢了，用 AI 我自己就修了好几个bug包括这个，半小时

[the-dev-z]

✅ 此问题已在 PR #668 中修复

感谢 @xqliu 的详细分析！你的分析完全正确。

问题根源

系统在 的两个地方使用了 可用余额（）而非 总资产（）：

1. • 创建交易员时设置初始余额
2. • 同步余额时更新初始余额

错误链条

开仓 → 保证金锁定 → 可用余额减少
  ↓
系统检测到余额变化 >5%
  ↓
误以为发生出金 → 重置 initialBalance
  ↓
盈亏计算基准错误 → 显示错误的盈亏百分比

你的案例

真实情况：
  初始: $200 → 当前: $194 = -$6 (-3%) ✓

系统错误显示：
  初始: $142 (被错误重置) → 当前: $194 = +$52 (+36%) ✗

修复方案

采用了你建议的 方案 1 + 方案 4：

1. ✅ 使用 totalEquity 替代 availableBalance

   totalEquity = totalWalletBalance + totalUnrealizedProfit

2. ✅ 增强日志输出，显示明细

   ✓ 查询到交易所总资产余额: 194.14 USDT (钱包: 194.80 + 未实现: -0.66)
   

影响范围

• ✅ 修复所有交易所（Binance、Hyperliquid、Aster）
• ✅ 修复创建交易员时的初始余额设置
• ✅ 修复余额同步时的错误重置
• ✅ 防止保证金占用被误判为出金

---

相关 PR: #668
修改文件: api/server.go (handleCreateTrader, handleSyncBalance)

感谢你用 AI 快速定位并分析了这个问题！🎉

[the-dev-z]

可能已通过 #634 修复 ✅

这个问题很可能已经被 PR #670（修复 #634）解决了。

问题分析

#637 显示的余额总盈亏这些数据都是错的很可能是由于 API 调用失败导致的数据显示错误。

#634 修复了一个关键的竞态条件问题：

• 问题: 页面刷新或直接访问时，SWR 立即发起 API 请求，但此时 AuthContext 还在从 localStorage 加载 token
• 结果: 请求带着 Authorization: Bearer null 发送，导致所有 API 调用失败
• 影响: 无法获取正确的账户数据、余额、盈亏等信息

修复内容

PR #670 在以下组件添加了认证检查：

• web/src/App.tsx: 5 个 SWR 调用（status, account, positions, decisions, stats）
• web/src/components/EquityChart.tsx: 2 个 SWR 调用（history, account）

现在所有 API 调用都会等待用户认证完成后才发起请求。

请验证

请在最新的 z-dev 分支上测试：

1. 清除浏览器缓存并刷新: Ctrl+Shift+R (或 Mac 上 Cmd+Shift+R)
2. 检查余额、总盈亏等数据是否正确显示
3. 打开浏览器 DevTools (F12) → Network 标签页
   • 确认不再有 Bearer null 错误
   • 确认 API 返回正确的数据

如果问题依然存在

如果验证后问题仍然存在，请提供：

1. 浏览器控制台错误截图（F12 → Console）
2. Network 标签页中失败的 API 请求截图
3. 具体哪些数据显示错误（余额？盈亏？其他？）

这样我们可以进一步排查是否还有其他问题。

🤖 Generated with Claude Code