From dffd5d5af23911887a14f93cf54ecb0a646e0e95 Mon Sep 17 00:00:00 2001 From: Olayinka Adelakun Date: Fri, 30 Jan 2026 16:20:10 -0500 Subject: [PATCH 1/4] fix: hide MCP tool from model & agent --- .../flowSidebarComponent/components/sidebarItemsList.tsx | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/frontend/src/pages/FlowPage/components/flowSidebarComponent/components/sidebarItemsList.tsx b/src/frontend/src/pages/FlowPage/components/flowSidebarComponent/components/sidebarItemsList.tsx index 7dfde7838838..20481ac17556 100644 --- a/src/frontend/src/pages/FlowPage/components/flowSidebarComponent/components/sidebarItemsList.tsx +++ b/src/frontend/src/pages/FlowPage/components/flowSidebarComponent/components/sidebarItemsList.tsx @@ -38,6 +38,9 @@ const SidebarItemsList = ({ }) .map((SBItemName) => { const currentItem = dataFilter[item.name][SBItemName]; + if (SBItemName === "MCPTools") { + return null; + } if (SBItemName === "ChatInput" || SBItemName === "Webhook") { return ( From ee4185c6bd85018bfbee4abba61fa1d03516f225 Mon Sep 17 00:00:00 2001 From: Olayinka Adelakun Date: Thu, 5 Feb 2026 17:04:58 -0500 Subject: [PATCH 2/4] fix: removing mcp searching --- .../components/flowSidebarComponent/index.tsx | 4 ++-- .../tests/extended/features/mcp-server-tab.spec.ts | 14 ++------------ 2 files changed, 4 insertions(+), 14 deletions(-) diff --git a/src/frontend/src/pages/FlowPage/components/flowSidebarComponent/index.tsx b/src/frontend/src/pages/FlowPage/components/flowSidebarComponent/index.tsx index 098b9a47abb5..3390bdbb0beb 100644 --- a/src/frontend/src/pages/FlowPage/components/flowSidebarComponent/index.tsx +++ b/src/frontend/src/pages/FlowPage/components/flowSidebarComponent/index.tsx @@ -67,7 +67,7 @@ export type SearchContextType = { // Additional properties for the sidebar to use search?: string; setSearch?: (value: string) => void; - searchInputRef?: React.RefObject; + searchInputRef?: React.RefObject; handleInputFocus?: () => void; handleInputBlur?: () => void; handleInputChange?: (event: React.ChangeEvent) => void; @@ -668,7 +668,7 @@ export function FlowSidebarComponent({ isLoading }: FlowSidebarComponentProps) { <> {hasResults ? ( <> - {showComponents && ( + {showComponents && !showMcp && ( Date: Thu, 5 Feb 2026 18:01:36 -0500 Subject: [PATCH 3/4] fix testcases --- src/frontend/tests/extended/features/mcp-server-tab.spec.ts | 2 -- src/frontend/tests/utils/open-add-mcp-server-modal.ts | 2 +- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/src/frontend/tests/extended/features/mcp-server-tab.spec.ts b/src/frontend/tests/extended/features/mcp-server-tab.spec.ts index 6851d78d2183..d1f5384e4438 100644 --- a/src/frontend/tests/extended/features/mcp-server-tab.spec.ts +++ b/src/frontend/tests/extended/features/mcp-server-tab.spec.ts @@ -239,8 +239,6 @@ test( await adjustScreenView(page, { numberOfZoomOut: 3 }); - await expect(page.getByTestId("dropdown_str_tool")).toBeHidden(); - await openAddMcpServerModal(page); await page.waitForSelector('[data-testid="json-input"]', { diff --git a/src/frontend/tests/utils/open-add-mcp-server-modal.ts b/src/frontend/tests/utils/open-add-mcp-server-modal.ts index 136b75e2bd61..95df2b3b6ef0 100644 --- a/src/frontend/tests/utils/open-add-mcp-server-modal.ts +++ b/src/frontend/tests/utils/open-add-mcp-server-modal.ts @@ -8,7 +8,7 @@ export async function openAddMcpServerModal(page: Page) { } else { // Otherwise use the dropdown await page.getByTestId("mcp-server-dropdown").click({ timeout: 3000 }); - await page.getByText("Add MCP Server", { exact: true }).click({ + await page.getByText("Add MCP Server", { exact: true }).last().click({ timeout: 5000, }); } From edeb8228569d225feaf83e69e49aabc86a991b55 Mon Sep 17 00:00:00 2001 From: Olayinka Adelakun Date: Thu, 5 Feb 2026 18:47:46 -0500 Subject: [PATCH 4/4] fix testcases --- .../extended/features/mcp-server.spec.ts | 72 +++---------------- 1 file changed, 10 insertions(+), 62 deletions(-) diff --git a/src/frontend/tests/extended/features/mcp-server.spec.ts b/src/frontend/tests/extended/features/mcp-server.spec.ts index a072fe27833f..d59d8e6f8987 100644 --- a/src/frontend/tests/extended/features/mcp-server.spec.ts +++ b/src/frontend/tests/extended/features/mcp-server.spec.ts @@ -16,18 +16,8 @@ test( timeout: 30000, }); await page.getByTestId("blank-flow").click(); - await page.getByTestId("sidebar-search-input").click(); - await page.getByTestId("sidebar-search-input").fill("mcp tools"); - - await page.waitForSelector('[data-testid="models_and_agentsMCP Tools"]', { - timeout: 30000, - }); - - await page - .getByTestId("models_and_agentsMCP Tools") - .dragTo(page.locator('//*[@id="react-flow-id"]'), { - targetPosition: { x: 100, y: 100 }, - }); + await page.getByTestId("sidebar-nav-mcp").click(); + await page.getByTestId("add-component-button-lf-starter_project").click(); // See if the color matches @@ -48,8 +38,6 @@ test( await adjustScreenView(page, { numberOfZoomOut: 3 }); - await expect(page.getByTestId("dropdown_str_tool")).toBeHidden(); - await openAddMcpServerModal(page); await page.getByTestId("stdio-tab").click(); @@ -337,18 +325,9 @@ test( timeout: 30000, }); await page.getByTestId("blank-flow").click(); - await page.getByTestId("sidebar-search-input").click(); - await page.getByTestId("sidebar-search-input").fill("mcp tools"); - - await page.waitForSelector('[data-testid="models_and_agentsMCP Tools"]', { - timeout: 30000, - }); + await page.getByTestId("sidebar-nav-mcp").click(); + await page.getByTestId("add-component-button-lf-starter_project").click(); - await page - .getByTestId("models_and_agentsMCP Tools") - .dragTo(page.locator('//*[@id="react-flow-id"]'), { - targetPosition: { x: 100, y: 100 }, - }); await adjustScreenView(page, { numberOfZoomOut: 3 }); await openAddMcpServerModal(page); @@ -490,18 +469,9 @@ test( timeout: 30000, }); await page.getByTestId("blank-flow").click(); - await page.getByTestId("sidebar-search-input").click(); - await page.getByTestId("sidebar-search-input").fill("mcp tools"); - - await page.waitForSelector('[data-testid="models_and_agentsMCP Tools"]', { - timeout: 30000, - }); + await page.getByTestId("sidebar-nav-mcp").click(); + await page.getByTestId("add-component-button-lf-starter_project").click(); - await page - .getByTestId("models_and_agentsMCP Tools") - .dragTo(page.locator('//*[@id="react-flow-id"]'), { - targetPosition: { x: 100, y: 100 }, - }); await adjustScreenView(page, { numberOfZoomOut: 3 }); await openAddMcpServerModal(page); @@ -660,18 +630,8 @@ test( timeout: 30000, }); await page.getByTestId("blank-flow").click(); - await page.getByTestId("sidebar-search-input").click(); - await page.getByTestId("sidebar-search-input").fill("mcp tools"); - - await page.waitForSelector('[data-testid="models_and_agentsMCP Tools"]', { - timeout: 30000, - }); - - await page - .getByTestId("models_and_agentsMCP Tools") - .dragTo(page.locator('//*[@id="react-flow-id"]'), { - targetPosition: { x: 100, y: 100 }, - }); + await page.getByTestId("sidebar-nav-mcp").click(); + await page.getByTestId("add-component-button-lf-starter_project").click(); await page.getByTestId("canvas_controls_dropdown").click(); @@ -680,8 +640,6 @@ test( await zoomOut(page, 3); await page.getByTestId("canvas_controls_dropdown").click({ force: true }); - await expect(page.getByTestId("dropdown_str_tool")).toBeHidden(); - await openAddMcpServerModal(page); await page.getByTestId("stdio-tab").click(); @@ -939,18 +897,8 @@ test( timeout: 30000, }); await page.getByTestId("blank-flow").click(); - await page.getByTestId("sidebar-search-input").click(); - await page.getByTestId("sidebar-search-input").fill("mcp tools"); - - await page.waitForSelector('[data-testid="models_and_agentsMCP Tools"]', { - timeout: 30000, - }); - - await page - .getByTestId("models_and_agentsMCP Tools") - .dragTo(page.locator('//*[@id="react-flow-id"]'), { - targetPosition: { x: 100, y: 100 }, - }); + await page.getByTestId("sidebar-nav-mcp").click(); + await page.getByTestId("add-component-button-lf-starter_project").click(); await adjustScreenView(page, { numberOfZoomOut: 3 });