- FILE-level review comments: Document that they are silently dropped by
GitHub API under certain conditions and the test doesn't fully verify them
- Directory deletion: Document that the test only deletes a single file
in a subdirectory, not actual recursive directory deletion
- Fix TestDirectoryDeletion: Create file in test-dir/ subdirectory to
match expected filename assertion
- Fix TestDirectoryDeletion: Search for deletion commit by message instead
of assuming first commit in list (order can vary)
- Fix TestPullRequestReviewCommentSubmit: Relax assertion from exactly 3
comments to at least 2 (FILE-level comments may not be returned by
ListReviewComments API)
* Initial plan
* Migrate actions toolset to modelcontextprotocol/go-sdk
- Removed //go:build ignore tags from actions.go and actions_test.go
- Updated imports to use modelcontextprotocol/go-sdk instead of mark3labs/mcp-go
- Updated all tool constructor signatures to return new types
- Converted all tool definitions from DSL format to jsonschema.Schema format
- Updated tool handler signatures to use generics
- Updated parameter extraction to use args map instead of request
- Replaced mcp.NewToolResultText/Error with utils package functions
- Updated all tests to work with new handler signature
- Updated toolsnaps for all actions tools
Co-authored-by: omgitsads <4619+omgitsads@users.noreply.github.com>
* re-add actions toolset
* create toolsnaps
* create toolsnaps
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: omgitsads <4619+omgitsads@users.noreply.github.com>
Co-authored-by: LuluBeatson <lulubeatson@github.com>
Co-authored-by: Adam Holt <me@adamholt.co.uk>
Build and Test Go Project / build (ubuntu-latest) (push) Has been cancelled
Build and Test Go Project / build (windows-latest) (push) Has been cancelled
CodeQL / Analyze (go) (push) Has been cancelled
CodeQL / Analyze (actions) (push) Has been cancelled
Docker / build (push) Has been cancelled
Build and Test Go Project / build (macos-latest) (push) Has been cancelled
GoReleaser Release / release (push) Has been cancelled
License Check / license-check (push) Has been cancelled
Publish to MCP Registry / publish (push) Has been cancelled
Temporarily removing package configuration for the GitHub MCP server because certain hosts incorrectly generate docker arguments based on it which results in a broken server configuration when installed via registry.
Will bring it back when the issue is resolved by MCP host.
* Initial plan
* Migrate dynamic toolset to modelcontextprotocol/go-sdk
This commit migrates the dynamic toolset (enable_toolset, list_available_toolsets,
get_toolset_tools) from mark3labs/mcp-go to modelcontextprotocol/go-sdk.
Changes:
- Removed //go:build ignore tag
- Updated imports to use modelcontextprotocol/go-sdk
- Migrated all tool functions to use new SDK patterns
- Updated ToolsetEnum helper to return []any instead of mcp.PropertyOption
- Converted DSL-based schema definitions to jsonschema.Schema structures
- Updated handler signatures to use map[string]any args
- Replaced old result helpers with utils package equivalents
- Fixed EnableToolset to use RegisterFunc instead of AddTools
- Created comprehensive test suite for all three tools
- Generated toolsnaps for the new tools
Related to #1428
Co-authored-by: omgitsads <4619+omgitsads@users.noreply.github.com>
* Enable dynamic tools
* Remove new test and toolsnaps, we can follow up with this
* Just return the tool and handler directly instead of assigning to variables first.
This stops copilot complaining in review that the variables are unused.
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: omgitsads <4619+omgitsads@users.noreply.github.com>
Co-authored-by: Adam Holt <me@adamholt.co.uk>
* Migrate repo resources to Go SDK
* Enable resources for repos
* Properly handle encoding and closing of the buffer
* Remove outdated comment
* Switch to StdEncoding, as it was originally
* fix casing for linter
* Update licenses
* Handle multiple path components
* Migrate repo resources to Go SDK
* Enable resources for repos
* Properly handle encoding and closing of the buffer
* Remove outdated comment
* Switch to StdEncoding, as it was originally
* fix casing for linter
* Update licenses
* Initial plan
* Migrate git toolset to modelcontextprotocol/go-sdk
- Remove //go:build ignore tag from git.go
- Update imports to use modelcontextprotocol/go-sdk
- Convert GetRepositoryTree tool schema to jsonschema format
- Update handler signature to use new generics pattern
- Update parameter extraction to use args map
- Replace mcp.NewToolResult* with utils package helpers
- Create dedicated git_test.go with updated test patterns
- Update toolsnaps for get_repository_tree
Related to #1428
Co-authored-by: omgitsads <4619+omgitsads@users.noreply.github.com>
* re-add git toolset
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: omgitsads <4619+omgitsads@users.noreply.github.com>
Co-authored-by: LuluBeatson <lulubeatson@github.com>
* Initial plan
* Migrate gists toolset to modelcontextprotocol/go-sdk
- Remove //go:build ignore tags from gists.go and gists_test.go
- Update imports to use modelcontextprotocol/go-sdk instead of mark3labs/mcp-go
- Migrate all 4 tools (ListGists, GetGist, CreateGist, UpdateGist):
- Updated tool definitions to use jsonschema.Schema for InputSchema
- Changed handler signatures to new SDK format with generics
- Updated parameter extraction to use args map instead of request object
- Replaced result helpers with utils package equivalents
- Updated all tests to match new handler signatures
- Added toolsnap tests for all 4 tools
- Added parseISOTimestamp utility function to minimal_types.go
- Created toolsnaps for all 4 tools
Related to #1428
Co-authored-by: omgitsads <4619+omgitsads@users.noreply.github.com>
* fix invalid schema, re-add gists toolset to server
* make schema types lowercase
* Don't assert without a testing.T
* just return the tool & handler
* Add Close method to IOLogger to close underlying reader and writer
* Update cmd/github-mcp-server/generate_docs.go
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* remove unnecessary translation
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: omgitsads <4619+omgitsads@users.noreply.github.com>
Co-authored-by: LuluBeatson <lulubeatson@github.com>
Co-authored-by: Adam Holt <me@adamholt.co.uk>
Co-authored-by: Adam Holt <omgitsads@github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
LuluBeatson