Skip to content

Upgrade dependencies and fix code formatting - #455

Merged
bernardladenthin merged 3 commits into
mainfrom
claude/busy-archimedes-lo9uyy
Sep 27, 2026
Merged

bernardladenthin merged 3 commits into
mainfrom
claude/busy-archimedes-lo9uyy

Conversation

@bernardladenthin

Copy link
Copy Markdown
Owner

Summary

  • Upgrade Gradle from 9.7.1 to 9.8.0 across all CI workflows and update AGP requirement comments from 9.3.0 to 9.4.0
  • Bump dependency versions: NullAway, Jackson, SLF4J, Logback, ArchUnit, Spotless, Palantir Java Format, Atmosphere, JLine, Exec Maven Plugin, Compose compiler plugin, AppCompat, Compose BOM, and langchain4j
  • Fix code formatting in LoopOptions.java (switch expression indentation)
  • Add safety guard in LlamaLoggerTest.restoreConsoleLogging() to skip cleanup when native library is unavailable (fixes SonarQube job failures)

Test plan

  • Affected unit tests pass locally (especially LlamaLoggerTest which now guards against missing native library)
  • CI is green on this branch (all workflows use updated Gradle 9.8.0)
  • No breaking changes; all dependency upgrades are patch/minor versions

Related issues / PRs

None

Checklist

  • I have read CONTRIBUTING.md and CODE_OF_CONDUCT.md
  • My commits follow Conventional Commits
  • No security-sensitive changes

https://claude.ai/code/session_01QSXAwbXqx9u5xsMArMvgT8

Cross-repo sweep of 2026-09-27, applied to all four Java repos together so the
tool-version matrix in workspace/crossrepostatus.md stays identical.

  llama:                  jackson 2.22.3, slf4j 2.0.20, logback 1.6.4 (test),
                          archunit 1.5.1, NullAway 0.14.2, spotless 3.10.3,
                          palantir-java-format 2.99.0
  llama-langchain4j:      langchain4j-core 1.20.1
  llama-atmosphere-agent: atmosphere-ai 4.0.71, jline 4.4.6, slf4j 2.0.20,
                          spotless 3.10.3, palantir 2.99.0, and exec-maven-plugin
                          3.5.0 -> 3.6.4 (the 2026-09-20 sweep missed this
                          standalone pom)

palantir 2.99.0 lays out an assigned switch expression differently;
spotless:apply rewrote LoopOptions.java, formatting only.

Verified: reactor install; llama verify (1772 tests, spotbugs:check);
llama-langchain4j verify (49); llama-kotlin verify (6);
llama-atmosphere-agent verify against the installed 5.2.0-SNAPSHOT (159) and
spotless:check.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QSXAwbXqx9u5xsMArMvgT8
…, Compose compiler plugin 2.4.20

- gradle-version 9.7.1 -> 9.8.0 in all seven setup-gradle steps. AGP 9.4.0 needs
  >= 9.6.0, so nothing else has to move; the four stale "AGP 9.3.0 requires
  Gradle >= 9.5.0" comments now name the real floor.
- android-llmservice: Compose BOM 2026.06.01 -> 2026.09.00, appcompat 1.7.1 ->
  1.8.0 (minSdk 23, the app is on 28), Compose compiler plugin 2.4.10 -> 2.4.20
  (now the same Kotlin as llama-kotlin).
- CLAUDE.md, android-llmservice/README.md and the Atmosphere version note follow.

Not buildable here: dl.google.com (AGP, androidx) is blocked in this sandbox, so
the Android side is validated by CI's build-android-llmservice and
android-consumer-test jobs, not locally.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QSXAwbXqx9u5xsMArMvgT8
JUnit runs @AfterEach even when the test body was aborted by an assumption, so
restoreConsoleLogging() called LlamaModel.setLogger on every skipped test. Without
the native library that fails in LlamaModel's static initializer
(UnsatisfiedLinkError, then NoClassDefFoundError), which is what has kept the
SonarQube workflow red on main and on every PR: 6 errors, all
LlamaLoggerTest.restoreConsoleLogging:85.

The teardown now uses the same nativeLibraryOnClasspath() guard as the tests.

Verified: mvn -f llama/pom.xml verify without a native library -- 1772 tests,
0 failures, 0 errors (was 6 errors).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QSXAwbXqx9u5xsMArMvgT8
@bernardladenthin
bernardladenthin merged commit 2f9bd00 into main Sep 27, 2026
10 of 15 checks passed
@bernardladenthin
bernardladenthin deleted the claude/busy-archimedes-lo9uyy branch September 27, 2026 09:47
@sonarqubecloud

Copy link
Copy Markdown

This branch had an error being deployed

1 failed deployment
startgate — 865b7410 Deployed Sep 27, 2026 by bernardladenthin via Start gate (abort window) #1035
maven-central — 865b7410 Deployed Sep 27, 2026 by bernardladenthin via Verify GPG signing key (no secrets printed) #1035
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