fix(android): detekt TooManyFunctions on AuthStore (14/11)
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) <noreply@anthropic.com>
This commit is contained in:
@@ -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,
|
||||
|
||||
Reference in New Issue
Block a user