ouyangbo/hermes-agent
1
0
Fork 0
Code Issues Pull Requests Actions 9 Packages Projects Releases Wiki Activity

refactor(hermes_cli): _group_display_name extraction; small collapses

Browse Source
This commit is contained in:
Teknium
2026-09-02 23:33:14 -07:00
parent b6de18eb0b
commit c540655e5f
2 changed files with 17 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files
hermes_cli/model_switch.py
+3 -4
View File
@@ -773,8 +773,8 @@ def _resolve_alias_fallback(
AmbiguousAliasError propagates: the alias exists on this provider, the user just has to
choose — trying the next provider would silently switch them somewhere they didn't ask for.
"""
return next((r for r in (resolve_alias(raw_input, p) for p in authenticated_providers or ("openrouter", "nous"))
if r is not None), None)
results = (resolve_alias(raw_input, p) for p in authenticated_providers or ("openrouter", "nous"))
return next((r for r in results if r is not None), None)
def resolve_display_context_length(
@@ -794,8 +794,7 @@ def resolve_display_context_length(
try:
from hermes_cli.route_identity import should_clear_context_pin
if should_clear_context_pin(
configured_model, model, configured_base_url, base_url, configured_provider, provider,
):
configured_model, model, configured_base_url, base_url, configured_provider, provider):
config_context_length = None
except Exception:
config_context_length = None
hermes_cli/model_switch_providers.py
+14 -13
View File
@@ -485,10 +485,19 @@ def _discover_flag(entry: dict):
def _display_prefix(name: str) -> str:
"""Text before the per-model separator Hermes's own writer uses ("—" / " - ")."""
for sep in ("—", " - "):
if sep in name:
return name.split(sep)[0].strip()
return name
return next((name.split(sep)[0].strip() for sep in ("—", " - ") if sep in name), name)
def _group_display_name(display_name: str) -> str:
"""Section-3 row label: strip the per-model suffix and trailing version tokens ("Palantir
Claude 4.7 Opus" -> "Palantir Claude") — cut at the first token containing a digit, only when
>= 2 words remain (avoids over-trimming)."""
grp_display = _display_prefix(display_name)
toks = grp_display.split()
cut_at = next((i for i, t in enumerate(toks) if any(c.isdigit() for c in t.strip(".,()"))), None)
if cut_at is not None and cut_at >= 2:
grp_display = " ".join(toks[:cut_at]).strip()
return grp_display or display_name
def _discover_endpoint_models(
@@ -831,18 +840,10 @@ def _lap_user_provider_rows(b: _PickerBuild, user_providers: dict) -> None:
group_key = (_norm_url(api_url), cred_identity, _entry_api_mode(ep_cfg), tuple(sorted(headers.items())))
if group_key not in ep_groups:
# Strip the per-model suffix and trailing version tokens ("Palantir Claude 4.7 Opus"
# -> "Palantir Claude"): cut at the first token with a digit, only when >=2 words
# remain (avoids over-trimming).
grp_display = _display_prefix(display_name)
toks = grp_display.split()
cut_at = next((i for i, t in enumerate(toks) if any(c.isdigit() for c in t.strip(".,()"))), None)
if cut_at is not None and cut_at >= 2:
grp_display = " ".join(toks[:cut_at]).strip()
# slug = first ep_name encountered; probe key from the first member (inline api_key,
# else key_env through the per-profile secret scope).
ep_groups[group_key] = {
"slug": ep_name, "name": grp_display or display_name, "api_url": api_url, "models": [],
"slug": ep_name, "name": _group_display_name(display_name), "api_url": api_url, "models": [],
"has_explicit_models": False, "api_key": inline_api_key or _scoped_key_env(key_env),
"headers": headers, "api_mode": ep_cfg.get("api_mode"),
"discovery_allowed": bool(api_url) and _discover_flag(ep_cfg), "raw_names": [], "aliases": set()}