Skip to content

Commit a16355e

Browse files
authored
fix(optimizer): hide advance if no group is selected (#1177)
1 parent 129ffb3 commit a16355e

File tree

2 files changed

+30
-2
lines changed

2 files changed

+30
-2
lines changed

renderer/src/routes/__tests__/mcp-optimizer.test.tsx

Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,19 @@ it('renders the MCP Optimizer page title', async () => {
4646
})
4747

4848
it('renders the Advanced dropdown menu button', async () => {
49+
server.use(
50+
http.get(mswEndpoint('/api/v1beta/workloads/:name'), ({ params }) => {
51+
if (params.name === META_MCP_SERVER_NAME) {
52+
return HttpResponse.json({
53+
name: META_MCP_SERVER_NAME,
54+
group: MCP_OPTIMIZER_GROUP_NAME,
55+
env_vars: { ALLOWED_GROUPS: 'default' },
56+
})
57+
}
58+
return HttpResponse.json(null, { status: 404 })
59+
})
60+
)
61+
4962
renderRoute(router)
5063

5164
await waitFor(() => {
@@ -365,6 +378,19 @@ it('refetches the selected group when navigating back to the page', async () =>
365378
})
366379

367380
it('clicking Meta-MCP logs in Advanced menu navigates to logs page', async () => {
381+
server.use(
382+
http.get(mswEndpoint('/api/v1beta/workloads/:name'), ({ params }) => {
383+
if (params.name === META_MCP_SERVER_NAME) {
384+
return HttpResponse.json({
385+
name: META_MCP_SERVER_NAME,
386+
group: MCP_OPTIMIZER_GROUP_NAME,
387+
env_vars: { ALLOWED_GROUPS: 'default' },
388+
})
389+
}
390+
return HttpResponse.json(null, { status: 404 })
391+
})
392+
)
393+
368394
const user = userEvent.setup()
369395
renderRoute(router)
370396

renderer/src/routes/mcp-optimizer.tsx

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ import { EditServerDialogProvider } from '@/features/mcp-servers/contexts/edit-s
2020
import { useEditServerDialog } from '@/features/mcp-servers/hooks/use-edit-server-dialog'
2121
import { WrapperDialogFormMcp } from '@/features/mcp-servers/components/wrapper-dialog-mcp'
2222
import { LinkViewTransition } from '@/common/components/link-view-transition'
23+
import { useOptimizedGroupName } from '@/common/hooks/use-optimize-group-name'
2324

2425
export const Route = createFileRoute('/mcp-optimizer')({
2526
component: McpOptimizerRoute,
@@ -28,6 +29,7 @@ export const Route = createFileRoute('/mcp-optimizer')({
2829
function McpOptimizerContent() {
2930
const groups = useMcpOptimizerGroups()
3031
const { state, openDialog, closeDialog } = useEditServerDialog()
32+
const optimizedGroupName = useOptimizedGroupName()
3133

3234
const handleCustomizeConfiguration = () => {
3335
const isRemote = false
@@ -39,7 +41,7 @@ function McpOptimizerContent() {
3941
<McpServersSidebar />
4042
<div className={'ml-sidebar min-w-0 flex-1'}>
4143
<TitlePage title="MCP Optimizer">
42-
<>
44+
{optimizedGroupName ? (
4345
<div className="flex gap-2 lg:ml-auto">
4446
<DropdownMenu>
4547
<DropdownMenuTrigger asChild>
@@ -74,7 +76,7 @@ function McpOptimizerContent() {
7476
</DropdownMenuContent>
7577
</DropdownMenu>
7678
</div>
77-
</>
79+
) : null}
7880
</TitlePage>
7981
<div className="p-6">
8082
<div className="mx-auto max-w-2xl">

0 commit comments

Comments
 (0)