Migrate projects toolset to modelcontextprotocol/go-sdk

[Copilot]

Migrates the projects toolset (9 tools) from mark3labs/mcp-go to modelcontextprotocol/go-sdk as part of the broader SDK migration (#1428).

Changes

Tool definitions

• Converted DSL-based schemas (mcp.WithString(), mcp.WithNumber()) to jsonschema.Schema structs
• Updated handler signatures from (ctx, req) -> (result, error) to (ctx, req, args) -> (result, data, error)
• Changed ReadOnlyHint from pointer type to boolean

Parameter extraction

• Replaced RequiredParam[T](req, "key") with RequiredParam[T](args, "key")
• Updated extractPaginationOptions() to accept map[string]any instead of mcp.CallToolRequest

Result construction

• Changed from mcp.NewToolResultText/Error() to utils.NewToolResultText/Error()

Tests

• Updated assertions to cast tool.InputSchema to *jsonschema.Schema
• Modified handler calls to pass &request and tc.requestArgs separately, expect 3 return values

Registration

• Enabled projects toolset in tools.go (previously commented out)

Example

Before:

func ListProjects(...) (tool mcp.Tool, handler server.ToolHandlerFunc) {
    return mcp.NewTool("list_projects",
        mcp.WithString("owner", mcp.Required()),
    ), func(ctx context.Context, req mcp.CallToolRequest) (*mcp.CallToolResult, error) {
        owner, _ := RequiredParam[string](req, "owner")
        return mcp.NewToolResultText(result), nil
    }
}

After:

func ListProjects(...) (mcp.Tool, mcp.ToolHandlerFor[map[string]any, any]) {
    return mcp.Tool{
        Name: "list_projects",
        InputSchema: &jsonschema.Schema{
            Properties: map[string]*jsonschema.Schema{
                "owner": {Type: "string"},
            },
            Required: []string{"owner"},
        },
    }, func(ctx context.Context, _ *mcp.CallToolRequest, args map[string]any) (*mcp.CallToolResult, any, error) {
        owner, _ := RequiredParam[string](args, "owner")
        return utils.NewToolResultText(result), nil, nil
    }
}

Original prompt

Migrate the projects 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]

Screenshots

⚠️ Can't get issues' database IDs with ONLY projects toolset

If projects is the only enabled toolset, then there isn't a way to get issues' database IDs which is needed for add_project_item.

[Copilot]

Pull request overview

This PR migrates the projects toolset (9 tools) from mark3labs/mcp-go to modelcontextprotocol/go-sdk as part of the broader SDK migration (#1428). The migration updates tool definitions from DSL-based schemas to jsonschema.Schema structs, changes handler signatures to the new SDK pattern, and enables the previously commented-out projects toolset.

Key Changes:

• Converted 9 project-related tools (list, get, add, update, delete operations for projects, fields, and items)
• Updated handler signatures to new 3-return-value pattern with separate args parameter
• Changed parameter extraction and result construction to use new SDK utilities
• Enabled projects toolset registration in tools.go

Reviewed changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
pkg/github/projects.go	Migrated all 9 project tool definitions and handlers to new SDK, updated parameter extraction to use args map, changed result construction to use utils package
pkg/github/projects_test.go	Updated tests to cast InputSchema to *jsonschema.Schema, modified handler calls to pass 3 arguments and expect 3 return values, removed build ignore directive
pkg/github/tools.go	Enabled projects toolset by uncommenting registration code
pkg/github/toolsnaps/*.snap	Updated tool snapshots to reflect new schema structure (field ordering changes, missing readOnlyHint: false for write operations)
README.md	Auto-generated documentation changes (incorrectly removed projects toolset from available toolsets)
docs/remote-server.md	Auto-generated documentation changes (incorrectly removed projects toolset from remote server toolsets)

[Copilot]

The toolsnap file is missing "readOnlyHint": false in the annotations for this write operation. Other write tools in the codebase (e.g., create_branch.snap, push_files.snap, star_repository.snap) include this field. Since AddProjectItem correctly sets ReadOnlyHint: false in the code (line 649 of projects.go), the toolsnap should reflect this.

The toolsnap should include:

"annotations": {
  "title": "Add project item",
  "readOnlyHint": false
},

This needs to be regenerated by running UPDATE_TOOLSNAPS=true go test ./....

Suggested change

	"title": "Add project item"
	"title": "Add project item",
	"readOnlyHint": false

[Copilot]

The toolsnap file is missing "readOnlyHint": false in the annotations for this write operation. Other write tools in the codebase (e.g., create_branch.snap, push_files.snap, star_repository.snap) include this field. Since UpdateProjectItem correctly sets ReadOnlyHint: false in the code (line 755 of projects.go), the toolsnap should reflect this.

The toolsnap should include:

"annotations": {
  "title": "Update project item",
  "readOnlyHint": false
},

This needs to be regenerated by running UPDATE_TOOLSNAPS=true go test ./....

Suggested change

	"title": "Update project item"
	"title": "Update project item",
	"readOnlyHint": false

[Copilot]

The toolsnap file is missing "readOnlyHint": false in the annotations for this write operation. Other write tools in the codebase (e.g., create_branch.snap, push_files.snap, star_repository.snap) include this field. Since DeleteProjectItem correctly sets ReadOnlyHint: false in the code (line 862 of projects.go), the toolsnap should reflect this.

The toolsnap should include:

"annotations": {
  "title": "Delete project item",
  "readOnlyHint": false
},

This needs to be regenerated by running UPDATE_TOOLSNAPS=true go test ./....

Suggested change

	"title": "Delete project item"
	"title": "Delete project item",
	"readOnlyHint": false

[SamMorrowDrums]

Are these generated docs changes deliberate in that PR