cleanup code to use mcp sdk types

Author: icecrasher321

apps/sim/app/api/mcp/discover/route.ts

@@ -11,33 +11,21 @@ const logger = createLogger('McpDiscoverAPI')
 export const dynamic = 'force-dynamic'
 
 /**
- * GET - Discover all published MCP servers available to the authenticated user
- *
- * This endpoint allows external MCP clients to discover available servers
- * using just their API key, without needing to know workspace IDs.
- *
- * Authentication: API Key (X-API-Key header) or Session
- *
- * Returns all published MCP servers from workspaces the user has access to.
+ * Discover all MCP servers available to the authenticated user.
  */
 export async function GET(request: NextRequest) {
   try {
-    // Authenticate the request
     const auth = await checkHybridAuth(request, { requireWorkflowId: false })
 
     if (!auth.success || !auth.userId) {
       return NextResponse.json(
-        {
-          success: false,
-          error: 'Authentication required. Provide X-API-Key header with your Sim API key.',
-        },
+        { success: false, error: 'Authentication required. Provide X-API-Key header.' },
         { status: 401 }
       )
     }
 
     const userId = auth.userId
 
-    // Get all workspaces the user has access to via permissions table
     const userWorkspacePermissions = await db
       .select({ entityId: permissions.entityId })
       .from(permissions)
@@ -46,23 +34,17 @@ export async function GET(request: NextRequest) {
     const workspaceIds = userWorkspacePermissions.map((w) => w.entityId)
 
     if (workspaceIds.length === 0) {
-      return NextResponse.json({
-        success: true,
-        servers: [],
-        message: 'No workspaces found for this user',
-      })
+      return NextResponse.json({ success: true, servers: [] })
     }
 
-    // Get all published MCP servers from user's workspaces with tool count
     const servers = await db
       .select({
         id: workflowMcpServer.id,
         name: workflowMcpServer.name,
         description: workflowMcpServer.description,
         workspaceId: workflowMcpServer.workspaceId,
         workspaceName: workspace.name,
-        isPublished: workflowMcpServer.isPublished,
-        publishedAt: workflowMcpServer.publishedAt,
+        createdAt: workflowMcpServer.createdAt,
         toolCount: sql<number>`(
           SELECT COUNT(*)::int 
           FROM "workflow_mcp_tool" 
@@ -71,31 +53,19 @@ export async function GET(request: NextRequest) {
       })
       .from(workflowMcpServer)
       .leftJoin(workspace, eq(workflowMcpServer.workspaceId, workspace.id))
-      .where(
-        and(
-          eq(workflowMcpServer.isPublished, true),
-          sql`${workflowMcpServer.workspaceId} IN ${workspaceIds}`
-        )
-      )
+      .where(sql`${workflowMcpServer.workspaceId} IN ${workspaceIds}`)
       .orderBy(workflowMcpServer.name)
 
     const baseUrl = getBaseUrl()
 
-    // Format response with connection URLs
     const formattedServers = servers.map((server) => ({
       id: server.id,
       name: server.name,
       description: server.description,
-      workspace: {
-        id: server.workspaceId,
-        name: server.workspaceName,
-      },
+      workspace: { id: server.workspaceId, name: server.workspaceName },
       toolCount: server.toolCount || 0,
-      publishedAt: server.publishedAt,
-      urls: {
-        http: `${baseUrl}/api/mcp/serve/${server.id}`,
-        sse: `${baseUrl}/api/mcp/serve/${server.id}/sse`,
-      },
+      createdAt: server.createdAt,
+      url: `${baseUrl}/api/mcp/serve/${server.id}`,
     }))
 
     logger.info(`User ${userId} discovered ${formattedServers.length} MCP servers`)
@@ -106,17 +76,6 @@ export async function GET(request: NextRequest) {
       authentication: {
         method: 'API Key',
         header: 'X-API-Key',
-        description: 'Include your Sim API key in the X-API-Key header for all MCP requests',
-      },
-      usage: {
-        listTools: {
-          method: 'POST',
-          body: '{"jsonrpc":"2.0","id":1,"method":"tools/list"}',
-        },
-        callTool: {
-          method: 'POST',
-          body: '{"jsonrpc":"2.0","id":1,"method":"tools/call","params":{"name":"TOOL_NAME","arguments":{}}}',
-        },
       },
     })
   } catch (error) {