diff --git a/.gitignore b/.gitignore index 04927700..80e2a6d7 100644 --- a/.gitignore +++ b/.gitignore @@ -5,6 +5,7 @@ # AI 工具 .claude/ +CLAUDE.md # 编译产物 nofx-auto @@ -29,7 +30,8 @@ Thumbs.db # 环境变量 .env config.json -config.db +config.db* +nofx.db configbak.json # 决策日志 @@ -61,4 +63,4 @@ rsa_key* # 加密相关 DATA_ENCRYPTION_KEY=* -*.enc \ No newline at end of file +*.enc diff --git a/api/server.go b/api/server.go index 543fb0a6..bbe71144 100644 --- a/api/server.go +++ b/api/server.go @@ -1,6 +1,7 @@ package api import ( + "context" "encoding/json" "fmt" "log" @@ -24,6 +25,7 @@ import ( // Server HTTP API服务器 type Server struct { router *gin.Engine + httpServer *http.Server traderManager *manager.TraderManager database *config.Database cryptoHandler *CryptoHandler @@ -2032,7 +2034,26 @@ func (s *Server) Start() error { log.Printf(" • GET /api/performance?trader_id=xxx - 指定trader的AI学习表现分析") log.Println() - return s.router.Run(addr) + // 创建 http.Server 以支持 graceful shutdown + s.httpServer = &http.Server{ + Addr: addr, + Handler: s.router, + } + + return s.httpServer.ListenAndServe() +} + +// Shutdown 优雅关闭 API 服务器 +func (s *Server) Shutdown() error { + if s.httpServer == nil { + return nil + } + + // 设置 5 秒超时 + ctx, cancel := context.WithTimeout(context.Background(), 5*time.Second) + defer cancel() + + return s.httpServer.Shutdown(ctx) } // handleGetPromptTemplates 获取所有系统提示词模板列表 diff --git a/config/database.go b/config/database.go index 2d756fb5..abbf8fd7 100644 --- a/config/database.go +++ b/config/database.go @@ -65,6 +65,24 @@ func NewDatabase(dbPath string) (*Database, error) { return nil, fmt.Errorf("打开数据库失败: %w", err) } + // 🔒 启用 WAL 模式,提高并发性能和崩溃恢复能力 + // WAL (Write-Ahead Logging) 模式的优势: + // 1. 更好的并发性能:读操作不会被写操作阻塞 + // 2. 崩溃安全:即使在断电或强制终止时也能保证数据完整性 + // 3. 更快的写入:不需要每次都写入主数据库文件 + if _, err := db.Exec("PRAGMA journal_mode=WAL"); err != nil { + db.Close() + return nil, fmt.Errorf("启用WAL模式失败: %w", err) + } + + // 🔒 设置 synchronous=FULL 确保数据持久性 + // FULL (2) 模式: 确保数据在关键时刻完全写入磁盘 + // 配合 WAL 模式,在保证数据安全的同时获得良好性能 + if _, err := db.Exec("PRAGMA synchronous=FULL"); err != nil { + db.Close() + return nil, fmt.Errorf("设置synchronous失败: %w", err) + } + database := &Database{db: db} if err := database.createTables(); err != nil { return nil, fmt.Errorf("创建表失败: %w", err) @@ -74,6 +92,7 @@ func NewDatabase(dbPath string) (*Database, error) { return nil, fmt.Errorf("初始化默认数据失败: %w", err) } + log.Printf("✅ 数据库已启用 WAL 模式和 FULL 同步,数据持久性得到保证") return database, nil } diff --git a/config/database_test.go b/config/database_test.go index c9d40521..11655bca 100644 --- a/config/database_test.go +++ b/config/database_test.go @@ -4,6 +4,7 @@ import ( "nofx/crypto" "os" "testing" + "time" ) // TestUpdateExchange_EmptyValuesShouldNotOverwrite 测试空值不应覆盖现有数据 @@ -587,3 +588,213 @@ func setupTestDB(t *testing.T) (*Database, func()) { return db, cleanup } + +// TestWALModeEnabled 测试 WAL 模式是否启用 +// TDD: 这个测试应该失败，因为当前代码没有启用 WAL 模式 +func TestWALModeEnabled(t *testing.T) { + db, cleanup := setupTestDB(t) + defer cleanup() + + // 查询当前的 journal_mode + var journalMode string + err := db.db.QueryRow("PRAGMA journal_mode").Scan(&journalMode) + if err != nil { + t.Fatalf("查询 journal_mode 失败: %v", err) + } + + // 期望是 WAL 模式 + if journalMode != "wal" { + t.Errorf("期望 journal_mode=wal，实际是 %s", journalMode) + } +} + +// TestSynchronousMode 测试 synchronous 模式设置 +// TDD: 验证数据持久性设置 +func TestSynchronousMode(t *testing.T) { + db, cleanup := setupTestDB(t) + defer cleanup() + + // 查询 synchronous 设置 + var synchronous int + err := db.db.QueryRow("PRAGMA synchronous").Scan(&synchronous) + if err != nil { + t.Fatalf("查询 synchronous 失败: %v", err) + } + + // 期望是 FULL (2) 以确保数据持久性 + if synchronous != 2 { + t.Errorf("期望 synchronous=2 (FULL)，实际是 %d", synchronous) + } +} + +// TestDataPersistenceAcrossReopen 测试数据在数据库关闭并重新打开后是否持久化 +// TDD: 模拟 Docker restart 场景 +func TestDataPersistenceAcrossReopen(t *testing.T) { + // 创建临时数据库文件 + tmpFile, err := os.CreateTemp("", "test_persistence_*.db") + if err != nil { + t.Fatalf("创建临时文件失败: %v", err) + } + tmpFile.Close() + dbPath := tmpFile.Name() + defer os.Remove(dbPath) + + // 设置加密服务 + rsaKeyPath := "test_rsa_key.pem" + cryptoService, err := crypto.NewCryptoService(rsaKeyPath) + if err != nil { + t.Fatalf("初始化加密服务失败: %v", err) + } + defer os.RemoveAll(rsaKeyPath) + + userID := "test-user-persistence" + testAPIKey := "test-api-key-should-persist" + testSecretKey := "test-secret-key-should-persist" + + // 第一次打开数据库并写入数据 + { + db, err := NewDatabase(dbPath) + if err != nil { + t.Fatalf("第一次创建数据库失败: %v", err) + } + db.SetCryptoService(cryptoService) + + // 写入交易所配置 + err = db.UpdateExchange( + userID, + "binance", + true, + testAPIKey, + testSecretKey, + false, + "", + "", + "", + "", + ) + if err != nil { + t.Fatalf("写入数据失败: %v", err) + } + + // 模拟正常关闭 + if err := db.Close(); err != nil { + t.Fatalf("关闭数据库失败: %v", err) + } + } + + // 第二次打开数据库并验证数据是否还在 + { + db, err := NewDatabase(dbPath) + if err != nil { + t.Fatalf("第二次打开数据库失败: %v", err) + } + db.SetCryptoService(cryptoService) + defer db.Close() + + // 读取数据 + exchanges, err := db.GetExchanges(userID) + if err != nil { + t.Fatalf("读取数据失败: %v", err) + } + + if len(exchanges) == 0 { + t.Fatal("数据丢失：没有找到任何交易所配置") + } + + // 验证数据完整性 + found := false + for _, ex := range exchanges { + if ex.ID == "binance" { + found = true + if ex.APIKey != testAPIKey { + t.Errorf("API Key 丢失或损坏，期望 %s，实际 %s", testAPIKey, ex.APIKey) + } + if ex.SecretKey != testSecretKey { + t.Errorf("Secret Key 丢失或损坏，期望 %s，实际 %s", testSecretKey, ex.SecretKey) + } + } + } + + if !found { + t.Error("数据丢失：找不到 binance 配置") + } + } +} + +// TestConcurrentWritesWithWAL 测试 WAL 模式下的并发写入 +// TDD: WAL 模式应该支持更好的并发性能 +func TestConcurrentWritesWithWAL(t *testing.T) { + db, cleanup := setupTestDB(t) + defer cleanup() + + // 这个测试验证多个并发写入可以成功 + // WAL 模式下并发性能更好,但 SQLite 仍然可能出现短暂的锁 + done := make(chan bool, 2) + errors := make(chan error, 10) + + // 并发写入1 + go func() { + for i := 0; i < 3; i++ { + err := db.UpdateExchange( + "user1", + "binance", + true, + "key1", + "secret1", + false, + "", + "", + "", + "", + ) + if err != nil { + errors <- err + } + // 小延迟减少锁冲突 + time.Sleep(10 * time.Millisecond) + } + done <- true + }() + + // 并发写入2 + go func() { + for i := 0; i < 3; i++ { + err := db.UpdateExchange( + "user2", + "hyperliquid", + true, + "key2", + "secret2", + false, + "0xWallet", + "", + "", + "", + ) + if err != nil { + errors <- err + } + // 小延迟减少锁冲突 + time.Sleep(10 * time.Millisecond) + } + done <- true + }() + + // 等待两个 goroutine 完成 + <-done + <-done + close(errors) + + // 检查是否有错误 + errorCount := 0 + for err := range errors { + t.Logf("并发写入错误: %v", err) + errorCount++ + } + + // WAL 模式下应该能处理并发,但可能有少量锁错误 + // 我们允许最多 2 个错误 + if errorCount > 2 { + t.Errorf("并发写入失败次数过多: %d", errorCount) + } +} diff --git a/docker-compose.yml b/docker-compose.yml index b85c9d3f..e83be07e 100644 --- a/docker-compose.yml +++ b/docker-compose.yml @@ -6,6 +6,7 @@ services: dockerfile: ./docker/Dockerfile.backend container_name: nofx-trading restart: unless-stopped + stop_grace_period: 30s # 允许应用有 30 秒时间优雅关闭 ports: - "${NOFX_BACKEND_PORT:-8080}:8080" volumes: diff --git a/main.go b/main.go index b8b1ed75..2fb4e83d 100644 --- a/main.go +++ b/main.go @@ -359,8 +359,28 @@ func main() { <-sigChan fmt.Println() fmt.Println() - log.Println("📛 收到退出信号，正在停止所有trader...") + log.Println("📛 收到退出信号，正在优雅关闭...") + + // 步骤 1: 停止所有交易员 + log.Println("⏸️ 停止所有交易员...") traderManager.StopAll() + log.Println("✅ 所有交易员已停止") + + // 步骤 2: 关闭 API 服务器 + log.Println("🛑 停止 API 服务器...") + if err := apiServer.Shutdown(); err != nil { + log.Printf("⚠️ 关闭 API 服务器时出错: %v", err) + } else { + log.Println("✅ API 服务器已安全关闭") + } + + // 步骤 3: 关闭数据库连接 (确保所有写入完成) + log.Println("💾 关闭数据库连接...") + if err := database.Close(); err != nil { + log.Printf("❌ 关闭数据库失败: %v", err) + } else { + log.Println("✅ 数据库已安全关闭，所有数据已持久化") + } fmt.Println() fmt.Println("👋 感谢使用AI交易系统！")