Fix parsing bug that breaks Docker Desktop client listing (#110)

* Fix a bug that breaks Docker Desktop client listing.

* Handle http in client configs.

Author: cmrigney

cmd/docker-mcp/client/config_test.go

@@ -32,7 +32,8 @@ func Test_yq_list(t *testing.T) {
 						Args:    []string{"mcp", "gateway", "run"},
 					},
 				},
-				SSEServers: []MCPServerSSE{},
+				SSEServers:  []MCPServerSSE{},
+				HTTPServers: []MCPServerHTTP{},
 			},
 		},
 		{
@@ -47,7 +48,8 @@ func Test_yq_list(t *testing.T) {
 						Args:    []string{"mcp", "gateway", "run"},
 					},
 				},
-				SSEServers: []MCPServerSSE{},
+				SSEServers:  []MCPServerSSE{},
+				HTTPServers: []MCPServerHTTP{},
 			},
 		},
 		{
@@ -65,7 +67,8 @@ func Test_yq_list(t *testing.T) {
 						Name: "my-server",
 					},
 				},
-				SSEServers: []MCPServerSSE{},
+				SSEServers:  []MCPServerSSE{},
+				HTTPServers: []MCPServerHTTP{},
 			},
 		},
 		{
@@ -93,6 +96,7 @@ func Test_yq_list(t *testing.T) {
 						Headers: map[string]string{"VERSION": "1.2"},
 					},
 				},
+				HTTPServers: []MCPServerHTTP{},
 			},
 		},
 	}

cmd/docker-mcp/client/ls.go

@@ -135,7 +135,7 @@ func prettyPrintBaseData(vendor string, data MCPClientCfgBase) {
 	circle := redCircle
 	nrServers := 0
 	if data.cfg != nil {
-		nrServers = len(data.cfg.STDIOServers) + len(data.cfg.SSEServers)
+		nrServers = len(data.cfg.STDIOServers) + len(data.cfg.SSEServers) + len(data.cfg.HTTPServers)
 	}
 	if nrServers > 0 {
 		circle = orangeCircle
@@ -155,6 +155,9 @@ func prettyPrintBaseData(vendor string, data MCPClientCfgBase) {
 	for _, server := range data.cfg.SSEServers {
 		fmt.Printf("   %s: %s (sse)\n", server.Name, server.String())
 	}
+	for _, server := range data.cfg.HTTPServers {
+		fmt.Printf("   %s: %s (http)\n", server.Name, server.String())
+	}
 }
 
 func (cfg GlobalConfig) GetData() any {

cmd/docker-mcp/client/parse.go

@@ -3,6 +3,7 @@ package client
 import (
 	"encoding/json"
 	"fmt"
+	"os"
 	"strings"
 )
 
@@ -14,6 +15,7 @@ import (
 type MCPJSONLists struct {
 	STDIOServers []MCPServerSTDIO
 	SSEServers   []MCPServerSSE
+	HTTPServers  []MCPServerHTTP
 }
 
 type MCPServerSTDIO struct {
@@ -41,10 +43,20 @@ type MCPServerSSE struct {
 	Headers map[string]string `json:"headers"`
 }
 
+type MCPServerHTTP struct {
+	Name    string            `json:"name"`
+	URL     string            `json:"url"`
+	Headers map[string]string `json:"headers"`
+}
+
 func (c *MCPServerSSE) String() string {
 	return c.URL
 }
 
+func (c *MCPServerHTTP) String() string {
+	return c.URL
+}
+
 func UnmarshalMCPJSONList(data []byte) (*MCPJSONLists, error) {
 	if len(data) == 0 {
 		return &MCPJSONLists{}, nil
@@ -56,6 +68,7 @@ func UnmarshalMCPJSONList(data []byte) (*MCPJSONLists, error) {
 	cfg := &MCPJSONLists{
 		STDIOServers: []MCPServerSTDIO{},
 		SSEServers:   []MCPServerSSE{},
+		HTTPServers:  []MCPServerHTTP{},
 	}
 	for _, raw := range temp {
 		itemType, _ := getType(raw) // type is an optional field, default to stdio if not explicitly set
@@ -72,8 +85,14 @@ func UnmarshalMCPJSONList(data []byte) (*MCPJSONLists, error) {
 				return nil, err
 			}
 			cfg.SSEServers = append(cfg.SSEServers, server)
+		case "http":
+			var server MCPServerHTTP
+			if err := json.Unmarshal(raw, &server); err != nil {
+				return nil, err
+			}
+			cfg.HTTPServers = append(cfg.HTTPServers, server)
 		default:
-			fmt.Printf("unknown server type for %q\n", itemType)
+			fmt.Fprintf(os.Stderr, "unknown server type for %q\n", itemType)
 		}
 	}
 	return cfg, nil

cmd/docker-mcp/client/parse_test.go

@@ -24,12 +24,18 @@ func Test_UnmarshalMCPJSONList(t *testing.T) {
    "name": "my-server",
    "type": "stdio"
  },
-	{
-     "name":"my-remote-server",
-     "type": "sse",
-     "url": "http://api.contoso.com/sse",
-     "headers": { "VERSION": "1.2" }
-   }
+ {
+   "name":"my-remote-server",
+   "type": "sse",
+   "url": "http://api.contoso.com/sse",
+   "headers": { "VERSION": "1.2" }
+ },
+ {
+   "name":"my-remote-server",
+   "type": "http",
+   "url": "http://api.contoso.com/http",
+   "headers": { "VERSION": "1.2" }
+ }
 ]`,
 			result: &MCPJSONLists{
 				STDIOServers: []MCPServerSTDIO{
@@ -38,6 +44,9 @@ func Test_UnmarshalMCPJSONList(t *testing.T) {
 				SSEServers: []MCPServerSSE{
 					{Name: "my-remote-server", URL: "http://api.contoso.com/sse", Headers: map[string]string{"VERSION": "1.2"}},
 				},
+				HTTPServers: []MCPServerHTTP{
+					{Name: "my-remote-server", URL: "http://api.contoso.com/http", Headers: map[string]string{"VERSION": "1.2"}},
+				},
 			},
 		},
 		{