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.
The body is highly actionable with a well-sequenced, validated workflow and concrete commands. It is concise and well-structured, with only minor trimming and organization opportunities.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Largely lean with executable commands and minimal padding, though inline rationales like 'a main-based mergeable flag does not prove it lands cleanly' and the explanatory prose around trailers could be trimmed slightly. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready commands throughout (git fetch, gh pr view/diff/comment, git merge-tree, cherry-pick, cargo fmt/test) with placeholders, plus a grounded real example (PR #3221) covering a common case. | 5 / 5 |
Workflow Clarity | A clear 7-step sequence with explicit validation checkpoints (merge-tree conflict check in step 3, 'only land green' in step 6), error-recovery guidance for conflicts in step 5, and a reinforcing 'Red flags / don't' checklist. | 5 / 5 |
Progressive Disclosure | Well-organized into 'When to use', 'Workflow', and 'Red flags' sections with one-level references to real repo paths (.github/AUTHOR_MAP, auto-close workflow) and a sibling skill; minor gaps as the dense workflow and grounded example are inlined rather than split out. | 4 / 5 |
Total | 18 / 20 Passed |