From 9d862a65831cf1a7dc8ba6a7e69900d3774dc85e Mon Sep 17 00:00:00 2001
From: mateo-berri <277851410+mateo-berri@users.noreply.github.com>
Date: Thu, 3 Sep 2026 04:27:43 -0700
Subject: [PATCH] style: drop explanatory comments from the agent publish fix

---
 litellm/proxy/agent_endpoints/endpoints.py   | 2 --
 tests/e2e/ui/tests/modelHub/modelHub.spec.ts | 3 +--
 2 files changed, 1 insertion(+), 4 deletions(-)

diff --git a/litellm/proxy/agent_endpoints/endpoints.py b/litellm/proxy/agent_endpoints/endpoints.py
index ec50500fdb2..cc17672553b 100644
--- a/litellm/proxy/agent_endpoints/endpoints.py
+++ b/litellm/proxy/agent_endpoints/endpoints.py
@@ -939,8 +939,6 @@ async def make_agent_public(
             if agent is None:
                 raise HTTPException(status_code=404, detail=f"Agent with ID {agent_id} not found")
 
-        # get_config() re-applies the DB's litellm_settings over the in-memory
-        # globals, so read it before deriving the new list and assign the global after
         config: Final = await proxy_config.get_config()
 
         current_public_agent_groups: Final = list(litellm.public_agent_groups or [])
diff --git a/tests/e2e/ui/tests/modelHub/modelHub.spec.ts b/tests/e2e/ui/tests/modelHub/modelHub.spec.ts
index 7c43b9f761c..1fa3e4c530e 100644
--- a/tests/e2e/ui/tests/modelHub/modelHub.spec.ts
+++ b/tests/e2e/ui/tests/modelHub/modelHub.spec.ts
@@ -22,8 +22,7 @@ test.describe("AI Hub (internal admin view)", () => {
     // on the disabled-Next button or the success toast.
     await expect(modal.getByText(/Select All \(\d+\)/)).toBeVisible({ timeout: 5_000 });
 
-    // Step 1: pick the seeded models via "Select All". check() rather than click() because
-    // the modal preselects groups that are already public, and a click would clear them.
+    // Step 1: pick the seeded models via "Select All"
     await modal.getByRole("checkbox", { name: /Select All/ }).check();
 
     // Move to confirm step