Skip to content

MCP: upgrade framework and test stateless tools/list - #2644

Draft
erichare wants to merge 3 commits into
mainfrom
fix/mcp-protocol-foundation
Draft

erichare wants to merge 3 commits into
mainfrom
fix/mcp-protocol-foundation

Conversation

@erichare

@erichare erichare commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

What this PR does:

Upgrades MCP from 1.13.0 to 2.0.3, tests session-free 2026-07-28 tools/list, and updates integration assertions for framework serverInfo metadata. Version restrictions depend on quarkiverse/quarkus-mcp-server#1078. OAuth follows separately.

Validation: regression red → green, seven focused tests, 17 affected DSE integration tests and packaging passed locally. Full DSE/HCD CI remains required.

Which issue(s) this PR fixes:

Part of #2625 (M1, M2, M8 in #2408). Leaves #2625 open.

Checklist

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

RED: McpProtocolTest executes against MCP 1.13.0 and fails because the first tools/list POST returns HTTP 400 instead of 200.
GREEN: the same raw HTTP McpProtocolTest passes with MCP BOM 2.0.3 on Quarkus 3.38.1 and JDK 21. Both independent tools/list requests return complete results without Mcp-Session-Id.
@github-actions

Copy link
Copy Markdown
Contributor

📈 Unit Test Coverage Delta vs Main Branch

Metric Value
Main Branch 54.71%
This PR 54.77%
Delta 🟢 +0.06%
✅ Coverage improved!

@github-actions

Copy link
Copy Markdown
Contributor

Unit Test Coverage Report

Overall Project 54.77% 🍏

There is no coverage information present for the Files changed

Both DSE and HCD CI exposed four obsolete empty-metadata assertions after the framework upgrade. Require only serverInfo and validate its name and version. The two affected DSE integration classes pass locally: 17 tests, zero failures.
@github-actions

Copy link
Copy Markdown
Contributor

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

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

@github-actions

Copy link
Copy Markdown
Contributor

Integration Test Coverage Report (dse69-it)

Overall Project 72.95% 🍏

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.27%
This PR 74.27%
Delta ⚪ 0.00%
ℹ️ Coverage unchanged

@github-actions

Copy link
Copy Markdown
Contributor

Integration Test Coverage Report (hcd-it)

Overall Project 74.27% 🍏

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.

2 participants