chore(mcp): drop redundant useCallback from startOauthForServer

The reference is only consumed by inline arrow handlers and is not
observed by any memoized child or effect dep array, so useCallback adds
overhead with no benefit.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/settings/components/mcp/mcp.tsx

@@ -225,41 +225,37 @@ export function MCP({ initialServerId }: MCPProps) {
   >({})
   const [expandedTools, setExpandedTools] = useState<Set<string>>(() => new Set())
 
-  const startOauthForServer = useCallback(
-    async (serverId: string) => {
-      setConnectingOauthServers((prev) => new Set(prev).add(serverId))
-      const clear = () => {
-        const existing = oauthPopupIntervalsRef.current.get(serverId)
-        if (existing !== undefined) {
-          window.clearInterval(existing)
-          oauthPopupIntervalsRef.current.delete(serverId)
-        }
-        setConnectingOauthServers((prev) => {
-          const next = new Set(prev)
-          next.delete(serverId)
-          return next
-        })
+  const startOauthForServer = async (serverId: string) => {
+    setConnectingOauthServers((prev) => new Set(prev).add(serverId))
+    const clear = () => {
+      const existing = oauthPopupIntervalsRef.current.get(serverId)
+      if (existing !== undefined) {
+        window.clearInterval(existing)
+        oauthPopupIntervalsRef.current.delete(serverId)
       }
-      try {
-        const result = await startOauthMutation.mutateAsync({ serverId, workspaceId })
-        if (result.status === 'already_authorized') {
-          clear()
-          return
-        }
-        const { popup } = result
-        const interval = window.setInterval(() => {
-          if (popup.closed) clear()
-        }, 500)
-        oauthPopupIntervalsRef.current.set(serverId, interval)
-      } catch (e) {
+      setConnectingOauthServers((prev) => {
+        const next = new Set(prev)
+        next.delete(serverId)
+        return next
+      })
+    }
+    try {
+      const result = await startOauthMutation.mutateAsync({ serverId, workspaceId })
+      if (result.status === 'already_authorized') {
         clear()
-        logger.error('Failed to start MCP OAuth', e)
-        toast.error(toError(e).message || 'Failed to start authorization')
+        return
       }
-    },
-    // eslint-disable-next-line react-hooks/exhaustive-deps -- mutateAsync is stable
-    [workspaceId]
-  )
+      const { popup } = result
+      const interval = window.setInterval(() => {
+        if (popup.closed) clear()
+      }, 500)
+      oauthPopupIntervalsRef.current.set(serverId, interval)
+    } catch (e) {
+      clear()
+      logger.error('Failed to start MCP OAuth', e)
+      toast.error(toError(e).message || 'Failed to start authorization')
+    }
+  }
 
   const handleRemoveServer = useCallback((serverId: string, serverName: string) => {
     setServerToDelete({ id: serverId, name: serverName })