Skip to content

Conversation

@DevelopmentCats
Copy link
Contributor

@DevelopmentCats DevelopmentCats commented Nov 24, 2025

Description

changes the claude-code mcp add command to claude-code mcp add-json instead, and updates usage examples with real world mcp server example.

Type of Change

  • New module
  • New template
  • Bug fix
  • Feature/enhancement
  • Documentation
  • Other

Module Information

Path: registry/coder/modules/claude-code
New version: v4.2.2
Breaking change: [ ] Yes [X] No

Testing & Validation

  • Tests pass (bun test)
  • Code formatted (bun fmt)
  • Changes tested locally

Related Issues

#562

Copy link
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 fixes a bug in the Claude Code module by correcting the MCP (Model Context Protocol) server configuration command from claude mcp add to claude mcp add-json. The module version is bumped from 4.2.1 to 4.2.2 to reflect this bug fix.

Key changes:

  • Updated the MCP server add command to use the correct add-json subcommand in the installation script
  • Bumped module version to 4.2.2 across all README examples
  • Fixed JSON syntax error in README example by adding missing trailing comma

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
registry/coder/modules/claude-code/scripts/install.sh Changed claude mcp add command to claude mcp add-json for proper MCP server configuration
registry/coder/modules/claude-code/README.md Updated version references from 4.2.1 to 4.2.2 in all example code blocks; fixed JSON syntax by adding trailing comma to MCP server configuration example

@DevelopmentCats DevelopmentCats marked this pull request as ready for review November 25, 2025 13:39
Copy link
Member

@matifali matifali left a comment

Choose a reason for hiding this comment

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

LGTM. Does it close #562?

@DevelopmentCats
Copy link
Contributor Author

This should but I'm thoroughly testing this to make sure there are no more hiccups with mcp servers and claude-code

@DevelopmentCats
Copy link
Contributor Author

There was an issue with the mcp servers being set for the root dir rather than the workdir and this has been addressed as well.

I have tested this, and updated the mcp config usage example to with a real world example as well

@DevelopmentCats DevelopmentCats merged commit dd412fb into main Nov 25, 2025
4 checks passed
@DevelopmentCats DevelopmentCats deleted the cat/claude-code-562 branch November 25, 2025 17:44
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