Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions hermes_cli/auth.py
Original file line number Diff line number Diff line change
Expand Up @@ -1495,6 +1495,7 @@ def resolve_provider(
"minimax-portal": "minimax-oauth", "minimax-global": "minimax-oauth", "minimax_oauth": "minimax-oauth",
"alibaba_coding": "alibaba-coding-plan", "alibaba-coding": "alibaba-coding-plan",
"alibaba_coding_plan": "alibaba-coding-plan",
"openai": "openai-api",
"claude": "anthropic", "claude-code": "anthropic",
"github": "copilot", "github-copilot": "copilot",
"github-models": "copilot", "github-model": "copilot",
Expand Down
1 change: 1 addition & 0 deletions hermes_cli/models.py
Original file line number Diff line number Diff line change
Expand Up @@ -1095,6 +1095,7 @@ def group_providers(slugs):
"minimax-portal": "minimax-oauth",
"minimax-global": "minimax-oauth",
"minimax_oauth": "minimax-oauth",
"openai": "openai-api",
"claude": "anthropic",
"claude-code": "anthropic",
"deep-seek": "deepseek",
Expand Down
43 changes: 43 additions & 0 deletions tests/hermes_cli/test_openai_provider_alias.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,43 @@
"""resolve_provider must accept "openai" as an alias for "openai-api".

hermes-agent registers its built-in OpenAI provider as ``openai-api`` in
``PROVIDER_REGISTRY``. Consumers (WebUI model picker, config.yaml, CLI
``--provider`` flag) may reasonably send the bare slug ``openai``.
Without an alias, ``resolve_provider("openai")`` raises ``AuthError``.
"""

import pytest

from hermes_cli.auth import resolve_provider, PROVIDER_REGISTRY


class TestOpenaiAlias:
"""The bare slug "openai" must resolve to the "openai-api" registry entry."""

def test_openai_resolves_to_openai_api(self):
assert resolve_provider("openai") == "openai-api"

def test_case_insensitive(self):
assert resolve_provider("OpenAI") == "openai-api"
assert resolve_provider("OPENAI") == "openai-api"

def test_canonical_openai_api_unchanged(self):
assert resolve_provider("openai-api") == "openai-api"

def test_openai_codex_unchanged(self):
assert resolve_provider("openai-codex") == "openai-codex"

def test_openai_api_in_registry(self):
assert "openai-api" in PROVIDER_REGISTRY


class TestOpenaiModelsAlias:
"""models.py alias table must also map "openai" -> "openai-api"."""

def test_models_alias(self):
from hermes_cli.models import _PROVIDER_ALIASES
assert _PROVIDER_ALIASES.get("openai") == "openai-api"

def test_models_normalize_provider(self):
from hermes_cli.models import normalize_provider
assert normalize_provider("openai") == "openai-api"