Skip to content

docs(readme): fix stale RL submodule section, /skills messaging row, test runner - #14758

Merged
teknium1 merged 1 commit into
mainfrom
hermes/hermes-a2ab431b
Apr 23, 2026
Merged

docs(readme): fix stale RL submodule section, /skills messaging row, test runner#14758
teknium1 merged 1 commit into
mainfrom
hermes/hermes-a2ab431b

Conversation

@teknium1

Copy link
Copy Markdown
Contributor

Summary

Three small README accuracy fixes found during a coverage audit.

Changes

  • RL Training section — the submodule instructions are stale. No .gitmodules exists, tinker-atropos/ is an empty directory, and atroposlib + tinker are already regular pip deps in pyproject.toml pulled in by .[all,dev]. Replaced the broken block with a one-line note.
  • CLI vs Messaging table/skills is registered as cli_only=True in COMMAND_REGISTRY, so it doesn't work on messaging platforms. Dropped it from the messaging column; /<skill-name> still works there.
  • Contributor test command — switched from bare python -m pytest tests/ -q to scripts/run_tests.sh, which AGENTS.md designates as the canonical runner (enforces hermetic env parity with CI: unset credential vars, TZ=UTC, LANG=C.UTF-8, -n 4 xdist workers).

Validation

  • All other README claims verified: 16/16 docs URLs return 200, every hermes subcommand referenced exists, every slash command in the table is in COMMAND_REGISTRY with correct CLI/gateway scoping, claw migrate flags match --help output, terminal-backends count matches tools/environments/.

…test runner

- Drop broken tinker-atropos submodule instructions: no .gitmodules exists,
  tinker-atropos/ is empty, and atroposlib + tinker are regular pip deps in
  pyproject.toml pulled in by .[all,dev]. Replace with a one-line note.
- CLI vs Messaging table: /skills is cli_only=True in COMMAND_REGISTRY, so
  remove it from the messaging column. /<skill-name> still works there.
- Point contributors at scripts/run_tests.sh (the canonical runner enforcing
  CI-parity env) instead of bare pytest.
@teknium1
teknium1 merged commit 692ae6d into main Apr 23, 2026
8 checks passed
@teknium1
teknium1 deleted the hermes/hermes-a2ab431b branch April 23, 2026 22:12
@alt-glitch alt-glitch added type/docs Documentation improvements P3 Low — cosmetic, nice to have labels Apr 23, 2026
nekorytaylor666 pushed a commit to nekorytaylor666/hermes-agent that referenced this pull request Apr 24, 2026
…test runner (NousResearch#14758)

- Drop broken tinker-atropos submodule instructions: no .gitmodules exists,
  tinker-atropos/ is empty, and atroposlib + tinker are regular pip deps in
  pyproject.toml pulled in by .[all,dev]. Replace with a one-line note.
- CLI vs Messaging table: /skills is cli_only=True in COMMAND_REGISTRY, so
  remove it from the messaging column. /<skill-name> still works there.
- Point contributors at scripts/run_tests.sh (the canonical runner enforcing
  CI-parity env) instead of bare pytest.
aj-nt pushed a commit to aj-nt/hermes-agent that referenced this pull request May 1, 2026
…test runner (NousResearch#14758)

- Drop broken tinker-atropos submodule instructions: no .gitmodules exists,
  tinker-atropos/ is empty, and atroposlib + tinker are regular pip deps in
  pyproject.toml pulled in by .[all,dev]. Replace with a one-line note.
- CLI vs Messaging table: /skills is cli_only=True in COMMAND_REGISTRY, so
  remove it from the messaging column. /<skill-name> still works there.
- Point contributors at scripts/run_tests.sh (the canonical runner enforcing
  CI-parity env) instead of bare pytest.
donald131 pushed a commit to donald131/hermes-agent that referenced this pull request May 2, 2026
…test runner (NousResearch#14758)

- Drop broken tinker-atropos submodule instructions: no .gitmodules exists,
  tinker-atropos/ is empty, and atroposlib + tinker are regular pip deps in
  pyproject.toml pulled in by .[all,dev]. Replace with a one-line note.
- CLI vs Messaging table: /skills is cli_only=True in COMMAND_REGISTRY, so
  remove it from the messaging column. /<skill-name> still works there.
- Point contributors at scripts/run_tests.sh (the canonical runner enforcing
  CI-parity env) instead of bare pytest.
02356abc pushed a commit to 02356abc/hermes-agent that referenced this pull request May 14, 2026
…test runner (NousResearch#14758)

- Drop broken tinker-atropos submodule instructions: no .gitmodules exists,
  tinker-atropos/ is empty, and atroposlib + tinker are regular pip deps in
  pyproject.toml pulled in by .[all,dev]. Replace with a one-line note.
- CLI vs Messaging table: /skills is cli_only=True in COMMAND_REGISTRY, so
  remove it from the messaging column. /<skill-name> still works there.
- Point contributors at scripts/run_tests.sh (the canonical runner enforcing
  CI-parity env) instead of bare pytest.
gweeteve pushed a commit to gweeteve/hermes-agent that referenced this pull request Jun 2, 2026
…test runner (NousResearch#14758)

- Drop broken tinker-atropos submodule instructions: no .gitmodules exists,
  tinker-atropos/ is empty, and atroposlib + tinker are regular pip deps in
  pyproject.toml pulled in by .[all,dev]. Replace with a one-line note.
- CLI vs Messaging table: /skills is cli_only=True in COMMAND_REGISTRY, so
  remove it from the messaging column. /<skill-name> still works there.
- Point contributors at scripts/run_tests.sh (the canonical runner enforcing
  CI-parity env) instead of bare pytest.
waefrebeorn pushed a commit to waefrebeorn/slermes that referenced this pull request Jul 2, 2026
…test runner (NousResearch#14758)

- Drop broken tinker-atropos submodule instructions: no .gitmodules exists,
  tinker-atropos/ is empty, and atroposlib + tinker are regular pip deps in
  pyproject.toml pulled in by .[all,dev]. Replace with a one-line note.
- CLI vs Messaging table: /skills is cli_only=True in COMMAND_REGISTRY, so
  remove it from the messaging column. /<skill-name> still works there.
- Point contributors at scripts/run_tests.sh (the canonical runner enforcing
  CI-parity env) instead of bare pytest.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P3 Low — cosmetic, nice to have type/docs Documentation improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants