Fix all clippy lint warnings in the project
73
60%
Does it follow best practices?
Impact
93%
1.34xAverage score across 3 eval scenarios
Passed
No known issues
Optimize this skill with Tessl
npx tessl skill review --optimize ./.claude/skills/fix-clippy/SKILL.mdQuality
Discovery
32%Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.
The description is a single imperative sentence that identifies a narrow task (fixing clippy warnings) but lacks explicit trigger guidance, detailed actions, and keyword variations. It reads more like a task command than a skill description, missing the 'when to use' clause entirely and providing minimal detail about capabilities.
Suggestions
Add an explicit 'Use when...' clause with trigger terms like 'cargo clippy', 'Rust lint warnings', 'clippy errors', 'lint fixes'.
Expand the capability description to list specific actions such as 'auto-apply clippy suggestions, refactor code to satisfy lints, suppress false-positive warnings with allow attributes'.
Rewrite in third-person declarative voice (e.g., 'Fixes all clippy lint warnings in a Rust project') and include file type or ecosystem context to improve distinctiveness.
| Dimension | Reasoning | Score |
|---|---|---|
Specificity | Names a specific domain (clippy lints) and a single action (fix warnings), but does not list multiple concrete actions or elaborate on what fixing entails (e.g., auto-apply suggestions, refactor code, suppress lints). | 2 / 3 |
Completeness | Describes what it does (fix clippy lint warnings) but has no 'Use when...' clause or any explicit trigger guidance for when Claude should select this skill, which per the rubric caps completeness at 2, and the 'when' is entirely missing so it scores 1. | 1 / 3 |
Trigger Term Quality | Includes 'clippy' and 'lint warnings' which are relevant keywords, but misses common variations like 'cargo clippy', 'Rust lints', 'clippy errors', 'lint fixes', or 'pedantic lints'. | 2 / 3 |
Distinctiveness Conflict Risk | Mentioning 'clippy' provides some specificity to the Rust ecosystem, but 'fix lint warnings' could overlap with other linting/code-quality skills for different languages or tools. | 2 / 3 |
Total | 7 / 12 Passed |
Implementation
87%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A concise, well-structured skill that provides actionable commands for fixing clippy warnings. The key insight about warnings vs errors is valuable. The main weakness is the lack of a verification step after manual fixes to confirm all warnings have been resolved.
Suggestions
Add a Step 3 verification step: re-run `cargo clippy --tests 2>&1 | grep "^warning:"` after manual fixes to confirm zero remaining warnings, creating a feedback loop.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Very lean and efficient. Every line serves a purpose. The note about warnings vs errors is a genuinely useful distinction Claude might not know. No unnecessary explanations. | 3 / 3 |
Actionability | Provides exact, copy-paste ready commands (`make fix`, `cargo clippy --tests 2>&1 | grep "^warning:" | sort -u`). Clear instruction for manual fixes of remaining warnings. | 3 / 3 |
Workflow Clarity | Steps are clearly sequenced (auto-fix first, then manual), but there's no validation/verification step after manual fixes to confirm all warnings are resolved. A feedback loop (fix -> re-run clippy -> verify clean) is missing for what is essentially a batch fix operation. | 2 / 3 |
Progressive Disclosure | This is a simple, single-purpose skill under 50 lines with no need for external references. The content is well-organized with clear section headers and a logical two-step structure. | 3 / 3 |
Total | 11 / 12 Passed |
Validation
100%Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.
Validation — 11 / 11 Passed
Validation for skill structure
No warnings or errors.
c61244b
Table of Contents
If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.