refactor: Cleanup

Author: allozaur

tools/server/webui/src/lib/components/app/chat/ChatProcessingInfo.svelte

@@ -7,12 +7,11 @@
 
 	const processingState = useProcessingState();
 
-	let processingDetails = $derived(processingState.getProcessingDetails());
-
 	let isCurrentConversationLoading = $derived(isLoading());
-
+	let processingDetails = $derived(processingState.getProcessingDetails());
 	let showSlotsInfo = $derived(isCurrentConversationLoading || config().keepStatsVisible);
 
+	// Track loading state reactively by checking if conversation ID is in loading conversations array
 	$effect(() => {
 		const keepStatsVisible = config().keepStatsVisible;
 
@@ -29,6 +28,7 @@
 		}
 	});
 
+	// Update processing state from stored timings
 	$effect(() => {
 		const conversation = activeConversation();
 		const messages = activeMessages() as DatabaseMessage[];
@@ -40,6 +40,8 @@
 				return;
 			}
 
+			// Search backwards through messages to find most recent assistant message with timing data
+			// Using reverse iteration for performance - avoids array copy and stops at first match
 			let foundTimingData = false;
 
 			for (let i = messages.length - 1; i >= 0; i--) {

tools/server/webui/src/lib/components/app/chat/ChatSidebar/ChatSidebarConversationItem.svelte

@@ -25,7 +25,6 @@
 	let renderActionsDropdown = $state(false);
 	let dropdownOpen = $state(false);
 
-	// Track loading state reactively by checking if conversation ID is in loading conversations array
 	let isLoading = $derived(getAllLoadingConversations().includes(conversation.id));
 
 	function handleEdit(event: Event) {

tools/server/webui/src/lib/services/slots.ts

@@ -172,6 +172,7 @@ export class SlotsService {
 
 		if (processingState === null) {
 			console.warn('Failed to parse timing data - skipping update');
+
 			return;
 		}
 
@@ -200,6 +201,7 @@ export class SlotsService {
 					...(apiKey ? { Authorization: `Bearer ${apiKey}` } : {})
 				}
 			});
+
 			if (response.ok) {
 				const slotsData = await response.json();
 				if (Array.isArray(slotsData) && slotsData.length > 0) {
@@ -236,6 +238,7 @@ export class SlotsService {
 
 		if (contextTotal === null) {
 			console.warn('No context total available - cannot calculate processing state');
+
 			return null;
 		}
 
@@ -276,6 +279,7 @@ export class SlotsService {
 	async getCurrentState(): Promise<ApiProcessingState | null> {
 		if (this.activeConversationId) {
 			const conversationState = this.conversationStates.get(this.activeConversationId);
+
 			if (conversationState) {
 				return conversationState;
 			}

tools/server/webui/src/lib/stores/chat.svelte.ts

@@ -97,14 +97,11 @@ class ChatStore {
 		this.activeConversation = conversation;
 		this.activeMessages = [];
 
-		// Set this conversation as active for statistics display
 		slotsService.setActiveConversation(conversation.id);
 
-		// Sync global isLoading state - new conversation is not loading
 		const isConvLoading = this.isConversationLoading(conversation.id);
 		this.isLoading = isConvLoading;
 
-		// Clear global currentResponse state - new conversation has no streaming
 		this.currentResponse = '';
 
 		await goto(`#/chat/${conversation.id}`);