updated colors, error throwing in mcp modal

Author: waleedlatif1

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

@@ -288,8 +288,9 @@ export function McpDeploy({
     onSubmittingChange?.(true)
     setSaveErrors([])
     try {
-      const nextServerToolsMap = { ...serverToolsMap }
       const errors: string[] = []
+      const addedEntries: Record<string, { tool: WorkflowMcpTool; isLoading: boolean }> = {}
+      const removedIds: string[] = []
 
       for (const serverId of toAdd) {
         setPendingServerChanges((prev) => new Set(prev).add(serverId))
@@ -302,7 +303,7 @@ export function McpDeploy({
             toolDescription: toolDescription.trim() || undefined,
             parameterSchema,
           })
-          nextServerToolsMap[serverId] = { tool: addedTool, isLoading: false }
+          addedEntries[serverId] = { tool: addedTool, isLoading: false }
           onAddedToServer?.()
           logger.info(`Added workflow ${workflowId} as tool to server ${serverId}`)
         } catch (error) {
@@ -319,7 +320,7 @@ export function McpDeploy({
       }
 
       for (const serverId of toRemove) {
-        const toolInfo = nextServerToolsMap[serverId]
+        const toolInfo = serverToolsMap[serverId]
         if (!toolInfo?.tool) continue
 
         setPendingServerChanges((prev) => new Set(prev).add(serverId))
@@ -329,7 +330,7 @@ export function McpDeploy({
             serverId,
             toolId: toolInfo.tool.id,
           })
-          delete nextServerToolsMap[serverId]
+          removedIds.push(serverId)
         } catch (error) {
           const serverName = servers.find((s) => s.id === serverId)?.name || serverId
           errors.push(`Failed to remove from ${serverName}`)
@@ -346,7 +347,7 @@ export function McpDeploy({
       if (shouldUpdateExisting) {
         for (const serverId of selectedServerIdsForForm) {
           if (toAdd.has(serverId)) continue
-          const toolInfo = nextServerToolsMap[serverId]
+          const toolInfo = serverToolsMap[serverId]
           if (!toolInfo?.tool) continue
 
           try {
@@ -366,7 +367,13 @@ export function McpDeploy({
         }
       }
 
-      setServerToolsMap(nextServerToolsMap)
+      setServerToolsMap((prev) => {
+        const next = { ...prev, ...addedEntries }
+        for (const id of removedIds) {
+          delete next[id]
+        }
+        return next
+      })
       if (errors.length > 0) {
         setSaveErrors(errors)
       } else {

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

@@ -434,7 +434,7 @@ export function TeamManagement() {
 
         {/* Team Information */}
         <details className='group overflow-hidden rounded-[6px] border border-[var(--border-1)] bg-[var(--surface-5)]'>
-          <summary className='flex cursor-pointer items-center justify-between px-[14px] py-[10px] font-medium text-[14px] text-[var(--text-primary)] transition-colors hover:bg-white/[0.04] group-open:rounded-b-none'>
+          <summary className='flex cursor-pointer items-center justify-between px-[14px] py-[10px] font-medium text-[14px] text-[var(--text-primary)] hover:bg-[var(--surface-4)] group-open:rounded-b-none'>
             <span>Team Information</span>
             <svg
               className='h-4 w-4 transition-transform group-open:rotate-180'
@@ -473,7 +473,7 @@ export function TeamManagement() {
         {/* Team Billing Information (only show for Team Plan, not Enterprise) */}
         {hasTeamPlan && !hasEnterprisePlan && (
           <details className='group overflow-hidden rounded-[6px] border border-[var(--border-1)] bg-[var(--surface-5)]'>
-            <summary className='flex cursor-pointer items-center justify-between px-[14px] py-[10px] font-medium text-[14px] text-[var(--text-primary)] transition-colors hover:bg-white/[0.04] group-open:rounded-b-none'>
+            <summary className='flex cursor-pointer items-center justify-between px-[14px] py-[10px] font-medium text-[14px] text-[var(--text-primary)] hover:bg-[var(--surface-4)] group-open:rounded-b-none'>
               <span>Billing Information</span>
               <svg
                 className='h-4 w-4 transition-transform group-open:rotate-180'