replacing log.Fatalf() by returning an error (#201)

Author: digitalghost-dev

cmd/card/card.go

@@ -54,7 +54,7 @@ func CardCommand() (string, error) {
 
 	if result.SeriesID != "" {
 		// Program 2: Sets selection
-		setsModel := SetsList(result.SeriesID)
+		setsModel, _ := SetsList(result.SeriesID)
 		finalSetsModel, err := tea.NewProgram(setsModel).Run()
 		if err != nil {
 			return "", fmt.Errorf("error running sets selection program: %w", err)

cmd/card/setslist.go

@@ -4,7 +4,6 @@ import (
 	"encoding/json"
 	"fmt"
 	"io"
-	"log"
 	"net/http"
 	"time"
 
@@ -72,12 +71,16 @@ type setData struct {
 	Symbol            string `json:"symbol"`
 }
 
-func SetsList(seriesID string) SetsModel {
-	body, _ := callSetsData("https://uoddayfnfkebrijlpfbh.supabase.co/rest/v1/sets")
+func SetsList(seriesID string) (SetsModel, error) {
+	body, err := callSetsData("https://uoddayfnfkebrijlpfbh.supabase.co/rest/v1/sets")
+	if err != nil {
+		return SetsModel{}, fmt.Errorf("error getting sets data: %v", err)
+	}
 	var allSets []setData
-	err := json.Unmarshal(body, &allSets)
+
+	err = json.Unmarshal(body, &allSets)
 	if err != nil {
-		log.Fatal(err)
+		return SetsModel{}, fmt.Errorf("error parsing sets data: %v", err)
 	}
 
 	// Filter sets by series_id and build ID map
@@ -102,10 +105,11 @@ func SetsList(seriesID string) SetsModel {
 	l.Styles.HelpStyle = helpStyle
 
 	return SetsModel{
-		List:       l,
-		SeriesName: seriesID,
-		setsIDMap:  setsIDMap,
-	}
+			List:       l,
+			SeriesName: seriesID,
+			setsIDMap:  setsIDMap,
+		},
+		nil
 }
 
 func callSetsData(url string) ([]byte, error) {