diff --git a/ui/litellm-dashboard/src/components/mcp_tools/MCPToolArgumentsForm.integration.test.tsx b/ui/litellm-dashboard/src/components/mcp_tools/MCPToolArgumentsForm.integration.test.tsx index fb0bb2f9f0e..0da3628a76b 100644 --- a/ui/litellm-dashboard/src/components/mcp_tools/MCPToolArgumentsForm.integration.test.tsx +++ b/ui/litellm-dashboard/src/components/mcp_tools/MCPToolArgumentsForm.integration.test.tsx @@ -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 }); }); diff --git a/ui/litellm-dashboard/src/components/mcp_tools/MCPToolArgumentsForm.tsx b/ui/litellm-dashboard/src/components/mcp_tools/MCPToolArgumentsForm.tsx index 64032659ed2..7d0b7fc1bd7 100644 --- a/ui/litellm-dashboard/src/components/mcp_tools/MCPToolArgumentsForm.tsx +++ b/ui/litellm-dashboard/src/components/mcp_tools/MCPToolArgumentsForm.tsx @@ -358,7 +358,9 @@ const MCPToolArgumentsForm = forwardRef - + + {field.value === "" ? "Empty string" : undefined} + {!required && Select {key}}