fix: ensure migration transaction is commited or rolled back

internal/database/pg.go

@@ -25,42 +25,44 @@ var postgresMigrations = []migration{
 		if err != nil {
 			return fmt.Errorf("failed to start transaction: %w", err)
 		}
-		defer tx.Rollback()
 
 		_, err = tx.Exec(`ALTER TABLE bookmark ADD COLUMN has_content BOOLEAN DEFAULT FALSE NOT NULL`)
 		if err != nil {
 			// Check if this is a "column already exists" error (PostgreSQL error code 42701)
+			// If it's not, return error.
+			// This is needed for users upgrading from >1.5.4 directly into this version.
 			pqErr, ok := err.(*pq.Error)
 			if ok && pqErr.Code == "42701" {
 				tx.Rollback()
 			} else {
 				return fmt.Errorf("failed to add has_content column to bookmark table: %w", err)
 			}
-		}
-
-		if errCommit := tx.Commit(); errCommit != nil {
-			return fmt.Errorf("failed to commit transaction: %w", errCommit)
+		} else {
+			if err := tx.Commit(); err != nil {
+				return fmt.Errorf("failed to commit transaction: %w", err)
+			}
 		}
 
 		tx, err = db.Begin()
 		if err != nil {
 			return fmt.Errorf("failed to start transaction: %w", err)
 		}
-		defer tx.Rollback()
 
 		_, err = tx.Exec(`ALTER TABLE account ADD COLUMN config JSONB NOT NULL DEFAULT '{}'`)
 		if err != nil {
 			// Check if this is a "column already exists" error (PostgreSQL error code 42701)
+			// If it's not, return error
+			// This is needed for users upgrading from >1.5.4 directly into this version.
 			pqErr, ok := err.(*pq.Error)
 			if ok && pqErr.Code == "42701" {
 				tx.Rollback()
 			} else {
 				return fmt.Errorf("failed to add config column to account table: %w", err)
 			}
-		}
-
-		if errCommit := tx.Commit(); errCommit != nil {
-			return fmt.Errorf("failed to commit transaction: %w", errCommit)
+		} else {
+			if err := tx.Commit(); err != nil {
+				return fmt.Errorf("failed to commit transaction: %w", err)
+			}
 		}
 
 		return nil