Content
85%Weight 40%Scale 1-3Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A well-structured, highly actionable skill body with strong workflow sequencing and validation feedback loops. The main weakness is redundancy between the prose Steps and the executable Commands sections.
Suggestions
Collapse the Steps section to focus on decision points and guardrails (force-with-lease, remote-auth handling, PR title reconsideration) instead of re-narrating the commands shown below, reducing overlap with the Commands block.
In the Commands block, replace the placeholder '<!-- ... -->' removal example with the actual placeholder strings expected by .github/PULL_REQUEST_TEMPLATE.md so the validation grep is copy-paste ready without inference.
Move the per-change PR-title guidance into the single place it applies (the gh pr create/edit commands) to avoid stating the title rules twice across Steps and Commands.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body assumes Claude's competence (no basic git/gh explanations), but the prose Steps section largely narrates what the Commands block already shows, which could be tightened. | 2 / 3 |
Actionability | Provides concrete, copy-paste-ready shell commands ('git push -u origin HEAD', 'gh pr create --title', 'ruff check app tests'); the title placeholder is appropriately justified per-change. | 3 / 3 |
Workflow Clarity | Clear 8-step sequence with explicit validation checkpoints (pre-push validation in step 2, PR-body placeholder check in step 7) and a push-rejection feedback loop (pull skill → re-validate → retry). | 3 / 3 |
Progressive Disclosure | Self-contained body with well-organized sections (Prerequisites, Goals, Related Skills, Steps, Commands, Notes); no nested references and no wall-of-text that should be split out, with no bundle files present. | 3 / 3 |
Total | 11 / 12 Passed |