feat(desktop): hoisted todo widget, JSON tool summaries, history grouping & timer fixes

- Hoist todo to first-class widget (shadcn checkboxes, brand colors, no tool-accordion). Header derives label from active task; non-active rows fade. - Replace raw JSON dumps with structured key/value summaries via formatToolResultSummary; nested error extraction for clearer failures. - Fix loaded-session grouping: stitch interleaved assistant/tool iterations into one bubble instead of orphaned synthetic messages. - Stable tool/thinking timers via keyed registry so unmount/scroll doesn't reset elapsed counts; gate "running" on real live thread state. - Reorganize chat-only assistant-ui components under components/chat/.
2026-05-11 16:34:25 -04:00
parent 4b3839a8ee
commit 4dd9732a94
34 changed files with 1234 additions and 306 deletions
@@ -0,0 +1,108 @@
+import { describe, expect, it } from 'vitest'
+
+import { extractToolErrorMessage, formatToolResultSummary } from './tool-result-summary'
+
+describe('formatToolResultSummary', () => {
+  it('unwraps wrapper payloads into structured key-value lines', () => {
+    const summary = formatToolResultSummary({
+      success: true,
+      result: {
+        data: {
+          path: '/tmp/demo.txt',
+          status: 'ok',
+          lines_written: 12,
+          checksum: 'abc123'
+        }
+      }
+    })
+
+    expect(summary).toContain('- Path: /tmp/demo.txt')
+    expect(summary).toContain('- Status: ok')
+    expect(summary).toContain('- Lines Written: 12')
+    expect(summary).not.toContain('"path"')
+  })
+
+  it('summarizes object arrays as readable list items', () => {
+    const summary = formatToolResultSummary([
+      { title: 'First result', snippet: 'alpha preview text' },
+      { title: 'Second result', status: 'cached' },
+      { title: 'Third result', summary: 'more details' },
+      { title: 'Fourth result', summary: 'line 4' },
+      { title: 'Fifth result', summary: 'line 5' },
+      { title: 'Sixth result', summary: 'line 6' },
+      { title: 'Seventh result', summary: 'line 7' }
+    ])
+
+    expect(summary).toContain('- First result - alpha preview text')
+    expect(summary).toContain('- Second result (cached)')
+    expect(summary).toContain('- … 1 more item')
+  })
+
+  it('truncates long field values for compact display', () => {
+    const summary = formatToolResultSummary({
+      message: 'ok',
+      details: `prefix ${'x'.repeat(500)}`
+    })
+
+    const detailsLine = summary
+      .split('\n')
+      .find(line => line.startsWith('- Details:'))
+
+    expect(detailsLine).toBeTruthy()
+    expect(detailsLine?.length).toBeLessThan(230)
+    expect(detailsLine).toContain('…')
+  })
+
+  it('formats stringified json payloads without raw dumps', () => {
+    const summary = formatToolResultSummary(
+      JSON.stringify({
+        data: {
+          title: 'Build report',
+          completed: true
+        }
+      })
+    )
+
+    expect(summary).toContain('- Title: Build report')
+    expect(summary).toContain('- Completed: true')
+  })
+})
+
+describe('extractToolErrorMessage', () => {
+  it('finds nested error messages through wrappers', () => {
+    const error = extractToolErrorMessage({
+      success: false,
+      result: {
+        output: {
+          error: {
+            message: 'Permission denied writing /tmp/demo.txt'
+          }
+        }
+      }
+    })
+
+    expect(error).toBe('Permission denied writing /tmp/demo.txt')
+  })
+
+  it('does not treat successful payload messages as errors', () => {
+    const error = extractToolErrorMessage({
+      success: true,
+      message: 'Completed successfully',
+      data: { count: 3 }
+    })
+
+    expect(error).toBe('')
+  })
+
+  it('ignores placeholder error fields in successful payloads', () => {
+    const error = extractToolErrorMessage({
+      success: true,
+      data: {
+        error: 'none',
+        status: 'ok'
+      }
+    })
+
+    expect(error).toBe('')
+  })
+})