fix: handle is_official type changes in comix provider and fix manga image rendering

Author: justchokingaround

internal/providers/manga/comix/comix.go

@@ -192,7 +192,7 @@ func (c *Comix) GetInfo(id string) (interface{}, error) {
 		ChapterID       int         `json:"chapter_id"`
 		Number          interface{} `json:"number"`
 		Name            string      `json:"name"`
-		IsOfficial      int         `json:"is_official"`
+		IsOfficial      any         `json:"is_official"`
 		ScanlationGroup struct {
 			Name string `json:"name"`
 		} `json:"scanlation_group"`
@@ -219,7 +219,7 @@ func (c *Comix) GetInfo(id string) (interface{}, error) {
 					ChapterID       int         `json:"chapter_id"`
 					Number          interface{} `json:"number"`
 					Name            string      `json:"name"`
-					IsOfficial      int         `json:"is_official"`
+					IsOfficial      any         `json:"is_official"`
 					ScanlationGroup struct {
 						Name string `json:"name"`
 					} `json:"scanlation_group"`
@@ -248,18 +248,28 @@ func (c *Comix) GetInfo(id string) (interface{}, error) {
 		ChapterID       int
 		Number          interface{}
 		Name            string
-		IsOfficial      int
+		IsOfficial      bool
 		ScanlationGroup string
 	}
 	chapterMap := make(map[string][]ChapterItem)
 
 	for _, item := range allChapters {
+		isOfficial := false
+		switch v := item.IsOfficial.(type) {
+		case bool:
+			isOfficial = v
+		case float64:
+			isOfficial = v == 1
+		case int:
+			isOfficial = v == 1
+		}
+
 		numStr := fmt.Sprintf("%v", item.Number)
 		chapterMap[numStr] = append(chapterMap[numStr], ChapterItem{
 			ChapterID:       item.ChapterID,
 			Number:          item.Number,
 			Name:            item.Name,
-			IsOfficial:      item.IsOfficial,
+			IsOfficial:      isOfficial,
 			ScanlationGroup: item.ScanlationGroup.Name,
 		})
 	}
@@ -288,7 +298,7 @@ func (c *Comix) GetInfo(id string) (interface{}, error) {
 
 		// 1. Try Official
 		for _, item := range items {
-			if item.IsOfficial == 1 {
+			if item.IsOfficial {
 				selectedItem = item
 				found = true
 				break

internal/providers/manga/comix/comix_test.go

@@ -0,0 +1,84 @@
+package comix
+
+import (
+	"context"
+	"fmt"
+	"testing"
+
+	"github.com/justchokingaround/greg/internal/providers"
+)
+
+func TestChainsawMan(t *testing.T) {
+	c := New()
+
+	ctx := context.Background()
+
+	// Step 1: Search for Chainsaw Man
+	t.Log("=== Step 1: Searching for Chainsaw Man ===")
+	results, err := c.Search(ctx, "chainsaw man")
+	if err != nil {
+		t.Fatalf("Search failed: %v", err)
+	}
+	if len(results) == 0 {
+		t.Fatal("No results found")
+	}
+
+	var chainsawMan providers.Media
+	for _, r := range results {
+		t.Logf("Found: %s (ID: %s)", r.Title, r.ID)
+		if r.Title == "Chainsaw Man" {
+			chainsawMan = r
+			break
+		}
+	}
+	if chainsawMan.Title == "" {
+		chainsawMan = results[0]
+	}
+	t.Logf("Selected: %s (ID: %s)", chainsawMan.Title, chainsawMan.ID)
+
+	// Step 2: Get manga details (includes chapters)
+	t.Log("=== Step 2: Getting manga details ===")
+	details, err := c.GetMediaDetails(ctx, chainsawMan.ID)
+	if err != nil {
+		t.Fatalf("GetMediaDetails failed: %v", err)
+	}
+	t.Logf("Title: %s", details.Title)
+	t.Logf("Synopsis: %s", details.Synopsis)
+	t.Logf("Status: %s", details.Status)
+	t.Logf("Seasons: %d", len(details.Seasons))
+
+	// Step 3: Get chapters (episodes)
+	t.Log("=== Step 3: Getting chapters ===")
+	episodes, err := c.GetEpisodes(ctx, chainsawMan.ID)
+	if err != nil {
+		t.Fatalf("GetEpisodes failed: %v", err)
+	}
+	t.Logf("Found %d chapters", len(episodes))
+	if len(episodes) == 0 {
+		t.Fatal("No chapters found")
+	}
+
+	// Print first 5 chapters
+	for i, ep := range episodes {
+		if i >= 5 {
+			break
+		}
+		t.Logf("  Chapter %d: %s (ID: %s)", ep.Number, ep.Title, ep.ID)
+	}
+
+	// Step 4: Get manga pages for first chapter
+	t.Log("=== Step 4: Getting manga pages ===")
+	pages, err := c.GetMangaPages(ctx, episodes[0].ID)
+	if err != nil {
+		t.Fatalf("GetMangaPages failed: %v", err)
+	}
+	t.Logf("Found %d pages", len(pages))
+	if len(pages) == 0 {
+		t.Fatal("No pages found")
+	}
+
+	// Print first page URL
+	fmt.Printf("First page URL: %s\n", pages[0])
+
+	t.Log("=== All tests passed! ===")
+}

internal/tui/components/manga/model.go

@@ -580,23 +580,54 @@ func (m *Model) renderPage() tea.Cmd {
 
 	return func() tea.Msg {
 		// Create a temp file
-		tmpFile, err := os.CreateTemp("", "greg-manga-*.jpg")
+		tmpFile, err := os.CreateTemp("", "greg-manga-*.png")
 		if err != nil {
 			return PageRenderedMsg{Err: fmt.Errorf("failed to create temp file: %w", err)}
 		}
-		defer func() { _ = os.Remove(tmpFile.Name()) }()
+		tmpFileName := tmpFile.Name()
+		defer func() { _ = os.Remove(tmpFileName) }()
 
 		// Download the image
-		resp, err := http.Get(url)
+		client := &http.Client{}
+		req, err := http.NewRequest("GET", url, nil)
+		if err != nil {
+			return PageRenderedMsg{Err: fmt.Errorf("failed to create request: %w", err)}
+		}
+		req.Header.Set("User-Agent", "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/120.0.0.0 Safari/537.36")
+		req.Header.Set("Accept", "image/webp,image/apng,image/svg+xml,image/*,*/*;q=0.8")
+		req.Header.Set("Accept-Language", "en-US,en;q=0.9")
+		req.Header.Set("Referer", "https://comix.to/")
+
+		resp, err := client.Do(req)
 		if err != nil {
 			return PageRenderedMsg{Err: fmt.Errorf("failed to download image: %w", err)}
 		}
 		defer func() { _ = resp.Body.Close() }()
 
-		_, err = io.Copy(tmpFile, resp.Body)
+		if resp.StatusCode != http.StatusOK {
+			return PageRenderedMsg{Err: fmt.Errorf("image download returned status %d", resp.StatusCode)}
+		}
+
+		downloadFile, err := os.CreateTemp("", "greg-manga-download-*.img")
+		if err != nil {
+			return PageRenderedMsg{Err: fmt.Errorf("failed to create download temp file: %w", err)}
+		}
+		downloadFileName := downloadFile.Name()
+		defer func() { _ = os.Remove(downloadFileName) }()
+
+		_, err = io.Copy(downloadFile, resp.Body)
 		if err != nil {
 			return PageRenderedMsg{Err: fmt.Errorf("failed to save image: %w", err)}
 		}
+		_ = downloadFile.Close()
+
+		// Convert image to PNG if needed (WebP, AVIF, etc.)
+		convertCmd := exec.Command("magick", downloadFileName, tmpFileName)
+		if err := convertCmd.Run(); err != nil {
+			// If convert fails, try using the original file
+			tmpFileName = downloadFileName
+		}
+
 		_ = tmpFile.Close()
 
 		// Run chafa
@@ -621,7 +652,7 @@ func (m *Model) renderPage() tea.Cmd {
 		// "symbols" or others will use default (no -f flag or specific one if needed, but chafa defaults to symbols)
 
 		// Add file path
-		args = append(args, tmpFile.Name())
+		args = append(args, tmpFileName)
 
 		cmd := exec.Command("chafa", args...)
 		output, err := cmd.Output()