From c2dbe5826d1a103d350ccb77bb3f497b97cb2918 Mon Sep 17 00:00:00 2001 From: "devin-ai-integration[bot]" <158243242+devin-ai-integration[bot]@users.noreply.github.com> Date: Wed, 1 Jul 2026 04:07:25 +0000 Subject: [PATCH] fix(ui): prevent destructure crash in Auto Router test connection prepareModelAddRequest returns an empty array for auto router forms (no model_mappings field), causing result[0] to be undefined when destructuring litellmParamsObj. Two fixes: - Guard against empty result array in model_connection_test.tsx - Pass properly constructed model_mappings from the auto router tab so test connection actually works for auto routers --- .../src/components/add_model/add_auto_router_tab.tsx | 12 ++++++++++-- .../components/add_model/model_connection_test.tsx | 2 +- 2 files changed, 11 insertions(+), 3 deletions(-) diff --git a/ui/litellm-dashboard/src/components/add_model/add_auto_router_tab.tsx b/ui/litellm-dashboard/src/components/add_model/add_auto_router_tab.tsx index 9e9a097e3fe..5eab6fd35de 100644 --- a/ui/litellm-dashboard/src/components/add_model/add_auto_router_tab.tsx +++ b/ui/litellm-dashboard/src/components/add_model/add_auto_router_tab.tsx @@ -436,11 +436,19 @@ const AddAutoRouterTab: React.FC = ({ form, handleOk, acc ]} width={700} > - {/* Only render the ConnectionErrorDisplay when modal is visible and we have a test ID */} {isResultModalVisible && ( = ({ console.log("Testing connection with form values:", formValues); const result = await prepareModelAddRequest(formValues, accessToken, null); - if (!result) { + if (!result || result.length === 0) { console.log("No result from prepareModelAddRequest"); setError("Failed to prepare model data. Please check your form inputs."); setIsSuccess(false);