common/hermes
2
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

fix(setup): use explicit key mapping for returning-user menu dispatch instead of positional index (#3083)

Browse Source 
Co-authored-by: ygd58 <buraysandro9@gmail.com>
This commit is contained in:
Teknium
2026-03-25 17:14:43 -07:00
committed by GitHub
parent 9783c9d5c1
commit 0d7f739675
1 changed files with 11 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
hermes_cli/setup.py
View File

@@ -3234,12 +3234,17 @@ def run_setup_wizard(args):
print_info("Exiting. Run 'hermes setup' again when ready.") print_info("Exiting. Run 'hermes setup' again when ready.")
return return
elif 3 <= choice <= 7: elif 3 <= choice <= 7:
# Individual section # Individual section — map by key, not by position.
section_idx = choice - 3 # SETUP_SECTIONS includes TTS but the returning-user menu skips it,
_, label, func = SETUP_SECTIONS[section_idx] # so positional indexing (choice - 3) would dispatch the wrong section.
func(config) _RETURNING_USER_SECTION_KEYS = ["model", "terminal", "gateway", "tools", "agent"]
save_config(config) section_key = _RETURNING_USER_SECTION_KEYS[choice - 3]
_print_setup_summary(config, hermes_home) section = next((s for s in SETUP_SECTIONS if s[0] == section_key), None)
if section:
_, label, func = section
func(config)
save_config(config)
_print_setup_summary(config, hermes_home)
return return
else: else:
# ── First-Time Setup ── # ── First-Time Setup ──