scriptoriumadmin/litellm
1
0
Fork 0
mirror of https://github.com/BerriAI/litellm.git synced 2026-09-14 23:21:35 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 102

fix(ui): label explicit empty tool selections

Browse source
This commit is contained in:
Yuneng Jiang 2026-09-11 19:16:09 -07:00
parent 0587e3b8f8
commit dea10d4fdb
No known key found for this signature in database
2 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
ui/litellm-dashboard/src/components/mcp_tools/MCPToolArgumentsForm.integration.test.tsx
View file

@ -307,5 +307,6 @@ it("should distinguish an unset enum from empty string and retain explicit false
await expect(submit(ref)).resolves.toEqual({ active: false });
await user.click(screen.getByRole("combobox", { name: "mode" }));
await user.click(await screen.findByRole("option", { name: "Empty string" }));
expect(screen.getByRole("combobox", { name: "mode" })).toHaveTextContent("Empty string");
await expect(submit(ref)).resolves.toEqual({ mode: "", active: false });
});

4
ui/litellm-dashboard/src/components/mcp_tools/MCPToolArgumentsForm.tsx
View file

@ -358,7 +358,9 @@ const MCPToolArgumentsForm = forwardRef<MCPToolArgumentsFormRef, MCPToolArgument
aria-invalid={field["aria-invalid"]}
className="w-full"
>
<SelectValue placeholder={`Select ${key}`} />
<SelectValue placeholder={`Select ${key}`}>
{field.value === "" ? "Empty string" : undefined}
</SelectValue>
</SelectTrigger>
<SelectContent>
{!required && <SelectItem value={null}>Select {key}</SelectItem>}