From c23fd6975eafa7d63b5a36e54145dbe25eee506f Mon Sep 17 00:00:00 2001
From: LTbinglingfeng <tan13318527080@163.com>
Date: Mon, 25 May 2026 01:54:58 +0800
Subject: [PATCH] fix(providers): preserve alias when applying discovered
 models

ModelDiscoveryPanel now hands the full ModelInfo objects to onApply
instead of just the model names, and BaseProviderForm copies each
incoming alias into the new form.models row so suggestions from
endpoints that return alias metadata are no longer silently dropped.
---
 .../providers/sheets/forms/BaseProviderForm.tsx  | 16 ++++++++++------
 .../sheets/forms/ModelDiscoveryPanel.tsx         | 10 ++++++----
 2 files changed, 16 insertions(+), 10 deletions(-)

diff --git a/src/features/providers/sheets/forms/BaseProviderForm.tsx b/src/features/providers/sheets/forms/BaseProviderForm.tsx
index 4bd1729..508f790 100644
--- a/src/features/providers/sheets/forms/BaseProviderForm.tsx
+++ b/src/features/providers/sheets/forms/BaseProviderForm.tsx
@@ -15,6 +15,7 @@ import type {
   OpenAIProviderConfig,
   ProviderKeyConfig,
 } from '@/types';
+import type { ModelInfo } from '@/utils/models';
 import { PROVIDER_DESCRIPTORS } from '../../descriptors';
 import type {
   ApiKeyEntryInput,
@@ -267,8 +268,8 @@ export function BaseProviderForm({
     setDiscoveryOpen(false);
   };
 
-  const applyDiscoveredModels = (names: string[]) => {
-    if (!names.length) return;
+  const applyDiscoveredModels = (incoming: ModelInfo[]) => {
+    if (!incoming.length) return;
     setForm((prev) => {
       const seen = new Set<string>();
       const next: ModelEntryInput[] = [];
@@ -284,14 +285,17 @@ export function BaseProviderForm({
       const placeholderIdx = next.findIndex(
         (it) => !(it.name ?? '').trim() && !(it.alias ?? '').trim()
       );
-      if (placeholderIdx !== -1 && names.length > 0) {
+      if (placeholderIdx !== -1) {
         next.splice(placeholderIdx, 1);
       }
-      names.forEach((name) => {
-        const trimmed = name.trim();
+      incoming.forEach((info) => {
+        const trimmed = info.name.trim();
         if (!trimmed || seen.has(trimmed)) return;
         seen.add(trimmed);
-        next.push({ name: trimmed, alias: '' });
+        next.push({
+          name: trimmed,
+          alias: (info.alias ?? '').trim(),
+        });
       });
       return { ...prev, models: next };
     });
diff --git a/src/features/providers/sheets/forms/ModelDiscoveryPanel.tsx b/src/features/providers/sheets/forms/ModelDiscoveryPanel.tsx
index dc18465..14f8a7a 100644
--- a/src/features/providers/sheets/forms/ModelDiscoveryPanel.tsx
+++ b/src/features/providers/sheets/forms/ModelDiscoveryPanel.tsx
@@ -16,7 +16,7 @@ interface ModelDiscoveryPanelProps {
   hasFetched: boolean;
   existingNames: Set<string>;
   mutating?: boolean;
-  onApply: (names: string[]) => void;
+  onApply: (picked: ModelInfo[]) => void;
   onReload: () => void;
   onClose: () => void;
 }
@@ -70,9 +70,11 @@ export function ModelDiscoveryPanel({
   };
 
   const handleApply = () => {
-    const names = Array.from(selected).filter((n) => !existingNames.has(n));
-    if (!names.length) return;
-    onApply(names);
+    const picked = models.filter(
+      (m) => selected.has(m.name) && !existingNames.has(m.name)
+    );
+    if (!picked.length) return;
+    onApply(picked);
     setSelected(new Set());
   };