Skip to content

Add mcp config for windows and claude#2097

Merged
cb-github-robot merged 1 commit intocloud-barista:mainfrom
seokho-son:main
Aug 17, 2025
Merged

Add mcp config for windows and claude#2097
cb-github-robot merged 1 commit intocloud-barista:mainfrom
seokho-son:main

Conversation

@seokho-son
Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: seokho-son <shsongist@gmail.com>
Copilot AI review requested due to automatic review settings August 17, 2025 14:09
@seokho-son seokho-son requested a review from yunkon-kim as a code owner August 17, 2025 14:09
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR reorganizes the MCP (Model Context Protocol) configuration documentation to provide platform-specific setup instructions for Windows and Claude desktop integration. The changes improve clarity by separating configurations for different operating systems and execution environments.

Key changes:

  • Reorganized configuration examples by platform (WSL, Windows native, Linux/macOS)
  • Added Windows-specific environment variables for proper encoding
  • Updated file paths to use Windows-style paths in the native Windows example

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

"args": [
"bash",
"-c",
"/home/username/.local/bin/uv run --with fastmcp /path/to/cb-tumblebug/src/interface/mcp/mcp-simple-proxy.py"
Copy link

Copilot AI Aug 17, 2025

Choose a reason for hiding this comment

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

The path '/home/username/.local/bin/uv run --with fastmcp /path/to/cb-tumblebug/src/interface/mcp/mcp-simple-proxy.py' appears to be incorrectly formatted. It should be split into the uv command and its arguments rather than being a single path string.

Suggested change
"/home/username/.local/bin/uv run --with fastmcp /path/to/cb-tumblebug/src/interface/mcp/mcp-simple-proxy.py"
"/home/username/.local/bin/uv",
"run",
"--with",
"fastmcp",
"/path/to/cb-tumblebug/src/interface/mcp/mcp-simple-proxy.py"

Copilot uses AI. Check for mistakes.
"fastmcp",
"/path/to/cb-tumblebug/src/interface/mcp/mcp-simple-proxy.py"
]
"C:\\Users\\admin\\Documents\\mcp-simple-proxy.py"
Copy link

Copilot AI Aug 17, 2025

Choose a reason for hiding this comment

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

The Windows path should use forward slashes or properly escaped backslashes in JSON. Consider using 'C:/Users/admin/Documents/mcp-simple-proxy.py' or ensure proper escaping with double backslashes.

Copilot uses AI. Check for mistakes.
@seokho-son
Copy link
Copy Markdown
Member Author

/approve

@github-actions github-actions bot added the approved This PR is approved and will be merged soon. label Aug 17, 2025
@cb-github-robot cb-github-robot merged commit 6ef27f3 into cloud-barista:main Aug 17, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved This PR is approved and will be merged soon.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants