refactor(locks): drop redundant ensureWorkflowAccess fetch in rename

waleedlatif1 · waleedlatif1 · commit 84affcf3bb56 · 2026-06-16T20:22:39.000-07:00
diff --git a/apps/sim/lib/copilot/tools/handlers/workflow/mutations.ts b/apps/sim/lib/copilot/tools/handlers/workflow/mutations.ts
@@ -637,10 +637,9 @@ export async function executeRenameWorkflow(
       return { success: false, error: 'Workflow name must be 200 characters or less' }
     }
 
-    await ensureWorkflowAccess(workflowId, context.userId, 'write')
+    const current = await ensureWorkflowAccess(workflowId, context.userId, 'write')
     await assertWorkflowMutable(workflowId)
     assertWorkflowMutationNotAborted(context)
-    const current = await ensureWorkflowAccess(workflowId, context.userId, 'write')
     if (!current.workspaceId) {
       return { success: false, error: 'Workflow workspace is required' }
     }

Original file line number	Diff line number	Diff line change
`@@ -637,10 +637,9 @@ export async function executeRenameWorkflow(`
`637`	`637`	`return { success: false, error: 'Workflow name must be 200 characters or less' }`
`638`	`638`	`}`
`639`	`639`
`640`		`- await ensureWorkflowAccess(workflowId, context.userId, 'write')`
	`640`	`+ const current = await ensureWorkflowAccess(workflowId, context.userId, 'write')`
`641`	`641`	`await assertWorkflowMutable(workflowId)`
`642`	`642`	`assertWorkflowMutationNotAborted(context)`
`643`		`- const current = await ensureWorkflowAccess(workflowId, context.userId, 'write')`
`644`	`643`	`if (!current.workspaceId) {`
`645`	`644`	`return { success: false, error: 'Workflow workspace is required' }`
`646`	`645`	`}`