Skip to content

fix(android-llmservice): migrate Kotlin jvmTarget to compilerOptions DSL; Gradle 8.14.4; allowBackup=false#311

Merged
bernardladenthin merged 1 commit into
mainfrom
claude/android-llmservice-build-fix
Jul 9, 2026
Merged

fix(android-llmservice): migrate Kotlin jvmTarget to compilerOptions DSL; Gradle 8.14.4; allowBackup=false#311
bernardladenthin merged 1 commit into
mainfrom
claude/android-llmservice-build-fix

Conversation

@bernardladenthin

Copy link
Copy Markdown
Owner

Summary

  • Fix the build-android-llmservice CI failure. With Kotlin 2.4.0 the android { kotlinOptions { jvmTarget = "17" } } string DSL is now a hard error. Migrated to the compilerOptions DSL: a top-level kotlin { compilerOptions { jvmTarget.set(JvmTarget.JVM_17) } } block (+ import org.jetbrains.kotlin.gradle.dsl.JvmTarget).
  • Bump the app CI job to Gradle 8.14.4 (clears the Kotlin-plugin deprecation warning for Gradle < 8.14.4 and picks up two Gradle security fixes). AGP 8.7.3 / Kotlin 2.4.0 stay; the AAR/lib jobs remain on 8.14.3 (they use no Kotlin Gradle plugin, so are unaffected).
  • allowBackup=false in the app manifest — resolves the CodeQL "Application backup allowed" alert and matches the app's privacy promise (the saved session + copied model in filesDir stay out of adb/cloud backup).

Test plan

  • Affected unit / integration tests pass locally — not runnable in this environment (no Android SDK); the failure was at Gradle config/script-compile, so downstream stages (Kotlin compile, Compose, R8, emulator UI test) are validated for the first time by this PR's CI.
  • CI is green on this branch — to be confirmed by this PR run (build-android-llmservice).
  • Docs / CHANGELOG updated where applicable — n/a: build-config + manifest fix; no doc surface changes.

Related issues / PRs

Refs #310 (introduced android-llmservice; its merge-to-main run surfaced this build failure + the CodeQL alert).

Checklist

  • I have read CONTRIBUTING.md and CODE_OF_CONDUCT.md
  • My commits follow Conventional Commits
  • No security-sensitive changes — the allowBackup=false change is a public hardening (resolves a code-scanning alert), not a sensitive vulnerability requiring private disclosure.

🤖 Generated with Claude Code

https://claude.ai/code/session_01HTS5FpMtBLJENTGoRUrp5m


Generated by Claude Code

…DSL; bump Gradle 8.14.4; allowBackup=false

CI build-android-llmservice failed at Gradle config: with Kotlin 2.4.0 the
android { kotlinOptions { jvmTarget = "17" } } string DSL is now a hard error.
- Migrate to the compilerOptions DSL: top-level kotlin { compilerOptions {
  jvmTarget.set(JvmTarget.JVM_17) } } (import org.jetbrains.kotlin.gradle.dsl.JvmTarget).
- Bump the app CI job to Gradle 8.14.4 (clears the Kotlin-plugin deprecation warning for
  Gradle < 8.14.4 and picks up two Gradle security fixes). AGP 8.7.3 stays; the AAR/lib
  jobs remain on 8.14.3 (they use no Kotlin Gradle plugin, so are unaffected).
- allowBackup=false: resolves the CodeQL "Application backup allowed" alert and matches the
  app's privacy promise — the saved session + copied model in filesDir stay out of adb/cloud backup.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HTS5FpMtBLJENTGoRUrp5m
@bernardladenthin bernardladenthin merged commit daf5043 into main Jul 9, 2026
11 of 15 checks passed
@bernardladenthin bernardladenthin deleted the claude/android-llmservice-build-fix branch July 9, 2026 20:32
@sonarqubecloud

sonarqubecloud Bot commented Jul 9, 2026

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants