Merge branch 'main' into 101-design-sweep-polish-chunk-page

This commit is contained in:
boneill-ds 2025-10-03 09:09:34 -06:00 committed by GitHub
commit 08d9f37e62
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 13 additions and 26 deletions

View file

@ -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]);

View file

@ -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
helperText="Model used for chat"
id="model-select"
<LabelWrapper
label="Language model"
required
helperText="Model used for chat"
id="embedding-model"
required={true}
>
<Select
value={
settings.agent?.llm_model ||
modelsData?.language_models?.find((m) => m.default)
?.value ||
"gpt-4"
}
<ModelSelector
options={modelsData?.language_models || []}
noOptionsPlaceholder={modelsData ? "No language models detected." : "Loading models..."}
icon={<OpenAILogo className="w-4 h-4" />}
value={modelsData ? settings.agent?.llm_model || "" : ""}
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">