Skip to content

Unhide skill CLI commands and update toolhive-cli-user skill#4443

Merged
JAORMX merged 2 commits intomainfrom
unhide-skill-commands-and-update-docs
Mar 30, 2026
Merged

Unhide skill CLI commands and update toolhive-cli-user skill#4443
JAORMX merged 2 commits intomainfrom
unhide-skill-commands-and-update-docs

Conversation

@JAORMX
Copy link
Copy Markdown
Collaborator

@JAORMX JAORMX commented Mar 30, 2026

Summary

The thv skill subcommands have been hidden behind Hidden: true since they were first implemented, pending feature readiness. They are now ready for release -- this PR removes the hidden flag so thv skill appears in CLI help, and updates the toolhive-cli-user agent skill with full documentation for all seven subcommands.

Type of change

  • New feature (non-breaking change which adds functionality)

Changes

File Change
cmd/thv/app/skill.go Remove Hidden: true and TODO comment from the skill cobra command
skills/toolhive-cli-user/SKILL.md Add "Skills Management" section, update description/version (0.2.0 -> 0.3.0), add guardrail and error table entries
skills/toolhive-cli-user/references/COMMANDS.md Add "Skill Commands" section with full reference for install, uninstall, list, info, build, push, validate
skills/toolhive-cli-user/references/EXAMPLES.md Add "Skills Management Examples" section with lifecycle, authoring, and workflow examples

Test plan

  • Verified go vet ./cmd/thv/app/ passes
  • Confirmed SKILL.md stays under 500-line budget (259 lines)
  • Run thv skill --help to confirm subcommands are visible
  • Run thv skill list --help to confirm flag documentation

Generated with Claude Code

@github-actions github-actions bot added the size/S Small PR: 100-299 lines changed label Mar 30, 2026
@JAORMX JAORMX changed the title Unhide skill CLI commands and add documentation Unhide skill CLI commands and update toolhive-cli-user skill Mar 30, 2026
@github-actions github-actions bot added size/S Small PR: 100-299 lines changed and removed size/S Small PR: 100-299 lines changed labels Mar 30, 2026
yrobla
yrobla previously approved these changes Mar 30, 2026
@JAORMX JAORMX force-pushed the unhide-skill-commands-and-update-docs branch from 2212aae to b2f1e45 Compare March 30, 2026 10:40
The skill subcommands (install, uninstall, list, info, build, push,
validate) are ready for release. This removes the Hidden flag from the
parent `thv skill` command and adds comprehensive documentation to the
toolhive-cli-user agent skill covering all seven subcommands, their
flags, and usage examples.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Juan Antonio Osorio <ozz@stacklok.com>
@JAORMX JAORMX force-pushed the unhide-skill-commands-and-update-docs branch from b2f1e45 to 98c0f13 Compare March 30, 2026 10:41
@github-actions github-actions bot added size/M Medium PR: 300-599 lines changed and removed size/S Small PR: 100-299 lines changed labels Mar 30, 2026
@github-actions github-actions bot added size/M Medium PR: 300-599 lines changed and removed size/M Medium PR: 300-599 lines changed labels Mar 30, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 30, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.49%. Comparing base (36af102) to head (2697efe).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4443      +/-   ##
==========================================
- Coverage   69.54%   69.49%   -0.06%     
==========================================
  Files         486      486              
  Lines       50026    50026              
==========================================
- Hits        34792    34765      -27     
- Misses      12550    12577      +27     
  Partials     2684     2684              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@JAORMX JAORMX merged commit 9824210 into main Mar 30, 2026
62 of 64 checks passed
@JAORMX JAORMX deleted the unhide-skill-commands-and-update-docs branch March 30, 2026 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/M Medium PR: 300-599 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants