Skip to content

Conversation

@koic
Copy link
Member

@koic koic commented Nov 29, 2025

Motivation and Context

This PR adds support for the optional websiteUrl parameter in serverInfo. This parameter has been supported since the 2025-11-25 MCP specification.

This PR also implements part of #127.

How Has This Been Tested?

The tests have been updated and are now passing.

Breaking Changes

None.

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

@koic koic force-pushed the add_website_url_to_server_info branch from 43f9409 to 2fa9916 Compare December 20, 2025 17:51
end

test "server response does not include title when not configured" do
test "server response does not include optionnal parameters when configured" do
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo: optionnal -> optional

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops! I fixed it.

This PR adds support for the optional `websiteUrl` parameter in `serverInfo`.
This parameter has been supported since the 2025-11-25 MCP specification.

This PR also implements part of modelcontextprotocol#127.
@koic koic force-pushed the add_website_url_to_server_info branch from 2fa9916 to 62615d6 Compare December 21, 2025 13:45
@koic koic merged commit 771163c into modelcontextprotocol:main Dec 23, 2025
6 checks passed
@koic koic deleted the add_website_url_to_server_info branch December 23, 2025 15:49
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