From 0774f5f55fb91230504fd8f3fc19de2011b2502d Mon Sep 17 00:00:00 2001 From: Bryan Van Deusen Date: Wed, 22 Jul 2026 23:16:17 -0400 Subject: [PATCH] =?UTF-8?q?fix(player):=20suppress=20TooManyFunctions=20on?= =?UTF-8?q?=20PlayerViewModel=20facade=20=E2=80=94=20#1944?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Adding the queue move/remove/clear pass-throughs pushed the VM to 12 functions (detekt cap 11). It's a thin transport facade forwarding to PlayerController, so suppress with a rationale rather than splitting the delegating surface. Co-Authored-By: Claude Opus 4.8 (1M context) --- .../java/com/fabledsword/minstrel/player/ui/PlayerViewModel.kt | 1 + 1 file changed, 1 insertion(+) diff --git a/android/app/src/main/java/com/fabledsword/minstrel/player/ui/PlayerViewModel.kt b/android/app/src/main/java/com/fabledsword/minstrel/player/ui/PlayerViewModel.kt index 674728f8..1f7810ce 100644 --- a/android/app/src/main/java/com/fabledsword/minstrel/player/ui/PlayerViewModel.kt +++ b/android/app/src/main/java/com/fabledsword/minstrel/player/ui/PlayerViewModel.kt @@ -25,6 +25,7 @@ import javax.inject.Inject * stub-test for ViewModel-level logic when it grows). */ @HiltViewModel +@Suppress("TooManyFunctions") // Thin transport facade — each fun forwards to PlayerController. class PlayerViewModel @Inject constructor( private val controller: PlayerController, private val likes: LikesRepository,