@nextclaw/ui 0.13.13 → 0.14.0

package/src/features/chat/managers/{ncp-chat-thread.manager.test.ts → __tests__/chat-thread.manager.test.ts}

@@ -1,6 +1,6 @@
 import { beforeEach, describe, expect, it, vi } from 'vitest';
 import { appQueryClient } from '@/app-query-client';
-import { NcpChatThreadManager } from '@/features/chat/managers/ncp-chat-thread.manager';
+import { ChatThreadManager } from '@/features/chat/managers/chat-thread.manager';
 import { useChatSessionListStore } from '@/features/chat/stores/chat-session-list.store';
 import { useChatThreadStore } from '@/features/chat/stores/chat-thread.store';
 import type * as SharedApi from '@/shared/lib/api';
@@ -49,19 +49,33 @@ beforeEach(() => {
   });
 });
 
-describe('NcpChatThreadManager', () => {
+function createUiManager(
+  overrides: Record<string, unknown> = {},
+): ConstructorParameters<typeof ChatThreadManager>[0] {
+  return {
+    goToSession: vi.fn(),
+    goToChatRoot: vi.fn(),
+    goToProviders: vi.fn(),
+    confirm: vi.fn(),
+    isCompactViewport: vi.fn(() => false),
+    showContent: vi.fn().mockResolvedValue(undefined),
+    ...overrides,
+  } as unknown as ConstructorParameters<typeof ChatThreadManager>[0];
+}
+
+describe('ChatThreadManager', () => {
   it('opens the child-session panel for the requested parent session and keeps focus on the chosen child', () => {
     const uiManager = {
       goToSession: vi.fn(),
       goToChatRoot: vi.fn(),
       goToProviders: vi.fn(),
       confirm: vi.fn(),
-    } as unknown as ConstructorParameters<typeof NcpChatThreadManager>[0];
+    } as unknown as ConstructorParameters<typeof ChatThreadManager>[0];
 
-    const manager = new NcpChatThreadManager(
+    const manager = new ChatThreadManager(
       uiManager,
-      {} as ConstructorParameters<typeof NcpChatThreadManager>[1],
-      {} as ConstructorParameters<typeof NcpChatThreadManager>[2],
+      {} as ConstructorParameters<typeof ChatThreadManager>[1],
+      {} as ConstructorParameters<typeof ChatThreadManager>[2],
     );
 
     manager.openChildSessionPanel({
@@ -86,12 +100,12 @@ describe('NcpChatThreadManager', () => {
       goToChatRoot: vi.fn(),
       goToProviders: vi.fn(),
       confirm: vi.fn(),
-    } as unknown as ConstructorParameters<typeof NcpChatThreadManager>[0];
+    } as unknown as ConstructorParameters<typeof ChatThreadManager>[0];
 
-    const manager = new NcpChatThreadManager(
+    const manager = new ChatThreadManager(
       uiManager,
-      {} as ConstructorParameters<typeof NcpChatThreadManager>[1],
-      {} as ConstructorParameters<typeof NcpChatThreadManager>[2],
+      {} as ConstructorParameters<typeof ChatThreadManager>[1],
+      {} as ConstructorParameters<typeof ChatThreadManager>[2],
     );
 
     manager.openChildSessionPanel({
@@ -108,12 +122,12 @@ describe('NcpChatThreadManager', () => {
       goToChatRoot: vi.fn(),
       goToProviders: vi.fn(),
       confirm: vi.fn(),
-    } as unknown as ConstructorParameters<typeof NcpChatThreadManager>[0];
+    } as unknown as ConstructorParameters<typeof ChatThreadManager>[0];
 
-    const manager = new NcpChatThreadManager(
+    const manager = new ChatThreadManager(
       uiManager,
-      {} as ConstructorParameters<typeof NcpChatThreadManager>[1],
-      {} as ConstructorParameters<typeof NcpChatThreadManager>[2],
+      {} as ConstructorParameters<typeof ChatThreadManager>[1],
+      {} as ConstructorParameters<typeof ChatThreadManager>[2],
     );
 
     manager.openSessionCronPanel('parent-session-1');
@@ -139,12 +153,12 @@ describe('NcpChatThreadManager', () => {
       goToChatRoot: vi.fn(),
       goToProviders: vi.fn(),
       confirm: vi.fn(),
-    } as unknown as ConstructorParameters<typeof NcpChatThreadManager>[0];
+    } as unknown as ConstructorParameters<typeof ChatThreadManager>[0];
 
-    const manager = new NcpChatThreadManager(
+    const manager = new ChatThreadManager(
       uiManager,
-      {} as ConstructorParameters<typeof NcpChatThreadManager>[1],
-      {} as ConstructorParameters<typeof NcpChatThreadManager>[2],
+      {} as ConstructorParameters<typeof ChatThreadManager>[1],
+      {} as ConstructorParameters<typeof ChatThreadManager>[2],
     );
 
     manager.openSessionCronPanel('parent-session-1');
@@ -158,12 +172,12 @@ describe('NcpChatThreadManager', () => {
       goToChatRoot: vi.fn(),
       goToProviders: vi.fn(),
       confirm: vi.fn(),
-    } as unknown as ConstructorParameters<typeof NcpChatThreadManager>[0];
+    } as unknown as ConstructorParameters<typeof ChatThreadManager>[0];
 
-    const manager = new NcpChatThreadManager(
+    const manager = new ChatThreadManager(
       uiManager,
-      {} as ConstructorParameters<typeof NcpChatThreadManager>[1],
-      {} as ConstructorParameters<typeof NcpChatThreadManager>[2],
+      {} as ConstructorParameters<typeof ChatThreadManager>[1],
+      {} as ConstructorParameters<typeof ChatThreadManager>[2],
     );
 
     manager.openFilePreview({
@@ -196,18 +210,18 @@ describe('NcpChatThreadManager', () => {
   });
 });
 
-describe('NcpChatThreadManager workspace navigation', () => {
+describe('ChatThreadManager workspace navigation', () => {
   it('navigates backward and forward through workspace selections', () => {
     const uiManager = {
       goToSession: vi.fn(),
       goToChatRoot: vi.fn(),
       goToProviders: vi.fn(),
       confirm: vi.fn(),
-    } as unknown as ConstructorParameters<typeof NcpChatThreadManager>[0];
-    const manager = new NcpChatThreadManager(
+    } as unknown as ConstructorParameters<typeof ChatThreadManager>[0];
+    const manager = new ChatThreadManager(
       uiManager,
-      {} as ConstructorParameters<typeof NcpChatThreadManager>[1],
-      {} as ConstructorParameters<typeof NcpChatThreadManager>[2],
+      {} as ConstructorParameters<typeof ChatThreadManager>[1],
+      {} as ConstructorParameters<typeof ChatThreadManager>[2],
     );
 
     manager.openChildSessionPanel({
@@ -261,10 +275,10 @@ describe('NcpChatThreadManager workspace navigation', () => {
         },
       ],
     });
-    const manager = new NcpChatThreadManager(
-      { goToSession: vi.fn() } as unknown as ConstructorParameters<typeof NcpChatThreadManager>[0],
-      {} as ConstructorParameters<typeof NcpChatThreadManager>[1],
-      {} as ConstructorParameters<typeof NcpChatThreadManager>[2],
+    const manager = new ChatThreadManager(
+      { goToSession: vi.fn() } as unknown as ConstructorParameters<typeof ChatThreadManager>[0],
+      {} as ConstructorParameters<typeof ChatThreadManager>[1],
+      {} as ConstructorParameters<typeof ChatThreadManager>[2],
     );
 
     manager.openChildSessionPanel({
@@ -287,10 +301,10 @@ describe('NcpChatThreadManager workspace navigation', () => {
   });
 
   it('removes a closed active file from workspace history and restores the previous entry', () => {
-    const manager = new NcpChatThreadManager(
-      { goToSession: vi.fn() } as unknown as ConstructorParameters<typeof NcpChatThreadManager>[0],
-      {} as ConstructorParameters<typeof NcpChatThreadManager>[1],
-      {} as ConstructorParameters<typeof NcpChatThreadManager>[2],
+    const manager = new ChatThreadManager(
+      { goToSession: vi.fn() } as unknown as ConstructorParameters<typeof ChatThreadManager>[0],
+      {} as ConstructorParameters<typeof ChatThreadManager>[1],
+      {} as ConstructorParameters<typeof ChatThreadManager>[2],
     );
 
     manager.openChildSessionPanel({
@@ -316,7 +330,246 @@ describe('NcpChatThreadManager workspace navigation', () => {
   });
 });
 
-describe('NcpChatThreadManager deletion', () => {
+describe('ChatThreadManager showContent', () => {
+  it('routes tool actions through the thread manager owner', async () => {
+    const uiManager = createUiManager({
+      goToSession: vi.fn(),
+    });
+    const manager = new ChatThreadManager(
+      uiManager,
+      {} as ConstructorParameters<typeof ChatThreadManager>[1],
+      {} as ConstructorParameters<typeof ChatThreadManager>[2],
+    );
+
+    await manager.handleToolAction({
+      kind: 'show-content',
+      label: 'Show content',
+      request: {
+        target: {
+          type: 'url',
+          payload: {
+            url: 'https://example.com/read',
+          },
+        },
+        title: 'Example URL',
+      },
+    });
+    await manager.handleToolAction({
+      kind: 'open-session',
+      sessionId: 'child-session-1',
+      sessionKind: 'session',
+    });
+
+    expect(uiManager.showContent).toHaveBeenCalledWith({
+      target: {
+        type: 'url',
+        payload: {
+          url: 'https://example.com/read',
+        },
+      },
+      title: 'Example URL',
+    });
+    expect(uiManager.goToSession).toHaveBeenCalledWith('child-session-1');
+  });
+
+  it('shows file content through the existing workspace file preview', async () => {
+    const manager = new ChatThreadManager(
+      { goToSession: vi.fn() } as unknown as ConstructorParameters<typeof ChatThreadManager>[0],
+      {} as ConstructorParameters<typeof ChatThreadManager>[1],
+      {} as ConstructorParameters<typeof ChatThreadManager>[2],
+    );
+
+    await manager.handleToolAction({
+      kind: 'show-content',
+      label: 'Show content',
+      request: {
+        target: {
+          type: 'file',
+          payload: {
+            path: 'docs/example.md',
+            line: 5,
+          },
+        },
+        title: 'Example',
+        purpose: 'read',
+      },
+    });
+
+    expect(useChatThreadStore.getState().snapshot).toMatchObject({
+      activeWorkspacePanelKind: 'file',
+      activeWorkspaceFileKey: 'parent-session-1::preview::docs/example.md',
+      workspaceFileTabs: [
+        expect.objectContaining({
+          path: 'docs/example.md',
+          label: 'Example',
+          line: 5,
+          viewMode: 'preview',
+        }),
+      ],
+    });
+  });
+
+  it('shows URL and panel app content through DocBrowser', async () => {
+    const uiManager = createUiManager();
+    const manager = new ChatThreadManager(
+      uiManager,
+      {} as ConstructorParameters<typeof ChatThreadManager>[1],
+      {} as ConstructorParameters<typeof ChatThreadManager>[2],
+    );
+
+    await manager.handleToolAction({
+      kind: 'show-content',
+      label: 'Show content',
+      request: {
+        target: {
+          type: 'url',
+          payload: {
+            url: 'https://example.com/read',
+          },
+        },
+        title: 'Example URL',
+      },
+    });
+    await manager.handleToolAction({
+      kind: 'show-content',
+      label: 'Show content',
+      request: {
+        target: {
+          type: 'panel_app',
+          payload: {
+            appId: 'reader',
+          },
+        },
+        title: 'Reader',
+      },
+    });
+
+    expect(uiManager.showContent).toHaveBeenCalledWith({
+      target: {
+        type: 'url',
+        payload: {
+          url: 'https://example.com/read',
+        },
+      },
+      title: 'Example URL',
+    });
+    expect(uiManager.showContent).toHaveBeenCalledWith({
+      target: {
+        type: 'panel_app',
+        payload: {
+          appId: 'reader',
+        },
+      },
+      title: 'Reader',
+    });
+  });
+
+  it('handles ui.show-content events once through the same owner path', async () => {
+    const uiManager = createUiManager();
+    const manager = new ChatThreadManager(
+      uiManager,
+      {} as ConstructorParameters<typeof ChatThreadManager>[1],
+      {} as ConstructorParameters<typeof ChatThreadManager>[2],
+    );
+
+    await manager.handleUiShowContentEvent({
+      id: 'tool:call-show-content-1:show-content',
+      toolCallId: 'call-show-content-1',
+      target: {
+        type: 'url',
+        payload: {
+          url: 'https://example.com/read',
+        },
+      },
+      title: 'Example URL',
+      purpose: 'read',
+    });
+    await manager.handleUiShowContentEvent({
+      id: 'tool:call-show-content-1:show-content',
+      toolCallId: 'call-show-content-1',
+      target: {
+        type: 'url',
+        payload: {
+          url: 'https://example.com/read',
+        },
+      },
+      title: 'Example URL',
+      purpose: 'read',
+    });
+
+    expect(uiManager.showContent).toHaveBeenCalledTimes(1);
+    expect(uiManager.showContent).toHaveBeenCalledWith({
+      target: {
+        type: 'url',
+        payload: {
+          url: 'https://example.com/read',
+        },
+      },
+      title: 'Example URL',
+    });
+  });
+
+  it('delegates panel app content to the UI manager owner', async () => {
+    const uiManager = createUiManager();
+    const manager = new ChatThreadManager(
+      uiManager,
+      {} as ConstructorParameters<typeof ChatThreadManager>[1],
+      {} as ConstructorParameters<typeof ChatThreadManager>[2],
+    );
+
+    await manager.handleToolAction({
+      kind: 'show-content',
+      label: 'Show content',
+      request: {
+        target: {
+          type: 'panel_app',
+          payload: {
+            appId: 'encoded-reader',
+          },
+        },
+        title: 'Reader',
+      },
+    });
+
+    expect(uiManager.showContent).toHaveBeenCalledWith({
+      target: {
+        type: 'panel_app',
+        payload: {
+          appId: 'encoded-reader',
+        },
+      },
+      title: 'Reader',
+    });
+  });
+});
+
+describe('ChatThreadManager visible workspace selection', () => {
+  it('delegates visible child-session read state to the session list owner', () => {
+    const sessionListManager = {
+      markVisibleWorkspaceChildRead: vi.fn(),
+    } as unknown as ConstructorParameters<typeof ChatThreadManager>[1];
+    const manager = new ChatThreadManager(
+      { goToSession: vi.fn() } as unknown as ConstructorParameters<typeof ChatThreadManager>[0],
+      sessionListManager,
+      {} as ConstructorParameters<typeof ChatThreadManager>[2],
+    );
+    const tab = {
+      sessionKey: 'child-session-1',
+      lastMessageAt: '2026-04-10T10:00:00.000Z',
+      readAt: null,
+      runStatus: 'completed',
+    };
+
+    manager.syncVisibleWorkspaceSelection({ kind: 'child-session', tab });
+    manager.syncVisibleWorkspaceSelection({ kind: 'file' });
+    manager.syncVisibleWorkspaceSelection(null);
+
+    expect(sessionListManager.markVisibleWorkspaceChildRead).toHaveBeenCalledTimes(1);
+    expect(sessionListManager.markVisibleWorkspaceChildRead).toHaveBeenCalledWith(tab);
+  });
+});
+
+describe('ChatThreadManager deletion', () => {
   it('clears the selected thread state after deleting the current session', async () => {
     const removeQueries = vi.spyOn(appQueryClient, 'removeQueries').mockImplementation(async () => undefined);
     const uiManager = {
@@ -324,24 +577,24 @@ describe('NcpChatThreadManager deletion', () => {
       goToChatRoot: vi.fn(),
       goToProviders: vi.fn(),
       confirm: vi.fn(async () => true),
-    } as unknown as ConstructorParameters<typeof NcpChatThreadManager>[0];
+    } as unknown as ConstructorParameters<typeof ChatThreadManager>[0];
     const sessionListManager = {
       setSelectedSessionKey: vi.fn((value: string | null) => {
         useChatSessionListStore.getState().setSnapshot({
           selectedSessionKey: value,
         });
       }),
-    } as unknown as ConstructorParameters<typeof NcpChatThreadManager>[1];
-    const streamActionsManager = {
-      resetStreamState: vi.fn(),
-    } as unknown as ConstructorParameters<typeof NcpChatThreadManager>[2];
-    const manager = new NcpChatThreadManager(
+    } as unknown as ConstructorParameters<typeof ChatThreadManager>[1];
+    const chatRunManager = {
+      clearRunState: vi.fn(),
+    } as unknown as ConstructorParameters<typeof ChatThreadManager>[2];
+    const manager = new ChatThreadManager(
       uiManager,
       sessionListManager,
-      streamActionsManager,
+      chatRunManager,
     );
 
-    await (manager as unknown as { deleteCurrentSession: () => Promise<void> }).deleteCurrentSession();
+    await manager.deleteSession();
 
     expect(sessionListManager.setSelectedSessionKey).toHaveBeenCalledWith(null);
     expect(useChatSessionListStore.getState().snapshot.selectedSessionKey).toBeNull();
@@ -357,7 +610,7 @@ describe('NcpChatThreadManager deletion', () => {
       workspaceNavigationHistory: [],
       workspaceNavigationHistoryIndex: 0,
     });
-    expect(streamActionsManager.resetStreamState).toHaveBeenCalledTimes(1);
+    expect(chatRunManager.clearRunState).toHaveBeenCalledTimes(1);
     expect(deleteSummaryMock).toHaveBeenCalledWith(appQueryClient, 'parent-session-1');
     expect(removeQueries).toHaveBeenCalledWith({
       queryKey: ['ncp-session-messages', 'parent-session-1'],

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

@@ -0,0 +1,126 @@
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+import { ChatUiManager } from '@/features/chat/managers/chat-ui.manager';
+import type { PanelAppEntryView } from '@/shared/lib/api';
+import type * as SharedApi from '@/shared/lib/api';
+
+const { listPanelAppsMock } = vi.hoisted(() => ({
+  listPanelAppsMock: vi.fn(),
+}));
+
+vi.mock('@/shared/lib/api', async (importOriginal) => {
+  const actual = await importOriginal<typeof SharedApi>();
+  return {
+    ...actual,
+    nextclawClient: {
+      panelApps: {
+        listPanelApps: listPanelAppsMock,
+      },
+    },
+  };
+});
+
+function createDocBrowserManager() {
+  return {
+    open: vi.fn(),
+    openTarget: vi.fn(),
+  };
+}
+
+function createPanelAppEntry(overrides: Partial<PanelAppEntryView> = {}): PanelAppEntryView {
+  return {
+    appId: 'reader',
+    clientDeclared: false,
+    clientGranted: false,
+    contentPath: '/api/panel-apps/encoded-reader/content',
+    createdAt: '2026-06-09T00:00:00.000Z',
+    favorite: false,
+    fileName: 'reader.panel',
+    id: 'encoded-reader',
+    kind: 'folder',
+    openCount: 0,
+    sizeBytes: 1024,
+    title: 'Reader',
+    updatedAt: '2026-06-09T00:00:00.000Z',
+    ...overrides,
+  };
+}
+
+describe('ChatUiManager content display', () => {
+  beforeEach(() => {
+    listPanelAppsMock.mockReset();
+    listPanelAppsMock.mockResolvedValue({ entries: [] });
+  });
+
+  it('shows URL content through DocBrowser', async () => {
+    const docBrowserManager = createDocBrowserManager();
+    const manager = new ChatUiManager(
+      docBrowserManager as unknown as ConstructorParameters<typeof ChatUiManager>[0],
+    );
+
+    await manager.showContent({
+      target: {
+        type: 'url',
+        payload: {
+          url: 'https://example.com/read',
+        },
+      },
+      title: 'Example URL',
+    });
+
+    expect(docBrowserManager.open).toHaveBeenCalledWith('https://example.com/read', {
+      title: 'Example URL',
+    });
+  });
+
+  it('opens a listed panel app as a managed DocBrowser target', async () => {
+    listPanelAppsMock.mockResolvedValue({
+      entries: [createPanelAppEntry()],
+    });
+    const docBrowserManager = createDocBrowserManager();
+    const manager = new ChatUiManager(
+      docBrowserManager as unknown as ConstructorParameters<typeof ChatUiManager>[0],
+    );
+
+    await manager.showContent({
+      target: {
+        type: 'panel_app',
+        payload: {
+          appId: 'reader',
+        },
+      },
+      title: 'Reader',
+    });
+
+    expect(docBrowserManager.openTarget).toHaveBeenCalledWith({
+      dedupeKey: 'panel-app:encoded-reader',
+      historyPolicy: 'managed',
+      kind: 'panel-app',
+      resourceUri: 'nextclaw://panel-app/encoded-reader',
+      title: 'Reader',
+      url: '/api/panel-apps/encoded-reader/content',
+    }, {
+      title: 'Reader',
+    });
+  });
+
+  it('falls back to the panel app resource URI when no listed entry matches', async () => {
+    const docBrowserManager = createDocBrowserManager();
+    const manager = new ChatUiManager(
+      docBrowserManager as unknown as ConstructorParameters<typeof ChatUiManager>[0],
+    );
+
+    await manager.showContent({
+      target: {
+        type: 'panel_app',
+        payload: {
+          appId: 'encoded-reader',
+        },
+      },
+      title: 'Reader',
+    });
+
+    expect(docBrowserManager.open).toHaveBeenCalledWith('nextclaw://panel-app/encoded-reader', {
+      title: 'Reader',
+    });
+  });
+});

package/src/features/chat/managers/{recent-selection.manager.test.ts → __tests__/recent-selection.manager.test.ts}

@@ -1,4 +1,4 @@
-import { RecentSelectionManager } from './recent-selection.manager';
+import { RecentSelectionManager } from '../recent-selection.manager';
 
 describe('RecentSelectionManager', () => {
   const storageKey = 'test.recent-selection-manager';