From 8c5e9d93bfd3281c82ae877a79294b805a832b94 Mon Sep 17 00:00:00 2001 From: sharindanic Date: Sun, 12 Apr 2026 15:39:17 +0530 Subject: [PATCH 1/2] fix: add placeholder for MCP tool return type field --- .../src/views/BI/ServiceDesigner/Forms/McpToolForm/index.tsx | 1 + 1 file changed, 1 insertion(+) diff --git a/workspaces/ballerina/ballerina-visualizer/src/views/BI/ServiceDesigner/Forms/McpToolForm/index.tsx b/workspaces/ballerina/ballerina-visualizer/src/views/BI/ServiceDesigner/Forms/McpToolForm/index.tsx index 17124757f9c..629d6cbcdc4 100644 --- a/workspaces/ballerina/ballerina-visualizer/src/views/BI/ServiceDesigner/Forms/McpToolForm/index.tsx +++ b/workspaces/ballerina/ballerina-visualizer/src/views/BI/ServiceDesigner/Forms/McpToolForm/index.tsx @@ -201,6 +201,7 @@ export function McpToolForm(props: McpToolFormProps) { editable: model.returnType.editable, advanced: model.returnType.advanced, documentation: model.returnType.metadata?.description || "", + placeholder: model.returnType.metadata?.placeholder || "e.g. string, int, json", value: model.returnType.value, types: model.returnType.types, }, From 6a26ca20854edc16508841d0cff527c0e501b3c1 Mon Sep 17 00:00:00 2001 From: sharindanic Date: Sun, 12 Apr 2026 16:04:57 +0530 Subject: [PATCH 2/2] fix: use PropertyModel.placeholder before metadata fallback --- .../src/views/BI/ServiceDesigner/Forms/McpToolForm/index.tsx | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/workspaces/ballerina/ballerina-visualizer/src/views/BI/ServiceDesigner/Forms/McpToolForm/index.tsx b/workspaces/ballerina/ballerina-visualizer/src/views/BI/ServiceDesigner/Forms/McpToolForm/index.tsx index 629d6cbcdc4..69685ffb812 100644 --- a/workspaces/ballerina/ballerina-visualizer/src/views/BI/ServiceDesigner/Forms/McpToolForm/index.tsx +++ b/workspaces/ballerina/ballerina-visualizer/src/views/BI/ServiceDesigner/Forms/McpToolForm/index.tsx @@ -201,7 +201,10 @@ export function McpToolForm(props: McpToolFormProps) { editable: model.returnType.editable, advanced: model.returnType.advanced, documentation: model.returnType.metadata?.description || "", - placeholder: model.returnType.metadata?.placeholder || "e.g. string, int, json", + placeholder: + model.returnType.placeholder ?? + model.returnType.metadata?.placeholder ?? + "e.g. string, int, json", value: model.returnType.value, types: model.returnType.types, },