refactor: rename CrashReportService to AcraCrashReporter

- refactor(crash): switch call sites to CrashReportService from :common
- refactor: update imports to use CrashReportService from :common

Assisted-by: Claude Opus 4 (1M context)

Author: criticalAY

AnkiDroid/src/androidTest/java/com/ichi2/anki/tests/ACRATest.kt

@@ -22,14 +22,19 @@ import androidx.core.content.edit
 import androidx.test.annotation.UiThreadTest
 import androidx.test.ext.junit.runners.AndroidJUnit4
 import com.ichi2.anki.AnkiDroidApp
-import com.ichi2.anki.CrashReportService
-import com.ichi2.anki.CrashReportService.FEEDBACK_REPORT_ALWAYS
-import com.ichi2.anki.CrashReportService.FEEDBACK_REPORT_ASK
 import com.ichi2.anki.R
+import com.ichi2.anki.acraCoreConfigBuilder
 import com.ichi2.anki.analytics.UsageAnalytics
+import com.ichi2.anki.common.crashreporting.CrashReportService
+import com.ichi2.anki.common.crashreporting.CrashReporter.Companion.FEEDBACK_REPORT_ALWAYS
+import com.ichi2.anki.common.crashreporting.CrashReporter.Companion.FEEDBACK_REPORT_ASK
+import com.ichi2.anki.common.crashreporting.CrashReporter.Companion.FEEDBACK_REPORT_KEY
+import com.ichi2.anki.common.crashreporting.CrashReporter.Companion.FEEDBACK_REPORT_NEVER
 import com.ichi2.anki.logging.ProductionCrashReportingTree
 import com.ichi2.anki.preferences.sharedPrefs
 import com.ichi2.anki.servicelayer.ThrowableFilterService
+import com.ichi2.anki.setDebugACRAConfig
+import com.ichi2.anki.setProductionACRAConfig
 import com.ichi2.anki.testutil.GrantStoragePermission
 import org.acra.ACRA
 import org.acra.builder.ReportBuilder
@@ -70,7 +75,7 @@ class ACRATest : InstrumentedTest() {
     @Throws(Exception::class)
     fun testDebugConfiguration() {
         // Debug mode overrides all saved state so no setup needed
-        CrashReportService.setDebugACRAConfig(sharedPrefs)
+        setDebugACRAConfig(sharedPrefs)
         assertArrayEquals(
             "Debug logcat arguments not set correctly",
             CrashReportService.acraCoreConfigBuilder
@@ -90,21 +95,21 @@ class ACRATest : InstrumentedTest() {
         )
         assertEquals(
             "ACRA feedback was not turned off correctly",
-            CrashReportService.FEEDBACK_REPORT_NEVER,
+            FEEDBACK_REPORT_NEVER,
             sharedPrefs
-                .getString(CrashReportService.FEEDBACK_REPORT_KEY, "undefined"),
+                .getString(FEEDBACK_REPORT_KEY, "undefined"),
         )
     }
 
     @Test
     @Throws(Exception::class)
     fun testProductionConfigurationUserDisabled() {
         // set up as if the user had prefs saved to disable completely
-        setReportConfig(CrashReportService.FEEDBACK_REPORT_NEVER)
+        setReportConfig(FEEDBACK_REPORT_NEVER)
 
         // ACRA initializes production logcat via annotation and we can't mock Build.DEBUG
         // That means we are restricted from verifying production logcat args and this is the debug case again
-        CrashReportService.setProductionACRAConfig(sharedPrefs)
+        setProductionACRAConfig(sharedPrefs)
         verifyDebugACRAPreferences()
     }
 
@@ -115,7 +120,7 @@ class ACRATest : InstrumentedTest() {
         setReportConfig(FEEDBACK_REPORT_ASK)
 
         // If the user is set to ask, then it's production, with interaction mode dialog
-        CrashReportService.setProductionACRAConfig(sharedPrefs)
+        setProductionACRAConfig(sharedPrefs)
         verifyACRANotDisabled()
 
         assertToastMessage(R.string.feedback_for_manual_toast_text)
@@ -134,7 +139,7 @@ class ACRATest : InstrumentedTest() {
 
         // If the user is set to always, then it's production, with interaction mode toast
         // will be useful with ACRA 5.2.0
-        CrashReportService.setProductionACRAConfig(sharedPrefs)
+        setProductionACRAConfig(sharedPrefs)
 
         // The same class/method combo is only sent once, so we face a new method each time (should test that system later)
         val crash = Exception("testCrashReportSend at " + System.currentTimeMillis())
@@ -173,7 +178,7 @@ class ACRATest : InstrumentedTest() {
         )
 
         // Now let's clear data
-        CrashReportService.deleteACRALimiterData(testContext)
+        CrashReportService.deleteLimiterData(testContext)
 
         // A third send should work again
         assertTrue(
@@ -192,7 +197,7 @@ class ACRATest : InstrumentedTest() {
         setReportConfig(FEEDBACK_REPORT_ALWAYS)
 
         // If the user is set to always, then it's production, with interaction mode toast
-        CrashReportService.setProductionACRAConfig(sharedPrefs)
+        setProductionACRAConfig(sharedPrefs)
         verifyACRANotDisabled()
 
         assertToastMessage(R.string.feedback_auto_toast_text)
@@ -207,13 +212,13 @@ class ACRATest : InstrumentedTest() {
         setReportConfig(FEEDBACK_REPORT_ALWAYS)
 
         // If the user is set to ask, then it's production, with interaction mode dialog
-        CrashReportService.setProductionACRAConfig(sharedPrefs)
+        setProductionACRAConfig(sharedPrefs)
         verifyACRANotDisabled()
 
         assertDialogEnabledStatus("dialog should be disabled when status is ALWAYS", false)
         assertToastMessage(R.string.feedback_auto_toast_text)
 
-        setAcraReportingMode(FEEDBACK_REPORT_ASK)
+        CrashReportService.setReportingMode(FEEDBACK_REPORT_ASK)
 
         assertDialogEnabledStatus("dialog should be re-enabled after changed to ASK", true)
         assertToastMessage(R.string.feedback_for_manual_toast_text)
@@ -226,12 +231,12 @@ class ACRATest : InstrumentedTest() {
         setReportConfig(FEEDBACK_REPORT_ASK)
 
         // If the user is set to ask, then it's production, with interaction mode dialog
-        CrashReportService.setProductionACRAConfig(sharedPrefs)
+        setProductionACRAConfig(sharedPrefs)
         verifyACRANotDisabled()
 
         assertToastMessage(R.string.feedback_for_manual_toast_text)
 
-        setAcraReportingMode(FEEDBACK_REPORT_ALWAYS)
+        CrashReportService.setReportingMode(FEEDBACK_REPORT_ALWAYS)
 
         assertToastMessage(R.string.feedback_auto_toast_text)
     }
@@ -281,10 +286,6 @@ class ACRATest : InstrumentedTest() {
         )
     }
 
-    private fun setAcraReportingMode(feedbackReportAlways: String) {
-        CrashReportService.setAcraReportingMode(feedbackReportAlways)
-    }
-
     @Throws(ACRAConfigurationException::class)
     private fun assertDialogEnabledStatus(
         message: String,
@@ -333,7 +334,7 @@ class ACRATest : InstrumentedTest() {
     }
 
     private fun setReportConfig(feedbackReportAsk: String) {
-        sharedPrefs.edit { putString(CrashReportService.FEEDBACK_REPORT_KEY, feedbackReportAsk) }
+        sharedPrefs.edit { putString(FEEDBACK_REPORT_KEY, feedbackReportAsk) }
     }
 
     private val sharedPrefs: SharedPreferences

…ava/com/ichi2/anki/CrashReportService.kt …java/com/ichi2/anki/AcraCrashReporter.ktAnkiDroid/src/main/java/com/ichi2/anki/CrashReportService.kt renamed to AnkiDroid/src/main/java/com/ichi2/anki/AcraCrashReporter.kt AnkiDroid/src/main/java/com/ichi2/anki/CrashReportService.kt renamed to AnkiDroid/src/main/java/com/ichi2/anki/AcraCrashReporter.kt

@@ -15,6 +15,7 @@
  */
 package com.ichi2.anki
 
+import android.app.Activity
 import android.app.Application
 import android.content.Context
 import android.content.SharedPreferences
@@ -26,6 +27,12 @@ import androidx.webkit.WebViewCompat
 import com.ichi2.anki.analytics.AnkiDroidCrashReportDialog
 import com.ichi2.anki.analytics.UsageAnalytics
 import com.ichi2.anki.analytics.UsageAnalytics.sendAnalyticsException
+import com.ichi2.anki.common.crashreporting.CrashReportService
+import com.ichi2.anki.common.crashreporting.CrashReporter
+import com.ichi2.anki.common.crashreporting.CrashReporter.Companion.FEEDBACK_REPORT_ALWAYS
+import com.ichi2.anki.common.crashreporting.CrashReporter.Companion.FEEDBACK_REPORT_ASK
+import com.ichi2.anki.common.crashreporting.CrashReporter.Companion.FEEDBACK_REPORT_KEY
+import com.ichi2.anki.common.crashreporting.CrashReporter.Companion.FEEDBACK_REPORT_NEVER
 import com.ichi2.anki.common.time.TimeManager
 import com.ichi2.anki.exception.ManuallyReportedException
 import com.ichi2.anki.exception.UserSubmittedException
@@ -43,13 +50,7 @@ import org.acra.config.ToastConfigurationBuilder
 import org.acra.sender.HttpSender
 import timber.log.Timber
 
-object CrashReportService {
-    // ACRA constants used for stored preferences
-    const val FEEDBACK_REPORT_KEY = "reportErrorMode"
-    const val FEEDBACK_REPORT_ASK = "2"
-    const val FEEDBACK_REPORT_NEVER = "1"
-    const val FEEDBACK_REPORT_ALWAYS = "0"
-
+private object AcraCrashReporter : CrashReporter {
     /** Our ACRA configurations, initialized during Application.onCreate()  */
     @JvmStatic
     private var logcatArgs =
@@ -166,15 +167,15 @@ object CrashReportService {
      */
     @JvmStatic
     fun initialize(application: Application) {
-        CrashReportService.application = application
+        this.application = application
         // FIXME ACRA needs to reinitialize after language is changed, but with the new language
         //   this is difficult because the Application (AnkiDroidApp) does not change it's baseContext
         //   perhaps a solution could be to change AnkiDroidApp to have a context wrapper that it sets
         //   as baseContext, and that wrapper allows a resources/configuration update, then
         //   in GeneralSettingsFragment for the language dialog change listener, the context wrapper
         //   could be updated directly with the new locale code so that calling getString on would fetch
         //   the new language string ?
-        toastText = ToastType.AUTO_TOAST.getToastMessage(CrashReportService.application)
+        toastText = ToastType.AUTO_TOAST.getToastMessage(application)
 
         // Setup logging and crash reporting
         if (BuildConfig.DEBUG) {
@@ -195,6 +196,8 @@ object CrashReportService {
      * Set the reporting mode for ACRA based on the value of the FEEDBACK_REPORT_KEY preference
      * @param value value of FEEDBACK_REPORT_KEY preference
      */
+    override fun setReportingMode(value: String) = setAcraReportingMode(value)
+
     fun setAcraReportingMode(value: String) {
         application.sharedPrefs().edit {
             // Set the ACRA disable value
@@ -262,17 +265,32 @@ object CrashReportService {
     }
 
     /** Used when we don't have an exception to throw, but we know something is wrong and want to diagnose it  */
-    fun sendExceptionReport(
+    override fun sendExceptionReport(
         message: String?,
         origin: String?,
     ) = sendExceptionReport(ManuallyReportedException(message), origin)
 
-    fun sendExceptionReport(
+    override fun sendExceptionReport(
         e: Throwable,
         origin: String?,
-        additionalInfo: String? = null,
-        onlyIfSilent: Boolean = false,
-        context: Context = application.applicationContext,
+        additionalInfo: String?,
+        onlyIfSilent: Boolean,
+    ) = sendExceptionReportInternal(e, origin, additionalInfo, onlyIfSilent, application.applicationContext)
+
+    override fun sendExceptionReport(
+        e: Throwable,
+        origin: String?,
+        additionalInfo: String?,
+        onlyIfSilent: Boolean,
+        context: Context,
+    ) = sendExceptionReportInternal(e, origin, additionalInfo, onlyIfSilent, context)
+
+    private fun sendExceptionReportInternal(
+        e: Throwable,
+        origin: String?,
+        additionalInfo: String?,
+        onlyIfSilent: Boolean,
+        context: Context,
     ) {
         sendAnalyticsException(e, false)
         AnkiDroidApp.sentExceptionReportHack = true
@@ -297,6 +315,11 @@ object CrashReportService {
 
     fun isProperServiceProcess(): Boolean = ACRA.isACRASenderServiceProcess()
 
+    override fun isEnabled(
+        context: Context,
+        defaultValue: Boolean,
+    ): Boolean = isAcraEnabled(context, defaultValue)
+
     fun isAcraEnabled(
         context: Context,
         defaultValue: Boolean,
@@ -314,6 +337,8 @@ object CrashReportService {
      *
      * @param context the context leading to the directory with ACRA limiter data
      */
+    override fun deleteLimiterData(context: Context) = deleteACRALimiterData(context)
+
     fun deleteACRALimiterData(context: Context) {
         try {
             LimiterData().store(context)
@@ -322,7 +347,7 @@ object CrashReportService {
         }
     }
 
-    fun onPreferenceChanged(
+    override fun onPreferenceChanged(
         ctx: Context,
         newValue: String,
     ) {
@@ -338,24 +363,24 @@ object CrashReportService {
      * @return the status of the report, true if the report was sent, false if the report is already
      *  submitted
      */
-    fun sendReport(ankiActivity: AnkiActivity): Boolean {
-        val preferences = ankiActivity.sharedPrefs()
+    override fun sendReport(activity: Activity): Boolean {
+        val preferences = activity.sharedPrefs()
         val reportMode = preferences.getString(FEEDBACK_REPORT_KEY, "")
         return if (FEEDBACK_REPORT_NEVER == reportMode) {
             preferences.edit { putBoolean(ACRA.PREF_DISABLE_ACRA, false) }
             toastText = ToastType.MANUAL_TOAST.getToastMessage(application)
             createAcraCoreConfigBuilder()
-            val sendStatus = sendReportFor(ankiActivity)
+            val sendStatus = sendReportFor(activity)
             dialogEnabled = false
             createAcraCoreConfigBuilder()
             preferences.edit { putBoolean(ACRA.PREF_DISABLE_ACRA, true) }
             sendStatus
         } else {
-            sendReportFor(ankiActivity)
+            sendReportFor(activity)
         }
     }
 
-    private fun sendReportFor(activity: AnkiActivity): Boolean {
+    private fun sendReportFor(activity: Activity): Boolean {
         val currentTimestamp = TimeManager.time.intTimeMS()
         val lastReportTimestamp = getTimestampOfLastReport(activity)
         return if (currentTimestamp - lastReportTimestamp > MIN_INTERVAL_MS) {
@@ -376,7 +401,7 @@ object CrashReportService {
      * @param activity the Activity used for Context access when interrogating ACRA reports
      * @return the timestamp of the most recent report, or -1 if no reports at all
      */
-    private fun getTimestampOfLastReport(activity: AnkiActivity): Long =
+    private fun getTimestampOfLastReport(activity: Activity): Long =
         LimiterData
             .load(activity)
             .reportMetadata
@@ -385,35 +410,23 @@ object CrashReportService {
 }
 
 /**
- * Runs the provided block, catching [Exception], logging it and reporting it to [CrashReportService]
- *
- * **Example**
- * ```
- * runCatchingWithReport("callingMethod", onlyIfSilent = true) {
- *     doSomethingRisky()
- * }
- * ```
- *
- * **Note**: This differs from [runCatching] - `Error` is thrown
- *
- * @param origin Data logged to Timber, and provided as the 'origin' field in the error report
- * @param onlyIfSilent Skip crash report if the crash reporting service is not 'always accept'
- * @param block Code to execute
- *
- * @throws Error If raised, this will be reported and rethrown
- *
- * @return A Result containing either the successful result of [block] or the [Exception] thrown
+ * Initializes ACRA crash reporting and wires it up as the
+ * global [CrashReportService] reporter.
  */
-fun <T> runCatchingWithReport(
-    origin: String?,
-    onlyIfSilent: Boolean = false,
-    block: () -> T,
-): Result<T> =
-    try {
-        Result.success(block())
-    } catch (e: Throwable) {
-        Timber.w(e, origin)
-        CrashReportService.sendExceptionReport(e, origin, onlyIfSilent = onlyIfSilent)
-        if (e is Error) throw e
-        Result.failure(e)
-    }
+context(application: Application)
+fun initializeAcraCrashReporter() {
+    AcraCrashReporter.initialize(application)
+    CrashReportService.setReporter(AcraCrashReporter)
+}
+
+fun isAcraSenderProcess(): Boolean = AcraCrashReporter.isProperServiceProcess()
+
+@VisibleForTesting(otherwise = VisibleForTesting.NONE)
+val CrashReportService.acraCoreConfigBuilder: CoreConfigurationBuilder
+    get() = AcraCrashReporter.acraCoreConfigBuilder
+
+@VisibleForTesting(otherwise = VisibleForTesting.NONE)
+fun setDebugACRAConfig(sharedPrefs: SharedPreferences) = AcraCrashReporter.setDebugACRAConfig(sharedPrefs)
+
+@VisibleForTesting(otherwise = VisibleForTesting.NONE)
+fun setProductionACRAConfig(sharedPrefs: SharedPreferences) = AcraCrashReporter.setProductionACRAConfig(sharedPrefs)

AnkiDroid/src/main/java/com/ichi2/anki/AnkiActivity.kt

@@ -61,6 +61,7 @@ import com.ichi2.anki.android.input.ShortcutGroup
 import com.ichi2.anki.android.input.ShortcutGroupProvider
 import com.ichi2.anki.android.input.shortcut
 import com.ichi2.anki.common.annotations.LegacyNotifications
+import com.ichi2.anki.common.crashreporting.CrashReportService
 import com.ichi2.anki.common.utils.annotation.KotlinCleanup
 import com.ichi2.anki.compat.CompatHelper
 import com.ichi2.anki.compat.CompatHelper.Companion.registerReceiverCompat