Skip to content

fix(cli): add loading state for slow slash commands#714

Closed
halfprecision wants to merge 1 commit into
NousResearch:mainfrom
halfprecision:fix/async-command-loading-indicator
Closed

fix(cli): add loading state for slow slash commands#714
halfprecision wants to merge 1 commit into
NousResearch:mainfrom
halfprecision:fix/async-command-loading-indicator

Conversation

@halfprecision

Copy link
Copy Markdown
Contributor

What changed

  • show an immediate loading message for slower slash commands like /skills ... and /reload-mcp
  • add a small spinner in the prompt while the command is running
  • make the input temporarily read-only so it doesn't feel like the CLI froze

Why

Right now these commands can sit there for a bit with no feedback, which makes it look like nothing is happening. This keeps the fix pretty small, but it gives the user immediate confirmation that Hermes is still working.

Testing

  • .venv/bin/python -m pytest tests/test_cli_loading_indicator.py tests/test_cli_model_command.py tests/hermes_cli/test_commands.py -q

Fixes #636

teknium1 pushed a commit that referenced this pull request Mar 11, 2026
Shows an immediate status message and braille spinner for slow slash
commands (/skills search|browse|inspect|install, /reload-mcp). Makes
input read-only while the command runs so the CLI doesn't appear frozen.

Cherry-picked from PR #714 by vilkasdev, rebased onto current main
with conflict resolution and bug fix (get_hint_text duplicate return).

Fixes #636

Co-authored-by: vilkasdev <vilkasdev@users.noreply.github.com>
teknium1 pushed a commit that referenced this pull request Mar 11, 2026
Shows an immediate status message and braille spinner for slow slash
commands (/skills search|browse|inspect|install, /reload-mcp). Makes
input read-only while the command runs so the CLI doesn't appear frozen.

Cherry-picked from PR #714 by vilkasdev, rebased onto current main
with conflict resolution and bug fix (get_hint_text duplicate return).

Fixes #636

Co-authored-by: vilkasdev <vilkasdev@users.noreply.github.com>
@teknium1

Copy link
Copy Markdown
Contributor

Merged via PR #882 — your commit was cherry-picked and rebased onto current main with your authorship preserved. Thanks for the contribution! Fixed a small merge artifact in get_hint_text() on top.

Closes #636.

@teknium1 teknium1 closed this Mar 11, 2026
angelburgosrosado pushed a commit to angelburgosrosado/hermes-agent that referenced this pull request Apr 27, 2026
Shows an immediate status message and braille spinner for slow slash
commands (/skills search|browse|inspect|install, /reload-mcp). Makes
input read-only while the command runs so the CLI doesn't appear frozen.

Cherry-picked from PR NousResearch#714 by vilkasdev, rebased onto current main
with conflict resolution and bug fix (get_hint_text duplicate return).

Fixes NousResearch#636

Co-authored-by: vilkasdev <vilkasdev@users.noreply.github.com>
02356abc pushed a commit to 02356abc/hermes-agent that referenced this pull request May 14, 2026
Shows an immediate status message and braille spinner for slow slash
commands (/skills search|browse|inspect|install, /reload-mcp). Makes
input read-only while the command runs so the CLI doesn't appear frozen.

Cherry-picked from PR NousResearch#714 by vilkasdev, rebased onto current main
with conflict resolution and bug fix (get_hint_text duplicate return).

Fixes NousResearch#636

Co-authored-by: vilkasdev <vilkasdev@users.noreply.github.com>
olympus-terminal pushed a commit to olympus-terminal/hermes-agent that referenced this pull request May 16, 2026
Shows an immediate status message and braille spinner for slow slash
commands (/skills search|browse|inspect|install, /reload-mcp). Makes
input read-only while the command runs so the CLI doesn't appear frozen.

Cherry-picked from PR NousResearch#714 by vilkasdev, rebased onto current main
with conflict resolution and bug fix (get_hint_text duplicate return).

Fixes NousResearch#636

Co-authored-by: vilkasdev <vilkasdev@users.noreply.github.com>
Egavasyug pushed a commit to Egavasyug/hermes-agent that referenced this pull request Jun 10, 2026
Shows an immediate status message and braille spinner for slow slash
commands (/skills search|browse|inspect|install, /reload-mcp). Makes
input read-only while the command runs so the CLI doesn't appear frozen.

Cherry-picked from PR NousResearch#714 by vilkasdev, rebased onto current main
with conflict resolution and bug fix (get_hint_text duplicate return).

Fixes NousResearch#636

Co-authored-by: vilkasdev <vilkasdev@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[UX]: Async slash commands (/skills search, etc.) need a loading indicator

2 participants