Doesn't fail

Signed-off-by: David Gageot <david.gageot@docker.com>

Author: dgageot

pkg/teamloader/teamloader.go

@@ -346,7 +346,7 @@ func createTool(ctx context.Context, toolset latest.Toolset, parentDir string, e
 
 		// TODO(dga): until the MCP Gateway supports oauth with cagent, we fetch the remote url and directly connect to it.
 		if serverSpec.Type == "remote" {
-			return mcp.NewRemoteToolset(serverSpec.Remote.URL, serverSpec.Remote.TransportType, nil, runtimeConfig.RedirectURI)
+			return mcp.NewRemoteToolset(serverSpec.Remote.URL, serverSpec.Remote.TransportType, nil, runtimeConfig.RedirectURI), nil
 		}
 
 		return mcp.NewGatewayToolset(mcpServerName, toolset.Config, envProvider), nil
@@ -365,7 +365,7 @@ func createTool(ctx context.Context, toolset latest.Toolset, parentDir string, e
 			headers[k] = expanded
 		}
 
-		return mcp.NewRemoteToolset(toolset.Remote.URL, toolset.Remote.TransportType, headers, runtimeConfig.RedirectURI)
+		return mcp.NewRemoteToolset(toolset.Remote.URL, toolset.Remote.TransportType, headers, runtimeConfig.RedirectURI), nil
 
 	default:
 		return nil, fmt.Errorf("unknown toolset type: %s", toolset.Type)

pkg/tools/mcp/stdio.go

@@ -6,8 +6,9 @@ import (
 	"iter"
 	"os/exec"
 
-	"github.com/docker/cagent/pkg/tools"
 	"github.com/modelcontextprotocol/go-sdk/mcp"
+
+	"github.com/docker/cagent/pkg/tools"
 )
 
 type stdioMCPClient struct {

pkg/tools/mcp/toolset.go

@@ -49,14 +49,14 @@ func NewToolsetCommand(command string, args, env []string) *Toolset {
 }
 
 // NewRemoteToolset creates a new MCP toolset from a remote MCP Server.
-func NewRemoteToolset(url, transport string, headers map[string]string, redirectURI string) (*Toolset, error) {
+func NewRemoteToolset(url, transport string, headers map[string]string, redirectURI string) *Toolset {
 	slog.Debug("Creating Remote MCP toolset", "url", url, "transport", transport, "headers", headers, "redirectURI", redirectURI)
 
 	return &Toolset{
 		mcpClient: newRemoteClient(url, transport, headers, redirectURI, NewInMemoryTokenStore()),
 		logType:   "remote",
 		logID:     url,
-	}, nil
+	}
 }
 
 func (ts *Toolset) Start(ctx context.Context) error {