Skip to content

feat(gateway): show home path and model in /status - #6750

Open
xinbenlv wants to merge 1 commit into
NousResearch:mainfrom
xinbenlv:feat/gateway-status-path-model
Open

xinbenlv wants to merge 1 commit into
NousResearch:mainfrom
xinbenlv:feat/gateway-status-path-model

Conversation

@xinbenlv

@xinbenlv xinbenlv commented Apr 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add the active Hermes home path to gateway /status
  • show the session's current model/provider in /status
  • keep the existing session title display intact and cover the new output with regression tests

Upstream context

Test Plan

  • ~/.hermes/hermes-agent/venv/bin/python -m pytest tests/gateway/test_status_command.py -q -o addopts=''

Contributed back from https://github.com/xinbenlv/zn-hermes-agent, which is being used and tested by https://github.com/xinbenlv.

Carried patch: patch-feat-pr6750
Contributed back from https://github.com/xinbenlv/zn-hermes-agent, which is being used and tested by https://github.com/xinbenlv.

xinbenlv added a commit to xinbenlv/zn-hermes-agent that referenced this pull request Apr 10, 2026
## Why carried
- Makes `/status` show which Hermes home/profile the gateway session is actually using.
- Exposes the active model/provider directly instead of making users guess from config or logs.

## Upstream
- PR: NousResearch#6750
- Status: open

## Summary
- Extends gateway `/status` output with home-path and model visibility.
- Keeps the status output operationally useful when multiple homes or models are in play.

## Drop condition
- Drop once PR NousResearch#6750 lands upstream and local `main` rebases onto the upstream commit.

## Files
- `gateway/run.py`
- `tests/gateway/test_status_command.py`
peteradams2026 pushed a commit to xinbenlv/zn-hermes-agent that referenced this pull request Apr 28, 2026
---
upstream: NousResearch#6750
files:
  - gateway/run.py
  - tests/gateway/test_status_command.py
---

## Summary
- Extend gateway `/status` with the active Hermes home/profile path.
- Show the effective model/provider directly in `/status`.

## Drop condition
- Drop once PR NousResearch#6750 lands upstream and local `main` rebases onto the upstream commit.
@alt-glitch alt-glitch added type/feature New feature or request P3 Low — cosmetic, nice to have comp/gateway Gateway runner, session dispatch, delivery labels Apr 29, 2026
xinbenlv added a commit to xinbenlv/zn-hermes-agent that referenced this pull request Apr 30, 2026
---
upstream: NousResearch#6750
files:
  - gateway/run.py
  - tests/gateway/test_status_command.py
---

## Upstream
- PR: NousResearch#6750
- Status: open

## Summary
- Extend gateway `/status` with the active Hermes home/profile path.
- Show the effective model/provider directly in `/status`.

## Drop condition
- Drop once PR NousResearch#6750 lands upstream and local main rebases onto the upstream commit.

@teknium1 teknium1 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the /status visibility improvement. The Hermes-home path is still absent on current main, but the model/provider portion has since been implemented more completely.

Problems

  • Current /status lives in gateway/slash_commands.py:493-666, following extraction commit 619bd7827; the PR's gateway/run.py handler is no longer the active implementation.
  • Model/provider status is already covered by gateway/slash_commands.py:550-610 and tests/gateway/test_status_command.py:181-275, added by ead38107a. It resolves live, cached, persisted, and configured state, which is stronger than the PR's older resolver.
  • Status output is localized (locales/en.yaml:285-306), so the remaining path line should use a new locale key rather than a hardcoded English string.

Suggested changes

  • Port only the profile-aware home-path addition into GatewaySlashCommandsMixin._handle_status_command, add locale entries, and add a focused regression test.

Automated hermes-sweeper review.

Comment thread gateway/run.py
"📊 **Hermes Gateway Status**",
"",
f"**Session ID:** `{session_entry.session_id}`",
f"**Path:** `{display_hermes_home()}`",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Current main moved /status to gateway/slash_commands.py and localizes its output through t(...). Please port the remaining path feature there as a gateway.status.path locale key (with catalog entries), rather than adding this hardcoded line to the former handler.

@teknium1 teknium1 added sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades sweeper:blast-contained Sweeper blast radius: contained — one narrow path / opt-in / few users labels Jul 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/gateway Gateway runner, session dispatch, delivery P3 Low — cosmetic, nice to have sweeper:blast-contained Sweeper blast radius: contained — one narrow path / opt-in / few users sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants