04ca8ac03b
Make Tool APIs future-proof by pushing generics into CallToolParams, and using CallToolParams throughout. This has several advantages: - We no longer need the awkward CallToolOptions type, which is almost but not quite like CallToolParams. - NewTool no longer needs to define its own handler type: it accepts a ToolHandler[TArgs]. This also means its signature is future-proof. Additionally, provide a `CallTool` helper (the analog of NewTool) to serve as a more strongly typed variant of `ClientSession.CallTool`. + design, protocol generation Change-Id: I80172a25bc50734dae84a541aa4776ed9161694e Reviewed-on: https://go-review.googlesource.com/c/tools/+/676495 Reviewed-by: Jonathan Amsterdam <jba@google.com> Auto-Submit: Robert Findley <rfindley@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
90 lines
2.6 KiB
Go
90 lines
2.6 KiB
Go
// Copyright 2025 The Go MCP SDK Authors. All rights reserved.
|
|
// Use of this source code is governed by an MIT-style
|
|
// license that can be found in the LICENSE file.
|
|
|
|
package mcp_test
|
|
|
|
import (
|
|
"context"
|
|
"testing"
|
|
|
|
"github.com/google/go-cmp/cmp"
|
|
"github.com/google/go-cmp/cmp/cmpopts"
|
|
"github.com/modelcontextprotocol/go-sdk/mcp"
|
|
"github.com/modelcontextprotocol/go-sdk/jsonschema"
|
|
)
|
|
|
|
// testToolHandler is used for type inference in TestNewTool.
|
|
func testToolHandler[T any](context.Context, *mcp.ServerSession, *mcp.CallToolParams[T]) (*mcp.CallToolResult, error) {
|
|
panic("not implemented")
|
|
}
|
|
|
|
func TestNewTool(t *testing.T) {
|
|
tests := []struct {
|
|
tool *mcp.ServerTool
|
|
want *jsonschema.Schema
|
|
}{
|
|
{
|
|
mcp.NewTool("basic", "", testToolHandler[struct {
|
|
Name string `json:"name"`
|
|
}]),
|
|
&jsonschema.Schema{
|
|
Type: "object",
|
|
Required: []string{"name"},
|
|
Properties: map[string]*jsonschema.Schema{
|
|
"name": {Type: "string"},
|
|
},
|
|
AdditionalProperties: &jsonschema.Schema{Not: new(jsonschema.Schema)},
|
|
},
|
|
},
|
|
{
|
|
mcp.NewTool("enum", "", testToolHandler[struct{ Name string }], mcp.Input(
|
|
mcp.Property("Name", mcp.Enum("x", "y", "z")),
|
|
)),
|
|
&jsonschema.Schema{
|
|
Type: "object",
|
|
Required: []string{"Name"},
|
|
Properties: map[string]*jsonschema.Schema{
|
|
"Name": {Type: "string", Enum: []any{"x", "y", "z"}},
|
|
},
|
|
AdditionalProperties: &jsonschema.Schema{Not: new(jsonschema.Schema)},
|
|
},
|
|
},
|
|
{
|
|
mcp.NewTool("required", "", testToolHandler[struct {
|
|
Name string `json:"name"`
|
|
Language string `json:"language"`
|
|
X int `json:"x,omitempty"`
|
|
Y int `json:"y,omitempty"`
|
|
}], mcp.Input(
|
|
mcp.Property("x", mcp.Required(true)))),
|
|
&jsonschema.Schema{
|
|
Type: "object",
|
|
Required: []string{"name", "language", "x"},
|
|
Properties: map[string]*jsonschema.Schema{
|
|
"language": {Type: "string"},
|
|
"name": {Type: "string"},
|
|
"x": {Type: "integer"},
|
|
"y": {Type: "integer"},
|
|
},
|
|
AdditionalProperties: &jsonschema.Schema{Not: new(jsonschema.Schema)},
|
|
},
|
|
},
|
|
{
|
|
mcp.NewTool("set_schema", "", testToolHandler[struct {
|
|
X int `json:"x,omitempty"`
|
|
Y int `json:"y,omitempty"`
|
|
}], mcp.Input(
|
|
mcp.Schema(&jsonschema.Schema{Type: "object"})),
|
|
),
|
|
&jsonschema.Schema{
|
|
Type: "object",
|
|
},
|
|
},
|
|
}
|
|
for _, test := range tests {
|
|
if diff := cmp.Diff(test.want, test.tool.Tool.InputSchema, cmpopts.IgnoreUnexported(jsonschema.Schema{})); diff != "" {
|
|
t.Errorf("NewTool(%v) mismatch (-want +got):\n%s", test.tool.Tool.Name, diff)
|
|
}
|
|
}
|
|
}
|