From 3daeec96aba77b43bc04543efb9a1d8c18bc3609 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E7=A8=8B=E6=99=BA=E8=B6=850668000959?= Date: Tue, 9 Jun 2026 09:48:52 +0800 Subject: [PATCH] fix(seahorse): explicitly ignore Close() errors on PRAGMA/migration failure paths --- pkg/seahorse/short_engine.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/pkg/seahorse/short_engine.go b/pkg/seahorse/short_engine.go index ef493bd1..ddada0e9 100644 --- a/pkg/seahorse/short_engine.go +++ b/pkg/seahorse/short_engine.go @@ -113,20 +113,20 @@ func NewEngine(config Config, completeFn CompleteFn) (*Engine, error) { // Configure SQLite for concurrent access if _, err := db.Exec("PRAGMA journal_mode = WAL;"); err != nil { - db.Close() + _ = db.Close() return nil, fmt.Errorf("enable WAL: %w", err) } if _, err := db.Exec("PRAGMA busy_timeout = 5000;"); err != nil { - db.Close() + _ = db.Close() return nil, fmt.Errorf("set busy_timeout: %w", err) } if _, err := db.Exec("PRAGMA synchronous = NORMAL;"); err != nil { - db.Close() + _ = db.Close() return nil, fmt.Errorf("set synchronous: %w", err) } if err := runSchema(db); err != nil { - db.Close() + _ = db.Close() return nil, fmt.Errorf("migrations: %w", err) }