feat(android): Warn for legacy Logs metadata

Detect explicit io.sentry.logs.enabled Android manifest metadata and emit tailored migration warnings without changing the integration-local Logs options.

Co-Authored-By: Claude <noreply@anthropic.com>

Author: adinauer

sentry-android-core/src/main/java/io/sentry/android/core/ManifestMetadataReader.java

@@ -706,6 +706,23 @@ static void applyMetadata(
           }
         }
 
+        if (metadata.containsKey(ENABLE_LOGS)) {
+          final boolean enableLogs = readBool(metadata, logger, ENABLE_LOGS, false);
+          if (enableLogs) {
+            logger.log(
+                SentryLevel.WARNING,
+                "The Android manifest option 'io.sentry.logs.enabled' is no longer supported. "
+                    + "Manual Sentry.logger() calls no longer require it, and automatic logging "
+                    + "integrations now require their own opt-ins.");
+          } else {
+            logger.log(
+                SentryLevel.WARNING,
+                "The Android manifest option 'io.sentry.logs.enabled' no longer disables manual "
+                    + "Sentry.logger() calls. Automatic logging integrations remain disabled "
+                    + "unless enabled through their own opt-ins.");
+          }
+        }
+
         options.setEnableTimberLogs(
             readBool(metadata, logger, ENABLE_TIMBER_LOGS, options.isEnableTimberLogs()));
 

sentry-android-core/src/test/java/io/sentry/android/core/ManifestMetadataReaderTest.kt

@@ -29,7 +29,7 @@ import org.mockito.kotlin.verify
 class ManifestMetadataReaderTest {
   private class Fixture {
     val logger = mock<ILogger>()
-    val options = SentryAndroidOptions().apply { setLogger(logger) }
+    val options = SentryAndroidOptions().apply { setLogger(this@Fixture.logger) }
     val buildInfoProvider = mock<BuildInfoProvider>()
 
     fun getContext(metaData: Bundle = Bundle()): Context =
@@ -1943,6 +1943,64 @@ class ManifestMetadataReaderTest {
     assertTrue(fixture.options.inAppExcludes.isEmpty())
   }
 
+  @Test
+  fun `applyMetadata does not warn when legacy logs enabled metadata is absent`() {
+    fixture.options.isDebug = true
+    val context = fixture.getContext()
+
+    ManifestMetadataReader.applyMetadata(context, fixture.options, fixture.buildInfoProvider)
+
+    verify(fixture.logger, never()).log(eq(SentryLevel.WARNING), any<String>())
+  }
+
+  @Test
+  fun `applyMetadata warns when legacy logs enabled metadata is true`() {
+    val bundle =
+      bundleOf(
+        ManifestMetadataReader.DEBUG to true,
+        ManifestMetadataReader.ENABLE_LOGS to true,
+      )
+    val context = fixture.getContext(metaData = bundle)
+
+    ManifestMetadataReader.applyMetadata(context, fixture.options, fixture.buildInfoProvider)
+
+    verify(fixture.logger)
+      .log(
+        SentryLevel.WARNING,
+        "The Android manifest option 'io.sentry.logs.enabled' is no longer supported. " +
+          "Manual Sentry.logger() calls no longer require it, and automatic logging " +
+          "integrations now require their own opt-ins.",
+        *emptyArray(),
+      )
+    assertThat(fixture.options.isEnableTimberLogs).isFalse()
+    assertThat(fixture.options.isEnableLogcatLogs).isFalse()
+  }
+
+  @Test
+  fun `applyMetadata warns when legacy logs enabled metadata is false`() {
+    fixture.options.isEnableTimberLogs = true
+    fixture.options.isEnableLogcatLogs = true
+    val bundle =
+      bundleOf(
+        ManifestMetadataReader.DEBUG to true,
+        ManifestMetadataReader.ENABLE_LOGS to false,
+      )
+    val context = fixture.getContext(metaData = bundle)
+
+    ManifestMetadataReader.applyMetadata(context, fixture.options, fixture.buildInfoProvider)
+
+    verify(fixture.logger)
+      .log(
+        SentryLevel.WARNING,
+        "The Android manifest option 'io.sentry.logs.enabled' no longer disables manual " +
+          "Sentry.logger() calls. Automatic logging integrations remain disabled unless " +
+          "enabled through their own opt-ins.",
+        *emptyArray(),
+      )
+    assertThat(fixture.options.isEnableTimberLogs).isTrue()
+    assertThat(fixture.options.isEnableLogcatLogs).isTrue()
+  }
+
   @Test
   fun `applyMetadata keeps Timber logs disabled if not found`() {
     val context = fixture.getContext()