mcp: typed tool handlers lose wrapped JSON-RPC errors

Abierto Apto para principiantes
  • ¿Algo no está bien?
#1,529 1 comentario 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

  • #1528 de @ktsoator — cerrado sin fusionar

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
85/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
go
Área
backend-api-design

Línea de trabajo

Empieza con toolForErr en el paquete MCP y con el manejo de errores JSON-RPC que usa ClientSession.CallTool; compara el manejo directo de *jsonrpc.Error con los errores envueltos. Ejecuta la reproducción provista de cliente/servidor en memoria y luego verifica que errors.As conserve el código, el mensaje y los datos de JSON-RPC y que el resultado no se marque incorrectamente como error de ejecución. Agrega o actualiza pruebas focalizadas para errores de protocolo envueltos y no envueltos.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Checklist
  • I searched the existing issues, and this has not been reported before.
  • This is a real problem that I ran into myself, using the latest release of the SDK.
  • I ran into it in a production system, not in a toy example, a benchmark, or an automated or AI-driven scan of the code.
  • I (a human) examined the relevant SDK code, and wrote or reviewed every line of this report. It is not unreviewed AI output.
  • This report is brief and concise, and sticks to the facts needed to understand the problem.
What happened?

A typed tool handler returning a wrapped *jsonrpc.Error is treated as a tool execution error. ClientSession.CallTool returns a nil Go error and a result with IsError: true, so the JSON-RPC error code and data are lost. Returning the same protocol error directly works.

toolForErr checks err.(*jsonrpc.Error) instead of searching the error chain. The expected result is a JSON-RPC error with the underlying code, message, and data preserved when wrapped with %w.

How to reproduce

Run this program with github.com/modelcontextprotocol/go-sdk v1.8.0:


package main

import (
	"context"
	"errors"
	"fmt"

	"github.com/modelcontextprotocol/go-sdk/jsonrpc"
	"github.com/modelcontextprotocol/go-sdk/mcp"
)

func main() {
	ctx := context.Background()
	s := mcp.NewServer(&mcp.Implementation{Name: "server", Version: "1"}, nil)
	mcp.AddTool(s, &mcp.Tool{Name: "demo"}, func(context.Context, *mcp.CallToolRequest, map[string]any) (*mcp.CallToolResult, any, error) {
		return nil, nil, fmt.Errorf("backend: %w", &jsonrpc.Error{
			Code:    jsonrpc.CodeInvalidParams,
			Message: "bad input",
		})
	})

	st, ct := mcp.NewInMemoryTransports()
	ss, err := s.Connect(ctx, st, nil)
	if err != nil {
		panic(err)
	}
	defer ss.Close()

	c := mcp.NewClient(&mcp.Implementation{Name: "client", Version: "1"}, nil)
	cs, err := c.Connect(ctx, ct, nil)
	if err != nil {
		panic(err)
	}
	defer cs.Close()

	result, err := cs.CallTool(ctx, &mcp.CallToolParams{Name: "demo"})
	var rpcErr *jsonrpc.Error
	fmt.Printf("protocolError=%v, err=%v\n", errors.As(err, &rpcErr), err)
	if result != nil {
		fmt.Printf("isError=%v\n", result.IsError)
	}
}

Observed:

protocolError=false, err=<nil>
isError=true

Expected: protocolError=true, with JSON-RPC code -32602.

Where did you hit this?

Found during local code inspection and reproduced with an in-memory client and server.

Go MCP SDK version

v1.8.0

Go version

go1.26.10 darwin/arm64, also reproduced on main at 0fa5e8407aa3fda65166e4207156c03d084bfd44.

Proposed fix

Use errors.As and return the underlying protocol error.

Lenguaje dominante
Go
Estrellas
5.3k
Forks
614
Merge medio
23 h 55 min
PR fusionados (30 d)
62

Preparar el entorno

Abrir en Codespaces

Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de modelcontextprotocol/go-sdk

Todos los issues de modelcontextprotocol/go-sdk

Issues similares

Más issues de Go

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.