webui: address review feedback from allozaur

Author: ServeurpersoCom

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

@@ -323,6 +323,7 @@ class ChatStore {
 				});
 			}
 		};
+
 		slotsService.startStreaming();
 
 		await chatService.sendMessage(allMessages, {

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

@@ -79,9 +79,7 @@ class ModelsStore {
 
 			const models: ModelOption[] = response.data.map((item, index) => {
 				const details = response.models?.[index];
-				const rawCapabilities = Array.isArray(details?.capabilities)
-					? [...(details?.capabilities ?? [])]
-					: [];
+				const rawCapabilities = Array.isArray(details?.capabilities) ? details?.capabilities : [];
 				const displayNameSource =
 					details?.name && details.name.trim().length > 0 ? details.name : item.id;
 				const displayName = this.toDisplayName(displayNameSource);
@@ -99,36 +97,17 @@ class ModelsStore {
 
 			this._models = models;
 
-			const persisted = this.readPersistedSelection();
-			let nextSelectionId = this._selectedModelId ?? persisted?.id ?? null;
-			let nextSelectionName = this._selectedModelName ?? persisted?.model ?? null;
-			if (nextSelectionId) {
-				const match = models.find((model) => model.id === nextSelectionId);
-				if (match) {
-					nextSelectionId = match.id;
-					nextSelectionName = match.model;
-				} else if (models[0]) {
-					nextSelectionId = models[0].id;
-					nextSelectionName = models[0].model;
-				} else {
-					nextSelectionId = null;
-					nextSelectionName = null;
-				}
-			} else if (models[0]) {
-				nextSelectionId = models[0].id;
-				nextSelectionName = models[0].model;
-			}
+			const selection = this.determineInitialSelection(models);
 
-			this._selectedModelId = nextSelectionId;
-			this._selectedModelName = nextSelectionName;
+			this._selectedModelId = selection.id;
+			this._selectedModelName = selection.model;
 			this.persistSelection(
-				nextSelectionId && nextSelectionName
-					? { id: nextSelectionId, model: nextSelectionName }
-					: null
+				selection.id && selection.model ? { id: selection.id, model: selection.model } : null
 			);
 		} catch (error) {
 			this._models = [];
 			this._error = error instanceof Error ? error.message : 'Failed to load models';
+
 			throw error;
 		} finally {
 			this._loading = false;
@@ -167,6 +146,38 @@ class ModelsStore {
 		return candidate && candidate.trim().length > 0 ? candidate : id;
 	}
 
+	/**
+	 * Determines which model should be selected after fetching the models list.
+	 * Priority: current selection > persisted selection > first available model > none
+	 */
+	private determineInitialSelection(models: ModelOption[]): {
+		id: string | null;
+		model: string | null;
+	} {
+		const persisted = this.readPersistedSelection();
+		let nextSelectionId = this._selectedModelId ?? persisted?.id ?? null;
+		let nextSelectionName = this._selectedModelName ?? persisted?.model ?? null;
+
+		if (nextSelectionId) {
+			const match = models.find((m) => m.id === nextSelectionId);
+			if (match) {
+				nextSelectionId = match.id;
+				nextSelectionName = match.model;
+			} else if (models[0]) {
+				nextSelectionId = models[0].id;
+				nextSelectionName = models[0].model;
+			} else {
+				nextSelectionId = null;
+				nextSelectionName = null;
+			}
+		} else if (models[0]) {
+			nextSelectionId = models[0].id;
+			nextSelectionName = models[0].model;
+		}
+
+		return { id: nextSelectionId, model: nextSelectionName };
+	}
+
 	private readPersistedSelection(): PersistedModelSelection | null {
 		if (!browser) {
 			return null;
@@ -183,6 +194,7 @@ class ModelsStore {
 				const id = parsed.id;
 				const model =
 					typeof parsed.model === 'string' && parsed.model.length > 0 ? parsed.model : id;
+
 				return { id, model };
 			}
 		} catch (error) {

tools/server/webui/src/routes/+layout.svelte

@@ -148,7 +148,7 @@
 
 <ModeWatcher />
 
-<Toaster position="top-center" richColors />
+<Toaster richColors />
 
 <ConversationTitleUpdateDialog
 	bind:open={titleUpdateDialogOpen}