From 56b09d9f72731b75eadaf14e1331cf5f597032ad Mon Sep 17 00:00:00 2001
From: yyh <yuanyouhuilyz@gmail.com>
Date: Tue, 20 Jan 2026 14:26:21 +0800
Subject: [PATCH] fix: download option trigger open tab

---
 .../workflow/skill/file-tree/menu-item.tsx    | 31 ++++++++++++-------
 1 file changed, 19 insertions(+), 12 deletions(-)

diff --git a/web/app/components/workflow/skill/file-tree/menu-item.tsx b/web/app/components/workflow/skill/file-tree/menu-item.tsx
index 8a95268cdf..3af2799292 100644
--- a/web/app/components/workflow/skill/file-tree/menu-item.tsx
+++ b/web/app/components/workflow/skill/file-tree/menu-item.tsx
@@ -52,20 +52,27 @@ const labelVariants = cva('system-sm-regular text-text-secondary', {
 export type MenuItemProps = {
   icon: React.ElementType
   label: string
-  onClick: () => void
+  onClick: React.MouseEventHandler<HTMLButtonElement>
   disabled?: boolean
 } & VariantProps<typeof menuItemVariants>
 
-const MenuItem: FC<MenuItemProps> = ({ icon: Icon, label, onClick, disabled, variant }) => (
-  <button
-    type="button"
-    onClick={onClick}
-    disabled={disabled}
-    className={cn(menuItemVariants({ variant }))}
-  >
-    <Icon className={cn(iconVariants({ variant }))} aria-hidden="true" />
-    <span className={cn(labelVariants({ variant }))}>{label}</span>
-  </button>
-)
+const MenuItem: FC<MenuItemProps> = ({ icon: Icon, label, onClick, disabled, variant }) => {
+  const handleClick = React.useCallback((event: React.MouseEvent<HTMLButtonElement>) => {
+    event.stopPropagation()
+    onClick(event)
+  }, [onClick])
+
+  return (
+    <button
+      type="button"
+      onClick={handleClick}
+      disabled={disabled}
+      className={cn(menuItemVariants({ variant }))}
+    >
+      <Icon className={cn(iconVariants({ variant }))} aria-hidden="true" />
+      <span className={cn(labelVariants({ variant }))}>{label}</span>
+    </button>
+  )
+}
 
 export default React.memo(MenuItem)