Fix admin panel showing 'default name': update user_name in-place on rename, improve session list display

app/admin/templates/admin.html

@@ -1002,16 +1002,22 @@
           $('session-list').innerHTML = '<div class="no-sessions">No sessions yet.</div>';
           return;
         }
-        $('session-list').innerHTML = sessions.map(s => `
-      <div class="session-item${s.session_id === _activeId ? ' active' : ''}"
-           data-id="${s.session_id}">
-        <div class="session-item-id" title="${s.user_name || s.session_id}">${s.user_name || shortId(s.session_id)}</div>
-        <div class="session-item-meta">
-          <span class="session-item-time">${fmtDate(s.last_active)}</span>
-          <span class="session-item-count">${s.message_count || 0} msg${s.message_count !== 1 ? 's' : ''}</span>
+        $('session-list').innerHTML = sessions.map(s => {
+          const isDefault = !s.user_name || s.user_name === 'default name' || s.user_name === 'default_name';
+          const displayName = isDefault
+            ? (s.session_id || '').replace(/_/g, ' ').trim() || shortId(s.session_id)
+            : s.user_name;
+          return `
+        <div class="session-item${s.session_id === _activeId ? ' active' : ''}"
+             data-id="${s.session_id}">
+          <div class="session-item-id" title="${displayName}">${displayName}</div>
+          <div class="session-item-meta">
+            <span class="session-item-time">${fmtDate(s.last_active)}</span>
+            <span class="session-item-count">${s.message_count || 0} msg${s.message_count !== 1 ? 's' : ''}</span>
+          </div>
         </div>
-      </div>
-    `).join('');
+      `;
+        }).join('');
         document.querySelectorAll('.session-item').forEach(el => {
           el.addEventListener('click', () => openSession(el.dataset.id));
         });
@@ -1019,7 +1025,10 @@
 
       $('search-inp').addEventListener('input', e => {
         const q = e.target.value.toLowerCase();
-        const filtered = _allSessions.filter(s => s.session_id.toLowerCase().includes(q));
+        const filtered = _allSessions.filter(s =>
+          s.session_id.toLowerCase().includes(q) ||
+          (s.user_name || '').toLowerCase().includes(q)
+        );
         renderSidebar(filtered);
       });
 

app/services/session_store.py

@@ -237,6 +237,25 @@ async def rename_session(old_id: str, new_name: str) -> str:
         counter += 1
 
     if new_id == old_id:
+        # Same filename — still update user_name in the JSON if it has changed.
+        # Without this, sessions whose file is already correctly named keep
+        # user_name = "default name" forever (the early return skips the update).
+        update_lock = await _get_lock(old_id)
+        async with update_lock:
+            try:
+                if old_path.exists():
+                    data = await asyncio.to_thread(_read_json, old_path)
+                    if data.get("user_name") != new_name:
+                        data["user_name"] = new_name
+                        await asyncio.to_thread(_write_json, old_path, data)
+                        logger.info(
+                            "session_store: updated user_name in-place for %s → '%s'",
+                            old_id, new_name
+                        )
+            except Exception as exc:
+                logger.error(
+                    "session_store: failed to update user_name for %s: %s", old_id, exc
+                )
         return old_id
 
     # Lock both IDs to prevent race conditions during move