.golangci.yml

@@ -7,6 +7,7 @@ linters:
     - asciicheck
     - bidichk
     - bodyclose
+    - depguard
     - dogsled
     - durationcheck
     - errcheck

core/media_streamer.go

@@ -129,11 +129,11 @@ func (s *Stream) EstimatedContentLength() int {
 func selectTranscodingOptions(ctx context.Context, ds model.DataStore, mf *model.MediaFile, reqFormat string, reqBitRate int) (format string, bitRate int) {
 	format = "raw"
 	if reqFormat == "raw" {
-		return format, 0
+		return
 	}
 	if reqFormat == mf.Suffix && reqBitRate == 0 {
 		bitRate = mf.BitRate
-		return format, bitRate
+		return
 	}
 	trc, hasDefault := request.TranscodingFrom(ctx)
 	var cFormat string
@@ -159,7 +159,7 @@ func selectTranscodingOptions(ctx context.Context, ds model.DataStore, mf *model
 		cBitRate = reqBitRate
 	}
 	if cBitRate == 0 && cFormat == "" {
-		return format, bitRate
+		return
 	}
 	t, err := ds.Transcoding(ctx).FindByFormat(cFormat)
 	if err == nil {

persistence/playlist_track_repository.go

@@ -5,7 +5,7 @@ import (
 	"github.com/deluan/rest"
 	"github.com/navidrome/navidrome/log"
 	"github.com/navidrome/navidrome/model"
-	"github.com/navidrome/navidrome/utils/slice"
+	"github.com/navidrome/navidrome/utils"
 )
 
 type playlistTrackRepository struct {
@@ -196,7 +196,7 @@ func (r *playlistTrackRepository) Reorder(pos int, newPos int) error {
 	if err != nil {
 		return err
 	}
-	newOrder := slice.Move(ids, pos-1, newPos-1)
+	newOrder := utils.MoveString(ids, pos-1, newPos-1)
 	return r.playlistRepo.updatePlaylist(r.playlistId, newOrder)
 }
 

utils/slice/slice.go

@@ -41,16 +41,3 @@ func MostFrequent[T comparable](list []T) T {
 
 	return topItem
 }
-
-func Insert[T any](slice []T, value T, index int) []T {
-	return append(slice[:index], append([]T{value}, slice[index:]...)...)
-}
-
-func Remove[T any](slice []T, index int) []T {
-	return append(slice[:index], slice[index+1:]...)
-}
-
-func Move[T any](slice []T, srcIndex int, dstIndex int) []T {
-	value := slice[srcIndex]
-	return Insert(Remove(slice, srcIndex), value, dstIndex)
-}

utils/slice/slice_test.go

@@ -14,8 +14,7 @@ func TestSlice(t *testing.T) {
 	RunSpecs(t, "Slice Suite")
 }
 
-var _ = Describe("Slice Utils", func() {
-	Describe("Map", func() {
+var _ = Describe("Map", func() {
 	It("returns empty slice for an empty input", func() {
 		mapFunc := func(v int) string { return strconv.Itoa(v * 2) }
 		result := slice.Map([]int{}, mapFunc)
@@ -27,9 +26,9 @@ var _ = Describe("Slice Utils", func() {
 		result := slice.Map([]int{1, 2, 3, 4}, mapFunc)
 		Expect(result).To(ConsistOf("2", "4", "6", "8"))
 	})
-	})
+})
 
-	Describe("Group", func() {
+var _ = Describe("Group", func() {
 	It("returns empty map for an empty input", func() {
 		keyFunc := func(v int) int { return v % 2 }
 		result := slice.Group([]int{}, keyFunc)
@@ -43,9 +42,9 @@ var _ = Describe("Slice Utils", func() {
 		Expect(result[0]).To(ConsistOf(2, 4, 6, 8, 10))
 		Expect(result[1]).To(ConsistOf(1, 3, 5, 7, 9, 11))
 	})
-	})
+})
 
-	Describe("MostFrequent", func() {
+var _ = Describe("MostFrequent", func() {
 	It("returns zero value if no arguments are passed", func() {
 		Expect(slice.MostFrequent([]int{})).To(BeZero())
 	})
@@ -56,17 +55,4 @@ var _ = Describe("Slice Utils", func() {
 	It("returns the item that appeared more times", func() {
 		Expect(slice.MostFrequent([]string{"1", "2", "1", "2", "3", "2"})).To(Equal("2"))
 	})
-	})
-
-	Describe("Move", func() {
-		It("moves item to end of slice", func() {
-			Expect(slice.Move([]string{"1", "2", "3"}, 0, 2)).To(ConsistOf("2", "3", "1"))
-		})
-		It("moves item to beginning of slice", func() {
-			Expect(slice.Move([]string{"1", "2", "3"}, 2, 0)).To(ConsistOf("3", "1", "2"))
-		})
-		It("keeps item in same position if srcIndex == dstIndex", func() {
-			Expect(slice.Move([]string{"1", "2", "3"}, 1, 1)).To(ConsistOf("1", "2", "3"))
-		})
-	})
 })

utils/strings.go

@@ -17,6 +17,19 @@ func NoArticle(name string) string {
 	return name
 }
 
+func InsertString(slice []string, value string, index int) []string {
+	return append(slice[:index], append([]string{value}, slice[index:]...)...)
+}
+
+func RemoveString(slice []string, index int) []string {
+	return append(slice[:index], slice[index+1:]...)
+}
+
+func MoveString(slice []string, srcIndex int, dstIndex int) []string {
+	value := slice[srcIndex]
+	return InsertString(RemoveString(slice, srcIndex), value, dstIndex)
+}
+
 func BreakUpStringSlice(items []string, chunkSize int) [][]string {
 	numTracks := len(items)
 	var chunks [][]string

utils/strings_test.go

@@ -35,6 +35,18 @@ var _ = Describe("Strings", func() {
 		})
 	})
 
+	Describe("MoveString", func() {
+		It("moves item to end of slice", func() {
+			Expect(MoveString([]string{"1", "2", "3"}, 0, 2)).To(ConsistOf("2", "3", "1"))
+		})
+		It("moves item to beginning of slice", func() {
+			Expect(MoveString([]string{"1", "2", "3"}, 2, 0)).To(ConsistOf("3", "1", "2"))
+		})
+		It("keeps item in same position if srcIndex == dstIndex", func() {
+			Expect(MoveString([]string{"1", "2", "3"}, 1, 1)).To(ConsistOf("1", "2", "3"))
+		})
+	})
+
 	Describe("BreakUpStringSlice", func() {
 		It("returns no chunks if slice is empty", func() {
 			var slice []string