Content
85%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.
A well-structured, highly actionable reviewer skill with excellent workflow sequencing, validation checkpoints, and clean one-level progressive disclosure to a real reference file. The only notable gaps are the missing submission command in the final posting step and minor duplicated content that could be trimmed.
Suggestions
Add the concrete command(s) for step 7 (e.g., the gh pr review / gh api invocation for posting inline comments and submitting the COMMENT review event), since that is the one step without an executable command.
Remove the duplicated trivial-PR exception (stated in both Review Mindset and step 7) and consolidate the csigs stop rule so it appears authoritatively in only one place.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and imperative throughout — no explanations of concepts Claude already knows, and every section carries operational weight. Minor trimmable duplication keeps it below anchor 5: the trivial-PR exception is stated twice (Review Mindset and step 7), and the csigs rule is repeated verbatim in review-rules.md §0. | 4 / 5 |
Actionability | Steps 1–4 provide exact copy-paste gh commands with flags, and the comment JSON schema plus template format are fully concrete. However, step 7 gives no actual command for posting inline comments or submitting the review event (e.g., the gh pr review / gh api invocation), leaving a gap in the final, most important step — anchor 4 rather than 5. | 4 / 5 |
Workflow Clarity | A clear 7-step numbered sequence with explicit validation checkpoints: retrieve the author before any work, form an independent assessment before reading the PR narrative, treat the PR description as claims to verify, check CI with a hard "never post LGTM if failing" rule, and verify concerns to avoid false positives. This matches the anchor-5 pattern of sequenced steps with explicit checkpoints and error-recovery guidance. | 5 / 5 |
Progressive Disclosure | SKILL.md is a workflow overview that defers all rule detail to a single, clearly signaled, one-level-deep reference (references/review-rules.md, step 5), which exists and holds the bulk material. The inline 9-item priority list is an appropriate summary, not content that belongs in a separate file; navigation is easy. | 5 / 5 |
Total | 18 / 20 Passed |