From 187726ae8d8ad8aa9e00bf2ae8dab195bee1ce66 Mon Sep 17 00:00:00 2001
From: kangfenmao <kangfenmao@qq.com>
Date: Mon, 17 Mar 2025 13:07:05 +0800
Subject: [PATCH] feat: enhance MCPSettings with server table and clean up
 NpxSearch component

- Added a debug log for MCP servers in MCPSettings.
- Refactored the MCPSettings component to streamline the server table rendering.
- Removed unnecessary styles from the NpxSearch component for cleaner layout.
---
 .../pages/settings/MCPSettings/NpxSearch.tsx  |  2 +-
 .../src/pages/settings/MCPSettings/index.tsx  | 30 ++++++++-----------
 2 files changed, 14 insertions(+), 18 deletions(-)

diff --git a/src/renderer/src/pages/settings/MCPSettings/NpxSearch.tsx b/src/renderer/src/pages/settings/MCPSettings/NpxSearch.tsx
index 752a90ce..c751ebc8 100644
--- a/src/renderer/src/pages/settings/MCPSettings/NpxSearch.tsx
+++ b/src/renderer/src/pages/settings/MCPSettings/NpxSearch.tsx
@@ -67,7 +67,7 @@ const NpxSearch: FC = () => {
   }
 
   return (
-    <SettingGroup theme={theme} style={{ marginTop: 15 }}>
+    <SettingGroup theme={theme}>
       <SettingTitle>{t('settings.mcp.npx_list.title')}</SettingTitle>
       <SettingDivider />
       <Paragraph type="secondary" style={{ margin: '0 0 10px 0' }}>
diff --git a/src/renderer/src/pages/settings/MCPSettings/index.tsx b/src/renderer/src/pages/settings/MCPSettings/index.tsx
index d74e54c8..a5d6d7be 100644
--- a/src/renderer/src/pages/settings/MCPSettings/index.tsx
+++ b/src/renderer/src/pages/settings/MCPSettings/index.tsx
@@ -19,6 +19,8 @@ const MCPSettings: FC = () => {
   const { Paragraph, Text } = Typography
   const mcpServers = useAppSelector((state) => state.mcp.servers)
 
+  console.debug(mcpServers)
+
   const handleDelete = (serverName: string) => {
     window.modal.confirm({
       title: t('settings.mcp.confirmDelete'),
@@ -145,24 +147,18 @@ const MCPSettings: FC = () => {
             {t('settings.mcp.editJson')}
           </Button>
         </HStack>
+        <Table
+          dataSource={mcpServers}
+          columns={columns}
+          rowKey="name"
+          pagination={false}
+          size="small"
+          locale={{ emptyText: t('settings.mcp.noServers') }}
+          rowClassName={(record) => (!record.isActive ? 'inactive-row' : '')}
+          onRow={(record) => ({ style: !record.isActive ? inactiveRowStyle : {} })}
+          style={{ marginTop: 15 }}
+        />
       </SettingGroup>
-      <Table
-        dataSource={mcpServers}
-        columns={columns}
-        rowKey="name"
-        pagination={false}
-        size="small"
-        locale={{ emptyText: t('settings.mcp.noServers') }}
-        rowClassName={(record) => (!record.isActive ? 'inactive-row' : '')}
-        onRow={(record) => ({
-          style: !record.isActive ? inactiveRowStyle : {}
-        })}
-        style={{
-          borderRadius: '8px',
-          overflow: 'hidden',
-          border: '0.5px solid var(--color-border)'
-        }}
-      />
       <NpxSearch />
     </SettingContainer>
   )