-
Notifications
You must be signed in to change notification settings - Fork 67
[docs] Update documentation for skill target-aware deployment (v0.8.7) #499
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
ff29a3b
c04c75c
88a9974
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -251,11 +251,11 @@ my-package/ | |
| └── SKILL.md # Sub-skill B | ||
| ``` | ||
|
|
||
| On install, APM promotes each sub-skill to a top-level `.github/skills/` entry alongside the parent — see [Sub-skill Promotion](#sub-skill-promotion) below. | ||
| On install, APM promotes each sub-skill to a top-level `{target}/skills/` entry alongside the parent — see [Sub-skill Promotion](#sub-skill-promotion) below. | ||
|
|
||
| ### Sub-skill Promotion | ||
|
|
||
| When a package contains sub-skills in `.apm/skills/*/` subdirectories, APM promotes each to a top-level entry under `.github/skills/`. This ensures Copilot can discover them independently, since it only scans direct children of `.github/skills/`. | ||
| When a package contains sub-skills in `.apm/skills/*/` subdirectories, APM promotes each to a top-level entry under `{target}/skills/` (e.g., `.github/skills/`, `.cursor/skills/`, `.opencode/skills/` depending on the active target). This ensures each supported IDE can discover them independently. | ||
|
Comment on lines
+254
to
+258
|
||
|
|
||
| ``` | ||
| # Installed package with sub-skills: | ||
|
|
@@ -266,12 +266,19 @@ apm_modules/org/repo/my-package/ | |
| └── azure-naming/ | ||
| └── SKILL.md | ||
|
|
||
| # Result after install: | ||
| # Result after install (default/copilot target): | ||
| .github/skills/ | ||
| ├── my-package/ # Parent skill | ||
| │ └── SKILL.md | ||
| └── azure-naming/ # Promoted sub-skill | ||
| └── SKILL.md | ||
|
|
||
| # Result after install (--target cursor): | ||
| .cursor/skills/ | ||
| ├── my-package/ | ||
| │ └── SKILL.md | ||
| └── azure-naming/ | ||
| └── SKILL.md | ||
| ``` | ||
|
|
||
| ## Package Detection | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This sentence/examples only mention copilot and cursor, but this section is about Claude Skills and APM also supports
.claude/skills/and.opencode/skills/. Consider expanding the examples to include all supported skill targets (or rephrase to avoid implying only those two targets exist).