From 5cc97e2baab5b2df9d25ff4436ce0c1c75bfd35d Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Sat, 18 Jul 2026 23:45:10 +0000 Subject: [PATCH] [log] Add debug logging to tool_registry_helpers.go Add meaningful debug logging calls to internal/server/tool_registry_helpers.go using the existing logToolRegistryHelpers logger (server:tool_registry_helpers namespace): - registerToolWithoutValidation: log tool name when registering - fetchBackendList: log serverID and method on entry, and error responses - registrationErrors.finish: log success summary when all backends registered These logs help diagnose tool registration issues and backend connectivity problems during development and production troubleshooting. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- internal/server/tool_registry_helpers.go | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/internal/server/tool_registry_helpers.go b/internal/server/tool_registry_helpers.go index f6f73fef..a34c169f 100644 --- a/internal/server/tool_registry_helpers.go +++ b/internal/server/tool_registry_helpers.go @@ -49,6 +49,7 @@ type launchResult struct { // This distinction relies on internal SDK behaviour and must be re-verified on every SDK upgrade. // Verified correct for go-sdk v1.6.1 (see server.go:Server.AddTool vs AddTool[In,Out]). func registerToolWithoutValidation(server *sdk.Server, tool *sdk.Tool, handler func(context.Context, *sdk.CallToolRequest, interface{}) (*sdk.CallToolResult, interface{}, error)) { + logToolRegistryHelpers.Printf("Registering tool without validation: %s", tool.Name) server.AddTool(tool, func(ctx context.Context, req *sdk.CallToolRequest) (*sdk.CallToolResult, error) { result, _, err := handler(ctx, req, nil) return result, err @@ -76,11 +77,13 @@ func fetchBackendList[T any]( handleResponseError func(code int, message string) error, handleParseError func(error) error, ) error { + logToolRegistryHelpers.Printf("Fetching backend list: serverID=%s, method=%s", serverID, method) result, err := conn.SendRequestWithServerID(ctx, method, nil, serverID) if err != nil { return handleRequestError(err) } if result.Error != nil { + logToolRegistryHelpers.Printf("Backend list error response: serverID=%s, method=%s, code=%d", serverID, method, result.Error.Code) return handleResponseError(result.Error.Code, result.Error.Message) } if err := json.Unmarshal(result.Result, listResult); err != nil { @@ -106,6 +109,8 @@ func (e *registrationErrors) finish() { if len(e.failed) > 0 { logger.LogError("backend", "Tool registration incomplete: %d of %d backends failed: %v — agents will not see tools from these servers", len(e.failed), e.total, e.failed) + } else { + logToolRegistryHelpers.Printf("Tool registration complete: all %d backend(s) registered successfully", e.total) } }