Implement skill list and info CLI commands with shared helpers#3987
Merged
Implement skill list and info CLI commands with shared helpers#3987
Conversation
Add skill_helpers.go with shared utilities (newSkillClient, completeSkillNames, formatSkillError, validateSkillScope) and replace the empty skill list/info stubs with real implementations that call the Skills HTTP client. Also add "skill" to the informationalCommands map since skill commands communicate with the HTTP API, not the container runtime. Closes #3653 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3987 +/- ##
==========================================
- Coverage 68.56% 68.55% -0.01%
==========================================
Files 437 437
Lines 44662 44662
==========================================
- Hits 30621 30617 -4
- Misses 11657 11663 +6
+ Partials 2384 2382 -2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
7 tasks
reyortiz3
pushed a commit
that referenced
this pull request
Mar 4, 2026
Add skill_helpers.go with shared utilities (newSkillClient, completeSkillNames, formatSkillError, validateSkillScope) and replace the empty skill list/info stubs with real implementations that call the Skills HTTP client. Also add "skill" to the informationalCommands map since skill commands communicate with the HTTP API, not the container runtime. Closes #3653 Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
skill_helpers.gowith shared utilities (newSkillClient,completeSkillNames,formatSkillError,validateSkillScope) used across all skill commandsskill listandskill infostubs with real implementations that call the Skills HTTP client"skill"to theinformationalCommandsmap since skill commands communicate via HTTP API, not container runtimeskill listsupports--scope,--client,--format,--project-rootflags with text (tabwriter) and JSON outputskill infosupports--scope,--format,--project-rootflags with detailed key-value text and JSON output, plus shell completion for skill namesPart 1 of 3 for #3653 (CLI Skills Commands).
Test plan
task lintpasses with no new warningstask testpasses (existing unit tests)task license-checkpasses (SPDX headers correct)thv skill --helpshows all subcommandsthv skill list --helpshows correct flags (--scope, --client, --format, --project-root)thv skill info --helpshows correct flags (--scope, --format, --project-root)🤖 Generated with Claude Code