Skip to content

fix: setup wizard overwrites custom endpoint config with OpenRouter defaults #3688

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
patlegu wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix: setup wizard overwrites custom endpoint config with OpenRouter defaults #3688

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
10 changes: 6 additions & 4 deletions hermes_cli/setup.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1054,10 +1054,12 @@ def setup_model_provider(config: dict):
# env saving, config.yaml updates, and custom_providers persistence.
from hermes_cli.main import _model_flow_custom
_model_flow_custom(config)
# _model_flow_custom handles model selection, config, env vars,
# and custom_providers. Keep selected_provider = "custom" so
# the model selection step below is skipped (line 1631 check)
# but vision and TTS setup still run.
# _model_flow_custom saves config independently (loads its own copy).
# Reload here so the final save_config(config) below doesn't overwrite it.
from hermes_cli.config import load_config as _reload_config
config.update(_reload_config())
# Keep selected_provider = "custom" so the model selection step below is
# skipped (line 1631 check) but vision and TTS setup still run.

elif provider_idx == 4: # Z.AI / GLM
selected_provider = "zai"
Expand Down