@nextclaw/ui 0.15.0 → 0.15.2

package/src/features/chat/features/workspace/components/chat-session-workspace-file-breadcrumbs.tsx

@@ -1,6 +1,6 @@
 import { Fragment, useState } from "react";
 import type { ChatFileOpenActionViewModel } from "@nextclaw/agent-chat-ui";
-import { ChevronRight, FileCode2, FolderTree } from "lucide-react";
+import { ChevronRight, FileCode2, Folder, FolderTree } from "lucide-react";
 import { Popover, PopoverContent, PopoverTrigger } from "@/shared/components/ui/popover";
 import { WorkspaceBreadcrumbBrowser } from "./chat-session-workspace-file-breadcrumb-browser";
 import type {
@@ -59,6 +59,8 @@ function WorkspaceBreadcrumbSegmentButton({
         >
           {segment.kind === "workspace" ? (
             <FolderTree className="h-3 w-3 shrink-0" />
+          ) : segment.kind === "directory" && segment.isCurrent ? (
+            <Folder className="h-3 w-3 shrink-0" />
           ) : segment.isCurrent ? (
             <FileCode2 className="h-3 w-3 shrink-0" />
           ) : null}

package/src/features/chat/features/workspace/components/chat-session-workspace-file-preview.tsx

@@ -4,12 +4,20 @@ import type {
   ChatFileOperationBlockViewModel,
   ChatFilePreviewViewer,
 } from "@nextclaw/agent-chat-ui";
-import { ChatMessageMarkdown, FileOperationCodeSurface } from "@nextclaw/agent-chat-ui";
+import {
+  ChatMessageMarkdown,
+  FileOperationCodeSurface,
+} from "@nextclaw/agent-chat-ui";
 import type { ChatWorkspaceFileTab } from "@/features/chat/stores/chat-thread.store";
+import { ChatSessionWorkspaceDirectoryBrowser } from "./chat-session-workspace-directory-browser";
 import { ChatSessionWorkspaceFileBreadcrumbs } from "./chat-session-workspace-file-breadcrumbs";
+import { useServerPathBrowse } from "@/shared/hooks/use-server-path-browse";
 import { useServerPathRead } from "@/shared/hooks/use-server-path-read";
 import { buildServerPathContentUrl } from "@/shared/lib/api";
-import { buildLineDiff, buildPreviewLines } from "@/features/chat/features/message/utils/file-operation/line-builder.utils";
+import {
+  buildLineDiff,
+  buildPreviewLines,
+} from "@/features/chat/features/message/utils/file-operation/line-builder.utils";
 import { t } from "@/shared/lib/i18n";
 import { buildWorkspaceFileBreadcrumb } from "@/shared/lib/session-project";
 import { cn } from "@/shared/lib/utils";
@@ -24,17 +32,18 @@ function inferPreviewKind(params: {
   return /\.mdx?$/i.test(params.path) ? "markdown" : "text";
 }
 
-function isHtmlFilePath(path: string): boolean {
-  return /\.html?$/i.test(path);
-}
-
 function resolveFilePreviewViewer(params: {
   path: string;
   viewer?: ChatFilePreviewViewer | null;
 }): "source" | "rendered" {
-  return params.viewer === "rendered" && isHtmlFilePath(params.path)
-    ? "rendered"
-    : "source";
+  return params.viewer === "rendered" && /\.html?$/i.test(params.path) ? "rendered" : "source";
+}
+
+function appendPreviewRefreshVersion(url: string, refreshVersion: number): string {
+  if (refreshVersion <= 0) {
+    return url;
+  }
+  return `${url}${url.includes("?") ? "&" : "?"}refresh=${refreshVersion}`;
 }
 
 function buildPreviewBlock(params: {
@@ -167,6 +176,7 @@ function WorkspaceHtmlRenderedPreview({
 }
 
 function WorkspacePreviewBody({
+  directoryQuery,
   onFileOpen,
   previewBlock,
   previewKind,
@@ -175,6 +185,7 @@ function WorkspacePreviewBody({
   previewUrl,
   previewViewer,
 }: {
+  directoryQuery: ReturnType<typeof useServerPathBrowse>;
   onFileOpen: (action: ChatFileOpenActionViewModel) => void;
   previewBlock: ChatFileOperationBlockViewModel | null;
   previewKind: "text" | "markdown" | "binary";
@@ -183,8 +194,26 @@ function WorkspacePreviewBody({
   previewUrl: string | null;
   previewViewer: "source" | "rendered";
 }) {
-  if (previewQuery.isLoading && !previewBlock) {
-    return <WorkspaceFilePreviewStatus text={t("chatWorkspaceLoadingFile")} />;
+  if (directoryQuery.data) {
+    return (
+      <ChatSessionWorkspaceDirectoryBrowser
+        browseQuery={directoryQuery}
+        onFileOpen={onFileOpen}
+        renderStatus={(params) => <WorkspaceFilePreviewStatus {...params} />}
+      />
+    );
+  }
+
+  if (directoryQuery.isLoading && previewQuery.error && !previewBlock) {
+    return (
+      <WorkspaceFilePreviewStatus text={t("chatWorkspaceLoadingDirectory")} />
+    );
+  }
+
+  if ((directoryQuery.isLoading || previewQuery.isLoading) && !previewBlock) {
+    return (
+      <WorkspaceFilePreviewStatus text={t("chatWorkspaceLoadingPreview")} />
+    );
   }
 
   if (previewQuery.data?.kind === "binary") {
@@ -235,6 +264,7 @@ function WorkspacePreviewBody({
 
 type ChatSessionWorkspaceFilePreviewProps = {
   file: ChatWorkspaceFileTab;
+  refreshVersion?: number;
   sessionProjectRoot: string | null;
   sessionWorkingDir: string | null;
   onFileOpen: (action: ChatFileOpenActionViewModel) => void;
@@ -242,6 +272,7 @@ type ChatSessionWorkspaceFilePreviewProps = {
 
 export function ChatSessionWorkspaceFilePreview({
   file,
+  refreshVersion = 0,
   sessionProjectRoot,
   sessionWorkingDir,
   onFileOpen,
@@ -252,6 +283,12 @@ export function ChatSessionWorkspaceFilePreview({
     basePath: sessionWorkingDir,
     enabled: isPreviewMode,
   });
+  const directoryQuery = useServerPathBrowse({
+    path: file.path,
+    basePath: sessionWorkingDir,
+    includeFiles: true,
+    enabled: isPreviewMode,
+  });
   const diffBlock = useMemo(
     () => (file.viewMode === "diff" ? buildDiffBlock(file) : null),
     [file],
@@ -262,14 +299,21 @@ export function ChatSessionWorkspaceFilePreview({
     path: previewQuery.data?.resolvedPath ?? file.path,
     serverKind: previewQuery.data?.kind,
   });
-  const resolvedPath = previewQuery.data?.resolvedPath ?? file.path;
+  const resolvedPath =
+    directoryQuery.data?.currentPath ??
+    previewQuery.data?.resolvedPath ??
+    file.path;
+  const previewPathKind = directoryQuery.data ? "directory" : "file";
   const previewViewer = resolveFilePreviewViewer({
     path: resolvedPath,
     viewer: file.previewViewer,
   });
   const previewUrl =
     previewViewer === "rendered" && previewQuery.data?.resolvedPath
-      ? buildServerPathContentUrl(previewQuery.data.resolvedPath)
+      ? appendPreviewRefreshVersion(
+          buildServerPathContentUrl(previewQuery.data.resolvedPath),
+          refreshVersion,
+        )
       : null;
   const previewBlock = useMemo(() => {
     if (!isPreviewMode || !previewText) {
@@ -295,12 +339,20 @@ export function ChatSessionWorkspaceFilePreview({
     () =>
       buildWorkspaceFileBreadcrumb({
         path: resolvedPath,
+        kind: previewPathKind,
         sessionProjectRoot: breadcrumbBasePath,
         line: file.line,
         column: file.column,
         truncated: isTruncated,
       }),
-    [breadcrumbBasePath, file.column, file.line, isTruncated, resolvedPath],
+    [
+      breadcrumbBasePath,
+      file.column,
+      file.line,
+      isTruncated,
+      previewPathKind,
+      resolvedPath,
+    ],
   );
 
   return (
@@ -315,6 +367,7 @@ export function ChatSessionWorkspaceFilePreview({
           <WorkspaceDiffBody diffBlock={diffBlock} />
         ) : (
           <WorkspacePreviewBody
+            directoryQuery={directoryQuery}
             onFileOpen={onFileOpen}
             previewBlock={previewBlock}
             previewKind={previewKind}

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

@@ -10,6 +10,7 @@ import { t } from "@/shared/lib/i18n";
 
 type ChatSessionWorkspacePanelContentProps = {
   activeSelection: WorkspaceSelection;
+  filePreviewRefreshVersion: number;
   sessionCronJobs: readonly CronJobView[];
   sessionProjectRoot: string | null;
   sessionWorkingDir: string | null;
@@ -67,6 +68,7 @@ function WorkspaceSideChatDraftHeader() {
 
 export function ChatSessionWorkspacePanelContent({
   activeSelection,
+  filePreviewRefreshVersion,
   sessionCronJobs,
   sessionProjectRoot,
   sessionWorkingDir,
@@ -113,6 +115,7 @@ export function ChatSessionWorkspacePanelContent({
     return (
       <ChatSessionWorkspaceFilePreview
         file={activeSelection.file}
+        refreshVersion={filePreviewRefreshVersion}
         sessionProjectRoot={sessionProjectRoot}
         sessionWorkingDir={sessionWorkingDir}
         onFileOpen={presenter.chatThreadManager.openFilePreview}

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

@@ -1,4 +1,15 @@
-import { AlarmClock, ArrowLeft, ArrowRight, FileCode2, MessageSquarePlus, MessageSquareText, X } from "lucide-react";
+import {
+  AlarmClock,
+  ArrowLeft,
+  ArrowRight,
+  FileCode2,
+  Maximize2,
+  MessageSquarePlus,
+  MessageSquareText,
+  Minimize2,
+  RefreshCw,
+  X,
+} from "lucide-react";
 import type { WorkspaceTabViewModel } from "@/features/chat/features/workspace/utils/chat-workspace-panel-view-model.utils";
 import { AgentIdentityAvatar } from "@/shared/components/common/agent-identity";
 import {
@@ -56,22 +67,54 @@ function buildCompactWorkspaceTabs(
 export function WorkspaceTabsBar({
   canGoBack,
   canGoForward,
+  isMaximized = false,
   onClose,
   onGoBack,
   onGoForward,
+  onRefreshFile,
+  onToggleMaximize,
   tabs,
 }: {
   canGoBack: boolean;
   canGoForward: boolean;
+  isMaximized?: boolean;
   onClose: () => void;
   onGoBack: () => void;
   onGoForward: () => void;
+  onRefreshFile?: () => void;
+  onToggleMaximize?: () => void;
   tabs: readonly WorkspaceTabViewModel[];
 }) {
   const compactTabs = buildCompactWorkspaceTabs(tabs);
   const actions: CompactTabStripAction[] = [
     { key: "back", icon: <ArrowLeft className="h-4 w-4" />, label: t("chatWorkspaceBack"), disabled: !canGoBack, onClick: onGoBack },
     { key: "forward", icon: <ArrowRight className="h-4 w-4" />, label: t("chatWorkspaceForward"), disabled: !canGoForward, onClick: onGoForward },
+    ...(onRefreshFile
+      ? [
+          {
+            key: "refresh-file",
+            icon: <RefreshCw className="h-4 w-4" />,
+            label: t("chatWorkspaceRefreshFile"),
+            onClick: onRefreshFile,
+          },
+        ]
+      : []),
+    ...(onToggleMaximize
+      ? [
+          {
+            key: "maximize",
+            icon: isMaximized ? (
+              <Minimize2 className="h-4 w-4" />
+            ) : (
+              <Maximize2 className="h-4 w-4" />
+            ),
+            label: isMaximized
+              ? t("chatWorkspaceRestorePanel")
+              : t("chatWorkspaceMaximizePanel"),
+            onClick: onToggleMaximize,
+          },
+        ]
+      : []),
     { key: "close", icon: <X className="h-4 w-4" />, label: t("chatWorkspaceClosePanel"), onClick: onClose },
   ];
 

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

@@ -1,4 +1,5 @@
-import { useEffect, useMemo } from "react";
+import { useEffect, useMemo, useState } from "react";
+import { useQueryClient } from "@tanstack/react-query";
 import type { CronJobView } from "@/shared/lib/api";
 import { useNcpChildSessionTabsView } from "@/features/chat/features/ncp/hooks/use-ncp-child-session-tabs-view";
 import { useChatSessionListStore } from "@/features/chat/stores/chat-session-list.store";
@@ -22,6 +23,8 @@ import {
   canGoBackInNavigationHistory,
   canGoForwardInNavigationHistory,
 } from "@/shared/lib/navigation-history";
+import { buildServerPathBrowseQueryKey } from "@/shared/hooks/use-server-path-browse";
+import { buildServerPathReadQueryKey } from "@/shared/hooks/use-server-path-read";
 
 type ChatSessionWorkspacePanelProps = {
   sessionKey: string | null;
@@ -55,6 +58,9 @@ export function ChatSessionWorkspacePanel({
   displayMode = "docked",
 }: ChatSessionWorkspacePanelProps) {
   const presenter = usePresenter();
+  const queryClient = useQueryClient();
+  const [isMaximized, setIsMaximized] = useState(false);
+  const [filePreviewRefreshVersion, setFilePreviewRefreshVersion] = useState(0);
   const resolvedChildTabs = useNcpChildSessionTabsView(childSessionTabs);
   const optimisticReadAtBySessionKey = useChatSessionListStore(
     (state) => state.optimisticReadAtBySessionKey,
@@ -110,30 +116,66 @@ export function ChatSessionWorkspacePanel({
     return null;
   }
 
+  const isContainerMaximized = displayMode === "docked" && isMaximized;
+  const isOverlayPanel = displayMode === "overlay" || isContainerMaximized;
+  const activeFile =
+    activeSelection.kind === "file" && activeSelection.file.viewMode === "preview"
+      ? activeSelection.file
+      : null;
+  const refreshActiveFile = activeFile
+    ? () => {
+        void queryClient.invalidateQueries({
+          queryKey: buildServerPathReadQueryKey({
+            path: activeFile.path,
+            basePath: sessionWorkingDir,
+          }),
+        });
+        void queryClient.invalidateQueries({
+          queryKey: buildServerPathBrowseQueryKey({
+            path: activeFile.path,
+            basePath: sessionWorkingDir,
+            includeFiles: true,
+          }),
+        });
+        setFilePreviewRefreshVersion((value) => value + 1);
+      }
+    : undefined;
+
   return (
     <ResizableRightPanel
+      data-testid="chat-session-workspace-panel"
       className={cn(
-        displayMode === "overlay"
+        isOverlayPanel
           ? "bg-white"
           : "hidden border-gray-200/70 bg-white/95 backdrop-blur-sm md:flex",
+        isContainerMaximized ? "shadow-xl" : null,
       )}
       defaultWidth={480}
       minWidth={360}
       maxWidth={860}
-      overlay={displayMode === "overlay"}
+      overlay={isOverlayPanel}
+      overlayScope={isContainerMaximized ? "container" : "viewport"}
     >
       <WorkspaceTabsBar
         tabs={workspaceTabs}
         canGoBack={canGoBackInNavigationHistory(workspaceHistory)}
         canGoForward={canGoForwardInNavigationHistory(workspaceHistory)}
+        isMaximized={isContainerMaximized}
         onGoBack={presenter.chatThreadManager.goBackWorkspacePanel}
         onGoForward={presenter.chatThreadManager.goForwardWorkspacePanel}
+        onRefreshFile={refreshActiveFile}
+        onToggleMaximize={
+          displayMode === "docked"
+            ? () => setIsMaximized((value) => !value)
+            : undefined
+        }
         onClose={presenter.chatThreadManager.closeWorkspacePanel}
       />
 
       <div className="flex min-h-0 flex-1 flex-col bg-white">
         <ChatSessionWorkspacePanelContent
           activeSelection={activeSelection}
+          filePreviewRefreshVersion={filePreviewRefreshVersion}
           sessionCronJobs={sessionCronJobs}
           sessionProjectRoot={sessionProjectRoot}
           sessionWorkingDir={sessionWorkingDir}

package/src/features/chat/features/workspace/utils/__tests__/chat-workspace-panel-view-model.utils.test.ts

@@ -158,4 +158,50 @@ describe("buildWorkspaceTabsViewModel", () => {
       active: true,
     });
   });
+
+  it("keeps distinct source and rendered file tabs without collapsing by path", () => {
+    const renderedTab = {
+      key: "parent::preview:rendered::demo.html",
+      parentSessionKey: "parent-1",
+      path: "demo.html",
+      label: "demo.html",
+      viewMode: "preview" as const,
+      previewViewer: "rendered" as const,
+    };
+    const sourceTab = {
+      key: "parent::preview::demo.html",
+      parentSessionKey: "parent-1",
+      path: "demo.html",
+      label: "demo.html",
+      viewMode: "preview" as const,
+      previewViewer: "source" as const,
+    };
+
+    const tabs = buildWorkspaceTabsViewModel({
+      resolvedChildTabs: [],
+      activeSideChatDraft: null,
+      workspaceFileTabs: [sourceTab, renderedTab],
+      sessionCronJobCount: 0,
+      activeSelection: {
+        kind: "file",
+        file: sourceTab,
+      },
+      optimisticReadAtBySessionKey: {},
+      onSelectSession: vi.fn(),
+      onSelectFile: vi.fn(),
+      onCloseFile: vi.fn(),
+      onSelectCronJobs: vi.fn(),
+    });
+
+    expect(tabs).toEqual([
+      expect.objectContaining({
+        key: "file:parent::preview::demo.html",
+        active: true,
+      }),
+      expect.objectContaining({
+        key: "file:parent::preview:rendered::demo.html",
+        active: false,
+      }),
+    ]);
+  });
 });

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

@@ -42,7 +42,10 @@ function resetChatSessionListManagerState() {
     snapshot: {
       ...useChatSessionListStore.getState().snapshot,
       selectedSessionKey: 'session-1',
-      listMode: 'time-first'
+      listMode: 'time-first',
+      pinnedSessionKeys: [],
+      pinnedProjectRoots: [],
+      collapsedProjectRoots: [],
     }
   });
   useChatThreadStore.setState({
@@ -191,6 +194,31 @@ describe('ChatSessionListManager list preference and read state', () => {
     });
   });
 
+  it('persists session and project list preferences through the list owner', () => {
+    const manager = new ChatSessionListManager(
+      {} as ConstructorParameters<typeof ChatSessionListManager>[0]
+    );
+
+    manager.toggleSessionPinned('session-2');
+    manager.toggleProjectPinned('/tmp/project-alpha');
+    manager.toggleProjectCollapsed('/tmp/project-alpha');
+
+    expect(useChatSessionListStore.getState().snapshot).toMatchObject({
+      pinnedSessionKeys: ['session-2'],
+      pinnedProjectRoots: ['/tmp/project-alpha'],
+      collapsedProjectRoots: ['/tmp/project-alpha'],
+    });
+    expect(persistStorage.get(chatSessionListModeStorageKey)).toMatchObject({
+      state: {
+        snapshot: {
+          pinnedSessionKeys: ['session-2'],
+          pinnedProjectRoots: ['/tmp/project-alpha'],
+          collapsedProjectRoots: ['/tmp/project-alpha'],
+        },
+      },
+    });
+  });
+
   it('marks a session as read through the session list owner boundary', () => {
     const manager = new ChatSessionListManager(
       {} as ConstructorParameters<typeof ChatSessionListManager>[0]
@@ -266,7 +294,10 @@ describe('ChatSessionListStore persistence', () => {
     useChatSessionListStore.setState({
       snapshot: {
         ...useChatSessionListStore.getState().snapshot,
-        listMode: 'time-first'
+        listMode: 'time-first',
+        pinnedSessionKeys: [],
+        pinnedProjectRoots: [],
+        collapsedProjectRoots: [],
       }
     });
   });

package/src/features/chat/managers/__tests__/chat-thread.manager.test.ts

@@ -331,6 +331,53 @@ describe('ChatThreadManager', () => {
 });
 
 describe('ChatThreadManager workspace file tabs', () => {
+  it('opens HTML previews in source mode when the viewer is omitted or auto', () => {
+    const manager = new ChatThreadManager(
+      createUiManager(),
+      {} as ConstructorParameters<typeof ChatThreadManager>[1],
+    );
+
+    manager.openFilePreview({
+      path: 'demo.html',
+      label: 'demo.html',
+      viewMode: 'preview',
+    });
+
+    expect(useChatThreadStore.getState().snapshot).toMatchObject({
+      activeWorkspaceFileKey: 'parent-session-1::preview::demo.html',
+      workspaceFileTabs: [
+        {
+          key: 'parent-session-1::preview::demo.html',
+          path: 'demo.html',
+          viewMode: 'preview',
+          previewViewer: 'source',
+        },
+      ],
+    });
+
+    manager.openFilePreview({
+      path: 'auto.html',
+      label: 'auto.html',
+      viewMode: 'preview',
+      previewViewer: 'auto',
+    });
+
+    expect(useChatThreadStore.getState().snapshot).toMatchObject({
+      activeWorkspaceFileKey: 'parent-session-1::preview::auto.html',
+      workspaceFileTabs: [
+        {
+          key: 'parent-session-1::preview::auto.html',
+          path: 'auto.html',
+          viewMode: 'preview',
+          previewViewer: 'source',
+        },
+        expect.objectContaining({
+          key: 'parent-session-1::preview::demo.html',
+        }),
+      ],
+    });
+  });
+
   it('keeps source and rendered previews for the same file as separate workspace tabs', () => {
     const manager = new ChatThreadManager(
       createUiManager(),

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

@@ -13,6 +13,17 @@ type WorkspaceChildReadState = {
   readAt?: string | null;
   runStatus?: string | null;
 };
+
+function toggleListValue(values: string[], value: string): string[] {
+  const normalizedValue = value.trim();
+  if (!normalizedValue) {
+    return values;
+  }
+  return values.includes(normalizedValue)
+    ? values.filter((item) => item !== normalizedValue)
+    : [normalizedValue, ...values];
+}
+
 export class ChatSessionListManager {
   constructor(private uiManager: ChatUiManager) {}
 
@@ -110,6 +121,27 @@ export class ChatSessionListManager {
     useChatSessionListStore.getState().setSnapshot({ listMode: value });
   };
 
+  toggleSessionPinned = (sessionKey: string) => {
+    const { pinnedSessionKeys } = useChatSessionListStore.getState().snapshot;
+    useChatSessionListStore.getState().setSnapshot({
+      pinnedSessionKeys: toggleListValue(pinnedSessionKeys, sessionKey),
+    });
+  };
+
+  toggleProjectPinned = (projectRoot: string) => {
+    const { pinnedProjectRoots } = useChatSessionListStore.getState().snapshot;
+    useChatSessionListStore.getState().setSnapshot({
+      pinnedProjectRoots: toggleListValue(pinnedProjectRoots, projectRoot),
+    });
+  };
+
+  toggleProjectCollapsed = (projectRoot: string) => {
+    const { collapsedProjectRoots } = useChatSessionListStore.getState().snapshot;
+    useChatSessionListStore.getState().setSnapshot({
+      collapsedProjectRoots: toggleListValue(collapsedProjectRoots, projectRoot),
+    });
+  };
+
   markSessionRead = (
     sessionKey: string | null | undefined,
     readAt: string | null | undefined,

package/src/features/chat/managers/chat-thread.manager.ts

@@ -111,14 +111,20 @@ export class ChatThreadManager {
       return null;
     }
     const normalizedParentSessionKey = parentSessionKey?.trim() || null;
-    const tabViewIdentity = action.viewMode === 'preview' && action.previewViewer === 'rendered' ? 'preview:rendered' : action.viewMode;
+    const previewViewer =
+      action.viewMode === 'preview' &&
+      action.previewViewer !== 'rendered' &&
+      (action.previewViewer === 'auto' || action.previewViewer === 'source' || /\.html?$/i.test(normalizedPath))
+        ? 'source'
+        : action.previewViewer ?? null;
+    const tabViewIdentity = action.viewMode === 'preview' && previewViewer === 'rendered' ? 'preview:rendered' : action.viewMode;
     return {
       key: `${normalizedParentSessionKey ?? 'draft'}::${tabViewIdentity}::${normalizedPath}`,
       parentSessionKey: normalizedParentSessionKey,
       path: normalizedPath,
       label: action.label?.trim() || null,
       viewMode: action.viewMode,
-      previewViewer: action.previewViewer ?? null,
+      previewViewer,
       line: action.line ?? null,
       column: action.column ?? null,
       rawText: action.rawText ?? null,