Skip to content

docs: document profile create --no-skills - #50241

Closed
asimons81 wants to merge 2 commits into
NousResearch:mainfrom
asimons81:docs/profile-create-no-skills
Closed

docs: document profile create --no-skills#50241
asimons81 wants to merge 2 commits into
NousResearch:mainfrom
asimons81:docs/profile-create-no-skills

Conversation

@asimons81

Copy link
Copy Markdown
Contributor

Summary

Adds the existing --no-skills flag to the hermes profile create command reference in the bundled hermes-agent skill.

Why

The flag already exists and is documented in the website docs, but it was missing from skills/autonomous-ai-agents/hermes-agent/SKILL.md, which agents and AI assistants use as a primary command reference.

This closes the remaining docs gap from #26446.

Fixes #26446.

What changed

  • Updated skills/autonomous-ai-agents/hermes-agent/SKILL.md
  • Added --no-skills to the hermes profile create NAME flag list

Validation

grep -rn "no-skills" skills/
grep -rn "no-skills" website/
git diff upstream/main..HEAD --stat
pytest tests/hermes_cli/test_profiles.py

Results:

  • grep -rn "no-skills" skills/ now finds the new SKILL.md entry
  • grep -rn "no-skills" website/ confirms the website docs already document the flag
  • Diff is limited to 1 file, 1 line
  • Profile tests: 136 passed, 2 pre-existing Windows failures reproduced identically on upstream/main

Risk

Zero behavioral risk. This is a one-line markdown documentation update only.

@alt-glitch alt-glitch added type/docs Documentation improvements P3 Low — cosmetic, nice to have duplicate This issue or pull request already exists labels Jun 21, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Duplicate of #26456 — both add --no-skills to the hermes profile create NAME flag line in skills/autonomous-ai-agents/hermes-agent/SKILL.md to close the #26446 docs gap. #26456 (open, 2026-05-15) makes the identical one-line edit plus broader doc/code coverage; this PR is the narrowest subset of that change. Other open competitors for the same gap: #36629 (SKILL.md + website mirror + zh-Hans i18n) and #30400 (user-guide profiles.md).

@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 correcting the bundled skill reference. The premise remains valid on current main: skills/autonomous-ai-agents/hermes-agent/SKILL.md:215 omits --no-skills, while the real CLI registers it at hermes_cli/subcommands/profile.py:53-57.

Problems

  • The public bundled-skill mirror has the same omission at website/docs/user-guide/skills/bundled/autonomous-ai-agents/autonomous-ai-agents-hermes-agent.md:219.

Suggested changes

  • Update that mirrored hermes profile create NAME line with --no-skills as well.

This is an automated hermes-sweeper review.

hermes profile list List all profiles
hermes profile create NAME Create (--clone, --clone-all, --clone-from)
hermes profile create NAME Create (--clone, --clone-all, --clone-from, --no-skills)
hermes profile use NAME Set sticky default

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.

Please make the same flag-list update in the published bundled-skill mirror at website/docs/user-guide/skills/bundled/autonomous-ai-agents/autonomous-ai-agents-hermes-agent.md:219; it currently has the identical omission.

The hermes-sweeper flagged that the public bundled-skill mirror at
website/docs/.../autonomous-ai-agents-hermes-agent.md:219 has the
same omission as the SKILL.md that PR NousResearch#50241 already fixed.

Refs: NousResearch#50241
@asimons81

Copy link
Copy Markdown
Contributor Author

Addressed the hermes-sweeper's suggestion: the website mirror at website/docs/user-guide/skills/bundled/autonomous-ai-agents/autonomous-ai-agents-hermes-agent.md:219 now also includes --no-skills.

This PR is now 2 files, 2 lines changed (both: +1 for --no-skills, -0). All 35 CI checks still pass.

Also noting the duplicate label references #26456, which has since been closed. The remaining open competitor (#36629) covers SKILL.md + website mirror + zh-Hans i18n -- if maintainers prefer the broader scope, happy to close this in favor of that PR.

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

Copy link
Copy Markdown
Contributor Author

Updated the website mirror doc per sweeper feedback (commit d5b7aaa). The duplicate label referencing #26456 appears stale -- that PR has been closed since June 24. All CI checks pass and this is a one-line docs fix. Could a maintainer re-evaluate the duplicate label?

@teknium1 teknium1 added the area/profiles Multi-profile isolation, HERMES_HOME scoping label Jul 19, 2026
@asimons81 asimons81 closed this Jul 30, 2026
@asimons81
asimons81 deleted the docs/profile-create-no-skills branch July 30, 2026 00:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/profiles Multi-profile isolation, HERMES_HOME scoping duplicate This issue or pull request already exists 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.

--no-skills flag is in RELEASE_NOTES and code, but missing from permanent docs

3 participants