This skill embodies the principles of "Clean Code" by Robert C. Martin (Uncle Bob). Use it to transform "code that works" into "code that is clean."
50
Quality
38%
Does it follow best practices?
Impact
Pending
No eval scenarios have been run
Passed
No known issues
Optimize this skill with Tessl
npx tessl skill review --optimize ./skills/clean-code/SKILL.mdQuality
Discovery
40%Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.
This description relies heavily on the 'Clean Code' book reference without explaining what concrete actions the skill performs. It lacks specific capabilities (e.g., improving naming, reducing function size, eliminating duplication) and doesn't provide clear trigger scenarios for when Claude should select this skill over other code-related skills.
Suggestions
Add specific concrete actions the skill performs, such as 'Improves naming conventions, reduces function complexity, eliminates code duplication, applies SOLID principles'
Include a proper 'Use when...' clause with natural trigger terms like 'Use when the user asks to refactor code, improve code readability, review code quality, or mentions Clean Code principles'
Add common user phrases and file contexts, such as 'code smell', 'technical debt', 'make this more readable', or 'review my code'
| Dimension | Reasoning | Score |
|---|---|---|
Specificity | The description uses vague, abstract language like 'embodies the principles' and 'transform code' without listing any concrete actions. It doesn't specify what transformations or refactoring operations are performed. | 1 / 3 |
Completeness | Has a weak 'what' (transform code to be clean) and includes 'Use it to...' which partially addresses 'when', but the trigger guidance is vague and doesn't provide explicit scenarios or user phrases that would invoke this skill. | 2 / 3 |
Trigger Term Quality | Contains some relevant terms like 'Clean Code', 'Robert C. Martin', and 'Uncle Bob' that users familiar with the book might use, but lacks common natural terms like 'refactor', 'code review', 'improve readability', 'naming', or 'functions'. | 2 / 3 |
Distinctiveness Conflict Risk | The 'Clean Code' and 'Uncle Bob' references provide some distinctiveness, but 'transform code' is generic enough to potentially conflict with other code improvement, refactoring, or review skills. | 2 / 3 |
Total | 7 / 12 Passed |
Implementation
37%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This skill serves as a decent reference card for Clean Code principles but doesn't add much beyond what Claude already knows. It lacks a clear workflow for applying these principles during refactoring, and the guidance is more descriptive than actionable. The content would benefit from concrete transformation examples and a step-by-step refactoring process.
Suggestions
Add a concrete refactoring workflow with validation steps (e.g., '1. Identify smell, 2. Write characterization test, 3. Apply transformation, 4. Verify tests pass, 5. Repeat')
Replace descriptive guidance with executable transformation patterns showing before/after code with specific refactoring commands
Remove the 'When to Use' and 'Core Philosophy' sections - Claude knows when to apply clean code principles
Consider splitting into SKILL.md (quick reference) and separate files for detailed examples per principle
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The skill is reasonably efficient but includes some unnecessary framing (e.g., 'When to Use' section explaining obvious use cases, the core philosophy quote). Claude already knows Clean Code principles; this reads more like a reference card than new knowledge. | 2 / 3 |
Actionability | Provides concrete examples and a checklist, but guidance is mostly descriptive rather than executable. The code snippets are illustrative comparisons rather than copy-paste-ready transformations or refactoring commands Claude could apply. | 2 / 3 |
Workflow Clarity | No clear workflow or sequence for applying these principles. The checklist at the end is helpful but there's no process for how to systematically refactor code, no validation steps, and no feedback loops for iterative improvement. | 1 / 3 |
Progressive Disclosure | Content is organized into clear sections with good headers, but it's a monolithic document with no references to external files for deeper dives. For a comprehensive topic like Clean Code, splitting detailed examples or language-specific guidance into separate files would improve navigation. | 2 / 3 |
Total | 7 / 12 Passed |
Validation
90%Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.
Validation — 10 / 11 Passed
Validation for skill structure
| Criteria | Description | Result |
|---|---|---|
frontmatter_unknown_keys | Unknown frontmatter key(s) found; consider removing or moving to metadata | Warning |
Total | 10 / 11 Passed | |
9c177eb
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.