From 4e31e1d3a8831844bf54b2f129ea67ca6d80feca Mon Sep 17 00:00:00 2001
From: Chaiyapruek Muangsiri <cmp.poon@gmail.com>
Date: Tue, 5 Aug 2025 07:31:11 +0800
Subject: [PATCH] remove unnecessary try catch block

---
 web-app/src/routes/settings/mcp-servers.tsx | 14 +++-----------
 1 file changed, 3 insertions(+), 11 deletions(-)

diff --git a/web-app/src/routes/settings/mcp-servers.tsx b/web-app/src/routes/settings/mcp-servers.tsx
index 995070d02..2cbf53015 100644
--- a/web-app/src/routes/settings/mcp-servers.tsx
+++ b/web-app/src/routes/settings/mcp-servers.tsx
@@ -83,11 +83,7 @@ function MCPServers() {
   }
 
   const handleSaveServer = async (name: string, config: MCPServerConfig) => {
-    try {
-      toggleServer(name, false)
-    } catch (error) {
-      console.error('Error deactivating server:', error)
-    }
+    toggleServer(name, false)
     if (editingKey) {
       // If server name changed, delete old one and add new one
       if (editingKey !== name) {
@@ -158,12 +154,8 @@ function MCPServers() {
       // Save all servers
       // Clear existing servers first
       Object.keys(mcpServers).forEach((serverKey) => {
-        try {
-          toggleServer(serverKey, false)
-          deleteServer(serverKey)
-        } catch (error) {
-          console.error('Error deactivating server:', error)
-        }
+        toggleServer(serverKey, false)
+        deleteServer(serverKey)
       })
 
       // Add all servers from the JSON