fix: three CLI quality-of-life fixes by teknium1 · Pull Request #10599 · NousResearch/hermes-agent

teknium1 · 2026-04-16T00:33:15Z

Summary

Three independent CLI fixes batched together. 3 files, 28 lines net.

1. `hermes auth add` crashes on non-interactive stdin (#10468)

input() for the label prompt was called without checking isatty(). In scripted/CI environments this raised EOFError. Fix: check sys.stdin.isatty() and fall back to the computed default label.

2. Subcommand help prints twice (#10230)

hermes dashboard -h printed help text twice because the SystemExit(0) from argparse was caught by the fallback retry logic, which re-parsed and printed help again. Fix: re-raise SystemExit with code 0 (help/version) immediately instead of falling through.

3. Duplicate entries in /model picker (#10526, #9545)

Kimi showed 2x because kimi-coding and kimi-coding-cn both map to the same models.dev ID (kimi-for-coding). Fix: track seen_mdev_ids and skip aliases — first one with valid credentials wins.
Providers showed 2-3x from case-variant slugs across the four loading paths (built-in, hermes-only, canonical, custom). Fix: normalize all seen_slugs checks to lowercase.

Tests

2081 hermes_cli tests pass. 1 pre-existing failure in test_model_validation (unrelated).

Three independent fixes batched together: 1. hermes auth add crashes on non-interactive stdin (#10468) input() for the label prompt was called without checking isatty(). In scripted/CI environments this raised EOFError. Fix: check sys.stdin.isatty() and fall back to the computed default label. 2. Subcommand help prints twice (#10230) 'hermes dashboard -h' printed help text twice because the SystemExit(0) from argparse was caught by the fallback retry logic, which re-parsed and printed help again. Fix: re-raise SystemExit with code 0 (help/version) immediately. 3. Duplicate entries in /model picker (#10526, #9545) - Kimi showed 2x because kimi-coding and kimi-coding-cn both mapped to the same models.dev ID. Fix: track seen mdev_ids and skip aliases. - Providers could show 2-3x from case-variant slugs across the four loading paths. Fix: normalize all seen_slugs membership checks and insertions to lowercase. Closes #10468, #10230, #10526, #9545

…h#10230, NousResearch#10526, NousResearch#9545) (NousResearch#10599) Three independent fixes batched together: 1. hermes auth add crashes on non-interactive stdin (NousResearch#10468) input() for the label prompt was called without checking isatty(). In scripted/CI environments this raised EOFError. Fix: check sys.stdin.isatty() and fall back to the computed default label. 2. Subcommand help prints twice (NousResearch#10230) 'hermes dashboard -h' printed help text twice because the SystemExit(0) from argparse was caught by the fallback retry logic, which re-parsed and printed help again. Fix: re-raise SystemExit with code 0 (help/version) immediately. 3. Duplicate entries in /model picker (NousResearch#10526, NousResearch#9545) - Kimi showed 2x because kimi-coding and kimi-coding-cn both mapped to the same models.dev ID. Fix: track seen mdev_ids and skip aliases. - Providers could show 2-3x from case-variant slugs across the four loading paths. Fix: normalize all seen_slugs membership checks and insertions to lowercase. Closes NousResearch#10468, NousResearch#10230, NousResearch#10526, NousResearch#9545

… collide If the gateway returns two providers that resolve to the same display name (e.g. `kimi-coding` and `kimi-coding-cn` both → "Kimi For Coding"), the picker now appends the slug so users can tell them apart, in both the provider list and the selected-provider header. No-op when names are already unique. Refs #10526 — the Python backend dedupe from #10599 skips one alias, but user-defined providers, canonical overlays, and future regressions can still surface as indistinguishable rows in the picker. This is a client-side safety net on top of that.

… collide If the gateway returns two providers that resolve to the same display name (e.g. `kimi-coding` and `kimi-coding-cn` both → "Kimi For Coding"), the picker now appends the slug so users can tell them apart, in both the provider list and the selected-provider header. No-op when names are already unique. Refs NousResearch#10526 — the Python backend dedupe from NousResearch#10599 skips one alias, but user-defined providers, canonical overlays, and future regressions can still surface as indistinguishable rows in the picker. This is a client-side safety net on top of that.