feat(gateway): add hermes gateway list to show all profiles' status (salvage #19129) - #21225
Merged
Conversation
…y status
Add a new `hermes gateway list` subcommand that shows the running
status of gateways across all profiles in a single view:
Gateways:
✓ default (current) — PID 155469
✓ wx1 — PID 166893
✗ dev — not running
Also includes `_print_other_profiles_gateway_status()` which appends
an "Other profiles" section to `hermes gateway status` output when
other profile gateways are running.
Both use existing `list_profiles()` and `find_profile_gateway_processes()`
— no new dependencies.
Closes #19127
Related: #19113, #4402, #4587
Contributor
🔎 Lint report:
|
alt-glitch
added
type/feature
1 task
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #19129 via salvage. Closes #19127. Related: #19113.
Summary
hermes gateway listshows gateway status across all profiles in one view:Uses existing
list_profiles()andfind_profile_gateway_processes()— no new deps.Note: The
_print_other_profiles_gateway_status()helper referenced by the original PR is already on main (landed separately); this salvage just adds the newlistsubcommand on top.Validation
Live-tested:
python -c 'from hermes_cli.gateway import _gateway_list; _gateway_list()'prints gateway table correctly on this box.scripts/run_tests.sh tests/hermes_cli/ -k gateway→ 317 passed (1 flaky failure intest_setup_openclaw_migrationunrelated — passes in isolation).Original author: @cixuuz.