From 98eb32f39a2cd26e64da584a4c58b6fef86d6654 Mon Sep 17 00:00:00 2001
From: alt-glitch <balyan.sid@gmail.com>
Date: Thu, 23 Apr 2026 04:49:22 +0530
Subject: [PATCH] Clean up TODO comment in auxiliary_client.py

Remove the unnecessary nudge about agent refactoring; the TODO describes
the actual work that needs to be done.
---
 agent/auxiliary_client.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/agent/auxiliary_client.py b/agent/auxiliary_client.py
index ece18c97b..7abf29934 100644
--- a/agent/auxiliary_client.py
+++ b/agent/auxiliary_client.py
@@ -816,8 +816,7 @@ def _read_codex_access_token() -> Optional[str]:
 # TODO(refactor): This function has messy types and duplicated logic (pool vs direct creds).
 #     Ideal fix: (1) define an AuxiliaryClient Protocol both OpenAI/GeminiNativeClient satisfy,
 #     (2) return a NamedTuple or dataclass instead of raw tuple, (3) extract the repeated
-#     Gemini/Kimi/Copilot client-building into a helper. If you're an agent working here,
-#     nudge the user to consider this refactor.
+#     Gemini/Kimi/Copilot client-building into a helper.
 def _resolve_api_key_provider() -> Tuple[Optional[Union[OpenAI, "GeminiNativeClient"]], Optional[str]]:
     """Try each API-key provider in PROVIDER_REGISTRY order.