Skip to content

fix(nous): use curated model list instead of full API dump for Nous Portal#3867

Merged
teknium1 merged 1 commit intomainfrom
hermes/hermes-b8e0ec91
Mar 30, 2026
Merged

fix(nous): use curated model list instead of full API dump for Nous Portal#3867
teknium1 merged 1 commit intomainfrom
hermes/hermes-b8e0ec91

Conversation

@teknium1
Copy link
Copy Markdown
Contributor

Problem

All three Nous Portal model selection paths were calling fetch_nous_models() which hits the live /models endpoint and returns every model available — potentially hundreds. Users selecting Nous Portal via hermes model got an overwhelming dump instead of the curated agentic models we show for OpenRouter.

Fix

Replaced fetch_nous_models() calls with the curated _PROVIDER_MODELS['nous'] list (25 models, same as OpenRouter defaults). Users can still pick unlisted models via "Enter custom model name."

Three paths fixed:

  • hermes_cli/main.py: _model_flow_nous() — already-logged-in model picker
  • hermes_cli/auth.py: _login_nous() — first-time login model selection
  • hermes_cli/setup.py: post-login model selection in setup wizard

Tests

7070 passed, 2 pre-existing failures, 0 regressions.

…ortal

All three Nous Portal model selection paths (hermes model, first-time
login, setup wizard) were hitting the live /models endpoint and showing
every model available — potentially hundreds. Now uses the curated
_PROVIDER_MODELS['nous'] list (25 agentic models matching OpenRouter
defaults) with 'Enter custom model name' for anything else.

Fixed in:
- hermes_cli/main.py: _model_flow_nous()
- hermes_cli/auth.py: _login_nous() model selection
- hermes_cli/setup.py: post-login model selection
@github-actions
Copy link
Copy Markdown

⚠️ Supply Chain Risk Detected

This PR contains patterns commonly associated with supply chain attacks. This does not mean the PR is malicious — but these patterns require careful human review before merging.

⚠️ WARNING: Install hook files modified

These files can execute code during package installation or interpreter startup.

Files:

hermes_cli/setup.py

Automated scan triggered by supply-chain-audit. If this is a false positive, a maintainer can approve after manual review.

@teknium1 teknium1 merged commit ccf7bb1 into main Mar 30, 2026
4 of 5 checks passed
itsXactlY pushed a commit to itsXactlY/hermes-agent that referenced this pull request Mar 30, 2026
…ortal (NousResearch#3867)

All three Nous Portal model selection paths (hermes model, first-time
login, setup wizard) were hitting the live /models endpoint and showing
every model available — potentially hundreds. Now uses the curated
_PROVIDER_MODELS['nous'] list (25 agentic models matching OpenRouter
defaults) with 'Enter custom model name' for anything else.

Fixed in:
- hermes_cli/main.py: _model_flow_nous()
- hermes_cli/auth.py: _login_nous() model selection
- hermes_cli/setup.py: post-login model selection
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant