Repository navigation
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Problem:
kotlinlang-mcp-serveris the only sample missing from thesamplesmatrix in.github/workflows/samples.yml(which listskotlin-mcp-client,kotlin-mcp-server,simple-streamable-server,weather-stdio-server). The checks on this PR confirm it: fourBuild Sample (...)jobs ran, none for the sample this PR actually changes. So this toolchain bump — and every future Dependabot bump to this directory — merges without ever being compiled or tested.Failure scenario: The gap has already bitten. Building this sample locally,
KotlinlangServerTest > server registers both tools with correct schema and annotationsfails withMcpException: Request timed out(Protocol.requestresolving underrunTest's virtual-time scheduler). It reproduces identically on2.4.20and2.4.21, so it is not caused by this bump — it is pre-existing red onmainthat no CI job observes. (Caveat: my sandbox had no JDK 21, so I retargetedjvmToolchain(21)to JDK 25 withjvmTarget = 21; the other 55 tests passed on both versions.) The next bump that genuinely breaks compilation will land just as silently.Suggested fix: Add
- kotlinlang-mcp-serverto the matrix in.github/workflows/samples.yml, then fix the timed-out test (give it a real dispatcher, or an explicit request timeout, instead of letting virtual time fast-forward the SDK's timeout) so the newly-enabled job starts green. Non-blocking for this bump itself.