fix(ui): only show 'session renamed' toast when name actually changes

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
arkon
2026-01-19 15:06:41 +01:00
co-authored by Claude Opus 4.5
parent 05322000f3
commit 17a53c4da2
+19 -13
View File
@@ -1467,22 +1467,28 @@ class ClaudemanApp {
async saveSessionOptions() { async saveSessionOptions() {
if (!this.editingSessionId) return; if (!this.editingSessionId) return;
const name = document.getElementById('sessionNameInput').value.trim(); const session = this.sessions.get(this.editingSessionId);
const newName = document.getElementById('sessionNameInput').value.trim();
const oldName = session?.name || '';
try { // Only update if name actually changed
const res = await fetch(`/api/sessions/${this.editingSessionId}/name`, { if (newName !== oldName) {
method: 'PUT', try {
headers: { 'Content-Type': 'application/json' }, const res = await fetch(`/api/sessions/${this.editingSessionId}/name`, {
body: JSON.stringify({ name }) method: 'PUT',
}); headers: { 'Content-Type': 'application/json' },
const data = await res.json(); body: JSON.stringify({ name: newName })
if (data.error) throw new Error(data.error); });
const data = await res.json();
if (data.error) throw new Error(data.error);
this.showToast('Session renamed', 'success'); this.showToast('Session renamed', 'success');
this.closeSessionOptions(); } catch (err) {
} catch (err) { this.showToast('Failed to rename: ' + err.message, 'error');
this.showToast('Failed to rename: ' + err.message, 'error'); }
} }
this.closeSessionOptions();
} }
// Inline rename on right-click // Inline rename on right-click