chore(build): update Gradle and Kotlin tooling

Update Gradle, Kotlin, Dokka, ktlint, cloud-build-logic, and GitHub
Actions. Configure the Kotlin language and API versions independently
from the compiler plugin and standard library versions.

Update Sonatype Central publishing to use the new snapshot repository,
Central Portal plugin APIs, and publication tasks.

Author: jpenilla

.github/workflows/build.yml

@@ -12,21 +12,21 @@ jobs:
     if: ${{ github.event_name != 'pull_request' || github.repository != github.event.pull_request.head.repo.full_name }}
     runs-on: "ubuntu-latest"
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v7
       - name: Set up JDK
-        uses: actions/setup-java@v4
+        uses: actions/setup-java@v5
         with:
           distribution: 'temurin'
           java-version: 21
-      - uses: gradle/actions/setup-gradle@v4
+      - uses: gradle/actions/setup-gradle@v6
         with:
           # allow master and *-dev branches to write caches (default is only master/main)
           cache-read-only: ${{ github.ref != 'refs/heads/master' && !(endsWith(github.ref, '-dev') && startsWith(github.ref, 'refs/heads/')) }}
       - name: Build
         run: ./gradlew build
       - name: Upload Test Results
         if: always()
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v7
         with:
           name: Test Results
           path: |
@@ -40,13 +40,13 @@ jobs:
           fi
       - name: Publish Snapshot
         if: "${{ env.STATUS != 'release' && github.event_name == 'push' && github.ref == 'refs/heads/master' }}"
-        run: ./gradlew publish
+        run: ./gradlew publishAllPublicationsToSonatypeSnapshots
         env:
           ORG_GRADLE_PROJECT_sonatypeUsername: "${{ secrets.SONATYPE_USERNAME }}"
           ORG_GRADLE_PROJECT_sonatypePassword: "${{ secrets.SONATYPE_PASSWORD }}"
       - name: Publish Release
         if: "${{ env.STATUS == 'release' && github.event_name == 'release' }}"
-        run: ./gradlew publishToSonatype closeAndReleaseSonatypeStagingRepository
+        run: ./gradlew publishReleaseCentralPortalBundle
         env:
           ORG_GRADLE_PROJECT_sonatypeUsername: "${{ secrets.SONATYPE_USERNAME }}"
           ORG_GRADLE_PROJECT_sonatypePassword: "${{ secrets.SONATYPE_PASSWORD }}"
@@ -59,7 +59,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Upload
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v7
         with:
           name: Event File
           path: ${{ github.event_path }}

.github/workflows/dependency_submission.yml

@@ -11,9 +11,9 @@ jobs:
   dependency-submission:
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@v4
-    - uses: actions/setup-java@v4
+    - uses: actions/checkout@v7
+    - uses: actions/setup-java@v5
       with:
         distribution: 'temurin'
         java-version: 21
-    - uses: gradle/actions/dependency-submission@v3
+    - uses: gradle/actions/dependency-submission@v6

.github/workflows/test_results.yml

@@ -20,7 +20,7 @@ jobs:
       actions: read
     steps:
       - name: Download and Extract Artifacts
-        uses: dawidd6/action-download-artifact@v3
+        uses: dawidd6/action-download-artifact@v6
         with:
             run_id: ${{ github.event.workflow_run.id }}
             path: artifacts

cloud-kotlin/cloud-kotlin-coroutines-annotations/src/main/kotlin/org/incendo/cloud/kotlin/coroutines/annotations/KotlinAnnotatedMethods.kt

@@ -143,8 +143,8 @@ private class KotlinSuggestionProvider<C>(
     private val instance: Any
 ) : SuggestionProvider<C> {
 
-    override fun suggestionsFuture(context: CommandContext<C>, input: CommandInput): CompletableFuture<Iterable<Suggestion>> {
-        return coroutineScope.future(coroutineContext) {
+    override fun suggestionsFuture(context: CommandContext<C>, input: CommandInput): CompletableFuture<Iterable<Suggestion>> =
+        coroutineScope.future(coroutineContext) {
             try {
                 if (kFunction.valueParameters.isEmpty()) {
                     kFunction.callSuspend(instance)
@@ -163,7 +163,6 @@ private class KotlinSuggestionProvider<C>(
                 else -> MethodSuggestionProvider.mapSuggestions(result)
             }
         }
-    }
 }
 
 private class KotlinMethodArgumentParserFactory<C>(
@@ -227,7 +226,8 @@ private class KotlinMethodArgumentParser<C, T>(
     private val suggestionProvider: SuggestionProvider<C>,
     javaMethod: Method,
     parameterInjectorRegistry: ParameterInjectorRegistry<C>
-) : FutureArgumentParser<C, T>, AnnotatedMethodHandler<C>(javaMethod, instance, parameterInjectorRegistry) {
+) : AnnotatedMethodHandler<C>(javaMethod, instance, parameterInjectorRegistry),
+    FutureArgumentParser<C, T> {
 
     private val paramsWithoutContinuation = parameters()
         .filterNot { Continuation::class.java == it.type }
@@ -236,33 +236,31 @@ private class KotlinMethodArgumentParser<C, T>(
     override fun parseFuture(
         commandContext: CommandContext<C>,
         commandInput: CommandInput
-    ): CompletableFuture<ArgumentParseResult<T>> =
-        executeSuspendFunction(
-            coroutineScope,
-            coroutineContext,
-            kFunction,
-            instance,
-            paramsWithoutContinuation,
-            commandContext,
-            listOf(commandInput)
-        )
-            .mapResult()
+    ): CompletableFuture<ArgumentParseResult<T>> = executeSuspendFunction(
+        coroutineScope,
+        coroutineContext,
+        kFunction,
+        instance,
+        paramsWithoutContinuation,
+        commandContext,
+        listOf(commandInput)
+    )
+        .mapResult()
 
     override fun suggestionProvider(): SuggestionProvider<C> = suggestionProvider
 
     @Suppress("UNCHECKED_CAST")
-    private fun <T> CompletableFuture<*>.mapResult(): CompletableFuture<ArgumentParseResult<T>> =
-        handle { result, exception ->
-            if (exception != null) {
-                ArgumentParseResult.failure(exception)
-            } else {
-                when (result) {
-                    null -> ArgumentParseResult.failure(IllegalArgumentException("Result not found"))
-                    is ArgumentParseResult<*> -> result as ArgumentParseResult<T>
-                    else -> ArgumentParseResult.success((result as T)!!)
-                }
+    private fun <T> CompletableFuture<*>.mapResult(): CompletableFuture<ArgumentParseResult<T>> = handle { result, exception ->
+        if (exception != null) {
+            ArgumentParseResult.failure(exception)
+        } else {
+            when (result) {
+                null -> ArgumentParseResult.failure(IllegalArgumentException("Result not found"))
+                is ArgumentParseResult<*> -> result as ArgumentParseResult<T>
+                else -> ArgumentParseResult.success((result as T)!!)
             }
         }
+    }
 }
 
 private fun <C> AnnotatedMethodHandler<C>.executeSuspendFunction(
@@ -324,5 +322,4 @@ private fun <C> AnnotatedMethodHandler<C>.executeSuspendFunction(
     }
 }
 
-private fun KParameter.hasType(clazz: Class<*>): Boolean =
-    GenericTypeReflector.erase(type.javaType) == clazz
+private fun KParameter.hasType(clazz: Class<*>): Boolean = GenericTypeReflector.erase(type.javaType) == clazz

cloud-kotlin/cloud-kotlin-coroutines-annotations/src/test/kotlin/org/incendo/cloud/kotlin/coroutines/annotations/KotlinAnnotatedMethodsTest.kt

@@ -219,23 +219,23 @@ class KotlinAnnotatedMethodsTest {
 
     public class TestCommandSender
 
-    private class TestCommandManager : CommandManager<TestCommandSender>(
-        ExecutionCoordinator.builder<TestCommandSender>()
-            .executor(executorService)
-            .build(),
-        CommandRegistrationHandler.nullCommandRegistrationHandler()
-    ) {
+    private class TestCommandManager :
+        CommandManager<TestCommandSender>(
+            ExecutionCoordinator.builder<TestCommandSender>()
+                .executor(executorService)
+                .build(),
+            CommandRegistrationHandler.nullCommandRegistrationHandler()
+        ) {
 
         override fun hasPermission(sender: TestCommandSender, permission: String): Boolean = true
     }
 
     public class CommandMethods {
 
         @Command("test")
-        public suspend fun suspendingCommand(): Unit =
-            withContext(Dispatchers.Default) {
-                println("called from thread: ${Thread.currentThread().name}")
-            }
+        public suspend fun suspendingCommand(): Unit = withContext(Dispatchers.Default) {
+            println("called from thread: ${Thread.currentThread().name}")
+        }
 
         @Command("test-exception")
         public suspend fun suspendingCommandWithException(): Unit = throw IllegalStateException()
@@ -268,10 +268,9 @@ class KotlinAnnotatedMethodsTest {
     class ParserMethods {
 
         @Parser
-        suspend fun suspendingParser(input: CommandInput): ParserResult =
-            withContext(Dispatchers.Default) {
-                ParserResult(input.lastRemainingToken().toInt())
-            }
+        suspend fun suspendingParser(input: CommandInput): ParserResult = withContext(Dispatchers.Default) {
+            ParserResult(input.lastRemainingToken().toInt())
+        }
 
         @Parser
         suspend fun suspendingParser2(input: CommandInput): ArgumentParseResult<ParserResult2> =
@@ -280,9 +279,8 @@ class KotlinAnnotatedMethodsTest {
             }
 
         @Parser
-        suspend fun exceptionParser(input: CommandInput): ParserResult3 =
-            withContext(Dispatchers.Default) {
-                throw IllegalStateException()
-            }
+        suspend fun exceptionParser(input: CommandInput): ParserResult3 = withContext(Dispatchers.Default) {
+            throw IllegalStateException()
+        }
     }
 }

cloud-kotlin/cloud-kotlin-coroutines/src/main/kotlin/org/incendo/cloud/kotlin/coroutines/SuspendingArgumentParser.kt

@@ -26,9 +26,9 @@ package org.incendo.cloud.kotlin.coroutines
 import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.GlobalScope
 import kotlinx.coroutines.future.future
-import org.incendo.cloud.CommandManager
 import org.incendo.cloud.context.CommandContext
 import org.incendo.cloud.context.CommandInput
+import org.incendo.cloud.execution.CommandExecutor
 import org.incendo.cloud.execution.ExecutionCoordinator
 import org.incendo.cloud.parser.ArgumentParseResult
 import org.incendo.cloud.parser.ArgumentParser
@@ -50,7 +50,7 @@ public fun interface SuspendingArgumentParser<C : Any, T : Any> {
      * Returns the result of parsing the given [commandInput].
      *
      * This method may be called when a command chain is being parsed for execution
-     * (using [CommandManager.executeCommand])
+     * (using [CommandExecutor.executeCommand])
      * or when a command is being parsed to provide context for suggestions
      * (using [SuggestionFactory.suggest]).
      * It is possible to use [CommandContext.isSuggestions] to see what the purpose of the

cloud-kotlin/cloud-kotlin-coroutines/src/main/kotlin/org/incendo/cloud/kotlin/coroutines/SuspendingSuggestionProvider.kt

@@ -77,12 +77,11 @@ public fun interface SuspendingSuggestionProvider<C : Any> {
             scope: CoroutineScope = GlobalScope,
             context: CoroutineContext = EmptyCoroutineContext,
             provider: SuspendingSuggestionProvider<C>
-        ): SuggestionProvider<C> =
-            SuggestionProvider { ctx, input ->
-                scope.future(context) {
-                    provider(ctx, input).toList()
-                }
+        ): SuggestionProvider<C> = SuggestionProvider { ctx, input ->
+            scope.future(context) {
+                provider(ctx, input).toList()
             }
+        }
     }
 }
 

cloud-kotlin/cloud-kotlin-coroutines/src/test/kotlin/org/incendo/cloud/kotlin/coroutines/SuspendingArgumentParserTest.kt

@@ -72,12 +72,13 @@ class SuspendingArgumentParserTest {
 
     private class TestCommandSender
 
-    private class TestCommandManager : CommandManager<TestCommandSender>(
-        ExecutionCoordinator.builder<TestCommandSender>()
-            .executor(executorService)
-            .build(),
-        CommandRegistrationHandler.nullCommandRegistrationHandler()
-    ) {
+    private class TestCommandManager :
+        CommandManager<TestCommandSender>(
+            ExecutionCoordinator.builder<TestCommandSender>()
+                .executor(executorService)
+                .build(),
+            CommandRegistrationHandler.nullCommandRegistrationHandler()
+        ) {
 
         override fun hasPermission(sender: TestCommandSender, permission: String): Boolean = true
     }

cloud-kotlin/cloud-kotlin-coroutines/src/test/kotlin/org/incendo/cloud/kotlin/coroutines/SuspendingHandlerTest.kt

@@ -59,12 +59,13 @@ class SuspendingHandlerTest {
 
     private class TestCommandSender
 
-    private class TestCommandManager : CommandManager<TestCommandSender>(
-        ExecutionCoordinator.builder<TestCommandSender>()
-            .executor(executorService)
-            .build(),
-        CommandRegistrationHandler.nullCommandRegistrationHandler()
-    ) {
+    private class TestCommandManager :
+        CommandManager<TestCommandSender>(
+            ExecutionCoordinator.builder<TestCommandSender>()
+                .executor(executorService)
+                .build(),
+            CommandRegistrationHandler.nullCommandRegistrationHandler()
+        ) {
 
         override fun hasPermission(sender: TestCommandSender, permission: String): Boolean = true
     }