ui improvements

Author: icecrasher321

apps/sim/app/api/mcp/serve/[serverId]/route.ts

@@ -170,13 +170,18 @@ async function handleToolsList(id: RequestId, serverId: string): Promise<NextRes
 
     const result: ListToolsResult = {
       tools: tools.map((tool) => {
-        const schema = tool.parameterSchema as { properties?: Record<string, unknown> } | null
+        const schema = tool.parameterSchema as {
+          type?: string
+          properties?: Record<string, unknown>
+          required?: string[]
+        } | null
         return {
           name: tool.toolName,
           description: tool.toolDescription || `Execute workflow: ${tool.toolName}`,
           inputSchema: {
             type: 'object' as const,
             properties: schema?.properties || {},
+            ...(schema?.required && schema.required.length > 0 && { required: schema.required }),
           },
         }
       }),

apps/sim/app/api/mcp/workflow-servers/[id]/tools/[toolId]/route.ts

@@ -5,6 +5,7 @@ import type { NextRequest } from 'next/server'
 import { createLogger } from '@/lib/logs/console/logger'
 import { getParsedBody, withMcpAuth } from '@/lib/mcp/middleware'
 import { createMcpErrorResponse, createMcpSuccessResponse } from '@/lib/mcp/utils'
+import { sanitizeToolName } from '@/lib/mcp/workflow-tool-schema'
 
 const logger = createLogger('WorkflowMcpToolAPI')
 
@@ -99,7 +100,7 @@ export const PATCH = withMcpAuth<RouteParams>('write')(
       }
 
       if (body.toolName !== undefined) {
-        updateData.toolName = body.toolName.trim()
+        updateData.toolName = sanitizeToolName(body.toolName)
       }
       if (body.toolDescription !== undefined) {
         updateData.toolDescription = body.toolDescription?.trim() || null

apps/sim/app/api/mcp/workflow-servers/[id]/tools/route.ts

@@ -184,8 +184,7 @@ export const POST = withMcpAuth<RouteParams>('write')(
         )
       }
 
-      // Generate tool name and description
-      const toolName = body.toolName?.trim() || sanitizeToolName(workflowRecord.name)
+      const toolName = sanitizeToolName(body.toolName?.trim() || workflowRecord.name)
       const toolDescription =
         body.toolDescription?.trim() ||
         workflowRecord.description ||