From a9f4c7df6f6f0c07df828e1687019008835e869c Mon Sep 17 00:00:00 2001 From: Brandon McAnsh Date: Wed, 1 Jul 2026 10:39:34 -0400 Subject: [PATCH] chore(logging): report all errors to Bugsnag, downgrade non-notifiable severity Previously only notifiable errors reached Bugsnag; everything else was dropped. Now all non-noise errors are reported, with severity derived from isNotifiable: WARNING for notifiable (needs attention), INFO for non-notifiable (reference only). Crashes remain ERROR. Slack routing should filter on severity != INFO to preserve today's notifiable-only behavior. Signed-off-by: Brandon McAnsh --- .../app/internal/startup/BugsnagErrorReporter.kt | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/apps/flipcash/app/src/main/kotlin/com/flipcash/app/internal/startup/BugsnagErrorReporter.kt b/apps/flipcash/app/src/main/kotlin/com/flipcash/app/internal/startup/BugsnagErrorReporter.kt index d2a4e6432..e45e57f2a 100644 --- a/apps/flipcash/app/src/main/kotlin/com/flipcash/app/internal/startup/BugsnagErrorReporter.kt +++ b/apps/flipcash/app/src/main/kotlin/com/flipcash/app/internal/startup/BugsnagErrorReporter.kt @@ -1,12 +1,17 @@ package com.flipcash.app.internal.startup import com.bugsnag.android.Bugsnag +import com.bugsnag.android.Severity import com.getcode.utils.ErrorReporter class BugsnagErrorReporter : ErrorReporter { override fun report(error: Throwable, cause: Throwable, isNotifiable: Boolean) { - if (!isNotifiable) return if (!Bugsnag.isStarted()) return - Bugsnag.notify(error) + Bugsnag.notify(error) { event -> + // WARNING = needs active attention (also drives the Slack filter); + // INFO = recorded for reference only, excluded from Slack. + event.severity = if (isNotifiable) Severity.WARNING else Severity.INFO + true + } } }