autoscroll servers

python/api/mcp_servers_apply.py

@@ -15,10 +15,11 @@ class McpServersApply(ApiHandler):
             # MCPConfig.update(mcp_servers) # done in settings automatically
             set_settings_delta({"mcp_servers": "[]"}) # to force reinitialization
             set_settings_delta({"mcp_servers": mcp_servers})
-            
+
             time.sleep(1) # wait at least a second
-            MCPConfig.wait_for_lock() # wait until config lock is released
+            # MCPConfig.wait_for_lock() # wait until config lock is released
+            status = MCPConfig.get_instance().get_servers_status()
+            return {"success": True, "status": status}
 
         except Exception as e:
             return {"success": False, "error": str(e)}
-        return {"success": True}

webui/components/settings/mcp/client/mcp-servers-store.js

@@ -72,7 +72,7 @@ const model = {
 
   async startStatusCheck() {
     this.statusCheck = true;
-    const firstLoad = true;
+    let firstLoad = true;
 
     while (this.statusCheck) {
       await this._statusCheck();
@@ -101,11 +101,15 @@ const model = {
     this.loading = true;
     try {
       scrollModal("mcp-servers-status");
-      await API.callJsonApi("mcp_servers_apply", {
+      const resp = await API.callJsonApi("mcp_servers_apply", {
         mcp_servers: this.getEditorValue(),
       });
-      // await sleep(5000); // just to prevent user from clicking apply multiple times
-      // api now waits for config locks automatically
+      if (resp.success) {
+        this.servers = resp.status;
+        this.servers.sort((a, b) => a.name.localeCompare(b.name));
+      }
+      this.loading = false;
+      await sleep(100); // wait for ui and scroll
       scrollModal("mcp-servers-status");
     } catch (error) {
       console.error("Failed to apply MCP servers:", error);