@nextclaw/ui 0.11.0 → 0.11.2

package/src/components/chat/ChatPage.tsx

@@ -1,16 +1,6 @@
-import { useLocation } from 'react-router-dom';
-import { resolveChatChain } from '@/components/chat/chat-chain';
 import type { ChatPageProps } from '@/components/chat/chat-page-shell';
-import { LegacyChatPage } from '@/components/chat/legacy/LegacyChatPage';
 import { NcpChatPage } from '@/components/chat/ncp/NcpChatPage';
 
 export function ChatPage({ view }: ChatPageProps) {
-  const location = useLocation();
-  const chatChain = resolveChatChain(location.search);
-
-  if (chatChain === 'ncp') {
-    return <NcpChatPage view={view} />;
-  }
-
-  return <LegacyChatPage view={view} />;
+  return <NcpChatPage view={view} />;
 }

package/src/components/chat/ChatSidebar.test.tsx

@@ -11,7 +11,6 @@ const mocks = vi.hoisted(() => ({
   setQuery: vi.fn(),
   selectSession: vi.fn(),
   docOpen: vi.fn(),
-  updateSession: vi.fn(),
   updateNcpSession: vi.fn()
 }));
 
@@ -33,15 +32,9 @@ vi.mock('@/components/doc-browser', () => ({
 
 vi.mock('@/components/chat/chat-session-label.service', () => ({
   useChatSessionLabelService: () => async (params: {
-    chatChain: 'legacy' | 'ncp';
     sessionKey: string;
     label: string | null;
-  }) => {
-    if (params.chatChain === 'ncp') {
-      return mocks.updateNcpSession(params.sessionKey, { label: params.label });
-    }
-    return mocks.updateSession(params.sessionKey, { label: params.label });
-  }
+  }) => mocks.updateNcpSession(params.sessionKey, { label: params.label })
 }));
 
 vi.mock('@/components/common/BrandHeader', () => ({
@@ -77,9 +70,7 @@ describe('ChatSidebar', () => {
     mocks.setQuery.mockReset();
     mocks.selectSession.mockReset();
     mocks.docOpen.mockReset();
-    mocks.updateSession.mockReset();
     mocks.updateNcpSession.mockReset();
-    mocks.updateSession.mockResolvedValue({});
     mocks.updateNcpSession.mockResolvedValue({});
 
     useChatInputStore.setState({
@@ -285,48 +276,9 @@ describe('ChatSidebar', () => {
         label: 'Renamed Label'
       });
     });
-    expect(mocks.updateSession).not.toHaveBeenCalled();
     expect(screen.getByText('Renamed Label')).not.toBeNull();
   });
 
-  it('routes inline session label edits to the legacy session api when chatChain=legacy', async () => {
-    useChatSessionListStore.setState({
-      snapshot: {
-        ...useChatSessionListStore.getState().snapshot,
-        sessions: [
-          {
-            key: 'session:legacy-1',
-            createdAt: '2026-03-19T09:00:00.000Z',
-            updatedAt: '2026-03-19T09:05:00.000Z',
-            label: 'Legacy Label',
-            sessionType: 'native',
-            sessionTypeMutable: false,
-            messageCount: 1
-          }
-        ]
-      }
-    });
-
-    render(
-      <MemoryRouter initialEntries={['/chat?chatChain=legacy']}>
-        <ChatSidebar />
-      </MemoryRouter>
-    );
-
-    fireEvent.click(screen.getByLabelText('Edit'));
-    fireEvent.change(screen.getByPlaceholderText('Session label (optional)'), {
-      target: { value: 'Legacy Renamed' }
-    });
-    fireEvent.click(screen.getByLabelText('Save'));
-
-    await waitFor(() => {
-      expect(mocks.updateSession).toHaveBeenCalledWith('session:legacy-1', {
-        label: 'Legacy Renamed'
-      });
-    });
-    expect(mocks.updateNcpSession).not.toHaveBeenCalled();
-  });
-
   it('cancels inline session label editing without saving', () => {
     useChatSessionListStore.setState({
       snapshot: {
@@ -357,7 +309,6 @@ describe('ChatSidebar', () => {
     });
     fireEvent.click(screen.getByLabelText('Cancel'));
 
-    expect(mocks.updateSession).not.toHaveBeenCalled();
     expect(mocks.updateNcpSession).not.toHaveBeenCalled();
     expect(screen.queryByDisplayValue('Should Not Persist')).toBeNull();
     expect(screen.getByText('Cancelable Label')).not.toBeNull();

package/src/components/chat/ChatSidebar.tsx

@@ -12,7 +12,6 @@ import { usePresenter } from '@/components/chat/presenter/chat-presenter-context
 import { useChatInputStore } from '@/components/chat/stores/chat-input.store';
 import { useChatRunStatusStore } from '@/components/chat/stores/chat-run-status.store';
 import { useChatSessionListStore } from '@/components/chat/stores/chat-session-list.store';
-import { resolveChatChain } from '@/components/chat/chat-chain';
 import { cn } from '@/lib/utils';
 import { LANGUAGE_OPTIONS, t, type I18nLanguage } from '@/lib/i18n';
 import { THEME_OPTIONS, type UiTheme } from '@/lib/theme';
@@ -21,7 +20,6 @@ import { useTheme } from '@/components/providers/ThemeProvider';
 import { useDocBrowser } from '@/components/doc-browser';
 import { SidebarActionItem, SidebarNavLinkItem, SidebarSelectItem } from '@/components/layout/sidebar-items';
 import { useUiStore } from '@/stores/ui.store';
-import { useLocation } from 'react-router-dom';
 import {
   AlarmClock,
   BookOpen,
@@ -117,7 +115,6 @@ const navItems = [
 export function ChatSidebar() {
   const presenter = usePresenter();
   const docBrowser = useDocBrowser();
-  const location = useLocation();
   const [isCreateMenuOpen, setIsCreateMenuOpen] = useState(false);
   const [editingSessionKey, setEditingSessionKey] = useState<string | null>(null);
   const [draftLabel, setDraftLabel] = useState('');
@@ -129,7 +126,6 @@ export function ChatSidebar() {
   const { language, setLanguage } = useI18n();
   const { theme, setTheme } = useTheme();
   const updateSessionLabel = useChatSessionLabelService();
-  const chatChain = resolveChatChain(location.search);
   const currentThemeLabel = t(THEME_OPTIONS.find((o) => o.value === theme)?.labelKey ?? 'themeWarm');
   const currentLanguageLabel = LANGUAGE_OPTIONS.find((o) => o.value === language)?.label ?? language;
 
@@ -182,7 +178,6 @@ export function ChatSidebar() {
     setSavingSessionKey(session.key);
     try {
       await updateSessionLabel({
-        chatChain,
         sessionKey: session.key,
         label: normalizedLabel || null
       });

package/src/components/chat/README.md

@@ -0,0 +1,2 @@
+## 目录预算豁免
+- 原因：聊天入口层需要并列承载页面壳、会话侧栏、会话偏好同步、展示组件与配套测试，且已通过 `ncp/` 等子目录承接更细分的实现，短期内仍需保留这组扁平入口文件。

package/src/components/chat/adapters/chat-message.adapter.test.ts

@@ -17,8 +17,15 @@ const defaultTexts = {
   reasoningLabel: "Reasoning",
   toolCallLabel: "Tool Call",
   toolResultLabel: "Tool Result",
+  toolInputLabel: "Input Summary",
+  toolCallIdLabel: "Call ID",
   toolNoOutputLabel: "No output",
   toolOutputLabel: "View Output",
+  toolStatusPreparingLabel: "Preparing",
+  toolStatusRunningLabel: "Running",
+  toolStatusCompletedLabel: "Completed",
+  toolStatusFailedLabel: "Failed",
+  toolStatusCancelledLabel: "Cancelled",
   imageAttachmentLabel: "Image attachment",
   fileAttachmentLabel: "File attachment",
   unknownPartLabel: "Unknown Part",
@@ -66,9 +73,7 @@ it("maps markdown, reasoning, and tool parts into UI view models", () => {
 
   expect(adapted).toHaveLength(1);
   expect(adapted[0]?.roleLabel).toBe("Assistant");
-  expect(adapted[0]?.timestampLabel).toBe(
-    "formatted:2026-03-17T10:00:00.000Z",
-  );
+  expect(adapted[0]?.timestampLabel).toBe("formatted:2026-03-17T10:00:00.000Z");
   expect(adapted[0]?.parts.map((part) => part.type)).toEqual([
     "markdown",
     "reasoning",
@@ -82,12 +87,67 @@ it("maps markdown, reasoning, and tool parts into UI view models", () => {
   expect(adapted[0]?.parts[2]).toMatchObject({
     type: "tool-card",
     card: {
+      callId: "call-1",
+      callIdLabel: "Call ID",
+      inputLabel: "Input Summary",
+      statusLabel: "Completed",
+      statusTone: "success",
       titleLabel: "Tool Result",
       outputLabel: "View Output",
     },
   });
 });
 
+it("maps tool lifecycle statuses into visible card state feedback", () => {
+  const adapted = adapt([
+    {
+      id: "assistant-tool-statuses",
+      role: "assistant",
+      parts: [
+        {
+          type: "tool-invocation",
+          toolInvocation: {
+            status: ToolInvocationStatus.PARTIAL_CALL,
+            toolCallId: "call-prep",
+            toolName: "web_search",
+            args: '{"q":"latest"}',
+          },
+        },
+        {
+          type: "tool-invocation",
+          toolInvocation: {
+            status: ToolInvocationStatus.ERROR,
+            toolCallId: "call-error",
+            toolName: "exec_command",
+            args: '{"cmd":"exit 1"}',
+            error: "Command failed",
+          },
+        },
+      ],
+    },
+  ] as unknown as ChatMessageSource[]);
+
+  expect(adapted[0]?.parts[0]).toMatchObject({
+    type: "tool-card",
+    card: {
+      statusTone: "running",
+      statusLabel: "Preparing",
+      titleLabel: "Tool Call",
+      callId: "call-prep",
+    },
+  });
+  expect(adapted[0]?.parts[1]).toMatchObject({
+    type: "tool-card",
+    card: {
+      statusTone: "error",
+      statusLabel: "Failed",
+      titleLabel: "Tool Result",
+      output: "Command failed",
+      callId: "call-error",
+    },
+  });
+});
+
 it("maps non-standard roles back to the generic message role", () => {
   const adapted = adapt([
     {
@@ -149,6 +209,7 @@ it("maps file parts into previewable attachment view models", () => {
           type: "file",
           mimeType: "image/png",
           data: "ZmFrZS1pbWFnZQ==",
+          sizeBytes: 4096,
         },
       ],
     },
@@ -160,6 +221,7 @@ it("maps file parts into previewable attachment view models", () => {
       label: "Image attachment",
       mimeType: "image/png",
       dataUrl: "data:image/png;base64,ZmFrZS1pbWFnZQ==",
+      sizeBytes: 4096,
       isImage: true,
     },
   });
@@ -176,6 +238,7 @@ it("keeps named non-image files as downloadable attachments", () => {
           name: "spec.pdf",
           mimeType: "application/pdf",
           data: "cGRm",
+          sizeBytes: 2048,
         },
       ],
     },
@@ -187,6 +250,7 @@ it("keeps named non-image files as downloadable attachments", () => {
       label: "spec.pdf",
       mimeType: "application/pdf",
       dataUrl: "data:application/pdf;base64,cGRm",
+      sizeBytes: 2048,
       isImage: false,
     },
   });
@@ -212,6 +276,7 @@ it("renders asset tool results as previewable files", () => {
                 name: "output.png",
                 mimeType: "image/png",
                 url: "/api/ncp/assets/content?uri=asset%3A%2F%2Fstore%2F2026%2F03%2F27%2Fasset_1",
+                sizeBytes: 5120,
               },
             },
           },
@@ -225,7 +290,9 @@ it("renders asset tool results as previewable files", () => {
     file: {
       label: "output.png",
       mimeType: "image/png",
-      dataUrl: "/api/ncp/assets/content?uri=asset%3A%2F%2Fstore%2F2026%2F03%2F27%2Fasset_1",
+      dataUrl:
+        "/api/ncp/assets/content?uri=asset%3A%2F%2Fstore%2F2026%2F03%2F27%2Fasset_1",
+      sizeBytes: 5120,
       isImage: true,
     },
   });