webui: stabilize dropdown, simplify model extraction, and init assistant model field

Author: ServeurpersoCom

tools/server/webui/src/lib/components/app/chat/ChatForm/ChatFormModelSelector.svelte

@@ -142,7 +142,7 @@
 	$effect(() => {
 		const optionCount = options.length;
 
-		if (!isOpen || optionCount < 0) return;
+		if (!isOpen || optionCount <= 0) return;
 
 		queueMicrotask(() => updateMenuPosition());
 	});

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

@@ -639,65 +639,35 @@ export class ChatService {
 	}
 
 	private extractModelName(data: unknown): string | undefined {
-		if (!data || typeof data !== 'object') {
-			return undefined;
-		}
-
-		const record = data as Record<string, unknown>;
-		const normalize = (value: unknown): string | undefined => {
-			if (typeof value !== 'string') {
-				return undefined;
-			}
-
-			const trimmed = value.trim();
+		const asRecord = (value: unknown): Record<string, unknown> | undefined => {
+			return typeof value === 'object' && value !== null
+				? (value as Record<string, unknown>)
+				: undefined;
+		};
 
-			return trimmed.length > 0 ? trimmed : undefined;
+		const getTrimmedString = (value: unknown): string | undefined => {
+			return typeof value === 'string' && value.trim() ? value.trim() : undefined;
 		};
 
-		const rootModel = normalize(record['model']);
-		if (rootModel) {
-			return rootModel;
-		}
+		const root = asRecord(data);
+		if (!root) return undefined;
 
-		const choices = record['choices'];
-		if (!Array.isArray(choices) || choices.length === 0) {
-			return undefined;
-		}
+		// 1) root (some implementations provide `model` at the top level)
+		const rootModel = getTrimmedString(root.model);
+		if (rootModel) return rootModel;
 
-		const firstChoice = choices[0] as Record<string, unknown> | undefined;
-		if (!firstChoice) {
-			return undefined;
-		}
+		// 2) streaming choice (delta) or final response (message)
+		const firstChoice = Array.isArray(root.choices) ? asRecord(root.choices[0]) : undefined;
+		if (!firstChoice) return undefined;
 
-		const choiceModel = normalize(firstChoice['model']);
-		if (choiceModel) {
-			return choiceModel;
-		}
+		// priority: delta.model (first chunk) else message.model (final response)
+		const deltaModel = getTrimmedString(asRecord(firstChoice.delta)?.model);
+		if (deltaModel) return deltaModel;
 
-		const delta = firstChoice['delta'] as Record<string, unknown> | undefined;
-		if (delta) {
-			const deltaModel = normalize(delta['model']);
-			if (deltaModel) {
-				return deltaModel;
-			}
-		}
-
-		const message = firstChoice['message'] as Record<string, unknown> | undefined;
-		if (message) {
-			const messageModel = normalize(message['model']);
-			if (messageModel) {
-				return messageModel;
-			}
-		}
-
-		const metadata = firstChoice['metadata'] as Record<string, unknown> | undefined;
-		if (metadata) {
-			const metadataModel = normalize(metadata['model']);
-			if (metadataModel) {
-				return metadataModel;
-			}
-		}
+		const messageModel = getTrimmedString(asRecord(firstChoice.message)?.model);
+		if (messageModel) return messageModel;
 
+		// avoid guessing from non-standard locations (metadata, etc.)
 		return undefined;
 	}
 

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

@@ -570,7 +570,8 @@ class ChatStore {
 				content: '',
 				timestamp: Date.now(),
 				thinking: '',
-				children: []
+				children: [],
+				model: null
 			},
 			parentId || null
 		);
@@ -1538,7 +1539,8 @@ class ChatStore {
 					role: 'assistant',
 					content: '',
 					thinking: '',
-					children: []
+					children: [],
+					model: null
 				},
 				parentMessage.id
 			);
@@ -1595,7 +1597,8 @@ class ChatStore {
 					role: 'assistant',
 					content: '',
 					thinking: '',
-					children: []
+					children: [],
+					model: null
 				},
 				userMessageId
 			);