Content
75%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 highly actionable, well-structured skill body with concrete commands, decision checkpoints, and complete output templates for a two-phase workflow. Its main weakness is token efficiency: the critical no-posting rules are duplicated in full and the output templates inflate the main file, which also limits progressive disclosure.
Suggestions
Deduplicate the 'CRITICAL: Do NOT Post Comments' section: state the rule once in the top-level CRITICAL RULES and reference it from Phase 2 instead of repeating the full table and workflow.
Move the full rewrite output template and the 'Quality Comparison Examples' worked examples into references (e.g. references/output-templates.md), keeping only a skeleton inline in SKILL.md.
Fold the 'Two-Phase Workflow' section into the opening paragraph and surface the 'write to pr-finalize-summary.md' step inside the Output Format section so the stated workflow matches the documented output.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Most sections (tables, title formula, output templates) earn their tokens, but the 'CRITICAL: Do NOT Post Comments' section is duplicated nearly verbatim (once in the opening rules, again in Phase 2), and the 'Two-Phase Workflow' intro restates the opening paragraph. This fits anchor 3 ('mostly efficient but could be tightened'); it is not anchor 2 because the verbosity is localized duplication rather than pervasive padding. | 3 / 5 |
Actionability | Copy-paste-ready gh commands (gh pr view/diff with --json/--jq variants), a concrete title formula with good/bad examples, complete markdown output templates for both the keep and rewrite paths, and worked good/poor description examples. Matches anchor 5: fully executable guidance covering the common cases. | 5 / 5 |
Workflow Clarity | Two phases are clearly sequenced and Phase 1 has an explicit decision checkpoint ('Is the existing description better than what my template would produce?') with keep/rewrite branches. Not anchor 5 because the 'write to pr-finalize-summary.md' step appears only in workflow prose and not in the output format, and Phase 2 (code review) has no corresponding checkpoint; not anchor 3 since checkpoints are explicit rather than missing. | 4 / 5 |
Progressive Disclosure | The single reference (references/complete-example.md, verified to exist, 63 lines, well-structured) is clearly signaled exactly once and is one level deep; the body has clear section headers. Not anchor 5 because a sizable chunk of detail (output templates and worked quality-comparison examples) is inlined in SKILL.md where a second reference file could carry it. | 4 / 5 |
Total | 16 / 20 Passed |