Sync spotless configuration with opentelemetry-java-instrumentation (#95)

* Initial plan

* Sync spotless configuration with opentelemetry-java-instrumentation

Co-authored-by: trask <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: trask <[email protected]>

Author: Copilot

.github/copilot-instructions.md

@@ -90,7 +90,7 @@ The agent JAR is built in **3 critical steps** (see `agent/agent/build.gradle.kt
   abstract class HttpClientTest {
     @Environment(TOMCAT_8_JAVA_8)
     static class Tomcat8Java8Test extends HttpClientTest {}
-    
+
     @Environment(TOMCAT_8_JAVA_11)
     static class Tomcat8Java11Test extends HttpClientTest {}
   }

buildSrc/src/main/kotlin/ai.spotless-conventions.gradle.kts

@@ -1,42 +1,123 @@
+import com.diffplug.gradle.spotless.SpotlessExtension
+
 plugins {
   id("com.diffplug.spotless")
 }
 
 spotless {
   java {
     googleJavaFormat()
-    licenseHeaderFile(rootProject.file("buildscripts/spotless.license.java"), "(package|import|public|// Includes work from:)")
+    licenseHeaderFile(
+      rootProject.file("buildscripts/spotless.license.java"),
+      "(package|import|public|// Includes work from:)"
+    )
+    toggleOffOn()
     target("src/**/*.java")
   }
+  plugins.withId("groovy") {
+    groovy {
+      licenseHeaderFile(
+        rootProject.file("buildscripts/spotless.license.java"),
+        "(package|import|(?:abstract )?class)"
+      )
+      endWithNewline()
+    }
+  }
+  plugins.withId("scala") {
+    scala {
+      scalafmt()
+      licenseHeaderFile(
+        rootProject.file("buildscripts/spotless.license.java"),
+        "(package|import|public)"
+      )
+      target("src/**/*.scala")
+    }
+  }
+  plugins.withId("org.jetbrains.kotlin.jvm") {
+    kotlin {
+      // not sure why it's not using the indent settings from .editorconfig
+      ktlint().editorConfigOverride(
+        mapOf(
+          "indent_size" to "2",
+          "continuation_indent_size" to "2",
+          "max_line_length" to "160",
+          "ktlint_standard_no-wildcard-imports" to "disabled",
+          "ktlint_standard_package-name" to "disabled",
+          // ktlint does not break up long lines, it just fails on them
+          "ktlint_standard_max-line-length" to "disabled",
+          // ktlint makes it *very* hard to locate where this actually happened
+          "ktlint_standard_trailing-comma-on-call-site" to "disabled",
+          // depends on ktlint_standard_wrapping
+          "ktlint_standard_trailing-comma-on-declaration-site" to "disabled",
+          // also very hard to find out where this happens
+          "ktlint_standard_wrapping" to "disabled"
+        )
+      )
+      licenseHeaderFile(
+        rootProject.file("buildscripts/spotless.license.java"),
+        "(package|import|class|// Includes work from:)"
+      )
+    }
+  }
   kotlinGradle {
-    ktlint().editorConfigOverride(mapOf(
-      "indent_size" to "2",
-      "continuation_indent_size" to "2",
-      "max_line_length" to "160",
-      "ktlint_standard_no-wildcard-imports" to "disabled",
-      // ktlint does not break up long lines, it just fails on them
-      "ktlint_standard_max-line-length" to "disabled",
-      // ktlint makes it *very* hard to locate where this actually happened
-      "ktlint_standard_trailing-comma-on-call-site" to "disabled",
-      // depends on ktlint_standard_wrapping
-      "ktlint_standard_trailing-comma-on-declaration-site" to "disabled",
-      // also very hard to find out where this happens
-      "ktlint_standard_wrapping" to "disabled"
-    ))
+    // not sure why it's not using the indent settings from .editorconfig
+    ktlint().editorConfigOverride(
+      mapOf(
+        "indent_size" to "2",
+        "continuation_indent_size" to "2",
+        "max_line_length" to "160",
+        "ktlint_standard_no-wildcard-imports" to "disabled",
+        // ktlint does not break up long lines, it just fails on them
+        "ktlint_standard_max-line-length" to "disabled",
+        // ktlint makes it *very* hard to locate where this actually happened
+        "ktlint_standard_trailing-comma-on-call-site" to "disabled",
+        // depends on ktlint_standard_wrapping
+        "ktlint_standard_trailing-comma-on-declaration-site" to "disabled",
+        // also very hard to find out where this happens
+        "ktlint_standard_wrapping" to "disabled",
+        // we use variable names like v1_10Deps
+        "ktlint_standard_property-naming" to "disabled",
+        // prevent moving comment to next line in latestDepTestLibrary("xxx") { // see xxx module
+        "ktlint_standard_function-literal" to "disabled"
+      )
+    )
   }
-  format("misc") {
-    // not using "**/..." to help keep spotless fast
-    target(
-      ".gitattributes",
-      ".gitconfig",
-      ".editorconfig",
-      "*.md",
-      "src/**/*.md",
-      "docs/**/*.md",
-      "*.sh",
-      "src/**/*.properties")
-    leadingTabsToSpaces()
-    trimTrailingWhitespace()
-    endWithNewline()
+}
+
+// Use root declared tool deps to avoid issues with high concurrency.
+// see https://github.com/diffplug/spotless/tree/main/plugin-gradle#dependency-resolution-modes
+if (project == rootProject) {
+  spotless {
+    format("misc") {
+      target(
+        ".gitignore",
+        ".gitattributes",
+        ".gitconfig",
+        ".editorconfig",
+        "**/*.md",
+        "**/*.sh",
+        "**/*.dockerfile",
+        "**/gradle.properties"
+      )
+      leadingTabsToSpaces()
+      trimTrailingWhitespace()
+      endWithNewline()
+    }
+    predeclareDeps()
+  }
+
+  with(extensions["spotlessPredeclare"] as SpotlessExtension) {
+    java {
+      googleJavaFormat()
+    }
+    scala {
+      scalafmt()
+    }
+    kotlin {
+      ktlint()
+    }
+    kotlinGradle {
+      ktlint()
+    }
   }
 }