Merge pull request #333 from bernardladenthin/claude/agp9-builtin-kotlin-fix-q3m8vz

Remove org.jetbrains.kotlin.android plugin for AGP 9 built-in Kotlin support

Author: bernardladenthin

CLAUDE.md

@@ -1652,11 +1652,16 @@ compiles/loads the API); this one has a UI and is driven end-to-end. The name is
 domain is the only string that must be globally unique.
 
 Structure (mirrors the consumer-test's plumbing):
-- **`settings.gradle.kts`** — `rootProject.name = "android-llmservice"`; pins AGP `9.2.0` + Kotlin
-  `2.4.0` + the Compose plugin; `mavenLocal()` first so the freshly-built AAR + façade resolve there
-  in CI (Maven Central for real users). AGP 9.x requires Gradle >= 9.4.1 and JDK 17+; CI already
-  runs JDK 21 everywhere (`env.JAVA_VERSION`), so only the `gradle-version` pin on the jobs that
-  build this project (and the `.github/android-consumer-test` fixture) needed bumping.
+- **`settings.gradle.kts`** — `rootProject.name = "android-llmservice"`; pins AGP `9.2.0` + the
+  Compose compiler plugin (`2.4.0`); `mavenLocal()` first so the freshly-built AAR + façade
+  resolve there in CI (Maven Central for real users). AGP 9.x requires Gradle >= 9.4.1 and
+  JDK 17+; CI already runs JDK 21 everywhere (`env.JAVA_VERSION`), so only the `gradle-version`
+  pin on the jobs that build this project (and the `.github/android-consumer-test` fixture)
+  needed bumping. AGP 9.0+ has **built-in Kotlin support** (a runtime dependency on Kotlin
+  Gradle plugin 2.2.10+), so the standalone `org.jetbrains.kotlin.android` plugin is no longer
+  applied — applying it now fails the build with "no longer required for Kotlin support since
+  AGP 9.0" (`app/build.gradle.kts` line 7). The Compose compiler plugin still applies
+  separately and its 2.4.0 pin exceeds AGP's 2.2.10 floor, so no other version changed.
 - **`app/build.gradle.kts`** — `namespace`/`applicationId` `net.ladenthin.android.llmservice`,
   `minSdk 28` (AAR floor), `compileSdk/targetSdk 35`, Jetpack Compose, `androidx.appcompat` (only for
   the per-app language API), ABIs `arm64-v8a` + `x86_64`. Depends on `net.ladenthin:llama-android` +

android-llmservice/app/build.gradle.kts

@@ -6,7 +6,8 @@ import org.jetbrains.kotlin.gradle.dsl.JvmTarget
 
 plugins {
     id("com.android.application")
-    id("org.jetbrains.kotlin.android")
+    // AGP 9.0+ built-in Kotlin support replaces the standalone kotlin-android plugin —
+    // see the note in settings.gradle.kts.
     id("org.jetbrains.kotlin.plugin.compose")
 }
 

android-llmservice/settings.gradle.kts

@@ -14,11 +14,14 @@ pluginManagement {
         gradlePluginPortal()
     }
     // Versions pinned here (KISS: no version catalog). AGP matches the consumer-test
-    // fixture; Kotlin matches llama-kotlin (2.4.0). AGP 9.x requires Gradle >= 9.4.1 —
-    // see the gradle-version pins on the CI jobs that build this project in publish.yml.
+    // fixture. AGP 9.x requires Gradle >= 9.4.1 — see the gradle-version pins on the CI
+    // jobs that build this project in publish.yml. AGP 9.0+ has built-in Kotlin support
+    // (runtime dependency on Kotlin Gradle plugin 2.2.10+), so the standalone
+    // org.jetbrains.kotlin.android plugin is no longer applied/needed — see
+    // https://developer.android.com/build/migrate-to-built-in-kotlin. The Compose
+    // compiler plugin (2.4.0) still applies separately and exceeds AGP's 2.2.10 floor.
     plugins {
         id("com.android.application") version "9.2.0"
-        id("org.jetbrains.kotlin.android") version "2.4.0"
         id("org.jetbrains.kotlin.plugin.compose") version "2.4.0"
     }
 }