PR #16807: fix(cli): show effective terminal timeout in /config

• Repository: NousResearch/hermes-agent
• Author: BlackishGreen33
• State: open | merged: False
• Link: https://github.com/NousResearch/hermes-agent/pull/16807

Description (problem / solution / changelog)

What does this PR do?

Fixes the interactive CLI /config display so the Terminal section uses the terminal settings already loaded by Hermes config before falling back to TERMINAL_* environment variables.

This addresses a case where /config could show Timeout: 60s even though config.yaml and hermes config show reported terminal.timeout: 3600.

Related Issue

Fixes #16723

Type of Change

• 🐛 Bug fix (non-breaking change that fixes an issue)
• ✨ New feature (non-breaking change that adds functionality)
• 🔒 Security fix
• 📝 Documentation update
• ✅ Tests (adding or improving test coverage)
• ♻️ Refactor (no behavior change)
• 🎯 New skill (bundled or hub)

Changes Made

• Updated HermesCLI.show_config() to read terminal backend / env_type, cwd, and timeout from the loaded CLI config first.
• Kept environment variables as a fallback when a value is not present in config.
• Added regression tests for config/env timeout mismatch and backend taking precedence over legacy env_type.

How to Test

1. source /Users/blackishgreen03/workspace/hermes-agent/venv/bin/activate && python -m pytest tests/cli/test_cli_init.py -q
2. Result: 28 passed
3. Full suite was attempted with python -m pytest tests/ -q, but this local environment is not clean for the whole repository: it fails with missing optional packages such as acp, numpy, and fastapi, plus many unrelated existing gateway/web/tool failures (16556 passed, 117 skipped, 191 failed, 66 errors).

Checklist

Code

• I've read the Contributing Guide
• My commit messages follow Conventional Commits (fix(scope):, feat(scope):, etc.)
• I searched for existing PRs to make sure this isn't a duplicate
• My PR contains only changes related to this fix/feature (no unrelated commits)
• I've run pytest tests/ -q and all tests pass
• I've added tests for my changes (required for bug fixes, strongly encouraged for features)
• I've tested on my platform: macOS, Python 3.11 using the existing local repo venv

Documentation & Housekeeping

• I've updated relevant documentation (README, docs/, docstrings) — or N/A
• I've updated cli-config.yaml.example if I added/changed config keys — or N/A
• I've updated CONTRIBUTING.md or AGENTS.md if I changed architecture or workflows — or N/A
• I've considered cross-platform impact (Windows, macOS) per the compatibility guide — or N/A
• I've updated tool descriptions/schemas if I changed tool behavior — or N/A

For New Skills

N/A

Screenshots / Logs

Targeted test:

28 passed in 11.83s

Changed files

• cli.py (modified, +19/-4)
• tests/cli/test_cli_init.py (modified, +38/-0)

PR #16840: fix(cli): show effective terminal timeout in /config

• Repository: NousResearch/hermes-agent
• Author: irun2themoney
• State: open | merged: False
• Link: https://github.com/NousResearch/hermes-agent/pull/16840

Description (problem / solution / changelog)

Summary

• read terminal backend/cwd/timeout from effective loaded config in /config display
• keep env values as fallback only, so /config no longer reports stale default timeout values
• align displayed timeout with config.yaml loaded runtime settings

Test plan

• Inspect /config implementation path and config load precedence
• Validate output logic now uses effective config values
• Run Hermes CLI and compare /config timeout against hermes config show

Made with Cursor

Changed files

• cli.py (modified, +14/-4)