Visitar URL original
fix(mcp): restore legacy tool names (#33593) · rldona/FlupCode@f0b93fe · GitHub
Skip to content

Commit f0b93fe

Browse files
authored
fix(mcp): restore legacy tool names (anomalyco#33593)
1 parent df20322 commit f0b93fe

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

‎packages/opencode/src/mcp/index.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -647,7 +647,7 @@ export const layer = Layer.effect(
647647
}
648648
const timeout = requestTimeout(s, clientName, mcpConfig, defaultTimeout)
649649
for (const mcpTool of listed) {
650-
const key = "mcp__" + McpCatalog.sanitize(clientName) + "__" + McpCatalog.sanitize(mcpTool.name)
650+
const key = McpCatalog.sanitize(clientName) + "_" + McpCatalog.sanitize(mcpTool.name)
651651
result[key] = McpCatalog.convertTool(mcpTool, client, timeout)
652652
}
653653
}

‎packages/opencode/test/mcp/lifecycle.test.ts‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -382,7 +382,7 @@ it.instance(
382382
command: ["echo", "test"],
383383
})
384384

385-
expect(Object.keys(yield* mcp.tools())).toEqual(["mcp__paged-server__tool-one", "mcp__paged-server__tool-two"])
385+
expect(Object.keys(yield* mcp.tools())).toEqual(["paged-server_tool-one", "paged-server_tool-two"])
386386
expect(Object.keys(yield* mcp.prompts())).toEqual(["paged-server:prompt-one", "paged-server:prompt-two"])
387387
expect(Object.keys(yield* mcp.resources())).toEqual(["paged-server:test://one", "paged-server:test://two"])
388388
expect(Object.keys(yield* mcp.resourceTemplates())).toEqual([
@@ -944,7 +944,7 @@ it.instance(
944944

945945
expect(statusName(result.status, "tools-only-server")).toBe("connected")
946946
expect(serverState.listToolsCalls).toBe(1)
947-
expect(Object.keys(yield* mcp.tools())).toEqual(["mcp__tools-only-server__test_tool"])
947+
expect(Object.keys(yield* mcp.tools())).toEqual(["tools-only-server_test_tool"])
948948
expect(yield* mcp.prompts()).toEqual({})
949949
expect(yield* mcp.resources()).toEqual({})
950950
expect(serverState.listPromptsCalls).toBe(0)
@@ -1137,7 +1137,7 @@ it.instance(
11371137
const keys = Object.keys(tools)
11381138

11391139
// Server name dots should be replaced with underscores
1140-
expect(keys.some((k) => k.startsWith("mcp__my_special-server__"))).toBe(true)
1140+
expect(keys.some((k) => k.startsWith("my_special-server_"))).toBe(true)
11411141
// Tool name dots should be replaced with underscores
11421142
expect(keys.some((k) => k.endsWith("tool_b"))).toBe(true)
11431143
expect(keys.length).toBe(2)

0 commit comments

Comments
 (0)