Vendor agent-skill packs (superpowers, matt-pocock, gstack, pstack) into .claude/skills and .agents/skills - #35
Conversation
- scripts/install-agent-skills.sh: reproducibly vendors the skill packs (slimmed, Markdown-only) into .claude/skills/ and .agents/skills/ - scripts/gen-skills-index.js: generates docs/agent-skills.md from SKILL.md frontmatter Co-authored-by: Prax Lannister <praxstack@users.noreply.github.com>
Slimmed (Markdown-only) installs of four MIT-licensed agent-skill packs into both .claude/skills/ (Claude Code, Cursor) and .agents/skills/ (Codex, Prime Agent). 162 skills per location. Each pack keeps its upstream LICENSE. - superpowers v6.3.0 (obra/superpowers) - mattpocock v1.2.3 (mattpocock/skills) - gstack v1.69.0.0 (garrytan/gstack) - pstack v0.9.14 (michael-denyer/pstack-claude; poteto's pstack port) Co-authored-by: Prax Lannister <praxstack@users.noreply.github.com>
- docs/agent-skills.md: browsable index of all 162 skills + project-specific recommendations mapping this Node/Express app's needs to skills - .claude/skills/README.md, .agents/skills/README.md: pack overview - AGENTS.md: '## Agent skills' discovery block for compatible agents Co-authored-by: Prax Lannister <praxstack@users.noreply.github.com>
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Co-authored-by: Prax Lannister <praxstack@users.noreply.github.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: aa8d43579a
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| (cd "$dir" && find . -name '*.md' -not -path '*/.git/*' | while read -r f; do | ||
| mkdir -p "$target/$(dirname "$f")"; cp "$f" "$target/$f"; done) |
There was a problem hiding this comment.
Preserve runtime assets required by installed skills
When an agent invokes several of these newly advertised skills, copying only *.md leaves required executables and templates absent: mattpocock/wizard/SKILL.md requires template.sh, git-guardrails-claude-code/SKILL.md requires scripts/block-dangerous-git.sh, and gstack/careful/SKILL.md invokes a missing bin/check-careful.sh while claiming its destructive-command protection is active. Preserve each skill's required runtime assets while excluding unrelated build/test output, or omit unusable skills from discovery.
AGENTS.md reference: AGENTS.md:L28-L29
Useful? React with 👍 / 👎.
| local pack="$1" repo="$2" subdir="$3" | ||
| local src="$TMP/$(echo "$repo" | tr '/' '_')" | ||
| echo ">> cloning $repo" | ||
| git clone --depth 1 "https://github.com/$repo.git" "$src" >/dev/null 2>&1 |
There was a problem hiding this comment.
Pin refresh clones to the reported pack versions
After any upstream repository advances, the documented refresh installs its latest default-branch contents here, but gen-skills-index.js:17-20 continues labeling those contents as the hard-coded versions 6.3.0, 1.2.3, 1.69.0.0, and 0.9.14. This makes subsequent bundles non-reproducible and causes the generated index to misidentify what was installed; clone pinned tags or commit SHAs and derive the displayed versions from the same lock data.
AGENTS.md reference: AGENTS.md:L25-L26
Useful? React with 👍 / 👎.
| @@ -0,0 +1,44 @@ | |||
| --- | |||
| name: tdd | |||
There was a problem hiding this comment.
Give overlapping skills unique discovery names
When a user asks for TDD, the discovery catalog contains this skill and mattpocock/tdd/SKILL.md under the identical name: tdd, despite their overlapping but different invocation rules; compatible runtimes that select or invoke skills by frontmatter name cannot unambiguously address them and may shadow or choose the unintended workflow. The same collision exists for teach and retro, so namespace these names by pack or deduplicate the alternatives in both vendored trees.
AGENTS.md reference: AGENTS.md:L14-L16
Useful? React with 👍 / 👎.
| - `~/.claude/skills/gstack/plan-ceo-review/SKILL.md` | ||
| - `~/.claude/skills/gstack/plan-design-review/SKILL.md` (only if UI scope detected) | ||
| - `~/.claude/skills/gstack/plan-eng-review/SKILL.md` | ||
| - `~/.claude/skills/gstack/plan-devex-review/SKILL.md` (only if DX scope detected) |
There was a problem hiding this comment.
Resolve gstack composition paths from the project install
On a fresh checkout without a separate global gstack installation, /autoplan reaches this mandatory composition step and tries to read every review skill from ~/.claude/skills/gstack, although this commit installs them only under the repository's .claude/skills/gstack and .agents/skills/gstack directories. Those reads therefore miss the files that are present in the checkout, preventing the advertised full review pipeline from loading; use paths relative to the active skill or add project-local fallbacks throughout the vendored gstack instructions.
AGENTS.md reference: AGENTS.md:L14-L16
Useful? React with 👍 / 👎.
📋 Description
Installs four agent-skill packs into the repo so they are available to every agent/harness, plus tooling to refresh them and a browsable index. Skills are vendored into both
.claude/skills/(Claude Code, Cursor) and.agents/skills/(Codex, Prime Agent), as requested.Installs are slimmed (Markdown only): only
SKILL.mdfiles and their Markdown references are vendored; upstream build scripts, browser extensions, tests, and binaries are excluded to keep the repo lean. This trims gstack from ~32 MB to a few MB. Each pack keeps its upstreamLICENSE(all MIT).Packs (162 skills per location)
superpowers/obra/superpowersmattpocock/mattpocock/skillsgstack/garrytan/gstackpstack/michael-denyer/pstack-claude(Claude Code/Codex port of poteto's pstack)Also added
docs/agent-skills.md— browsable index of all 162 skills + a "Recommended for this project" section mapping this Node/Express app's needs (security review, code review, testing, CI, QA) to specific vendored skills, and notes on complementary Cursor tooling (CodeRabbit, Security Review, Bugbot).scripts/install-agent-skills.sh— reproducibly re-vendors the packs from upstream.scripts/gen-skills-index.js— regenerates the index fromSKILL.mdfrontmatter..claude/skills/README.md,.agents/skills/README.md— pack overviews.AGENTS.md— adds an## Agent skillsdiscovery block (the convention these packs expect).🔄 Type of Change
📝 Additional Notes
scripts/install-agent-skills.shrefreshes them from upstream on demand.BhaskarManam/pstack(the product-management pack), I can swap it.