From c2f9bbaff451249e86b720cf887a76c59a7e639d Mon Sep 17 00:00:00 2001 From: Bryan Van Deusen Date: Thu, 11 Jun 2026 22:19:13 -0400 Subject: [PATCH] fix(android): suppress detekt TooManyFunctions on HomeRepository MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The two new you-might-like observe accessors pushed HomeRepository from 11 to 13 functions, tripping detekt's per-class default. It's accessor density (one observe method per Home row) on a thin pass-through repository, not complexity — suppressed with a one-line rationale per the project convention. ktlint already passed; this was the only detekt finding. Co-Authored-By: Claude Opus 4.8 (1M context) --- .../java/com/fabledsword/minstrel/home/data/HomeRepository.kt | 3 +++ 1 file changed, 3 insertions(+) diff --git a/android/app/src/main/java/com/fabledsword/minstrel/home/data/HomeRepository.kt b/android/app/src/main/java/com/fabledsword/minstrel/home/data/HomeRepository.kt index 26c54578..25b3908c 100644 --- a/android/app/src/main/java/com/fabledsword/minstrel/home/data/HomeRepository.kt +++ b/android/app/src/main/java/com/fabledsword/minstrel/home/data/HomeRepository.kt @@ -39,6 +39,9 @@ import javax.inject.Singleton * pre-warms the top artists via [HomeArtistPrewarmer]. */ @Singleton +// Per-section observe accessors (one per Home row) inflate the function +// count past detekt's 11 default — accessor density, not complexity. +@Suppress("TooManyFunctions") class HomeRepository @Inject constructor( private val homeIndexDao: CachedHomeIndexDao, private val metadataProvider: MetadataProvider,