fix: replace hardcoded provider prefixes with generic colon-split parsing in ProviderModelPicker

cursoragent · cursoragent · commit 7bb9501a873e · 2026-02-27T18:37:51.000Z
diff --git a/apps/web/src/components/ChatView.tsx b/apps/web/src/components/ChatView.tsx
@@ -2826,25 +2826,14 @@ const ProviderModelPicker = memo(function ProviderModelPicker(props: {
     <Select
       items={items}
       value={`${props.provider}:${props.model}`}
-      onValueChange={(value) =>
-        value
-          ? (() => {
-              if (value.startsWith("codex:")) {
-                props.onProviderModelChange(
-                  "codex",
-                  resolveModelSlugForProvider("codex", value.slice("codex:".length)),
-                );
-                return;
-              }
-              if (value.startsWith("claudeCode:")) {
-                props.onProviderModelChange(
-                  "claudeCode",
-                  resolveModelSlugForProvider("claudeCode", value.slice("claudeCode:".length)),
-                );
-              }
-            })()
-          : undefined
-      }
+      onValueChange={(value) => {
+        if (!value) return;
+        const colonIdx = value.indexOf(":");
+        if (colonIdx === -1) return;
+        const provider = value.slice(0, colonIdx) as ProviderKind;
+        const slug = value.slice(colonIdx + 1);
+        props.onProviderModelChange(provider, resolveModelSlugForProvider(provider, slug));
+      }}
     >
       <SelectTrigger size="sm" variant="ghost">
         <span className="flex min-w-0 items-center gap-2">
