Skip to content

MCP: serve only protocol 2026-07-28 - #2659

Open
erichare wants to merge 3 commits into
mainfrom
feat/mcp-protocol-2026-07-28
Open

erichare wants to merge 3 commits into
mainfrom
feat/mcp-protocol-2026-07-28

Conversation

@erichare

@erichare erichare commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

What this PR does:

Moves Quarkiverse MCP Server to 2.1.0.Beta1 and serves only protocol 2026-07-28. Older and unknown versions, initialize, sessions, GET, DELETE, and the legacy HTTP+SSE routes are rejected. Discovery and tool-list cache hints are private with ttlMs 0. Shared test clients use stateless mode, and McpProtocolTest covers the raw HTTP wire contract.

Which issue(s) this PR fixes:

Part of #2625 and #2626.

Verification

JAVA_HOME=$(/usr/libexec/java_home -v 21) ./mvnw -B -ntp com.spotify.fmt:fmt-maven-plugin:check -Dtest=McpProtocolTest test

Passed: 33 tests, 0 failures, 0 errors, 0 skipped. Formatter checked 1,194 files with 0 non-complying.

Not run locally: the full unit suite and DSE/HCD integration suites. OAuth remains in #2642. The framework still advertises logging, returns 500 when clientInfo has no name, and returns HTTP 200 with a JSON-RPC parse error for malformed JSON.

Checklist

  • Changes manually tested
  • Automated Tests added/updated
  • Documentation added/updated
  • CLA Signed: DataStax CLA

Move Quarkiverse MCP Server to 2.1.0.Beta1 and restrict the endpoint to
2026-07-28 with supported-protocol-versions. Keep discovery and tool list
cache hints private and immediately stale. Switch the shared test clients
to stateless mode and check the raw HTTP contract in McpProtocolTest.
A test class that injects ToolManager makes the framework advertise
tools.listChanged in every QuarkusTest app. The packaged server has no
such injection point, so assert it there instead.
@erichare
erichare marked this pull request as ready for review October 23, 2026 22:05
@erichare
erichare requested a review from a team as a code owner October 23, 2026 22:05
@github-actions

Copy link
Copy Markdown
Contributor

➡️ Unit Test Coverage Delta vs Main Branch

Metric Value
Main Branch 55.07%
This PR 55.07%
Delta ⚪ 0.00%
ℹ️ Coverage unchanged

@github-actions

Copy link
Copy Markdown
Contributor

Unit Test Coverage Report

Overall Project 55.07% 🍏

There is no coverage information present for the Files changed

@github-actions

Copy link
Copy Markdown
Contributor

➡️ Integration Test Coverage Delta vs Main Branch (hcd-it)

Metric Value
Main Branch 74.35%
This PR 74.35%
Delta ⚪ 0.00%
ℹ️ Coverage unchanged

@github-actions

Copy link
Copy Markdown
Contributor

Integration Test Coverage Report (hcd-it)

Overall Project 74.35% 🍏

There is no coverage information present for the Files changed

@github-actions

Copy link
Copy Markdown
Contributor

📉 Integration Test Coverage Delta vs Main Branch (dse69-it)

Metric Value
Main Branch 73.04%
This PR 73.03%
Delta 🔴 -0.00%
⚠️ Coverage decreased

@github-actions

Copy link
Copy Markdown
Contributor

Integration Test Coverage Report (dse69-it)

Overall Project 73.03% 🍏

There is no coverage information present for the Files changed

This branch has not been deployed

No deployments
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.

1 participant