From 5455cd5d80a3dc7b015b910a05f9e731fc237679 Mon Sep 17 00:00:00 2001 From: Bryan Van Deusen Date: Tue, 26 May 2026 16:53:39 -0400 Subject: [PATCH] fix(android): detekt TooManyFunctions on AuthStore (14/11) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit CacheSettings persistence in b438772 pushed AuthStore from 12 to 14 functions. Same shape of fix as PlayerController (484ad6c-era): @Suppress at the class with rationale — function count scales with the pref count, splitting would scatter shared dao/scope/json plumbing for no gain. Co-Authored-By: Claude Opus 4.7 (1M context) --- .../main/java/com/fabledsword/minstrel/auth/AuthStore.kt | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/android/app/src/main/java/com/fabledsword/minstrel/auth/AuthStore.kt b/android/app/src/main/java/com/fabledsword/minstrel/auth/AuthStore.kt index 7319b662..c4f493f8 100644 --- a/android/app/src/main/java/com/fabledsword/minstrel/auth/AuthStore.kt +++ b/android/app/src/main/java/com/fabledsword/minstrel/auth/AuthStore.kt @@ -28,6 +28,14 @@ import javax.inject.Singleton * sees the new value immediately; the DAO write coroutine catches up * shortly after. */ +// AuthStore is the single-row facade over auth_session (de-facto +// app_preferences — see entity comment). It legitimately owns one +// getter + one setter + one persist helper per pref column; the +// function count scales with the pref count, not with feature +// complexity. Splitting into per-pref stores would scatter the +// shared dao + scope + json plumbing for no real gain. Suppress +// the cap rather than fragment. +@Suppress("TooManyFunctions") @Singleton class AuthStore @Inject constructor( private val dao: AuthSessionDao,