refactor(claude): Convert commands to atomic skills #783
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
…taMask actions Only allow updates from 'actions/*' (official GitHub Actions) and 'MetaMask/*' (MetaMask organization actions). Other actions must be pinned to commit hashes and reviewed before updating. Co-Authored-By: Claude <noreply@anthropic.com>
Move commit, commit-push, and commit-push-pr from commands to skills for better reusability. Update settings.json to include skill permissions. Co-Authored-By: Claude <noreply@anthropic.com>
…ills - Remove commit-push and commit-push-pr composite skills - Add atomic push skill for pushing to remote - Convert pr command to pr skill with unsandboxed flag - Update pr skill to use narrative description format - Update settings.json with new skill permissions Co-Authored-By: Claude <noreply@anthropic.com>
- Rename check to lint-build-test for clarity - Remove last command, now all are skills - Update commit skill to remove /check reference - Update settings.json with new skill name Co-Authored-By: Claude <noreply@anthropic.com>
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
Co-Authored-By: Claude <noreply@anthropic.com>
Contributor
Coverage Report
File CoverageNo changed files found. |
Skip unnecessary checks based on changed files: - No changes: skip entirely - Only meta files (.md, .yml, .json, etc.): run lint:misc only - Source files: run full lint, build, test Co-Authored-By: Claude <noreply@anthropic.com>
sirtimid
reviewed
Jan 27, 2026
Comment on lines
25
to
27
Member
Author
There was a problem hiding this comment.
Ah, accidentally carried over from a local branch. It already got merged to main. Disappeared by updating the branch.
sirtimid
previously approved these changes
Jan 27, 2026
github-merge-queue bot
pushed a commit
that referenced
this pull request
Jan 27, 2026
This PR refactors the Claude Code configuration from commands to skills, making them more modular and reusable. ## Summary - Convert commit, commit-push, commit-push-pr commands to atomic skills - Replace composite skills (commit-push, commit-push-pr) with individual push and pr skills - Convert check command to lint-build-test skill for clarity - Convert pr command to pr skill - Remove the commands directory entirely - everything is now a skill ## Changes - **commit**: Now a standalone skill for committing code - **push**: New atomic skill for pushing to remote - **pr**: Converted from command to skill, with updated PR body format (narrative description, no checkbox test plan) - **lint-build-test**: Renamed from "check" for clarity about what it does The skills are now atomic building blocks that Claude can compose based on user intent, rather than rigid composite commands. ## Testing Changes were tested by invoking the skills directly during development. The skills correctly handle git operations, pushing, and would create PRs with the expected format. <!-- CURSOR_SUMMARY --> --- > [!NOTE] > Refactors Claude Code config from monolithic commands to atomic skills. > > - Removes command files in `.claude/commands` and replaces them with skills in `.claude/skills` > - Adds/updates skills: `commit` (reworked steps/args), `push` (new), `pr` (converted, `unsandboxed`, defines PR body format), `lint-build-test` (new/renamed from `check` with detailed run matrix) > - Updates `.claude/settings.json` to allow new skills: `lint-build-test`, `code-review`, `commit`, `glossary`, `pr`, `push` > - Eliminates composite flows (`commit-push`, `commit-push-pr`) in favor of composing atomic skills > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit e8369db. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> --------- Co-authored-by: Claude <noreply@anthropic.com>
Add instructions to check if PR builds on another feature branch with an open PR. If so, create the PR as a draft. Co-Authored-By: Claude <noreply@anthropic.com>
sirtimid
approved these changes
Jan 27, 2026
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.
This PR refactors the Claude Code configuration from commands to skills, making them more modular and reusable.
Summary
Changes
The skills are now atomic building blocks that Claude can compose based on user intent, rather than rigid composite commands.
Testing
Changes were tested by invoking the skills directly during development. The skills correctly handle git operations, pushing, and would create PRs with the expected format.
Note
Replaces legacy command-based setup with modular, atomic skills and updates permissions accordingly.
.claude/commands/*(check,commit-push,commit-push-pr,pr)commit,push,pr,lint-build-test; updates.claude/settings.jsonto allow thesecommit: introducescheck/forceargs, standardized commit steps/message (incl. Co-Authored-By), safety checkslint-build-test: determines scope from changed files and runs targeted lint/build/test (workspace or monorepo, quiet tests)pr: scripted flow with status validation, stacked PR detection, and narrative PR body requirementspush: basic push flow with upstream handlingWritten by Cursor Bugbot for commit fff6c78. This will update automatically on new commits. Configure here.