Skip to content

fix(mcp): keep pre-snake_case tool names as deprecated aliases - #1782

Merged
graphite-app[bot] merged 1 commit into
mainfrom
mcp-legacy-tool-aliases
Oct 7, 2026
Merged

graphite-app[bot] merged 1 commit into
mainfrom
mcp-legacy-tool-aliases

Conversation

@MaheshtheDev

Copy link
Copy Markdown
Member

#1665 renamed getDocument, listSpaces, listDocuments, listMemories and whoAmI to snake_case with no aliases. Clients with a cached tool list (ChatGPT, Codex, Claude) still call the old names and fail 100% of the time (~2k errors from ~300 users in the last 10 days, per PostHog). The old names are now registered again with the same handler and a "Deprecated alias" description. They stay listed because ChatGPT rejects calls to tools missing from tools/list.

Copy link
Copy Markdown
Member Author

How to use the Graphite Merge Queue

Add the label Main to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@MaheshtheDev
MaheshtheDev marked this pull request as ready for review October 7, 2026 17:29
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
🔵 In progress
View logs
supermemory-app 203fb6e Oct 07 2026, 06:51 PM

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
supermemory-mcp 203fb6e Oct 07 2026, 06:52 PM

Comment thread apps/mcp/src/server/tools/index.ts Outdated
Comment thread apps/mcp/src/server/tools/index.ts
Comment thread apps/mcp/src/server/tools/index.ts
Comment thread apps/mcp/src/server/tools/index.ts
Comment thread apps/mcp/e2e/discovery.test.ts
@graphite-app

graphite-app Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Merge activity

#1665 renamed getDocument, listSpaces, listDocuments, listMemories and whoAmI to snake_case with no aliases. Clients with a cached tool list (ChatGPT, Codex, Claude) still call the old names and fail 100% of the time (~2k errors from ~300 users in the last 10 days, per PostHog). The old names are now registered again with the same handler and a "Deprecated alias" description. They stay listed because ChatGPT rejects calls to tools missing from tools/list.
@graphite-app
graphite-app Bot force-pushed the mcp-legacy-tool-aliases branch from 4f0f952 to 203fb6e Compare October 7, 2026 18:51
@graphite-app
graphite-app Bot merged commit 203fb6e into main Oct 7, 2026
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants