Skip to content

Change sample to work with Claude Desktop #48

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

SebastianAigner
Copy link
Contributor

@SebastianAigner SebastianAigner commented Mar 7, 2025

Claude Desktop expects the following:

  • No stdout emissions before the actual JSON interactions (removed the Server running... text)
  • Tool name to conform to a regex:
Screenshot 2025-03-07 at 11 56 22

This PR makes the required changes to the example project.

After these changes, both MCP inspector and Claude Desktop behave correctly:

Screenshot 2025-03-07 at 11 58 35

… not emit any text on stdout when running in stdio mode
@e5l e5l enabled auto-merge (squash) March 7, 2025 11:41
@e5l
Copy link
Contributor

e5l commented Mar 7, 2025

Hey @SebastianAigner, thank you for the PR. LGTM

@e5l e5l merged commit c57c56e into modelcontextprotocol:main Mar 7, 2025
1 check passed
@SebastianAigner SebastianAigner deleted the aigner-fix-for-claude-desktop branch March 7, 2025 15:36
tinycrops pushed a commit to tinycrops/kotlin-sdk that referenced this pull request Mar 8, 2025
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