Skip to content

Ag/client conformance#292

Closed
atesgoral wants to merge 5 commits intomainfrom
ag/client-conformance
Closed

Ag/client conformance#292
atesgoral wants to merge 5 commits intomainfrom
ag/client-conformance

Conversation

@atesgoral
Copy link
Copy Markdown
Contributor

Motivation and Context

How Has This Been Tested?

Breaking Changes

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

keisku and others added 5 commits April 4, 2026 00:30
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
When an SSE stream is active, the server responds with 202 Accepted
(no Content-Type, empty body) to POST requests, delivering the actual
response via the SSE stream. Previously this raised an unsupported
Content-Type error.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Move event_stream_parser from Gemfile to gemspec as runtime dependency
- Add brief descriptions to TODO comments in client/http.rb and
  streamable_http_transport.rb
- Add description to spec URL comment in client.rb
- Use SESSION_ID_HEADER constant for session ID header lookup in connect

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add conformance/client.rb for running MCP client conformance scenarios
(initialize, tools_call). Require event_stream_parser in http.rb so SSE
responses from the conformance server are parsed correctly.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Document how to run client conformance tests alongside the existing
server conformance documentation.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@atesgoral atesgoral closed this Apr 4, 2026
@koic
Copy link
Copy Markdown
Member

koic commented Apr 4, 2026

I had been working locally on organizing the client conformance tests with the intention of opening a PR after #210 settles, but I’ve opened it now as a PR #293 for the time being.

@atesgoral atesgoral deleted the ag/client-conformance branch April 4, 2026 15:57
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.

3 participants