Use when you need to review, improve, or refactor Java code for object-oriented design quality — including applying SOLID, DRY, and YAGNI principles, improving class and interface design, fixing OOP concept misuse (encapsulation, inheritance, polymorphism), identifying and resolving code smells (God Class, Feature Envy, Data Clumps), or improving object creation patterns, method design, and exception handling. Part of the skills-for-java project
88
85%
Does it follow best practices?
Impact
Pending
No eval scenarios have been run
Passed
No known issues
Quality
Discovery
100%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 is an excellent skill description that clearly articulates its purpose, provides comprehensive trigger terms, and explicitly states when to use it. The description is specific to Java OOP design quality with concrete examples of principles, patterns, and code smells it addresses. The 'Use when...' clause at the beginning provides clear activation guidance.
| Dimension | Reasoning | Score |
|---|---|---|
Specificity | Lists multiple specific concrete actions: 'applying SOLID, DRY, and YAGNI principles', 'improving class and interface design', 'fixing OOP concept misuse (encapsulation, inheritance, polymorphism)', 'identifying and resolving code smells (God Class, Feature Envy, Data Clumps)', 'improving object creation patterns, method design, and exception handling'. | 3 / 3 |
Completeness | Explicitly answers both what (review, improve, refactor Java code for OOP design quality with specific techniques listed) and when ('Use when you need to...' clause at the start provides clear trigger guidance). | 3 / 3 |
Trigger Term Quality | Excellent coverage of natural terms users would say: 'Java code', 'SOLID', 'DRY', 'YAGNI', 'refactor', 'code smells', 'God Class', 'Feature Envy', 'encapsulation', 'inheritance', 'polymorphism', 'exception handling'. These are terms developers naturally use when discussing OOP design issues. | 3 / 3 |
Distinctiveness Conflict Risk | Highly distinctive with clear niche: specifically Java + OOP design quality + named principles (SOLID, DRY, YAGNI) + specific code smells. Unlikely to conflict with general code review or other language-specific skills. | 3 / 3 |
Total | 12 / 12 Passed |
Implementation
70%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This skill content is well-organized with excellent workflow clarity and progressive disclosure - it clearly establishes safety constraints and points to detailed reference material. However, it lacks concrete code examples in the main file, relying entirely on the reference document for actionable patterns. The 'When to use this skill' section adds little value beyond what's already stated.
Suggestions
Add 1-2 brief, executable code examples showing a common before/after refactoring pattern (e.g., a God Class smell and its fix) to make the skill immediately actionable
Remove or consolidate the 'When to use this skill' section as it largely duplicates the description and 'What is covered' content
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The skill is reasonably efficient but includes some redundancy - the 'When to use this skill' section largely repeats information from the description and 'What is covered' sections. The bullet lists are appropriately concise. | 2 / 3 |
Actionability | Provides concrete commands (mvnw compile, mvn clean verify) but lacks executable code examples. The actual guidance is deferred entirely to the reference file, making this more of a pointer than actionable instruction. | 2 / 3 |
Workflow Clarity | Clear workflow with explicit validation checkpoints: compile before changes, stop on failure (blocking condition), verify after changes. The sequence is unambiguous with appropriate safety gates. | 3 / 3 |
Progressive Disclosure | Well-structured overview with clear one-level-deep reference to the detailed guidance file. The SKILL.md appropriately serves as an index/overview pointing to comprehensive documentation. | 3 / 3 |
Total | 10 / 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.
7772a1b
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.