refactor(skills): install all mattpocock/skills and make every install-all entry explicit - #160
Conversation
Change the mattpocock/skills entry from the curated 16-skill subset to install all skills from the repo. The upstream repo has grown to 38 skills (engineering, productivity, misc, personal, deprecated, and in-progress categories), and several previously-listed skills were renamed or removed upstream. Dropping the filter keeps the entry in sync with the repo and matches the "omit skills list to install all" convention. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01VxgujtX7yvLr4Ht35vz4Js
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
Disabled knowledge base sources:
📝 WalkthroughSummary by CodeRabbit
WalkthroughThis PR updates SKILLS.txt by converting numerous previously repo-only "keep all" entries into explicit comma-separated skill-slug lists across the file, with corresponding "(N total)" comment counts adjusted for each affected repo entry. ChangesSKILLS.txt curated skill list expansion
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Poem
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
Expand every "install all" (bare repo) entry into an explicit, comma-separated list of skill names, and refresh each "(N total)" count to match the repo's current skill set. Skill names were enumerated with the same `skills add <repo> --list` command the Makefile uses, so the explicit lists resolve to the same set the implicit form installed. Several upstream repos have changed since the counts were first written (e.g. google/skills 30 -> 58, addyosmani/agent-skills 22 -> 24, coinbase/agentic-wallet-skills 8 -> 1); the explicit lists now reflect current reality. Four entries can't be enumerated from CI and stay implicit, each with an inline note explaining why: - https://index.how/to/articulate (well-known endpoint exposes no skills) - shunkakinoki/private-skills (private repo, requires auth) - vercel-labs/next-browser, vercel-labs/next-skills (no SKILL.md upstream) Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01VxgujtX7yvLr4Ht35vz4Js
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
Summary
Two related changes to
SKILLS.txt:mattpocock/skills— the entry previously pinned a curated 16-skill subset that had drifted out of date (several listed skills no longer exist upstream). It now installs all 38 skills the repo currently exposes.(N total)comment is refreshed to match the repo's current skill set.How the lists were generated
Skill names were enumerated with the exact command the repo's own
Makefileuses:Because each explicit list is the complete set the CLI reports, these entries resolve to the same skills the implicit form installed — this pins the current set rather than changing what gets installed. Counts were cross-checked against the CLI's own "Found N skills" line for every repo.
Notable count changes (upstream drift since the entries were first written)
google/skillsmattpocock/skillsaddyosmani/agent-skillscloudflare/skillselvisun/newsjackemilkowalski/skillcoinbase/agentic-wallet-skillsbetter-auth/skillsdonnfelker/donnfelker-plugin-marketplace(Full detail in the diff.)
Entries left implicit (with inline notes)
Four sources can't be enumerated from this environment and remain bare, each annotated with the reason:
https://index.how/to/articulate— the.well-knownendpoint currently exposes no skillsshunkakinoki/private-skills— private repo, clone requires auth not available herevercel-labs/next-browser,vercel-labs/next-skills— upstream currently has noSKILL.mdNotes
antfu/skills,getsentry/skills,trailofbits/skills) are intentionally left unchanged — they were already explicit.make sync. This PR only updates the declarative manifest.🤖 Generated with Claude Code
https://claude.ai/code/session_01VxgujtX7yvLr4Ht35vz4Js