Skip to content

Add GitHub Copilot CLI as a supported MCP client#5287

Merged
rdimitrov merged 1 commit into
mainfrom
add-copilot-cli-client
May 15, 2026
Merged

Add GitHub Copilot CLI as a supported MCP client#5287
rdimitrov merged 1 commit into
mainfrom
add-copilot-cli-client

Conversation

@danbarr
Copy link
Copy Markdown
Collaborator

@danbarr danbarr commented May 14, 2026

Summary

GitHub Copilot CLI supports MCP servers via ~/.copilot/mcp-config.json using the standard mcpServers map format that ToolHive already handles. Adding it here means users can run thv client register copilot-cli and have ToolHive-managed servers wired up automatically, alongside the other 27 supported clients.

The Copilot CLI docs list four transport type values (local, stdio, http, sse). Since ToolHive always proxies servers as streamable HTTP, registered entries get "type": "http" (matching the VS Code / Cursor pattern), with "sse" reserved for the legacy SSE transport.

Type of change

  • New feature (non-breaking change which adds functionality)

Test plan

  • task lint-fix — clean
  • go test ./pkg/client/... — passes
  • task docs regenerates docs/cli/thv_client_register.md, thv_client_remove.md, and the OpenAPI spec with copilot-cli listed
  • Manually verified end-to-end against the Copilot CLI: thv client register copilot-cli writes a valid entry to ~/.copilot/mcp-config.json and Copilot CLI picks it up

Does this introduce a user-facing change?

Yes. copilot-cli is now a valid argument to thv client register, thv client remove, and shows up in thv client status.

Copilot CLI reads MCP server entries from ~/.copilot/mcp-config.json
using the standard mcpServers map. Register it with type "http" for
streamable HTTP and SSE for the legacy transport, matching the
behaviour we already use for VS Code and Cursor.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added size/XS Extra small PR: < 100 lines changed and removed size/XS Extra small PR: < 100 lines changed labels May 14, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.37%. Comparing base (8fd704e) to head (4102020).
⚠️ Report is 12 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5287      +/-   ##
==========================================
+ Coverage   68.22%   68.37%   +0.14%     
==========================================
  Files         618      619       +1     
  Lines       63215    63318     +103     
==========================================
+ Hits        43131    43291     +160     
+ Misses      16861    16794      -67     
- Partials     3223     3233      +10     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Member

@aponcedeleonch aponcedeleonch left a comment

Choose a reason for hiding this comment

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

nice!

@rdimitrov rdimitrov merged commit 836c570 into main May 15, 2026
48 of 49 checks passed
@rdimitrov rdimitrov deleted the add-copilot-cli-client branch May 15, 2026 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XS Extra small PR: < 100 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants