Skip to content

Navigation Menu

Sign in
Sign up
This repository was archived by the owner on Jul 17, 2026. It is now read-only.

docs: Add MCP server naming constraints - #1268

Open
codegen-sh[bot] wants to merge 1 commit into
develop from
codegen-bot/add-mcp-naming-constraints-1758215523
Open

docs: Add MCP server naming constraints #1268
codegen-sh[bot] wants to merge 1 commit into
develop from
codegen-bot/add-mcp-naming-constraints-1758215523

Conversation

@codegen-sh

@codegen-sh codegen-sh Bot commented Sep 18, 2025

Copy link
Copy Markdown
Contributor

Summary

Adds documentation for MCP server naming constraints to prevent connection failures caused by invalid server names.

Changes

  • Added naming requirements section to MCP servers documentation
  • Documented regex pattern: ^[a-zA-Z0-9_-]{1,128}$
  • Specified allowed characters: alphanumeric, hyphens, and underscores only
  • Added warning about dots and special characters causing failures
  • Included example showing correct naming convention

Context

This addresses the issue where MCP servers with dots in their names (like awslabs.aws-documentation-mcp-server) cause malformed tool calls due to Anthropic's regex validation requirements. The documentation now clearly specifies the naming constraints to prevent these failures.

Files Changed


💻 View my work • 💬 Initiated in #customer-requestsAbout Codegen
⛔ Remove Codegen from PR🚫 Ban action checks

- Document required naming pattern: ^[a-zA-Z0-9_-]{1,128}$
- Specify allowed characters: alphanumeric, hyphens, underscores only
- Add warning about dots and special characters causing failures
- Include example of correct naming convention
Fixes issue where MCP servers with dots in names (like awslabs.aws-documentation-mcp-server) cause malformed tool calls due to Anthropic's regex validation.
Co-authored-by: Edo Pujol <ed@codegen.com>

codecov Bot commented Sep 18, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@codegen-team codegen-team Awaiting requested review from codegen-team

At least 1 approving review is required to merge this pull request.

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

AltStyle によって変換されたページ (->オリジナル) /