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 terse and action-oriented but lacks depth. It identifies a specific tool (clippy) and a single action (fix warnings) without elaborating on the range of capabilities or providing explicit trigger guidance for when Claude should select this skill. Adding a 'Use when...' clause and more concrete actions would significantly improve its utility for skill selection.
Suggestions
Add an explicit 'Use when...' clause, e.g., 'Use when the user asks to fix clippy warnings, run cargo clippy, resolve Rust lint issues, or clean up Rust code quality.'
Expand the capability description with more concrete actions, e.g., 'Runs cargo clippy, auto-applies suggested fixes, refactors code to resolve lint warnings, and optionally suppresses specific lints with allow attributes.'
Include natural trigger term variations such as 'cargo clippy', 'Rust lints', 'lint errors', and 'code quality warnings' to improve keyword coverage.
| 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 explicit 'Use when...' clause or trigger guidance, which per the rubric caps completeness at 2, and the 'what' is also thin—so this lands at 1. | 1 / 3 |
Trigger Term Quality | Includes 'clippy', 'lint', and 'warnings' which are relevant keywords, but misses common variations like 'cargo clippy', 'Rust lints', 'clippy errors', 'lint fixes', or '#[allow(...)]'. | 2 / 3 |
Distinctiveness Conflict Risk | Mentioning 'clippy' narrows the domain to Rust tooling, which is fairly distinct, but the description is brief enough that it could overlap with a general Rust code quality or linting skill. | 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 clear executable 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:' | sort -u` after manual fixes to confirm zero remaining warnings, creating a feedback loop.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Very lean and efficient. The one explanatory sentence about warnings vs errors is genuinely useful context that prevents a common mistake. Every other token is actionable. | 3 / 3 |
Actionability | Provides exact, copy-paste ready commands for both the auto-fix step and the manual inspection step. The grep/sort pipeline is a concrete, executable command. | 3 / 3 |
Workflow Clarity | Steps are clearly sequenced (auto-fix first, then manual fix), 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. | 2 / 3 |
Progressive Disclosure | This is a simple, single-purpose skill under 50 lines. The content is well-organized into clear sequential sections with no need for external references. | 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.
cfb53e9
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.