docs: Update Claude Code config to use claude mcp add commands

[ElanHasson]

Summary

• Replace manual JSON configuration with claude mcp add CLI commands
• Add multiple Docker configuration options (basic, file access, restricted)
• Update dnx configuration examples with proper syntax
• Remove docker-compose.yml file and all references
• Simplify documentation with modern Claude CLI integration

Changes

Documentation Updates

• Claude Code section: Converted from manual JSON editing to claude mcp add commands
• Docker options: Added basic, file access, and restricted directory configurations
• Syntax correction: Used proper -- separator between command and arguments
• dnx examples: Updated with environment variable configuration

File Removal

• Removed docker-compose.yml (no longer needed)
• Updated project structure documentation accordingly

Benefits

• Easier setup: Users can now use simple CLI commands instead of manual JSON editing
• Better UX: Multiple configuration options for different use cases
• Modern approach: Leverages latest Claude CLI capabilities
• Cleaner repo: Removes unused docker-compose configuration

🤖 Generated with Claude Code

[claude]

Claude finished @ElanHasson's task —— View job

---

PR Review Complete ✅

This PR successfully modernizes the documentation by replacing manual JSON configuration with claude mcp add CLI commands and cleaning up outdated files.

Todo List:

• Read and analyze the PR context and changed files
• Review README.md changes for documentation quality and accuracy
• Verify docker-compose.yml removal is appropriate
• Check for any security concerns in the new configuration examples
• Provide comprehensive feedback on code quality and best practices

Review Feedback

✅ Excellent Improvements

1. Modern CLI Integration: The transition from manual JSON configuration to claude mcp add commands is a significant UX improvement. This follows current best practices for Claude CLI usage.

2. Multiple Configuration Options: The three Docker configuration examples (basic, file access, restricted) provide flexibility for different use cases:

   • Basic: Simple setup for testing
   • File access: Full filesystem access with volume mounting
   • Restricted: Security-focused with read-only script directory

3. Improved Documentation Structure: Better formatting with clear sections and bullet points enhances readability.

4. Proper Cleanup: Removing docker-compose.yml is appropriate since:

   • The file was basic and didn't add significant value
   • Docker run commands are more straightforward for MCP usage
   • Only reference in codebase was in .dockerignore (which is appropriate)

✅ Security Considerations

The new configuration examples maintain good security practices:

• Restricted directory example uses read-only mounting (/scripts:ro) - README.md:210
• Environment variable usage for path restrictions is properly documented
• No exposure of sensitive information in the examples

✅ Code Quality & Best Practices

1. Command Syntax: Proper use of -- separator between claude mcp add and Docker arguments - README.md:198,204,210
2. Environment Variables: Correct -e flag usage for setting CSX_ALLOWED_PATH - README.md:210,216
3. Consistency: All examples follow the same pattern and formatting
4. Documentation Accuracy: Volume mounting explanation is maintained and accurate - README.md:219

🔧 Minor Suggestions

1. Line 204: Consider adding a brief comment about when to use the file access option vs restricted option
2. Line 216: The dnx example shows a hardcoded path /Users/your-username/scripts - could use "${HOME}/scripts" for consistency with Docker examples

✅ Overall Assessment

This is a high-quality documentation update that:

• Improves user experience with modern CLI commands
• Maintains security best practices
• Follows current Claude CLI patterns
• Removes technical debt (docker-compose.yml)
• Enhances documentation clarity

Recommendation: ✅ Approve - This PR significantly improves the setup experience while maintaining security and following current best practices.

---