try to fix xerafin import bug

Author: domino14

internal/wordvault/leitner_converter.go

@@ -6,6 +6,7 @@ import (
 	"encoding/json"
 	"errors"
 	"fmt"
+	"strconv"
 	"time"
 
 	"github.com/jackc/pgx/v5/pgtype"
@@ -74,9 +75,9 @@ func LeitnerImport(ctx context.Context, searchServer *searchserver.Server, lexic
 			correct       int
 			incorrect     int
 			streak        int
-			lastCorrect   sql.NullInt32
+			lastCorrect   interface{}
 			cardbox       sql.NullInt32
-			nextScheduled sql.NullInt32
+			nextScheduled interface{}
 		)
 
 		if err := rows.Scan(&question, &correct, &incorrect, &streak, &lastCorrect, &cardbox, &nextScheduled); err != nil {
@@ -152,16 +153,54 @@ func LeitnerImport(ctx context.Context, searchServer *searchserver.Server, lexic
 	return totalInserted, unimportedAlphagrams, nil
 
 }
-func convertLeitnerToFsrs(correct, incorrect, streak int, lastCorrect, nextScheduled, cardbox sql.NullInt32,
+func convertLeitnerToFsrs(correct, incorrect, streak int, lastCorrect, nextScheduled interface{}, cardbox sql.NullInt32,
 	now time.Time) (stores.Card, []stores.ReviewLog, pgtype.Timestamptz) {
 
-	lc := lastCorrect.Int32
-	if !lastCorrect.Valid {
-		lc = 0
+	// Convert lastCorrect from interface{} to int64
+	var lc int64
+	if lastCorrect != nil {
+		switch v := lastCorrect.(type) {
+		case int64:
+			lc = v
+		case int32:
+			lc = int64(v)
+		case float64:
+			lc = int64(v)
+		case string:
+			// Try to parse as float first (handles scientific notation)
+			if f, err := strconv.ParseFloat(v, 64); err == nil {
+				lc = int64(f)
+			} else if i, err := strconv.ParseInt(v, 10, 64); err == nil {
+				lc = i
+			}
+		}
 	}
 
-	nextScheduledTime := time.Unix(int64(nextScheduled.Int32), 0).UTC()
-	if !nextScheduled.Valid || nextScheduled.Int32 == 0 {
+	// Convert nextScheduled from interface{} to time
+	var nextScheduledTime time.Time
+	if nextScheduled != nil {
+		var ts int64
+		switch v := nextScheduled.(type) {
+		case int64:
+			ts = v
+		case int32:
+			ts = int64(v)
+		case float64:
+			ts = int64(v)
+		case string:
+			// Try to parse as float first (handles scientific notation)
+			if f, err := strconv.ParseFloat(v, 64); err == nil {
+				ts = int64(f)
+			} else if i, err := strconv.ParseInt(v, 10, 64); err == nil {
+				ts = i
+			}
+		}
+		if ts > 0 {
+			nextScheduledTime = time.Unix(ts, 0).UTC()
+		} else {
+			nextScheduledTime = now
+		}
+	} else {
 		nextScheduledTime = now
 	}
 
@@ -170,7 +209,7 @@ func convertLeitnerToFsrs(correct, incorrect, streak int, lastCorrect, nextSched
 		cb = -1
 	}
 
-	lastCorrectTime := time.Unix(int64(lc), 0).UTC()
+	lastCorrectTime := time.Unix(lc, 0).UTC()
 	reviewLog := []stores.ReviewLog{{
 		ImportLog: &stores.ImportLog{
 			ImportedDate:    now,

internal/wordvault/leitner_converter_test.go

@@ -15,8 +15,8 @@ func TestConvertToFSRS(t *testing.T) {
 	is := is.New(t)
 	// Convert a card we've never missed.
 	now := time.Date(2024, 11, 14, 3, 4, 5, 0, time.UTC)
-	card, revLog, _ := convertLeitnerToFsrs(21, 0, 21, sql.NullInt32{Int32: 1712380948, Valid: true},
-		sql.NullInt32{Int32: 1721117219, Valid: true},
+	card, revLog, _ := convertLeitnerToFsrs(21, 0, 21, int64(1712380948),
+		int64(1721117219),
 		sql.NullInt32{Int32: 7, Valid: true},
 		now)
 
@@ -38,8 +38,8 @@ func TestConvertToFSRS(t *testing.T) {
 	// Convert a card that was last correct after it was due. This can happen
 	// if you quiz on it outside of cardbox.
 	card, revLog, _ = convertLeitnerToFsrs(2, 2, 1,
-		sql.NullInt32{Int32: 1730935552, Valid: true},
-		sql.NullInt32{Int32: 1730330752, Valid: true},
+		int64(1730935552),
+		int64(1730330752),
 		sql.NullInt32{Int32: 1, Valid: true},
 		now)
 	// stability should be handwaved to a small number
@@ -49,4 +49,12 @@ func TestConvertToFSRS(t *testing.T) {
 	newCard = schedulingCards[rating].Card
 	is.True(!math.IsNaN(newCard.Stability))
 	fmt.Println("newCard", newCard.Stability, newCard.Due, newCard.Difficulty)
+
+	// Test handling of scientific notation string (the bug we're fixing)
+	card, _, _ = convertLeitnerToFsrs(5, 1, 4,
+		"1712380948.0",
+		"1.756833312857143e+09", // This is the problematic value from the issue
+		sql.NullInt32{Int32: 3, Valid: true},
+		now)
+	is.True(card.Stability > 0) // Should parse correctly and not crash
 }