test: unskip use_mcp_tool e2e suite

Author: roomote

apps/vscode-e2e/src/fixtures/use-mcp-tool.ts

@@ -0,0 +1,109 @@
+import * as path from "path"
+
+import { LLMock } from "@copilotkit/aimock"
+
+const TEST_DIR_NAME = "use-mcp-tool-fixture"
+const FILESYSTEM_SERVER_NAME = "filesystem"
+
+type UseMcpToolFixture = {
+	userMessagePattern: string
+	toolCallId: string
+	toolName: string
+	toolArguments?: Record<string, unknown>
+	serverName?: string
+	result: string
+	id: string
+}
+
+export function addUseMcpToolResultFixtures(mock: InstanceType<typeof LLMock>, workspaceDir: string) {
+	const readFilePath = path.join(workspaceDir, TEST_DIR_NAME, "mcp-read-target.txt")
+	const writeFilePath = path.join(workspaceDir, TEST_DIR_NAME, "mcp-write-target.txt")
+
+	const fixtures: UseMcpToolFixture[] = [
+		{
+			userMessagePattern: "USE_MCP_TOOL_READ_FILE_SMOKE",
+			toolCallId: "call_use_mcp_tool_read_file_001",
+			toolName: "read_file",
+			toolArguments: { path: readFilePath },
+			result: "Read the requested file through the MCP filesystem server.",
+			id: "call_use_mcp_tool_read_file_002",
+		},
+		{
+			userMessagePattern: "USE_MCP_TOOL_WRITE_FILE_SMOKE",
+			toolCallId: "call_use_mcp_tool_write_file_001",
+			toolName: "write_file",
+			toolArguments: { path: writeFilePath, content: "Hello from MCP!" },
+			result: "Created the requested file through the MCP filesystem server.",
+			id: "call_use_mcp_tool_write_file_002",
+		},
+		{
+			userMessagePattern: "USE_MCP_TOOL_LIST_DIRECTORY_SMOKE",
+			toolCallId: "call_use_mcp_tool_list_directory_001",
+			toolName: "list_directory",
+			toolArguments: { path: path.join(workspaceDir, TEST_DIR_NAME) },
+			result: "Listed the requested directory through the MCP filesystem server.",
+			id: "call_use_mcp_tool_list_directory_002",
+		},
+		{
+			userMessagePattern: "USE_MCP_TOOL_DIRECTORY_TREE_SMOKE",
+			toolCallId: "call_use_mcp_tool_directory_tree_001",
+			toolName: "directory_tree",
+			toolArguments: { path: path.join(workspaceDir, TEST_DIR_NAME) },
+			result: "Returned the directory tree through the MCP filesystem server.",
+			id: "call_use_mcp_tool_directory_tree_002",
+		},
+		{
+			userMessagePattern: "USE_MCP_TOOL_GET_FILE_INFO_SMOKE",
+			toolCallId: "call_use_mcp_tool_get_file_info_001",
+			toolName: "get_file_info",
+			toolArguments: { path: readFilePath },
+			result: "Returned the requested file metadata through the MCP filesystem server.",
+			id: "call_use_mcp_tool_get_file_info_002",
+		},
+		{
+			userMessagePattern: "USE_MCP_TOOL_UNKNOWN_SERVER_SMOKE",
+			toolCallId: "call_use_mcp_tool_unknown_server_001",
+			serverName: "nonexistent-server",
+			toolName: "read_file",
+			toolArguments: { path: readFilePath },
+			result: "Handled the missing MCP server gracefully.",
+			id: "call_use_mcp_tool_unknown_server_002",
+		},
+	]
+
+	for (const fixture of fixtures) {
+		mock.addFixture({
+			match: {
+				userMessage: new RegExp(fixture.userMessagePattern),
+			},
+			response: {
+				toolCalls: [
+					{
+						name: "use_mcp_tool",
+						arguments: JSON.stringify({
+							server_name: fixture.serverName ?? FILESYSTEM_SERVER_NAME,
+							tool_name: fixture.toolName,
+							arguments: fixture.toolArguments,
+						}),
+						id: fixture.toolCallId,
+					},
+				],
+			},
+		})
+
+		mock.addFixture({
+			match: {
+				toolCallId: fixture.toolCallId,
+			},
+			response: {
+				toolCalls: [
+					{
+						name: "attempt_completion",
+						arguments: JSON.stringify({ result: fixture.result }),
+						id: fixture.id,
+					},
+				],
+			},
+		})
+	}
+}

apps/vscode-e2e/src/runTest.ts

@@ -10,6 +10,7 @@ import { addExecuteCommandResultFixtures } from "./fixtures/execute-command"
 import { addListFilesResultFixtures } from "./fixtures/list-files"
 import { addReadFileResultFixtures } from "./fixtures/read-file"
 import { addSearchFilesResultFixtures } from "./fixtures/search-files"
+import { addUseMcpToolResultFixtures } from "./fixtures/use-mcp-tool"
 import { addWriteToFileResultFixtures } from "./fixtures/write-to-file"
 
 async function main() {
@@ -39,6 +40,9 @@ async function main() {
 	let testWorkspace: string | undefined
 
 	try {
+		// Create a temporary workspace folder for tests
+		testWorkspace = await fs.mkdtemp(path.join(os.tmpdir(), "roo-test-workspace-"))
+
 		if (useMock) {
 			const fixturesDir = path.resolve(__dirname, "../fixtures")
 
@@ -67,6 +71,7 @@ async function main() {
 				addListFilesResultFixtures(mock)
 				addReadFileResultFixtures(mock)
 				addSearchFilesResultFixtures(mock)
+				addUseMcpToolResultFixtures(mock, testWorkspace)
 				addWriteToFileResultFixtures(mock)
 
 				// The modes test (switch_mode → ask) triggers a second API call whose last
@@ -90,9 +95,6 @@ async function main() {
 
 			await mock.start()
 		}
-
-		// Create a temporary workspace folder for tests
-		testWorkspace = await fs.mkdtemp(path.join(os.tmpdir(), "roo-test-workspace-"))
 		// Get test filter from command line arguments or environment variable
 		// Usage examples:
 		// - npm run test:e2e -- --grep "write-to-file"