Bump io.opentelemetry.instrumentation:gradle-plugins from 2.19.0-alpha to 2.26.1-alpha

agent/agent-tooling/src/main/java/com/microsoft/applicationinsights/agent/internal/classicsdk/ApplicationInsightsAppenderClassFileTransformer.java

@@ -72,10 +72,10 @@ public MethodVisitor visitMethod(
         @Nullable String signature,
         @Nullable String[] exceptions) {
       MethodVisitor mv = cw.visitMethod(access, name, descriptor, signature, exceptions);
-      if (name.equals("append")
-          && (descriptor.equals("(Lch/qos/logback/classic/spi/ILoggingEvent;)V")
-              || descriptor.equals("(Lorg/apache/log4j/spi/LoggingEvent;)V")
-              || descriptor.equals("(Lorg/apache/logging/log4j/core/LogEvent;)V"))) {
+      // match append(SomeLoggingEvent): the descriptor type-strings are not checked here
+      // because Shadow relocates the "ch/qos/logback" literal, breaking the match at runtime;
+      // class-name filtering in transform() already guarantees we are in the right class
+      if (name.equals("append") && descriptor.startsWith("(L") && descriptor.endsWith(";)V")) {
         // no-op the append() method
         mv.visitCode();
         mv.visitInsn(RETURN);

buildSrc/build.gradle.kts

@@ -32,11 +32,11 @@ dependencies {
   // When updating, update above in plugins too
   implementation("com.diffplug.spotless:spotless-plugin-gradle:8.2.1")
   implementation("com.github.spotbugs.snom:spotbugs-gradle-plugin:6.4.8")
-  implementation("com.gradleup.shadow:shadow-gradle-plugin:8.3.9")
+  implementation("com.gradleup.shadow:shadow-gradle-plugin:9.3.2")
 
   implementation("org.owasp:dependency-check-gradle:12.2.0")
 
-  implementation("io.opentelemetry.instrumentation:gradle-plugins:2.19.0-alpha")
+  implementation("io.opentelemetry.instrumentation:gradle-plugins:2.26.1-alpha")
 
   implementation("net.ltgt.gradle:gradle-errorprone-plugin:4.3.0")
   implementation("net.ltgt.gradle:gradle-nullaway-plugin:3.0.0")

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

@@ -5,11 +5,28 @@ plugins {
 }
 
 tasks.withType<ShadowJar>().configureEach {
-  mergeServiceFiles {
-    include("inst/META-INF/services/*")
-  }
+  mergeServiceFiles()
+  mergeServiceFiles("inst/META-INF/services")
 
   exclude("**/module-info.class")
+  exclude("META-INF/*.SF")
+  exclude("META-INF/*.DSA")
+  exclude("META-INF/*.RSA")
+  exclude("META-INF/maven/**")
+  exclude("META-INF/INDEX.LIST")
+
+  // Keep service loaders merged while deduplicating non-functional META-INF metadata.
+  eachFile {
+    if (path.startsWith("META-INF/") && !path.startsWith("META-INF/services/")) {
+      duplicatesStrategy = DuplicatesStrategy.EXCLUDE
+    }
+  }
+  filesMatching("META-INF/services/**") {
+    duplicatesStrategy = DuplicatesStrategy.INCLUDE
+  }
+  filesMatching("inst/META-INF/services/**") {
+    duplicatesStrategy = DuplicatesStrategy.INCLUDE
+  }
 
   // Prevents conflict with other SLF4J instances. Important for premain.
   relocate("org.slf4j", "io.opentelemetry.javaagent.slf4j")

buildSrc/src/main/kotlin/ai.smoke-test-jar.gradle.kts

@@ -15,13 +15,17 @@ val aiSmokeTest = extensions.getByType(AiSmokeTestExtension::class)
 tasks.named<ShadowJar>("shadowJar") {
   archiveClassifier.set("")
   mergeServiceFiles()
-
+
+  filesMatching("META-INF/spring.factories") {
+    duplicatesStrategy = DuplicatesStrategy.INCLUDE
+  }
+
   // Properly merge spring.factories files from all dependencies
   // This is required for Spring Boot auto-configuration to work
   // Use PropertiesFileTransformer to merge duplicate keys instead of simple append
   transform(PropertiesFileTransformer::class.java) {
     paths = listOf("META-INF/spring.factories")
-    mergeStrategy = "append"
+    mergeStrategy = PropertiesFileTransformer.MergeStrategy.Append
   }
 
   // Set main class - can be overridden by individual projects via mainClassName property