update server enable/disable to support catalogs

Author: slimslenderslacks

cmd/docker-mcp/internal/catalog/catalog.go

@@ -140,9 +140,27 @@ func GetWithOptions(ctx context.Context, useConfigured bool, additionalCatalogs
 		catalogPaths = append(catalogPaths, additionalCatalogs...)
 	}
 
+	// Remove duplicates while preserving order
+	catalogPaths = removeDuplicates(catalogPaths)
+
 	return ReadFrom(ctx, catalogPaths)
 }
 
+// removeDuplicates removes duplicate strings while preserving order (first occurrence wins)
+func removeDuplicates(slice []string) []string {
+	keys := make(map[string]bool)
+	result := []string{}
+
+	for _, item := range slice {
+		if !keys[item] {
+			keys[item] = true
+			result = append(result, item)
+		}
+	}
+
+	return result
+}
+
 // getConfiguredCatalogs reads the catalog registry and returns the list of configured catalog files
 func getConfiguredCatalogs() ([]string, error) {
 	homeDir, err := user.HomeDir()

cmd/docker-mcp/server/enable.go

@@ -32,7 +32,7 @@ func update(ctx context.Context, docker docker.Client, add []string, remove []st
 		return fmt.Errorf("parsing registry config: %w", err)
 	}
 
-	catalog, err := catalog.Get(ctx)
+	catalog, err := catalog.GetWithOptions(ctx, true, nil)
 	if err != nil {
 		return err
 	}

cmd/docker-mcp/server/server_test.go

@@ -229,6 +229,18 @@ func withCatalog(yaml string) option {
 	return func(t *testing.T, home string, _ *fakeDocker) {
 		t.Helper()
 		writeFile(t, filepath.Join(home, ".docker/mcp/catalogs/docker-mcp.yaml"), []byte(yaml))
+
+		// Create catalog.json registry file to register the docker-mcp catalog
+		catalogRegistry := `{
+  "catalogs": {
+    "docker-mcp": {
+      "displayName": "Docker MCP Default Catalog",
+      "url": "docker-mcp.yaml",
+      "lastUpdate": "2024-01-01T00:00:00Z"
+    }
+  }
+}`
+		writeFile(t, filepath.Join(home, ".docker/mcp/catalog.json"), []byte(catalogRegistry))
 	}
 }