From d2bfd703033fbaa8e7fde038dddf76a8c788d7e8 Mon Sep 17 00:00:00 2001
From: Frank_zhu <58329837+Frank-zhu0404@users.noreply.github.com>
Date: Thu, 17 Sep 2026 17:40:11 +0000
Subject: [PATCH] fix: treat missing MCP schema required as optional (#6955)

When inputSchema has no required field (or it is empty), optional
chaining made `undefined !== -1` true and marked every param required.
Default missing/empty required to [] so params stay optional.
---
 ui/src/workflow/nodes/mcp-node/index.vue | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/ui/src/workflow/nodes/mcp-node/index.vue b/ui/src/workflow/nodes/mcp-node/index.vue
index 5e5a64e9b4f..7947092365d 100644
--- a/ui/src/workflow/nodes/mcp-node/index.vue
+++ b/ui/src/workflow/nodes/mcp-node/index.vue
@@ -482,11 +482,11 @@ function changeTool() {
           },
           input_type: input_type,
           source: 'referencing',
-          required: args_schema.properties[item].required?.indexOf(item2) !== -1,
+          required: (args_schema.properties[item].required || []).indexOf(item2) !== -1,
           props_info: {
             rules: [
               {
-                required: args_schema.properties[item].required?.indexOf(item2) !== -1,
+                required: (args_schema.properties[item].required || []).indexOf(item2) !== -1,
                 message: t('dynamicsForm.tip.requiredMessage'),
                 trigger: 'blur',
               },
@@ -518,11 +518,11 @@ function changeTool() {
         },
         input_type: input_type,
         source: 'referencing',
-        required: args_schema.required?.indexOf(item) !== -1,
+        required: (args_schema.required || []).indexOf(item) !== -1,
         props_info: {
           rules: [
             {
-              required: args_schema.required?.indexOf(item) !== -1,
+              required: (args_schema.required || []).indexOf(item) !== -1,
               message: t('dynamicsForm.tip.requiredMessage'),
               trigger: 'blur',
             },