Merge remote-tracking branch 'origin/main' into fix/model_selector

frontend/src/app/onboarding/components/model-selector.tsx

@@ -44,7 +44,7 @@ export function ModelSelector({
 	const [searchValue, setSearchValue] = useState("");
 
 	useEffect(() => {
-		if (value && (!options.find((option) => option.value === value) && !custom)) {
+		if (value && value !== "" && (!options.find((option) => option.value === value) && !custom)) {
 			onValueChange("");
 		}
 	}, [options, value, custom, onValueChange]);

frontend/src/app/settings/page.tsx

@@ -13,6 +13,7 @@ import {
 import { useGetSettingsQuery } from "@/app/api/queries/useGetSettingsQuery";
 import { ConfirmationDialog } from "@/components/confirmation-dialog";
 import { LabelWrapper } from "@/components/label-wrapper";
+import OpenAILogo from "@/components/logo/openai-logo";
 import { ProtectedRoute } from "@/components/protected-route";
 import { Button } from "@/components/ui/button";
 import {
@@ -40,6 +41,7 @@ import {
 	UI_CONSTANTS,
 } from "@/lib/constants";
 import { useDebounce } from "@/lib/debounce";
+import { ModelSelector } from "../onboarding/components/model-selector";
 import { getFallbackModels, type ModelProvider } from "./helpers/model-helpers";
 import { ModelSelectItems } from "./helpers/model-select-item";
 
@@ -864,34 +866,19 @@ function KnowledgeSourcesPage() {
 				<CardContent>
 					<div className="space-y-6">
 						<div className="space-y-2">
-							<LabelWrapper
+						<LabelWrapper
-								helperText="Model used for chat"
-								id="model-select"
 								label="Language model"
-								required
+								helperText="Model used for chat"
+								id="embedding-model"
+								required={true}
 							>
-								<Select
+								<ModelSelector
-									value={
+									options={modelsData?.language_models || []}
-										settings.agent?.llm_model ||
+                  noOptionsPlaceholder={modelsData ? "No language models detected." : "Loading models..."}
-										modelsData?.language_models?.find((m) => m.default)
+									icon={<OpenAILogo className="w-4 h-4" />}
-											?.value ||
+									value={modelsData ? settings.agent?.llm_model || "" : ""}
-										"gpt-4"
-									}
 									onValueChange={handleModelChange}
-								>
+								/>
-									<SelectTrigger id="model-select">
-										<SelectValue placeholder="Select a model" />
-									</SelectTrigger>
-									<SelectContent>
-										<ModelSelectItems
-											models={modelsData?.language_models}
-											fallbackModels={
-												getFallbackModels(currentProvider).language
-											}
-											provider={currentProvider}
-										/>
-									</SelectContent>
-								</Select>
 							</LabelWrapper>
 						</div>
 						<div className="space-y-2">