fix(android): detekt — suppress SwallowedException + rename LikesWire
- LikesRepository.kt: the catch around best-effort like REST is
intentional swallow (the queue replays later). Added
`@Suppress("SwallowedException")` alongside the existing
`TooGenericExceptionCaught`, plus a comment explaining the choice
so the next reader doesn't "fix" it.
- models/wire/LikesWire.kt → LikedIdsWire.kt: file held a single
top-level declaration (LikedIdsWire); detekt's
MatchingDeclarationName rule wants the filename to match.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -94,8 +94,12 @@ class LikesRepository @Inject constructor(
|
|||||||
if (desiredState) api.like(kindPath, entityId) else api.unlike(kindPath, entityId)
|
if (desiredState) api.like(kindPath, entityId) else api.unlike(kindPath, entityId)
|
||||||
true
|
true
|
||||||
} catch (
|
} catch (
|
||||||
@Suppress("TooGenericExceptionCaught") e: Throwable,
|
@Suppress("TooGenericExceptionCaught", "SwallowedException") e: Throwable,
|
||||||
) {
|
) {
|
||||||
|
// Intentional swallow: the queue replays this later; surfacing
|
||||||
|
// the network error here would force callers (UI) to handle it
|
||||||
|
// even though it's already been buffered. The replayer logs
|
||||||
|
// per-attempt diagnostics when it lands in Phase 12.2.
|
||||||
mutationQueue.enqueueLikeToggle(entityType, entityId, desiredState)
|
mutationQueue.enqueueLikeToggle(entityType, entityId, desiredState)
|
||||||
false
|
false
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user