From bb679d02be28440acbc860146cbcb3809224aa09 Mon Sep 17 00:00:00 2001
From: ginnoir <ginnoir@users.noreply.github.com>
Date: Wed, 8 Jul 2026 18:59:41 -0500
Subject: [PATCH] fix(agent): improve model selector legibility

---
 src/modules/agent/components/assistant-panel.tsx | 2 +-
 tests/e2e/assistant.spec.ts                      | 3 +++
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/modules/agent/components/assistant-panel.tsx b/src/modules/agent/components/assistant-panel.tsx
index 45d17e6..1a786d7 100644
--- a/src/modules/agent/components/assistant-panel.tsx
+++ b/src/modules/agent/components/assistant-panel.tsx
@@ -257,7 +257,7 @@ export function AssistantPanel({ configured, userId, assistantName, assistantMod
                 value={selectedModel}
                 onChange={(event) => changeModel(event.target.value)}
                 disabled={modelsLoading || savingModel || isPending}
-                className="h-7 w-full max-w-36 appearance-none truncate rounded-[min(var(--radius-md),10px)] border border-input bg-transparent py-1 pr-7 pl-2.5 text-sm outline-none transition-colors focus-visible:border-ring focus-visible:ring-3 focus-visible:ring-ring/50 disabled:cursor-not-allowed disabled:opacity-50"
+                className="h-9 w-full max-w-36 appearance-none truncate rounded-[min(var(--radius-md),10px)] border border-input bg-[var(--card)] py-0 pr-8 pl-3 text-[13px] leading-9 text-[var(--ink)] outline-none transition-colors focus-visible:border-ring focus-visible:ring-3 focus-visible:ring-ring/50 disabled:cursor-not-allowed disabled:opacity-50"
               >
                 {modelOptions.map((model) => (
                   <option key={model.id} value={model.id}>
diff --git a/tests/e2e/assistant.spec.ts b/tests/e2e/assistant.spec.ts
index b495409..129f7ef 100644
--- a/tests/e2e/assistant.spec.ts
+++ b/tests/e2e/assistant.spec.ts
@@ -29,6 +29,9 @@ test("assistant chat smoke after opt-in", async ({ page }) => {
   const modelSelector = page.getByRole("combobox", { name: "Assistant model" });
   await expect(modelSelector).toBeVisible();
   await expect.poll(() => modelSelector.evaluate((node) => node.tagName)).toBe("SELECT");
+  await expect
+    .poll(() => modelSelector.evaluate((node) => node.getBoundingClientRect().height))
+    .toBeGreaterThanOrEqual(36);
 
   await page.getByLabel("Message for Assistant").fill("hello assistant");
   await page.getByRole("button", { name: "Send" }).click();