Content
33%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 maximally concise but substantively empty: it describes a helper file that is not present in the bundle and provides no instructions, steps, or validation for the work it names. As written it offers context trivia rather than actionable guidance.
Suggestions
Add explicit instructions for the single action this skill performs (e.g., how to run static analysis over query.py, with the exact command), which would lift actionability and workflow clarity.
Ship query.py in the bundle (e.g., under scripts/) or remove the broken link, so the one reference in the body resolves.
Include a validation/verification checkpoint appropriate to database-query analysis so the workflow is not just a bare description.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is three lean lines with zero padding and no explanation of concepts Claude already knows — 'The inert helper query.py preserves the historical unsafe string construction. Static analysis only; no database is contacted.' Every token is informational, matching 'Lean and efficient; assumes Claude's competence'. | 5 / 5 |
Actionability | The body only describes what query.py is ('preserves the historical unsafe string construction'); it gives no code, commands, or steps for what to actually do, matching 'Entirely vague or abstract; only describes rather than instructs'. It is not 2 because there are not even high-level hints toward executable steps, and the referenced helper file does not exist in the bundle. | 1 / 5 |
Workflow Clarity | No sequence of steps exists at all — no analysis procedure, no commands, no validation checkpoints — matching 'Steps missing or incoherent; no sequence; no validation for risky operations'. The simple-skill exception does not apply because the single action (what to do with query.py) is never stated; even for a database-context skill, the content falls at the bottom anchor rather than the feedback-loop cap of 3. | 1 / 5 |
Progressive Disclosure | The body's sole link '[query.py](query.py)' is a dangling reference — no query.py exists alongside SKILL.md and no references/, scripts/, or assets/ directories are present, so navigation fails entirely. This fits 'Minimal structure; ... references are buried' (broken navigation) rather than 1, since the body is not a monolithic wall of text; it cannot reach the simple-skill 5 because the one reference it makes does not resolve. | 2 / 5 |
Total | 9 / 20 Passed |