refactor: Improve error handling for response body closure and file removal in downloader and player

Author: alvarorichard

internal/api/enhanced.go

@@ -382,7 +382,7 @@ func sanitizeFilename(name string) string {
 }
 
 // Basic download function (placeholder - integrate with your existing downloader)
-func downloadFromURL(url, filename string) error {
+func downloadFromURL(_url, _filename string) error {
 	// This is a placeholder that should fail to trigger fallback to the proper downloader
 	util.Debugf("Enhanced API downloadFromURL is a placeholder - returning error to trigger fallback")
 	return fmt.Errorf("enhanced download not implemented - use legacy downloader")

internal/downloader/downloader.go

@@ -468,7 +468,11 @@ func (d *EpisodeDownloader) estimateContentLengthForAllAnime(url string, client
 		util.Debugf("Range request failed, using default size estimate")
 		return 300 * 1024 * 1024, nil // 300MB default
 	}
-	defer resp.Body.Close()
+	defer func() {
+		if closeErr := resp.Body.Close(); closeErr != nil {
+			util.Warnf("Failed to close response body: %v", closeErr)
+		}
+	}()
 
 	// Check Content-Range header for total size
 	contentRange := resp.Header.Get("Content-Range")

internal/player/player.go

@@ -441,7 +441,9 @@ func downloadAndPlayEpisode(
 		if stat, err := os.Stat(episodePath); err == nil {
 			if stat.Size() < 1024 {
 				fmt.Println("File is too small, re-downloading...")
-				os.Remove(episodePath) // Remove invalid file
+				if removeErr := os.Remove(episodePath); removeErr != nil {
+					util.Warnf("Failed to remove invalid file: %v", removeErr)
+				}
 				return downloadAndPlayEpisode(videoURL, episodes, selectedEpisodeNum, animeURL, episodeNumberStr, animeMalID, updater)
 			}
 		}

internal/player/scraper.go

@@ -166,7 +166,11 @@ func estimateContentLengthForAllAnime(url string, client *http.Client) (int64, e
 		util.Debugf("Range request failed, using default size estimate")
 		return 300 * 1024 * 1024, nil // 300MB default
 	}
-	defer resp.Body.Close()
+	defer func() {
+		if closeErr := resp.Body.Close(); closeErr != nil {
+			util.Warnf("Failed to close response body: %v", closeErr)
+		}
+	}()
 
 	// Check Content-Range header for total size
 	contentRange := resp.Header.Get("Content-Range")