mirrors/hermes-agent
1
0
Fork 0
mirror of https://github.com/NousResearch/hermes-agent.git synced 2026-04-25 00:51:20 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Clean up TODO comment in auxiliary_client.py

Browse source 
Remove the unnecessary nudge about agent refactoring; the TODO describes
the actual work that needs to be done.
This commit is contained in:
alt-glitch 2026-04-23 04:49:22 +05:30
parent 2df306e6cd
commit 98eb32f39a
1 changed files with 1 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

3
agent/auxiliary_client.py
View file

@ -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.