-
Notifications
You must be signed in to change notification settings - Fork 0
docs: fix install snippets across skills pages #27
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
Changes from all commits
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 |
|---|---|---|
|
|
@@ -20,6 +20,8 @@ Each subdirectory is a single skill with a `SKILL.md` using standard YAML frontm | |
|
|
||
| ## Install into Claude Code | ||
|
|
||
| Run from the `mogplex-docs` repo root (the directory that contains this `skills/` folder): | ||
|
|
||
| ```bash | ||
| # Global install | ||
| mkdir -p ~/.claude/skills && cp -R skills/* ~/.claude/skills/ | ||
|
|
@@ -28,6 +30,8 @@ mkdir -p ~/.claude/skills && cp -R skills/* ~/.claude/skills/ | |
| mkdir -p .claude/skills && cp -R skills/* .claude/skills/ | ||
| ``` | ||
|
|
||
| If you pulled the skills tree with `npx degit webrenew/mogplex-docs/skills mogplex-skills` instead (so there is no outer `skills/` directory), run the copy as `cp -R mogplex-skills/* ~/.claude/skills/`. | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Suggestion: skills/README.md degit note only covers the global install path The new sentence added at the end of the install section reads:
It only shows the global target ( Suggestion — expand the note to cover both variants, matching the structure above it: If you pulled the skills tree with `npx degit webrenew/mogplex-docs/skills mogplex-skills` instead (so there is no outer `skills/` directory):
```bash
# Global
mkdir -p ~/.claude/skills && cp -R mogplex-skills/* ~/.claude/skills/
# Or per-project (run from your project root)
mkdir -p .claude/skills && cp -R mogplex-skills/* .claude/skills/ |
||
|
|
||
| ## Install into an Agent SDK app | ||
|
|
||
| ```ts | ||
|
|
||
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.
The per-project command tells users to run from their project root, but
SKILLSis set to a relative path (mogplex-docs/skillsormogplex-skills), socp -R "$SKILLS"/* .claude/skills/fails as soon as theycdto a different directory. This makes the documented per-project install path non-functional for the common case where the app repo is separate from the downloaded docs/skills folder (the same pattern appears incontent/docs/extend/skills.mdx).Useful? React with 👍 / 👎.