resolve PR comments

Author: waleedlatif1

apps/sim/app/api/mcp/servers/[id]/refresh/route.ts

@@ -55,7 +55,7 @@ export const POST = withMcpAuth('read')(
       let lastError: string | null = null
 
       try {
-        const tools = await mcpService.discoverServerTools(userId, serverId, workspaceId, true) // Force refresh
+        const tools = await mcpService.discoverServerTools(userId, serverId, workspaceId)
         connectionStatus = 'connected'
         toolCount = tools.length
         logger.info(

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

@@ -27,7 +27,7 @@ export const GET = withMcpAuth('read')(
 
       let tools
       if (serverId) {
-        tools = await mcpService.discoverServerTools(userId, serverId, workspaceId, forceRefresh)
+        tools = await mcpService.discoverServerTools(userId, serverId, workspaceId)
       } else {
         tools = await mcpService.discoverTools(userId, workspaceId, forceRefresh)
       }
@@ -79,7 +79,7 @@ export const POST = withMcpAuth('read')(
 
       const results = await Promise.allSettled(
         serverIds.map(async (serverId: string) => {
-          const tools = await mcpService.discoverServerTools(userId, serverId, workspaceId, true)
+          const tools = await mcpService.discoverServerTools(userId, serverId, workspaceId)
           return { serverId, toolCount: tools.length }
         })
       )

apps/sim/app/api/mcp/tools/execute/route.ts

@@ -54,7 +54,7 @@ export const POST = withMcpAuth('read')(
 
       let tool = null
       try {
-        const tools = await mcpService.discoverServerTools(userId, serverId, workspaceId, false) // Use cache
+        const tools = await mcpService.discoverServerTools(userId, serverId, workspaceId)
         tool = tools.find((t) => t.name === toolName)
 
         if (!tool) {

apps/sim/app/api/tools/google_calendar/calendars/route.ts

@@ -21,7 +21,7 @@ interface CalendarListItem {
  * Get calendars from Google Calendar
  */
 export async function GET(request: NextRequest) {
-  const requestId = generateRequestId() // Generate a short request ID for correlation
+  const requestId = generateRequestId()
   logger.info(`[${requestId}] Google Calendar calendars request received`)
 
   try {

apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/workflow-block/components/sub-block/components/mcp-dynamic-args/mcp-dynamic-args.tsx

@@ -32,7 +32,15 @@ export function McpDynamicArgs({
 
   const currentArgs = useCallback(() => {
     if (isPreview && previewValue) {
-      return typeof previewValue === 'string' ? JSON.parse(previewValue) : previewValue
+      if (typeof previewValue === 'string') {
+        try {
+          return JSON.parse(previewValue)
+        } catch (error) {
+          console.warn('Failed to parse preview value as JSON:', error)
+          return previewValue
+        }
+      }
+      return previewValue
     }
     if (typeof toolArgs === 'string') {
       try {

apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/workflow-block/components/sub-block/components/mcp-server-modal/mcp-tool-selector.tsx

@@ -13,13 +13,10 @@ import {
   CommandList,
 } from '@/components/ui/command'
 import { Popover, PopoverContent, PopoverTrigger } from '@/components/ui/popover'
-import { createLogger } from '@/lib/logs/console/logger'
 import { useSubBlockValue } from '@/app/workspace/[workspaceId]/w/[workflowId]/components/workflow-block/components/sub-block/hooks/use-sub-block-value'
 import type { SubBlockConfig } from '@/blocks/types'
 import { useMcpTools } from '@/hooks/use-mcp-tools'
 
-const logger = createLogger('McpToolSelector')
-
 interface McpToolSelectorProps {
   blockId: string
   subBlock: SubBlockConfig

apps/sim/app/workspace/[workspaceId]/w/components/sidebar/components/settings-modal/components/mcp/mcp.tsx

@@ -743,7 +743,7 @@ export function MCP() {
                             <SelectValue />
                           </SelectTrigger>
                           <SelectContent>
-                            <SelectItem value='streamable-http'>Streamable HTTP </SelectItem>
+                            <SelectItem value='streamable-http'>Streamable HTTP</SelectItem>
                             <SelectItem value='http'>HTTP</SelectItem>
                             <SelectItem value='sse'>Server-Sent Events</SelectItem>
                           </SelectContent>

…im/db/migrations/0087_brief_gargoyle.sql …im/db/migrations/0087_wealthy_landau.sqlapps/sim/db/migrations/0087_brief_gargoyle.sql renamed to apps/sim/db/migrations/0087_wealthy_landau.sql apps/sim/db/migrations/0087_brief_gargoyle.sql renamed to apps/sim/db/migrations/0087_wealthy_landau.sql

@@ -1,5 +1,5 @@
 {
-  "id": "abef926f-8dce-4a71-93f9-16a87f3020f9",
+  "id": "2091645c-e3df-44b6-9c7f-8e066741b51e",
   "prevId": "ef439418-5e82-4e59-9013-8b116258d511",
   "version": "7",
   "dialect": "postgresql",

apps/sim/db/migrations/meta/0087_snapshot.json

@@ -607,8 +607,8 @@
     {
       "idx": 87,
       "version": "7",
-      "when": 1757444334787,
-      "tag": "0087_brief_gargoyle",
+      "when": 1757448419436,
+      "tag": "0087_wealthy_landau",
       "breakpoints": true
     }
   ]