diff --git a/web/src/app/home/mcp/MCPDetailContent.tsx b/web/src/app/home/mcp/MCPDetailContent.tsx index 7868bbe5..90550ea2 100644 --- a/web/src/app/home/mcp/MCPDetailContent.tsx +++ b/web/src/app/home/mcp/MCPDetailContent.tsx @@ -1,6 +1,6 @@ 'use client'; -import { useState, useEffect, useCallback } from 'react'; +import { useState, useEffect, useCallback, useRef } from 'react'; import { useRouter } from 'next/navigation'; import { Button } from '@/components/ui/button'; import { Switch } from '@/components/ui/switch'; @@ -21,6 +21,7 @@ import { DialogFooter, } from '@/components/ui/dialog'; import MCPForm from '@/app/home/mcp/components/mcp-form/MCPForm'; +import type { MCPFormHandle } from '@/app/home/mcp/components/mcp-form/MCPForm'; import { httpClient, systemInfo } from '@/app/infra/http/HttpClient'; import { useSidebarData } from '@/app/home/components/home-sidebar/SidebarDataContext'; import { useTranslation } from 'react-i18next'; @@ -50,6 +51,10 @@ export default function MCPDetailContent({ id }: { id: string }) { // Track whether the form has unsaved changes const [formDirty, setFormDirty] = useState(false); + // Ref to MCPForm for triggering test from header + const formRef = useRef(null); + const [mcpTesting, setMcpTesting] = useState(false); + // Enable state managed here so the header switch works const [serverEnabled, setServerEnabled] = useState(true); const [enableLoaded, setEnableLoaded] = useState(false); @@ -142,26 +147,38 @@ export default function MCPDetailContent({ id }: { id: string }) { {/* Header */}

{t('mcp.createServer')}

- +
+ + +
{/* Content */}
@@ -193,19 +210,31 @@ export default function MCPDetailContent({ id }: { id: string }) { )} - +
+ + +
{/* Content */}
{/* Card: Danger Zone */} diff --git a/web/src/app/home/mcp/components/mcp-form/MCPForm.tsx b/web/src/app/home/mcp/components/mcp-form/MCPForm.tsx index fdf71c9c..615b3156 100644 --- a/web/src/app/home/mcp/components/mcp-form/MCPForm.tsx +++ b/web/src/app/home/mcp/components/mcp-form/MCPForm.tsx @@ -1,6 +1,12 @@ 'use client'; -import React, { useState, useEffect, useRef } from 'react'; +import React, { + useState, + useEffect, + useRef, + forwardRef, + useImperativeHandle, +} from 'react'; import { useTranslation } from 'react-i18next'; import { Resolver, useForm } from 'react-hook-form'; import { zodResolver } from '@hookform/resolvers/zod'; @@ -215,14 +221,25 @@ interface MCPFormProps { onFormSubmit: () => void; onNewServerCreated: (serverName: string) => void; onDirtyChange?: (dirty: boolean) => void; + onTestingChange?: (testing: boolean) => void; } -export default function MCPForm({ - initServerName, - onFormSubmit, - onNewServerCreated, - onDirtyChange, -}: MCPFormProps) { +// Handle exposed to parent via ref +export interface MCPFormHandle { + testMcp: () => void; + isTesting: boolean; +} + +const MCPForm = forwardRef(function MCPForm( + { + initServerName, + onFormSubmit, + onNewServerCreated, + onDirtyChange, + onTestingChange, + }, + ref, +) { const { t } = useTranslation(); const formSchema = getFormSchema(t); const isEditMode = !!initServerName; @@ -262,6 +279,21 @@ export default function MCPForm({ onDirtyChange?.(isDirty); }, [isDirty, onDirtyChange]); + // Notify parent when testing state changes + useEffect(() => { + onTestingChange?.(mcpTesting); + }, [mcpTesting, onTestingChange]); + + // Expose test action and testing state to parent + useImperativeHandle( + ref, + () => ({ + testMcp: () => testMcp(), + isTesting: mcpTesting, + }), + [mcpTesting], + ); + // Load server data useEffect(() => { isInitializing.current = true; @@ -647,15 +679,6 @@ export default function MCPForm({ )} - - )} @@ -895,19 +918,9 @@ export default function MCPForm({ - - {/* Test button (create mode only, edit mode has it in the status card) */} - {!isEditMode && ( - - )} ); -} +}); + +export default MCPForm;