Content
88%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 body is a strong, actionable contribution workflow with executable commands, explicit validation checkpoints, and clean delegation of mechanics to referenced repo files. Its main weakness is moderate repetition of the 403 error and pre-existing-failure examples across sections.
Suggestions
Consolidate the GraphQL 403 `AddLabelsToLabelable` example and the pre-existing-failure package list so each appears once (e.g. in the verification protocol) and is referenced elsewhere, trimming conciseness waste.
Move the anti-patterns table and the detailed 'Standard honest-rewrite pattern' into a reference file (e.g. references/honest-rewrites.md) and link to it from SKILL.md to reduce the inline body length.
Add a couple of trigger-term synonyms in the description (e.g. 'reviewing a PR', 'PR comment') to round out trigger-term coverage.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Dense, high-signal, repo-specific content that assumes Claude's competence (no basic explanations of PRs or Conventional Commits); not a 5 because the GraphQL 403 example and the pre-existing-failure list are each repeated across multiple sections and could be trimmed. | 4 / 5 |
Actionability | Fully executable guidance throughout: concrete `gh pr view --json ...` commands, git branch commands, the exact branch-name and title regexes, and a literal PR-body section template with emojis covering the common cases. | 5 / 5 |
Workflow Clarity | The PR workflow is a clearly numbered 6-step sequence with an explicit 'Local validation' checkpoint, the verification protocol provides validate-then-retry feedback loops, and a pre-PR self-audit checklist closes the loop for this batch/permission-sensitive process. | 5 / 5 |
Progressive Disclosure | Well-sectioned body with a dedicated References list and a 'Source of truth' table that points one level deep to external repo files (CONTRIBUTING.md, sibling SKILL.md files) instead of duplicating mechanics; not a 5 because at ~290 lines some inlined material (anti-patterns table, detailed honest-rewrite pattern) could live in a reference file, and no bundle files are provided to offload it. | 4 / 5 |
Total | 18 / 20 Passed |