Repository navigation
Update palantir-java-format, Checker, langchain4j, the agent's deps and clang-format - #477
Merged
Merged
Conversation
…nd clang-format - palantir-java-format 2.100.0 -> 2.102.0 (core + agent): no reformatting. - Checker Framework 4.2.3 -> 4.3.0 (processor and checker-qual together, as they must share a major version). - langchain4j-core 1.20.2 -> 1.21.0 (llama-langchain4j). - Agent: Atmosphere 4.0.71 -> 4.0.72, Jetty 12.1.13 -> 12.1.14, JLine 4.4.6 -> 4.4.7. None of the three files the carried JLine fixes change (Status, Display, LineReaderImpl) differs between 4.4.6 and 4.4.7; 4.4.7 only fixes AttributedStringBuilder tabs, Curses.tputs, telnet and a history ReDoS. - clang-format 23.1.1 -> 23.1.3: zero churn over the checked tree. Checked: reactor install (SpotBugs included) green; core 1835 tests, llama-langchain4j 49, agent 277 (-Dllama.natives=none), 0 failures; check-natives, check-shared-files, check-run-scripts clean. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AytmJF9faEiQEVt6eetQS2
bernardladenthin
had a problem deploying
to
maven-central
October 6, 2026 22:11 — with
GitHub Actions
Failure
bernardladenthin
had a problem deploying
to
startgate
October 6, 2026 22:11 — with
GitHub Actions
Error
bernardladenthin
had a problem deploying
to
maven-central
October 6, 2026 22:11 — with
GitHub Actions
Failure
Review Summary ✅This is a well-executed dependency update PR with excellent documentation and testing. The changes are clean, consistent, and properly documented. Verification NotesStrengths:
Minor Note:
Recommendations
|
Detailed Change VerificationVersion Consistency Check ✅All version updates are properly coordinated across modules:
Bytecode Compatibility ✅
Documentation Consistency ✅All documentation accurately reflects version changes. No stale references found. Overall Assessment: This PR exhibits excellent quality control with thorough pre-CI validation. All cross-module dependencies are synchronized, documentation is complete, and the testing is comprehensive. |
|
This branch had an error being deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.



Summary
spotless:applychanges no source file.checker-qualmove together, because they must share a major version.llama-langchain4j).Status,DisplayandLineReaderImpl, and none of those three files differs between 4.4.6 and 4.4.7. 4.4.7 itself only fixesAttributedStringBuildertab positions,Curses.tputs, Telnet NEW-ENVIRON limits and a ReDoS in the history commands.The same palantir/Checker versions go to BitcoinAddressFinder, srcmorph and streambuffer in parallel PRs (streambuffer already has Checker 4.3.0 from Dependabot).
Test plan
install, including SpotBugs, is greenllama-langchain4j verify: 49 tests, 0 failuresverify -Dllama.natives=none: 277 tests, 0 failurescheck-natives,check-shared-filesandcheck-run-scriptsare cleanRelated issues / PRs
Part of the 2026-10-06 cross-repo dependency sweep. OpenJML 21.0.28 is left out on purpose; it is being evaluated separately.
Checklist
🤖 Generated with Claude Code
https://claude.ai/code/session_01AytmJF9faEiQEVt6eetQS2
Generated by Claude Code