Content
75%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 lean, highly actionable instruction-only skill: every hunting category carries concrete examples, confidence bands and evidence rules act as quality gates, and the output is a strict JSON contract. Weaknesses are minor: some duplicated rule-matching guidance, an unstated end-to-end review order, and a findings schema that is referenced but neither bundled nor shown.
Suggestions
Include the findings schema inline or as a bundled reference file, and show one complete example finding (with rule quote, diff lines, and confidence) in the output section.
Deduplicate the rule-matching guidance between 'Standards discovery' and 'What you don't flag' into a single statement.
State the review sequence explicitly (read standards → read diff → match rules → check categories → calibrate → emit JSON) at the top of the body.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense, imperative, and assumes competence — no concept explanations or padding, e.g. "You audit code changes against the project's own standards files... not to invent new rules." Not a 5 because the rule-matching idea is duplicated ("A skill compliance checklist does not apply to a TypeScript converter change" appears in both Standards discovery and What you don't flag), which could be tightened. | 4 / 5 |
Actionability | Concrete, executable guidance throughout: specific violation categories with inline examples ("TodoWrite" instead of "TaskCreate", "learnings-researcher" instead of the fully qualified name), numeric confidence bands (0.80+/0.60-0.79/below 0.60), a two-part evidence requirement with a drop rule, and a copy-paste JSON output block. Not a 5 because it says "matching the findings schema" without showing that schema, and the output example omits where the calibrated confidence value goes — a minor gap for an instruction-only skill. | 4 / 5 |
Workflow Clarity | The sequence is coherent: standards discovery (two paths, one with numbered steps) → match rules to file types → hunt violations → calibrate confidence → apply exclusions → evidence check → JSON output. Checkpoints exist (confidence suppression rule, evidence drop rule, exclusion list). Not a 5 because the overall review order is implied rather than stated as an explicit step sequence, and there is no verify/re-check loop before emitting findings. | 4 / 5 |
Progressive Disclosure | Well-organized sections (discovery, hunting list, calibration, exclusions, evidence, output) with no bundle files to disclose; at ~72 lines of compact persona guidance, nothing clearly belongs in a separate file, so it sits just outside the under-50-line 5 exception. The one buried reference — "matching the findings schema" with no path or included schema — keeps it at 4 rather than 5. | 4 / 5 |
Total | 16 / 20 Passed |