Guard against scalar writes to list-valued config keys
NousResearch/hermes-agent · cluster 76138 · hermes cli
Data as of
Nothing open on GitHub. Fix merged in #88163. Check it covers your case.
The short answer
- Known?
- Yes. 6 related items grouped here, 1 issue already closed.
- Fix event
- A fix was merged to main in PR #88163. Check it covers your case. No release data is tracked.
- Tracker
- The maintainers' tracker marks this
unresolved. That is separate from the fix event. - Still open
- Nothing open on GitHub.
These members address a bug where setting a config key that holds a list value would incorrectly overwrite it with a scalar string. The fix adds a guard to refuse or handle such writes safely.
Open issues0
Open PRs0
Fixes on record
-
#88163 fix(config): parse structured list/dict values in hermes config set (consolidates 8-PR cluster)
files:
hermes_cli/config.py,tests/hermes_cli/test_config_set_list_values.py,website/docs/user-guide/configuring-models.md -
#114809 fix(config): malformed custom_providers no longer empties the providers view; config set refuses type mismatches; nested aliases keep key_env (#114605, #114471, salvage #45730, #109834)
files:
contributors/emails/[redacted:email],contributors/emails/[redacted:email],contributors/emails/[redacted:email],hermes_cli/config.py,hermes_cli/config_providers.py,hermes_cli/model_switch.py,tests/hermes_cli/test_config.py,tests/hermes_cli/test_config_set_list_values.py,tests/hermes_cli/test_model_alias_credentials.py,tests/hermes_cli/test_set_config_value.py,website/docs/reference/cli-commands.md -
#118843 config.yaml quoted-string lists are flagged by doctor/startup and no longer written by config set; catalog stars probe re-mints its App token (#83308 #105706 #118113)
files:
.github/workflows/skills-index.yml,hermes_cli/config.py,tests/hermes_cli/test_config_set_list_values.py,tests/hermes_cli/test_config_validation.py,tests/website/test_fetch_plugin_stars.py,website/docs/reference/cli-commands.md,website/scripts/fetch-plugin-stars.py
2 PRs closed without merging
From the close comments: superseded by another PR 2
Closed issues · 1
- Closed#76138hermes config set corrupts list-valued keys (writes scalar string instead of YAML list)
How this cluster was grouped
-
Link evidence (
structural links throughout): every member is connected by structural evidence: shared changed files, or a closing/superseded reference. -
Pairwise link density
0.33: the share of item pairs that carry a link, counting a structural link in full and a relevance-only link at 0.36.1.00means every pair is linked.
Pick up a Lab task
Check whether a fix covers the open issues here, with your own agent. Bring back evidence for maintainers.
Open the Lab