Content
73%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 strong, actionable skill body: sequenced phases, confirmation gates before destructive edits, a verification checklist, and a real, well-scoped reference file. The two weak spots are repeated restatement of the must-ask-standards constraint and ripgrep commands written with a grep-style --include flag instead of -g, which keeps them from being copy-paste correct.
Suggestions
State the must-ask-standards constraint once (e.g., in Phase 1 or Rules of Engagement) and reference it elsewhere rather than restating it in Mindset, the workflow, and Troubleshooting.
Fix the ripgrep invocations: ripgrep has no --include flag — use `rg -g '*.{js,ts,vue}' '\bvar\b'` (or `rg --glob`) so the examples are executable as written.
Dedupe the standards-to-tooling pointer, which currently appears in both When to Use and When NOT to Use, and consider moving the full report template into references/ to slim the main file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient — phases, tables, and terse BAD/GOOD pairs — but the core "MUST ask for the standards" constraint is restated roughly six times (frontmatter echo, Mindset, Phase 1, Rules of Engagement, Troubleshooting) and the `standards-to-tooling` pointer appears in both When to Use and When NOT to Use. Fits "mostly efficient but includes some unnecessary explanation or could be tightened"; not 4 because the repetition is more than minor. | 3 / 5 |
Actionability | Concrete, executable guidance dominates: real scan commands ("rg --include '*.{js,ts,vue}' '\bvar\b'", "find <source-dir> -name '*[A-Z]*'"), a parse-classification table, an exact report template, and verbatim user prompts. Not 5 because the ripgrep examples use a nonexistent `--include` flag (ripgrep's glob flag is `-g`), so the commands are not copy-paste correct as written. | 4 / 5 |
Workflow Clarity | Five clearly sequenced phases with explicit validation checkpoints: "MUST present findings before offering to fix", confirmation gates before destructive edits ("Apply fixes?"), a Troubleshooting table as error-recovery feedback loops, and a final Verification checklist. Matches the top anchor; the batch/destructive-operation cap does not apply because validation and confirmation steps are explicit. | 5 / 5 |
Progressive Disclosure | Well-sectioned ~210-line body with one clearly signaled, one-level-deep reference ([edge-cases.md](references/edge-cases.md), verified to exist) that appropriately offloads edge-case handling. Not 5 because sizable inline content — the full report template and Rules of Engagement WHY/BAD/GOOD blocks — could arguably live in a reference file; not 3 because what is referenced is clearly signaled and well organized. | 4 / 5 |
Total | 16 / 20 Passed |