fix(web): avoid fake session recency and preserve interrupted tool states

Author: Cai-Tang-www

web/package-lock.json

@@ -40,6 +40,7 @@
     "@types/react": "^18.3.18",
     "@types/react-dom": "^18.3.5",
     "@vitejs/plugin-react": "^4.3.4",
+    "@vitest/coverage-v8": "^4.1.5",
     "electron": "^33.2.0",
     "electron-builder": "^25.1.8",
     "happy-dom": "^20.9.0",

web/package.json

@@ -134,6 +134,27 @@ describe('useSessionStore', () => {
     expect(session.time).toBe('2026-05-09T09:30:00.000Z')
   })
 
+  it('fetchSessions uses stable fallback time when created_at and updated_at are both invalid', async () => {
+    const mockListSessions = vi.fn().mockResolvedValue({
+      payload: {
+        sessions: [{
+          id: 'sess-invalid-time',
+          title: 'InvalidTime',
+          created_at: 'not-a-date',
+          updated_at: '',
+        }],
+      },
+    })
+    const mockBindStream = vi.fn().mockResolvedValue({})
+    const mockLoadSession = vi.fn().mockResolvedValue({ payload: { messages: [] } })
+    const mockAPI = { listSessions: mockListSessions, bindStream: mockBindStream, loadSession: mockLoadSession } as any
+
+    await useSessionStore.getState().fetchSessions(mockAPI)
+
+    const session = useSessionStore.getState().projects[0].sessions[0]
+    expect(session.time).toBe('1970-01-01T00:00:00.000Z')
+  })
+
   it('switchSession concurrently fetches todos and checkpoints', async () => {
     const mockBindStream = vi.fn().mockResolvedValue({})
     const mockLoadSession = vi.fn().mockResolvedValue({

web/src/stores/useSessionStore.test.ts

@@ -120,7 +120,8 @@ function selectSessionDisplayTime(session: APISessionSummary): Date {
   }
   if (updatedValid) return updated
   if (createdValid) return created
-  return new Date()
+  // 当后端时间字段都无效时，不伪造“当前时间”，避免损坏记录冒充“刚刚更新”并扰乱排序。
+  return new Date(0)
 }
 
 export type BackendMessage = {

web/src/stores/useSessionStore.ts

@@ -263,6 +263,80 @@ describe('eventBridge', () => {
     expect(useChatStore.getState().messages[0].toolStatus).toBe('done')
   })
 
+  it('StopReasonDecided keeps running tool calls unresolved for non-fatal reasons', () => {
+    const api = createMockGatewayAPI()
+    useChatStore.getState().addMessage({
+      id: 'tool-running-nonfatal',
+      role: 'tool',
+      type: 'tool_call',
+      content: '',
+      toolName: 'bash',
+      toolCallId: 'tc-nonfatal',
+      toolStatus: 'running',
+      timestamp: Date.now(),
+    })
+
+    handleGatewayEvent({
+      type: EventType.StopReasonDecided,
+      payload: { payload: { runtime_event_type: EventType.StopReasonDecided, payload: { reason: 'user_interrupt' } } },
+      session_id: 'sess-1',
+      run_id: 'run-1',
+    }, api)
+
+    expect(useChatStore.getState().messages[0].toolStatus).toBe('running')
+  })
+
+  it('StopReasonDecided marks running tool calls as error on fatal_error', () => {
+    const api = createMockGatewayAPI()
+    useChatStore.getState().addMessage({
+      id: 'tool-running-fatal',
+      role: 'tool',
+      type: 'tool_call',
+      content: '',
+      toolName: 'bash',
+      toolCallId: 'tc-fatal',
+      toolStatus: 'running',
+      timestamp: Date.now(),
+    })
+
+    handleGatewayEvent({
+      type: EventType.StopReasonDecided,
+      payload: {
+        payload: {
+          runtime_event_type: EventType.StopReasonDecided,
+          payload: { reason: 'fatal_error', detail: 'fatal' },
+        },
+      },
+      session_id: 'sess-1',
+      run_id: 'run-1',
+    }, api)
+
+    expect(useChatStore.getState().messages[0].toolStatus).toBe('error')
+  })
+
+  it('RunCanceled does not convert running tool calls to done', () => {
+    const api = createMockGatewayAPI()
+    useChatStore.getState().addMessage({
+      id: 'tool-running-canceled',
+      role: 'tool',
+      type: 'tool_call',
+      content: '',
+      toolName: 'bash',
+      toolCallId: 'tc-canceled',
+      toolStatus: 'running',
+      timestamp: Date.now(),
+    })
+
+    handleGatewayEvent({
+      type: EventType.RunCanceled,
+      payload: { payload: { runtime_event_type: EventType.RunCanceled, payload: {} } },
+      session_id: 'sess-1',
+      run_id: 'run-1',
+    }, api)
+
+    expect(useChatStore.getState().messages[0].toolStatus).toBe('running')
+  })
+
   it('BudgetChecked updates runtime insight budget state', () => {
     const api = createMockGatewayAPI()
     handleGatewayEvent({

web/src/utils/eventBridge.test.ts

@@ -523,7 +523,9 @@ export function handleGatewayEvent(frame: MessageFrame, gatewayAPI: GatewayAPI)
       const detail = strField(eventPayload, 'detail')
       useChatStore.getState().setStopReason(reason)
       useChatStore.getState().setGenerating(false)
-      useChatStore.getState().finalizeRunningToolCalls(reason === 'fatal_error' ? 'error' : 'done')
+      if (reason === 'fatal_error') {
+        useChatStore.getState().finalizeRunningToolCalls('error')
+      }
       if (reason === 'fatal_error') {
         uiStore.showToast(detail || '模型调用失败，请检查配置', 'error')
       }
@@ -537,7 +539,6 @@ export function handleGatewayEvent(frame: MessageFrame, gatewayAPI: GatewayAPI)
 
     case EventType.RunCanceled: {
       useChatStore.getState().resetGeneratingState()
-      useChatStore.getState().finalizeRunningToolCalls('done')
       uiStore.showToast('Run cancelled', 'info')
       break
     }