From c56fc6189c56d6072cf7f6644a5d1f935535842e Mon Sep 17 00:00:00 2001
From: Tobias Gesellchen <tobias@gesellix.de>
Date: Sat, 5 Sep 2026 14:09:47 +0200
Subject: [PATCH] refactor(tunein): consolidate the Items/body fallback into
 one helper

TuneInSearch, TuneInSearchNext, and TuneInNavigateProfile each
duplicated the identical "Items" (v1.3)-then-"body" (legacy) fallback.
Extracted tuneInRawItems, used by all three. Pure refactor, no
behavior change.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
---
 pkg/service/bmx/tunein.go | 29 +++++++++++++++--------------
 1 file changed, 15 insertions(+), 14 deletions(-)

diff --git a/pkg/service/bmx/tunein.go b/pkg/service/bmx/tunein.go
index 684a7d03..0e83d365 100644
--- a/pkg/service/bmx/tunein.go
+++ b/pkg/service/bmx/tunein.go
@@ -99,6 +99,18 @@ func isTuneInOpmlURI(rawURL string) bool {
 	return strings.EqualFold(u.Hostname(), "opml.radiotime.com")
 }
 
+// tuneInRawItems extracts a response's item list, trying "Items" (the
+// v1.3 search/profiles API shape) first, then falling back to "body"
+// (the legacy/OPML shape).
+func tuneInRawItems(data map[string]interface{}) []interface{} {
+	items, ok := data["Items"].([]interface{})
+	if !ok {
+		items, _ = data["body"].([]interface{})
+	}
+
+	return items
+}
+
 // tuneInRenderJSONURI returns the URL with render=json set as a query parameter,
 // replacing any existing render value instead of appending a duplicate.
 func tuneInRenderJSONURI(rawURL string) string {
@@ -350,11 +362,7 @@ func TuneInSearch(query string) (*models.BmxNavResponse, error) {
 		Layout: "classic",
 	}
 
-	// Try "Items" (v1.3) first, then "body" (legacy)
-	items, ok := data["Items"].([]interface{})
-	if !ok {
-		items, _ = data["body"].([]interface{})
-	}
+	items := tuneInRawItems(data)
 
 	for idx, item := range items {
 		m, ok := item.(map[string]interface{})
@@ -479,10 +487,7 @@ func TuneInSearchNext(encodedCursor string) (*models.BmxNavResponse, error) {
 		return nil, err
 	}
 
-	rawItems, ok := data["Items"].([]interface{})
-	if !ok {
-		rawItems, _ = data["body"].([]interface{})
-	}
+	rawItems := tuneInRawItems(data)
 
 	navItems := make([]models.BmxNavItem, 0, len(rawItems))
 	for _, raw := range rawItems {
@@ -662,11 +667,7 @@ func TuneInNavigateProfile(encodedURI string) (*models.BmxNavResponse, error) {
 			return nil, err
 		}
 
-		// "Items" (v1.3 profiles/search API) first, then "body" (legacy/OPML).
-		rawItems, ok := contents["Items"].([]interface{})
-		if !ok {
-			rawItems, _ = contents["body"].([]interface{})
-		}
+		rawItems := tuneInRawItems(contents)
 
 		// The Contents pivot doesn't return playable items directly: each
 		// top-level entry is a "Container" (identified by a "ContainerType"