Content
72%Weight 40%Scale 1-3Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A tight, well-structured instruction skill that respects the context budget, but its guidance is more descriptive than executable and lacks validation checkpoints around the auto-fix step. Tightening the output spec and adding a verify step would lift the weaker dimensions.
Suggestions
Specify the exact format/content of the prioritized list in `post-merge-state.md` (e.g., columns or fields per item) so output is reproducible.
Add an explicit validation/verification checkpoint before and after any auto-fix attempt (e.g., re-run lint/tests, confirm the fix applies) since auto-fix is a batch operation.
Give concrete commands for the scan steps beyond `git diff HEAD~10` (e.g., how to enumerate recent merged PRs or surface introduced lint warnings).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's competence — short bullet lists with no conceptual padding — so every token earns its place. | 3 / 3 |
Actionability | It gives some concrete specifics (`git diff HEAD~10`, `post-merge-state.md`) but mostly describes tasks rather than instructing; the output format and how to produce the prioritized list are underspecified. | 2 / 3 |
Workflow Clarity | Sections imply a scan → classify → output sequence, but there are no explicit validation checkpoints, and auto-fix is a batch/destructive-style operation where missing verification caps clarity at 2. | 2 / 3 |
Progressive Disclosure | This is a simple under-50-line skill with no need for external references, and its sections (Scan Sources, Classification, Output, Rules) are well organized — meeting the simple-skill allowance for a 3. | 3 / 3 |
Total | 10 / 12 Passed |