From 4c3be18d002f418b5d2d9d6eeeb5e80d1dfc7666 Mon Sep 17 00:00:00 2001
From: Daniel Han <danielhanchen@gmail.com>
Date: Sun, 24 May 2026 17:46:33 +0000
Subject: [PATCH] Preserve explicit serviceTier="auto" through the settings
 picker (PR #5711)

Round 13 P1 finding: the Service tier picker rendered `null` as the
displayed `auto` and converted any explicit `auto` selection back to
`null`, so the chat-adapter's truthy guard then omitted `service_tier`
on the wire. For Anthropic the docs distinguish:

  - omitting `service_tier`     -> provider default
  - `service_tier="auto"`       -> opts into Priority Tier when available
  - `service_tier="standard_only"` -> opts out

Drop the auto -> null conversion so the user's explicit pick reaches
the adapter and the wire reflects it. `null` still means "never set"
and falls through to the provider default; the existing serviceTier
allowlist already includes "auto" everywhere it matters.
---
 .../frontend/src/features/chat/chat-settings-sheet.tsx | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/studio/frontend/src/features/chat/chat-settings-sheet.tsx b/studio/frontend/src/features/chat/chat-settings-sheet.tsx
index 938c0183af..422a16b63b 100644
--- a/studio/frontend/src/features/chat/chat-settings-sheet.tsx
+++ b/studio/frontend/src/features/chat/chat-settings-sheet.tsx
@@ -1375,10 +1375,12 @@ export function ChatSettingsPanel({
                 <Select
                   value={params.serviceTier ?? "auto"}
                   onValueChange={(value) => {
-                    if (value === "auto") {
-                      set("serviceTier")(null);
-                      return;
-                    }
+                    // Store every selected tier verbatim, including the
+                    // explicit "auto". On Anthropic the docs distinguish
+                    // omitting service_tier (provider default) from
+                    // setting "auto" (opt into Priority Tier when
+                    // available); preserve the user's choice through to
+                    // the adapter so the wire reflects it.
                     const allowed: readonly ServiceTier[] = serviceTierOptions;
                     if (allowed.includes(value as ServiceTier)) {
                       set("serviceTier")(value as ServiceTier);