From 36816461fd3fb421e03ffc967687388488abafbe Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Tue, 9 Jun 2026 00:00:38 +0000 Subject: [PATCH 1/2] config/config_stdin: add debug logging to helper functions Add logStdin.Printf calls to three previously unlogged functions: - assignLegacyIntAlias: log when a deprecated snake_case alias (connect_timeout/tool_timeout) is applied, helping identify configs that should migrate to camelCase equivalents - stripExtensionFieldsForValidation: log server count processed when stripping gateway-specific extension fields before schema validation - convertStdinServerConfig (HTTP branch): log when custom connectTimeout or toolTimeout are configured on an HTTP server Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- internal/config/config_stdin.go | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/internal/config/config_stdin.go b/internal/config/config_stdin.go index 61a98d43..44d726ef 100644 --- a/internal/config/config_stdin.go +++ b/internal/config/config_stdin.go @@ -267,6 +267,7 @@ func assignLegacyIntAlias(rawFields map[string]json.RawMessage, alias string, ta if err := json.Unmarshal(raw, &value); err != nil { return fmt.Errorf("invalid %s value: %w", alias, err) } + logStdin.Printf("Applying legacy alias %q: value=%d (prefer camelCase equivalent)", alias, value) *target = &value return nil } @@ -304,7 +305,9 @@ func stripExtensionFieldsForValidation(data []byte) ([]byte, error) { delete(config, "guards") // Strip per-server "guard" and "auth" extension fields + serverCount := 0 if servers, ok := config["mcpServers"].(map[string]interface{}); ok { + serverCount = len(servers) for _, server := range servers { if serverMap, ok := server.(map[string]interface{}); ok { delete(serverMap, "guard") @@ -314,6 +317,7 @@ func stripExtensionFieldsForValidation(data []byte) ([]byte, error) { } } + logStdin.Printf("Stripped gateway extension fields for schema validation: %d servers processed", serverCount) return json.Marshal(config) } @@ -516,6 +520,9 @@ func convertStdinServerConfig(name string, server *StdinServerConfig, customSche if server.ToolTimeout != nil { serverCfg.ToolTimeout = *server.ToolTimeout } + if server.ConnectTimeout != nil || server.ToolTimeout != nil { + logStdin.Printf("HTTP server %q: custom timeouts configured: connectTimeout=%v, toolTimeout=%v", name, server.ConnectTimeout, server.ToolTimeout) + } if server.Auth != nil { serverCfg.Auth = &AuthConfig{ Type: server.Auth.Type, From 0f27024bab6a74f4f71fc9a2b958d5a607d27c79 Mon Sep 17 00:00:00 2001 From: Landon Cox Date: Tue, 9 Jun 2026 07:39:42 -0700 Subject: [PATCH 2/2] Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --- internal/config/config_stdin.go | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/internal/config/config_stdin.go b/internal/config/config_stdin.go index 44d726ef..991626ea 100644 --- a/internal/config/config_stdin.go +++ b/internal/config/config_stdin.go @@ -521,7 +521,15 @@ func convertStdinServerConfig(name string, server *StdinServerConfig, customSche serverCfg.ToolTimeout = *server.ToolTimeout } if server.ConnectTimeout != nil || server.ToolTimeout != nil { - logStdin.Printf("HTTP server %q: custom timeouts configured: connectTimeout=%v, toolTimeout=%v", name, server.ConnectTimeout, server.ToolTimeout) + var connectTimeout any + if server.ConnectTimeout != nil { + connectTimeout = *server.ConnectTimeout + } + var toolTimeout any + if server.ToolTimeout != nil { + toolTimeout = *server.ToolTimeout + } + logStdin.Printf("HTTP server %q: custom timeouts configured: connectTimeout=%v, toolTimeout=%v", name, connectTimeout, toolTimeout) } if server.Auth != nil { serverCfg.Auth = &AuthConfig{