Skip to content

docs(reference): sync slash-command tables with the command registry - #69639

Open
A-KH17 wants to merge 1 commit into
NousResearch:mainfrom
A-KH17:docs/slash-commands-registry-sync
Open

docs(reference): sync slash-command tables with the command registry#69639
A-KH17 wants to merge 1 commit into
NousResearch:mainfrom
A-KH17:docs/slash-commands-registry-sync

Conversation

@A-KH17

@A-KH17 A-KH17 commented Jul 22, 2026

Copy link
Copy Markdown

What does this PR do?

Brings the zh-Hans slash-commands reference back in sync with the command registry in hermes_cli/commands.py:

  • Removes /credits and /billing — neither command is registered anywhere at HEAD (CLI registry, gateway, or TUI). Billing was unified under /topup in v0.19.0, but the zh-Hans reference still tells users to type the retired commands.
  • Adds /topup (CLI + messaging) and /subscription (alias /upgrade, CLI-only) — both previously undocumented in zh-Hans.
  • Adds /journey (aliases /learning, /memory-graph) — available in the classic CLI, as a TUI overlay, and in the desktop app (Star Map panel); not available on messaging platforms.
  • Updates the CLI-only / both-surfaces availability lists accordingly.

The English half of the original sync landed upstream in 43874d1a; this PR now carries only the still-missing zh-Hans corrections.

Related Issue

N/A — small, self-verifying docs correction; no issue filed. Happy to open one if preferred.

Type of Change

  • 📝 Documentation update

Changes Made

  • website/i18n/zh-Hans/docusaurus-plugin-content-docs/current/reference/slash-commands.md: Info table (−/credits, −/billing, +/topup, +/subscription), Session table (+/journey), Messaging table (−/credits, +/topup), CLI-only list (−/billing, +/journey, +/subscription), both-surfaces list (−/credits, +/topup).
  • contributors/emails/: email→login mapping for the commit author.

How to Test

  1. grep -n 'CommandDef("topup"\|CommandDef("subscription"\|CommandDef("journey"' hermes_cli/commands.py — all three registered.
  2. grep -nE 'credits|billing' website/i18n/zh-Hans/docusaurus-plugin-content-docs/current/reference/slash-commands.md — only the /topup row's historical note remains.
  3. /journey TUI registration: ui-tui/src/app/slash/commands/ops.ts:329-334.

Checklist

  • I've read the Contributing Guide
  • My commit messages follow Conventional Commits
  • I searched for existing PRs — no duplicate found
  • My PR contains only changes related to this fix
  • pytest — N/A: docs-only change, no code paths touched
  • I've considered cross-platform impact — N/A (documentation only)

@alt-glitch alt-glitch added type/docs Documentation improvements P3 Low — cosmetic, nice to have comp/cli CLI entry point, hermes_cli/, setup wizard comp/gateway Gateway runner, session dispatch, delivery area/billing Account usage, credit usage, billing (cross-cutting) labels Jul 22, 2026

@teknium1 teknium1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for identifying the stale command reference. The zh-Hans corrections remain needed: current main still lists /credits and /billing at website/i18n/zh-Hans/docusaurus-plugin-content-docs/current/reference/slash-commands.md:109-110 and /credits at :218.

Problems

  • The English changes have already landed in 43874d1a96134729ff65d8a0da84a38ef0fc723b; current main has the replacement entries at website/docs/reference/slash-commands.md:126-127 and :240. The PR is now conflicting, and reapplying its English /journey row would duplicate the existing rows at :66 and :68.
  • The new zh-Hans /journey wording says CLI-only, but the TUI registers the command and its aliases at ui-tui/src/app/slash/commands/ops.ts:329-334.

Suggested changes

  • Salvage the zh-Hans command-table and availability-list corrections only, and align the /journey wording with its TUI support.

Automated hermes-sweeper review.

@@ -58,6 +58,7 @@ Hermes 有两个斜杠命令入口,均由 `hermes_cli/commands.py` 中的中
| `/agents`(别名:`/tasks`) | 显示当前会话中的活动 agent 和运行中的任务。 |
| `/background <prompt>`(别名:`/bg`、`/btw`) | 在独立的后台会话中运行 prompt。agent 独立处理你的 prompt——当前会话保持空闲可继续其他工作。任务完成后结果以面板形式显示。见 [CLI 后台会话](/user-guide/cli#background-sessions)。 |
| `/branch [name]`(别名:`/fork`) | 分支当前会话(探索不同路径) |
| `/journey [list\|delete <id>\|edit <id>]`(别名:`/learning`、`/memory-graph`) | 打开学习旅程时间线——查看 Hermes 随时间学到的 skill 和记忆,并可编辑或删除单条条目。仅限 CLI。 |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

cli_only excludes the messaging gateway, but /journey is also registered in the TUI at ui-tui/src/app/slash/commands/ops.ts:329-334. Please describe its local-surface availability consistently rather than calling it CLI-only.

@teknium1 teknium1 added the sweeper:blast-contained Sweeper blast radius: contained — one narrow path / opt-in / few users label Jul 30, 2026
…egistry

The zh-Hans reference still documented two commands that are no longer
registered anywhere (/credits, /billing — unified under /topup in v0.19.0)
and omitted three registered ones (/topup, /subscription, /journey).

The English half of the original sync landed upstream in 43874d1; this
change now carries only the zh-Hans corrections. /journey availability
wording reflects its TUI overlay + desktop Star Map support rather than
calling it CLI-only.
@A-KH17
A-KH17 force-pushed the docs/slash-commands-registry-sync branch from cea5c24 to cf81ea8 Compare August 2, 2026 23:59
@A-KH17

A-KH17 commented Aug 3, 2026

Copy link
Copy Markdown
Author

Thanks for the review — both points addressed in the reworked branch (now a single commit on current main):

  1. English changes dropped — confirmed they already landed in 43874d1a; the PR now carries only the zh-Hans corrections, which are still needed on main (/credits at :109-110 and :218, /billing in the CLI-only list). No conflict, no /journey row duplication.
  2. /journey wording fixed — the zh-Hans row now mirrors the current English wording: works in the classic CLI, as a TUI overlay, and in the desktop app (Star Map panel); not available on messaging platforms. Registered in the TUI at ui-tui/src/app/slash/commands/ops.ts:329-334, as you noted.

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

Labels

area/billing Account usage, credit usage, billing (cross-cutting) comp/cli CLI entry point, hermes_cli/, setup wizard 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 type/docs Documentation improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants