feat(sessions): improve session title handling and tool message insertion

- Replace string comparison with ToolType enum in SessionManager for type safety
- Use GetProjectWithProvider instead of GetProject for proper provider context
- Add insertToolMessage function to correctly position tool messages after anchor points
- Implement onFirstMessage callback to auto-generate session titles from first user message
- Update SessionAwareProjectChat to handle first message and update session title
- Add patch method mock to useSessionManager tests for API client compatibility
- Improve tool message ordering by finding last consecutive tool message before insertion
- Enhance session title generation by truncating first message to 50 characters

Author: 239573049

src/MyYuCode/Services/Sessions/SessionManager.cs

@@ -158,7 +158,7 @@ public async Task UpdateSessionTitleAsync(Guid sessionId, string title)
     /// </summary>
     public async Task SyncProjectSessionTitlesAsync(Guid projectId)
     {
-        var project = _dataStore.GetProject(projectId);
+        var project = _dataStore.GetProjectWithProvider(projectId);
         if (project == null) return;
 
         var sessions = _dataStore.GetSessionsByProject(projectId);
@@ -167,7 +167,7 @@ public async Task SyncProjectSessionTitlesAsync(Guid projectId)
         bool changed = false;
         var userHome = Environment.GetFolderPath(Environment.SpecialFolder.UserProfile);
 
-        if (string.Equals(project.ToolType, "Codex", StringComparison.OrdinalIgnoreCase))
+        if (project.ToolType == ToolType.Codex)
         {
             var historyPath = Path.Combine(userHome, ".codex", "history.jsonl");
             if (File.Exists(historyPath))
@@ -232,7 +232,7 @@ public async Task SyncProjectSessionTitlesAsync(Guid projectId)
                 }
             }
         }
-        else if (string.Equals(project.ToolType, "ClaudeCode", StringComparison.OrdinalIgnoreCase))
+        else if (project.ToolType == ToolType.ClaudeCode)
         {
             foreach (var session in sessions)
             {

web/src/components/project-workspace/ProjectChat.tsx

@@ -2311,6 +2311,37 @@ function insertAfterMessage(
   return next
 }
 
+// 插入 tool 消息时，找到最后一个 tool 消息或锚点消息，确保新 tool 在正确位置
+function insertToolMessage(
+  prev: ChatMessage[],
+  anchorMessageId: string | null | undefined,
+  message: ChatMessage,
+): ChatMessage[] {
+  // 先找锚点位置
+  let anchorIndex = anchorMessageId ? prev.findIndex((m) => m.id === anchorMessageId) : -1
+  
+  // 从锚点位置向后找最后一个连续的 tool 消息
+  if (anchorIndex >= 0) {
+    let lastToolIndex = anchorIndex
+    for (let i = anchorIndex + 1; i < prev.length; i++) {
+      if (prev[i].kind === 'tool' && prev[i].role === 'agent') {
+        lastToolIndex = i
+      } else {
+        break
+      }
+    }
+    // 如果找到了 tool 消息，在最后一个 tool 后面插入
+    if (lastToolIndex > anchorIndex || (prev[anchorIndex]?.kind === 'tool' && prev[anchorIndex]?.role === 'agent')) {
+      const next = [...prev]
+      next.splice(lastToolIndex + 1, 0, message)
+      return next
+    }
+  }
+  
+  // 否则使用默认的插入逻辑
+  return insertAfterMessage(prev, anchorMessageId, message)
+}
+
 export function ProjectChat({
   project,
   detailsOpen,
@@ -3543,6 +3574,14 @@ export function ProjectChat({
     if (sending) return
     if ((!text && readyImages.length === 0) || hasBlockingUploads) return
 
+    // 如果是第一条消息，通知父组件（用于设置会话标题）
+    const isFirstMessage = messages.length === 0
+    if (isFirstMessage && text && onFirstMessage) {
+      // 截取前50个字符作为标题
+      const title = text.length > 50 ? text.slice(0, 47) + '...' : text
+      onFirstMessage(title)
+    }
+
     setChatError(null)
     setDraft('')
     setMentionToken(null)
@@ -3986,7 +4025,7 @@ export function ProjectChat({
                   return next
                 }
                 inserted = true
-                return insertAfterMessage(prev, anchorAfterId, toolMessage)
+                return insertToolMessage(prev, anchorAfterId, toolMessage)
               })
 
               if (inserted) {
@@ -4048,7 +4087,7 @@ export function ProjectChat({
                 }
 
                 inserted = true
-                return insertAfterMessage(prev, anchorAfterId, toolMessage)
+                return insertToolMessage(prev, anchorAfterId, toolMessage)
               })
 
               if (inserted) {
@@ -4142,7 +4181,7 @@ export function ProjectChat({
               }
 
               const anchorAfterId = streamTailMessageId
-              setMessages((prev) => insertAfterMessage(prev, anchorAfterId, toolMessage))
+              setMessages((prev) => insertToolMessage(prev, anchorAfterId, toolMessage))
               toolChainTailMessageId = toolMessageId
               markStreamTail('tool', toolMessageId)
 
@@ -4246,7 +4285,9 @@ export function ProjectChat({
     apiBase,
     draft,
     draftImages,
+    messages.length,
     modelSelection,
+    onFirstMessage,
     onToolOutput,
     project.id,
     project.toolType,

web/src/components/project-workspace/SessionAwareProjectChat.tsx

@@ -133,6 +133,17 @@ export function SessionAwareProjectChat({
     }
   }, [updateSessionTitle]);
 
+  // Handle first message - update session title
+  const handleFirstMessage = useCallback(async (text: string) => {
+    if (currentSession?.id) {
+      try {
+        await updateSessionTitle(currentSession.id, text);
+      } catch (error) {
+        console.error('Failed to update session title:', error);
+      }
+    }
+  }, [currentSession?.id, updateSessionTitle]);
+
   // Toggle session panel
   const toggleSessionPanel = useCallback(() => {
     setSessionPanelOpen(prev => !prev);
@@ -212,6 +223,7 @@ export function SessionAwareProjectChat({
             onToolOutput={onToolOutput}
             currentToolType={currentToolType}
             sessionId={currentSession?.id}
+            onFirstMessage={handleFirstMessage}
           />
         </div>
       </div>

web/src/hooks/useSessionManager.test.ts

@@ -8,6 +8,7 @@ vi.mock('@/api/client', () => ({
     get: vi.fn(),
     post: vi.fn(),
     put: vi.fn(),
+    patch: vi.fn(),
     delete: vi.fn(),
   },
 }));
@@ -18,6 +19,7 @@ const mockApiClient = apiClient as {
   get: ReturnType<typeof vi.fn>;
   post: ReturnType<typeof vi.fn>;
   put: ReturnType<typeof vi.fn>;
+  patch: ReturnType<typeof vi.fn>;
   delete: ReturnType<typeof vi.fn>;
 };