From 148f92f92d0d097b41d63a3abb07ff5ea0499b92 Mon Sep 17 00:00:00 2001
From: Joel <iamjoel007@gmail.com>
Date: Fri, 16 Jan 2026 17:20:11 +0800
Subject: [PATCH] fix: allow all fileds and not allow model set to auto

---
 .../tool-selector/reasoning-config-form.tsx   | 23 +++++++++++--------
 .../tool-setting/tool-settings-section.tsx    | 20 ++++------------
 2 files changed, 18 insertions(+), 25 deletions(-)

diff --git a/web/app/components/plugins/plugin-detail-panel/tool-selector/reasoning-config-form.tsx b/web/app/components/plugins/plugin-detail-panel/tool-selector/reasoning-config-form.tsx
index 86141eb055..346cb06f93 100644
--- a/web/app/components/plugins/plugin-detail-panel/tool-selector/reasoning-config-form.tsx
+++ b/web/app/components/plugins/plugin-detail-panel/tool-selector/reasoning-config-form.tsx
@@ -64,6 +64,8 @@ const ReasoningConfigForm: React.FC<Props> = ({
   }
 
   const handleAutomatic = (key: string, val: any, type: FormTypeEnum) => {
+    if (type === FormTypeEnum.modelSelector)
+      return
     onChange({
       ...value,
       [key]: {
@@ -151,7 +153,8 @@ const ReasoningConfigForm: React.FC<Props> = ({
       placeholder,
       options,
     } = schema
-    const auto = value[variable]?.auto
+    const canUseAuto = type !== FormTypeEnum.modelSelector
+    const auto = canUseAuto ? value[variable]?.auto : 0
     const tooltipContent = (tooltip && (
       <Tooltip
         popupContent={(
@@ -241,14 +244,16 @@ const ReasoningConfigForm: React.FC<Props> = ({
             )}
 
           </div>
-          <div className="flex cursor-pointer items-center gap-1 rounded-[6px] border border-divider-subtle bg-background-default-lighter px-2 py-1 hover:bg-state-base-hover" onClick={() => handleAutomatic(variable, !auto, type)}>
-            <span className="system-xs-medium text-text-secondary">{t('detailPanel.toolSelector.auto', { ns: 'plugin' })}</span>
-            <Switch
-              size="xs"
-              defaultValue={!!auto}
-              onChange={val => handleAutomatic(variable, val, type)}
-            />
-          </div>
+          {canUseAuto && (
+            <div className="flex cursor-pointer items-center gap-1 rounded-[6px] border border-divider-subtle bg-background-default-lighter px-2 py-1 hover:bg-state-base-hover" onClick={() => handleAutomatic(variable, !auto, type)}>
+              <span className="system-xs-medium text-text-secondary">{t('detailPanel.toolSelector.auto', { ns: 'plugin' })}</span>
+              <Switch
+                size="xs"
+                defaultValue={!!auto}
+                onChange={val => handleAutomatic(variable, val, type)}
+              />
+            </div>
+          )}
         </div>
         {auto === 0 && (
           <div className={cn('gap-1', !(isShowJSONEditor && isConstant) && 'flex')}>
diff --git a/web/app/components/workflow/skill/editor/skill-editor/tool-setting/tool-settings-section.tsx b/web/app/components/workflow/skill/editor/skill-editor/tool-setting/tool-settings-section.tsx
index d7117dac0e..ef43a4f5ae 100644
--- a/web/app/components/workflow/skill/editor/skill-editor/tool-setting/tool-settings-section.tsx
+++ b/web/app/components/workflow/skill/editor/skill-editor/tool-setting/tool-settings-section.tsx
@@ -45,7 +45,6 @@ const ToolSettingsSection: FC<ToolSettingsSectionProps> = ({
   const settingsFormSchemas = useMemo(() => toolParametersToFormSchemas(currentToolSettings), [currentToolSettings])
   const paramsFormSchemas = useMemo(() => toolParametersToFormSchemas(currentToolParams), [currentToolParams])
 
-  const allowReasoning = !!safeNodeId
   const handleSettingsFormChange = (v: Record<string, any>) => {
     if (!value || !onChange)
       return
@@ -71,7 +70,7 @@ const ToolSettingsSection: FC<ToolSettingsSectionProps> = ({
     return null
 
   const showSettingsSection = currentToolSettings.length > 0
-  const showParamsSection = allowReasoning && currentToolParams.length > 0
+  const showParamsSection = currentToolParams.length > 0
   const getVarKindType = (type: FormTypeEnum) => {
     if (type === FormTypeEnum.file || type === FormTypeEnum.files)
       return VarKindType.variable
@@ -109,20 +108,9 @@ const ToolSettingsSection: FC<ToolSettingsSectionProps> = ({
   return (
     <>
       <Divider className="my-1 w-full" />
-      {showSettingsSection && (
-        <div className="p-4 pb-1">
-          <div className="system-sm-semibold-uppercase text-text-primary">{t('detailPanel.toolSelector.settings', { ns: 'plugin' })}</div>
-        </div>
-      )}
-      {showParamsSection && (
-        <div className="mb-1 p-4 pb-1">
-          <div className="system-sm-semibold-uppercase text-text-primary">{t('detailPanel.toolSelector.params', { ns: 'plugin' })}</div>
-          <div className="pb-1">
-            <div className="system-xs-regular text-text-tertiary">{t('detailPanel.toolSelector.paramsTip1', { ns: 'plugin' })}</div>
-            <div className="system-xs-regular text-text-tertiary">{t('detailPanel.toolSelector.paramsTip2', { ns: 'plugin' })}</div>
-          </div>
-        </div>
-      )}
+      <div className="p-4 pb-1">
+        <div className="system-sm-semibold-uppercase text-text-primary">{t('detailPanel.toolSelector.settings', { ns: 'plugin' })}</div>
+      </div>
       {showSettingsSection && (
         <ReasoningConfigForm
           value={getSafeConfigValue(value?.settings, settingsFormSchemas)}