From e2cc1ea0f7fa6a832bb3296a3b4fd22cda3cbf76 Mon Sep 17 00:00:00 2001
From: Peter Steinberger <peter@steipete.me>
Date: Mon, 13 Jul 2026 11:47:29 -0700
Subject: [PATCH] refactor(ui): avoid model group map spreads (#106732)

---
 ui/src/pages/model-providers/default-models-view.ts | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/ui/src/pages/model-providers/default-models-view.ts b/ui/src/pages/model-providers/default-models-view.ts
index 1caeeafca4c7..65a939b9f4fb 100644
--- a/ui/src/pages/model-providers/default-models-view.ts
+++ b/ui/src/pages/model-providers/default-models-view.ts
@@ -48,12 +48,11 @@ function modelGroups(models: ModelPickerEntry[]): ModelOptionGroup[] {
     group.models.push({ ref, label: model.name || ref });
     groups.set(groupKey, group);
   }
-  return [...groups.values()]
-    .map((group) => ({
-      ...group,
-      models: group.models.toSorted((a, b) => a.label.localeCompare(b.label)),
-    }))
-    .toSorted((a, b) => a.label.localeCompare(b.label));
+  const sortedGroups = [...groups.values()];
+  for (const group of sortedGroups) {
+    group.models = group.models.toSorted((a, b) => a.label.localeCompare(b.label));
+  }
+  return sortedGroups.toSorted((a, b) => a.label.localeCompare(b.label));
 }
 
 function renderModelOptions(models: ModelPickerEntry[], selected = "") {