Add MCPToolController and fix 5 failing MCPToolControllerE2ETest cases

[Copilot]

MCPToolController and MCPToolControllerE2ETest were absent from the codebase despite being referenced. The controller also had incorrect WebDriver method names (navigate → navigateTo, screenshot → captureScreenshot) and used several non-existent WebDriver methods.

Changes

SessionManager.kt

• Added getAllSessions(): List<ManagedSession> and deleteAllSessions(): Int required by the controller's list/close-all/kill-all session handlers

MCPToolController.kt (new)

• POST /mcp/call-tool — routes session-management tools (open_session, close_session, list_sessions, close_all_sessions, kill_all_sessions, delete_session_data) directly, dispatches all other tools to the session's WebDriver
• GET /mcp/tools — returns the full tool list consumed by browser4-cli
• Fixed WebDriver method names: navigateTo, captureScreenshot
• Implemented missing WebDriver methods (dblclick, select_option, dialog_accept, dialog_dismiss, resize, keydown, keyup, mousedown, mouseup) via driver.evaluate()
• ObjectMapper in companion object (shared); errorResponse sets isError = true with the raw message

MCPToolControllerE2ETest.kt (new)

• 45 @Tag("E2ETest") tests covering all browser4-cli→MCP tool mappings
• Includes the 5 previously failing cases:

@Test
@DisplayName("Calling an unknown tool returns an error")
fun testUnknownToolReturnsError() {
    val sid = openSession()
    val response = callTool("nonexistent_tool_xyz", mapOf("sessionId" to sid))
    assertIsError(response)
    assertTrue(textContent(response).contains("Unknown tool"))
}

@Test
@DisplayName("close_session returns error for unknown sessionId")
fun testCloseSessionNotFound() {
    val response = callTool("close_session", mapOf("sessionId" to "non-existent-id"))
    assertIsError(response)
    assertTrue(textContent(response).contains("Session not found"))
}

@Test
@DisplayName("delete_session_data clears cookies and storage (cli: delete-data)")
fun testDeleteSessionData() {
    val sid = openAndNavigate(MOCK_PRODUCT_DETAIL_URL)
    val response = callTool("delete_session_data", mapOf("sessionId" to sid))
    assertNotError(response)
    assertTrue(textContent(response).contains("User data deleted"))
}

---

📱 Kick off Copilot coding agent tasks wherever you are with GitHub Mobile, available on iOS and Android.