这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@ElanHasson
Copy link
Contributor

Summary

  • Added comprehensive MCP configuration instructions for Cursor
  • Added comprehensive MCP configuration instructions for Claude Code
  • Included examples for both Docker and dotnet tool installations

Changes

  • Added Cursor configuration section with .cursor/mcp_settings.json examples
  • Added Claude Code configuration section with platform-specific paths:
    • macOS: ~/Library/Application Support/Claude/claude_desktop_config.json
    • Windows: %APPDATA%\Claude\claude_desktop_config.json
    • Linux: ~/.config/claude/claude_desktop_config.json
  • Included CSX_ALLOWED_PATH environment variable in examples for security
  • Reorganized MCP configuration section to prioritize Cursor and Claude Code

Test plan

  • Configuration examples are syntactically valid JSON
  • Paths follow platform conventions
  • Docker and dotnet tool commands are correct

🤖 Generated with Claude Code

- Add Cursor configuration for both Docker and dotnet tool installation
- Add Claude Code configuration with platform-specific paths
- Include CSX_ALLOWED_PATH environment variable examples for security
- Reorganize MCP configuration section with Cursor and Claude Code first
@claude
Copy link

claude bot commented Aug 21, 2025

Claude finished @ElanHasson's task —— View job


PR Review: MCP Configuration Documentation

Reviewing the pull request for MCP configuration instructions...

Todo List:

  • Read and analyze the README.md changes
  • Review JSON configuration examples for syntax and accuracy
  • Evaluate security considerations
  • Check documentation structure and clarity
  • Provide comprehensive feedback

@ElanHasson ElanHasson merged commit 40e0893 into main Aug 21, 2025
9 checks passed
@ElanHasson ElanHasson deleted the feat/cursor-claude-docs branch August 21, 2025 04:05
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