Skip to content

fix: make config set examples use placeholder syntax#957

Closed
stablegenius49 wants to merge 1 commit intoNousResearch:mainfrom
stablegenius49:pr-factory/issue-926-config-placeholders
Closed

fix: make config set examples use placeholder syntax#957
stablegenius49 wants to merge 1 commit intoNousResearch:mainfrom
stablegenius49:pr-factory/issue-926-config-placeholders

Conversation

@stablegenius49
Copy link
Contributor

Summary

  • replace literal KEY VALUE examples with <key> <value> placeholders in the setup summary and config CLI output
  • update the config set usage text so copy-paste examples no longer write a literal KEY: VALUE entry into config
  • add focused CLI output tests covering the setup summary and missing-args usage path

Testing

  • /opt/homebrew/bin/python3.11 -m pytest -o addopts='' tests/hermes_cli/test_config.py tests/hermes_cli/test_set_config_value.py tests/hermes_cli/test_placeholder_usage.py -q

Closes #926

@teknium1
Copy link
Contributor

Merged via PR #1322. Your substantive commit was cherry-picked onto current main with authorship preserved, with a small follow-up for the remaining fallback help text. Thanks for the fix.

@teknium1 teknium1 closed this Mar 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Setup completion screen: 'hermes config set KEY VALUE' example is copy-paste unsafe

3 participants