Fixes tool name prefix stripping to use server name instead of catalog name.
Fixes #1179 /claim #1179
Tools are stored as {serverName}__{toolName} but the code was trying to strip {catalogName}__, causing MCP servers to receive incorrectly prefixed tool names.
Strip server name prefix (the correct prefix) as the primary behavior, removing the incorrect catalog name logic.
[!NOTE] Fixes tool name prefixing to strip server-name (with catalog-name fallback) and aligns local tool persistence; adds tests and minor e2e config updates.
- Backend
- Tool call prefixing: In
src/clients/mcp-client.ts, stripmcpServerNameprefix fromtoolCall.name; if unchanged and catalog differs, fall back to strippingcatalogName.- Local tool persistence: In
src/routes/mcp-server.ts, persist tools usingmcpServer.nameas the prefix (remove catalog-name usage).- Tests
- Add cases in
src/clients/mcp-client.test.tsverifying server-name stripping and catalog-name fallback for mixed naming (n8n-lidar__*vsn8n__*).- Maintain existing behaviors across transports and response modifier templates.
- Config
- Update
platform/e2e-tests/knip.config.tsentries (addfixtures.tsand auth setup files) and ignoretscbinary.Written by Cursor Bugbot for commit bc1cfb60dae6badc1efde22a44b35c1dff468247. This will update automatically on new commits. Configure here.
Harsh Kumar
@thisisharsh7
Archestra
@archestra-ai