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.
A well-structured, highly actionable protocol skill with concrete commands and strong validation gates around an outward-facing operation. Its only weakness is minor repetition of the core voice rule across sections.
Suggestions
Consolidate the no-hedging/required-changes rule so it is stated once authoritatively in "The rule" and only referenced (not restated) in "Voice" and "Before posting".
If the voice section grows, consider extracting the worked review-body templates into a references file so SKILL.md stays a lean overview.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and directive without explaining concepts Claude already knows, but the no-hedging/required-changes rule is restated across "The rule", "Voice", and "Before posting" sections and could be tightened. | 4 / 5 |
Actionability | Provides copy-paste-ready commands covering the common cases — `gh pr review --request-changes --body-file`, `gh pr view ... --json`, the full `gh api .../reviews -X POST --input -` JSON payload with `side`/`start_line` anchoring, and `gh issue comment --body-file` — plus concrete review-body examples. | 5 / 5 |
Workflow Clarity | Clear before/during/after sequencing with explicit validation gates: `gh auth status` once, refresh the PR via `gh pr view --json state,isDraft,author,headRefOid` immediately before each autonomous post, and a "Before posting" checklist that re-reads the body and confirms the event/requests agreement. | 5 / 5 |
Progressive Disclosure | No bundle files exist and none are needed; the skill is a single cohesive document organized under clear headers (Who is posting, The rule, Voice, Structure, How to post, After posting, Before posting) with no nested references, though at ~100 lines some voice-rule content could conceivably be split out. | 4 / 5 |
Total | 18 / 20 Passed |