fix: always register activity lifecycle callbacks for accurate session tracking

Author: polbins

android/src/main/java/com/amplitude/android/Amplitude.kt

@@ -64,10 +64,8 @@ open class Amplitude internal constructor(
     init {
         registerShutdownHook()
 
-        if (configuration.autocapture.any { it in AutocaptureOption.REQUIRES_ACTIVITY_CALLBACKS }) {
-            with(configuration.context as Application) {
-                registerActivityLifecycleCallbacks(activityLifecycleCallbacks)
-            }
+        with(configuration.context as Application) {
+            registerActivityLifecycleCallbacks(activityLifecycleCallbacks)
         }
     }
 

android/src/main/java/com/amplitude/android/AutocaptureOptions.kt

@@ -58,18 +58,6 @@ enum class AutocaptureOption {
                 FRUSTRATION_INTERACTIONS,
             )
 
-        /**
-         * Set of autocapture options that require Android Activity lifecycle callbacks to function properly.
-         *
-         * These options need access to activity lifecycle events and therefore require the ActivityLifecycleObserver to be registered with the Application.
-         */
-        val REQUIRES_ACTIVITY_CALLBACKS =
-            setOf(
-                APP_LIFECYCLES,
-                SCREEN_VIEWS,
-                ELEMENT_INTERACTIONS,
-                DEEP_LINKS,
-            )
     }
 }
 

android/src/test/kotlin/com/amplitude/android/ConfigurationTest.kt

@@ -138,22 +138,6 @@ class ConfigurationTest {
         Assertions.assertFalse(AutocaptureOption.SCREEN_VIEWS in configuration.autocapture)
     }
 
-    @Test
-    fun autocaptureOption_requiresActivityCallbacks_containsExpectedOptions() {
-        val expectedOptions =
-            setOf(
-                AutocaptureOption.APP_LIFECYCLES,
-                AutocaptureOption.SCREEN_VIEWS,
-                AutocaptureOption.ELEMENT_INTERACTIONS,
-                AutocaptureOption.DEEP_LINKS,
-            )
-
-        Assertions.assertEquals(expectedOptions, AutocaptureOption.REQUIRES_ACTIVITY_CALLBACKS)
-
-        // Verify SESSIONS is NOT included (it doesn't need activity callbacks)
-        Assertions.assertFalse(AutocaptureOption.SESSIONS in AutocaptureOption.REQUIRES_ACTIVITY_CALLBACKS)
-    }
-
     @Suppress("DEPRECATION")
     @Test
     fun configuration_defaultTracking_replaces_autocapture_entirely_configuration() {