npm - @nextclaw/ui - Versions diffs - 0.12.30 → 0.12.32 - Mend

@nextclaw/ui 0.12.30 → 0.12.32

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (66) hide show

package/dist/index.html CHANGED Viewed

@@ -78,35 +78,35 @@
     })();
   </script>
   <title>NextClaw</title>
-  <script type="module" crossorigin src="/assets/index-CTfzteME.js"></script>
+  <script type="module" crossorigin src="/assets/index-BEn7o-oH.js"></script>
   <link rel="modulepreload" crossorigin href="/assets/i18n-D1144VAA.js">
   <link rel="modulepreload" crossorigin href="/assets/createLucideIcon-DzY6wN61.js">
-  <link rel="modulepreload" crossorigin href="/assets/cpu-DPPwMzoC.js">
-  <link rel="modulepreload" crossorigin href="/assets/api-CKKNxbsy.js">
-  <link rel="modulepreload" crossorigin href="/assets/es2015-ZJ-nWL4L.js">
-  <link rel="modulepreload" crossorigin href="/assets/select-C2c10F3e.js">
-  <link rel="modulepreload" crossorigin href="/assets/dist-DN-YM_0Y.js">
+  <link rel="modulepreload" crossorigin href="/assets/cpu-DKc3bQic.js">
+  <link rel="modulepreload" crossorigin href="/assets/api-rASx2-N4.js">
+  <link rel="modulepreload" crossorigin href="/assets/es2015-CVAuMqlM.js">
+  <link rel="modulepreload" crossorigin href="/assets/select-HI5kiwX_.js">
+  <link rel="modulepreload" crossorigin href="/assets/dist-NBjcwlsw.js">
   <link rel="modulepreload" crossorigin href="/assets/x-BeyYA_h6.js">
-  <link rel="modulepreload" crossorigin href="/assets/dialog-DWZeMf9V.js">
-  <link rel="modulepreload" crossorigin href="/assets/popover-2H9DS7Cv.js">
+  <link rel="modulepreload" crossorigin href="/assets/dialog-CR80q3Rm.js">
+  <link rel="modulepreload" crossorigin href="/assets/popover-DeAliE_4.js">
   <link rel="modulepreload" crossorigin href="/assets/search-CRtQwr-h.js">
   <link rel="modulepreload" crossorigin href="/assets/book-open-BcnAiKde.js">
+  <link rel="modulepreload" crossorigin href="/assets/external-link-CpEvG65F.js">
+  <link rel="modulepreload" crossorigin href="/assets/plus-B-YHtTNC.js">
+  <link rel="modulepreload" crossorigin href="/assets/doc-browser-context-BPeeiNJ_.js">
+  <link rel="modulepreload" crossorigin href="/assets/doc-browser-B19cYGxa.js">
   <link rel="modulepreload" crossorigin href="/assets/download-CMM8po31.js">
   <link rel="modulepreload" crossorigin href="/assets/key-round-DUq47t0P.js">
-  <link rel="modulepreload" crossorigin href="/assets/notice-card-BM2_YNoc.js">
+  <link rel="modulepreload" crossorigin href="/assets/notice-card-CF4AJPM3.js">
   <link rel="modulepreload" crossorigin href="/assets/status-dot-l3kPFdq_.js">
-  <link rel="modulepreload" crossorigin href="/assets/app-manager-provider-BM1hziPv.js">
+  <link rel="modulepreload" crossorigin href="/assets/app-manager-provider-CUkcbdWk.js">
   <link rel="modulepreload" crossorigin href="/assets/settings-MWL2SMyk.js">
   <link rel="modulepreload" crossorigin href="/assets/refresh-cw-CrbD8EkT.js">
   <link rel="modulepreload" crossorigin href="/assets/sparkles-BmgOD4nY.js">
-  <link rel="modulepreload" crossorigin href="/assets/external-link-CpEvG65F.js">
-  <link rel="modulepreload" crossorigin href="/assets/plus-B-YHtTNC.js">
-  <link rel="modulepreload" crossorigin href="/assets/doc-browser-context-FukQHvyo.js">
-  <link rel="modulepreload" crossorigin href="/assets/doc-browser-rZIQIjuw.js">
-  <link rel="modulepreload" crossorigin href="/assets/doc-browser-CAhfnm0D.js">
-  <link rel="modulepreload" crossorigin href="/assets/use-config-jAYudJxy.js">
-  <link rel="modulepreload" crossorigin href="/assets/desktop-CmIbyIrg.js">
-  <link rel="stylesheet" crossorigin href="/assets/index-DKC9hvsP.css">
+  <link rel="modulepreload" crossorigin href="/assets/doc-browser-CDBYvb8A.js">
+  <link rel="modulepreload" crossorigin href="/assets/use-config-BFqzVF8Y.js">
+  <link rel="modulepreload" crossorigin href="/assets/desktop-DJpQzxsz.js">
+  <link rel="stylesheet" crossorigin href="/assets/index-DV57tBmg.css">
 </head>
 <body>

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "@nextclaw/ui",
-  "version": "0.12.30",
+  "version": "0.12.32",
   "private": false,
   "type": "module",
   "license": "MIT",
@@ -28,14 +28,14 @@
     "tailwind-merge": "^2.5.4",
     "zod": "^3.23.8",
     "zustand": "^5.0.2",
-    "@nextclaw/agent-chat": "0.1.18",
-    "@nextclaw/client-sdk": "0.1.9",
-    "@nextclaw/agent-chat-ui": "0.3.20",
-    "@nextclaw/ncp": "0.5.13",
-    "@nextclaw/ncp-http-agent-client": "0.3.25",
-    "@nextclaw/server": "0.12.21",
-    "@nextclaw/ncp-react": "0.4.33",
-    "@nextclaw/shared": "0.1.7"
+    "@nextclaw/agent-chat": "0.1.20",
+    "@nextclaw/agent-chat-ui": "0.3.22",
+    "@nextclaw/client-sdk": "0.1.11",
+    "@nextclaw/ncp": "0.5.15",
+    "@nextclaw/ncp-http-agent-client": "0.3.27",
+    "@nextclaw/ncp-react": "0.4.35",
+    "@nextclaw/server": "0.12.23",
+    "@nextclaw/shared": "0.1.9"
   },
   "devDependencies": {
     "@testing-library/react": "^16.3.0",

package/src/app/components/layout/app-layout.test.tsx CHANGED Viewed

@@ -1,6 +1,8 @@
 import { render, screen } from "@testing-library/react";
+import { QueryClient, QueryClientProvider } from "@tanstack/react-query";
+import type { ReactNode } from "react";
 import { MemoryRouter, Route, Routes } from "react-router-dom";
-import { describe, expect, it, vi } from "vitest";
+import { beforeEach, describe, expect, it, vi } from "vitest";
 import { AppLayout } from "@/app/components/layout/app-layout";
 import { I18nProvider } from "@/app/components/i18n-provider";
@@ -22,7 +24,22 @@ vi.mock("@/app/components/layout/sidebar", () => ({
   ),
 }));
+vi.mock("@/platforms/mobile", () => ({
+  MobileAppShell: ({ children }: { children: ReactNode }) => (
+    <div data-testid="mobile-app-shell">{children}</div>
+  ),
+}));
 describe("AppLayout", () => {
+  beforeEach(() => {
+    useViewportLayoutMock.mockReturnValue({
+      mode: "desktop",
+      isMobile: false,
+      isDesktop: true,
+    });
+    window.nextclawDesktop = undefined;
+  });
   it("treats /agents as a main workspace route instead of the settings shell", () => {
     const { container } = render(
       <I18nProvider>
@@ -97,13 +114,50 @@ describe("AppLayout", () => {
     );
     expect(screen.getByTestId("chat-content")).toBeTruthy();
+    expect(screen.getByTestId("mobile-app-shell")).toBeTruthy();
     expect(screen.queryByTestId("settings-sidebar-header")).toBeNull();
     expect(container.querySelector("aside")).toBeNull();
+  });
+  it("keeps the desktop shell on desktop hosts even when the viewport is narrow", () => {
+    window.nextclawDesktop = {
+      platform: "win32",
+    } as typeof window.nextclawDesktop;
     useViewportLayoutMock.mockReturnValue({
-      mode: "desktop",
-      isMobile: false,
-      isDesktop: true,
+      mode: "mobile",
+      isMobile: true,
+      isDesktop: false,
     });
+    const queryClient = new QueryClient({
+      defaultOptions: {
+        queries: {
+          retry: false,
+        },
+      },
+    });
+    render(
+      <QueryClientProvider client={queryClient}>
+        <I18nProvider>
+          <MemoryRouter initialEntries={["/chat"]}>
+            <Routes>
+              <Route
+                path="*"
+                element={
+                  <AppLayout>
+                    <div data-testid="chat-content">Chat Content</div>
+                  </AppLayout>
+                }
+              />
+            </Routes>
+          </MemoryRouter>
+        </I18nProvider>
+      </QueryClientProvider>,
+    );
+    expect(screen.getByTestId("chat-content")).toBeTruthy();
+    expect(screen.getByTestId("desktop-window-chrome")).toBeTruthy();
+    expect(screen.getByTestId("desktop-window-chrome-main-drag-region")).toBeTruthy();
   });
 });

package/src/app/components/layout/app-layout.tsx CHANGED Viewed

@@ -7,7 +7,7 @@ import {
 import { useDocLinkInterceptor } from "@/shared/components/doc-browser/use-doc-link-interceptor";
 import { useI18n } from "@/app/components/i18n-provider";
 import { useViewportLayout } from "@/app/hooks/use-viewport-layout";
-import { DesktopAppShell } from "@/platforms/desktop";
+import { DesktopAppShell, getDesktopHostPlatform } from "@/platforms/desktop";
 import { MobileAppShell } from "@/platforms/mobile";
 import { resolveUiDocumentTitle } from "@/shared/lib/ui-document-title";
@@ -26,7 +26,7 @@ function AppLayoutInner({ children }: AppLayoutProps) {
     document.title = resolveUiDocumentTitle(pathname);
   }, [pathname, language]);
-  if (isMobile) {
+  if (isMobile && getDesktopHostPlatform() === null) {
     return (
       <MobileAppShell pathname={pathname} isDocBrowserOpen={isOpen}>
         {children}

package/src/app/components/theme-provider.tsx CHANGED Viewed

@@ -28,7 +28,6 @@ export function ThemeProvider({ children }: { children: ReactNode }) {
   useEffect(() => {
     pwaShellThemeManager.syncTheme(theme);
-    void window.nextclawDesktop?.setShellTheme?.(theme);
   }, [theme]);
   useEffect(() => {

package/src/app/hooks/use-viewport-layout.ts CHANGED Viewed

@@ -10,6 +10,5 @@ export function useViewportLayout() {
   return {
     mode,
     isMobile: mode === "mobile",
-    isDesktop: mode === "desktop",
   };
 }

package/src/features/chat/components/chat-session-workspace-panel.tsx CHANGED Viewed

@@ -30,6 +30,7 @@ import { usePresenter } from "@/features/chat/components/providers/chat-presente
 import { ChatSessionWorkspaceFilePreview } from "./chat-session-workspace-file-preview";
 import { AgentIdentityAvatar } from "@/shared/components/common/agent-identity";
 import { SessionCronJobContent } from "@/features/chat/components/workspace/session-cron-job-content";
+import { ResizableRightPanel } from "@/shared/components/resizable-right-panel/resizable-right-panel";
 import { t } from "@/shared/lib/i18n";
 import { cn } from "@/shared/lib/utils";
@@ -314,63 +315,65 @@ export function ChatSessionWorkspacePanel({
   }
   return (
-    <aside
+    <ResizableRightPanel
       className={cn(
         displayMode === "overlay"
-          ? "fixed inset-0 z-40 flex bg-white"
-          : "hidden shrink-0 border-l border-gray-200/70 bg-white/95 backdrop-blur-sm md:flex md:w-[26rem] lg:w-[30rem] xl:w-[34rem]",
+          ? "bg-white"
+          : "hidden border-gray-200/70 bg-white/95 backdrop-blur-sm md:flex",
       )}
+      defaultWidth={480}
+      minWidth={360}
+      maxWidth={860}
+      overlay={displayMode === "overlay"}
     >
-      <div className="flex h-full min-h-0 w-full flex-col">
-        <div className="flex items-center justify-between gap-3 border-b border-gray-200/70 px-4 py-2.5">
-          <button
-            type="button"
-            onClick={presenter.chatThreadManager.goToParentSession}
-            className={cn(
-              "inline-flex items-center gap-1 text-xs font-medium text-gray-600 transition-colors hover:text-gray-900",
-              !hasParentSession && "pointer-events-none opacity-0",
-            )}
-          >
-            <ArrowLeft className="h-3.5 w-3.5" />
-            <span>{t("chatBackToParent")}</span>
-          </button>
-          <button
-            type="button"
-            onClick={presenter.chatThreadManager.closeWorkspacePanel}
-            className="rounded-full border border-gray-200/80 p-1.5 text-gray-500 transition-colors hover:bg-gray-100 hover:text-gray-900"
-            aria-label={t("chatWorkspaceClosePanel")}
-          >
-            <X className="h-4 w-4" />
-          </button>
-        </div>
+      <div className="flex items-center justify-between gap-3 border-b border-gray-200/70 px-4 py-2.5">
+        <button
+          type="button"
+          onClick={presenter.chatThreadManager.goToParentSession}
+          className={cn(
+            "inline-flex items-center gap-1 text-xs font-medium text-gray-600 transition-colors hover:text-gray-900",
+            !hasParentSession && "pointer-events-none opacity-0",
+          )}
+        >
+          <ArrowLeft className="h-3.5 w-3.5" />
+          <span>{t("chatBackToParent")}</span>
+        </button>
+        <button
+          type="button"
+          onClick={presenter.chatThreadManager.closeWorkspacePanel}
+          className="rounded-full border border-gray-200/80 p-1.5 text-gray-500 transition-colors hover:bg-gray-100 hover:text-gray-900"
+          aria-label={t("chatWorkspaceClosePanel")}
+        >
+          <X className="h-4 w-4" />
+        </button>
+      </div>
-        <WorkspaceTabsBar tabs={workspaceTabs} />
+      <WorkspaceTabsBar tabs={workspaceTabs} />
-        <div className="flex min-h-0 flex-1 flex-col bg-white">
-          {activeSelection.kind === "child-session" ? (
-            <>
-              <WorkspaceActiveChildHeader tab={activeSelection.tab} />
-              <div className="flex-1 min-h-0">
-                <ChildSessionContent
-                  sessionKey={activeSelection.tab.sessionKey}
-                  onToolAction={
-                    presenter.chatThreadManager.openSessionFromToolAction
-                  }
-                  onFileOpen={presenter.chatThreadManager.openFilePreview}
-                />
-              </div>
-            </>
-          ) : activeSelection.kind === "file" ? (
-            <ChatSessionWorkspaceFilePreview
-              file={activeSelection.file}
-              sessionProjectRoot={sessionProjectRoot}
-              onFileOpen={presenter.chatThreadManager.openFilePreview}
-            />
-          ) : (
-            <SessionCronJobContent jobs={sessionCronJobs} />
-          )}
-        </div>
+      <div className="flex min-h-0 flex-1 flex-col bg-white">
+        {activeSelection.kind === "child-session" ? (
+          <>
+            <WorkspaceActiveChildHeader tab={activeSelection.tab} />
+            <div className="flex-1 min-h-0">
+              <ChildSessionContent
+                sessionKey={activeSelection.tab.sessionKey}
+                onToolAction={
+                  presenter.chatThreadManager.openSessionFromToolAction
+                }
+                onFileOpen={presenter.chatThreadManager.openFilePreview}
+              />
+            </div>
+          </>
+        ) : activeSelection.kind === "file" ? (
+          <ChatSessionWorkspaceFilePreview
+            file={activeSelection.file}
+            sessionProjectRoot={sessionProjectRoot}
+            onFileOpen={presenter.chatThreadManager.openFilePreview}
+          />
+        ) : (
+          <SessionCronJobContent jobs={sessionCronJobs} />
+        )}
       </div>
-    </aside>
+    </ResizableRightPanel>
   );
 }

package/src/features/chat/components/conversation/chat-conversation-panel.test.tsx CHANGED Viewed

@@ -2,7 +2,7 @@ import { render, screen } from "@testing-library/react";
 import userEvent from "@testing-library/user-event";
 import { beforeEach, describe, expect, it, vi } from "vitest";
 import { ChatConversationPanel } from "@/features/chat/components/conversation/chat-conversation-panel";
-import { ChatSessionWorkspacePanel } from "@/features/chat";
+import { ChatSessionWorkspacePanel } from "@/features/chat/components/chat-session-workspace-panel";
 import type { ResolvedChildSessionTab } from "@/features/chat/hooks/use-ncp-child-session-tabs-view";
 import type { CronJobView } from "@/shared/lib/api";
 import { useChatInputStore } from "@/features/chat/stores/chat-input.store";
@@ -556,6 +556,7 @@ describe("ChatSessionWorkspacePanel", () => {
     );
     expect(screen.queryByText("Child sessions")).toBeNull();
+    expect(screen.getByTestId("resizable-right-panel-handle")).toBeTruthy();
     expect(screen.getAllByText("北京天气")).toHaveLength(2);
     expect(screen.getByText("Codex")).toBeTruthy();
     expect(screen.getByText("openai/gpt-5.3-codex")).toBeTruthy();

package/src/features/chat/managers/chat-session-list.manager.test.ts CHANGED Viewed

@@ -42,8 +42,9 @@ describe('ChatSessionListManager', () => {
       snapshot: {
         ...useChatThreadStore.getState().snapshot,
         workspacePanelParentKey: 'session-1',
+        activeWorkspacePanelKind: 'child-session',
         activeChildSessionKey: 'child-session-1',
-        activeWorkspaceFileKey: 'session-1::/tmp/demo.md',
+        activeWorkspaceFileKey: null,
       },
     });
   });
@@ -155,7 +156,7 @@ describe('ChatSessionListManager', () => {
     expect(useChatInputStore.getState().snapshot.pendingProjectRootSessionKey).toBeNull();
   });
-  it('delegates existing-session selection to routing without eagerly mutating the selected session state', () => {
+  it('delegates existing-session selection to routing while preserving workspace panel state', () => {
     const uiManager = {
       goToChatRoot: vi.fn(),
       goToSession: vi.fn(),
@@ -170,8 +171,9 @@ describe('ChatSessionListManager', () => {
     expect(uiManager.goToSession).toHaveBeenCalledWith('session-2');
     expect(useChatSessionListStore.getState().snapshot.selectedSessionKey).toBe('session-1');
-    expect(useChatThreadStore.getState().snapshot.workspacePanelParentKey).toBeNull();
-    expect(useChatThreadStore.getState().snapshot.activeChildSessionKey).toBeNull();
+    expect(useChatThreadStore.getState().snapshot.workspacePanelParentKey).toBe('session-1');
+    expect(useChatThreadStore.getState().snapshot.activeWorkspacePanelKind).toBe('child-session');
+    expect(useChatThreadStore.getState().snapshot.activeChildSessionKey).toBe('child-session-1');
     expect(useChatThreadStore.getState().snapshot.activeWorkspaceFileKey).toBeNull();
   });

package/src/features/chat/managers/chat-session-list.manager.ts CHANGED Viewed

@@ -154,12 +154,6 @@ export class ChatSessionListManager {
   };
   selectSession = (sessionKey: string) => {
-    useChatThreadStore.getState().setSnapshot({
-      workspacePanelParentKey: null,
-      activeWorkspacePanelKind: null,
-      activeChildSessionKey: null,
-      activeWorkspaceFileKey: null,
-    });
     this.uiManager.goToSession(sessionKey);
   };

package/src/features/chat/utils/chat-message-invalid-tool-arguments.utils.test.ts ADDED Viewed

@@ -0,0 +1,79 @@
+import { ToolInvocationStatus, type UiMessage } from "@nextclaw/agent-chat";
+import { adaptChatMessages } from "./chat-message.utils";
+import type { ChatMessageSource } from "./chat-message.utils";
+const defaultTexts = {
+  roleLabels: {
+    user: "You",
+    assistant: "Assistant",
+    tool: "Tool",
+    system: "System",
+    fallback: "Message",
+  },
+  reasoningLabel: "Reasoning",
+  toolCallLabel: "Tool Call",
+  toolResultLabel: "Tool Result",
+  toolInputLabel: "Input",
+  toolNoOutputLabel: "No output",
+  toolOutputLabel: "Output",
+  toolStatusPreparingLabel: "Preparing",
+  toolStatusRunningLabel: "Running",
+  toolStatusCompletedLabel: "Completed",
+  toolStatusFailedLabel: "Failed",
+  toolStatusCancelledLabel: "Cancelled",
+  imageAttachmentLabel: "Image attachment",
+  fileAttachmentLabel: "File attachment",
+  unknownPartLabel: "Unknown Part",
+};
+function adapt(uiMessages: UiMessage[]) {
+  return adaptChatMessages({
+    uiMessages: uiMessages as unknown as ChatMessageSource[],
+    formatTimestamp: (value) => `formatted:${value}`,
+    texts: defaultTexts,
+  });
+}
+it("renders invalid tool arguments as a failed argument card instead of a normal raw-args card", () => {
+  const rawArgumentsText = '{"task":"spawn","request">\n<｜｜DSML｜｜parameter: "final_reply"}';
+  const adapted = adapt([
+    {
+      id: "assistant-invalid-tool-args",
+      role: "assistant",
+      parts: [
+        {
+          type: "tool-invocation",
+          toolInvocation: {
+            status: ToolInvocationStatus.RESULT,
+            toolCallId: "call-invalid-args",
+            toolName: "sessions_spawn",
+            args: rawArgumentsText,
+            result: {
+              ok: false,
+              error: {
+                code: "invalid_tool_arguments",
+                message: "Tool arguments are invalid.",
+                toolCallId: "call-invalid-args",
+                toolName: "sessions_spawn",
+                rawArgumentsText,
+                issues: ["Invalid JSON"],
+              },
+            },
+          },
+        },
+      ],
+    },
+  ]);
+  expect(adapted[0]?.parts[0]).toMatchObject({
+    type: "tool-card",
+    card: {
+      toolName: "sessions_spawn",
+      summary: "Invalid arguments: Invalid JSON",
+      input: rawArgumentsText,
+      output: "Invalid JSON",
+      statusTone: "error",
+      statusLabel: "Failed",
+    },
+  });
+});

package/src/features/chat/utils/chat-message-part.utils.ts CHANGED Viewed

@@ -21,7 +21,6 @@ import { buildFileOperationCardData } from "./file-operation/card.utils";
 import { buildSessionRequestToolCard } from "./chat-message-session-request-tool-card.utils";
 import type {
   ChatMessagePartViewModel,
-  ChatToolPartViewModel,
 } from "@nextclaw/agent-chat-ui";
 import type {
   ChatMessageAdapterTexts,
@@ -39,6 +38,17 @@ type ChatMessagePartAdapterParams = {
   texts: ChatMessageAdapterTexts;
 };
+function readInvalidToolArgumentsError(value: unknown): Record<string, unknown> | null {
+  const error = typeof value === "object" && value !== null && !Array.isArray(value)
+    ? (value as { error?: unknown }).error
+    : null;
+  return typeof error === "object" &&
+    error !== null &&
+    !Array.isArray(error) &&
+    (error as { code?: unknown }).code === "invalid_tool_arguments"
+    ? error as Record<string, unknown>
+    : null;
+}
 function buildReasoningPart(
   part: Extract<ChatMessagePartSource, { type: "reasoning" }>,
@@ -113,6 +123,10 @@ function buildToolInvocationPart(
     result: invocation.result,
     texts,
   });
+  const invalidArgsError = readInvalidToolArgumentsError(invocation.result);
+  const invalidArgsIssue = Array.isArray(invalidArgsError?.issues)
+    ? invalidArgsError.issues.find((issue): issue is string => typeof issue === "string" && issue.trim().length > 0)
+    : undefined;
   const fileOperationCardData = buildFileOperationCardData({
     toolName: invocation.toolName,
     status: invocation.status,
@@ -121,14 +135,14 @@ function buildToolInvocationPart(
     parsedArgs: invocation.parsedArgs,
     result: invocation.result,
   });
-  const detail =
-    fileOperationCardData?.summary ??
-    summarizeToolArgs(invocation.parsedArgs ?? invocation.args);
-  const input = fileOperationCardData
-    ? undefined
-    : buildToolInvocationInput(invocation.args, invocation.parsedArgs);
+  const detail = invalidArgsIssue
+    ? `Invalid arguments: ${invalidArgsIssue}`
+    : fileOperationCardData?.summary ?? summarizeToolArgs(invocation.parsedArgs ?? invocation.args);
+  const input = fileOperationCardData ? undefined : buildToolInvocationInput(invocation.args, invocation.parsedArgs);
   const rawResult =
-    typeof invocation.error === "string" && invocation.error.trim()
+    invalidArgsIssue
+      ? invalidArgsIssue
+      : typeof invocation.error === "string" && invocation.error.trim()
       ? invocation.error.trim()
       : invocation.result != null
         ? stringifyUnknown(invocation.result).trim()
@@ -149,8 +163,8 @@ function buildToolInvocationPart(
     outputData: invocation.result,
     callId: invocation.toolCallId || undefined,
     hasResult: statusView.hasResult,
-    statusTone: statusView.statusTone,
-    statusLabel: statusView.statusLabel,
+    statusTone: invalidArgsError ? "error" : statusView.statusTone,
+    statusLabel: invalidArgsError ? texts.toolStatusFailedLabel : statusView.statusLabel,
     ...(fileOperationCardData?.fileOperation
       ? { fileOperation: fileOperationCardData.fileOperation }
       : {}),
@@ -217,17 +231,18 @@ function isToolInvocationPart(
 export function adaptChatMessagePart(
   params: ChatMessagePartAdapterParams,
 ): ChatMessagePartViewModel | null {
-  if (isTextPart(params.part)) {
-    return buildTextPart(params.part, params.inlineTokens);
+  const { inlineTokens, part, texts } = params;
+  if (isTextPart(part)) {
+    return buildTextPart(part, inlineTokens);
   }
-  if (isReasoningPart(params.part)) {
-    return buildReasoningPart(params.part, params.texts);
+  if (isReasoningPart(part)) {
+    return buildReasoningPart(part, texts);
   }
-  if (isFilePart(params.part)) {
-    return buildFilePart(params.part, params.texts);
+  if (isFilePart(part)) {
+    return buildFilePart(part, texts);
   }
-  if (isToolInvocationPart(params.part)) {
-    return buildToolInvocationPart(params.part, params.texts);
+  if (isToolInvocationPart(part)) {
+    return buildToolInvocationPart(part, texts);
   }
-  return buildUnknownPart(params.part, params.texts);
+  return buildUnknownPart(part, texts);
 }

package/src/main.tsx CHANGED Viewed

@@ -1,18 +1,20 @@
 import { StrictMode } from 'react';
 import { createRoot } from 'react-dom/client';
-import { BrowserRouter } from 'react-router-dom';
+import { BrowserRouter, MemoryRouter } from 'react-router-dom';
 import App from './app';
 import { I18nProvider } from '@/components/providers/I18nProvider';
 import { ThemeProvider } from '@/components/providers/ThemeProvider';
 import './index.css';
+const AppRouter = window.nextclawDesktop?.platform === 'win32' ? MemoryRouter : BrowserRouter;
 createRoot(document.getElementById('root')!).render(
   <StrictMode>
     <ThemeProvider>
       <I18nProvider>
-        <BrowserRouter>
+        <AppRouter>
           <App />
-        </BrowserRouter>
+        </AppRouter>
       </I18nProvider>
     </ThemeProvider>
   </StrictMode>

package/src/platforms/desktop/components/desktop-app-shell.test.tsx CHANGED Viewed

@@ -48,14 +48,26 @@ describe("DesktopAppShell", () => {
     const chrome = screen.getByTestId("desktop-window-chrome");
     const sidebarChrome = screen.getByTestId("desktop-window-chrome-sidebar");
-    const mainChrome = screen.getByTestId("desktop-window-chrome-main");
+    const mainDragRegion = screen.getByTestId("desktop-window-chrome-main-drag-region");
+    const resizeStrip = screen.getByTestId("desktop-window-chrome-resize-strip");
+    const controls = screen.getByTestId("desktop-window-controls");
     expect(chrome).toBeTruthy();
     expect(chrome.parentElement?.style.getPropertyValue("--desktop-titlebar-height")).toBe("40px");
+    expect(chrome.className).toContain("bg-secondary");
+    expect(chrome.className).toContain("border-b");
+    expect(chrome.className).toContain("desktop-window-drag");
     expect(sidebarChrome.className).toContain("w-[var(--desktop-sidebar-width)]");
-    expect(sidebarChrome.className).not.toContain("border-b");
-    expect(mainChrome.className).toContain("border-b");
-    expect(mainChrome.className).toContain("pr-[var(--desktop-caption-safe-right)]");
+    expect(sidebarChrome.className).toContain("desktop-window-no-drag");
+    expect(mainDragRegion.className).toContain("flex-1");
+    expect(mainDragRegion.className).toContain("desktop-window-drag");
+    expect(resizeStrip.className).toContain("desktop-window-no-drag");
+    expect(resizeStrip.className).toContain("top-0");
+    expect(resizeStrip.className).toContain("h-1");
+    expect(controls.className).toContain("desktop-window-no-drag");
+    expect(screen.getByLabelText("Minimize").className).toContain("desktop-window-no-drag");
+    expect(screen.getByLabelText("Maximize").className).toContain("desktop-window-no-drag");
+    expect(screen.getByLabelText("Close").className).toContain("desktop-window-no-drag");
     expect(screen.getByTestId("app-content")).toBeTruthy();
   });