More accurate, faster spotless config

This PR ensures that spotless is actually running over all our files. It
wasn't before and you can see this generated a massive diff.
There are still a number of exceptions that were suppressed in order to
save time going through the hundreds of them.

In other news, this moves the spotless logic to its own convention
plugin and runs over each source set of each project separately. This
means that most of the tasks should be cached independently.

Author: runningcode

build-logic/build.gradle.kts

@@ -1,7 +1,11 @@
 plugins {
-    `kotlin-dsl`
+  `kotlin-dsl`
 }
 
 repositories {
-    gradlePluginPortal()
+  gradlePluginPortal()
+}
+
+dependencies {
+  implementation(libs.spotlessLib)
 }

build-logic/settings.gradle.kts

@@ -1 +1,9 @@
+dependencyResolutionManagement {
+    versionCatalogs {
+        create("libs") {
+            from(files("../gradle/libs.versions.toml"))
+        }
+    }
+}
+
 rootProject.name = "build-logic"

build-logic/src/main/kotlin/io.sentry.spotless.gradle.kts

@@ -0,0 +1,37 @@
+import com.diffplug.spotless.LineEnding
+
+plugins {
+    id("com.diffplug.spotless")
+}
+
+spotless {
+    lineEndings = LineEnding.UNIX
+    java {
+        target("**/*.java")
+        removeUnusedImports()
+        googleJavaFormat()
+        targetExclude("src/**/java/io/sentry/vendor/**")
+    }
+    kotlin {
+        target("**/*.kt")
+        ktlint()
+        targetExclude("src/test/java/io/sentry/apollo4/generated/**", "src/test/java/io/sentry/apollo3/adapter/**")
+        suppressLintsFor {
+            step = "ktlint"
+            shortCode = "standard:max-line-length"
+        }
+        suppressLintsFor {
+            step = "ktlint"
+            shortCode = "standard:property-naming"
+        }
+        suppressLintsFor {
+            step = "ktlint"
+            shortCode = "standard:function-naming"
+        }
+    }
+    kotlinGradle {
+        target("**/*.kts")
+        ktlint()
+    }
+}
+

build.gradle.kts

@@ -1,4 +1,3 @@
-import com.diffplug.spotless.LineEnding
 import com.vanniktech.maven.publish.JavaLibrary
 import com.vanniktech.maven.publish.JavadocJar
 import com.vanniktech.maven.publish.MavenPublishBaseExtension
@@ -10,7 +9,7 @@ import org.gradle.api.tasks.testing.logging.TestLogEvent
 
 plugins {
     `java-library`
-    alias(libs.plugins.spotless)
+    alias(libs.plugins.spotless) apply false
     jacoco
     alias(libs.plugins.detekt)
     `maven-publish`
@@ -107,6 +106,8 @@ allprojects {
 }
 
 subprojects {
+    apply { plugin("io.sentry.spotless") }
+
     val jacocoAndroidModules = listOf(
         "sentry-android-core",
         "sentry-android-fragment",
@@ -227,26 +228,6 @@ subprojects {
     }
 }
 
-spotless {
-    lineEndings = LineEnding.UNIX
-    java {
-        target("**/*.java")
-        removeUnusedImports()
-        googleJavaFormat()
-        targetExclude("**/generated/**", "**/vendor/**", "**/sentry-native/**")
-    }
-    kotlin {
-        target("**/*.kt")
-        ktlint()
-        targetExclude("**/sentry-native/**", "**/build/**")
-    }
-    kotlinGradle {
-        target("**/*.kts")
-        ktlint()
-        targetExclude("**/sentry-native/**")
-    }
-}
-
 tasks.register("buildForCodeQL") {
     subprojects
         .filter {

gradle/libs.versions.toml

@@ -32,6 +32,7 @@ springboot3 = "3.5.0"
 targetSdk = "34"
 compileSdk = "34"
 minSdk = "21"
+spotless = "7.0.4"
 
 [plugins]
 kotlin-android = { id = "org.jetbrains.kotlin.android", version.ref = "kotlin" }
@@ -43,7 +44,7 @@ binary-compatibility-validator = { id = "org.jetbrains.kotlinx.binary-compatibil
 compose-compiler = { id = "org.jetbrains.compose", version.ref = "jetbrainsCompose" }
 errorprone = { id = "net.ltgt.errorprone", version = "3.0.1" }
 gradle-versions = { id = "com.github.ben-manes.versions", version = "0.42.0" }
-spotless = { id = "com.diffplug.spotless", version = "6.11.0" }
+spotless = { id = "com.diffplug.spotless", version.ref = "spotless" }
 detekt = { id = "io.gitlab.arturbosch.detekt", version = "1.23.8" }
 jacoco-android = { id = "com.mxalbert.gradle.jacoco-android", version = "0.2.0" }
 kover = { id = "org.jetbrains.kotlinx.kover", version = "0.7.3" }
@@ -121,6 +122,7 @@ servlet-jakarta-api = { module = "jakarta.servlet:jakarta.servlet-api", version
 slf4j-api = { module = "org.slf4j:slf4j-api", version.ref = "slf4j" }
 slf4j-jdk14 = { module = "org.slf4j:slf4j-jdk14", version.ref = "slf4j" }
 slf4j2-api = { module = "org.slf4j:slf4j-api", version = "2.0.5" }
+spotlessLib = { module = "com.diffplug.spotless:com.diffplug.spotless.gradle.plugin", version.ref = "spotless"}
 springboot-starter = { module = "org.springframework.boot:spring-boot-starter", version.ref = "springboot2" }
 springboot-starter-graphql = { module = "org.springframework.boot:spring-boot-starter-graphql", version.ref = "springboot2" }
 springboot-starter-quartz = { module = "org.springframework.boot:spring-boot-starter-quartz", version.ref = "springboot2" }

sentry-android-core/build.gradle.kts

@@ -11,11 +11,17 @@ plugins {
 }
 
 android {
-    compileSdk = libs.versions.compileSdk.get().toInt()
+    compileSdk =
+        libs.versions.compileSdk
+            .get()
+            .toInt()
     namespace = "io.sentry.android.core"
 
     defaultConfig {
-        minSdk = libs.versions.minSdk.get().toInt()
+        minSdk =
+            libs.versions.minSdk
+                .get()
+                .toInt()
 
         testInstrumentationRunner = "androidx.test.runner.AndroidJUnitRunner"
 

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

@@ -47,6 +47,7 @@ final class ANRWatchDog extends Thread {
   private final ANRListener anrListener;
   private final MainLooperHandler uiHandler;
   private final ICurrentDateProvider timeProvider;
+
   /** the interval in which we check if there's an ANR, in ms */
   private long pollingIntervalMs;
 

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

@@ -516,6 +516,7 @@ private void setOptionsTags(final @NotNull SentryBaseEvent event) {
       }
     }
   }
+
   // endregion
 
   @Override

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

@@ -17,6 +17,7 @@ public final class BuildInfoProvider {
   public BuildInfoProvider(final @NotNull ILogger logger) {
     this.logger = Objects.requireNonNull(logger, "The ILogger object is required.");
   }
+
   /**
    * Returns the Build.VERSION.SDK_INT
    *

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

@@ -160,6 +160,7 @@ public final class SentryAndroidOptions extends SentryOptions {
   @NotNull
   private NdkHandlerStrategy ndkHandlerStrategy =
       NdkHandlerStrategy.SENTRY_HANDLER_STRATEGY_DEFAULT;
+
   /**
    * Enable the Java to NDK Scope sync. The default value for sentry-java is disabled and enabled
    * for sentry-android.