Skip to content

fix(cli): /whoami no longer reports Unknown command in the classic CLI (salvage #26047) - #90381

Merged
teknium1 merged 1 commit into
mainfrom
fix/cli-whoami-dispatch
Aug 20, 2026
Merged

teknium1 merged 1 commit into
mainfrom
fix/cli-whoami-dispatch

Conversation

@teknium1

Copy link
Copy Markdown
Collaborator

Summary

/whoami now dispatches in the classic CLI instead of printing Unknown command: /whoami.

Salvage of #26047 by @kronexoi (earliest of 4 independent fixes for this bug), cherry-picked onto current main with authorship preserved.

Root cause: the command exists in the shared COMMAND_REGISTRY (so it appears in /help and autocomplete on every surface), but cli.py's process_command() never had a dispatch branch for it. Refs #74594, #35892.

Changes

  • cli.py: _handle_whoami_command() + dispatch branch in process_command()
  • tests/cli/test_cli_init.py: regression test — /whoami must not fall through to Unknown command

Validation

Before After
/whoami in hermes chat Unknown command: /whoami prints access summary (verified live in tmux sandbox)
targeted test n/a tests/cli/test_cli_init.py -k whoami → 1 passed

Duplicate cluster to close on merge: #26047 (salvaged, credit), #52170 (also fixed /indicator — already fixed on main), #61416, #77920.

Infographic

whoami fix infographic

@github-actions

github-actions Bot commented Aug 20, 2026 •

Copy link
Copy Markdown
Contributor

૮ >ﻌ< ა ci review

ran on 1de5414 — fix(cli): wire /whoami slash command in classic CLI

⚠️ Warnings

OSV vulnerability scan · View job

7 known vulnerabilities found in pinned dependencies.

How to fix:

Review the findings in the Security tab. Update the affected dependencies if a patched version is available.


debug info

CI timings

CI timings · View report · View job

Wall time 7m54s vs 9m28s (-16.5%). 10 job(s) slower, 10 faster, 4 unchanged.

  • Python tests / Run tests slice 1/12: -42.0s
  • Python tests / Run tests slice 5/12: -34.0s
  • Python tests / Run tests slice 12/12: +23.0s
  • Python tests / Run tests slice 9/12: -22.0s
  • Python tests / Run tests slice 8/12: -21.0s

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/cli CLI entry point, hermes_cli/, setup wizard P3 Low — cosmetic, nice to have type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants