@nextclaw/ui 0.12.10 → 0.12.11

package/src/components/chat/ncp/tests/ncp-chat-input.manager.test.ts

@@ -4,6 +4,7 @@ import { NcpChatInputManager } from '@/components/chat/ncp/ncp-chat-input.manage
 import { useChatInputStore } from '@/components/chat/stores/chat-input.store';
 import { useChatSessionListStore } from '@/components/chat/stores/chat-session-list.store';
 import { useChatThreadStore } from '@/components/chat/stores/chat-thread.store';
+import { useSystemStatusStore } from '@/features/system-status';
 
 describe('NcpChatInputManager', () => {
   beforeEach(() => {
@@ -15,8 +16,23 @@ describe('NcpChatInputManager', () => {
         attachments: [],
         selectedSkills: [],
         selectedSessionType: 'native',
-        selectedModel: '',
+        selectedModel: 'gpt-5',
         selectedThinkingLevel: null,
+        isProviderStateResolved: true,
+        modelOptions: [
+          {
+            value: 'gpt-5',
+            modelLabel: 'GPT-5',
+            providerLabel: 'OpenAI',
+            thinkingCapability: null,
+          },
+        ],
+      },
+    });
+    useSystemStatusStore.setState({
+      state: {
+        ...useSystemStatusStore.getState().state,
+        lifecyclePhase: 'ready',
       },
     });
     useChatSessionListStore.setState({
@@ -96,4 +112,38 @@ describe('NcpChatInputManager', () => {
     );
     expect(sessionListManager.promoteRootDraftSessionRoute).toHaveBeenCalledWith('draft-root-session');
   });
+
+  it('does not send while the runtime is still blocked during startup', async () => {
+    useChatInputStore.setState({
+      snapshot: {
+        ...useChatInputStore.getState().snapshot,
+        isProviderStateResolved: false,
+        modelOptions: [],
+      },
+    });
+    useSystemStatusStore.setState({
+      state: {
+        ...useSystemStatusStore.getState().state,
+        lifecyclePhase: 'cold-starting',
+      },
+    });
+    const streamActionsManager = {
+      sendMessage: vi.fn().mockResolvedValue(undefined),
+      stopCurrentRun: vi.fn().mockResolvedValue(undefined),
+    } as unknown as ConstructorParameters<typeof NcpChatInputManager>[1];
+    const sessionListManager = {
+      ensureDraftSession: vi.fn(() => 'draft-session'),
+      promoteRootDraftSessionRoute: vi.fn(),
+    } as unknown as ConstructorParameters<typeof NcpChatInputManager>[2];
+    const manager = new NcpChatInputManager(
+      {} as ConstructorParameters<typeof NcpChatInputManager>[0],
+      streamActionsManager,
+      sessionListManager,
+    );
+
+    await manager.send();
+
+    expect(streamActionsManager.sendMessage).not.toHaveBeenCalled();
+    expect(sessionListManager.promoteRootDraftSessionRoute).not.toHaveBeenCalled();
+  });
 });

package/src/components/config/desktop-update-config.test.tsx

@@ -2,8 +2,8 @@ import { render, screen } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
 import { beforeEach, describe, expect, it, vi } from 'vitest';
 import { DesktopUpdateConfig } from '@/components/config/desktop-update-config';
-import { useDesktopUpdateStore } from '@/desktop/stores/desktop-update.store';
 import { setLanguage } from '@/lib/i18n';
+import { useDesktopUpdateStore } from '@/platforms/desktop';
 
 const mocks = vi.hoisted(() => ({
   start: vi.fn(),
@@ -15,9 +15,15 @@ const mocks = vi.hoisted(() => ({
   updateChannel: vi.fn()
 }));
 
-vi.mock('@/desktop/managers/desktop-update.manager', () => ({
-  desktopUpdateManager: mocks
-}));
+vi.mock('@/platforms/desktop', async () => {
+  const actual = await vi.importActual<typeof import('@/platforms/desktop')>(
+    '@/platforms/desktop'
+  );
+  return {
+    ...actual,
+    desktopUpdateManager: mocks,
+  };
+});
 
 describe('DesktopUpdateConfig', () => {
   beforeEach(() => {

package/src/components/config/desktop-update-config.tsx

@@ -5,11 +5,13 @@ import { Label } from '@/components/ui/label';
 import { Select, SelectContent, SelectItem, SelectTrigger, SelectValue } from '@/components/ui/select';
 import { Switch } from '@/components/ui/switch';
 import { PageHeader, PageLayout } from '@/components/layout/page-layout';
-import { desktopUpdateManager } from '@/desktop/managers/desktop-update.manager';
-import type { DesktopReleaseChannel } from '@/desktop/desktop-update.types';
-import { useDesktopUpdateStore } from '@/desktop/stores/desktop-update.store';
 import { formatDateTime, t } from '@/lib/i18n';
 import { cn } from '@/lib/utils';
+import {
+  desktopUpdateManager,
+  type DesktopReleaseChannel,
+  useDesktopUpdateStore,
+} from '@/platforms/desktop';
 import { Download, ExternalLink, RefreshCw, RotateCw } from 'lucide-react';
 
 function formatVersion(value: string | null): string {

package/src/components/config/runtime-control-card.test.tsx

@@ -1,17 +1,44 @@
-import { QueryClient, QueryClientProvider } from '@tanstack/react-query';
 import { render, screen, waitFor } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
-import type { ReactNode } from 'react';
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
 import { toast } from 'sonner';
 import { RuntimeControlCard } from '@/components/config/runtime-control-card';
 import { setLanguage } from '@/lib/i18n';
 
+const baseControlView = {
+  environment: 'managed-local-service' as const,
+  lifecycle: 'healthy' as const,
+  serviceState: 'running' as const,
+  message: 'runtime healthy',
+  pendingRestart: null,
+  canStartService: {
+    available: false,
+    requiresConfirmation: false,
+    impact: 'brief-ui-disconnect' as const,
+    reasonIfUnavailable: '当前页面已经由运行中的本地服务托管。',
+  },
+  canRestartService: {
+    available: true,
+    requiresConfirmation: false,
+    impact: 'brief-ui-disconnect' as const,
+  },
+  canStopService: {
+    available: true,
+    requiresConfirmation: true,
+    impact: 'brief-ui-disconnect' as const,
+  },
+  canRestartApp: {
+    available: false,
+    requiresConfirmation: true,
+    impact: 'full-app-relaunch' as const,
+    reasonIfUnavailable: 'desktop only',
+  },
+  managementHint: 'This page is served by the running local service.',
+};
+
 const mocks = vi.hoisted(() => ({
-  useRuntimeControl: vi.fn(),
-  useRuntimeServiceAction: vi.fn(),
-  waitForRecovery: vi.fn(),
-  restartApp: vi.fn(),
+  useRuntimeControlPanelView: vi.fn(),
+  runRuntimeControlAction: vi.fn(),
 }));
 
 vi.mock('sonner', () => ({
@@ -21,106 +48,28 @@ vi.mock('sonner', () => ({
   },
 }));
 
-vi.mock('@/hooks/use-runtime-control', () => ({
-  useRuntimeControl: (...args: unknown[]) => mocks.useRuntimeControl(...args),
-  useRuntimeServiceAction: (...args: unknown[]) => mocks.useRuntimeServiceAction(...args),
-}));
-
-vi.mock('@/runtime-control/runtime-control.manager', () => ({
-  runtimeControlManager: {
-    waitForRecovery: (...args: unknown[]) => mocks.waitForRecovery(...args),
-    restartApp: (...args: unknown[]) => mocks.restartApp(...args),
+vi.mock('@/features/system-status', () => ({
+  useRuntimeControlPanelView: (...args: unknown[]) =>
+    mocks.useRuntimeControlPanelView(...args),
+  systemStatusManager: {
+    runRuntimeControlAction: (...args: unknown[]) =>
+      mocks.runRuntimeControlAction(...args),
   },
 }));
 
-function createWrapper(queryClient: QueryClient) {
-  return function Wrapper({ children }: { children: ReactNode }) {
-    return <QueryClientProvider client={queryClient}>{children}</QueryClientProvider>;
-  };
-}
-
 describe('RuntimeControlCard', () => {
   beforeEach(() => {
     setLanguage('zh');
     vi.clearAllMocks();
-    mocks.useRuntimeControl.mockReturnValue({
-      data: {
-        environment: 'managed-local-service',
-        lifecycle: 'healthy',
-        serviceState: 'running',
-        message: 'runtime healthy',
-        pendingRestart: null,
-        canStartService: {
-          available: false,
-          requiresConfirmation: false,
-          impact: 'brief-ui-disconnect',
-          reasonIfUnavailable: '当前页面已经由运行中的本地服务托管。'
-        },
-        canRestartService: {
-          available: true,
-          requiresConfirmation: false,
-          impact: 'brief-ui-disconnect',
-        },
-        canStopService: {
-          available: true,
-          requiresConfirmation: true,
-          impact: 'brief-ui-disconnect',
-        },
-        canRestartApp: {
-          available: false,
-          requiresConfirmation: true,
-          impact: 'full-app-relaunch',
-          reasonIfUnavailable: 'desktop only',
-        },
-        managementHint: 'This page is served by the running local service.'
-      },
-      isError: false,
-      error: null,
-    });
-    mocks.useRuntimeServiceAction.mockReturnValue({
-      mutateAsync: vi.fn().mockResolvedValue({
-        accepted: true,
-        action: 'restart-service',
-        lifecycle: 'restarting-service',
-        message: 'Restart scheduled. This page may disconnect for a few seconds.',
-      }),
-      isPending: false,
-    });
-    mocks.waitForRecovery.mockResolvedValue({
-      environment: 'managed-local-service',
-      lifecycle: 'healthy',
-      serviceState: 'running',
-      message: 'runtime healthy',
+    mocks.useRuntimeControlPanelView.mockReturnValue({
+      controlView: baseControlView,
+      visibleLifecycle: 'healthy',
+      visibleServiceState: 'running',
+      visibleMessage: 'runtime healthy',
+      busyAction: null,
+      busy: false,
       pendingRestart: null,
-      canStartService: {
-        available: false,
-        requiresConfirmation: false,
-        impact: 'brief-ui-disconnect',
-        reasonIfUnavailable: '当前页面已经由运行中的本地服务托管。'
-      },
-      canRestartService: {
-        available: true,
-        requiresConfirmation: false,
-        impact: 'brief-ui-disconnect',
-      },
-      canStopService: {
-        available: true,
-        requiresConfirmation: true,
-        impact: 'brief-ui-disconnect',
-      },
-      canRestartApp: {
-        available: false,
-        requiresConfirmation: true,
-        impact: 'full-app-relaunch',
-        reasonIfUnavailable: 'desktop only',
-      },
-      managementHint: 'This page is served by the running local service.'
-    });
-    mocks.restartApp.mockResolvedValue({
-      accepted: true,
-      action: 'restart-app',
-      lifecycle: 'restarting-app',
-      message: 'NextClaw app restart scheduled.',
+      errorMessage: null,
     });
   });
 
@@ -129,14 +78,15 @@ describe('RuntimeControlCard', () => {
   });
 
   it('renders service management actions from the current capability view', () => {
-    const queryClient = new QueryClient();
+    render(<RuntimeControlCard />);
 
-    render(<RuntimeControlCard />, {
-      wrapper: createWrapper(queryClient),
-    });
+    const startButton = screen.getByRole('button', {
+      name: '启动服务',
+    }) as HTMLButtonElement;
+    const restartAppButton = screen.getByRole('button', {
+      name: '重启应用',
+    }) as HTMLButtonElement;
 
-    const startButton = screen.getByRole('button', { name: '启动服务' }) as HTMLButtonElement;
-    const restartAppButton = screen.getByRole('button', { name: '重启应用' }) as HTMLButtonElement;
     expect(screen.getByText('服务管理')).toBeTruthy();
     expect(screen.getByText('服务运行中')).toBeTruthy();
     expect(screen.getByRole('button', { name: '重启服务' })).toBeTruthy();
@@ -146,85 +96,61 @@ describe('RuntimeControlCard', () => {
     expect(screen.getByText('desktop only')).toBeTruthy();
   });
 
-  it('runs the restart service flow and waits for recovery', async () => {
-    const queryClient = new QueryClient();
+  it('runs the restart service flow through the system status manager', async () => {
     const user = userEvent.setup();
-    const invalidateQueriesSpy = vi.spyOn(queryClient, 'invalidateQueries');
-    const mutateAsync = vi.fn().mockResolvedValue({
+    mocks.runRuntimeControlAction.mockResolvedValue({
       accepted: true,
       action: 'restart-service',
       lifecycle: 'restarting-service',
       message: 'Restart scheduled. This page may disconnect for a few seconds.',
     });
-    mocks.useRuntimeServiceAction.mockReturnValue({
-      mutateAsync,
-      isPending: false,
-    });
 
-    render(<RuntimeControlCard />, {
-      wrapper: createWrapper(queryClient),
-    });
+    render(<RuntimeControlCard />);
 
     await user.click(screen.getByRole('button', { name: '重启服务' }));
 
     await waitFor(() => {
-      expect(mutateAsync).toHaveBeenCalledWith('restart-service');
-      expect(mocks.waitForRecovery).toHaveBeenCalledTimes(1);
+      expect(mocks.runRuntimeControlAction).toHaveBeenCalledWith(
+        'restart-service'
+      );
     });
-    expect(toast.success).toHaveBeenCalledWith('Restart scheduled. This page may disconnect for a few seconds.');
-    expect(invalidateQueriesSpy).toHaveBeenCalledWith({ queryKey: ['runtime-control'] });
+    expect(toast.success).toHaveBeenCalledWith(
+      'Restart scheduled. This page may disconnect for a few seconds.'
+    );
   });
 
   it('runs the stop service flow after confirmation', async () => {
-    const queryClient = new QueryClient();
     const user = userEvent.setup();
-    const mutateAsync = vi.fn().mockResolvedValue({
+    const confirmSpy = vi.spyOn(window, 'confirm').mockReturnValue(true);
+    mocks.runRuntimeControlAction.mockResolvedValue({
       accepted: true,
       action: 'stop-service',
       lifecycle: 'stopping-service',
       message: 'Stop scheduled. This page will disconnect shortly.',
     });
-    mocks.useRuntimeServiceAction.mockReturnValue({
-      mutateAsync,
-      isPending: false,
-    });
-    const confirmSpy = vi.spyOn(window, 'confirm').mockReturnValue(true);
 
-    render(<RuntimeControlCard />, {
-      wrapper: createWrapper(queryClient),
-    });
+    render(<RuntimeControlCard />);
 
     await user.click(screen.getByRole('button', { name: '停止服务' }));
 
     await waitFor(() => {
       expect(confirmSpy).toHaveBeenCalledTimes(1);
-      expect(mutateAsync).toHaveBeenCalledWith('stop-service');
+      expect(mocks.runRuntimeControlAction).toHaveBeenCalledWith(
+        'stop-service'
+      );
     });
-    expect(mocks.waitForRecovery).not.toHaveBeenCalled();
-    expect(toast.success).toHaveBeenCalledWith('Stop scheduled. This page will disconnect shortly.');
+    expect(toast.success).toHaveBeenCalledWith(
+      'Stop scheduled. This page will disconnect shortly.'
+    );
   });
 
   it('runs the desktop restart app flow after confirmation', async () => {
-    const queryClient = new QueryClient();
     const user = userEvent.setup();
-
-    mocks.useRuntimeControl.mockReturnValue({
-      data: {
+    const confirmSpy = vi.spyOn(window, 'confirm').mockReturnValue(true);
+    mocks.useRuntimeControlPanelView.mockReturnValue({
+      controlView: {
+        ...baseControlView,
         environment: 'desktop-embedded',
-        lifecycle: 'healthy',
-        serviceState: 'running',
-        message: 'runtime healthy',
-        pendingRestart: null,
-        canStartService: {
-          available: false,
-          requiresConfirmation: false,
-          impact: 'none',
-        },
-        canRestartService: {
-          available: true,
-          requiresConfirmation: false,
-          impact: 'brief-ui-disconnect',
-        },
         canStopService: {
           available: false,
           requiresConfirmation: true,
@@ -235,76 +161,72 @@ describe('RuntimeControlCard', () => {
           requiresConfirmation: true,
           impact: 'full-app-relaunch',
         },
-        managementHint: 'desktop launcher hint'
+        managementHint: 'desktop launcher hint',
       },
-      isError: false,
-      error: null,
+      visibleLifecycle: 'healthy',
+      visibleServiceState: 'running',
+      visibleMessage: 'runtime healthy',
+      busyAction: null,
+      busy: false,
+      pendingRestart: null,
+      errorMessage: null,
     });
-
-    const confirmSpy = vi.spyOn(window, 'confirm').mockReturnValue(true);
-
-    render(<RuntimeControlCard />, {
-      wrapper: createWrapper(queryClient),
+    mocks.runRuntimeControlAction.mockResolvedValue({
+      accepted: true,
+      action: 'restart-app',
+      lifecycle: 'restarting-app',
+      message: 'NextClaw app restart scheduled.',
     });
 
+    render(<RuntimeControlCard />);
+
     await user.click(screen.getByRole('button', { name: '重启应用' }));
 
     await waitFor(() => {
       expect(confirmSpy).toHaveBeenCalledTimes(1);
-      expect(mocks.restartApp).toHaveBeenCalledTimes(1);
+      expect(mocks.runRuntimeControlAction).toHaveBeenCalledWith(
+        'restart-app'
+      );
     });
-    expect(toast.success).toHaveBeenCalledWith('NextClaw app restart scheduled.');
+    expect(toast.success).toHaveBeenCalledWith(
+      'NextClaw app restart scheduled.'
+    );
   });
 
   it('shows a pending restart notice instead of auto-applying hidden restarts', () => {
-    const queryClient = new QueryClient();
-
-    mocks.useRuntimeControl.mockReturnValue({
-      data: {
-        environment: 'managed-local-service',
-        lifecycle: 'healthy',
-        serviceState: 'running',
+    mocks.useRuntimeControlPanelView.mockReturnValue({
+      controlView: {
+        ...baseControlView,
         message: 'Saved changes are waiting for a manual restart.',
         pendingRestart: {
           changedPaths: ['plugins', 'ui'],
           message: 'Saved changes are waiting for a manual restart.',
           reasons: ['config reload requires restart: plugins, ui'],
-          requestedAt: '2026-04-17T10:00:00.000Z'
+          requestedAt: '2026-04-17T10:00:00.000Z',
         },
-        canStartService: {
-          available: false,
-          requiresConfirmation: false,
-          impact: 'brief-ui-disconnect',
-          reasonIfUnavailable: '当前页面已经由运行中的本地服务托管。'
-        },
-        canRestartService: {
-          available: true,
-          requiresConfirmation: false,
-          impact: 'brief-ui-disconnect',
-        },
-        canStopService: {
-          available: true,
-          requiresConfirmation: true,
-          impact: 'brief-ui-disconnect',
-        },
-        canRestartApp: {
-          available: false,
-          requiresConfirmation: true,
-          impact: 'full-app-relaunch',
-          reasonIfUnavailable: 'desktop only',
-        },
-        managementHint: 'This page is served by the running local service.'
       },
-      isError: false,
-      error: null,
+      visibleLifecycle: 'healthy',
+      visibleServiceState: 'running',
+      visibleMessage: 'Saved changes are waiting for a manual restart.',
+      busyAction: null,
+      busy: false,
+      pendingRestart: {
+        changedPaths: ['plugins', 'ui'],
+        message: 'Saved changes are waiting for a manual restart.',
+        reasons: ['config reload requires restart: plugins, ui'],
+        requestedAt: '2026-04-17T10:00:00.000Z',
+      },
+      errorMessage: null,
     });
 
-    render(<RuntimeControlCard />, {
-      wrapper: createWrapper(queryClient),
-    });
+    render(<RuntimeControlCard />);
 
     expect(screen.getByText('待重启')).toBeTruthy();
-    expect(screen.getByText('这次改动已经保存，但系统不会自动重启。请在你方便的时候手动重启，重启完成后该提示会自动清空。')).toBeTruthy();
+    expect(
+      screen.getByText(
+        '这次改动已经保存，但系统不会自动重启。请在你方便的时候手动重启，重启完成后该提示会自动清空。'
+      )
+    ).toBeTruthy();
     expect(screen.getByText('plugins')).toBeTruthy();
     expect(screen.getByText('ui')).toBeTruthy();
   });