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 an exceptionally lean, well-structured set of maintainer rules with strong validation gates in the merge and release workflows. Its main weakness is actionability: much of the guidance is stated as principles rather than concrete, copy-paste commands or complete examples.
Suggestions
Add a few concrete command examples for the most common actions (e.g. a sample `gh pr list --search` invocation, a merge-gate shell snippet that captures `$?`).
Provide one short fail-before/pass-after regression-test template to make the 'every fix carries a test' rule immediately executable.
Consider a minimal close-with-reopen-door message template so the communication guidance is copy-paste ready.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Lean throughout: every line is a distilled failure-lesson with no padding, no explanation of concepts Claude already knows, and no tutorial fluff about what an issue or PR is. | 5 / 5 |
Actionability | Guidance is concrete and executable in places (gh pr list, Co-authored-by, pytest | tail, capture $?) but most of it is principle-level directive prose rather than copy-paste commands or complete examples covering common cases. | 3 / 5 |
Workflow Clarity | Sequenced workflows with explicit validation gates are present (run full gates before bumping versions; merge only on structural check; validate built release like a skeptic), with only minor checkpoint gaps in the triage/close flow. | 4 / 5 |
Progressive Disclosure | Well-organized into clearly headed sections with no bundle files to over-split; the body is a single self-contained overview that navigates cleanly by section, appropriate for a guidance skill needing no external references. | 5 / 5 |
Total | 17 / 20 Passed |