Skip to content

fix: make config set examples use placeholder syntax#1322

Merged
teknium1 merged 2 commits intomainfrom
hermes/hermes-2f2b4807
Mar 14, 2026
Merged

fix: make config set examples use placeholder syntax#1322
teknium1 merged 2 commits intomainfrom
hermes/hermes-2f2b4807

Conversation

@teknium1
Copy link
Contributor

Summary

  • cherry-pick stablegenius49's placeholder syntax fix from fix: make config set examples use placeholder syntax #957 onto current main
  • update the remaining config fallback help text to use <key> <value> too
  • add regression tests for setup summary, config usage, show config output, and fallback help output

Testing

  • source /home/teknium/.hermes/hermes-agent/.venv/bin/activate && python -m pytest tests/hermes_cli/test_placeholder_usage.py tests/hermes_cli/test_set_config_value.py tests/hermes_cli/test_config.py -n0 -q
  • source /home/teknium/.hermes/hermes-agent/.venv/bin/activate && python -m pytest tests/ -n0 -q
  • source /home/teknium/.hermes/hermes-agent/.venv/bin/activate && python -m hermes_cli.main config set
  • source /home/teknium/.hermes/hermes-agent/.venv/bin/activate && python -m hermes_cli.main config show

Closes #926

Credit to @stablegenius49 for the original fix in #957.

dadi-sisi and others added 2 commits March 14, 2026 10:34
Update the unknown-subcommand config help output to use placeholder syntax too,
and extend the placeholder regression tests to cover show_config() and that
fallback help path.
@teknium1 teknium1 merged commit d2e2d6e into main Mar 14, 2026
1 check passed
@teknium1 teknium1 deleted the hermes/hermes-2f2b4807 branch March 14, 2026 17:38
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

2 participants