feat(skills): port Matt Pocock's skills as plugin skills and drop the dependency - #41
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Ports thirteen of Matt Pocock's skills into the plugin as first-class skills, rewritten in plain English, and removes the
mattpocock-skillsdependency.Skills added
grillmerges grilling, grill-me, and grill-with-docs. It runsdomain-modelingalongside when the repo has aCONTEXT.md.buildmerges implement, tdd, and implement-spec. It takes the conversation as the ask by default and decides which public functions get tests without a confirm step. It reviews the diff against the ask throughopus-medium, with the repo's own review instructions winning, and commits only when told. A spec with tickets fans out one worktree per ticket.specmerges to-spec and to-tickets. It writes a trimmed spec and offers tickets only when the work will not fit one session.routineis loop-me under a name that does not collide with the built-in/loop.wayfinderkeeps the map, decisions, unresolved questions, and out-of-scope sections as files, at a third of the length, with the claim and assignee rules cut.handoffmerges handoff and claude-handoff and writes underdocs/handoff/or into Bear, never the temp directory.research,wait-what,teach,writing-for-agents,domain-modeling,diagnosing-bugs, andresolving-merge-conflictsare rewritten from their upstream bodies.teachpublishes lessons as Claude artifacts.writing-for-agentsdrops the leading-words section and states the positive-form rule instead.Where documents go
reference/output-locations.mdholds the one rule. A repo can carry.claude/gborges-standard.jsonwithdocsandtrackerkeys, written by the new/setup-repocommand andscripts/setup-repo.sh. Without it, everything goes underdocs/and nothing touches GitHub unless asked.Codex
The ten user-only skills,
pair-debateandmodel-routing-reviewincluded, getagents/openai.yamlwithallow_implicit_invocation: false, since Codex ignores thedisable-model-invocationfrontmatter.Also in this PR
RULES.md, and incheck.py..claude/skills/upstream-check/is a repo-only skill that lists what changed upstream since commit3cca18b.A machine with the mattpocock plugin still installed at user scope will list two skills for each shared name until it is uninstalled.