From 081c60a15f12b93907132b2cae124e8d60adc1c2 Mon Sep 17 00:00:00 2001
From: ekko <fqsy1416@gmail.com>
Date: Mon, 13 Apr 2026 00:52:34 +0800
Subject: [PATCH] feat: add session grouping by source with collapsible
 accordion

- Group sessions by source in sidebar (api_server first, cron last)
- Accordion behavior: only one group expanded at a time
- Auto-select first session when expanding a group
- Backfill session titles from first user message in listSessions
- Remove chat header model badge
- Fix toolPreview type error for non-string values
- New chats default to api_server source

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 server/src/services/hermes-cli.ts |  10 +-
 src/components/chat/ChatPanel.vue | 200 ++++++++++++++++++++----------
 src/stores/chat.ts                |  20 +--
 3 files changed, 144 insertions(+), 86 deletions(-)

diff --git a/server/src/services/hermes-cli.ts b/server/src/services/hermes-cli.ts
index 68310b2..24a7c2c 100644
--- a/server/src/services/hermes-cli.ts
+++ b/server/src/services/hermes-cli.ts
@@ -53,12 +53,20 @@ export async function listSessions(source?: string, limit?: number): Promise<Her
     for (const line of lines) {
       try {
         const raw: HermesSessionFull = JSON.parse(line)
+        let title = raw.title
+        if (!title && raw.messages) {
+          const firstUser = raw.messages.find((m: any) => m.role === 'user')
+          if (firstUser?.content) {
+            const t = String(firstUser.content).slice(0, 40)
+            title = t + (String(firstUser.content).length > 40 ? '...' : '')
+          }
+        }
         sessions.push({
           id: raw.id,
           source: raw.source,
           user_id: raw.user_id,
           model: raw.model,
-          title: raw.title,
+          title,
           started_at: raw.started_at,
           ended_at: raw.ended_at,
           end_reason: raw.end_reason,
diff --git a/src/components/chat/ChatPanel.vue b/src/components/chat/ChatPanel.vue
index 9d3fbe9..d485714 100644
--- a/src/components/chat/ChatPanel.vue
+++ b/src/components/chat/ChatPanel.vue
@@ -1,14 +1,12 @@
 <script setup lang="ts">
 import { renameSession } from '@/api/sessions'
-import { useAppStore } from '@/stores/app'
-import { useChatStore } from '@/stores/chat'
+import { useChatStore, type Session } from '@/stores/chat'
 import { NButton, NDropdown, NInput, NModal, NPopconfirm, NTooltip, useMessage } from 'naive-ui'
-import { computed, nextTick, ref } from 'vue'
+import { computed, nextTick, ref, watch } from 'vue'
 import ChatInput from './ChatInput.vue'
 import MessageList from './MessageList.vue'
 
 const chatStore = useChatStore()
-const appStore = useAppStore()
 const message = useMessage()
 
 const showSessions = ref(true)
@@ -16,11 +14,68 @@ const showRenameModal = ref(false)
 const renameValue = ref('')
 const renameSessionId = ref<string | null>(null)
 const renameInputRef = ref<InstanceType<typeof NInput> | null>(null)
+const collapsedGroups = ref<Set<string>>(new Set())
 
-const sortedSessions = computed(() => {
-  return [...chatStore.sessions].sort((a, b) => b.createdAt - a.createdAt)
+// Source sort order: api_server first, cron last, others alphabetical
+function sourceSortKey(source: string): number {
+  if (source === 'api_server') return -1
+  if (source === 'cron') return 999
+  return 0
+}
+
+// Group sessions by source, with sort order
+interface SessionGroup {
+  source: string
+  label: string
+  sessions: Session[]
+}
+
+const groupedSessions = computed<SessionGroup[]>(() => {
+  const all = [...chatStore.sessions].sort((a, b) => b.createdAt - a.createdAt)
+
+  const map = new Map<string, Session[]>()
+  for (const s of all) {
+    const key = s.source || ''
+    if (!map.has(key)) map.set(key, [])
+    map.get(key)!.push(s)
+  }
+
+  const keys = [...map.keys()].sort((a, b) => {
+    const ka = sourceSortKey(a)
+    const kb = sourceSortKey(b)
+    if (ka !== kb) return ka - kb
+    return a.localeCompare(b)
+  })
+
+  return keys.map(key => ({
+    source: key,
+    label: key ? getSourceLabel(key) : 'Other',
+    sessions: map.get(key)!,
+  }))
 })
 
+function toggleGroup(source: string) {
+  const isExpanded = !collapsedGroups.value.has(source)
+  if (isExpanded) {
+    collapsedGroups.value = new Set([...collapsedGroups.value, source])
+  } else {
+    collapsedGroups.value = new Set(
+      groupedSessions.value.map(g => g.source).filter(s => s !== source)
+    )
+    // Auto-select the first session in the expanded group
+    const group = groupedSessions.value.find(g => g.source === source)
+    if (group?.sessions.length) {
+      chatStore.switchSession(group.sessions[0].id)
+    }
+  }
+}
+
+// Default: expand only the first group, collapse the rest
+watch(groupedSessions, (groups) => {
+  if (collapsedGroups.value.size > 0) return
+  collapsedGroups.value = new Set(groups.map(g => g.source))
+}, { once: true })
+
 const activeSessionTitle = computed(() =>
   chatStore.activeSession?.title || 'New Chat',
 )
@@ -29,9 +84,6 @@ const activeSessionSource = computed(() =>
   chatStore.activeSession?.source || '',
 )
 
-const sessionModelLabel = computed(() =>
-  chatStore.activeSession?.model || appStore.selectedModel || '',
-)
 
 const sourceLabel: Record<string, string> = {
   telegram: 'Telegram',
@@ -50,6 +102,7 @@ const sourceLabel: Record<string, string> = {
   weixin: 'WeChat',
   bluebubbles: 'iMessage',
   mattermost: 'Mattermost',
+  cron: 'Cron',
 }
 
 function getSourceLabel(source?: string): string {
@@ -151,36 +204,44 @@ async function handleRenameConfirm() {
         </NButton>
       </div>
       <div v-if="showSessions" class="session-items">
-        <div v-if="chatStore.isLoadingSessions && sortedSessions.length === 0" class="session-loading">Loading...</div>
-        <div v-else-if="sortedSessions.length === 0" class="session-empty">No sessions</div>
-        <button
-          v-for="s in sortedSessions"
-          :key="s.id"
-          class="session-item"
-          :class="{ active: s.id === chatStore.activeSessionId }"
-          @click="chatStore.switchSession(s.id)"
-          @contextmenu="handleContextMenu($event, s.id)"
-        >
-          <div class="session-item-content">
-            <span class="session-item-title">{{ s.title }}</span>
-            <span class="session-item-meta">
-              <span v-if="s.model" class="session-item-model">{{ s.model }}</span>
-              <!-- <span v-if="s.source" class="session-item-source">{{ getSourceLabel(s.source) }}</span> -->
-              <span class="session-item-time">{{ formatTime(s.createdAt) }}</span>
-            </span>
+        <div v-if="chatStore.isLoadingSessions && chatStore.sessions.length === 0" class="session-loading">Loading...</div>
+        <div v-else-if="chatStore.sessions.length === 0" class="session-empty">No sessions</div>
+        <template v-for="group in groupedSessions" :key="group.source">
+          <div class="session-group-header" @click="toggleGroup(group.source)">
+            <svg width="10" height="10" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" class="group-chevron" :class="{ collapsed: collapsedGroups.has(group.source) }"><polyline points="9 18 15 12 9 6"/></svg>
+            <span class="session-group-label">{{ group.label }}</span>
+            <span class="session-group-count">{{ group.sessions.length }}</span>
           </div>
-          <NPopconfirm
-            v-if="s.id !== chatStore.activeSessionId || sortedSessions.length > 1"
-            @positive-click="handleDeleteSession(s.id)"
-          >
-            <template #trigger>
-              <button class="session-item-delete" @click.stop>
-                <svg width="12" height="12" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2"><line x1="18" y1="6" x2="6" y2="18"/><line x1="6" y1="6" x2="18" y2="18"/></svg>
-              </button>
-            </template>
-            Delete this session?
-          </NPopconfirm>
-        </button>
+          <template v-if="!collapsedGroups.has(group.source)">
+            <button
+              v-for="s in group.sessions"
+              :key="s.id"
+              class="session-item"
+              :class="{ active: s.id === chatStore.activeSessionId }"
+              @click="chatStore.switchSession(s.id)"
+              @contextmenu="handleContextMenu($event, s.id)"
+            >
+              <div class="session-item-content">
+                <span class="session-item-title">{{ s.title }}</span>
+                <span class="session-item-meta">
+                  <span v-if="s.model" class="session-item-model">{{ s.model }}</span>
+                  <span class="session-item-time">{{ formatTime(s.createdAt) }}</span>
+                </span>
+              </div>
+              <NPopconfirm
+                v-if="s.id !== chatStore.activeSessionId || chatStore.sessions.length > 1"
+                @positive-click="handleDeleteSession(s.id)"
+              >
+                <template #trigger>
+                  <button class="session-item-delete" @click.stop>
+                    <svg width="12" height="12" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2"><line x1="18" y1="6" x2="6" y2="18"/><line x1="6" y1="6" x2="18" y2="18"/></svg>
+                  </button>
+                </template>
+                Delete this session?
+              </NPopconfirm>
+            </button>
+          </template>
+        </template>
       </div>
     </aside>
 
@@ -225,9 +286,6 @@ async function handleRenameConfirm() {
           <span class="header-session-title">{{ activeSessionTitle }}</span>
           <span v-if="activeSessionSource" class="source-badge">{{ getSourceLabel(activeSessionSource) }}</span>
         </div>
-        <div class="header-center">
-          <span v-if="sessionModelLabel" class="model-badge">{{ sessionModelLabel }}</span>
-        </div>
         <div class="header-actions">
           <NTooltip trigger="hover">
             <template #trigger>
@@ -295,6 +353,39 @@ async function handleRenameConfirm() {
   letter-spacing: 0.5px;
 }
 
+.session-group-header {
+  display: flex;
+  align-items: center;
+  gap: 4px;
+  padding: 6px 10px 4px;
+  cursor: pointer;
+  user-select: none;
+}
+
+.group-chevron {
+  flex-shrink: 0;
+  transition: transform 0.15s ease;
+  transform: rotate(90deg);
+
+  &.collapsed {
+    transform: rotate(0deg);
+  }
+}
+
+.session-group-label {
+  font-size: 10px;
+  font-weight: 600;
+  color: $text-muted;
+  text-transform: uppercase;
+  letter-spacing: 0.5px;
+}
+
+.session-group-count {
+  font-size: 10px;
+  color: $text-muted;
+  font-weight: 400;
+}
+
 .session-items {
   flex: 1;
   overflow-y: auto;
@@ -379,17 +470,6 @@ async function handleRenameConfirm() {
   white-space: nowrap;
 }
 
-.session-item-source {
-  font-size: 10px;
-  color: $text-muted;
-  background: rgba($text-muted, 0.1);
-  padding: 0 5px;
-  border-radius: 3px;
-  line-height: 16px;
-  flex-shrink: 0;
-  white-space: nowrap;
-}
-
 .session-item-delete {
   flex-shrink: 0;
   opacity: 0;
@@ -433,12 +513,6 @@ async function handleRenameConfirm() {
   min-width: 0;
 }
 
-.header-center {
-  flex-shrink: 0;
-  max-width: 240px;
-  min-width: 140px;
-}
-
 .header-session-title {
   font-size: 14px;
   font-weight: 500;
@@ -459,16 +533,6 @@ async function handleRenameConfirm() {
   line-height: 16px;
 }
 
-.model-badge {
-  font-size: 11px;
-  color: $text-muted;
-  background: rgba($accent-primary, 0.1);
-  padding: 2px 8px;
-  border-radius: 10px;
-  border: 1px solid $border-color;
-  flex-shrink: 0;
-}
-
 .header-actions {
   display: flex;
   align-items: center;
diff --git a/src/stores/chat.ts b/src/stores/chat.ts
index 6b94a35..53e2459 100644
--- a/src/stores/chat.ts
+++ b/src/stores/chat.ts
@@ -118,7 +118,7 @@ function mapHermesMessages(msgs: HermesMessage[]): Message[] {
         timestamp: Math.round(msg.timestamp * 1000),
         toolName,
         toolArgs,
-        toolPreview: preview.slice(0, 100) || undefined,
+        toolPreview: typeof preview === 'string' ? preview.slice(0, 100) || undefined : undefined,
         toolResult: msg.content || undefined,
         toolStatus: 'done',
       })
@@ -166,22 +166,6 @@ export const useChatStore = defineStore('chat', () => {
     try {
       const list = await fetchSessions()
       sessions.value = list.map(mapHermesSession)
-      // Backfill titles from first user message for sessions with null title
-      const nullTitleSessions = sessions.value.filter(s => s.title === 'New Chat')
-      if (nullTitleSessions.length > 0) {
-        await Promise.allSettled(
-          nullTitleSessions.map(async (s) => {
-            const detail = await fetchSession(s.id)
-            if (detail?.messages) {
-              const firstUser = detail.messages.find(m => m.role === 'user')
-              if (firstUser) {
-                const t = firstUser.content.slice(0, 40)
-                s.title = t + (firstUser.content.length > 40 ? '...' : '')
-              }
-            }
-          })
-        )
-      }
       // Auto-select the most recent session
       if (!activeSessionId.value && sessions.value.length > 0) {
         await switchSession(sessions.value[0].id)
@@ -193,10 +177,12 @@ export const useChatStore = defineStore('chat', () => {
     }
   }
 
+
   function createSession(): Session {
     const session: Session = {
       id: uid(),
       title: 'New Chat',
+      source: 'api_server',
       messages: [],
       createdAt: Date.now(),
       updatedAt: Date.now(),