Publish temporal-spring-ai from the JDK 11 release job via toolchains

The module was previously excluded from the build whenever Gradle ran on
JDK <17, which silently dropped it from the release and snapshot publish
jobs (both use JDK 11). Switch to the same pattern temporal-sdk already
uses: resolve a JDK 17+ compiler/javadoc/launcher via Gradle toolchains
when the running JVM is older, and drop the conditional include guards
in settings.gradle and the BOM so the artifact always publishes.

Shared javadoc config in gradle/java.gradle now keys flag choices off
the javadoc tool's language version instead of JavaVersion.current(),
so modules that override javadocTool (temporal-spring-ai on JDK 11)
don't get JDK 9-12-only flags passed to a JDK 17 tool.

Add JDK 17 alongside JDK 11 in the publish workflows so toolchain
discovery has a matching installation to resolve.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: donald-pinckney

.github/workflows/prepare-release.yml

@@ -94,7 +94,11 @@ jobs:
       - name: Set up Java
         uses: actions/setup-java@v5
         with:
-          java-version: "11"
+          # JDK 17 is required for temporal-spring-ai's Gradle toolchain (Spring AI requires Java 17+).
+          # The primary publish JVM remains JDK 11 (listed last so it's the default JAVA_HOME).
+          java-version: |
+            17
+            11
           distribution: "temurin"
 
       - name: Set up Gradle

.github/workflows/publish-snapshot.yml

@@ -38,7 +38,11 @@ jobs:
       - name: Set up Java
         uses: actions/setup-java@v5
         with:
-          java-version: '11'
+          # JDK 17 is required for temporal-spring-ai's Gradle toolchain (Spring AI requires Java 17+).
+          # The primary publish JVM remains JDK 11 (listed last so it's the default JAVA_HOME).
+          java-version: |
+            17
+            11
           distribution: 'temurin'
 
       - name: Set up Gradle

gradle/java.gradle

@@ -34,14 +34,19 @@ subprojects {
         options.encoding = 'UTF-8'
         options.addStringOption('Xdoclint:reference', '-quiet')
         options.addBooleanOption('Werror', true)
-        if (JavaVersion.current().isJava9Compatible()) {
-            options.addBooleanOption('html5', true)
-        }
-        if (JavaVersion.current().isCompatibleWith(JavaVersion.VERSION_1_9) &&
-                !JavaVersion.current().isCompatibleWith(JavaVersion.VERSION_13)) {
-            //https://stackoverflow.com/questions/53732632/gradle-javadoc-search-redirects-to-undefined-url
-            //the flag is removed in java13: https://bugs.openjdk.java.net/browse/JDK-8215582
-            options.addBooleanOption('-no-module-directories', true)
+        // Base flag decisions on the javadoc tool that will actually run, not the Gradle JVM.
+        // Modules that override javadocTool via toolchains (e.g. temporal-spring-ai on JDK 11)
+        // would otherwise get flags that don't apply to the tool being invoked.
+        doFirst {
+            int toolVersion = javadocTool.get().metadata.languageVersion.asInt()
+            if (toolVersion >= 9) {
+                options.addBooleanOption('html5', true)
+            }
+            if (toolVersion >= 9 && toolVersion < 13) {
+                //https://stackoverflow.com/questions/53732632/gradle-javadoc-search-redirects-to-undefined-url
+                //the flag is removed in java13: https://bugs.openjdk.java.net/browse/JDK-8215582
+                options.addBooleanOption('-no-module-directories', true)
+            }
         }
     }
 

settings.gradle

@@ -6,11 +6,7 @@ include 'temporal-testing'
 include 'temporal-test-server'
 include 'temporal-opentracing'
 include 'temporal-kotlin'
-// temporal-spring-ai requires Java 17+ (Spring AI dependency).
-// Exclude from builds running on older JDKs.
-if (JavaVersion.current().isCompatibleWith(JavaVersion.VERSION_17)) {
-    include 'temporal-spring-ai'
-}
+include 'temporal-spring-ai'
 include 'temporal-spring-boot-autoconfigure'
 include 'temporal-spring-boot-starter'
 include 'temporal-remote-data-encoder'

temporal-bom/build.gradle

@@ -12,9 +12,7 @@ dependencies {
         api project(':temporal-sdk')
         api project(':temporal-serviceclient')
         api project(':temporal-shaded')
-        if (JavaVersion.current().isCompatibleWith(JavaVersion.VERSION_17)) {
-            api project(':temporal-spring-ai')
-        }
+        api project(':temporal-spring-ai')
         api project(':temporal-spring-boot-autoconfigure')
         api project(':temporal-spring-boot-starter')
         api project(':temporal-test-server')

temporal-spring-ai/build.gradle

@@ -10,22 +10,43 @@ ext {
 // When edgeDepsTest is set, use 21 to match other modules (avoids Gradle JVM compatibility rejection).
 ext {
     springAiJavaVersion = project.hasProperty("edgeDepsTest") ? JavaVersion.VERSION_21 : JavaVersion.VERSION_17
-    springAiRelease = project.hasProperty("edgeDepsTest") ? '21' : '17'
+    springAiReleaseInt = project.hasProperty("edgeDepsTest") ? 21 : 17
 }
 
 java {
     sourceCompatibility = springAiJavaVersion
     targetCompatibility = springAiJavaVersion
 }
 
+// Use a JDK 17+ compiler/javadoc/launcher via Gradle toolchains when the running JVM is older.
+// This lets the module build and publish from the same JDK 11 release job used for the rest of
+// the SDK, matching the pattern used by temporal-sdk for its Java 17/21 source sets.
 compileJava {
+    if (!JavaVersion.current().isCompatibleWith(springAiJavaVersion)) {
+        javaCompiler = javaToolchains.compilerFor {
+            languageVersion = JavaLanguageVersion.of(springAiReleaseInt)
+        }
+    }
     options.compilerArgs.removeAll(['--release', '8'])
-    options.compilerArgs.addAll(['--release', springAiRelease])
+    options.release = springAiReleaseInt
 }
 
 compileTestJava {
+    if (!JavaVersion.current().isCompatibleWith(springAiJavaVersion)) {
+        javaCompiler = javaToolchains.compilerFor {
+            languageVersion = JavaLanguageVersion.of(springAiReleaseInt)
+        }
+    }
     options.compilerArgs.removeAll(['--release', '8'])
-    options.compilerArgs.addAll(['--release', springAiRelease])
+    options.release = springAiReleaseInt
+}
+
+javadoc {
+    if (!JavaVersion.current().isCompatibleWith(springAiJavaVersion)) {
+        javadocTool = javaToolchains.javadocToolFor {
+            languageVersion = JavaLanguageVersion.of(springAiReleaseInt)
+        }
+    }
 }
 
 dependencies {
@@ -59,4 +80,9 @@ dependencies {
 
 tasks.test {
     useJUnitPlatform()
+    if (!JavaVersion.current().isCompatibleWith(springAiJavaVersion)) {
+        javaLauncher = javaToolchains.launcherFor {
+            languageVersion = JavaLanguageVersion.of(springAiReleaseInt)
+        }
+    }
 }