🐛 Fix skill directory structure for Claude Code compatibility#9
🐛 Fix skill directory structure for Claude Code compatibility#9clubanderson merged 1 commit intomainfrom
Conversation
Migrate from legacy `commands/<name>.md` structure to the current `skills/<name>/SKILL.md` format required by Claude Code. Also fix `allowed-tools` patterns to use correct plugin names (kubestellar-ops and kubestellar-deploy) instead of the old kubectl-claude reference, and add proper YAML frontmatter to kubestellar-deploy skills. Fixes: #8 Co-Authored-By: Claude Opus 4.6 <[email protected]> Signed-off-by: Andrew Anderson <[email protected]>
kubestellar-prow
Bot
added
the
dco-signoff: yes
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Welcome to KubeStellar! 🚀 Thank you for submitting this Pull Request. Before your PR can be merged, please ensure: ✅ DCO Sign-off - All commits must be signed off with ✅ PR Title - Must start with an emoji: ✨ (feature), 🐛 (bug fix), 📖 (docs), 🌱 (infra/tests), Getting Started with KubeStellar: Contributor Resources:
🌟 Help KubeStellar Grow - We Need Adopters! Our roadmap is driven entirely by adopter feedback. Whether you're using KubeStellar yourself or know someone who could benefit from multi-cluster Kubernetes: 📋 Take our Multi-Cluster Survey - Share your use cases and help shape our direction! A maintainer will review your PR soon. Feel free to ask questions in the comments or on Slack! |
kubestellar-prow
Bot
added
the
size/L
|
🎉 Thank you for your contribution! Your PR has been successfully merged. 🌟 Help KubeStellar Grow - We Need Adopters! Our roadmap is driven entirely by adopter feedback - nothing else. Whether you're using KubeStellar yourself or know organizations that could benefit from multi-cluster Kubernetes, we need your help: 📋 Take our Multi-Cluster Survey - Share your use cases and help shape our direction! 🗣️ Spread the word - Tell colleagues, write blog posts, present at meetups 💬 Share feedback on Slack #kubestellar-dev Every adopter story helps us prioritize what matters most. Thank you for being part of the KubeStellar community! |
There was a problem hiding this comment.
Pull request overview
Migrates this marketplace repo to Claude Code’s newer skill layout so the plugins’ slash-invoked skills are discoverable and the MCP tool permission patterns match the actual plugin/server names.
Changes:
- Standardized skills to
plugins/<plugin>/skills/<skill>/SKILL.mdwith YAML frontmatter (name,description,allowed-tools) - Updated
allowed-toolsfrom legacykubectl-claudepatterns tokubestellar-ops/kubestellar-deploy - Removed legacy
commands/*.mdforkubestellar-deployand updated the rootREADME.mdaccordingly
Reviewed changes
Copilot reviewed 18 out of 18 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| plugins/kubestellar-ops/skills/k8s-upgrade/SKILL.md | Adds name frontmatter and updates allowed-tools/tool wording to kubestellar-ops. |
| plugins/kubestellar-ops/skills/k8s-upgrade-check/SKILL.md | Adds name frontmatter and updates allowed-tools/tool wording to kubestellar-ops. |
| plugins/kubestellar-ops/skills/k8s-security/SKILL.md | Adds name frontmatter and updates allowed-tools/tool wording to kubestellar-ops. |
| plugins/kubestellar-ops/skills/k8s-rbac/SKILL.md | Adds name frontmatter and updates allowed-tools/tool wording to kubestellar-ops. |
| plugins/kubestellar-ops/skills/k8s-ownership/SKILL.md | Adds name frontmatter and updates allowed-tools to kubestellar-ops. |
| plugins/kubestellar-ops/skills/k8s-issues/SKILL.md | Adds name frontmatter and updates allowed-tools/tool wording to kubestellar-ops. |
| plugins/kubestellar-ops/skills/k8s-health/SKILL.md | Adds name frontmatter and updates allowed-tools/tool wording to kubestellar-ops. |
| plugins/kubestellar-ops/skills/k8s-audit-kubeconfig/SKILL.md | Adds name frontmatter and updates allowed-tools/tool wording to kubestellar-ops. |
| plugins/kubestellar-ops/skills/k8s-analyze/SKILL.md | Adds name frontmatter and updates allowed-tools/tool wording to kubestellar-ops. |
| plugins/kubestellar-deploy/skills/gitops-sync/SKILL.md | Converts legacy markdown command content into SKILL format with frontmatter and tool constraints. |
| plugins/kubestellar-deploy/skills/gitops-drift/SKILL.md | Adds new SKILL file (frontmatter + task flow) for drift detection. |
| plugins/kubestellar-deploy/skills/deploy/SKILL.md | Converts legacy markdown command content into SKILL format with frontmatter and tool constraints. |
| plugins/kubestellar-deploy/skills/app-status/SKILL.md | Adds new SKILL file (frontmatter + task flow) for app status. |
| plugins/kubestellar-deploy/skills/app-logs/SKILL.md | Adds new SKILL file (frontmatter + task flow) for app logs. |
| plugins/kubestellar-deploy/commands/gitops-drift.md | Removes legacy commands/ format file (replaced by skills/ structure). |
| plugins/kubestellar-deploy/commands/app-status.md | Removes legacy commands/ format file (replaced by skills/ structure). |
| plugins/kubestellar-deploy/commands/app-logs.md | Removes legacy commands/ format file (replaced by skills/ structure). |
| README.md | Updates installation/docs to new plugin names, skill structure, and allowed-tools patterns. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary
commands/<name>.mdtoskills/<name>/SKILL.mdformat required by Claude Codeallowed-toolspatterns from oldmcp__plugin_kubectl-claude_kubectl-claude__*to correctmcp__plugin_kubestellar-ops_kubestellar-ops__*andmcp__plugin_kubestellar-deploy_kubestellar-deploy__*Fixes #8
Test plan
/menu when plugin is installed/k8s-healthand other ops skills invoke correctly with MCP tools/app-statusand other deploy skills invoke correctly with MCP toolsallowed-toolspatterns match actual MCP tool names🤖 Generated with Claude Code