build: Compile the SDK for Java 11 (#6459)

* build(android): Compile the Android modules for Java 11

AGP 9 defaults compileOptions to Java 11. The published Android libraries now emit Java 11
bytecode for both Java and Kotlin instead of Java 8. The JVM modules, sentry included, keep
publishing Java 8 bytecode so that Java 8 backends can still use them.

Fixes JAVA-660

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

* changelog

* build: Compile the JVM modules for Java 11

Extend the Java 11 target to the JVM modules, so every published module now emits Java 11
bytecode and the SDK no longer runs on Java 8. Modules that already targeted Java 17 are
unchanged. The samples that pinned Java 8 move to Java 11 with the SDK.

Refs JAVA-660

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

* changelog

* build: Drop the Java 8 API signature check

With Java 11 as the minimum, the Java 8 signature only blocked APIs the SDK may now use.
javac's release flag and Kotlin's -Xjdk-release already limit main code to the Java 11 API, so
the JVM-only modules no longer apply animal sniffer. The modules that Android apps also use keep
checking against the Android API 21 signature.

Refs JAVA-660

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

* changelog

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>

Author: runningcode

.cursor/BUGBOT.md

@@ -58,7 +58,7 @@ rule file in [`.cursor/rules/`](rules) for the area the diff touches (`api`, `op
 
 ## Java and Android specifics
 
-- The core `sentry` module is Java 8 and must not reference Android or JVM-only APIs. Reach optional
+- The core `sentry` module is Java 11 and must not reference Android or JVM-only APIs. Reach optional
   platform code through `Platform`, `LoadClass`, or a separate module.
 - Android code calling an API newer than `minSdk` must be guarded by
   `BuildInfoProvider.getSdkInfoVersion()`.

AGENTS.md

@@ -123,7 +123,7 @@ The repository is organized into multiple modules:
 ## Development Guidelines
 
 ### Code Style
-- **Languages**: Java 8+ and Kotlin
+- **Languages**: Java 11+ and Kotlin
 - **Formatting**: Enforced via Spotless
 - **API Compatibility**: Binary compatibility is enforced. `.api` files are generated, never hand-edited
 

CHANGELOG.md

@@ -4,6 +4,7 @@
 
 ### Breaking Changes
 
+- The SDK is now compiled for Java 11 instead of Java 8. JVM applications need Java 11 or newer ([#6459](https://github.com/getsentry/sentry-java/pull/6459))
 - Default trace context status to `ok` when an active span has no explicit status ([#6418](https://github.com/getsentry/sentry-java/pull/6418))
 - Remove the deprecated user feedback APIs ([#6337](https://github.com/getsentry/sentry-java/pull/6337))
   - `UserFeedback` and `Sentry.captureUserFeedback(...)`: use `Sentry.feedback().capture(...)` with `io.sentry.protocol.Feedback` instead

build-logic/build.gradle.kts

@@ -16,10 +16,6 @@ gradlePlugin {
       id = "io.sentry.jdk-target"
       implementationClass = "io.sentry.gradle.JdkTargetConvention"
     }
-    register("sentryAnimalSniffer") {
-      id = "io.sentry.animalsniffer"
-      implementationClass = "io.sentry.gradle.SentryAnimalSnifferPlugin"
-    }
     register("sentryAnimalSnifferAndroid") {
       id = "io.sentry.animalsniffer.android"
       implementationClass = "io.sentry.gradle.SentryAnimalSnifferAndroidPlugin"

build-logic/src/main/kotlin/io/sentry/gradle/JdkTargetConvention.kt

@@ -20,18 +20,18 @@ import org.jetbrains.kotlin.gradle.tasks.KotlinJvmCompile
 /** The JDK every module compiles and tests with, whatever JDK runs Gradle. */
 private val BUILD_JDK = JavaLanguageVersion.of(21)
 
-/** The level the SDK publishes, so that Java 8 consumers can still use it. */
-private val PUBLISHED_JAVA_VERSION = JavaVersion.VERSION_1_8
+/** The level the SDK publishes. */
+private val PUBLISHED_JAVA_VERSION = JavaVersion.VERSION_11
 
 /**
  * Holds every toolchain, bytecode level decision and compilation option of a module. It does the
  * following
  * - sample apps target the build JDK (21)
  * - unit test code will target the build JDK (21) (required for Mockito 5)
- * - published JVM modules main code pins the Java 1.8 release, for Java and for Kotlin
+ * - published JVM modules main code pins the Java 11 release, for Java and for Kotlin
  * - published modules Java code drops the MethodParameters attribute javac 21 adds, and their
  *   archives are checked for it
- * - published Android modules main code targets Java 1.8
+ * - published Android modules main code targets Java 11
  * - minSdk is set for Android libraries
  */
 class JdkTargetConvention : Plugin<Project> {
@@ -75,7 +75,7 @@ class JdkTargetConvention : Plugin<Project> {
         }
       }
       // -Xjdk-release is the Kotlin counterpart of release: without it Kotlin resolves against the
-      // build JDK and happily compiles e.g. Files.readString() into Java 8 bytecode. The version
+      // build JDK and happily compiles e.g. Stream.toList() into Java 11 bytecode. The version
       // follows jvmTarget, which the modules that publish a higher level set themselves.
       project.plugins.withId("org.jetbrains.kotlin.jvm") {
         project.tasks.named("compileKotlin", KotlinJvmCompile::class.java) {
@@ -110,8 +110,8 @@ class JdkTargetConvention : Plugin<Project> {
       .toInt()
 
   /**
-   * Mockito 5 and mockito-kotlin 6 are built for Java 11, so test code - which is never published -
-   * cannot share the Java 8 target of the main code. AGP has no per variant compileOptions, so this
+   * Test code - which is never published - targets the build JDK, as it does in the JVM modules,
+   * instead of the Java 11 target of the main code. AGP has no per variant compileOptions, so this
    * goes through the Kotlin compilations: their compile task providers are configured after AGP
    * wrote its own compileOptions, which an apply time configureEach would lose against.
    */

build-logic/src/main/kotlin/io/sentry/gradle/SentryAnimalSnifferPlugin.kt

@@ -20,14 +20,19 @@ abstract class SentryAnimalSnifferExtension {
   }
 }
 
-class SentryAnimalSnifferPlugin : Plugin<Project> {
+/**
+ * Checks the main code of the JVM modules that Android apps also use against the Android API 21
+ * signature. The JVM API needs no signature: javac's release flag and Kotlin's -Xjdk-release
+ * already limit main code to the published Java version, see JdkTargetConvention.
+ */
+class SentryAnimalSnifferAndroidPlugin : Plugin<Project> {
   override fun apply(project: Project) {
     project.pluginManager.apply("ru.vyarus.animalsniffer")
 
     val extension =
       project.extensions.create("sentryAnimalSniffer", SentryAnimalSnifferExtension::class.java)
 
-    project.addSignatureDependency("java8-signature")
+    project.addSignatureDependency("gummy-bears-api21")
 
     project.tasks.named("animalsnifferMain", AnimalSniffer::class.java).configure {
       ignoreClasses = ignoreClasses + extension.ignoredClasses.get()
@@ -38,14 +43,6 @@ class SentryAnimalSnifferPlugin : Plugin<Project> {
   }
 }
 
-class SentryAnimalSnifferAndroidPlugin : Plugin<Project> {
-  override fun apply(project: Project) {
-    project.pluginManager.apply(SentryAnimalSnifferPlugin::class.java)
-
-    project.addSignatureDependency("gummy-bears-api21")
-  }
-}
-
 private fun Project.addSignatureDependency(libraryName: String) {
   val libs = extensions.getByType(VersionCatalogsExtension::class.java).named("libs")
   dependencies.add("signature", signatureNotation(libs.findLibrary(libraryName).get().get()))

gradle/libs.versions.toml

@@ -13,7 +13,6 @@ coroutines = "1.6.1"
 espresso = "3.7.0"
 feign = "11.6"
 gummyBears = "0.12.0"
-java8Signature = "1.0"
 jackson = "2.18.3"
 jetbrainsCompose = "1.6.11"
 kotlin = "2.3.21"
@@ -252,7 +251,6 @@ timber = { module = "com.jakewharton.timber:timber", version = "4.7.1" }
 
 # Animalsniffer signature
 gummy-bears-api21 = { module = "com.toasttab.android:gummy-bears-api-21", version.ref = "gummyBears" }
-java8-signature = { module = "org.codehaus.mojo.signature:java18", version.ref = "java8Signature" }
 
 # tomcat libraries
 tomcat-catalina = { module = "org.apache.tomcat:tomcat-catalina", version = "9.0.108" }

sentry-android-core/build.gradle.kts

@@ -1,6 +1,6 @@
 import net.ltgt.gradle.errorprone.errorprone
 import org.jetbrains.kotlin.config.KotlinCompilerVersion
-import org.jetbrains.kotlin.gradle.dsl.JvmTarget.JVM_1_8
+import org.jetbrains.kotlin.gradle.dsl.JvmTarget.JVM_11
 
 plugins {
   id("com.android.library")
@@ -39,7 +39,7 @@ android {
   // disabled, so unit tests must target release to run at all.
   testBuildType = "release"
 
-  kotlin { compilerOptions.jvmTarget = JVM_1_8 }
+  kotlin { compilerOptions.jvmTarget = JVM_11 }
 
   testOptions {
     animationsDisabled = true

sentry-android-distribution/build.gradle.kts

@@ -1,4 +1,4 @@
-import org.jetbrains.kotlin.gradle.dsl.JvmTarget.JVM_1_8
+import org.jetbrains.kotlin.gradle.dsl.JvmTarget.JVM_11
 import org.jetbrains.kotlin.gradle.dsl.KotlinVersion
 
 plugins {
@@ -28,7 +28,7 @@ android {
 }
 
 kotlin {
-  compilerOptions.jvmTarget = JVM_1_8
+  compilerOptions.jvmTarget = JVM_11
   compilerOptions.languageVersion = KotlinVersion.KOTLIN_1_9
   explicitApi()
 }

sentry-android-fragment/build.gradle.kts

@@ -31,7 +31,7 @@ android {
   testBuildType = "release"
 
   kotlin {
-    compilerOptions.jvmTarget = JvmTarget.JVM_1_8
+    compilerOptions.jvmTarget = JvmTarget.JVM_11
     compilerOptions.languageVersion = KotlinVersion.KOTLIN_1_9
     compilerOptions.apiVersion = KotlinVersion.KOTLIN_1_9
   }
@@ -75,5 +75,5 @@ dependencies {
 
 tasks.withType<Detekt>().configureEach {
   // Target version of the generated JVM bytecode. It is used for type resolution.
-  jvmTarget = JavaVersion.VERSION_1_8.toString()
+  jvmTarget = JavaVersion.VERSION_11.toString()
 }