Merge remote-tracking branch 'origin/chore-unskip-e2e-use-mcp-tool' into chore-unskip-e2e-subtasks

roomote[bot] · roomote[bot] · commit 895516b7aab0 · 2026-05-19T05:09:10.000Z
diff --git a/apps/vscode-e2e/src/suite/tools/use-mcp-tool.test.ts b/apps/vscode-e2e/src/suite/tools/use-mcp-tool.test.ts
@@ -129,9 +129,12 @@ suite("Roo Code use_mcp_tool Tool", function () {
 
 			if (message.type === "ask" && message.ask === "use_mcp_server" && message.text && !mcpRequest) {
 				try {
-					mcpRequest = JSON.parse(message.text) as ParsedMcpRequest
+					const parsed = JSON.parse(message.text) as ParsedMcpRequest
+					if (parsed.serverName && parsed.toolName) {
+						mcpRequest = parsed
+					}
 				} catch {
-					mcpRequest = null
+					// Ignore partial JSON; a later complete ask will overwrite.
 				}
 			}
 

Original file line number	Diff line number	Diff line change
`@@ -129,9 +129,12 @@ suite("Roo Code use_mcp_tool Tool", function () {`
`129`	`129`
`130`	`130`	`if (message.type === "ask" && message.ask === "use_mcp_server" && message.text && !mcpRequest) {`
`131`	`131`	`try {`
`132`		`- mcpRequest = JSON.parse(message.text) as ParsedMcpRequest`
	`132`	`+ const parsed = JSON.parse(message.text) as ParsedMcpRequest`
	`133`	`+ if (parsed.serverName && parsed.toolName) {`
	`134`	`+ mcpRequest = parsed`
	`135`	`+ }`
`133`	`136`	`} catch {`
`134`		`- mcpRequest = null`
	`137`	`+ // Ignore partial JSON; a later complete ask will overwrite.`
`135`	`138`	`}`
`136`	`139`	`}`
`137`	`140`