Add GitHub Copilot CLI as a supported MCP client (#5287)

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: Dan Barr <6922515+danbarr@users.noreply.github.com>
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: danbarr

docs/cli/thv_client_register.md

@@ -98,6 +98,8 @@ const (
 	KimiCli ClientApp = "kimi-cli"
 	// Factory represents the Factory.ai Droid CLI.
 	Factory ClientApp = "factory"
+	// CopilotCli represents the GitHub Copilot CLI.
+	CopilotCli ClientApp = "copilot-cli"
 )
 
 const (
@@ -1003,6 +1005,25 @@ var supportedClientIntegrations = []clientAppConfig{
 		SkillsGlobalPath:  []string{".factory", skillsDirName},
 		SkillsProjectPath: []string{".factory", skillsDirName},
 	},
+	{
+		ClientType:           CopilotCli,
+		Description:          "GitHub Copilot CLI",
+		SettingsFile:         "mcp-config.json",
+		MCPServersPathPrefix: "/mcpServers",
+		RelPath:              []string{".copilot"},
+		Extension:            JSON,
+		SupportedTransportTypesMap: map[types.TransportType]string{
+			types.TransportTypeStdio:          httpTransportLabel,
+			types.TransportTypeSSE:            "sse",
+			types.TransportTypeStreamableHTTP: httpTransportLabel,
+		},
+		IsTransportTypeFieldSupported: true,
+		MCPServersUrlLabelMap: map[types.TransportType]string{
+			types.TransportTypeStdio:          defaultURLFieldName,
+			types.TransportTypeSSE:            defaultURLFieldName,
+			types.TransportTypeStreamableHTTP: defaultURLFieldName,
+		},
+	},
 	{
 		// Xcode does not support MCP; it is an LLM-gateway-only entry.
 		// Cast LLMClientApp → ClientApp for internal config storage; the type

docs/cli/thv_client_remove.md

@@ -504,7 +504,7 @@ func TestSuccessfulClientConfigOperations(t *testing.T) {
 			case AmpWindsurf:
 				assert.Contains(t, string(content), `"mcpServers":`,
 					"AmpWindsurf config should contain mcpServers key")
-			case LMStudio, Trae, Kiro, Antigravity, GeminiCli, KimiCli, Factory:
+			case LMStudio, Trae, Kiro, Antigravity, GeminiCli, KimiCli, Factory, CopilotCli:
 				assert.Contains(t, string(content), `"mcpServers":`,
 					"Config should contain mcpServers key")
 			case VSCodeServer:
@@ -563,7 +563,7 @@ func TestSuccessfulClientConfigOperations(t *testing.T) {
 					"VSCode config should contain the server URL")
 			case Cursor, RooCode, ClaudeCode, Cline, Windsurf, WindsurfJetBrains, AmpCli,
 				AmpVSCode, AmpCursor, AmpVSCodeInsider, AmpWindsurf, LMStudio, Goose, Trae, Continue, OpenCode, Kiro, Antigravity, Zed, GeminiCli, VSCodeServer,
-				MistralVibe, Codex, KimiCli, Factory:
+				MistralVibe, Codex, KimiCli, Factory, CopilotCli:
 				assert.Contains(t, string(content), testURL,
 					"Config should contain the server URL")
 			}
@@ -1292,8 +1292,8 @@ func TestGetAllClients(t *testing.T) {
 
 	clients := GetAllClients()
 
-	// Should return all 27 supported clients
-	assert.Len(t, clients, 27, "Expected 27 supported clients")
+	// Should return all 28 supported clients
+	assert.Len(t, clients, 28, "Expected 28 supported clients")
 
 	// Verify the list is sorted alphabetically
 	for i := 1; i < len(clients); i++ {
@@ -1507,7 +1507,7 @@ func TestGetClientListCSV(t *testing.T) {
 			clientNames[i-1], clientNames[i])
 	}
 
-	// Count the number of clients (should be 25)
+	// Count the number of clients (should be 28)
 	clients := strings.Split(csv, ", ")
-	assert.Len(t, clients, 27, "Expected 27 clients in CSV list")
+	assert.Len(t, clients, 28, "Expected 28 clients in CSV list")
 }