Adds conversation deletion functionality to chat sidebar

Implements permanent conversation deletion with confirmation dialog and safety checks.

Prevents deletion of note-to-self conversations and requires user confirmation before proceeding. Automatically clears active conversation if deleted and refreshes the conversation list to reflect changes.

Includes proper error handling and user feedback for failed deletion attempts.

Author: ralyodio

src/lib/components/ChatSidebar.svelte

@@ -166,6 +166,49 @@
 		}
 	}
 
+	// Handle delete conversation action
+	async function handleDeleteConversation(conversation) {
+		contextMenu.show = false;
+		
+		// Prevent deleting note-to-self conversations
+		if (conversation.type === 'note_to_self') {
+			console.log('Cannot delete note-to-self conversations');
+			return;
+		}
+		
+		// Confirm deletion
+		const conversationName = conversation.name || conversation.conversation_name || 'this conversation';
+		const confirmed = confirm(
+			`Are you sure you want to permanently delete ${conversationName}?\n\n` +
+			`This will delete all messages and files for all participants and cannot be undone.`
+		);
+		
+		if (!confirmed) {
+			return;
+		}
+		
+		try {
+			const result = await chat.deleteConversation(conversation.id);
+			
+			if (result.success) {
+				// If the deleted conversation was active, clear it
+				if (activeConversationId === conversation.id) {
+					onConversationSelect(null);
+				}
+				
+				// Reload conversations to reflect changes
+				hasLoadedConversations = false;
+				await loadConversationsData();
+			} else {
+				console.error('Failed to delete conversation:', result.error);
+				alert(`Failed to delete conversation: ${result.error}`);
+			}
+		} catch (error) {
+			console.error('Delete conversation error:', error);
+			alert('Failed to delete conversation. Please try again.');
+		}
+	}
+
 	// Handle conversation selection
 	function handleConversationSelect(/** @type {string} */ conversationId) {
 		onConversationSelect(conversationId);
@@ -496,6 +539,15 @@
 			</svg>
 			{contextMenu.conversation?.is_archived ? 'Unarchive' : 'Archive'}
 		</button>
+		<button
+			class="context-item delete-item"
+			onclick={() => handleDeleteConversation(contextMenu.conversation)}
+		>
+			<svg width="16" height="16" viewBox="0 0 24 24" fill="currentColor">
+				<path d="M6 19c0 1.1.9 2 2 2h8c1.1 0 2-.9 2-2V7H6v12zM19 4h-3.5l-1-1h-5l-1 1H5v2h14V4z"/>
+			</svg>
+			Delete
+		</button>
 	</div>
 {/if}
 
@@ -974,4 +1026,13 @@
 	.context-item:hover {
 		background: var(--color-bg-secondary);
 	}
+
+	.context-item.delete-item {
+		color: var(--color-error);
+	}
+
+	.context-item.delete-item:hover {
+		background: var(--color-error);
+		color: white;
+	}
 </style>

src/lib/stores/chat.js

@@ -644,6 +644,13 @@ function createChatStore() {
 	const unarchiveConversation = (conversationId) =>
 		conversationUtils.unarchiveConversation(apiPost, loadConversations, conversationId);
 
+	/**
+	 * Delete a conversation permanently
+	 * @param {string} conversationId - Conversation ID
+	 */
+	const deleteConversation = (conversationId) =>
+		conversationUtils.deleteConversation(apiPost, loadConversations, conversationId);
+
 	return {
 		subscribe,
 
@@ -658,6 +665,7 @@ function createChatStore() {
 		createConversation,
 		archiveConversation,
 		unarchiveConversation,
+		deleteConversation,
 
 		// Message management
 		loadMessages,

src/lib/utils/conversation-utils.js

@@ -45,4 +45,26 @@ export async function unarchiveConversation(apiPost, loadConversations, conversa
 		console.error('Failed to unarchive conversation:', error);
 		return { success: false, error: 'Failed to unarchive conversation' };
 	}
+}
+
+/**
+ * Delete a conversation permanently
+ * @param {Function} apiPost - API POST function (from chat store)
+ * @param {Function} loadConversations - Function to reload conversations
+ * @param {string} conversationId - Conversation ID
+ * @returns {Promise<Object>} Result object with success status
+ */
+export async function deleteConversation(apiPost, loadConversations, conversationId) {
+	try {
+		const response = await apiPost('/api/conversations/delete', { conversationId });
+
+		if (response.success) {
+			await loadConversations();
+			return { success: true };
+		}
+		return { success: false, error: response.error || 'Failed to delete conversation' };
+	} catch (error) {
+		console.error('Failed to delete conversation:', error);
+		return { success: false, error: 'Failed to delete conversation' };
+	}
 }

src/routes/api/conversations/delete/+server.js

@@ -0,0 +1,84 @@
+/**
+ * @fileoverview Delete Conversation API Endpoint
+ * Handles permanently deleting a conversation and all associated data for all participants
+ */
+
+import { json } from '@sveltejs/kit';
+import { withAuth } from '$lib/api/middleware/auth.js';
+
+export const POST = withAuth(async ({ request, locals }) => {
+	try {
+		const { conversationId } = await request.json();
+
+		if (!conversationId) {
+			return json({ error: 'Missing conversationId' }, { status: 400 });
+		}
+
+		const { supabase, user: authUser } = locals;
+
+		// First, verify the user is a participant in this conversation
+		const { data: participant, error: participantError } = await supabase
+			.from('conversation_participants')
+			.select('id')
+			.eq('conversation_id', conversationId)
+			.eq('user_id', authUser.id)
+			.single();
+
+		if (participantError || !participant) {
+			console.error('User not a participant:', participantError);
+			return json({ 
+				error: 'Conversation not found or you are not a participant' 
+			}, { status: 404 });
+		}
+
+		// Delete all messages in the conversation
+		const { error: messagesError } = await supabase
+			.from('messages')
+			.delete()
+			.eq('conversation_id', conversationId);
+
+		if (messagesError) {
+			console.error('Error deleting messages:', messagesError);
+			return json({ error: 'Failed to delete messages' }, { status: 500 });
+		}
+
+		// Delete all file attachments associated with the conversation
+		const { error: filesError } = await supabase
+			.from('file_attachments')
+			.delete()
+			.eq('conversation_id', conversationId);
+
+		if (filesError) {
+			console.error('Error deleting file attachments:', filesError);
+			// Continue even if file deletion fails
+		}
+
+		// Delete all conversation participants
+		const { error: participantsError } = await supabase
+			.from('conversation_participants')
+			.delete()
+			.eq('conversation_id', conversationId);
+
+		if (participantsError) {
+			console.error('Error deleting participants:', participantsError);
+			return json({ error: 'Failed to delete participants' }, { status: 500 });
+		}
+
+		// Finally, delete the conversation itself
+		const { error: conversationError } = await supabase
+			.from('conversations')
+			.delete()
+			.eq('id', conversationId);
+
+		if (conversationError) {
+			console.error('Error deleting conversation:', conversationError);
+			return json({ error: 'Failed to delete conversation' }, { status: 500 });
+		}
+
+		console.log(`✅ Successfully deleted conversation ${conversationId} and all associated data`);
+		return json({ success: true });
+	} catch (error) {
+		console.error('Delete conversation error:', error);
+		return json({ error: 'Internal server error' }, { status: 500 });
+	}
+});