From bb01fd00fc0a432589120fdf1cafcc57a0e6a592 Mon Sep 17 00:00:00 2001 From: Lucas Oliveira <62335616+lucaseduoli@users.noreply.github.com> Date: Tue, 19 Aug 2025 15:33:28 -0300 Subject: [PATCH] fix: remove duplicate embeddings category, refactor render code (#9446) Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com> --- .../flowSidebarComponent/components/bundleItems.tsx | 7 ------- .../flowSidebarComponent/components/sidebarBundles.tsx | 10 +++++++--- src/frontend/src/utils/styleUtils.ts | 2 -- 3 files changed, 7 insertions(+), 12 deletions(-) diff --git a/src/frontend/src/pages/FlowPage/components/flowSidebarComponent/components/bundleItems.tsx b/src/frontend/src/pages/FlowPage/components/flowSidebarComponent/components/bundleItems.tsx index e0547600d..6406dd577 100644 --- a/src/frontend/src/pages/FlowPage/components/flowSidebarComponent/components/bundleItems.tsx +++ b/src/frontend/src/pages/FlowPage/components/flowSidebarComponent/components/bundleItems.tsx @@ -20,13 +20,6 @@ export const BundleItem = memo( sensitiveSort, handleKeyDownInput, }: BundleItemProps) => { - if ( - !dataFilter[item.name] || - Object.keys(dataFilter[item.name]).length === 0 - ) { - return null; - } - const isOpen = openCategories.includes(item.name); const handleOpenChange = useCallback( diff --git a/src/frontend/src/pages/FlowPage/components/flowSidebarComponent/components/sidebarBundles.tsx b/src/frontend/src/pages/FlowPage/components/flowSidebarComponent/components/sidebarBundles.tsx index e37d5e7c0..505ebb8f5 100644 --- a/src/frontend/src/pages/FlowPage/components/flowSidebarComponent/components/sidebarBundles.tsx +++ b/src/frontend/src/pages/FlowPage/components/flowSidebarComponent/components/sidebarBundles.tsx @@ -1,4 +1,4 @@ -import { memo, useCallback, useMemo, useState } from "react"; +import { memo, useMemo } from "react"; import { SidebarGroup, SidebarGroupContent, @@ -28,8 +28,12 @@ export const MemoizedSidebarGroup = memo( referenceArray.findIndex((value) => value === a.name) - referenceArray.findIndex((value) => value === b.name) ); - }); - }, [BUNDLES, search, sortedCategories]); + }).filter( + (item: { name: string | number }) => + dataFilter[item.name] && + Object.keys(dataFilter[item.name]).length > 0, + ); + }, [BUNDLES, search, sortedCategories, dataFilter]); return ( diff --git a/src/frontend/src/utils/styleUtils.ts b/src/frontend/src/utils/styleUtils.ts index bddf5caf2..9b36c496a 100644 --- a/src/frontend/src/utils/styleUtils.ts +++ b/src/frontend/src/utils/styleUtils.ts @@ -242,7 +242,6 @@ export const SIDEBAR_BUNDLES = [ { display_name: "Amazon", name: "amazon", icon: "Amazon" }, { display_name: "Anthropic", name: "anthropic", icon: "Anthropic" }, { display_name: "Apify", name: "apify", icon: "Apify" }, - { display_name: "arXiv", name: "arxiv", icon: "arXiv" }, { display_name: "AssemblyAI", name: "assemblyai", icon: "AssemblyAI" }, { display_name: "Azure", name: "azure", icon: "Azure" }, @@ -258,7 +257,6 @@ export const SIDEBAR_BUNDLES = [ { display_name: "DeepSeek", name: "deepseek", icon: "DeepSeek" }, { display_name: "Docling", name: "docling", icon: "Docling" }, { display_name: "DuckDuckGo", name: "duckduckgo", icon: "DuckDuckGo" }, - { display_name: "Embeddings", name: "embeddings", icon: "Binary" }, { display_name: "Exa", name: "exa", icon: "Exa" }, { display_name: "Firecrawl", name: "firecrawl", icon: "FirecrawlCrawlApi" }, { display_name: "Git", name: "git", icon: "GitLoader" },