Skip to content

feat(gateway): add model and provider info to /status command - #16079

Open
KickUasS wants to merge 1 commit into
NousResearch:mainfrom
KickUasS:feature/status-model-info
Open

feat(gateway): add model and provider info to /status command#16079
KickUasS wants to merge 1 commit into
NousResearch:mainfrom
KickUasS:feature/status-model-info

Conversation

@KickUasS

Copy link
Copy Markdown

Enhance the /status command to display current model and provider information.

Changes made:

  • Read model/provider from config.yaml as baseline
  • Check _session_model_overrides for active session-level overrides
  • Display model name with (session override) indicator when applicable
  • Show provider label for better readability

This makes it easy to verify which model is being used by the gateway.

Enhance the /status command to display current model and provider information.
Shows session-level overrides when /model command is used to switch models.

- Read model/provider from config.yaml as baseline
- Check _session_model_overrides for active session-level overrides
- Display model name with (session override) indicator when applicable
- Show provider label for better readability
@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 26, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Likely duplicate of #11079 — same feature (add model/provider info to /status), same file changed (gateway/run.py).

@teknium1

Copy link
Copy Markdown
Contributor

Thanks for addressing gateway model visibility. The broader model/provider status feature was already salvaged into main in ead38107a2f2b6d6a71e92e978fe98a684bd3be8, but the session-override edge case remains worth preserving.

Problems

  • This patch targets gateway/run.py, but /status now lives in gateway/slash_commands.py:493 after 619bd782738adfa87544d0fda7a1114defe86e4c; it needs a targeted port.
  • The current gap is specifically post-/model: the handler stores an override at gateway/slash_commands.py:1868-1883 and evicts the cache at :1891, while /status falls back through SessionDB/config at :582-610. hermes_state.py:2363-2375 persists only the model column, so provider can be stale after a cross-provider session switch.
  • tests/gateway/test_status_command.py:180-276 has model/provider coverage but no immediate /model/status regression case.

Suggested changes

  • Port override-aware resolution into GatewaySlashCommandsMixin._handle_status_command, preserving the localized status output at gateway/slash_commands.py:605-610.
  • Cover an evicted-cache, cross-provider session switch; include the persisted SessionEntry.model_override path.

Automated hermes-sweeper review.

@teknium1 teknium1 added sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades sweeper:blast-moderate Sweeper blast radius: moderate — a subsystem or single platform 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-moderate Sweeper blast radius: moderate — a subsystem or single platform 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.

4 participants