Migrate labels toolset to modelcontextprotocol/go-sdk

[Copilot]

Closes: Part of #1428

Migrates GetLabel, ListLabels, and LabelWrite tools from mark3labs/mcp-go to modelcontextprotocol/go-sdk.

Changes

• Tool definitions: Converted from DSL-based mcp.NewTool() to struct-based mcp.Tool{} with explicit jsonschema.Schema definitions
• Handler signatures: Updated from func(context.Context, mcp.CallToolRequest) (*mcp.CallToolResult, error) to generic handlers func(context.Context, *mcp.CallToolRequest, map[string]any) (*mcp.CallToolResult, any, error)
• Parameter extraction: Changed from RequiredParam[T](request, "param") to RequiredParam[T](args, "param") using passed args map
• Result constructors: Replaced mcp.NewToolResult* with utils.NewToolResult* helpers
• Toolsnaps: Updated to reflect property ordering changes (semantics unchanged)

Example

Before:

func GetLabel(...) (mcp.Tool, server.ToolHandlerFunc) {
    return mcp.NewTool("get_label",
        mcp.WithString("owner", mcp.Required(), ...),
        ...
    ),
    func(ctx context.Context, request mcp.CallToolRequest) (*mcp.CallToolResult, error) {
        owner, err := RequiredParam[string](request, "owner")
        ...
        return mcp.NewToolResultText(string(out)), nil
    }
}

After:

func GetLabel(...) (mcp.Tool, mcp.ToolHandlerFor[map[string]any, any]) {
    tool := mcp.Tool{
        Name: "get_label",
        InputSchema: &jsonschema.Schema{
            Properties: map[string]*jsonschema.Schema{
                "owner": {Type: "string", Description: "..."},
                ...
            },
            Required: []string{"owner", "repo", "name"},
        },
    }
    handler := mcp.ToolHandlerFor[map[string]any, any](
        func(ctx context.Context, _ *mcp.CallToolRequest, args map[string]any) (*mcp.CallToolResult, any, error) {
            owner, err := RequiredParam[string](args, "owner")
            ...
            return utils.NewToolResultText(string(out)), nil, nil
        })
    return tool, handler
}

Original prompt

Migrate the labels toolset to modelcontextprotocol/go-sdk.

This will be merged into the parent PR #1428, so reference that PR in the description to track the migration

Custom agent used: go-sdk-tool-migrator
Agent specializing in migrating MCP tools from mark3labs/mcp-go to modelcontextprotocol/go-sdk

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[LuluBeatson]

[Copilot]

Pull Request Overview

This PR migrates the labels toolset (GetLabel, ListLabels, LabelWrite) from the legacy mark3labs/mcp-go SDK to the new modelcontextprotocol/go-sdk, continuing the migration effort tracked in #1428.

Key changes:

• Converted tool definitions from DSL-based to struct-based with explicit JSON schemas
• Updated handler signatures to use generic mcp.ToolHandlerFor[map[string]any, any] type
• Modified parameter extraction to use passed args map instead of request object
• Updated result constructors to use utils.NewToolResult* helpers

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
pkg/github/labels.go	Migrated GetLabel, ListLabels, and LabelWrite tool definitions and handlers to new SDK patterns with struct-based schemas and generic handler types
pkg/github/labels_test.go	Removed //go:build ignore directive, updated test assertions to check ReadOnlyHint annotations directly, and adapted handler invocations to new 3-return signature
pkg/github/tools.go	Uncommented and re-enabled labels toolset registration in the default toolset group
pkg/github/toolsnaps/get_label.snap	Reordered JSON properties and moved type before required in schema (no semantic changes)
pkg/github/toolsnaps/list_label.snap	Reordered JSON properties and moved type before required in schema (no semantic changes)
pkg/github/toolsnaps/label_write.snap	Reordered JSON properties, moved type before required, and removed explicit readOnlyHint: false (omitted false values per new SDK conventions)