Repository navigation
chore(mcp): remove the deprecated Python MCP server - #260
Merged
Merged
Conversation
Without filters, list_events loaded every event and paged in memory; it now asks ListEvents for the page, newest first. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The MCP endpoint served by the Go server at /mcp covers the eight tools of the Python server, with authentication and team scope. The Python server called four routes that do not exist (get_event, list_catalog, get_catalog_service, get_lock) and two search filters the API ignores. Dependabot no longer watches mcp-server; FEATURES.md documents the /mcp endpoint, and docs/MCP_SERVER.md keeps the migration notes. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Removes
mcp-server/(Python, stdio), deprecated since #238. The MCP endpoint served by the Go server at/mcp(server/mcp.go) replaces it.Parity check, tool by tool
/mcplist_eventstoday_eventssearch_eventsimpact,slack_idper_page/page.impactandslack_idare ignored by the API (utils.CreateFilternever reads them); looking an event up by its Slack id is done byget_eventget_event/events/{id}(404, the route is/event/{id})list_catalog/catalog/list(404, the route is/catalogs/list)get_catalog_service/catalog/{name}(no such route)list_locksget_lock/locks/{id}(404, the route is/lock/{id})On top: authentication (API keys, sessions, anonymous permissions), team scope, read-only annotations, nothing to install on the client.
Checked against a running
servwith a catalog entry, a lock and two events:tools/listreturns the 8 tools and each one answers (list_eventswith and without filter,today_events,search_events,get_event,list_catalog,get_catalog_service,list_locks,get_lock); a search without filter and an unknown enum value come back as tool errors.Also
list_eventswithout filter loaded every event and paged in memory; it now asksListEvents(paginated since perf: paginate ListEvents, gzip the HTTP responses, cache the assets #257) for the page. NewTestMCPListEventsPagesUnfilteredList./mcp-server(uv).FEATURES.md: the MCP section described the Python install (uv,TRACKER_URL); it now documents/mcpand points todocs/MCP_SERVER.md.docs/MCP_SERVER.md: deprecation notice removed, migration notes kept.CLAUDE.mdand the Copilot instructions updated.Checks
MONGO_TEST_URI=… go test ./...(all MCP end-to-end tests), gosec, staticcheck: OK.🤖 Generated with Claude Code