From ca50522f80c90a9ce988796d7a64b6674e872f7b Mon Sep 17 00:00:00 2001
From: Joel <iamjoel007@gmail.com>
Date: Fri, 1 Nov 2024 15:21:30 +0800
Subject: [PATCH 1/3] feat: set tool params

---
 .../app/configuration/config/agent/agent-tools/index.tsx | 2 +-
 .../workflow/block-selector/tool/action-item.tsx         | 9 +++++++--
 web/app/components/workflow/block-selector/tool/tool.tsx | 7 +++++++
 web/app/components/workflow/block-selector/types.ts      | 1 +
 4 files changed, 16 insertions(+), 3 deletions(-)

diff --git a/web/app/components/app/configuration/config/agent/agent-tools/index.tsx b/web/app/components/app/configuration/config/agent/agent-tools/index.tsx
index bbc352b9b4..57d2e5f632 100644
--- a/web/app/components/app/configuration/config/agent/agent-tools/index.tsx
+++ b/web/app/components/app/configuration/config/agent/agent-tools/index.tsx
@@ -91,7 +91,7 @@ const AgentTools: FC = () => {
         provider_name: tool.provider_name,
         tool_name: tool.tool_name,
         tool_label: tool.tool_label,
-        tool_parameters: {},
+        tool_parameters: tool.params,
         enabled: true,
       })
     })
diff --git a/web/app/components/workflow/block-selector/tool/action-item.tsx b/web/app/components/workflow/block-selector/tool/action-item.tsx
index c233200b05..e33f625861 100644
--- a/web/app/components/workflow/block-selector/tool/action-item.tsx
+++ b/web/app/components/workflow/block-selector/tool/action-item.tsx
@@ -10,14 +10,12 @@ import { useGetLanguage } from '@/context/i18n'
 import BlockIcon from '../../block-icon'
 
 type Props = {
-  className?: string
   provider: ToolWithProvider
   payload: Tool
   onSelect: (type: BlockEnum, tool?: ToolDefaultValue) => void
 }
 
 const ToolItem: FC<Props> = ({
-  className,
   provider,
   payload,
   onSelect,
@@ -46,6 +44,12 @@ const ToolItem: FC<Props> = ({
         key={payload.name}
         className='rounded-lg pl-[21px] hover:bg-state-base-hover cursor-pointer'
         onClick={() => {
+          const params: Record<string, string> = {}
+          if (payload.parameters) {
+            payload.parameters.forEach((item) => {
+              params[item.name] = ''
+            })
+          }
           onSelect(BlockEnum.Tool, {
             provider_id: provider.id,
             provider_type: provider.type,
@@ -53,6 +57,7 @@ const ToolItem: FC<Props> = ({
             tool_name: payload.name,
             tool_label: payload.label[language],
             title: payload.label[language],
+            params,
           })
         }}
       >
diff --git a/web/app/components/workflow/block-selector/tool/tool.tsx b/web/app/components/workflow/block-selector/tool/tool.tsx
index 679f0b0e2e..f7433b8e60 100644
--- a/web/app/components/workflow/block-selector/tool/tool.tsx
+++ b/web/app/components/workflow/block-selector/tool/tool.tsx
@@ -66,6 +66,12 @@ const Tool: FC<Props> = ({
               toggleFold()
               return
             }
+            // TODO: get workflow and custom tool params
+            // if (payload.parameters) {
+            //   payload.parameters.forEach((item) => {
+            //     params[item.name] = ''
+            //   })
+            // }
             onSelect(BlockEnum.Tool, {
               provider_id: payload.id,
               provider_type: payload.type,
@@ -73,6 +79,7 @@ const Tool: FC<Props> = ({
               tool_name: payload.name,
               tool_label: payload.label[language],
               title: payload.label[language],
+              params: {},
             })
           }}
         >
diff --git a/web/app/components/workflow/block-selector/types.ts b/web/app/components/workflow/block-selector/types.ts
index affa2488b9..9bdbf5cb3c 100644
--- a/web/app/components/workflow/block-selector/types.ts
+++ b/web/app/components/workflow/block-selector/types.ts
@@ -25,4 +25,5 @@ export type ToolDefaultValue = {
   tool_name: string
   tool_label: string
   title: string
+  params: Record<string, any>
 }

From 5d5db7c6c19f54cfa94e67c4e9a986771286e773 Mon Sep 17 00:00:00 2001
From: Joel <iamjoel007@gmail.com>
Date: Fri, 1 Nov 2024 15:28:10 +0800
Subject: [PATCH 2/3] fix: key too long breaks ui

---
 web/app/components/plugins/base/key-value-item.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/web/app/components/plugins/base/key-value-item.tsx b/web/app/components/plugins/base/key-value-item.tsx
index 50d3b53535..faa81f64e7 100644
--- a/web/app/components/plugins/base/key-value-item.tsx
+++ b/web/app/components/plugins/base/key-value-item.tsx
@@ -46,7 +46,7 @@ const KeyValueItem: FC<Props> = ({
     <div className='flex items-center gap-1'>
       <span className={cn('flex flex-col justify-center items-start text-text-tertiary system-xs-medium', labelWidthClassName)}>{label}</span>
       <div className='flex justify-center items-center gap-0.5'>
-        <span className='max-w-[300px] truncate system-xs-medium text-text-secondary'>
+        <span className='max-w-[162px] truncate system-xs-medium text-text-secondary'>
           {value}
         </span>
         <Tooltip popupContent={t(`common.operation.${isCopied ? 'copied' : 'copy'}`)} position='top'>

From 4caa8f38bcb8a197589f2d2fb54badd5854df62f Mon Sep 17 00:00:00 2001
From: JzoNg <jzongcode@gmail.com>
Date: Fri, 1 Nov 2024 15:33:15 +0800
Subject: [PATCH 3/3] hide plugin detail

---
 .../plugins/plugin-page/plugins-panel.tsx      | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/web/app/components/plugins/plugin-page/plugins-panel.tsx b/web/app/components/plugins/plugin-page/plugins-panel.tsx
index 8dbbf8eaa5..768af9d706 100644
--- a/web/app/components/plugins/plugin-page/plugins-panel.tsx
+++ b/web/app/components/plugins/plugin-page/plugins-panel.tsx
@@ -44,14 +44,16 @@ const PluginsPanel = () => {
           <List pluginList={filteredList} />
         </div>
       </div>
-      <PluginDetailPanel
-        pluginDetail={currentPluginDetail}
-        endpointList={currentPluginEndpoints}
-        onHide={() => {
-          setCurrentPluginDetail(undefined)
-          setCurrentEndpoints([])
-        }}
-      />
+      {false && (
+        <PluginDetailPanel
+          pluginDetail={currentPluginDetail}
+          endpointList={currentPluginEndpoints}
+          onHide={() => {
+            setCurrentPluginDetail(undefined)
+            setCurrentEndpoints([])
+          }}
+        />
+      )}
     </>
   )
 }