Content
77%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A highly actionable, well-sequenced instruction skill with strong validation feedback loops, weakened mainly by duplicated content in a monolithic file with no reference split-out.
Suggestions
Remove the duplication: drop the verbatim description repeat in 'When to Use' and keep the Source:-line explanation in only one place (it currently appears in both the report-mode approach and the fix-mode recipe).
Split the report-format template and/or the fix-mode recipe into a reference file (e.g. references/report-format.md, references/fix-recipe.md) and link to them from SKILL.md to improve progressive disclosure and shorten the body.
De-duplicate the flow-picker logic, which is stated once in 'Picking a flow' and restated in the fallback note immediately after.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient and directive, but the 'When to Use' section repeats the description verbatim and the Source:-line explanation is duplicated (appearing both in the report-mode approach and the fix-mode recipe), so it could be tightened rather than earning the lean score-3 anchor. | 2 / 3 |
Actionability | Gives concrete, parameterized tool calls — audit_diff({ audit_name: "before" }), format: "compact", explain_rule({ id }), audit_live — plus an exact report template and explicit bail conditions; for an instruction-only skill this is copy-ready, actionable guidance matching the score-3 anchor. | 3 / 3 |
Workflow Clarity | Fix mode is a clearly sequenced Baseline → Plan+apply → Verify loop with an explicit validation checkpoint (confirm -fixed covers targets and +new is empty) and a 'When to bail' feedback loop, matching the score-3 anchor with validation steps and error-recovery feedback. | 3 / 3 |
Progressive Disclosure | No bundle files exist and the 107-line body is a single monolithic file; internal sections are well organized but nothing is split out into one-level-deep references, so it lands at the score-2 'some structure, content that could be separate is inline' anchor rather than the split-out score-3 anchor (the under-50-line simple-skill exemption does not apply). | 2 / 3 |
Total | 10 / 12 Passed |