Content
78%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 well-structured, actionable pre-merge checklist with an explicit verification step and clean sectioning. It is concise and appropriately organized, with only minor over-explanation in the intro and a few small validation gaps.
Suggestions
Trim the opening output-formatting paragraph to one sentence (e.g. 'Output only actionable items; omit areas needing no action.') to push conciseness toward 5.
Add an explicit final verification step in 'When you're done' (e.g. re-run the checklist items to confirm none remain open before pushing) to strengthen workflow_clarity.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is largely lean and assumes Claude's competence, with concrete instructions rather than concept explanations. The opening paragraph ('Please make sure to only output actionable items; if one area does not require further action, don't say so, just omit it. The user should receive a list of actionable todos.') is mildly over-explained and could be trimmed, keeping it just below the fully lean score-5 anchor. | 4 / 5 |
Actionability | It gives concrete, specific guidance — named required PR-body sections (Why / What changed / Test plan), a concrete tool to invoke ('use `chatto-api-compatibility`'), a required label ('api-breaking-change'), and Conventional Commit syntax. As an instruction-only skill it has no code, but the guidance is actionable; minor gaps (e.g. no exact command for reading the body back from GitHub) keep it at 4 rather than 5. | 4 / 5 |
Workflow Clarity | There is a clear sequenced checklist (familiarize → fill test gaps → update ADRs/docs → PR body → breaking-change classification → push) and an explicit verification checkpoint ('read the stored body back from GitHub and confirm it accurately represents the full diff'). Minor validation gaps elsewhere (e.g. no explicit re-check after fixing test gaps) prevent a 5. | 4 / 5 |
Progressive Disclosure | This is a single-file, sub-50-line, single-purpose skill with no bundle files (references/scripts/assets absent) and clear section headers (PR Body Quality, Breaking Changes Checklist, When you're done). Per the simple-skill scoring note, well-organized sections alone warrant a 5 here. | 5 / 5 |
Total | 17 / 20 Passed |