Content
57%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 concise, well-sequenced workflow with concrete gh commands and useful judgment checkpoints, but it defers critical execution detail (git identity, branch naming, PR body) to a non-existent pr-conventions.md and lacks a fix-verification step before filing.
Suggestions
Either ship references/pr-conventions.md with the branch-naming, git-identity, and PR-body rules, or inline those rules so the skill is self-contained.
Add an explicit verification step before filing (e.g. run the repo's tests / reproduce the fix locally) so the workflow clears the validation checkpoint required for outward-facing operations.
Keep the lean prose but cut lines that restate the overview (e.g. 'do the investigation properly first, don't jump to a fix without evidence') to push conciseness toward 5.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean, assumes Claude's competence, and avoids explaining concepts Claude already knows; only minor phrases like 'do the investigation properly first, don't jump to a fix without evidence' could be trimmed. Not a 5 because a few sentences are mildly redundant with the overview; not a 3 because there is no real padding or over-explanation. | 4 / 5 |
Actionability | Concrete commands are present ('gh pr list --search "<keywords>" --state open', 'gh repo clone <owner>/<repo>'), but the commit/push/PR-body and git-identity steps defer to 'pr-conventions.md', which is not present in the bundle, leaving key execution details missing. Not a 4 because core execution detail is offloaded to an absent file; not a 2 because two real executable commands are given. | 3 / 5 |
Workflow Clarity | An 8-step sequence is clearly listed with some checkpoints (existing-PR search, PR-right-tool judgment, 'gh auth status' check), but there is no validation step confirming the fix actually works before filing an outward-facing PR, which per the rubric caps workflow clarity at 3. Not a 4 because the verification loop is absent; not a 2 because the sequence is coherent and several checkpoints exist. | 3 / 5 |
Progressive Disclosure | Sections are well organized (Overview, Workflow, What's a good fix, Restricted tools) and references are signaled, but 'pr-conventions.md' is referenced as a bundle file and does not exist in references/scripts/assets. Not a 4 because a referenced detail file is missing; not a 2 because structure is reasonable and references are clearly signaled rather than buried. | 3 / 5 |
Total | 13 / 20 Passed |