Skip to content

fix(mcp-server): advertise server capabilities in initialize response#5

Open
jonathanNi98 wants to merge 1 commit into
openJiuwen-ai:developfrom
jonathanNi98:issue_fix_2
Open

fix(mcp-server): advertise server capabilities in initialize response#5
jonathanNi98 wants to merge 1 commit into
openJiuwen-ai:developfrom
jonathanNi98:issue_fix_2

Conversation

@jonathanNi98
Copy link
Copy Markdown

  • Fix InitializeResult serializer to emit actual capability fields instead of hardcoded empty object
  • Add BuildServerCapabilities() to dynamically compute capabilities from registered tools, prompts, and resources
  • Add RefreshServerCapabilities() called after every Add/Remove operation to keep capabilities in sync
  • Wire SetServerCapabilities() into InitializeServerManager(), StdioServerManagerStart(), and NewSession() so all sessions receive current capabilities
  • Fix case-insensitive Accept/Content-Type header lookup per RFC 7230

Fixes #2

- Fix InitializeResult serializer to emit actual capability fields
  instead of hardcoded empty object
- Add BuildServerCapabilities() to dynamically compute capabilities
  from registered tools, prompts, and resources
- Add RefreshServerCapabilities() called after every Add/Remove
  operation to keep capabilities in sync
- Wire SetServerCapabilities() into InitializeServerManager(),
  StdioServerManagerStart(), and NewSession() so all sessions
  receive current capabilities
- Fix case-insensitive Accept/Content-Type header lookup per RFC 7230

Fixes openJiuwen-ai#2
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