Prepare for release 0.24.2.

CHANGELOG.md

@@ -4,6 +4,15 @@ Changelog
 **Unreleased**
 --------------
 
+0.24.2
+------
+
+_2025-01-30_
+
+- Fix loading of optional properties with defined default values.
+- Update to Kotlin `2.1.10`.
+- Build against KSP `2.1.10-1.0.29`.
+
 0.24.1
 ------
 

gradle.properties

@@ -52,4 +52,4 @@ POM_DEVELOPER_ID=slackhq
 POM_DEVELOPER_NAME=Slack Technologies, Inc.
 POM_DEVELOPER_URL=https://github.com/slackhq
 POM_INCEPTION_YEAR=2022
-VERSION_NAME=1.0.0-SNAPSHOT
+VERSION_NAME=0.24.2

gradle/libs.versions.toml

@@ -24,7 +24,7 @@ jvmTarget = "17"
 jewel = "0.27.0"
 jna = "5.16.0"
 kaml = "0.67.0"
-kotlin = "2.1.0"
+kotlin = "2.1.10"
 kotlinx-serialization = "1.8.0"
 ksp = "2.1.10-1.0.29"
 kotlinPoet = "2.0.0"

platforms/gradle/better-gradle-properties/src/main/kotlin/foundry/gradle/properties/PropertyResolver.kt

@@ -137,7 +137,9 @@ public class PropertyResolver(
     blankBehavior: BlankBehavior = BlankBehavior.ERROR,
   ): Provider<String> {
     return providerFor(key)
-      .let { defaultValue?.let { providers.provider { defaultValue } } ?: it }
+      .let { provider ->
+        defaultValue?.let { provider.orElse(providers.provider { defaultValue }) } ?: provider
+      }
       .filter {
         when (blankBehavior) {
           BlankBehavior.FILTER -> {

platforms/gradle/foundry-gradle-plugin/src/main/kotlin/foundry/gradle/AnnotationProcessing.kt

@@ -153,6 +153,7 @@ internal abstract class BasicAptOptionsConfig : AptOptionsConfig {
           configure<KaptExtension> {
             arguments {
               baseConfig.globalOptions(foundryProperties).forEach { (key, value) ->
+                logger.lifecycle("Adding kapt args to $path: $key=$value")
                 arg(key, value)
               }
             }
@@ -193,8 +194,8 @@ internal object AptOptionsConfigs {
   object Dagger : BasicAptOptionsConfig() {
     override val targetDependency: String = "dagger-compiler"
 
-    override fun globalOptions(foundryProperties: FoundryProperties): Map<String, String> =
-      foundryProperties.daggerOptions.getOrElse(DEFAULT_ARGS)
+    override fun globalOptions(properties: FoundryProperties): Map<String, String> =
+      properties.daggerOptions.getOrElse(DEFAULT_ARGS)
 
     private val DEFAULT_ARGS =
       mapOf(
@@ -214,7 +215,7 @@ internal object AptOptionsConfigs {
   object Moshi : BasicAptOptionsConfig() {
     override val targetDependency: String = "moshi-kotlin-codegen"
 
-    override fun globalOptions(foundryProperties: FoundryProperties): Map<String, String> =
+    override fun globalOptions(properties: FoundryProperties): Map<String, String> =
       mapOf("moshi.generated" to "javax.annotation.Generated")
   }
 }

platforms/gradle/foundry-gradle-plugin/src/main/kotlin/foundry/gradle/FoundryProperties.kt

@@ -735,10 +735,14 @@ internal constructor(
    */
   public val daggerOptions: Provider<Map<String, String>>
     get() =
-      resolver.optionalStringProvider("foundry.dagger.options", defaultValue = "").map { value ->
-        if (value.isBlank()) return@map emptyMap<String, String>()
-        value.splitToSequence(',').associate { kv -> kv.trim().split('=').let { it[0] to it[1] } }
-      }
+      resolver
+        .optionalStringProvider(
+          key = "foundry.dagger.options",
+          blankBehavior = PropertyResolver.BlankBehavior.FILTER,
+        )
+        .map { value ->
+          value.splitToSequence(',').associate { kv -> kv.trim().split('=').let { it[0] to it[1] } }
+        }
 
   /** Overrides the kotlin language version if present. */
   public val kaptLanguageVersion: Provider<KotlinVersion>