@nextclaw/ui 0.15.23 → 0.15.25

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

@@ -6,7 +6,10 @@ import { useChatThreadStore } from '@/features/chat/stores/chat-thread.store';
 import type * as SharedApi from '@/shared/lib/api';
 
 const { deleteNcpSessionMock, deleteSummaryMock } = vi.hoisted(() => ({
-  deleteNcpSessionMock: vi.fn(async () => ({ deleted: true, sessionId: 'parent-session-1' })),
+  deleteNcpSessionMock: vi.fn(async () => ({
+    deleted: true,
+    sessionId: 'parent-session-1',
+  })),
   deleteSummaryMock: vi.fn(),
 }));
 const persistStorage = new Map<string, unknown>();
@@ -34,8 +37,12 @@ vi.mock('@/shared/lib/api', async (importOriginal) => {
 
 beforeEach(() => {
   persistStorage.clear();
-  useChatSessionListStore.persist.setOptions({ storage: createPersistStorage() as never });
-  useChatThreadStore.persist.setOptions({ storage: createPersistStorage() as never });
+  useChatSessionListStore.persist.setOptions({
+    storage: createPersistStorage() as never,
+  });
+  useChatThreadStore.persist.setOptions({
+    storage: createPersistStorage() as never,
+  });
   useChatSessionListStore.setState({
     optimisticReadAtBySessionKey: {},
     snapshot: {
@@ -67,9 +74,7 @@ beforeEach(() => {
   });
 });
 
-function createUiManager(
-  overrides: Record<string, unknown> = {},
-): ConstructorParameters<typeof ChatThreadManager>[0] {
+function createUiManager(overrides: Record<string, unknown> = {}): ConstructorParameters<typeof ChatThreadManager>[0] {
   return {
     goToSession: vi.fn(),
     goToChatRoot: vi.fn(),
@@ -90,10 +95,7 @@ describe('ChatThreadManager', () => {
       confirm: vi.fn(),
     } as unknown as ConstructorParameters<typeof ChatThreadManager>[0];
 
-    const manager = new ChatThreadManager(
-      uiManager,
-      {} as ConstructorParameters<typeof ChatThreadManager>[1],
-    );
+    const manager = new ChatThreadManager(uiManager, {} as ConstructorParameters<typeof ChatThreadManager>[1]);
 
     manager.openChildSessionPanel({
       parentSessionKey: 'parent-session-1',
@@ -119,10 +121,7 @@ describe('ChatThreadManager', () => {
       confirm: vi.fn(),
     } as unknown as ConstructorParameters<typeof ChatThreadManager>[0];
 
-    const manager = new ChatThreadManager(
-      uiManager,
-      {} as ConstructorParameters<typeof ChatThreadManager>[1],
-    );
+    const manager = new ChatThreadManager(uiManager, {} as ConstructorParameters<typeof ChatThreadManager>[1]);
 
     manager.openChildSessionPanel({
       parentSessionKey: 'parent-session-1',
@@ -140,10 +139,7 @@ describe('ChatThreadManager', () => {
       confirm: vi.fn(),
     } as unknown as ConstructorParameters<typeof ChatThreadManager>[0];
 
-    const manager = new ChatThreadManager(
-      uiManager,
-      {} as ConstructorParameters<typeof ChatThreadManager>[1],
-    );
+    const manager = new ChatThreadManager(uiManager, {} as ConstructorParameters<typeof ChatThreadManager>[1]);
 
     manager.openSessionCronPanel('parent-session-1');
 
@@ -194,10 +190,7 @@ describe('ChatThreadManager', () => {
     const uiManager = createUiManager({
       goToSession: vi.fn(),
     });
-    const manager = new ChatThreadManager(
-      uiManager,
-      {} as ConstructorParameters<typeof ChatThreadManager>[1],
-    );
+    const manager = new ChatThreadManager(uiManager, {} as ConstructorParameters<typeof ChatThreadManager>[1]);
 
     manager.openSideChatDraft('parent-session-1');
 
@@ -223,10 +216,7 @@ describe('ChatThreadManager', () => {
   });
 
   it('materializes an active side chat draft into a real child-session tab', () => {
-    const manager = new ChatThreadManager(
-      createUiManager(),
-      {} as ConstructorParameters<typeof ChatThreadManager>[1],
-    );
+    const manager = new ChatThreadManager(createUiManager(), {} as ConstructorParameters<typeof ChatThreadManager>[1]);
 
     manager.openSideChatDraft('parent-session-1');
     const draftKey = useChatThreadStore.getState().snapshot.activeSideChatDraft?.draftKey;
@@ -256,9 +246,7 @@ describe('ChatThreadManager', () => {
           sessionKey: 'child-session-1',
         },
       ],
-      workspaceNavigationHistory: [
-        { kind: 'child-session', key: 'child-session-side' },
-      ],
+      workspaceNavigationHistory: [{ kind: 'child-session', key: 'child-session-side' }],
       workspaceNavigationHistoryIndex: 0,
     });
   });
@@ -277,10 +265,7 @@ describe('ChatThreadManager', () => {
       confirm: vi.fn(),
     } as unknown as ConstructorParameters<typeof ChatThreadManager>[0];
 
-    const manager = new ChatThreadManager(
-      uiManager,
-      {} as ConstructorParameters<typeof ChatThreadManager>[1],
-    );
+    const manager = new ChatThreadManager(uiManager, {} as ConstructorParameters<typeof ChatThreadManager>[1]);
 
     manager.openSessionCronPanel('parent-session-1');
 
@@ -295,10 +280,7 @@ describe('ChatThreadManager', () => {
       confirm: vi.fn(),
     } as unknown as ConstructorParameters<typeof ChatThreadManager>[0];
 
-    const manager = new ChatThreadManager(
-      uiManager,
-      {} as ConstructorParameters<typeof ChatThreadManager>[1],
-    );
+    const manager = new ChatThreadManager(uiManager, {} as ConstructorParameters<typeof ChatThreadManager>[1]);
 
     manager.openFilePreview({
       path: 'chart.html',
@@ -331,39 +313,40 @@ describe('ChatThreadManager', () => {
       ]),
     );
   });
-
 });
 
 describe('ChatThreadManager workspace file tabs', () => {
   it.each([
     { path: 'demo.html', previewViewer: undefined, expectedViewer: 'source' },
-    { path: 'auto.html', previewViewer: 'auto' as const, expectedViewer: 'source' },
+    {
+      path: 'auto.html',
+      previewViewer: 'auto' as const,
+      expectedViewer: 'source',
+    },
     { path: 'README.md', previewViewer: undefined, expectedViewer: 'rendered' },
-    { path: 'report.docx', previewViewer: 'auto' as const, expectedViewer: 'auto' },
+    {
+      path: 'report.docx',
+      previewViewer: 'auto' as const,
+      expectedViewer: 'auto',
+    },
   ])('normalizes the viewer for $path', ({ path, previewViewer, expectedViewer }) => {
-      const manager = new ChatThreadManager(
-        createUiManager(),
-        {} as ConstructorParameters<typeof ChatThreadManager>[1],
-      );
-
-      manager.openFilePreview({
-        path,
-        label: path,
-        viewMode: 'preview',
-        previewViewer,
-      });
-
-      expect(useChatThreadStore.getState().snapshot.workspaceFileTabs[0]).toMatchObject({
-        path,
-        previewViewer: expectedViewer,
-      });
+    const manager = new ChatThreadManager(createUiManager(), {} as ConstructorParameters<typeof ChatThreadManager>[1]);
+
+    manager.openFilePreview({
+      path,
+      label: path,
+      viewMode: 'preview',
+      previewViewer,
+    });
+
+    expect(useChatThreadStore.getState().snapshot.workspaceFileTabs[0]).toMatchObject({
+      path,
+      previewViewer: expectedViewer,
+    });
   });
 
   it('opens source and rendered previews for the same file as adjacent workspace tabs', () => {
-    const manager = new ChatThreadManager(
-      createUiManager(),
-      {} as ConstructorParameters<typeof ChatThreadManager>[1],
-    );
+    const manager = new ChatThreadManager(createUiManager(), {} as ConstructorParameters<typeof ChatThreadManager>[1]);
 
     manager.openFilePreview({
       path: 'demo.html',
@@ -403,6 +386,56 @@ describe('ChatThreadManager workspace file tabs', () => {
       ]),
     });
   });
+
+  it('remaps open file tabs and navigation history after a file or folder rename', () => {
+    const manager = new ChatThreadManager(createUiManager(), {} as ConstructorParameters<typeof ChatThreadManager>[1]);
+    manager.openFilePreview({
+      path: '/workspace/docs/guide.md',
+      label: 'guide.md',
+      viewMode: 'preview',
+    });
+    const previousKey = useChatThreadStore.getState().snapshot.activeWorkspaceFileKey!;
+
+    manager.renameWorkspacePath({
+      previousPath: '/workspace/docs',
+      path: '/workspace/designs',
+      label: 'designs',
+    });
+
+    const { snapshot } = useChatThreadStore.getState();
+    expect(snapshot.workspaceFileTabs[0]).toMatchObject({
+      path: '/workspace/designs/guide.md',
+      label: 'guide.md',
+    });
+    expect(snapshot.activeWorkspaceFileKey).toBe('parent-session-1::preview:rendered::/workspace/designs/guide.md');
+    expect(snapshot.workspaceNavigationHistory).toContainEqual({
+      kind: 'file',
+      key: snapshot.activeWorkspaceFileKey,
+    });
+    expect(snapshot.activeWorkspaceFileKey).not.toBe(previousKey);
+  });
+
+  it('closes every open tab below a deleted path and restores the prior file', () => {
+    const manager = new ChatThreadManager(createUiManager(), {} as ConstructorParameters<typeof ChatThreadManager>[1]);
+    manager.openFilePreview({
+      path: '/workspace/README.md',
+      label: 'README.md',
+      viewMode: 'preview',
+    });
+    manager.openFilePreview({
+      path: '/workspace/docs/guide.md',
+      label: 'guide.md',
+      viewMode: 'preview',
+    });
+
+    manager.removeWorkspacePath('/workspace/docs');
+
+    expect(useChatThreadStore.getState().snapshot).toMatchObject({
+      activeWorkspacePanelKind: 'file',
+      activeWorkspaceFileKey: 'parent-session-1::preview:rendered::/workspace/README.md',
+      workspaceFileTabs: [expect.objectContaining({ path: '/workspace/README.md' })],
+    });
+  });
 });
 
 describe('ChatThreadManager workspace navigation', () => {
@@ -413,10 +446,7 @@ describe('ChatThreadManager workspace navigation', () => {
       goToProviders: vi.fn(),
       confirm: vi.fn(),
     } as unknown as ConstructorParameters<typeof ChatThreadManager>[0];
-    const manager = new ChatThreadManager(
-      uiManager,
-      {} as ConstructorParameters<typeof ChatThreadManager>[1],
-    );
+    const manager = new ChatThreadManager(uiManager, {} as ConstructorParameters<typeof ChatThreadManager>[1]);
 
     manager.openChildSessionPanel({
       parentSessionKey: 'parent-session-1',
@@ -517,9 +547,7 @@ describe('ChatThreadManager workspace navigation', () => {
       activeWorkspacePanelKind: 'child-session',
       activeChildSessionKey: 'child-session-1',
       activeWorkspaceFileKey: null,
-      workspaceNavigationHistory: [
-        { kind: 'child-session', key: 'child-session-1' },
-      ],
+      workspaceNavigationHistory: [{ kind: 'child-session', key: 'child-session-1' }],
       workspaceNavigationHistoryIndex: 0,
     });
   });
@@ -535,10 +563,7 @@ describe('ChatThreadManager tool actions', () => {
     const uiManager = createUiManager({
       goToSession: vi.fn(),
     });
-    const manager = new ChatThreadManager(
-      uiManager,
-      {} as ConstructorParameters<typeof ChatThreadManager>[1],
-    );
+    const manager = new ChatThreadManager(uiManager, {} as ConstructorParameters<typeof ChatThreadManager>[1]);
 
     await manager.handleToolAction({
       kind: 'open-session',
@@ -562,9 +587,7 @@ describe('ChatThreadManager tool actions', () => {
           agentId: 'verifier-agent',
         },
       ],
-      workspaceNavigationHistory: [
-        { kind: 'child-session', key: 'child-session-9' },
-      ],
+      workspaceNavigationHistory: [{ kind: 'child-session', key: 'child-session-9' }],
     });
     expect(uiManager.goToSession).not.toHaveBeenCalled();
   });
@@ -575,10 +598,7 @@ describe('ChatThreadManager showContent', () => {
     const uiManager = createUiManager({
       goToSession: vi.fn(),
     });
-    const manager = new ChatThreadManager(
-      uiManager,
-      {} as ConstructorParameters<typeof ChatThreadManager>[1],
-    );
+    const manager = new ChatThreadManager(uiManager, {} as ConstructorParameters<typeof ChatThreadManager>[1]);
 
     await manager.handleToolAction({
       kind: 'show-content',
@@ -652,10 +672,7 @@ describe('ChatThreadManager showContent', () => {
 
   it('shows URL and panel app content through DocBrowser', async () => {
     const uiManager = createUiManager();
-    const manager = new ChatThreadManager(
-      uiManager,
-      {} as ConstructorParameters<typeof ChatThreadManager>[1],
-    );
+    const manager = new ChatThreadManager(uiManager, {} as ConstructorParameters<typeof ChatThreadManager>[1]);
 
     await manager.handleToolAction({
       kind: 'show-content',
@@ -706,10 +723,7 @@ describe('ChatThreadManager showContent', () => {
 
   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],
-    );
+    const manager = new ChatThreadManager(uiManager, {} as ConstructorParameters<typeof ChatThreadManager>[1]);
 
     await manager.handleUiShowContentEvent({
       id: 'tool:call-show-content-1:show-content',
@@ -752,10 +766,7 @@ describe('ChatThreadManager showContent', () => {
 
   it('delegates panel app content to the UI manager owner', async () => {
     const uiManager = createUiManager();
-    const manager = new ChatThreadManager(
-      uiManager,
-      {} as ConstructorParameters<typeof ChatThreadManager>[1],
-    );
+    const manager = new ChatThreadManager(uiManager, {} as ConstructorParameters<typeof ChatThreadManager>[1]);
 
     await manager.handleToolAction({
       kind: 'show-content',
@@ -781,16 +792,12 @@ describe('ChatThreadManager showContent', () => {
       title: 'Reader',
     });
   });
-
 });
 
 describe('ChatThreadManager inline showContent', () => {
   it('does not auto-open inline panel app content outside the message card', async () => {
     const uiManager = createUiManager();
-    const manager = new ChatThreadManager(
-      uiManager,
-      {} as ConstructorParameters<typeof ChatThreadManager>[1],
-    );
+    const manager = new ChatThreadManager(uiManager, {} as ConstructorParameters<typeof ChatThreadManager>[1]);
 
     await manager.handleUiShowContentEvent({
       id: 'tool:call-show-content-inline:show-content',
@@ -845,10 +852,7 @@ describe('ChatThreadManager deletion', () => {
       confirm: vi.fn(async () => true),
     } as unknown as ConstructorParameters<typeof ChatThreadManager>[0];
     const sessionListManager = {} as ConstructorParameters<typeof ChatThreadManager>[1];
-    const manager = new ChatThreadManager(
-      uiManager,
-      sessionListManager,
-    );
+    const manager = new ChatThreadManager(uiManager, sessionListManager);
 
     await manager.deleteSession();
 

package/src/features/chat/managers/chat-composer-intent.manager.ts

@@ -1,7 +1,9 @@
 import {
+  CHAT_CONVERSATION_EXCERPT_TOKEN_KIND,
+  CHAT_WORKSPACE_DIRECTORY_TOKEN_KIND,
   CHAT_WORKSPACE_EXCERPT_TOKEN_KIND,
   CHAT_WORKSPACE_FILE_TOKEN_KIND,
-} from "@nextclaw/shared";
+} from '@nextclaw/shared';
 
 type ChatComposerReferenceIntentBase = {
   id: number;
@@ -14,6 +16,10 @@ export type ChatComposerFileReferenceIntent = ChatComposerReferenceIntentBase &
   kind: typeof CHAT_WORKSPACE_FILE_TOKEN_KIND;
 };
 
+export type ChatComposerDirectoryReferenceIntent = ChatComposerReferenceIntentBase & {
+  kind: typeof CHAT_WORKSPACE_DIRECTORY_TOKEN_KIND;
+};
+
 export type ChatComposerExcerptReferenceIntent = ChatComposerReferenceIntentBase & {
   kind: typeof CHAT_WORKSPACE_EXCERPT_TOKEN_KIND;
   path: string;
@@ -22,13 +28,33 @@ export type ChatComposerExcerptReferenceIntent = ChatComposerReferenceIntentBase
   endLine: number | null;
 };
 
+export type ChatComposerConversationExcerptIntent = ChatComposerReferenceIntentBase & {
+  kind: typeof CHAT_CONVERSATION_EXCERPT_TOKEN_KIND;
+  messageId: string;
+  role: 'assistant' | 'user';
+  excerpt: string;
+};
+
 export type ChatComposerReferenceIntent =
   | ChatComposerFileReferenceIntent
-  | ChatComposerExcerptReferenceIntent;
+  | ChatComposerDirectoryReferenceIntent
+  | ChatComposerExcerptReferenceIntent
+  | ChatComposerConversationExcerptIntent;
 
 type ChatComposerReferenceRequest =
-  | Omit<ChatComposerFileReferenceIntent, "id">
-  | Omit<ChatComposerExcerptReferenceIntent, "id">;
+  | Omit<ChatComposerFileReferenceIntent, 'id'>
+  | Omit<ChatComposerDirectoryReferenceIntent, 'id'>
+  | Omit<ChatComposerExcerptReferenceIntent, 'id'>
+  | Omit<ChatComposerConversationExcerptIntent, 'id'>;
+
+function createExcerptTokenKey(prefix: string, identity: string): string {
+  let hash = 2166136261;
+  for (let index = 0; index < identity.length; index += 1) {
+    hash ^= identity.charCodeAt(index);
+    hash = Math.imul(hash, 16777619);
+  }
+  return `${prefix}#excerpt-${(hash >>> 0).toString(36)}`;
+}
 
 type ChatComposerIntentListener = (intent: ChatComposerReferenceIntent) => void;
 
@@ -45,9 +71,11 @@ export class ChatComposerIntentManager {
   private publish = (intent: ChatComposerReferenceRequest) => {
     const id = this.nextId + 1;
     const targetSessionKey = intent.targetSessionKey?.trim() || null;
-    const nextIntent: ChatComposerReferenceIntent = intent.kind === CHAT_WORKSPACE_FILE_TOKEN_KIND
-      ? { ...intent, id, targetSessionKey }
-      : { ...intent, id, targetSessionKey };
+    const nextIntent: ChatComposerReferenceIntent = {
+      ...intent,
+      id,
+      targetSessionKey,
+    };
     this.nextId = nextIntent.id;
     this.pendingIntent = nextIntent;
     this.subscriptions.forEach((subscription) => {
@@ -57,11 +85,7 @@ export class ChatComposerIntentManager {
     });
   };
 
-  requestFileReference = (params: {
-    targetSessionKey: string | null;
-    tokenKey: string;
-    label: string;
-  }) => {
+  requestFileReference = (params: { targetSessionKey: string | null; tokenKey: string; label: string }) => {
     const tokenKey = params.tokenKey.trim();
     const label = params.label.trim();
     if (!tokenKey || !label) {
@@ -75,6 +99,20 @@ export class ChatComposerIntentManager {
     });
   };
 
+  requestDirectoryReference = (params: { targetSessionKey: string | null; tokenKey: string; label: string }) => {
+    const tokenKey = params.tokenKey.trim();
+    const label = params.label.trim();
+    if (!tokenKey || !label) {
+      return;
+    }
+    this.publish({
+      kind: CHAT_WORKSPACE_DIRECTORY_TOKEN_KIND,
+      targetSessionKey: params.targetSessionKey?.trim() || null,
+      tokenKey,
+      label,
+    });
+  };
+
   requestExcerptReference = (params: {
     targetSessionKey: string | null;
     path: string;
@@ -83,30 +121,17 @@ export class ChatComposerIntentManager {
     startLine: number | null;
     endLine: number | null;
   }) => {
-    const {
-      endLine,
-      excerpt: rawExcerpt,
-      label: rawLabel,
-      path: rawPath,
-      startLine,
-      targetSessionKey,
-    } = params;
+    const { endLine, excerpt: rawExcerpt, label: rawLabel, path: rawPath, startLine, targetSessionKey } = params;
     const path = rawPath.trim();
     const label = rawLabel.trim();
     const excerpt = rawExcerpt.trim();
     if (!path || !label || !excerpt) {
       return;
     }
-    const identity = `${path}:${startLine ?? "x"}:${endLine ?? "x"}:${excerpt}`;
-    let hash = 2166136261;
-    for (let index = 0; index < identity.length; index += 1) {
-      hash ^= identity.charCodeAt(index);
-      hash = Math.imul(hash, 16777619);
-    }
     this.publish({
       kind: CHAT_WORKSPACE_EXCERPT_TOKEN_KIND,
       targetSessionKey,
-      tokenKey: `${path}#excerpt-${(hash >>> 0).toString(36)}`,
+      tokenKey: createExcerptTokenKey(path, `${path}:${startLine ?? 'x'}:${endLine ?? 'x'}:${excerpt}`),
       path,
       label,
       excerpt,
@@ -115,9 +140,36 @@ export class ChatComposerIntentManager {
     });
   };
 
-  consumePending = (
-    targetSessionKey: string | null,
-  ): ChatComposerReferenceIntent | null => {
+  requestConversationExcerptReference = (params: {
+    targetSessionKey: string | null;
+    messageId: string;
+    role: 'assistant' | 'user';
+    label: string;
+    excerpt: string;
+  }) => {
+    const {
+      targetSessionKey,
+      messageId: requestedMessageId,
+      role,
+      label: requestedLabel,
+      excerpt: requestedExcerpt,
+    } = params;
+    const messageId = requestedMessageId.trim();
+    const label = requestedLabel.trim();
+    const excerpt = requestedExcerpt.trim();
+    if (!messageId || !label || !excerpt) return;
+    this.publish({
+      kind: CHAT_CONVERSATION_EXCERPT_TOKEN_KIND,
+      targetSessionKey,
+      tokenKey: createExcerptTokenKey(messageId, `${messageId}:${excerpt}`),
+      messageId,
+      role,
+      label,
+      excerpt,
+    });
+  };
+
+  consumePending = (targetSessionKey: string | null): ChatComposerReferenceIntent | null => {
     if (this.pendingIntent?.targetSessionKey !== targetSessionKey) {
       return null;
     }
@@ -132,10 +184,7 @@ export class ChatComposerIntentManager {
     }
   };
 
-  subscribe = (
-    targetSessionKey: string | null,
-    listener: ChatComposerIntentListener,
-  ) => {
+  subscribe = (targetSessionKey: string | null, listener: ChatComposerIntentListener) => {
     const subscription = { targetSessionKey, listener };
     this.subscriptions.add(subscription);
     return () => {