Skip to content

Fix typo for Claude desktop config file path correction and add more example for Windows, Linux, and MacOS #107

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
merged 5 commits into from
Apr 9, 2025

Conversation

Rayato159
Copy link
Contributor

Fix typo in Claude Desktop config file path and add additional example usage for Windows, Linux, and macOS environments.

Motivation and Context

Previously, the documentation had a typo in the Claude Desktop config file path, which may cause confusion, especially for new users. Additionally, the examples provided were limited and OS-agnostic, which may not be intuitive for developers on specific platforms.

This PR:

  • Fixes the path format to match Claude Desktop's expected configuration directory.
  • Add additional examples demonstrating how to run an MCP server on Windows, Linux, and macOS, making onboarding more approachable for each environment while building and add them to claude_desktop_config.json.

How Has This Been Tested?

Tested locally with the following setup:

  • Claude Desktop 0.9.1 (Windows 11)
  • Rust MCP Server (servers_std_io)
  • Verified path behavior and config loading across different platforms.

Breaking Changes

No breaking changes. This PR updates documentation only.

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

No additional context.

jokemanfire
jokemanfire previously approved these changes Apr 8, 2025
@jokemanfire
Copy link
Collaborator

jokemanfire commented Apr 9, 2025

Sorry for the conflict caused by the late merging. Could you resolve it?

@Rayato159
Copy link
Contributor Author

Dear @jokemanfire,
I've fixed the conflict and some corrections.

Please review again when you have a chance 🙇‍♂️.

@Rayato159 Rayato159 requested a review from jokemanfire April 9, 2025 02:35
jokemanfire
jokemanfire previously approved these changes Apr 9, 2025
@Rayato159
Copy link
Contributor Author

Dear @jokemanfire,
Now, I've fixed you can review and merge.
This should be the final version, I’ve double-checked everything.

Sorry again for the confusion with the last conflict 🙇‍♂️🙇‍♂️🙇‍♂️.

@Rayato159 Rayato159 requested a review from jokemanfire April 9, 2025 02:49
Copy link
Collaborator

@jokemanfire jokemanfire left a comment

Choose a reason for hiding this comment

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

Thanks

@jokemanfire jokemanfire merged commit feb224c into modelcontextprotocol:main Apr 9, 2025
8 checks passed
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