Skip to content

Commit 26c1362

Browse files
committed
use reactquery
1 parent c45759e commit 26c1362

File tree

3 files changed

+2
-18
lines changed
  • apps/sim/app
    • api/mcp/workflow-servers
    • workspace/[workspaceId]/w
      • [workflowId]/components/panel/components/deploy/components/deploy-modal/components/mcp
      • components/sidebar/components/settings-modal/components/workflow-mcp-servers

3 files changed

+2
-18
lines changed

apps/sim/app/api/mcp/workflow-servers/route.ts

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -131,12 +131,10 @@ export const POST = withMcpAuth('write')(
131131
})
132132
.returning()
133133

134-
// If workflowIds are provided, create tools for each workflow
135134
const workflowIds: string[] = body.workflowIds || []
136135
const addedTools: Array<{ workflowId: string; toolName: string }> = []
137136

138137
if (workflowIds.length > 0) {
139-
// Fetch all workflows in one query
140138
const workflows = await db
141139
.select({
142140
id: workflow.id,
@@ -148,23 +146,19 @@ export const POST = withMcpAuth('write')(
148146
.from(workflow)
149147
.where(inArray(workflow.id, workflowIds))
150148

151-
// Create tools for each valid workflow
152149
for (const workflowRecord of workflows) {
153-
// Skip if workflow doesn't belong to this workspace
154150
if (workflowRecord.workspaceId !== workspaceId) {
155151
logger.warn(
156152
`[${requestId}] Skipping workflow ${workflowRecord.id} - does not belong to workspace`
157153
)
158154
continue
159155
}
160156

161-
// Skip if workflow is not deployed
162157
if (!workflowRecord.isDeployed) {
163158
logger.warn(`[${requestId}] Skipping workflow ${workflowRecord.id} - not deployed`)
164159
continue
165160
}
166161

167-
// Skip if workflow doesn't have a start block
168162
const hasStartBlock = await hasValidStartBlock(workflowRecord.id)
169163
if (!hasStartBlock) {
170164
logger.warn(`[${requestId}] Skipping workflow ${workflowRecord.id} - no start block`)

apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/deploy/components/deploy-modal/components/mcp/mcp.tsx

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -98,11 +98,7 @@ export function McpDeploy({
9898
const workspaceId = params.workspaceId as string
9999
const openSettingsModal = useSettingsModalStore((state) => state.openModal)
100100

101-
const {
102-
data: servers = [],
103-
isLoading: isLoadingServers,
104-
refetch: refetchServers,
105-
} = useWorkflowMcpServers(workspaceId)
101+
const { data: servers = [], isLoading: isLoadingServers } = useWorkflowMcpServers(workspaceId)
106102
const addToolMutation = useAddWorkflowMcpTool()
107103
const deleteToolMutation = useDeleteWorkflowMcpTool()
108104
const updateToolMutation = useUpdateWorkflowMcpTool()
@@ -346,7 +342,6 @@ export function McpDeploy({
346342
toolDescription: toolDescription.trim() || undefined,
347343
parameterSchema,
348344
})
349-
refetchServers()
350345
onAddedToServer?.()
351346
logger.info(`Added workflow ${workflowId} as tool to server ${serverId}`)
352347
} catch (error) {
@@ -375,7 +370,6 @@ export function McpDeploy({
375370
delete next[serverId]
376371
return next
377372
})
378-
refetchServers()
379373
} catch (error) {
380374
logger.error('Failed to remove tool:', error)
381375
} finally {
@@ -398,7 +392,6 @@ export function McpDeploy({
398392
parameterSchema,
399393
addToolMutation,
400394
deleteToolMutation,
401-
refetchServers,
402395
onAddedToServer,
403396
]
404397
)

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

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ interface ServerDetailViewProps {
5959
type McpClientType = 'cursor' | 'claude-code' | 'claude-desktop' | 'vscode'
6060

6161
function ServerDetailView({ workspaceId, serverId, onBack }: ServerDetailViewProps) {
62-
const { data, isLoading, error, refetch } = useWorkflowMcpServer(workspaceId, serverId)
62+
const { data, isLoading, error } = useWorkflowMcpServer(workspaceId, serverId)
6363
const { data: deployedWorkflows = [], isLoading: isLoadingWorkflows } =
6464
useDeployedWorkflows(workspaceId)
6565
const deleteToolMutation = useDeleteWorkflowMcpTool()
@@ -146,7 +146,6 @@ function ServerDetailView({ workspaceId, serverId, onBack }: ServerDetailViewPro
146146
setShowAddWorkflow(false)
147147
setSelectedWorkflowId(null)
148148
setActiveServerTab('workflows')
149-
refetch()
150149
} catch (err) {
151150
logger.error('Failed to add workflow:', err)
152151
}
@@ -253,7 +252,6 @@ function ServerDetailView({ workspaceId, serverId, onBack }: ServerDetailViewPro
253252
isPublic: editServerIsPublic,
254253
})
255254
setShowEditServer(false)
256-
refetch()
257255
} catch (err) {
258256
logger.error('Failed to update server:', err)
259257
}
@@ -676,7 +674,6 @@ function ServerDetailView({ workspaceId, serverId, onBack }: ServerDetailViewPro
676674
toolDescription: editingDescription.trim() || undefined,
677675
parameterSchema: updatedSchema,
678676
})
679-
refetch()
680677
setToolToView(null)
681678
setEditingDescription('')
682679
setEditingParameterDescriptions({})

0 commit comments

Comments
 (0)