code-reviewer
Reviews Java 25 and Spring Boot 4 codebases, pull requests, files, and modules for migration risks, architecture boundary violations, JSpecify null-safety issues, security flaws, performance regressions, and Spring Data pitfalls. Use when the task is a concrete Java or Spring code review with code context. Do not use for Kotlin-only code, non-Spring frameworks, or generic review advice without files or diffs.
95
92%
Does it follow best practices?
Impact
100%
1.23xAverage score across 3 eval scenarios
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 is specific, well-scoped, and highly actionable. It lists concrete review concerns, provides clear trigger conditions with a 'Use when' clause, and includes explicit exclusion criteria that reduce ambiguity. The description is concise yet comprehensive, making it easy for Claude to select this skill appropriately from a large pool.
| Dimension | Reasoning | Score |
|---|---|---|
Specificity | Lists multiple specific concrete actions: reviewing for migration risks, architecture boundary violations, JSpecify null-safety issues, security flaws, performance regressions, and Spring Data pitfalls. Very detailed and actionable. | 3 / 3 |
Completeness | Clearly answers both 'what' (reviews Java/Spring codebases for specific issues) and 'when' (use for concrete Java or Spring code review with code context). Also includes explicit 'do not use' exclusions, which further clarifies scope. | 3 / 3 |
Trigger Term Quality | Includes strong natural keywords users would say: 'Java 25', 'Spring Boot 4', 'pull requests', 'code review', 'migration risks', 'null-safety', 'security flaws', 'performance regressions', 'diffs', 'files'. Good coverage of terms a developer would naturally use. | 3 / 3 |
Distinctiveness Conflict Risk | Highly distinctive with a clear niche: Java 25 + Spring Boot 4 code review. The explicit exclusions (Kotlin-only, non-Spring frameworks, generic advice without files) sharply delineate boundaries and minimize conflict with other skills. | 3 / 3 |
Total | 12 / 12 Passed |
Implementation
85%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This is a well-structured, high-quality review skill that provides clear workflows, specific triggers for findings, and excellent progressive disclosure through reference files. Its main weakness is moderate verbosity—the quick triggers and common review modes sections partially overlap with the main workflow, adding some redundancy. Overall, it's a strong skill that would effectively guide Claude through Java 25 and Spring Boot 4 code reviews.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The skill is mostly efficient and avoids explaining basic concepts, but some sections are somewhat verbose—the 'Quick triggers for findings' section partially duplicates what the review passes already cover, and the 'Common review modes' section repeats workflow logic. Some tightening is possible. | 2 / 3 |
Actionability | The skill provides concrete, specific guidance throughout: exact file references to load, specific patterns to flag (e.g., 'TestRestTemplate instead of RestTestClient', '@ImportHttpServices'), a structured report template with file/line citation format, and clear pass-by-pass review instructions. While it doesn't contain executable code (appropriate for a review skill), the guidance is highly specific and actionable. | 3 / 3 |
Workflow Clarity | The workflow is clearly sequenced across three steps (confirm scope → load references → run passes), with explicit ordering of passes A-F for full reviews. The skill includes validation checkpoints like 'Never review without code context' and 'confirm Java and Spring Boot versions from the build files.' The common review modes provide clear abbreviated workflows for specific scenarios. | 3 / 3 |
Progressive Disclosure | The skill excellently structures content with a clear overview and well-signaled one-level-deep references via the reference table. It appropriately delegates to other skills (spring-data-jpa, springboot-migration) when scope exceeds this skill's purpose. The main SKILL.md stays at the right level of abstraction while pointing to 8 specific reference files. | 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.
- Commit
efa5546
- Reviewed
Table of Contents
Is this your skill?
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.