Content
81%Weight 40%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The content is well-structured, actionable, and includes explicit validation and feedback loops for a destructive-ish operation. Main improvement areas are tightening minor redundancy and making the PR-body authoring steps fully executable.
Suggestions
Make the PR-body authoring section executable (e.g. concrete gh pr edit --body-file invocation with template-driven content) instead of commentary placeholders.
De-duplicate the force-with-lease and sync-vs-auth guidance that appears in both Steps and Notes.
Consider mentioning the 'PR' synonym in the description trigger phrases to improve trigger term coverage.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's knowledge of git/PRs, with only minor redundancy between the Steps, Commands, and Notes sections (e.g. force-with-lease and sync-vs-auth guidance repeated) that could be trimmed. | 4 / 5 |
Actionability | Provides concrete executable commands (git push -u origin HEAD, gh pr create/edit, mix pr_body.check), but the PR-body writing portion is commentary/placeholders rather than fully executable code. | 4 / 5 |
Workflow Clarity | Eight clearly sequenced steps with explicit validation checkpoints (make -C elixir all, mix pr_body.check) and a feedback loop for rejected pushes (pull skill, re-validate, retry), plus checklist-style PR-state handling. | 5 / 5 |
Progressive Disclosure | Well-organized into clear sections with one-level-deep, clearly signaled references (the pull skill, .github/pull_request_template.md) and no nested references; at ~110 lines it slightly exceeds the under-50 simple-skill exception. | 4 / 5 |
Total | 17 / 20 Passed |