Skip to content

Commit 93c19bb

Browse files
authored
feat: hide "filesystem" MCP temporarily (#487)
* feat: temporarily remove the filesystem server * refactor test
1 parent 6a1d4fb commit 93c19bb

File tree

2 files changed

+30
-11
lines changed

2 files changed

+30
-11
lines changed

renderer/src/features/registry-servers/components/grid-cards-registry-server.tsx

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import type { RegistryImageMetadata } from '@/common/api/generated/types.gen'
22
import { CardRegistryServer } from './card-registry-server'
33
import { FormRunFromRegistry } from './form-run-from-registry'
4-
import { useState } from 'react'
4+
import { useState, useMemo } from 'react'
55
import { useFilterSort } from '@/common/hooks/use-filter-sort'
66
import { InputSearch } from '@/common/components/ui/input-search'
77

@@ -13,12 +13,18 @@ export function GridCardsRegistryServer({
1313
const [selectedServer, setSelectedServer] =
1414
useState<RegistryImageMetadata | null>(null)
1515
const [isModalOpen, setIsModalOpen] = useState(false)
16+
17+
// Filter out filesystem servers
18+
const filteredServers = useMemo(() => {
19+
return servers.filter((server) => server.name !== 'filesystem')
20+
}, [servers])
21+
1622
const {
1723
filter,
1824
setFilter,
19-
filteredData: filteredServers,
25+
filteredData: filteredAndSortedServers,
2026
} = useFilterSort({
21-
data: servers,
27+
data: filteredServers,
2228
filterFields: (server) => [server.name || '', server.description || ''],
2329
sortBy: (server) => server.name || '',
2430
})
@@ -36,15 +42,15 @@ export function GridCardsRegistryServer({
3642
placeholder="Search..."
3743
/>
3844
<div className="grid grid-cols-1 gap-4 md:grid-cols-2 lg:grid-cols-4">
39-
{filteredServers.map((server) => (
45+
{filteredAndSortedServers.map((server) => (
4046
<CardRegistryServer
4147
key={server.name}
4248
server={server}
4349
onClick={() => handleCardClick(server)}
4450
/>
4551
))}
4652
</div>
47-
{filteredServers.length === 0 && (
53+
{filteredAndSortedServers.length === 0 && (
4854
<div className="text-muted-foreground py-12 text-center">
4955
<p className="text-sm">
5056
No registry servers found matching the current filter

renderer/src/routes/__tests__/registry.test.tsx

Lines changed: 19 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -25,13 +25,26 @@ it('renders list of MCP servers', async () => {
2525

2626
renderRoute(router)
2727
await waitFor(() => {
28-
for (const mcpServer of MOCK_REGISTRY_RESPONSE) {
29-
expect(
30-
screen.queryByText(mcpServer.name),
31-
`Expected ${mcpServer.name} to be in the document`
32-
).toBeVisible()
33-
}
28+
expect(
29+
screen.queryByText('atlassian'),
30+
'Expected atlassian to be in the document'
31+
).toBeVisible()
3432
})
33+
34+
expect(
35+
screen.queryByText('mongodb'),
36+
'Expected mongodb to be in the document'
37+
).toBeVisible()
38+
expect(
39+
screen.queryByText('redis'),
40+
'Expected redis to be in the document'
41+
).toBeVisible()
42+
43+
// Filesystem was temporarily hidden
44+
expect(
45+
screen.queryByText('filesystem'),
46+
'Expected filesystem to NOT be in the document'
47+
).not.toBeInTheDocument()
3548
})
3649

3750
const REGISTRY_SERVER = {

0 commit comments

Comments
 (0)