Skip to content

Upgrade dependencies and document llama.cpp 5.2.0 test failures - #210

Merged
bernardladenthin merged 1 commit into
mainfrom
claude/busy-archimedes-lo9uyy
Sep 27, 2026
Merged

bernardladenthin merged 1 commit into
mainfrom
claude/busy-archimedes-lo9uyy

Conversation

@bernardladenthin

Copy link
Copy Markdown
Owner

Summary

  • Document four latent test failures in LlamaCppJniProviderSupportTest against llama.cpp 5.2.0 core, discovered 2026-09-27. The failures stem from assertions that call InferenceParameters.toString() expecting JSON, but in 5.2.0 toString() is a redacted debug view; tests should call toJson() instead.
  • Upgrade minor/patch versions across the dependency tree: Jackson 2.22.2 → 2.22.3, SLF4J 2.0.19 → 2.0.20, Logback 1.6.3 → 1.6.4, NullAway 0.14.1 → 0.14.2, ArchUnit 1.5.0 → 1.5.1, Spotless 3.10.2 → 3.10.3, Palantir Java Format 2.98.0 → 2.99.0.
  • Update CLAUDE.md dependency table to reflect SLF4J and Logback version bumps.

Test plan

  • CI is green on this branch
  • No code logic changes; dependency upgrades are transitive and configuration-only
  • Documentation updates (TODO.md, CLAUDE.md) are informational

Related issues / PRs

Refs the llama.cpp 5.2.0 core pin (documented in TODO.md as a blocking issue).

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

…10.3, palantir 2.99.0, NullAway 0.14.2, archunit 1.5.1

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. CLAUDE.md's
dependency table follows.

Verified as far as this repo currently allows. The <llama.version>5.2.0</llama.version>
pin still does not resolve from Central (TODO.md), so the build was run against
java-llama.cpp main installed locally (-Dllama.version=5.2.0-SNAPSHOT):

- spotless:apply under palantir 2.99.0 changes no file in any of the three modules.
- srcmorph / srcmorph-cli / srcmorph-maven-plugin: 634 / 39 / 33 tests green.
- Two things stay red and are NOT caused by this change -- both reproduce on the
  unmodified tree:
  * 4 findsecbugs findings in LlamaCppJniProviderSupport (already in TODO.md);
  * 4 failures in LlamaCppJniProviderSupportTest (buildInferenceParameters_*),
    which read InferenceParameters.toString() as JSON. In the 5.2.0-SNAPSHOT core
    toString() is a redacted debug view and toJson() is the request body, so those
    assertions need to move to toJson() when the 5.2.0 pin is resolved.

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

This branch had an error being deployed

1 failed deployment
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