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

fix(config): expand provider base_url templates before validation

Browse source
This commit is contained in:
刘昊 2026-04-23 17:20:25 +08:00
parent 5e76c650bb
commit 88d71e7535
3 changed files with 48 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

34
tests/hermes_cli/test_config.py
View file

@ -482,7 +482,7 @@ class TestCustomProviderCompatibility:
"providers": {
"openai-direct": {
"api": "https://api.openai.com/v1",
"api_key": "test-key",
"api_key": "***",
"default_model": "gpt-5-mini",
"name": "OpenAI Direct",
"transport": "codex_responses",
@ -502,6 +502,38 @@ class TestCustomProviderCompatibility:
assert compatible[0]["provider_key"] == "openai-direct"
assert compatible[0]["api_mode"] == "codex_responses"
def test_providers_dict_expands_env_placeholders_before_url_validation(self, tmp_path):
"""Runtime compat view should accept env-backed provider URLs from providers dict."""
config_path = tmp_path / "config.yaml"
config_path.write_text(
yaml.safe_dump(
{
"_config_version": 17,
"providers": {
"provider-a": {
"name": "Provider A",
"base_url": "${PROVIDER_A_BASE_URL}",
}
},
}
),
encoding="utf-8",
)
with patch.dict(
os.environ,
{"HERMES_HOME": str(tmp_path), "PROVIDER_A_BASE_URL": "https://provider-a.example.com/v1"},
):
compatible = get_compatible_custom_providers()
assert compatible == [
{
"name": "Provider A",
"base_url": "https://provider-a.example.com/v1",
"provider_key": "provider-a",
}
]
def test_compatible_custom_providers_prefers_base_url_then_url_then_api(self, tmp_path):
"""URL field precedence is base_url > url > api (PR #9332)."""
config_path = tmp_path / "config.yaml"