From 7a9af4eaf4483870be04f2333c9b976dc5c6e113 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Wed, 11 Feb 2026 05:21:21 +0000 Subject: [PATCH 1/2] Initial plan From ceecc3865e946c10762a1cfd4b47e383d6103c1b Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Wed, 11 Feb 2026 05:27:15 +0000 Subject: [PATCH 2/2] Fix MCP logs guardrail integration test after jq removal Remove references to SuggestedQueries field which was removed when jq support was removed from MCP server tools. The test was checking for suggested jq queries which are no longer part of the guardrail response structure. Fixes build failure in CI workflow run 21893400216. Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> --- pkg/cli/mcp_logs_guardrail_integration_test.go | 14 -------------- 1 file changed, 14 deletions(-) diff --git a/pkg/cli/mcp_logs_guardrail_integration_test.go b/pkg/cli/mcp_logs_guardrail_integration_test.go index c59fa5de2f4..2350ac99ebd 100644 --- a/pkg/cli/mcp_logs_guardrail_integration_test.go +++ b/pkg/cli/mcp_logs_guardrail_integration_test.go @@ -90,10 +90,6 @@ func TestMCPServer_LogsGuardrail(t *testing.T) { t.Error("Schema should have fields") } - if len(guardrail.SuggestedQueries) == 0 { - t.Error("Should have suggested queries") - } - // Verify some expected fields are in the schema expectedFields := []string{"summary", "runs", "tool_usage", "errors_and_warnings"} for _, field := range expectedFields { @@ -101,15 +97,5 @@ func TestMCPServer_LogsGuardrail(t *testing.T) { t.Errorf("Schema should include field '%s'", field) } } - - // Verify suggested queries have proper structure - for i, query := range guardrail.SuggestedQueries { - if query.Description == "" { - t.Errorf("Query %d should have description", i) - } - if query.Query == "" { - t.Errorf("Query %d should have query string", i) - } - } }) }