From 0caf0d9b5af8693d163b82e69d727dbedbffce94 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Sat, 17 Jan 2026 00:08:27 +0000 Subject: [PATCH 1/2] Initial plan From 4c92e2b4d6a27b98797fdb212eec77fe4f696d37 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Sat, 17 Jan 2026 00:12:38 +0000 Subject: [PATCH 2/2] Fix panic in test mock handlers - add safe type assertions and error handling Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com> --- internal/server/call_backend_tool_test.go | 24 +++++++++++++++++++---- 1 file changed, 20 insertions(+), 4 deletions(-) diff --git a/internal/server/call_backend_tool_test.go b/internal/server/call_backend_tool_test.go index 2611ebeb5..9ad54b1b6 100644 --- a/internal/server/call_backend_tool_test.go +++ b/internal/server/call_backend_tool_test.go @@ -29,9 +29,16 @@ func TestCallBackendTool_ReturnsNonNilCallToolResult(t *testing.T) { backend := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { var req map[string]interface{} err := json.NewDecoder(r.Body).Decode(&req) - require.NoError(err) + if err != nil { + w.WriteHeader(http.StatusBadRequest) + return + } - method := req["method"].(string) + method, ok := req["method"].(string) + if !ok { + w.WriteHeader(http.StatusBadRequest) + return + } switch method { case "initialize": @@ -147,8 +154,17 @@ func TestCallBackendTool_ErrorStillReturnsCallToolResult(t *testing.T) { // Create a mock HTTP backend that returns an error backend := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { var req map[string]interface{} - json.NewDecoder(r.Body).Decode(&req) - method := req["method"].(string) + err := json.NewDecoder(r.Body).Decode(&req) + if err != nil { + w.WriteHeader(http.StatusBadRequest) + return + } + + method, ok := req["method"].(string) + if !ok { + w.WriteHeader(http.StatusBadRequest) + return + } switch method { case "initialize":